REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2313
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1954
The EDK2 Kernel PciBusDxe driver is enhanced to enable the configuration
of PCI features like
(1) Max_Payload_Size
(2) Max_Read
I've given R-b for other patches in this series. I just give R-b this one which
is only changed
in v7.
Reviewed-by: Jian J Wang
Regards,
Jian
> -Original Message-
> From: Kubacki, Michael A
> Sent: Saturday, November 02, 2019 1:35 AM
> To: devel@edk2.groups.io
> Cc: Bi, Dandan ; Ard B
Hi Laszlo,
Everything is back online now. Will be doing more extensive
CI unit tests next.
If there is a merge conflict at the time the PR is submitted
you should be notified by email. I think you ran into what
should be an impossible case when the service is running
correctly. The fact it was
Hello,
This RFC proposes adding a new directory to edk2-platforms called
"Features". The purpose of this directory is to add support code
for platform features. Intel would like to maintain features in the
form of feature packages in which a feature may be composed of
multiple libraries, modules,
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2164
Replaced Cmos.c and Cmos.h with BoardModulePkg's
Cmos library CmosAccessLib
Cc: Ankit Sinha
Cc: Nate DeSimone
Cc: Kubacki Michael A
Signed-off-by: Prince Agyeman
---
.../BoardInitLib/PeiX58Ich10InitPreMemLib.c | 46 ---
Laszlo,
You're right. I'll do more tests for secure boot. I'll leave https boot to you.
Regards,
Jian
> -Original Message-
> From: Laszlo Ersek
> Sent: Saturday, November 02, 2019 6:03 AM
> To: Wang, Jian J ; Zhang, Shenglei
> ; devel@edk2.groups.io
> Cc: Lu, XiaoyuX ; Gao, Liming
> Su
On 11/01/19 23:39, Laszlo Ersek wrote:
> * When I submitted PR#87, there had been no conflicts; the PR was simply
> blocked on the CI tasks. Now, there are conflicts (I assume due to
> intervening pushes, while my PR#87 was blocked). It's useful that the
> WebUI points out this change in the statu
Hi Laszlo,
I did not make any changes to the OvmfPkg patch and I forgot to carry forward
the R-b.
I'll keep that in mind in the future.
For V7, I request the MdeModulePkg maintainers please add the R-b for the
patches
not changed. If this is not acceptable, I will be happy to send out a patch
On 11/01/19 23:05, Ashish Singhal wrote:
> Please refer to https://edk2.groups.io/g/devel/message/49627 for the patch.
Thanks! The subject line is:
[PATCH] MdeModulePkg/UefiBootManagerLib: Support skipping BM enumeration
This looks like a glitch to me -- a failure to reflect your
message to *so
Hi Mike,
On 11/01/19 06:03, Kinney, Michael D wrote:
> Hi Laszlo,
>
> ===
> Note for all reviewers:
> ---
> Pull requests against edk2-staging/edk2-ci are not
> being processed right now. We are worki
Hello Michael,
On 11/01/19 18:34, Michael Kubacki wrote:
> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2220
>
> V7 Changes:
> [PATCH V6 07/10] MdeModulePkg/Variable: Add RT GetVariable() cache support
> * Remove VariableRuntimeCache.c and VariableRuntimeCache.h from
>VariableSmmRunti
Please refer to https://edk2.groups.io/g/devel/message/49627 for the patch.
Thanks
Ashish
-Original Message-
From: Laszlo Ersek
Sent: Friday, November 1, 2019 3:42 PM
To: Ashish Singhal ; devel@edk2.groups.io;
jian.j.w...@intel.com; hao.a...@intel.com; Ray Ni ; Zhichao
Gao
Subject: R
On 11/01/19 08:31, Wang, Jian J wrote:
> Hi Laszlo,
>
> I did simple ovmf boot tests (shell, linux, windows) and all passed. Let me
> know if you have
> any comments or want to do more tests against v4 before check in.
>
> Based on my review and tests,
>Reviewed-by: Jian J Wang
I can get t
For the patch series...
Reviewed-by: Nate DeSimone
-Original Message-
From: Kubacki, Michael A
Sent: Friday, November 1, 2019 2:03 PM
To: devel@edk2.groups.io
Cc: Bu, Daocheng ; Desimone, Nathaniel L
; Gillispie, Thad ;
Kinney, Michael D ; Lu, Shifei A
; Oram, Isaac W ; Piwko, Macie
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Agyeman, Prince
Sent: Friday, November 1, 2019 12:51 PM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Chiu, Chasel
; Desimone, Nathaniel L
Subject: [edk2-devel] [edk2-platforms] [PATCH 3/5] BoardM
Reviewed-by: Nate DeSimone
-Original Message-
From: Agyeman, Prince
Sent: Friday, November 1, 2019 12:51 PM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Chiu, Chasel
; Desimone, Nathaniel L
Subject: [edk2-platforms] [PATCH 2/5] Platform/Intel: Move Sio Dxe Driver
Moved LegacySio
Reviewed-by: Nate DeSimone
-Original Message-
From: Agyeman, Prince
Sent: Friday, November 1, 2019 12:51 PM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Chiu, Chasel
; Desimone, Nathaniel L
Subject: [edk2-platforms] [PATCH 1/5] Platform/Intel: Add
gBoardModulePkgTokenSpaceGuid
This is actually a really generic library. Please move it to BoardModulePkg and
rename it to BdsPs2KeyboardLib.
This isn't really a "null library" please update your commit message. I'd
recommend "Add BdsPs2KeyboardLib".
-Original Message-
From: Agyeman, Prince
Sent: Friday, November
Please implement a patch 6 and 7 adding the same changes you made here for
GalagoPro3 to KabylakeRvp3 and WhiskeyLakeURvp. Both of those boards have PS/2
keyboard connectors and your change is valuable there too.
-Original Message-
From: devel@edk2.groups.io On Behalf Of Agyeman, Prince
On 10/31/19 13:42, Ashish Singhal wrote:
> Hello Laszlo,
>
> This is the cover letter. The patch was also submitted.
Can you please state the groups.io permalink for the posted patch,
and/or the Message-Id header for the posted patch?
> Please let me know if you are unable to find it and I'll re
For the patch series...
Reviewed-by: Isaac Oram
-Original Message-
From: Kubacki, Michael A
Sent: Friday, November 1, 2019 2:03 PM
To: devel@edk2.groups.io
Cc: Bu, Daocheng ; Desimone, Nathaniel L
; Gillispie, Thad ;
Kinney, Michael D ; Lu, Shifei A
; Oram, Isaac W ; Piwko, Maciej
;
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
The current set of maintainers do not plan to maintain this package
moving forward. Simply leaving the code as unmaintained is
undesirable for several reasons including presence of build issues,
functional issues, and lack of consistency with
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2325
The current set of maintainers do not plan to maintain this package
moving forward. Simply leaving the code as unmaintained is
undesirable for several reasons including presence of build issues,
functional issues, and lack of consistency with
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
The current set of maintainers do not plan to maintain this package
moving forward. Simply leaving the code as unmaintained is
undesirable for several reasons including presence of build issues,
functional issues, and lack of consistency with
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
The current set of maintainers do not plan to maintain this package
moving forward. Simply leaving the code as unmaintained is
undesirable for several reasons including presence of build issues,
functional issues, and lack of consistency with
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
The current set of maintainers do not plan to maintain this package
moving forward. Simply leaving the code as unmaintained is
undesirable for several reasons including presence of build issues,
functional issues, and lack of consistency with
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2325
Due to some Intel packages being removed, this change removes the
corresponding packages from Maintainers.txt.
Cc: Bu, Daocheng
Cc: Nate DeSimone
Cc: Gillispie, Thad
Cc: Michael D Kinney
Cc: Shifei A Lu
Cc: Isaac W Oram
Cc: Piwko, Maci
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
Removes the BoardMtOlympus build configuration from build.cfg as
the package will be removed in a future change.
Cc: Nate DeSimone
Cc: Michael D Kinney
Cc: Shifei A Lu
Cc: Isaac W Oram
Cc: Xiaohu Zhou
Signed-off-by: Michael Kubacki
---
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
Removes all references to PurleyOpenBoardPkg in Readme.md files in
edk2-platforms as the package will be removed in a future change.
Cc: Nate DeSimone
Cc: Michael D Kinney
Cc: Shifei A Lu
Cc: Isaac W Oram
Cc: Xiaohu Zhou
Signed-off-by:
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2322
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2323
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2324
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2325
RFC:https://edk2.groups.io/g/devel/message/49360
This change implements a
Added GalagoPro3 board ps2 keyboard library
that adds ps2 device path to ConIn and ConInDev
Uefi variables
Cc: Michael Kubacki
Cc: Chasel Chiu
Cc: Nate DeSimone
Signed-off-by: Prince Agyeman
---
.../GalagoPro3/Library/Ps2KbcLib/Ps2KbcLib.c | 202 ++
.../GalagoPro3/Library/Ps
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2228
This patch series enables BIOS Ps2 keyboard in GalagoPro3
What was done:
Patch 0001 adds PCDs to BoardModulePkg that will enable/disable,
describe Super I/O , Ps2 keyboard/mouse, uart1 and uart2 com ports
Patch 0002 moves the generic Supe
Added gBoardModulePkgTokenSpaceGuid to BoardModulePkg
Added ps2 keyboard and Uart enable PCDs.
Added PCDs to define IO ports for Uart1 and Uart2
Enabled KabylakeOpenBoardPkg and WhiskeylakeOpenBoardPkg
boards to use the ps2 keyboard enable PCD.
Cc: Michael Kubacki
Cc: Chasel Chiu
Cc: Nate DeS
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2228
Added Ps2Keyboard DXE driver.
Added Sio dxe driver to publish the SIO protocol
needed by the Ps2Keyboard DXE driver.
Included a Ps2 null Library to BdsDxe driver
to add the Ps2 device path to ConIn and ConInDev
variables
Configured the Su
Moved LegacySioDxe from SimicsOpenBoardPkg
to BoardModulePkg as LegacySioDxe is generic
and can be shared between open board packages
Removed unused libraries and functions in LegacySioDxe
Pointed BoardX58Ich10 to the current LegacySioDxe
location
Cc: Michael Kubacki
Cc: Chasel Chiu
Cc: Nate D
Included PCDs to the Sio dxe driver,
to allow the enable/disable of Ps2 keyboard/mouse,
UART1 and UART2 com ports.
Renamed the Sio driver's base name
Cc: Michael Kubacki
Cc: Chasel Chiu
Cc: Nate DeSimone
Signed-off-by: Prince Agyeman
---
.../LegacySioDxe/LegacySioDxe.inf | 15 ++
With Michael's comments addressed...
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, October 31, 2019 3:28 AM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Desimone, Nathaniel L
; Gao, Liming
Subject: [edk2-devel]
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, October 31, 2019 3:28 AM
To: devel@edk2.groups.io
Cc: Wei, David Y ; Agyeman, Prince
Subject: [edk2-devel] [edk2-platforms: PATCH v3 6/6]
SimicsOpenBoardPkg/BoardX58Ich10
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, October 31, 2019 3:28 AM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Desimone, Nathaniel L
Subject: [edk2-devel] [edk2-platforms: PATCH v3 4/6] WhiskeylakeOpenBoardP
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, October 31, 2019 3:28 AM
To: devel@edk2.groups.io
Cc: Lu, Shifei A ; Zhou, Bowen ;
Oram, Isaac W
Subject: [edk2-devel] [edk2-platforms: PATCH v3 5/6]
PurleyOpenBoardPkg/B
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, October 31, 2019 3:28 AM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Desimone, Nathaniel L
; Gao, Liming ; Jeremy
Soller
Subject: [edk2-devel] [edk2-platforms: PATC
Reviewed-by: Nate DeSimone
-Original Message-
From: devel@edk2.groups.io On Behalf Of Chiu, Chasel
Sent: Thursday, October 31, 2019 3:28 AM
To: devel@edk2.groups.io
Cc: Kubacki, Michael A ; Desimone, Nathaniel L
; Gao, Liming
Subject: [edk2-devel] [edk2-platforms: PATCH v3 2/6] MinPlat
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2220
This change reduces SMIs for GetVariable () by maintaining a
UEFI variable cache in Runtime DXE in addition to the pre-
existing cache in SMRAM. When the Runtime Service GetVariable()
is invoked, a Runtime DXE cache is used instead of trigger
This change enables the variable runtime cache by default by setting
gEfiMdeModulePkgTokenSpaceGuid.PcdEnableVariableRuntimeCache in
MdeModulePkg.dec to TRUE.
Cc: Dandan Bi
Cc: Ard Biesheuvel
Cc: Eric Dong
Cc: Laszlo Ersek
Cc: Liming Gao
Cc: Michael D Kinney
Cc: Ray Ni
Cc: Jian J Wang
Cc:
The majority of logic related to GetNextVariableName () is currently
implemented in VariableServiceGetNextVariableInternal (). The list
of variable stores to search for the given variable name and variable
GUID is defined in the function body. This change adds a new parameter
so that the caller mus
Sets gEfiMdeModulePkgTokenSpaceGuid.PcdEnableVariableRuntimeCache
to FALSE in OvmfPkgIa32.dsc, OvmfPkgIa32X64.dsc, and OvmfPkgX64.dsc
so that when SMM_REQUIRE is TRUE, the SMM variable driver will not
use the runtime variable cache.
This is done for OvmfPkg because it currently depends upon a SMM
UpdateVariableInfo () currently accepts parameters regarding updates
to be made to a global variable of type VARIABLE_INFO_ENTRY. This
change passes the structure by pointer to UpdateVariableInfo ()
so structures other than the fixed global variable can be updated.
Cc: Dandan Bi
Cc: Ard Biesheuve
The file VariableParsing.c provides generic functionality related
to parsing variable related structures and information. In order to
calculate offsets for certain operations, the functions must know if
authenticated variables are enabled as this increases the size of
variable headers.
This change
This change moves the following functions into a dedicated file
so they may be used in other variable files as needed. These are
commonly needed for basic variable data structure parsing
operations. The functions are grouped together in VariableParsing.c
to support cohesiveness for these operations
https://bugzilla.tianocore.org/show_bug.cgi?id=2220
This change implements the Runtime Service GetNextVariableName()
using the runtime cache in VariableSmmRuntimeDxe. Runtime Service
calls to GetNextVariableName() will no longer trigger a SW SMI
when gEfiMdeModulePkgTokenSpaceGuid.PcdEnableVariabl
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2220
V7 Changes:
[PATCH V6 07/10] MdeModulePkg/Variable: Add RT GetVariable() cache support
* Remove VariableRuntimeCache.c and VariableRuntimeCache.h from
VariableSmmRuntimeDxe.inf since they are not needed to build the module.
V6 Changes:
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2220
The current VariableInfo application only checks for variable
statistics from SMM if the variable information entries are
not present in the UEFI System Configuration table as published
by the DXE UEFI variable driver (VariableRuntimeDxe).
T
This change adds a dedicated file for variable operations specific
to non-volatile variables. This decreases the overall length of the
relatively large Variable.c file.
Cc: Dandan Bi
Cc: Ard Biesheuvel
Cc: Eric Dong
Cc: Laszlo Ersek
Cc: Liming Gao
Cc: Michael D Kinney
Cc: Ray Ni
Cc: Jian J
Tested-by: Sudipto Paul https://community.arm.com/developer/tools-software/oss-platforms/w/docs/440/neoverse-n1-sdp
-Sudipto
-Original Message-
From: Sami Mujawar
Sent: 21 October 2019 17:06
To: Krzysztof Koch ; devel@edk2.groups.io
Cc: jaben.car...@intel.com; ray...@intel.com; zhichao.
Hi Prabhakar,
On Fri, Nov 1, 2019 at 10:22 AM Prabhakar Kushwaha
wrote:
>
> Hi Bhupesh,
>
> On Fri, Nov 1, 2019 at 1:59 AM Bhupesh Sharma wrote:
> >
> > Hi Prabhakar,
> >
> > On Wed, Oct 30, 2019 at 1:47 PM Prabhakar Kushwaha
> > wrote:
> > >
> > > On Wed, Oct 30, 2019 at 1:14 PM Ard Biesheuve
On Fri, Nov 01, 2019 at 10:17:30AM +0530, Prabhakar Kushwaha wrote:
> On Fri, Nov 1, 2019 at 2:04 AM dann frazier
> wrote:
> >
> > On Thu, Oct 31, 2019 at 12:55:10PM -0600, dann frazier wrote:
> > > On Thu, Oct 31, 2019 at 6:16 AM Leif Lindholm
> > > wrote:
> > > >
> > > > On Thu, Oct 31, 2019
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The code changes are made to enable the configuration of new PCI feature
Max_Payload_Size (MPS), which defines the data packet size for the PCI
transactions, as per the PCI Base Specification 4 Revision 1.
The code changes are made to calibr
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2313
The code changes are made; as per the PCI Base Specification 4 Revision
1; to enable the configuration of new PCI feature Completion Timeout
(CTO), which enables the PCI function to wait on programmed duration for
its transactions before time
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The code changes are made to enable the configuration of new PCI feature
Max_Read_Req_Size (MRRS), which defines the memory read request size for
the PCI transactions, as per the PCI Base Specification 4 Revision 1.
The code changes are made
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2313
The code changes are made; as per the PCI Base Specification 4 Revision
1; to enable the configuration of new PCI feature No-Snoop (NS), which
enables the PCI function to initiate requests if it does not require har-
dware enforced cache-cohe
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2313
The code changes are made to enable the configuration of new PCI feature
Relax Ordering (OR), which enables the PCI function to initiate requests
if it does not require strong write ordering for its transactions; as per
the PCI Base Specifica
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The following legacy PCI Platform Protocol usage is reorganized in the
separate source files:-
(1) PlatformPrepController
(2) PlatformNotify
(3) GetPlatformPolicy
(4) GetPciRom
This code changes are made to support the new PCI Platform Proto
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The code changes are made to integrate the setup infrastructure for the
PCI feature enumeration, in the last phase of the PCI Bus driver, after
its registration and its option ROM loading phase is complete.
Signed-off-by: Ashraf Javeed
Cc:
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
Definition of bit masks for the new PCD for the following new PCI feature
set:-
1. Maximum Payload Size (MPS)
2. Maximum Read Request Size (MRRS)
3. Completion Timeout (CTO)
4. Relax Order (RO) Enable
5. No Snoop (NS)
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The code changes are made to setup the following internal sub-phases for
enumerating the PCI features in the late phase of the PCI Bus driver.
(1) PciFeatureRootBridgeScan - initial phase in configuring the other PCI
features to record th
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The separation of the PCI device registration phase includes only the
installation of the PCI IO Protocol on the PCI node to acquire the EFI
handles, and loading of its applicable PCI Option ROM.
The separation of the PCI device start phase
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The code changes are made to support the new PCI Platform Protocol, as
well as the legacy PCI Platform Protocol interfaces.
The code change is made to consume the new interface to acquire the PCI
device-specific platform policy.
This code c
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2194
The code changes are made to record the PCI device's PCI-Express
Capability Structure register set during early PCI enumeration phase.
This data shall be used during PCI feature enumeration phase.
Signed-off-by: Ashraf Javeed
Cc: Jian J Wan
The EDK2 Kernel PciBusDxe driver is enhanced to enable the configuration
of PCI features like
(1) Max_Payload_Size
(2) Max_Read_Req_Size
(3) Relax Ordering
(4) No-Snoop
(5) Completion Timeout
Max_Payload_Size:- The PCI Device Control register provides this feature
register field which controls the
On Fri, Nov 1, 2019 at 6:57 AM Jon Nettleton via Groups.Io
wrote:
>
> On Thu, Oct 31, 2019 at 5:44 PM Andrew Fish via Groups.Io
> wrote:
> >
> > Jon,
> >
> > Its a little confusing but gBS->GetMemoryMap () only returns information
> > about DRAM and any address that requires a kernel virtual add
Hi Laszlo,
I did simple ovmf boot tests (shell, linux, windows) and all passed. Let me
know if you have
any comments or want to do more tests against v4 before check in.
Based on my review and tests,
Reviewed-by: Jian J Wang
Regards,
Jian
> -Original Message-
> From: Zhang, Shengle
71 matches
Mail list logo