Re: [edk2] [PATCH v1 1/1] ShellPkg/App: Fix memory leak and save resources.

2016-05-19 Thread Carsey, Jaben
Marvin, I will review your patches this week. -Jaben > -Original Message- > From: Marvin Häuser [mailto:marvin.haeu...@outlook.com] > Sent: Thursday, May 19, 2016 1:20 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subject: RE: [edk2] [PATCH v1 1/1] ShellP

Re: [edk2] [PATCH v1 1/1] ShellPkg/App: Fix memory leak and save resources.

2016-05-19 Thread Carsey, Jaben
Can we safely remove the ASSERt now? > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Marvin Häuser > Sent: Thursday, May 19, 2016 12:04 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subje

Re: [edk2] System Table and more

2016-05-18 Thread Carsey, Jaben
Those pointers are dependent on library constructors from associated libraries. You need to correctly include the library to make them function. The table pointer in your module's entry point should always be valid though. -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-dev

Re: [edk2] [PATCH] [MdePkg/BaseLib]: Remove overreaction to SourceLen in StrCpyS

2016-05-18 Thread Carsey, Jaben
unhelpful. I like return values way more than I do asserts. The same way we have libraries in EDKII that have constructors and no destructors which means they are risky and potentially useless for any code that must support unload. -Jaben > On May 18, 2016, at 9:57 AM, Carsey, Jaben wr

Re: [edk2] [PATCH] [MdePkg/BaseLib]: Remove overreaction to SourceLen in StrCpyS

2016-05-18 Thread Carsey, Jaben
Sure. Reviewed by: Jaben Carsey -Jaben > On May 18, 2016, at 9:44 AM, El-Haj-Mahmoud, Samer > wrote: > > Thanks Jaben! > > Will you give it a reviewed-by ? > > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf

Re: [edk2] [PATCH] [MdePkg/BaseLib]: Remove overreaction to SourceLen in StrCpyS

2016-05-18 Thread Carsey, Jaben
> -Original Message- > From: Palmer, Thomas [mailto:thomas.pal...@hpe.com] > Sent: Wednesday, May 18, 2016 9:21 AM > To: jim_dai...@dell.com; Carsey, Jaben ; edk2- > de...@lists.01.org > Cc: Kinney, Michael D ; Yao, Jiewen > ; Gao, Liming > Subject: RE: [edk2]

Re: [edk2] [PATCH] [MdePkg/BaseLib]: Remove overreaction to SourceLen in StrCpyS

2016-05-18 Thread Carsey, Jaben
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Thomas Palmer > Sent: Wednesday, May 18, 2016 9:01 AM > To: edk2-devel@lists.01.org > Cc: Kinney, Michael D ; Yao, Jiewen > ; Gao, Liming > Subject: [edk2] [PATCH] [MdePkg/BaseLib]: Remove ove

Re: [edk2] [Patch] Use a local variable to cache the pointer.

2016-05-12 Thread Carsey, Jaben
Amended for commit. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Jordan Justen > Sent: Thursday, May 12, 2016 3:20 PM > To: Carsey, Jaben ; edk2-devel@lists.01.org > Cc: Qiu, Shumin > Subject: Re: [edk2] [Patch] Us

Re: [edk2] [Patch] ShellPkg: Add argument to set block size for tftp command.

2016-05-06 Thread Carsey, Jaben
Why write the function UintnToAscDec to convert UINTN to ascii string? PrintLib can do that for you. Reviewed-by: Jaben Carsey > -Original Message- > From: Fu, Siyuan > Sent: Thursday, May 05, 2016 7:33 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shum

Re: [edk2] [PATCH] ShellPkg: Remove debug message in release binaries.

2016-05-05 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Wednesday, May 04, 2016 11:05 PM > To: edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > > Subject: [PATCH] ShellPkg: Remove debug message in release binaries. > I

Re: [edk2] [PATCH v3 1/2] ShellPkg/UefiDpLib: Fix the error message "Timer library instance error!"

2016-05-03 Thread Carsey, Jaben
Thanks! > -Original Message- > From: Zeng, Star > Sent: Monday, May 02, 2016 9:39 PM > To: Carsey, Jaben ; Cinnamon Shia > ; edk2-devel@lists.01.org > Cc: Qiu, Shumin ; samer.el-haj-mahm...@hpe.com > Subject: RE: [PATCH v3 1/2] ShellPkg/UefiDpLib: Fix the error mes

Re: [edk2] [PATCH v3 1/2] ShellPkg/UefiDpLib: Fix the error message "Timer library instance error!"

2016-05-02 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Zeng, Star > Sent: Monday, May 02, 2016 1:25 AM > To: Cinnamon Shia ; edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > ; samer.el-haj-mahm...@hpe.com > Subject: RE: [PATCH v3 1/2] ShellPkg/UefiDpLib:

Re: [edk2] [PATCH] PerformancePkg Dp_App: Delete InitCumulativeData()

