Reviewed-by: Nate DeSimone
On 1/22/20, 1:55 PM, "devel@edk2.groups.io on behalf of Bjorge, Erik C"
wrote:
On Ubuntu the script final_copy.py was not able to run without
prepending ./ to the script name.
Signed-off-by: Erik Bjorge
Cc: Nate DeSimone
Cc: Ashley DeSimon
Actually I have no permission to create the pull request. I would help to send
the patch to Ray and he would create the pull request.
Thanks,
Zhichao
> -Original Message-
> From: Ashish Singhal [mailto:ashishsin...@nvidia.com]
> Sent: Tuesday, January 28, 2020 1:29 AM
> To: Gao, Zhichao
Hi,
I have tried with the commit. It pass the check.
Thanks,
Zhichao
> -Original Message-
> From: devel@edk2.groups.io [mailto:devel@edk2.groups.io] On Behalf Of
> Philippe Mathieu-Daudé
> Sent: Tuesday, January 21, 2020 7:24 PM
> To: Gao, Zhichao ; devel@edk2.groups.io
> Cc: Ni, Ray
>
Removes Michael Kubacki as an Intel maintainer.
Cc: Prince Agyeman
Cc: Sai Chaganty
Cc: Chasel Chiu
Cc: Nate DeSimone
Cc: Liming Gao
Signed-off-by: Michael Kubacki
---
Maintainers.txt | 6 --
1 file changed, 6 deletions(-)
diff --git a/Maintainers.txt b/Maintainers.txt
index e97ea5b343
Hi Laszlo,
I have reviewed this patch series. All the patches look
good. The detailed description of each change made it
easy to review.
Series Reviewed-by: Michael D Kinney
I have one question about that is not directly related to
this logic change.
I see this logic that checks for invalid
Hi Derik,
The Linux service clock_gettime() returns a structure
with seconds and nanoseconds and you are retuning the
number of nanoseconds from QueryPerformanceCounter().
Doesn't that imply that the rate of the performance
counter is 1 ns? Does this allow you to remove the
calibration code? Dep
On 01/30/20 18:10, Kinney, Michael D wrote:
> 3) EDK II community defines a small number of profiles
>for the PEI, DXE, SMM versions of these modules and
>publishes released binaries for each profile. Platforms
>integrate the published binary modules. This may
>provide more serv
Hi Laszlo,
Great feedback. The reason I chose to use a Structured PCD
is to provide maximum flexibility. I agree we should review
the behavior in RELEASE builds when a call is made to a
service that is disabled. Perhaps REPORT_STATUS_CODE would
be a good option for a platform provide its own h
Reviewed-by: Sami Mujawar
Regards,
Sami Mujawar
-Original Message-
From: Krzysztof Koch
Sent: 30 January 2020 16:20
To: devel@edk2.groups.io
Cc: ray...@intel.com; zhichao@intel.com; Matteo Carlini
; Sami Mujawar ; nd
Subject: [PATCH v1 1/1] ShellPkg: acpiview: Validate ACPI tabl
Check if the ACPI table length, as reported in the ACPI table header, is
big enough to fit at least the header itself.
If not, report an error to the user and stop parsing the table in order
to prevent buffer overruns.
Signed-off-by: Krzysztof Koch
---
Changes can be seet at:
https://github.co
I've understood, thanks a lot!
Sergey
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#53584): https://edk2.groups.io/g/devel/message/53584
Mute This Topic: https://groups.io/mt/70267536/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubsc
On 30/01/2020 12:44, Anthony PERARD wrote:
> On Thu, Jan 30, 2020 at 10:12:51AM +0100, Roger Pau Monné wrote:
>> On Wed, Jan 29, 2020 at 12:12:34PM +, Anthony PERARD wrote:
>>> + Parameters.domid = DOMID_SELF;
>>> + Parameters.gpfn = (UINTN) PagePtr >> EFI_PAGE_SHIFT;
>>> + ReturnCode = XenH
On 01/30/20 13:57, serges...@yandex.ru wrote:
> Thank you, may I just create a new GUID (sysHostGuid),
Right, just get your own GUID with "uuidgen".
> then call BuildGuidDataHob passing in it the sysHostGuid
Yes.
> and SYSHOST structure?
Well, not exactly. It depends.
If your SYSHOST structur
On 01/30/20 08:00, Michael D Kinney wrote:
> https://bugzilla.tianocore.org/show_bug.cgi?id=2420
>
> Based on the following package with changes to merge into
> CryptoPkg.
>
> https://github.com/microsoft/mu_plus/tree/dev/201908/SharedCryptoPkg
>
> Add the CryptoPei, CryptoDxe, and CryptoSmm mod
Thank you, may I just create a new GUID (sysHostGuid), then call
BuildGuidDataHob passing in it the sysHostGuid and SYSHOST structure? Will I be
able to find the structure in the DXE phase looking up by the sysHostGuid?
Thank you,
Sergey
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all
On Thu, Jan 30, 2020 at 10:12:51AM +0100, Roger Pau Monné wrote:
> On Wed, Jan 29, 2020 at 12:12:34PM +, Anthony PERARD wrote:
> > + Parameters.domid = DOMID_SELF;
> > + Parameters.gpfn = (UINTN) PagePtr >> EFI_PAGE_SHIFT;
> > + ReturnCode = XenHypercallMemoryOp (XENMEM_remove_from_physmap,
Reviewed-by: Sami Mujawar
Regards,
Sami Mujawar
-Original Message-
From: Krzysztof Koch
Sent: 30 January 2020 11:43
To: devel@edk2.groups.io
Cc: ray...@intel.com; zhichao@intel.com; Matteo Carlini
; Sami Mujawar ; nd
Subject: [PATCH v1 1/1] ShellPkg: acpiview: Remove duplicate A
Remove duplicate Generic Address Structure (GAS) and ACPI table header
size definitions. Replace any dependencies with the definitions from
Acpi63.h.
AcpiParser.h redefines the size of Generic Address Structure (GAS) and
the ACPI table header. The motivation is to 'minimise any reference to
the ED
Reviewed-by: Sami Mujawar
Regards,
Sami Mujawar
-Original Message-
From: Krzysztof Koch
Sent: 30 January 2020 11:14
To: devel@edk2.groups.io
Cc: michael.d.kin...@intel.com; liming@intel.com; Matteo Carlini
; Sami Mujawar ; nd
Subject: [PATCH v1 1/1] MdePkg: Add PCCT table signat
On 01/30/20 12:03, serges...@yandex.ru wrote:
> I need to use some code from PEIM driver in DXE phase. This code is
> using the SYSHOST structure and I think it not easy to restore it
> from HOBs becuase there are a lot of information. So I thought it
> easy to save the structure completely in the
Acpi63.h contains definitions describing the Platform Communications
Channel Table's (PCCT) structure. However, a macro for the table's
signature is missing.
This patch adds the definition of Platform Communications Channel
Table's signature.
References:
- ACPI 6.3, January 2019, Section 14.1
Si
I need to use some code from PEIM driver in DXE phase. This code is using the
SYSHOST structure and I think it not easy to restore it from HOBs becuase there
are a lot of information. So I thought it easy to save the structure completely
in the memory.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: Y
On Wed, Jan 29, 2020 at 05:14:35PM +0100, Laszlo Ersek wrote:
> On 01/29/20 13:12, Anthony PERARD wrote:
> Assuming the OVMF platforms continue to build at this stage into the
> series, and provided that (1) and (2) are fixed:
I'll fix (1) and (2).
I've build tests both OvmfXen and OvmfPkgX64, an
Hey,
Are HOBs ("Hand-Off Blocks") what you are looking for? They are passed
from PEI to DXE (see PI 1.6, 9.3 "Passing the Hand-Off Block (HOB) List").
Regards,
Marvin
Am 30.01.2020 um 10:51 schrieb serges...@yandex.ru:
> Is it possible to save SYSHOST data structure at PEI phase in memory at
>
Is it possible to save SYSHOST data structure at PEI phase in memory at some
default location to use it at DXE phase by some UEFI application?
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#53570): https://edk2.groups.io/g/devel/message/
Reviewed-by: Sai Chaganty
-Original Message-
From: Kubacki, Michael A
Sent: Tuesday, January 28, 2020 1:17 AM
To: devel@edk2.groups.io
Cc: Chaganty, Rangasai V ; Desimone, Nathaniel L
Subject: [tianocore-docs][PATCH V1 1/1] edk2-MinimumPlatformSpecification:
Advanced Feature section
On Thu, 30 Jan 2020 at 06:08, Gaurav Jain wrote:
>
> ASSERT in SetTime_Conf and SetWakeupTime_Conf Consistency Test.
> SCT Test expect return as Invalid Parameter.
> So removed ASSERT().
>
This is not all this patch does.
> Signed-off-by: Gaurav Jain
> ---
> EmbeddedPkg/RealTimeClockRuntimeDxe
On Wed, Jan 29, 2020 at 12:12:34PM +, Anthony PERARD wrote:
> Calculate the frequency of the APIC timer that Xen provides.
>
> Even though the frequency is currently hard-coded, it isn't part of
> the public ABI that Xen provides and thus may change at any time. OVMF
> needs to determine the f
ASSERT in CopyMem_Conf, PollMem_Conf, SetBarAttributes_Conf
Conformance Test.
SCT Test expect return as Invalid Parameter.
So removed ASSERT().
Signed-off-by: Gaurav Jain
---
.../NonDiscoverablePciDeviceIo.c | 20 ---
1 file changed, 17 insertions(+), 3 deletions(-)
29 matches
Mail list logo
