Re: [edk2-devel] [PATCH v3 13/39] UefiCpuPkg: Add CpuMmuLib.h to UefiCpuPkg

2023-12-03 Thread Chao Li
Hi Ray, For this patch, I checked again and here are my opinions: 1. (Set|Get)MemoryRegionAttribute is difficult to merge together, because the parameters between the tow APIs are not similar. So I suggest they be independent. 2. The EfiAttributeConverse, GetMemoryRegionAttribute, SetMemory

Re: [edk2-devel] [PATCH V3 9/9] RedfishPkg/HostInterfaceBmcUsbNic: Fix potential memory corruption issue

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 9/9] RedfishPkg/HostInterfaceBmcUsbNic: Fix pot

Re: [edk2-devel] [PATCH V3 8/9] RedfishPkg/HostInterfaceBmcUsbNic: Fix incorrect HI protocol record size

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 8/9] RedfishPkg/HostInterfaceBmcUsbNic: Fix inc

Re: [edk2-devel] [PATCH V3 7/9] RedfishPkg/HostInterfaceBmcUsbNic: Fix incorrect reference of MAC address pointer

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 7/9] RedfishPkg/HostInterfaceBmcUsbNic: Fix inc

Re: [edk2-devel] [PATCH V3 6/9] RedfishPkg/RedfishDiscovery: Refine SMBIOS 42h code

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 6/9] RedfishPkg/RedfishDiscovery: Refine SMBIOS

Re: [edk2-devel] [PATCH V3 0/9] Refine BMC USB NIC discovery and Redfish service enablement

2023-12-03 Thread Mike Maslenkin
On Mon, Nov 27, 2023 at 8:31 AM wrote: > > From: Abner Chang > > In V3: Address Mike's comments to V2. > > In V2: Send additional two patches those are missed >in V1. > > This patch set updates the algorithm of BMC USB > NIC discovery and fixes some bugs. > > - Add a new protocol to trigg

Re: [edk2-devel] [PATCH V3 5/9] RedfishPkg/RedfishDiscovery: Add more debug message

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 5/9] RedfishPkg/RedfishDiscovery: Add more debu

Re: [edk2-devel] [PATCH V3 4/9] RedfishPkg/RedfishConfigHandler: Correct the prototype of callback function

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 4/9] RedfishPkg/RedfishConfigHandler: Correct t

Re: [edk2-devel] [PATCH V3 3/9] RedfishPkg/RedfishConfigHandler: Use Redfish HI readiness notification

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 3/9] RedfishPkg/RedfishConfigHandler: Use Redfi

Re: [edk2-devel] [PATCH V3 2/9] RedfishPkg/RedfishHostInterfaceDxe: Add Redfish HI readiness notification

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: abner.ch...@amd.com > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Nickle Wang ; Igor Kulchytskyy ; > Mike Maslenkin > Subject: [PATCH V3 2/9] RedfishPkg/RedfishHostInterfaceDxe: Add Re

