Liming,
That was exactly what I was looking for.
Thanks
Sean
-Original Message-
From: Gao, Liming
Sent: Wednesday, November 7, 2018 10:01 PM
To: Sean Brogan
Cc: edk2-devel@lists.01.org
Subject: RE: Edk2 uni file encoding
Sean:
EDKII UNI spec
Yeah, there is some good resource on the web.
ACPI spec defined APEI for error handling.
https://www.uefi.org/sites/default/files/resources/ACPI_6_2.pdf
Below ppt introduced basic info on memory technology such as spare or mirror.
ftp://ftp.hp.com/pub/c-products/servers/options/c00256943.pdf
From: "bob.c.f...@intel.com"
https://bugzilla.tianocore.org/show_bug.cgi?id=1288
This patch is one of build tool performance improvement
series patches.
This patch is going to add cache for UniToHexList function.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: BobCF
https://bugzilla.tianocore.org/show_bug.cgi?id=1288
This patch is one of build tool performance improvement
series patches.
This patch is going to customize the deepcopy function for
SkuClass, PcdClassObject and python dictionary.
python deepcopy copy everything of a object, but for our current
Sean:
EDKII UNI spec
(https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Specifications)
Chapter 2 defines UNI file format. EdkCompatibilityPkg is obsolete. BZ
https://bugzilla.tianocore.org/show_bug.cgi?id=1103 is submitted to delete
EdkCompatibilityPkg from edk2/master. We will
Star:
There is no special process for Hard Feature Freeze. If the people has the
comments or rejection for the bug fix in this phase, they can give their
comments in review mail.
Yes. I will send the mail today to announce Hard Feature Freeze.
Laszlo:
Thanks for your feedback. We will
Hi Ray,
Right. No SQL dependency any more after this patch.
Thanks,
Bob
-Original Message-
From: Ni, Ruiyu
Sent: Thursday, November 8, 2018 1:37 PM
To: Feng, Bob C ; edk2-devel@lists.01.org
Cc: Carsey, Jaben ; Gao, Liming
Subject: Re: [edk2] [Patch] BaseTools: Replace the sqlite
On 11/8/2018 10:28 AM, Shenglei Zhang wrote:
https://bugzilla.tianocore.org/show_bug.cgi?id=1298
The change is based on the change below:
https://lists.01.org/pipermail/edk2-devel/2018-November/031877.html
Cc: Jordan Justen
Cc: Andrew Fish
Cc: Ruiyu Ni
Contributed-under: TianoCore
On 11/8/2018 11:15 AM, BobCF wrote:
https://bugzilla.tianocore.org/show_bug.cgi?id=1288
This patch is one of build tool performance improvement
series patches.
This patch is going to use python list to store the parser data
instead of using sqlite database.
The replacement solution is as
> -Original Message-
> From: Zhang, Shenglei
> Sent: Thursday, November 08, 2018 10:28 AM
> To: edk2-devel@lists.01.org
> Cc: Ni, Ruiyu; Wu, Hao A
> Subject: [PATCH] Nt32Pkg: Remove PcdShellFile in DSC
>
> https://bugzilla.tianocore.org/show_bug.cgi?id=1298
> This change is based on the
Has been the time to announce Hard Feature Freeze?
During Hard Feature Freeze, every checkin needs to be approved by
stewards, right?
Thanks,
Star
On 2018/11/8 3:13, Laszlo Ersek wrote:
On 11/07/18 16:38, Leif Lindholm wrote:
Hi Laszlo,
Can I inject an alternative interpretation? (feel
In current implementation, core level semaphore use same container
with package level semaphore. This design will let the core level
semaphore not works as expected in below case:
1. Feature A has CPU_FEATURE_CORE_BEFORE dependence with Feature B.
2. Feature C has CPU_FEATURE_PACKAGE_AFTER
In current implementation, core level semaphore use same container
with package level semaphore. This design will let the core level
semaphore not works as expected in below case:
1. Feature A has CPU_FEATURE_CORE_BEFORE dependence with Feature B.
2. Feature C has CPU_FEATURE_PACKAGE_AFTER
https://bugzilla.tianocore.org/show_bug.cgi?id=1298
This change is based on the change below:
https://lists.01.org/pipermail/edk2-devel/2018-November/031875.html
Cc: Ruiyu Ni
Cc: Hao Wu
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang
---
https://bugzilla.tianocore.org/show_bug.cgi?id=1298
The change is based on the change below:
https://lists.01.org/pipermail/edk2-devel/2018-November/031877.html
Cc: Jordan Justen
Cc: Andrew Fish
Cc: Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Shenglei Zhang
In current implementation, core level semaphore use same container
with package level semaphore. This design will let the core level
semaphore not works as expected in below case:
1. Feature A has CPU_FEATURE_CORE_BEFORE dependence with Feature B.
2. Feature C has CPU_FEATURE_PACKAGE_AFTER
From: Tomasz Michalec
Extend MvBoardDescSdMmcGet function to fill MV_BOARD_SDMMC_DESC
with Xenon specific info obtained from ArmadaBoardDescLib.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas
---
Silicon/Marvell/Drivers/BoardDesc/MvBoardDescDxe.inf | 1 +
From: Tomasz Michalec
This patch implements ArmadaBoarDescLib library for
Armada8040 Development Board and add to it ArmadaBoardDescSdMmcGet
function with description of connected Xenon host controllers.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas
---
From: Tomasz Michalec
This patch implements ArmadaBoarDescLib library for
Armada80x0McBin comunity board and add to it ArmadaBoardDescSdMmcGet
function with description of connected Xenon host controllers.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas
---
From: Tomasz Michalec
Added fields specific for Xenon host controller and declaration
of ArmadaBoardDescSdMmcGet function.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas
---
Silicon/Marvell/Include/Library/ArmadaBoardDescLib.h | 17 -
1
The newest changes in the SdMmcOverride protocol added additional
arguments to the NotifyPhase and Capability routines. Update
according places in the Synquacer Emmc driver.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Marcin Wojtas
---
Hi,
The second version of the patchset simply adjust Synquacer and
Xenon drivers to the modified SdMmcOverride. The details can
be found in the changelog below.
Patches are available in the github:
https://github.com/MarvellEmbeddedProcessors/edk2-open-platform/commits/xenon-upstream-r20181108
Some SdMmc host controllers are run by clocks with different
frequency than it is reflected in Capabilities Register 1.
It is allowed by SDHCI specification ver. 4.2 - if BaseClkFreq
field value of the Capability Register 1 is zero, the clock
frequency must be obtained via another method.
Because
From: Tomasz Michalec
Some SD Host Controlers need to do additional opperations after clock
frequency switch.
This patch add new callback type to NotifyPhase of the SdMmcOverride
protocol. It is called after EmmcSwitchClockFreq and SdMmcHcClockSupply.
Contributed-under: TianoCore Contribution
From: Tomasz Michalec
Some SD Host Controllers use different values in Host Control 2 Register
to select UHS Mode. This patch adds a new UhsSignaling type routine to
the NotifyPhase of the SdMmcOverride protocol.
UHS signaling configuration is moved to a common, default routine
In order to ensure bigger flexibility in the NotifyPhase
routine of the SdMmcOverride protocol, enable using an
optional phase-specific data. This will allow to exchange
more information between the protocol producer driver
and SdMmcPciHcDxe in the newly added callbacks.
Contributed-under:
Hi,
This is the third version of the patchset, which addresses
all remarks that came up during the v2 review, such as
updating the bus mode related enum/macro, headers cleanup,
etc. Details can be found in the changelog below.
Patches are available in the github:
Hi Jaben,
The patch already has been pushed after received your/Siyuan reviewed-by tag.
Thanks,
Jiaxin
> -Original Message-
> From: Carsey, Jaben
> Sent: Thursday, November 8, 2018 9:09 AM
> To: Fu, Siyuan ; Wu, Jiaxin ;
> edk2-devel@lists.01.org
> Cc: Ye, Ting
> Subject: RE: [PATCH
Reviewed-by: Star Zeng
-Original Message-
From: Bi, Dandan
Sent: Wednesday, November 7, 2018 10:53 PM
To: edk2-devel@lists.01.org
Cc: Gao, Liming ; Dong, Eric ; Zeng,
Star ; Wu, Hao A
Subject: [patch] MdeModulePkg/DisplayEngine: Remove useless NULL ptr check for
NewPos
REF:
Wu,
I plan to push this patch tomorrow, but I would like to add this to the commit
message. What do you think?
"This fixes the bug where parameter value 0 causes failure."
-Jaben
> -Original Message-
> From: Fu, Siyuan
> Sent: Monday, November 05, 2018 10:51 PM
> To: Wu, Jiaxin ;
> -Original Message-
> From: Ni, Ruiyu
> Sent: Wednesday, November 07, 2018 5:37 PM
> To: edk2-devel@lists.01.org
> Cc: Jim Dailey; Wu, Hao A
> Subject: [PATCH] ShellPkg/ShellLib: Fix potential NULL deference issue
>
> Contributed-under: TianoCore Contribution Agreement 1.1
>
Cc to the package maintainers.
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jeff
> Brasen
> Sent: Wednesday, October 31, 2018 5:31 AM
> To: edk2-devel@lists.01.org
> Cc: Jeff Brasen
> Subject: [edk2] [PATCH 1/2]
Reviewed-by: Liming Gao
> -Original Message-
> From: Bi, Dandan
> Sent: Wednesday, November 7, 2018 10:53 PM
> To: edk2-devel@lists.01.org
> Cc: Gao, Liming ; Dong, Eric ;
> Zeng, Star ; Wu, Hao A
>
> Subject: [patch] MdeModulePkg/DisplayEngine: Remove useless NULL ptr check
> for
Is there a definitive answer for the file encoding for all UNI files in edk2?
If not I would like to propose one. Incorrect encoding causes tool issues and
is something we can easily check for and fix.
Proposal: All UNI files in edk2 should be
1. UTF-8
Or
1. Use a BOM and be UTF-16
On Tue, 2018-11-06 at 17:46 +, Leif Lindholm wrote:
> On Tue, Nov 06, 2018 at 02:17:41PM +0530, Sakar Arora wrote:
> > From: John Powell
> >
> > Tests SBBR requirements 3.4.1 and 3.4.4.
It would be better if we add comments on how SBBR requirements 3.4.1
and 3.4.4 relate to the UEFI
Hi Ray,
On 11/07/18 16:54, Ruiyu Ni wrote:
> Today's code generates assembly code as below for
> InternalSyncIncrement:
> __asm__ __volatile__ (
> "movl$1, %%eax \n\t"
> "lock \n\t"
> "xadd%%eax, %1 \n\t"
> "inc %%eax \n\t"
> : "=a" (Result),
Eric,
Could you please split these as PATCH (perhaps 3 or more) series as
there are more than 10 files changed in a single patch?.
Please see comments inline.
On Wed, 2018-11-07 at 14:47 +0800, Eric Jin wrote:
> UEFI drivers which implement the EFI_SIMPLE_TEXT_INPUT_EX protocol
> are required
Thanks.
Reviewed-by: Supreeth Venkatesh
On Fri, 2018-11-02 at 03:08 -0500, Eric Jin wrote:
> The LUN is an 8 byte array that is displayed in hexadecimal
> format with byte 0 first (i.e., on the left) and byte 7 last
> (i.e, on the right).
>
> Cc: Supreeth Venkatesh
> Contributed-under:
Some of the same comments of PATCH v1 apply to v2.
Please see comments for v1 and resend a patch v3 as a series.
Thanks,
Supreeth
On Wed, 2018-11-07 at 07:19 +, Jin, Eric wrote:
> It is the patch v2 below.
>
> -Original Message-
> From: edk2-devel
On Wed, 2018-11-07 at 06:45 +, Jin, Eric wrote:
> Hi Supreeth,
>
> Thank for comment. The magic num 7 will be removed in the v2 patch.
Thanks.
>
> Best Regards
> Eric
>
> -Original Message-
> From: Supreeth Venkatesh
> Sent: Monday, October 15, 2018 10:18 AM
> To: Jin, Eric ;
On 11/07/18 17:47, Philippe Mathieu-Daudé wrote:
> On 7/11/18 14:38, Ard Biesheuvel wrote:
>> On 7 November 2018 at 14:13, Ard Biesheuvel
>> wrote:
>>> BaseIoLibIntrinsicArmVirt was created to prevent LTO from merging
>>> accesses to MMIO regions, resulting in instructions with multiple
>>>
On 11/07/18 16:38, Leif Lindholm wrote:
> Hi Laszlo,
>
> Can I inject an alternative interpretation? (feel free to shoot it down)
>
> On Wed, Nov 07, 2018 at 04:14:01PM +0100, Laszlo Ersek wrote:
>> Hi,
>>
>> On 11/07/18 02:12, Gao, Liming wrote:
>>> Hi, all
>>
>>>
On 11/07/18 16:27, Ni, Ruiyu wrote:
>
>
>> -Original Message-
>> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
>> Sent: Wednesday, November 7, 2018 10:46 PM
>> To: Laszlo Ersek
>> Cc: Ni, Ruiyu ; edk2-devel@lists.01.org; Kinney, Michael
>> D
>> ; Gao, Liming
>> Subject: Re:
Reviewed-by: Jaben Carsey
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of
> Ruiyu Ni
> Sent: Wednesday, November 07, 2018 1:37 AM
> To: edk2-devel@lists.01.org
> Cc: Wu, Hao A ; Jim Dailey
> Subject: [edk2] [PATCH] ShellPkg/ShellLib: Fix
On 5/11/18 13:54, Marvin Häuser wrote:
The second argument of "UnicodeVSPrintAsciiFormat" is "BufferSize",
which takes the size of the buffer in bytes. Replace the currently
used MAX_DEBUG_MESSAGE_LENGTH reference, which is the buffer's length,
with the actual buffer size.
Contributed-under:
On 5/11/18 13:54, Marvin Häuser wrote:
The second argument of "UnicodeVSPrintAsciiFormat" is "BufferSize",
which takes the size of the buffer in bytes. Replace the currently
used MAX_DEBUG_MESSAGE_LENGTH reference, which is the buffer's length,
with the actual buffer size.
Contributed-under:
On 7/11/18 10:36, Ruiyu Ni wrote:
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ruiyu Ni
Cc: Jim Dailey
Cc: Hao A Wu
Reviewed-by: Philippe Mathieu-Daudé
---
ShellPkg/Library/UefiShellLib/UefiShellLib.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
On 11/07/18 04:47, yuchen...@synology.com wrote:
> From: yuchenlin
>
> BAR |std vga | vmsvga
> -
> 0| Framebuffer | I/O space
> 1| Reserved | Framebuffer
> 2| MMIO | FIFO
>
> - We cannot recognize VMW SVGA as BOCHS because
On 7/11/18 14:38, Ard Biesheuvel wrote:
On 7 November 2018 at 14:13, Ard Biesheuvel wrote:
BaseIoLibIntrinsicArmVirt was created to prevent LTO from merging
accesses to MMIO regions, resulting in instructions with multiple
output registers that KVM on ARM cannot emulate (since the exception
Today's code generates assembly code as below for
InternalSyncIncrement:
__asm__ __volatile__ (
"movl$1, %%eax \n\t"
"lock \n\t"
"xadd%%eax, %1 \n\t"
"inc %%eax \n\t"
: "=a" (Result), // %0
"+m" (*Value) // %1
:
Hi Laszlo,
Can I inject an alternative interpretation? (feel free to shoot it down)
On Wed, Nov 07, 2018 at 04:14:01PM +0100, Laszlo Ersek wrote:
> Hi,
>
> On 11/07/18 02:12, Gao, Liming wrote:
> > Hi, all
>
> > https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
> >
Reviewed-by: Ruiyu Ni
> -Original Message-
> From: Dong, Eric
> Sent: Wednesday, November 7, 2018 4:26 PM
> To: edk2-devel@lists.01.org
> Cc: Laszlo Ersek ; Ni, Ruiyu
> Subject: [Patch] UefiCpuPkg/RegisterCpuFeaturesLib: Adjust Order.
>
> In current code logic, only adjust feature
> -Original Message-
> From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org]
> Sent: Wednesday, November 7, 2018 10:46 PM
> To: Laszlo Ersek
> Cc: Ni, Ruiyu ; edk2-devel@lists.01.org; Kinney, Michael D
> ; Gao, Liming
> Subject: Re: [edk2] [PATCH] MdePkg/BaseSynchronizationLib XCODE:
On 7 November 2018 at 16:07, Marc Zyngier wrote:
> On 07/11/18 14:48, Ard Biesheuvel wrote:
>> (+ Marc)
>>
>>
>> On 29 October 2018 at 05:57, Ming Huang wrote:
>>> As GicV3 Spec, Interrupt Routing Modes should be 0 for
>>> routing the SPIs to the primary CPU.
>>>
>>> Contributed-under: TianoCore
Hi,
On 11/07/18 02:12, Gao, Liming wrote:
> Hi, all
> https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning
> lists edk2-stable201811 tag planning. Now, we enter into Soft Feature
> Freeze phase. In this phase, the feature under review will not be
> allowed to be pushed.
On 7 November 2018 at 16:00, Zeng, Star wrote:
> On 2018/11/6 22:37, Zeng, Star wrote:
>>
>> On 2018/11/6 17:49, Ard Biesheuvel wrote:
>>>
>>> On 31 October 2018 at 05:38, Zeng, Star wrote:
Good feedback.
On 2018/10/30 20:50, Leif Lindholm wrote:
>
>
> On Tue, Oct
On 2018/11/6 22:37, Zeng, Star wrote:
On 2018/11/6 17:49, Ard Biesheuvel wrote:
On 31 October 2018 at 05:38, Zeng, Star wrote:
Good feedback.
On 2018/10/30 20:50, Leif Lindholm wrote:
On Tue, Oct 30, 2018 at 09:39:24AM -0300, Ard Biesheuvel wrote:
(add back the list)
Oi! Go back on
I agree my 1st version commit message is too general.
Because I don't know if RAS can be mentioned:)
> -Original Message-
> From: Laszlo Ersek [mailto:ler...@redhat.com]
> Sent: Wednesday, November 7, 2018 10:46 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Yao, Jiewen ; Dong, Eric
> -Original Message-
> From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Laszlo
> Ersek
> Sent: Wednesday, November 7, 2018 10:42 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Kinney, Michael D ; Gao, Liming
>
> Subject: Re: [edk2] [PATCH]
On Tue, Nov 06, 2018 at 10:10:16PM +, Kinney, Michael D wrote:
> Hi Ard,
>
> Can you please add CC lines to the commit message
> for the developers that have contributed to the
> edk2-staging/Intel_UNDI branch?
>
> This would include:
>
> Cc: Maciej Rabeda
> Cc: Kamil Kacperski
Could
> -Original Message-
> From: Laszlo Ersek [mailto:ler...@redhat.com]
> Sent: Wednesday, November 7, 2018 10:32 PM
> To: Ni, Ruiyu ; edk2-devel@lists.01.org
> Cc: Gao, Liming ; Kinney, Michael D
> ; Philippe Mathieu-Daudé ;
> Andrew Fish
> Subject: Re: [PATCH]
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1306
In function UiDisplayMenu, the NewPos ptr which used to point to the
highlight menu entry. It will always point to the menu entry which
need to be highlighted or the gMenuOption menu if the highlight menu
is not found.
So we can remove the
(+ Marc)
On 29 October 2018 at 05:57, Ming Huang wrote:
> As GicV3 Spec, Interrupt Routing Modes should be 0 for
> routing the SPIs to the primary CPU.
>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ming Huang
> ---
> ArmPkg/Drivers/ArmGic/GicV3/ArmGicV3Dxe.c | 2
On 11/07/18 09:59, Ruiyu Ni wrote:
> From: Jiewen Yao
>
> When static paging is disabled, page table for below 4GB is created
> and page table for above 4GB is created dynamically in page fault
> handler.
>
> Today's implementation only allow SMM access-out to below types of
> memory address no
On 7 November 2018 at 15:31, Laszlo Ersek wrote:
> (+Andrew)
>
> Hi Ray,
>
> On 11/07/18 05:03, Ruiyu Ni wrote:
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303
>>
>> XCODE disassembly code of InternalSyncDecrement with today's code is:
>>
>> __asm__ __volatile__ (
>> "movl
On 11/07/18 15:31, Laszlo Ersek wrote:
> (+Andrew)
>
> Hi Ray,
>
> On 11/07/18 05:03, Ruiyu Ni wrote:
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303
>>
>> XCODE disassembly code of InternalSyncDecrement with today's code is:
>>
>> __asm__ __volatile__ (
>> "movl$1, %%eax
(+Andrew)
Hi Ray,
On 11/07/18 05:03, Ruiyu Ni wrote:
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1303
>
> XCODE disassembly code of InternalSyncDecrement with today's code is:
>
> __asm__ __volatile__ (
> "movl$1, %%eax \n\t"
> "lock \n\t"
> "xadd
On 6/11/18 18:58, Ard Biesheuvel wrote:
Unlike Visual Studio, GCC does not permit implicit conversion between
a pointer-to-void-pointer and pointer to a typed pointer. So add the
explicit casts where necessary.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard
On 11/07/18 09:26, Eric Dong wrote:
> In current code logic, only adjust feature position if current CPU
> feature position not follow the request order. Just like Feature A
> need to be executed before feature B, but current feature A registers
> after feature B. So code will adjust the position
On 7 November 2018 at 14:13, Ard Biesheuvel wrote:
> BaseIoLibIntrinsicArmVirt was created to prevent LTO from merging
> accesses to MMIO regions, resulting in instructions with multiple
> output registers that KVM on ARM cannot emulate (since the exception
> syndrome information that KVM relies
On 11/07/18 09:27, Yao, Jiewen wrote:
> If Static paging is not used, then the SMM uses dynamic paging.
>
> Here dynamic paging means the page fault request on-demand.
>
> 2 special examples here:
> 1) above 4G MMIO. By default, SMM only setup paging table for 4G memory. If
> MMIO above 4G,
BaseIoLibIntrinsicArmVirt was created to prevent LTO from merging
accesses to MMIO regions, resulting in instructions with multiple
output registers that KVM on ARM cannot emulate (since the exception
syndrome information that KVM relies on can only describe a single
output register)
However,
Reviewed-by: Ruiyu Ni
Thanks/Ray
> -Original Message-
> From: Zeng, Star
> Sent: Tuesday, November 6, 2018 10:34 PM
> To: edk2-devel@lists.01.org
> Cc: Zeng, Star ; Ni, Ruiyu ; Wu,
> Hao A ; Wang, Jian J ; Yao,
> Jiewen ; Leif Lindholm ;
> Ard Biesheuvel
> Subject: [PATCH 0/2] Revert
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ruiyu Ni
Cc: Jim Dailey
Cc: Hao A Wu
---
ShellPkg/Library/UefiShellLib/UefiShellLib.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/ShellPkg/Library/UefiShellLib/UefiShellLib.c
From: "bob.c.f...@intel.com"
https://bugzilla.tianocore.org/show_bug.cgi?id=1292
This patch is going to enable Array data type for PCD.
1. Support Pcd ARRAY as Structure PCD type
including basic datatype array and structure array.
For example:
On 6/11/18 18:58, Ard Biesheuvel wrote:
Unlike Visual Studio, GCC does not permit implicit conversion between
a pointer-to-void-pointer and pointer to a typed pointer. So add the
explicit casts where necessary.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard
On 6/11/18 21:35, Ard Biesheuvel wrote:
On 6 November 2018 at 21:31, Philippe Mathieu-Daudé wrote:
Hi Ard,
On 6/11/18 18:58, Ard Biesheuvel wrote:
UINT8 and CHAR8 are not the same underlying type on all architectures,
so add an explicit cast where necessary.
Contributed-under: TianoCore
On 6/11/18 18:58, Ard Biesheuvel wrote:
Drop variables that are assigned but never used, or move them into
a conditional preprocessor block if the only references occur from
such code.
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel
---
reviewed-by: jiewen@intel.com
thank you!
Yao, Jiewen
> 在 2018年11月7日,下午4:58,Ni, Ruiyu 写道:
>
> From: Jiewen Yao
>
> When static paging is disabled, page table for below 4GB is created
> and page table for above 4GB is created dynamically in page fault
> handler.
>
> Today's
Leif,
Thanks for the comments. I'll push the patch after removing the change log
from the commit log.
Regards,
Jian
> -Original Message-
> From: Leif Lindholm [mailto:leif.lindh...@linaro.org]
> Sent: Wednesday, November 07, 2018 3:53 PM
> To: Wang, Jian J
> Cc:
If Static paging is not used, then the SMM uses dynamic paging.
Here dynamic paging means the page fault request on-demand.
2 special examples here:
1) above 4G MMIO. By default, SMM only setup paging table for 4G memory. If
MMIO above 4G, then SMM need use #PF handler to grant MMIO access.
2)
In current code logic, only adjust feature position if current CPU
feature position not follow the request order. Just like Feature A
need to be executed before feature B, but current feature A registers
after feature B. So code will adjust the position for feature A, move
it to just before
Hi Fu Siyan,
On Wed, Nov 07, 2018 at 08:12:55AM +, Fu, Siyuan wrote:
> Hi, Leif and Ard
>
> I just realized that you may not be CCed in this email. I resent these
> patches a few minutes ago, my Git Bash send-email reports it added you to CC
> receiver, but the Outlook received email still
Hi, Leif and Ard
I just realized that you may not be CCed in this email. I resent these patches
a few minutes ago, my Git Bash send-email reports it added you to CC receiver,
but the Outlook received email still doesn't have your name in CC list.
Do you have any comments for this v2 patch of
V2:
Additional fixups required for NetworkPkg migration
The PXE/iSCSI/TCP drivers in MdeModulePkg are going to be deprecated. All
platform DSC/FDF files should be updated to use the dual-stack drivers in
NetworkPkg.
The NetworkPkg driver have all the functionality compared with MdeModulePkg
one,
From: Leif Lindholm
Commit 1a13dfd37fe7 ("Hisilicon/D0x: Switch to generic PciHostBridge
driver") introduced some incorrect line endings, fix those here.
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Michael D Kinney
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Leif
Patch 1/2 fixes the line ending issue introduced by Ccommit 1a13dfd37fe7
("Hisilicon/D0x: Switch to generic PciHostBridge driver").
Patch 2/2 updates all platform DSC/FDF files for NetworkPkg migration.
Cc: Ard Biesheuvel
Cc: Leif Lindholm
Cc: Michael D Kinney
Fu Siyuan (1):
Edk2Platforms:
87 matches
Mail list logo