2016-05-02 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Shia, Cinnamon [mailto:cinnamon.s...@hpe.com] > Sent: Monday, May 02, 2016 2:23 AM > To: Zeng, Star ; edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Gao, Liming > > Subject: RE: [PATCH] Performa

Re: [edk2] [PATCH] ShellPkg UefiDpLib: Delete one comment introduced by 196ccda08fc4

2016-05-02 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Shia, Cinnamon > Sent: Monday, May 02, 2016 2:23 AM > To: Zeng, Star ; edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Gao, Liming

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Carsey, Jaben
I missed that the default was from the RFC. I agree with you. -Jaben > -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Friday, April 29, 2016 11:00 AM > To: Carsey, Jaben > Cc: Marcin Wojtas ; Fu, Siyuan ; > ha...@marvell.com; edk2-

[edk2] [review]Please review changes to UNI file format document

2016-04-29 Thread Carsey, Jaben
Updated UNI spec to remove info specific to some consumers, not format. Updated UNI spec to allow non-ascii characters in strings. Cc: Qiu Shumin mailto:shumin@intel.com>> Cc: Erik Bjorge Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jaben Carsey mailto:jaben.car

Re: [edk2] [PATCH 1/2] ShellPkg/UefiDpLib: Fix the error message "Timer library instance error!"

2016-04-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Thursday, April 28, 2016 7:42 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Zeng, Star > Su

Re: [edk2] [PATCH 2/2] PerformancePkg/Dp_App: Fix the error message "Timer library instance error!"

2016-04-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Thursday, April 28, 2016 7:42 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Zeng, Star > Su

Re: [edk2] [RFC] MdeModulePkg, ShellPkg: increase TFTP block size

2016-04-29 Thread Carsey, Jaben
Maybe we should have a PCD control the default value so that people who build it can set it as per their own best value. Adding a parameter seems like a great (but separate) solution. -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of >

Re: [edk2] accessing IP info. set using ifconfig shell command in EFI application.

2016-04-28 Thread Carsey, Jaben
You could look at the source code for that command since you know that does what you want. It's located here: ShellPkg/Library/UefiShellNetwork1CommandsLib/Ifconfig.c > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > devendra rawat > Sent: Thu