Re: [edk2-devel] [PATCH V3 1/9] RedfishPkg/BmcUsbNicLib: Update BMC USB NIC searching algorithm

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: devel@edk2.groups.io On Behalf Of Chang, Abner > via groups.io > Sent: Monday, November 27, 2023 1:31 PM > To: devel@edk2.groups.io > Cc: Mike Maslenkin ; Nickle Wang > ; Igor Kulchytskyy > Subject: [edk2-devel] [

Re: [edk2-devel] [PATCH 3/3] RedfishClientPkg: warn user about reboot if allowed.

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: Mike Maslenkin > Sent: Friday, December 1, 2023 7:19 AM > To: devel@edk2.groups.io > Cc: abner.ch...@amd.com; Nickle Wang ; > ig...@ami.com > Subject: [PATCH 3/3] RedfishClientPkg: warn user about reboot if allowed

Re: [edk2-devel] [PATCH 2/3] RedfishClientPkg: fix format specifier in DEBUG macro

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: Mike Maslenkin > Sent: Friday, December 1, 2023 7:19 AM > To: devel@edk2.groups.io > Cc: abner.ch...@amd.com; Nickle Wang ; > ig...@ami.com > Subject: [PATCH 2/3] RedfishClientPkg: fix format specifier in DEBUG mac

Re: [edk2-devel] [PATCH 1/3] RedfishClientPkg: fix misleading texts in debug traces.

2023-12-03 Thread Nickle Wang via groups.io
Reviewed-by: Nickle Wang Regards, Nickle > -Original Message- > From: Mike Maslenkin > Sent: Friday, December 1, 2023 7:19 AM > To: devel@edk2.groups.io > Cc: abner.ch...@amd.com; Nickle Wang ; > ig...@ami.com > Subject: [PATCH 1/3] RedfishClientPkg: fix misleading texts in debug tra

[edk2-devel] [PATCH v4 1/1] MdeModulePkg/AcpiTableDxe: Prefer xDSDT over DSDT when installing tables

2023-12-03 Thread Dhaval Sharma
As per ACPI Spec 6.5+ Table 5-9 if xDSDT is available, it should be used first. Handle required flow when xDSDT is absent or present. Test: Tested on RISCV64 Qemu platform with xDSDT and booted to linux kernel. Cc: Liming Gao Cc: Zhiguang Liu Cc: Dandan Bi Signed-off-by: Dhaval Sharma Acked-b

[edk2-devel] [PATCH v4 0/1] Prefer XDSDT table over DSDT if available

2023-12-03 Thread Dhaval Sharma
Enable detection of XDSDT table from ACPI HOB and use it to comply with ACPI spec 6.5+ Table 5-9. Dhaval (1): MdeModulePkg/AcpiTableDxe: Prefer xDSDT over DSDT when installing tables MdeModulePkg/Universal/Acpi/AcpiTableDxe/AcpiTableProtocol.c | 23 ++-- 1 file changed, 16

Re: [edk2-devel] [Patch V2 1/4] BaseTools: fixing FMMT ShrinkFv issue

2023-12-03 Thread Yuwei Chen
Hi Liming, To avoid wrong operation. I create a new PR for this patch series with Reviewed-by. Please help adding push label. Thanks. https://github.com/tianocore/edk2/pull/5102 Thanks, Christine > -Original Message- > From: gaoliming > Sent: Wednesday, November 29, 2023 11:53 PM > To:

[edk2-devel] Event: Tools, CI, Code base construction meeting series - Monday, December 4, 2023 #cal-reminder

2023-12-03 Thread Group Notification
*Reminder: Tools, CI, Code base construction meeting series* *When:* Monday, December 4, 2023 4:30pm to 5:30pm (UTC-08:00) America/Los Angeles *Where:* https://teams.microsoft.com/l/meetup-join/19%3ameeting_ZDI2ZDg4NmMtMjI1My00MzI5LWFmYjAtMGQyNjUzNTBjZGYw%40thread.v2/0?context=%7b%22Tid%22%3a%227

Re: [edk2-devel] [PATCH] Ext4Pkg: Fix CRC16 checksumming on block groups

2023-12-03 Thread Pedro Falcato
On Sun, Dec 3, 2023 at 10:56 PM Marvin Häuser wrote: > > Reviewed-by: Marvin Häuser Thank you! Pushed as b95395b. -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#111999): https://edk2.groups.io/g/devel/message/111999 Mute Th

Re: [edk2-devel] [PATCH] Ext4Pkg: Fix CRC16 checksumming on block groups

2023-12-03 Thread Marvin Häuser
Reviewed-by: Marvin Häuser > On Dec 3, 2023, at 23:41, Pedro Falcato wrote: > > Old filesystems (around 2008 and older) do not use CRC32c > but rather CRC16-ANSI. Previously, the CalculateCrc16Ansi function was > broken and gave us wrong checksums. Adapt to the new interface. > > And while we

[edk2-devel] [PATCH] Ext4Pkg: Fix CRC16 checksumming on block groups

2023-12-03 Thread Pedro Falcato
Old filesystems (around 2008 and older) do not use CRC32c but rather CRC16-ANSI. Previously, the CalculateCrc16Ansi function was broken and gave us wrong checksums. Adapt to the new interface. And while we're at it, fix the checksum algorithm itself - the crc16 algorithm just skips over the bg_che

Re: [edk2-devel] [Patch 1/1] EmulatorPkg/Win/Host: Remove /NODEFAULTLIB in WinHost.inf

2023-12-03 Thread Pedro Falcato
On Sun, Dec 3, 2023 at 1:44 AM Michael D Kinney wrote: > > Update WinHost.inf to remove /NODEFAULTLIB option from DLINK_FLAGS. > WinHost builds a Windows application that depends on the compiler > default libraries. By removing /NODEFAULTLIB, fewer libraries > have to be specified on the link comm

Re: [edk2-devel] [PATCH 1/2] UnitTestFrameworkPkg: Fix Google Test components with multiple files

2023-12-03 Thread Pedro Falcato
On Sun, Dec 3, 2023 at 2:37 AM Michael D Kinney wrote: > > Merged: https://github.com/tianocore/edk2/pull/5098 I was going to submit a v2, but thanks for cleaning things up and pushing! -- Pedro -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply O