[U-Boot] [RFC PATCH 6/8] net: TFTP over IPv6

[Joe Hershberger]

Hi Jean-Pierre,

On Tue, Nov 3, 2015 at 4:41 AM, Jean-Pierre Tosoni <jp.tosoni at acksys.fr> wrote:
> Hi all,
>
>> -----Message d'origine-----
>> De : Joe Hershberger [mailto:joe.hershberger at gmail.com]
>> Envoyé : lundi 2 novembre 2015 21:44
>> À : Chris Packham
>> Cc : u-boot; Joe Hershberger; jp.tosoni at acksys.fr; Hanna Hawa; Angga
>> Objet : Re: [U-Boot] [RFC PATCH 6/8] net: TFTP over IPv6
>>
> (...)
>> > ---
>> > One problem with the [hostIpAddr:]fileName syntax is that IPv6
>> > addresses contains colons. So tftp_start() would be confused by
>> > 'tftpboot6 $loadaddr 2001:db8::1:zImage'. It is probably possible to
>> > change the parsing to separate the host from the filename by parsing
>> > from the end (i.e. use strrchr() instead of strchr()) but then there
>> > are error cases that may not be handled correctly (e.g. omitting the
>> filename).
>>
>> I think we should just change the filename separator for tftp6. How about
>> ','?
>>
>
> How about a '/' to make it look like an URL ? More intuitive maybe.

That might work, but it also could be part of the path to the file
itself. It doesn't seem good to be ambiguous about that or to require
2001:db8::1//kernel.tgz

-Joe