Dear "Fillod Stephane", In message <0b45e93c5ff65740aeae690bf3848b7a02087...@rennsmail04.eu.thmulti.com> you wrote: > > I'm encountering a problem when tftp'ing a file of size 1747851 bytes > with > CONFIG_IP_DEFRAG defined and CONFIG_TFTP_BLOCKSIZE set to 4096. U-Boot's > tftp > times out on the last chunk. Undefining > CONFIG_IP_DEFRAG/CONFIG_TFTP_BLOCKSIZE > or downloading with a Linux tftp client are all well, so it does not > look like > a server problem. >... > The patch below appears to solve my problem. WARNING: since I haven't > rolled > my mind yet around the NetDefragment IP/UDP stack, I don't know yet what > are > the side effects of my patch. > > --- a/net/net.c 31 Mar 2010 21:54:39 > +++ b/net/net.c 4 Jun 2010 15:09:08 > @@ -1201,7 +1201,7 @@ static IP_t *__NetDefragment(IP_t *ip, i > h = payload + h->next_hole; > } > > - if (offset8 + (len / 8) <= h - payload) { > + if (offset8 + (len / 8) < h - payload) { > /* no overlap with holes (dup fragment?) */ > return NULL; > }
Can you please resubmit this (eventually integrating Alessandro's comments) and add your Signed-off-by: line, so we can integrate this fix into mainline? Thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de What about WRITING it first and rationalizing it afterwords? :-) - Larry Wall in <8...@jpl-devvax.jpl.nasa.gov> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot