[U-Boot-Users] [PATCH v3] Fix initrd length miscalculation in bootm command when using a dtu
timur at freescale.com
Fri May 4 15:25:55 CEST 2007
Wolfgang Denk wrote:
I'm glad you fixed the bug, so I'll just add a few comments:
> Your comment is actually wrong. The use of "len" is not limited to
> that purpose.
If you apply my patch, then the comment becomes correct. My goal was to
lock the variables 'len' and 'data' into one purpose. The reason the
bug existed is because the other developer didn't realize this. He used
'len' thinking it was available. In a sense, I was trying to implement
some "defensive programming", so that the next time someone hacks up
do_bootm_linux(), he won't re-introduce the bug.
Now, you might say that that won't happen again, but I disagree. I
think it can, for two reasons:
1) It happened once already, last year. You approved and applied a
patch which does overwrite the variable.
2) The libfdt code introduced a number of other bugs relating to dtu
usage, which have not yet been fixed.
So I believe there is a real possibility that another patch could
re-introduce this bug. If you had applied my patch as-is, that
possibility would have been eliminated. This is why I think my patch is
better than yours.
But I guess only time will tell who's right. :-)
> And please accept my apologies thatt his was so complicated and took
> so long. [Nevertheless you still might want to try to find a way to
> access the repository I created for you in case you have more
Stefan said he had a testing repo of some kind. How about we just use
that? If Stefan is willing, he can apply my emailed patches to his repo.
More information about the U-Boot