Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 1 April 2016 at 16:07, Yao, Jiewen wrote: > That is enough. Thanks! > > I suggest you use this way as temp work-around. > > As final solution, I think we can construct table after all ReadyToBoot > events are processed, which need more update in DxeCore. > For now, the

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

heuvel [mailto:ard.biesheu...@linaro.org] > Sent: Friday, April 1, 2016 10:02 PM > To: Yao, Jiewen <jiewen@intel.com> > Cc: edk2-devel@lists.01.org <edk2-de...@ml01.01.org>; Gao, Liming > <liming....@intel.com> > Subject: Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

2-de...@ml01.01.org>; Gao, Liming >> <liming....@intel.com> >> Subject: Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table >> type issue >> >> On 1 April 2016 at 14:16, Ard Biesheuvel <ard.biesheu...@linaro.org> wrote: >> > On

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

day, April 1, 2016 8:19 PM > To: Yao, Jiewen <jiewen@intel.com> > Cc: edk2-devel@lists.01.org <edk2-de...@ml01.01.org>; Gao, Liming > <liming....@intel.com> > Subject: Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table > type issue > > On

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 1 April 2016 at 14:16, Ard Biesheuvel wrote: > On 1 April 2016 at 09:25, Ard Biesheuvel wrote: >> On 1 April 2016 at 09:24, Ard Biesheuvel wrote: >>> On 31 March 2016 at 15:33, Ard Biesheuvel

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 1 April 2016 at 09:24, Ard Biesheuvel wrote: > On 31 March 2016 at 15:33, Ard Biesheuvel wrote: >> On 31 March 2016 at 15:20, Yao, Jiewen wrote: >>> Hi Ard >>> Thanks for your log. Yes, it seems new issue, I have

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 31 March 2016 at 15:33, Ard Biesheuvel wrote: > On 31 March 2016 at 15:20, Yao, Jiewen wrote: >> Hi Ard >> Thanks for your log. Yes, it seems new issue, I have never encountered >> before. >> >> Here is what I found: >> >> 1)

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

vel@lists.01.org > <edk2-de...@ml01.01.org> > Subject: Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table > type issue > > On 31 March 2016 at 16:09, Yao, Jiewen <jiewen@intel.com> wrote: > > Thanks for your quick response. > > > >

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

rd. >> -Original Message- >> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] >> Sent: Thursday, March 31, 2016 9:34 PM >> To: Yao, Jiewen <jiewen@intel.com> >> Cc: Gao, Liming <liming@intel.com>; edk2-devel@lists.01.org >> <e

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

; From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Thursday, March 31, 2016 9:34 PM > To: Yao, Jiewen <jiewen@intel.com> > Cc: Gao, Liming <liming@intel.com>; edk2-devel@lists.01.org > <edk2-de...@ml01.01.org> > Subject: Re: [edk2] [patch V3] Md

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 31 March 2016 at 15:20, Yao, Jiewen wrote: > Hi Ard > Thanks for your log. Yes, it seems new issue, I have never encountered before. > > Here is what I found: > > 1) MemoryAttributeTable is always installed in ReadyToBoot event. (Sorry, I > gave wrong info on EndOfDxe,

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

gt;> To: Yao, Jiewen <jiewen@intel.com>; Ard Biesheuvel >>> <ard.biesheu...@linaro.org> >>> Cc: edk2-devel@lists.01.org <edk2-de...@ml01.01.org>; Gao, Liming >>> <liming@intel.com> >>> Subject: RE: [edk2] [patch V3] MdeModule

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

t; Sent: Thursday, March 31, 2016 4:51 PM >> To: Yao, Jiewen <jiewen@intel.com>; Ard Biesheuvel >> <ard.biesheu...@linaro.org> >> Cc: edk2-devel@lists.01.org <edk2-de...@ml01.01.org>; Gao, Liming >> <liming@intel.com> >> Subject: RE: [edk2]

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 31 March 2016 at 10:48, Yao, Jiewen wrote: > Hi Ard > I think you replied mail say this patch works fine at Wednesday, February 24, > 2016 4:15 PM. > > May I know if this is new issue or old issue? > This is a new issue. > > Also, in order to narrow down issue, would

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

t; Ard Biesheuvel > Sent: Thursday, March 31, 2016 4:37 PM > To: Yao, Jiewen <jiewen@intel.com> > Cc: edk2-devel@lists.01.org <edk2-de...@ml01.01.org>; Gao, Liming > <liming@intel.com> > Subject: Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table &

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

On 23 February 2016 at 05:46, Jiewen Yao wrote: > According to the spec, each entry in the Memory > Attributes table shall have the same type as > the region it was carved out of in the UEFI memory map. > The current attribute uses RTData for PE Data, but > it should be

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

Thank you very much! From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] Sent: Wednesday, February 24, 2016 4:15 PM To: Yao, Jiewen Cc: Gao, Liming; edk2-de...@ml01.01.org Subject: Re: [patch V3] MdeModulePkg: Fix Memory Attributes table type issue On 24 February 2016 at 07:02, Yao, Jiewen

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

Hi Ard Liming gave me minor update comment, and I have finished V3. Should I wait for your validation on AArch64? Thank you Yao Jiewen -Original Message- From: Gao, Liming Sent: Tuesday, February 23, 2016 1:10 PM To: Yao, Jiewen; edk2-de...@ml01.01.org Cc: Ard Biesheuvel Subject: RE:

Re: [edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

Reviewed-by: Liming Gao -Original Message- From: Yao, Jiewen Sent: Tuesday, February 23, 2016 12:47 PM To: edk2-de...@ml01.01.org Cc: Yao, Jiewen ; Ard Biesheuvel ; Gao, Liming Subject: [patch

[edk2] [patch V3] MdeModulePkg: Fix Memory Attributes table type issue

According to the spec, each entry in the Memory Attributes table shall have the same type as the region it was carved out of in the UEFI memory map. The current attribute uses RTData for PE Data, but it should be RTCode. This patch fixed the issue. It is validated with or without PropertiesTable.