Hi Chris,
I agree with you that the second parameter for Exit() shall be type EFI_STATUS.
And my point is according to spec the second parameter is the image's exit code
if we pass EFI_ABORTED to Exit() the "%lasterror%" is always 0x15. So how about
using another EFI_STATUS type local variable to get exit code and passing to
Exit()?
Thanks
Shumin
From: Phillips, Chris J (Plano, TX) [mailto:[email protected]]
Sent: Friday, September 12, 2014 2:12 AM
To: [email protected]; Kirmeier, Peter; Olivier Martin
Subject: Re: [edk2] [Patch]ShellPkg: Make shell image exit with exit code
instead of EFI_ABORTED while exit status is other than SHELL_SUCCESS.
Shumin,
Please wait for Olivier to look at this. I do not believe this change should
be made.
The Exit() boot service requires the second parameter to be of type EFI_STATUS.
Also, Olivier previously submitted a change so ExitData contains NULL string
followed by ExitStatus.
Thanks,
Chris
From: Carsey, Jaben [mailto:[email protected]]
Sent: Thursday, September 11, 2014 12:11 PM
To: Kirmeier, Peter;
[email protected]<mailto:[email protected]>;
Olivier Martin
Subject: Re: [edk2] [Patch]ShellPkg: Make shell image exit with exit code
instead of EFI_ABORTED while exit status is other than SHELL_SUCCESS.
Reviewed-by: Jaben Carsey
<[email protected]<mailto:[email protected]>>
From: Kirmeier, Peter [mailto:[email protected]]
Sent: Wednesday, September 10, 2014 11:38 PM
To: [email protected]<mailto:[email protected]>;
Carsey, Jaben; Olivier Martin
Subject: RE: [edk2] [Patch]ShellPkg: Make shell image exit with exit code
instead of EFI_ABORTED while exit status is other than SHELL_SUCCESS.
Importance: High
Signed-off-by: Peter Kirmeier
<[email protected]<mailto:[email protected]>>
From: Qiu, Shumin [mailto:[email protected]]
Sent: Thursday, September 11, 2014 8:01 AM
To: Carsey, Jaben; Olivier Martin
Cc: [email protected]<mailto:[email protected]>
Subject: [edk2] [Patch]ShellPkg: Make shell image exit with exit code instead
of EFI_ABORTED while exit status is other than SHELL_SUCCESS.
Hi Jaben and Olivier,
Could you help to review the patch? When exit status is not SHELL_SUCCESS make
the shell exit with the exit code instead of EFI_ABORTED.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Qiu Shumin <[email protected]<mailto:[email protected]>>
Thanks,
Shumin
------------------------------------------------------------------------------
Want excitement?
Manually upgrade your production database.
When you want reliability, choose Perforce
Perforce version control. Predictably reliable.
http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel
