[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCHv3] xen: Add EFI_LOAD_OPTION support



>>> On 21.05.18 at 18:59, <tamas@xxxxxxxxxxxxx> wrote:
> After closer inspection the problem is with the following line here:
> 
>>>          for ( i = 1; i < argc; ++i )
> 
> This assumes that argv[0] is the EFI executable filename, which is not
> true when EFI_LOAD_OPTION is used. That's why in my v3 patch I had the
> "elo_active" variable to determine whether to start the iteration from
> 0 or from 1.

Oh, I see. I didn't mean to drop that code of yours, but I also didn't mean
to use it as is; I simply forgot that I need to make further changes here.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.