Re: [edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Carsey, Jaben
arsey > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Wednesday, April 27, 2016 1:12 PM > To: Carsey, Jaben ; edk2-devel-01 de...@ml01.01.org> > Cc: Ni, Ruiyu ; Justen, Jordan L > ; Tian, Feng ; Yao, Jiewen > ; Zeng, Star > Subject: Re:

Re: [edk2] [PATCH] ShellPkg: Fix pci command for '_e' option

2016-04-27 Thread Carsey, Jaben
Done. > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Attar, Abdul Lateef > Sent: Tuesday, April 26, 2016 3:42 AM > To: Qiu, Shumin ; Carsey, Jaben > ; Ni, Ruiyu ; edk2- > de...@lists.01.org > Cc: Parthasar

Re: [edk2] [PATCH 1/3] MdeModulePkg: PiDxeS3BootScriptLib: honor PcdAcpiS3Enable

2016-04-27 Thread Carsey, Jaben
Laszlo, Does the library destructor not get called? Shouldn't that destructor unregister the protocol notify and leave the callback pointer in DXE core correct? -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Laszlo Ersek > Sent:

Re: [edk2] [PATCH] ShellPkg: Fix typos and EDK2 coding style issues

2016-04-27 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of El- > Haj-Mahmoud, Samer > Sent: Wednesday, April 27, 2016 8:11 AM > To: Shah, Tapan ; edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subje

Re: [edk2] [Patch] ShellPkg: Enhance ping6 to select the interface automatically

2016-04-27 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Wu, Jiaxin > Sent: Tuesday, April 26, 2016 6:21 PM > To: Bhupesh Sharma ; edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Ye, Ting ; > Fu, Siyuan > Subject: RE: [edk2] [Patch] ShellPkg: Enhance ping6 to

Re: [edk2] [PATCH] CryptoPkg: Fix TLS Lib issue when certificate is added to store multiple times

2016-04-25 Thread Carsey, Jaben
Ok. You can rb me, but I don't maintain that pkg so... Reviewed-by Jaben Carsey -Jaben > On Apr 25, 2016, at 11:17 AM, El-Haj-Mahmoud, Samer > wrote: > > Thanks Jaben. I did notice this and tried to submit the patch with the > modified subject (using git send-email wit

Re: [edk2] [PATCH] CryptoPkg: Fix TLS Lib issue when certificate is added to store multiple times

2016-04-25 Thread Carsey, Jaben
Samer, Note: This is not in the EDK2 yet, but in the staging... change [PATCH] to [Staging/HTTPS-TLS PATCH] (or something like that per the rules...) 2 questions inline also. -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Samer El

Re: [edk2] [PATCH] ShellPkg: Fix Shell treats every .EFI file as an executable application.

2016-04-25 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Sunday, April 24, 2016 10:56 PM > To: edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > > Subject: [PATCH] ShellPkg: Fix Shell treats every .EFI file as an executable > applica

Re: [edk2] [PATCH] ShellPkg: Fix pci command for '_e' option

2016-04-21 Thread Carsey, Jaben
Looks good to me. I will let Shumin or Ray review also. Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Abdul Lateef Attar > Sent: Thursday, April 21, 2016 4:24 AM > To: Carsey, Jaben ; Ni, Ruiyu

Re: [edk2] [PATCH] ShellPkg: Support connect a device handle recursively with '-r'.

2016-04-20 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey From: Qiu, Shumin Sent: Tuesday, April 19, 2016 7:52 PM To: Carsey, Jaben Cc: Qiu, Shumin ; edk2-devel@lists.01.org Subject: RE: [edk2] [PATCH] ShellPkg: Support connect a device handle recursively with '-r'. Importance: High Hi Jaben, 'ShellComm

Re: [edk2] [PATCH] ShellPkg: Support connect a device handle recursively with '-r'.

2016-04-19 Thread Carsey, Jaben
- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Qiu > Shumin > Sent: Monday, April 18, 2016 1:18 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: [edk2] [PATCH] ShellPkg: Support connect a device handle > recursively wi

Re: [edk2] [Patch] ShellPkg: Enhance ping to select the interface automatically

2016-04-19 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Wu, Jiaxin > Sent: Monday, April 18, 2016 6:52 PM > To: edk2-devel@lists.01.org > Cc: David Van Arnem ; Bhupesh Sharma > ; Carsey, Jaben ; Ye, > Ting ; Fu, Siyuan > Subject: [Patch] ShellPkg: Enhance ping

Re: [edk2] [Patch] ShellPkg: Update ping command options to sync with Spec

2016-04-18 Thread Carsey, Jaben
That sounds like a good enhancement to me. -Jaben > -Original Message- > From: Wu, Jiaxin > Sent: Thursday, April 14, 2016 8:52 PM > To: Carsey, Jaben ; David Van Arnem > ; Bhupesh Sharma ; > Laszlo Ersek ; edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan

Re: [edk2] [PATCH v2] PerformancePkg: Make Dp print help information with -? flag in Shell.

2016-04-13 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Zeng, Star > Sent: Wednesday, April 13, 2016 1:07 AM > To: Qiu, Shumin ; edk2-devel@lists.01.org > Cc: Ni, Ruiyu ; Carsey, Jaben > Subject:

Re: [edk2] [Patch] ShellPkg: Update ping command options to sync with Spec

2016-04-13 Thread Carsey, Jaben
.@ml01.01.org; Ye, Ting ; Carsey, Jaben > ; Fu, Siyuan ; David Van > Arnem > Subject: Re: [edk2] [Patch] ShellPkg: Update ping command options to sync > with Spec > Importance: High > > On 04/13/16 05:16, Jiaxin Wu wrote: > > This patch is used to update ping command optio

Re: [edk2] [PATCH v2] ShellPkg : Cache the environment variable into memory to enhance the performance.

2016-04-13 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ni, > Ruiyu > Sent: Wednesday, April 13, 2016 12:16 AM > To: Qiu, Shumin ; edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH v2] ShellPkg : Cache the environment vari

Re: [edk2] [PATCH] ShellPkg : Cache the environment variable into memory to enhance the performance.

2016-04-11 Thread Carsey, Jaben
2 comments below. Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Sunday, April 10, 2016 5:55 AM > To: edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > ; Ni, Ruiyu > Subject: [PATCH] ShellPkg : Cache the environment va

Re: [edk2] [PATCH] ShellPkg: Fix Shell ASSERT when mv file with cwd is NULL.

2016-04-08 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Ni, Ruiyu > Sent: Friday, April 08, 2016 3:10 AM > To: Qiu, Shumin ; edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: RE: [edk2] [PATCH] ShellPkg: Fix Shell ASSERT when mv file with cwd &g

Re: [edk2] [PATCH 03/12] FatBinPkg: Change to 2-clause BSD license

2016-04-07 Thread Carsey, Jaben
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Laszlo Ersek > Sent: Thursday, April 07, 2016 12:30 AM > To: Justen, Jordan L ; Ni, Ruiyu > ; edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH 03/12] FatBinPkg: Change to 2-clause BSD lice

Re: [edk2] Using multiple SNP drivers on Shell

2016-04-04 Thread Carsey, Jaben
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo > Ersek > Sent: Monday, April 04, 2016 10:47 AM > To: Bhupesh Sharma > Cc: edk2-devel@lists.01.org ; David Van Arnem > > Subject: Re: [edk2] Using multiple SNP drivers on Shell > Importa

Re: [edk2] How to initiate a PXE boot from the UEFI shell

