From mboxrd@z Thu Jan  1 00:00:00 1970
From: AKASHI Takahiro <takahiro.akashi@linaro.org>
Date: Wed, 11 Sep 2019 15:14:00 +0900
Subject: [U-Boot] efi_loader: LoadOptions (bootargs)
In-Reply-To: <20190822234226.GE14152@linaro.org>
References: <20190822090322.GB14152@linaro.org>
 <67df244d-7678-1ce6-c27c-b843dc305c95@gmx.de>
 <20190822234226.GE14152@linaro.org>
Message-ID: <20190911061359.GO4398@linaro.org>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Heinrich,

On Fri, Aug 23, 2019 at 08:42:27AM +0900, AKASHI Takahiro wrote:
> On Thu, Aug 22, 2019 at 07:53:46PM +0200, Heinrich Schuchardt wrote:
> > On 8/22/19 11:03 AM, AKASHI Takahiro wrote:
> > >Heinrich,
> > >
> > >I'm now wondering whether LoadedImage's LoadOptions, which comes
> > >from "bootargs" variable, should contain a command(application) name
> > >as a first argument or not.
> > >
> > >When I tried some efi application (efitools), I found that it expected
> > >so. For example, efitools' UpdateVars.efi takes
> > >     Usage: UpdateVars.efi: [-g guid] [-a] [-e] [-b] var file
> > >
> > >and I had to passed arguments by specifying "foo db DB.auth" for
> > >"bootargs" where foo makes no sense.
> > >
> > >What do you think about this issue?
> > 
> > Do you relate to
> > https://git.kernel.org/pub/scm/linux/kernel/git/jejb/efitools.git?
> 
> Yes.
> 
> > This style of parsing LoadOptions is defined by the EFI shell. See
> > function ParseCommandLineToArgs() in
> > ShellPkg/Application/Shell/ShellParametersProtocol.c.
> 
> So do you mean that Shell.efi is responsible for adding a command name
> to LoadOptions (or bootargs) as a first parameter or that LoadOptions
> is solely for Shell environment?
> 
> If so, should we do the same thing at bootefi?

Any comment?

-Takahiro Akashi


> > If UpdateVars.efi would work differently it could not be launched via
> > the shell.
> 
> Well, I'm trying to run UpdateVars.efi in a standalone way
> by invoking it directly from bootefi/bootmgr and it obviously fails
> due to this issue.
> 
> Thanks,
> -Takashiro Akashi
> 
> 
> > Best regards
> > 
> > Heinrich