On August 17, 2021 4:38 PM, Gerd Hoffmann wrote:
>
> Hi,
>
> > In the I/O functions of above files, if IsTdxGuest() returns TRUE,
> > then Td I/O routine is called, otherwise the legacy I/O routine is called.
> > Td I/O routines are declared in IoLibTdx.h and implemented in
> > IoLibInternalTdx
[PATCH] Package: CryptoPkg/BaseCryptLib: fix incorrect param order
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3550
Function ConvertAsn1TimeToEfiTime initializes timestamp to zeroes with SetMem,
but the actual parameters are out of order.
The result is the SetMem operation has no effect.
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Add BsdSocketLib and EfiSocketLib to [LibraryClasses] section
of Python-2.7.10.inf. Even if sockets are disabled in Python
config, with the /WHOLEARCHIVE linker option, there are still
references to the socket related functions. Add these
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Update Python configuration to enable the HAVE_COPYSIGN define
to prevent multiple defined symbols link failure for copysign().
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
AppPkg/Applications/Python/Ia32/pyconfig.h
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Removing the ldtoa.c file from the INF removes the ldtoa()
function from this library. The ldtoa() function is not used
and when /WHOLEARCHIVE linker flag is enabled this unused
function makes references to other functions that are not
impl
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
SocketDxe generates build failures with VS2017 and VS2019 due to
use of weak references and /WHOLEARCHIVE. Disable SocketDxe until
weak references can be removed.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
StdLib/StdLib.inc |
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Disable warning 4706 to allow assignments in conditional
statements.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
StdLib/BsdSocketLib/BsdSocketLib.inf | 3 +++
1 file changed, 3 insertions(+)
diff --git a/StdLib/BsdSocketLib/Bsd
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Add LibNetUtil to BsdSocketLib INF file that fails with missing
symbols for inet_ntoa(), inet_aton(), inet_ntop(), inet_network()
when BsdSocketLib is linked to another component.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
StdL
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Add /Wv:11 to VS2017/VS2017 CC_FLAGS to match the warning level
used by VS2015.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
StdLib/StdLib.inc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/StdLib/StdLib.inc b/StdLib/StdLi
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Fix missing RegisterFilterLib build failure by adding MdeLibs.dsc.inc
after the [Defines] section of AppPkg.dsc.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
AppPkg/AppPkg.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a
Update AppPkg and StdLib* to build using latest edk2/master and
support VS2017 and VS2019.
Most of the changes are related to warning levels, missing symbols and
duplicate symbols that were uncovered by the use of the /WHOLEARCHIVE
flag.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
Michae
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3549
Fix missing RegisterFilterLib build failure by adding MdeLibs.dsc.inc
after the [Defines] section of StdLib.dsc.
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
StdLib/StdLib.dsc | 2 ++
1 file changed, 2 insertions(+)
diff --git a
pushed to edk2-platforms/master
109380a7a8..c2ab6bab4b
Mike
> -Original Message-
> From: Kinney, Michael D
> Sent: Tuesday, August 17, 2021 10:16 PM
> To: devel@edk2.groups.io; pedro.falc...@gmail.com; Kinney, Michael D
>
> Cc: Leif Lindholm ; Bret Barkelew
>
> Subject: RE: [edk2-d
Series Reviewed-by: Michael D Kinney
Mike
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Pedro Falcato
> Sent: Monday, August 16, 2021 11:07 AM
> To: devel@edk2.groups.io
> Cc: Pedro Falcato ; Leif Lindholm
> ; Kinney, Michael D
> ; Bret Barkelew
> Subject: [edk2-devel
Yuwei:
This change may impact the third part tool. Customer may have their tool
to parse BIOS image. They may not support FileSystemGuid3.
So, I suggest to defer this patch to next stable tag so that there are
enough time to collect the feedback.
For this patch, I add my comments below.
Hi Caden/Nate,
This patch aims to make the terminal output character set to be expected as
https://github.com/tianocore/tianocore.github.io/wiki/Tasks-Terminal-driver-improvements.
It is better to add this info to the commit message. Because of the commit
message length limitation, I would sugge
Reviewed-by: Jiewen Yao
Thanks Liming. Yes. I agree to merge it for stable tag.
Will you help to do that?
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of gaoliming
> Sent: Wednesday, August 18, 2021 9:18 AM
> To: 'Stewart, Chris (SW Architect)' ;
> devel@edk2.groups.io
>
Reviewed-by: Liming Gao
This is a clear bug fix. I suggest to merge it for this stable tag 202108.
Thanks
Liming
> -邮件原件-
> 发件人: Stewart, Chris (SW Architect)
> 发送时间: 2021年8月17日 20:48
> 收件人: devel@edk2.groups.io
> 抄送: jiewen@intel.com; jian.j.w...@intel.com; xiaoyux...@intel.com;
>
TianoCore Community Meeting
August 12, 2021
EVENTS:
Google Summer of Code:
* 7 projects were selected, one of the projects got dropped before
mid-term. All projects are listed here:
https://summerofcode.withgoogle.com/organizations/6376892141142016/
* All 6 are doing well. Code rev
Pushed: https://github.com/tianocore/edk2-platforms/commit/109380
-Original Message-
From: devel@edk2.groups.io On Behalf Of Benjamin Doron
Sent: Wednesday, August 4, 2021 1:43 PM
To: devel@edk2.groups.io
Cc: Dong, Eric ; Liming Gao
Subject: [edk2-devel] [edk2-platforms][PATCH v1 1/2] Ac
Hi Benjamin,
I’ll just push the AcpiDebugFeaturePkg change.
Thanks,
Nate
From: devel@edk2.groups.io On Behalf Of Benjamin Doron
Sent: Tuesday, August 17, 2021 11:04 AM
To: Desimone, Nathaniel L
Cc: devel@edk2.groups.io; Oram, Isaac W ; Liming Gao
Subject: Re: [edk2-devel] [edk2-platforms][PA
Reviewed-by: Rebecca Cran
On 8/17/21 9:16 AM, Michael D Kinney wrote:
Cc: Rebecca Cran
Cc: Daryl McDaniel
Cc: Jaben Carsey
Signed-off-by: Michael D Kinney
---
Maintainers.txt | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Maintainers.txt b/Maintainers.txt
ind
It appears that this fix was already committed as part of commit 2333ea4.
How do I drop this patch?
On Tue., Aug. 17, 2021, 1:59 p.m. Desimone, Nathaniel L, <
nathaniel.l.desim...@intel.com> wrote:
> Reviewed-by: Nate DeSimone
>
> > -Original Message-
> > From: Benjamin Doron
> > Sent:
Reviewed-by: Nate DeSimone
> -Original Message-
> From: Benjamin Doron
> Sent: Wednesday, August 4, 2021 1:43 PM
> To: devel@edk2.groups.io
> Cc: Oram, Isaac W ; Desimone, Nathaniel L
> ; Liming Gao
>
> Subject: [edk2-platforms][PATCH v1 2/2] IpmiFeaturePkg: BmcAcpi is
> DRIVER_ACPITABL
Reviewed-by: Nate DeSimone
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of
> Benjamin Doron
> Sent: Wednesday, August 4, 2021 1:43 PM
> To: devel@edk2.groups.io
> Cc: Dong, Eric ; Liming Gao
>
> Subject: [edk2-devel] [edk2-platforms][PATCH v1 1/2]
> AcpiDebugFeaturePkg: M
Pushed: https://github.com/tianocore/edk2-platforms/commit/d410a0a
> -Original Message-
> From: devel@edk2.groups.io On Behalf Of Nate
> DeSimone
> Sent: Wednesday, August 11, 2021 6:22 PM
> To: devel@edk2.groups.io
> Cc: Desimone, Nathaniel L ; Michael
> Kubacki ; Chiu, Chasel
> ; Chagan
Reviewed-by: Jaben Carsey
-Original Message-
From: Kinney, Michael D
Sent: Tuesday, August 17, 2021 8:17 AM
To: devel@edk2.groups.io
Cc: Rebecca Cran ; Daryl McDaniel
; Carsey, Jaben
Subject: [edk2-libc Patch] Maintainers.txt: Update maintainers for AppPkg and
StdLib*
Cc: Rebecca Cr
Hi Marvin,
It would be beneficial in the sense that it could reduce the amount of
work manually needed elsewhere.
PeiCore can migrate pointers in global databases it knows about such as
the PPI list that point to global data areas it has knowledge of such as
Temporary RAM. Prior to this chan
On Sat, 7 Aug 2021 at 21:37, Marcin Wojtas wrote:
>
> Hi,
>
> The second version comes with a minor improvement, which
> is a result of the TF-A support update and changes around the
> default IO windows configuration.
>
> The patches are also available on public branches:
> https://github.com/sem
On Sat, 7 Aug 2021 at 21:37, Marcin Wojtas wrote:
>
> This patch introduces the top device tree for the CN913x CEx7
> Evaluation Board, based on the sources which are common for the
> Cn913x SoCs. Also an .inf file is added to allow its compilation.
>
> Signed-off-by: Marcin Wojtas
Pushed as de
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3361
Cc: Rebecca Cran
Signed-off-by: Michael D Kinney
---
StdLib/PosixLib/GetPass/GetPass.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/StdLib/PosixLib/GetPass/GetPass.c
b/StdLib/PosixLib/GetPass/GetPass.c
index 4651
On 8/17/21 10:12 AM, Ard Biesheuvel wrote:
On Mon, 16 Aug 2021 at 22:27, Jeremy Linton wrote:
One of the many issues with the PCIe on this platform is
its inbound DMA is either constrained to the lower 3G, or
on later SoC's a translation can be used. That translation
was problematic with some
Cc: Rebecca Cran
Cc: Daryl McDaniel
Cc: Jaben Carsey
Signed-off-by: Michael D Kinney
---
Maintainers.txt | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Maintainers.txt b/Maintainers.txt
index 4e05e8a..d0419be 100644
--- a/Maintainers.txt
+++ b/Maintainers.txt
@@ -6
On Mon, 16 Aug 2021 at 22:27, Jeremy Linton wrote:
>
> One of the many issues with the PCIe on this platform is
> its inbound DMA is either constrained to the lower 3G, or
> on later SoC's a translation can be used. That translation
> was problematic with some of the OS's expected to boot
> on thi
On Tue, 10 Aug 2021 at 05:36, Jeremy Linton wrote:
>
> A new Arm standard DEN0115A specifies how
> platforms that don't have standard ECAM can
> use the firmware to handle config read/write
> operations. This is mostly implemented in TFA
> but UEFI needs to assure that there is a
> description of
Hi Pedro,
Your suggestion is interesting. We will try to enhance the clang Linux build
and remove the libgcc dependency in the future.
This patch change is for a project of Google Summer of Code 2021, which is to
build BaseTools with clang + gnu make in windows. We hope to focus the windows
par
On Tue, 17 Aug 2021 at 09:26, Grzegorz Bernacki wrote:
>
> Commit 2f0188b56ef4 ("Revert "Platform/RaspberryPi: Setup option for...")
> mistakenly introduced to files which are residues from a
> conflict resolution. Fix that.
>
> Signed-off-by: Grzegorz Bernacki
Pushed as b291607503dc..f24aa7708d
On Mon, 16 Aug 2021 at 09:10, Ard Biesheuvel wrote:
>
> On Fri, 13 Aug 2021 at 07:05, Jeremy Linton wrote:
> >
> > Map->Operation is used to select whether a DMA region that
> > is being bounced has the source buffer copied to it. Except
> > Map->Operation isn't yet set, so the behavior is somewh
Bugzilla: 3568 (https://bugzilla.tianocore.org/show_bug.cgi?id=3568)
Update the FADT generator to use Acpi64.h. As the FADT revision tracks
the ACPI revision this updates the minor revision value generated.
Mantis ID for change: https://mantis.uefi.org/mantis/view.php?id=2105
Signed-off-by: Chri
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3429
Both the TDX and SEV support needs to reserve a page in MEMFD as a work
area. The page will contain meta data specific to the guest type.
Currently, the SEV-ES support reserves a page in MEMFD
(PcdSevEsWorkArea) for the work area. This page c
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3429
While build the initial page table, the SetCr3ForPageTables64 checks
whether SEV-ES is enabled. If so, clear the page encryption mask from the
GHCB page. Move the logic to clear the page encryption mask in the
AmdSev.asm.
Cc: James Bottomley
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3429
Update the SEV support to switch to using the newer work area format.
Cc: James Bottomley
Cc: Min Xu
Cc: Jiewen Yao
Cc: Tom Lendacky
Cc: Jordan Justen
Cc: Ard Biesheuvel
Cc: Erdem Aktas
Signed-off-by: Brijesh Singh
---
OvmfPkg/Reset
Based on the discussion on the mailing list, we agreed that instead
of wasting extra page in the MEMFD, we can reuse the SevEsWorkArea
buffer for the TDX. To avoid any confusion, lets introduce a OvmfWorkArea
that will contains 32 bytes of header followed by the actual workarea.
While at it, move
This adds more helper functions that assist in calculating the checksum,
locating an ACPI table by signature, and updating an AML integer object.
Cc: Leif Lindholm
Cc: Ard Biesheuvel
Cc: Abner Chang
Cc: Daniel Schaefer
Signed-off-by: Nhi Pham
---
EmbeddedPkg/Library/AcpiLib/AcpiLib.inf | 2
Hi Gerd,
On 8/17/21 11:23 AM, Gerd Hoffmann wrote:
> Add support for virtio 1.0 to the mmio transport. virtio 0.9.5 uses
> page size, page frame number and a fixed layout for the ring. virtio
> 1.0 uses the physical addresses for base address, used bits and
> available bits instead.
>
> The rin
On 8/17/21 11:23 AM, Gerd Hoffmann wrote:
> Use QueueNumMax as QueueNum default for drivers which do not
> explicitly call VIRTIO_DEVICE_PROTOCOL->SetQueueSize().
>
> Signed-off-by: Gerd Hoffmann
> ---
> .../Library/VirtioMmioDeviceLib/VirtioMmioDevice.h | 1 +
> .../VirtioMmioDeviceLib/Virti
virtio 1.0 has 64 feature bits instead of 32.
Signed-off-by: Gerd Hoffmann
---
.../VirtioMmioDeviceFunctions.c | 28 +++
1 file changed, 23 insertions(+), 5 deletions(-)
diff --git a/OvmfPkg/Library/VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c
b/OvmfPkg/Library
Add support for virtio 1.0 to the mmio transport. virtio 0.9.5 uses
page size, page frame number and a fixed layout for the ring. virtio
1.0 uses the physical addresses for base address, used bits and
available bits instead.
The ring layout is not changed, so a 0.9.5 compatible layout is used in
Use QueueNumMax as QueueNum default for drivers which do not
explicitly call VIRTIO_DEVICE_PROTOCOL->SetQueueSize().
Signed-off-by: Gerd Hoffmann
---
.../Library/VirtioMmioDeviceLib/VirtioMmioDevice.h | 1 +
.../VirtioMmioDeviceLib/VirtioMmioDeviceFunctions.c | 12 +++-
2 files chang
Add defines for the config space offsets for virtio 1.0 mmio transport.
Signed-off-by: Gerd Hoffmann
Reviewed-by: Philippe Mathieu-Daude
---
OvmfPkg/Include/IndustryStandard/Virtio10.h | 12
1 file changed, 12 insertions(+)
diff --git a/OvmfPkg/Include/IndustryStandard/Virtio10.h
This little series adds virtio 1.0 support for the virtio-mmio
transport. For the mmio transport the difference between 0.9.5 and 1.0
is rather small (when compared to the pci transport), it is just a bunch
of new registers for the changed virtio queue initialization. So the
patch series is small
Hi,
> > Also: Why there are separate TdxProbeLib + TdxLib libs?
> This is because TdxLib wrap the operations of TdCall and TdVmcall. While
> TdxProbeLib
> is a library to probe (call CPUID(0x21)) if it is td guest or not. I am open
> to merge these 2
> libs into one if the community think it i
On August 17, 2021 4:21 PM, Gerd Hoffmann wrote:
> On Tue, Aug 17, 2021 at 12:14:44AM +, Min Xu wrote:
> > On Monday, August 16, 2021 5:43 PM, Gerd Hoffmann wrote:
> > > > +++ b/MdePkg/Library/TdxProbeLib/X64/TdProbe.nasm
> > >
> > > Any specific reason why you code up your own instead of using
Hi,
> In the I/O functions of above files, if IsTdxGuest() returns TRUE, then
> Td I/O routine is called, otherwise the legacy I/O routine is called.
> Td I/O routines are declared in IoLibTdx.h and implemented in
> IoLibInternalTdx.c.
Sorry, I'm a bit late to the party, but what is the overall
Reviewed-by: Chasel Chiu
> -Original Message-
> From: Loo, Tung Lun
> Sent: Tuesday, August 17, 2021 3:43 PM
> To: devel@edk2.groups.io
> Cc: Loo, Tung Lun ; Ma, Maurice
> ; Desimone, Nathaniel L
> ; Zeng, Star ; Chiu,
> Chasel
> Subject: [PATCH] IntelFsp2Pkg: Support Config File and
On Tue, Aug 17, 2021 at 12:14:44AM +, Min Xu wrote:
> On Monday, August 16, 2021 5:43 PM, Gerd Hoffmann wrote:
> > > +++ b/MdePkg/Library/TdxProbeLib/X64/TdProbe.nasm
> >
> > Any specific reason why you code up your own instead of using the existing
> > cpuid functions in BaseLib ?
> Actually
From: "Loo, Tung Lun"
BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3567
This patch is to enable config editor to have a new feature that
can spell out the delta between the default configuration files'
data, such as YAML and BSF, against the data stored in the binary.
This can help users u
Commit 2f0188b56ef4 ("Revert "Platform/RaspberryPi: Setup option for...")
mistakenly introduced to files which are residues from a
conflict resolution. Fix that.
Signed-off-by: Grzegorz Bernacki
---
Platform/RaspberryPi/Library/DualSerialPortLib/DualSerialPortLib.c.orig |
613
58 matches
Mail list logo