2016-03-30 Thread Carsey, Jaben
If your BDS supports PXE booting, you should be able to "exit" the shell and then use BDS to initiate a PXE boot.. I think you're right that there is no built in command to initiate a PXE boot. There is no command to initiate any boot type. -Jaben > -Original Message- > From: edk2-dev

Re: [edk2] [PATCH] ShellPkg: Refine the comparisons code in ShellPkg.

2016-03-28 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Sunday, March 27, 2016 11:48 PM > To: edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > > Subject: [PATCH] ShellPkg: Refine the comparisons code in ShellPkg. > Importance: High

Re: [edk2] [PATCH] ShellPkg: Make the USB mouse behavior in 'edit' consistent with 'hexedit'.

2016-03-23 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Tuesday, March 22, 2016 1:36 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subject: [PATCH] ShellPkg: Make the USB mouse behavior in 'edit' consistent > with 'hexed

Re: [edk2] Shell Aliases and Case-Sensitivity

2016-03-22 Thread Carsey, Jaben
-Jaben > On Mar 22, 2016, at 10:32 AM, "jim_dai...@dell.com" > wrote: > > >> Jaben asked: > >> Why would we want to add something case sensitive when everything else >> is not case sensitive? Do we really want to have to add DIR, dIR, >> Dir, >> (etc...) to properly handle the built in al

Re: [edk2] Shell Aliases and Case-Sensitivity

2016-03-22 Thread Carsey, Jaben
Jim, Why would we want to add something case sensitive when everything else is not case sensitive? Do we really want to have to add DIR, dIR, Dir, (etc...) to properly handle the built in alias that is required in the shell spec? What if we requested to change the spec to say alias is case ins

Re: [edk2] [PATCH] MdePkg: Add EFI_FIRMWARE_IMAGE_DESCRIPTOR V1 and V2 structure definition.

2016-03-19 Thread Carsey, Jaben
Looks good. Reviewed-by: Jaben Carsey > -Original Message- > From: Tapan Shah [mailto:tapands...@hpe.com] > Sent: Wednesday, March 16, 2016 10:31 AM > To: edk2-devel@lists.01.org > Cc: samer.el-haj-mahm...@hpe.com; Carsey, Jaben > ; Tapan Shah > Subject

Re: [edk2] [PATCH] ShellPkg: Modify the 'dh' Shell command to dump the Firmware Management Protocol Image Descriptor Information.

2016-03-19 Thread Carsey, Jaben
We need to split into 2 patches (a series), one for MdePkg and one for ShellPkg. Comments below inline. > -Original Message- > From: Tapan Shah [mailto:tapands...@hpe.com] > Sent: Wednesday, March 16, 2016 8:55 AM > To: edk2-devel@lists.01.org > Cc: samer.el-haj-mahm...@

Re: [edk2] [PATCH] ShellPkg: Modify 'dh' Shell command to dump the Firmware Management Protocol Image Descriptor Information.

2016-03-19 Thread Carsey, Jaben
One final question below. > -Original Message- > From: Tapan Shah [mailto:tapands...@hpe.com] > Sent: Wednesday, March 16, 2016 10:32 AM > To: edk2-devel@lists.01.org > Cc: samer.el-haj-mahm...@hpe.com; Carsey, Jaben > ; Tapan Shah > Subject: [PATCH] ShellPkg: Modi

Re: [edk2] [PATCH] ShellPkg: Modify 'dh' Shell command to dump the Firmware Management Protocol Image Descriptor Information.

2016-03-19 Thread Carsey, Jaben
Got it. makes sense. Reviewed-by: Jaben Carsey > -Original Message- > From: Shah, Tapan [mailto:tapands...@hpe.com] > Sent: Wednesday, March 16, 2016 11:08 AM > To: Carsey, Jaben ; edk2-devel@lists.01.org > Cc: El-Haj-Mahmoud, Samer > Subject: RE: [PATCH] ShellPkg:

Re: [edk2] [PATCH] ShellPkg: Modify the 'dh' Shell command to dump the Firmware Management Protocol Image Descriptor Information.

2016-03-19 Thread Carsey, Jaben
Looks good to me. I will let Shumin check also. Reviewed-by: Jaben Carsey > -Original Message- > From: Tapan Shah [mailto:tapands...@hpe.com] > Sent: Friday, March 18, 2016 9:32 AM > To: edk2-devel@lists.01.org > Cc: samer.el-haj-mahm...@hpe.com; Carsey, Jaben > ; Tap

Re: [edk2] [PATCH] ShellPkg:

2016-03-15 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: jim_dai...@dell.com [mailto:jim_dai...@dell.com] > Sent: Tuesday, March 15, 2016 2:09 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: [PATCH] ShellPkg: > Importance: High >

Re: [edk2] [PATCH] ShellPkg: Fix Shell ASSERT when read 'TAB' key fail from 'ConIn'.

2016-03-14 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Ni, Ruiyu > Sent: Sunday, March 13, 2016 7:14 PM > To: Qiu, Shumin ; edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subject: RE: [PATCH] ShellPkg: Fix Shell ASSERT when read 'TAB' key fail from &

Re: [edk2] EDK2 Setup & Configuration Issue

2016-03-08 Thread Carsey, Jaben
BaseTools and Conf should be one directory up I think. From: Jim Slaughter [mailto:jwslau...@yahoo.com] Sent: Tuesday, March 08, 2016 4:19 PM To: Carsey, Jaben ; Gao, Liming ; Tian, Feng ; edk2-devel@lists.01.org Subject: Re: [edk2] EDK2 Setup & Configuration Issue Importance: High Win

Re: [edk2] EDK2 Setup & Configuration Issue

2016-03-08 Thread Carsey, Jaben
Jim, You need to download or build the BaseTools that are used during the build. Have you done that? -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jim > Slaughter > Sent: Tuesday, March 08, 2016 2:21 PM > To: Gao, Liming ; Tian, Fen

Re: [edk2] [PATCH 3/3] BaseTools ConvertMasmToNasm: Support Python 3

2016-03-08 Thread Carsey, Jaben
> -Original Message- > From: Justen, Jordan L > Sent: Tuesday, March 08, 2016 11:37 AM > To: Carsey, Jaben > Cc: edk2-devel@lists.01.org; Gao, Liming ; Bjorge, Erik > C > Subject: Re: [edk2] [PATCH 3/3] BaseTools ConvertMasmToNasm: Support > Python 3 > Impor

Re: [edk2] [PATCH 3/3] BaseTools ConvertMasmToNasm: Support Python 3

2016-03-08 Thread Carsey, Jaben
> On Mar 7, 2016, at 7:16 PM, Jordan Justen wrote: > > The script is updated to support both python 2.7 and python 3. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jordan Justen > Cc: Yonghong Zhu > Cc: Liming Gao > Cc: Erik Bjorge > --- > BaseTools/Scripts/C

Re: [edk2] [PATCH] ShellPkg: Use DOS format end of line.

2016-03-08 Thread Carsey, Jaben
Fix Intel's copyright years in the modified files. If you do that, then good. Reviewed-By: Jaben Carsey -Jaben Sent from my iPad. > On Mar 8, 2016, at 4:57 AM, Qiu, Shumin wrote: > > Use DOS format end of line(CR, LF). > > Cc: Jaben Carsey > Contributed-under:

Re: [edk2] [PATCH v2 1/2] PerformancePkg/Dp_App: Support execution break

2016-03-07 Thread Carsey, Jaben
Also for series. Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of El- > Haj-Mahmoud, Samer > Sent: Monday, March 07, 2016 4:57 AM > To: edk2-devel@lists.01.org; Shia, Cinnamon > Subject: Re: [edk2] [PATCH v2 1/2] Per

Re: [edk2] [PATCH] ShellPkg: Fix smbiosview decode of PCIe Extended Capabilities

2016-03-04 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey And commited. > -Original Message- > From: Samer El-Haj-Mahmoud [mailto:samer.el-haj-mahm...@hpe.com] > Sent: Thursday, March 03, 2016 7:32 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Samer El-Haj-Mahmoud mahm...@

Re: [edk2] [patch] ShellPkg: Merge Ping6 and Ifconfig6 tools to Shell command.

2016-03-04 Thread Carsey, Jaben
01, 2016 6:14 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Wu, Jiaxin > ; Ye, Ting ; Fu, Siyuan > > Subject: [patch] ShellPkg: Merge Ping6 and Ifconfig6 tools to Shell command. > Importance: High > > According to the new Shell spec, we add Network2 profile and >

Re: [edk2] [Patch] ShellPkg: Update 'ifconfig -r' implementation

2016-03-03 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Subramanian, Sriram (EG Servers Platform SW) [mailto:sriram- > s...@hpe.com] > Sent: Wednesday, March 02, 2016 7:13 PM > To: Wu, Jiaxin ; edk2-devel@lists.01.org > Cc: Ye, Ting ; Fu, Siyuan ; Carsey, > Jaben ;

Re: [edk2] [Patch] ShellPkg: Update 'ifconfig -r' implementation

2016-03-02 Thread Carsey, Jaben
:57 PM >> To: Laszlo Ersek >> Cc: Wu, Jiaxin ; edk2-devel@lists.01.org > de...@ml01.01.org>; Ye, Ting ; Carsey, Jaben >> ; Fu, Siyuan >> Subject: Re: [edk2] [Patch] ShellPkg: Update 'ifconfig -r' implementation >> >>> On 2 March 2016 at 09:02,

Re: [edk2] [PATCH] ShellPkg: Don't strip BOM when piping UCS2 data to a UEFI app

2016-03-01 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey This patch worked great! > -Original Message- > From: jim_dai...@dell.com [mailto:jim_dai...@dell.com] > Sent: Tuesday, March 01, 2016 1:02 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: [PATCH] ShellPkg:

Re: [edk2] [EDK2] [PATCH] ShellPkg: Ensure UEFI apps open the correct StdOut

2016-03-01 Thread Carsey, Jaben
; Sent: Tuesday, March 01, 2016 12:48 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: [edk2] [EDK2] [PATCH] ShellPkg: Ensure UEFI apps open the correct > StdOut > Importance: High > > Dell - Internal Use - Confidential > > ShellPkg: E

Re: [edk2] Force shell device mapping

2016-03-01 Thread Carsey, Jaben
You could use map to delete the standard map names for sure, then remap them how you like. I think the problem would be parsing the HD node in a script file though. -Jaben > -Original Message- > From: Foster, Matthew I > Sent: Tuesday, March 01, 2016 9:00 AM > To: C

Re: [edk2] Force shell device mapping

2016-03-01 Thread Carsey, Jaben
Matthew, I don't know how to do that quickly given the current code. When you change the HD node of the device path, the whole thing changes. The consistent name is designed to accommodate adding or removing devices and not l losing the names of the other devices in the system. This is not

Re: [edk2] [PATCH] ShellPkg: Add FileSize member to shell memory file structure.

2016-02-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel-boun...@lists.01.org [mailto:edk2-devel- > boun...@lists.01.org] On Behalf Of jim_dai...@dell.com > Sent: Thursday, February 18, 2016 6:41 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, S

Re: [edk2] RESEND: [PATCH] ShellPkg: Do not write the UNICODE BOM on ConOut

2016-02-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel-boun...@lists.01.org [mailto:edk2-devel- > boun...@lists.01.org] On Behalf Of jim_dai...@dell.com > Sent: Thursday, February 18, 2016 6:47 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, S

Re: [edk2] [PATCH] ShellPkg: Increase reallocation size for temp memory files

2016-02-29 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel-boun...@lists.01.org [mailto:edk2-devel- > boun...@lists.01.org] On Behalf Of jim_dai...@dell.com > Sent: Thursday, February 18, 2016 7:52 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, S

Re: [edk2] [PATCH] ShellPkg: Fix unexpected behavior of mouse cursor in Editor.

2016-02-26 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Thursday, February 25, 2016 10:54 PM > To: edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > ; Ni, Ruiyu > Subject: [PATCH] ShellPkg: Fix unexpected behavior of mouse cursor in &

Re: [edk2] [PATCH v4 0/2] Enhance Shell to support finding help message embeded in resource section.

2016-02-23 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Qiu > Shumin > Sent: Monday, February 22, 2016 11:01 PM > To: edk2-devel@lists.01.org > Subject: [edk2] [PATCH v4 0/2] Enhance Shell to support finding help message > em

Re: [edk2] [PATCH] ShellPkg: Do NULL pointer check before the pointer is used.

2016-02-21 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey -Jaben Sent from my iPad. > On Feb 21, 2016, at 6:49 PM, Qiu, Shumin wrote: > > Do NULL pointer check before the pointer is used to avoid dereferenced. > > Cc: Jaben Carsey > Cc: Ruiyu Ni > Contributed-under: TianoCore Contribution Agreemen

Re: [edk2] [PATCH] ShellPkg: Increase reallocation size for temp memory files

2016-02-18 Thread Carsey, Jaben
That sounds good to me. > -Original Message- > From: jim_dai...@dell.com [mailto:jim_dai...@dell.com] > Sent: Thursday, February 18, 2016 8:26 AM > To: Carsey, Jaben > Cc: Qiu, Shumin ; edk2-devel@lists.01.org > Subject: RE: [edk2] [PATCH] ShellPkg: Increase realloca

Re: [edk2] [PATCH] ShellPkg: Increase reallocation size for temp memory files

2016-02-18 Thread Carsey, Jaben
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > jim_dai...@dell.com > Sent: Thursday, February 18, 2016 7:52 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: [edk2] [PATCH] ShellPk

Re: [edk2] [PATCH v1 1/1] StdLib/BsdSocketLib: Fix minor memory leak.

2016-02-17 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Daryl McDaniel > Sent: Tuesday, February 16, 2016 12:09 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Colin King > > Subject: [edk

Re: [edk2] [PATCH] ShellPkg: Do NULL pointer check before the pointer is used.

2016-02-17 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Qiu > Shumin > Sent: Monday, February 15, 2016 6:07 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > > Subject: [edk2] [

Re: [edk2] [PATCH] ShellPkg: ShellFileHandleReadLine must return UCS2 lines

2016-02-10 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: Wednesday, February 10, 2016 6:16 AM > To: jim_dai...@dell.com > Cc: edk2-devel@lists.01.org; Carsey, Jaben ; Qiu, > Shumin > Subject: Re: [e

Re: [edk2] [PATCH] Maintainers.txt: Update maintainers for ShellBinPkg

2016-02-09 Thread Carsey, Jaben
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Leif > Lindholm > Sent: Tuesday, February 09, 2016 10:13 AM > To: Carsey, Jaben > Cc: edk2-devel@lists.01.org; Qiu, Shumin ; Ard > Biesheuvel > Subject: Re: [edk2]

Re: [edk2] [PATCH] ShellPkg Fix ASCII and UNICODE file pipes

2016-02-09 Thread Carsey, Jaben
Ryan Harkin > Sent: Tuesday, February 09, 2016 10:15 AM > To: Carsey, Jaben > Cc: jim_dai...@dell.com; edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH] ShellPkg Fix ASCII and UNICODE file pipes > Importance: High > > Hi Jaben, > > On 8 February 2016 at 19:35, Carsey

Re: [edk2] [PATCH] Maintainers.txt: Update maintainers for ShellBinPkg

2016-02-09 Thread Carsey, Jaben
el@lists.01.org; Carsey, Jaben ; Qiu, > Shumin > Subject: Re: [PATCH] Maintainers.txt: Update maintainers for ShellBinPkg > Importance: High > > On 9 February 2016 at 18:18, Leif Lindholm > wrote: > > Separate ShellPkg and ShellBinPkg into separate entities. >

Re: [edk2] [PATCH] Maintainers.txt: Update maintainers for ShellBinPkg

2016-02-09 Thread Carsey, Jaben
> -Original Message- > From: Leif Lindholm [mailto:leif.lindh...@linaro.org] > Sent: Tuesday, February 09, 2016 9:19 AM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Ard Biesheuvel > Subject: [PATCH] Maintainers.txt: Update maintainers for She

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2016-02-09 Thread Carsey, Jaben
olm > Sent: Tuesday, February 09, 2016 8:54 AM > To: Carsey, Jaben > Cc: edk2-devel@lists.01.org; Laszlo Ersek ; Qiu, Shumin > ; Ard Biesheuvel > Subject: Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update. > Importance: High > > Thanks! > > (Argh, forgot

Re: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update.

2016-02-09 Thread Carsey, Jaben
-devel@lists.01.org > Cc: Carsey, Jaben ; Qiu, Shumin > ; Laszlo Ersek ; Ard Biesheuvel > > Subject: [edk2] [PATCH] ShellBinPkg: Arm/AArch64 Shell binary update. > Importance: High > > The binaries of ShellBinPkg are generated with ShellPkg from > 9a8f6fb40712 (&qu

Re: [edk2] [PATCH] Update Shell Library INF file version

2016-02-08 Thread Carsey, Jaben
Committed and done. Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > jim_dai...@dell.com > Sent: Monday, February 08, 2016 1:53 PM > To: edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subject:

Re: [edk2] [PATCH] ShellPkg Fix ASCII and UNICODE file pipes

2016-02-08 Thread Carsey, Jaben
to:jim_dai...@dell.com] Sent: Monday, February 08, 2016 11:03 AM To: Carsey, Jaben Cc: Qiu, Shumin Subject: RE: [edk2] [PATCH] ShellPkg Fix ASCII and UNICODE file pipes Importance: High Strange. I didn't see them on the mailing list posting, but I assumed the list had stripped them. They are

Re: [edk2] [PATCH] ShellPkg/UefiDpLib: Refine the code of locating all handles in DpTrace.c.

2016-02-08 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Cinnamon Shia > Sent: Monday, February 08, 2016 9:39 AM > To: edk2-devel@lists.01.org > Subject: [edk2] [PATCH] ShellPkg/UefiDpLib: Refine the code of locating all > h

Re: [edk2] manually booting efi file

2016-02-05 Thread Carsey, Jaben
Since you asked here is a little more info: The file's EFI_FILE_PROTOCOL is insufficient information to find it as it just contains the path in file system for the file. That's the equivalent to saying "I want to open file directory/foo.txt". you need to give more context for success. To loa

Re: [edk2] [PATCH v2 6/6] Revert "ArmPlatformPkg: Create an ARM Platform DSC / FDF / ArmPlatformLib template"

2016-02-04 Thread Carsey, Jaben
ro-u...@lists.linaro.org; Carsey, Jaben > > Subject: Re: [edk2] [PATCH v2 6/6] Revert "ArmPlatformPkg: Create an ARM > Platform DSC / FDF / ArmPlatformLib template" > Importance: High > > On 02/04/16 12:53, Ryan Harkin wrote: > > On 3 February 2016 at 17:38, Leif

Re: [edk2] [PATCH v2 6/6] Revert "ArmPlatformPkg: Create an ARM Platform DSC / FDF / ArmPlatformLib template"

2016-02-03 Thread Carsey, Jaben
> -Original Message- > From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: Wednesday, February 03, 2016 9:25 AM > To: Carsey, Jaben > Cc: edk2-de...@ml01.01.org; linaro-u...@lists.linaro.org; Leif Lindholm > ; Justen, Jordan L ; Ard > Biesheuvel > Subject

Re: [edk2] [PATCH v2 6/6] Revert "ArmPlatformPkg: Create an ARM Platform DSC / FDF / ArmPlatformLib template"

2016-02-03 Thread Carsey, Jaben
Change looks good. Question - Is there a way to optimize the patch/email for this type of change so that we do not see the "diff" for a file that is deleted? It seems redundant to show a whole file of "-" lines... -Jaben > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun.

Re: [edk2] [PATCH] Maintainers.txt: Update source repository urls

2016-02-02 Thread Carsey, Jaben
I like that a lot. Reviewed-by: Jaben Carsey > -Original Message- > From: Justen, Jordan L > Sent: Tuesday, February 02, 2016 9:35 AM > To: Carsey, Jaben ; edk2-devel@lists.01.org > Cc: Carsey, Jaben > Subject: RE: [edk2] [PATCH] Maintainers.txt: Update sour

Re: [edk2] [PATCH] Maintainers.txt: Update source repository urls

2016-02-02 Thread Carsey, Jaben
Looks like good info. Should we clarify that SVN is read only also? Reviewed-by: Jaben Carsey > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Jordan Justen > Sent: Tuesday, February 02, 2016 9:03 AM > To: edk2-devel@lists.01.org > Cc: Juste

Re: [edk2] [PATCH] ShellPkg: Refine the code to avoid use a constant number as offset.

2016-01-25 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Qiu, Shumin > Sent: Monday, January 25, 2016 12:01 AM > To: edk2-devel@lists.01.org > Cc: Qiu, Shumin ; Carsey, Jaben > > Subject: [PATCH] ShellPkg: Refine the code to avoid use a constant number as > of

Re: [edk2] [PATCH v2 00/14] ShellPkg: BCFG dump improvements

2016-01-21 Thread Carsey, Jaben
For series. Reviewed-by: Jaben Carsey > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, January 21, 2016 9:07 AM > To: edk2-devel-01 > Cc: Carsey, Jaben ; Ryan Harkin > > Subject: [PATCH v2 00/14] ShellPkg: BCFG dump improvemen

Re: [edk2] [PATCH v2 01/14] ShellPkg: BcfgDisplayDump(): update whitespace & layout

2016-01-21 Thread Carsey, Jaben
Reviewed-by: Jaben Carsey > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Thursday, January 21, 2016 9:07 AM > To: edk2-devel-01 > Cc: Carsey, Jaben ; Ryan Harkin > > Subject: [PATCH v2 01/14] ShellPkg: BcfgDisplayDump(): update

Re: [edk2] [PATCH] [ShellPkg] fix operator

2016-01-21 Thread Carsey, Jaben
> -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of > Laszlo Ersek > Sent: Thursday, January 21, 2016 8:13 AM > To: Carsey, Jaben ; Ryan Harkin > > Cc: edk2-devel@lists.01.org > Subject: Re: [edk2] [PATCH] [ShellPkg] f

Re: [edk2] [PATCH] [ShellPkg] fix operator

2016-01-21 Thread Carsey, Jaben
> -Original Message- > From: Ryan Harkin [mailto:ryan.har...@linaro.org] > Sent: Thursday, January 21, 2016 12:47 AM > To: Laszlo Ersek > Cc: Carsey, Jaben ; edk2-devel@lists.01.org de...@ml01.01.org> > Subject: Re: [PATCH] [ShellPkg] fix operator > Importance:

Re: [edk2] [PATCH 00/13] ShellPkg: BCFG dump improvements

2016-01-20 Thread Carsey, Jaben
l-boun...@lists.01.org] On Behalf Of Laszlo > Ersek > Sent: Wednesday, January 20, 2016 5:12 PM > To: edk2-de...@ml01.01.org > Cc: Carsey, Jaben ; Ryan Harkin > > Subject: [edk2] [PATCH 00/13] ShellPkg: BCFG dump improvements > Importance: High > > This series cleans up the &q

Re: [edk2] [PATCH] [ShellPkg] fix operator

2016-01-20 Thread Carsey, Jaben
Note that when you specify your 1 2 3 it’s interpreted as a keystroke code. If you want a string you should use “1 2 3”… From: Ryan Harkin [mailto:ryan.har...@linaro.org] Sent: Wednesday, January 20, 2016 2:18 PM To: Carsey, Jaben Cc: edk2-devel@lists.01.org ; Laszlo Ersek Subject: RE

Re: [edk2] Shell BCFG command - adding option data

2016-01-20 Thread Carsey, Jaben
ER_OPT (x) (BCFG_VAR_BUFFER_DPATH(x)+BCFG_VAR_BUFFER_LEN(x)) -Jaben > -Original Message- > From: Laszlo Ersek [mailto:ler...@redhat.com] > Sent: Wednesday, January 20, 2016 1:26 PM > To: Ryan Harkin ; Carsey, Jaben > > Cc: edk2-devel@lists.01.org ; Linaro UEFI >

<    1   2   3   4   5   6   7   8   >