[edk2-devel] [PATCH v3 0/16] Redfish related fixes and improvements

2023-12-24 Thread Mike Maslenkin
This is an expanded version of the original set of 9 patches with additional patch introducing brackets for IPv6 Redfish resource location with required changes according to comments for the initial RFC patch. PR: https://github.com/tianocore/edk2/pull/5149 diff from v2: added Abner's R-b ref

[edk2-devel] [PATCH v3 01/16] RedfishPkg: fix RedfishPlatformHostInterfaceLib library class name typo.

2023-12-24 Thread Mike Maslenkin
PlatformHostInterfaceBmcUsbNicLib is the library instance name not the class name. Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- .../PlatformHostInterfaceBmcUsbNicLib.inf | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[edk2-devel] [PATCH v3 02/16] RedfishPkg: fix RedfishPlatformCredentialLib library class name typo.

2023-12-24 Thread Mike Maslenkin
RedfishPlatformCredentialIpmiLib is the library instance name not the class name. Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- .../RedfishPlatformCredentialIpmiLib.inf| 2 +- 1 file changed, 1 insertion(+), 1 deletion(

[edk2-devel] [PATCH v3 03/16] RedfishPkg: get rid of unused definitions from RedfishCrtLib.h

2023-12-24 Thread Mike Maslenkin
It seems that initial implementation of this header file is based on CrtLibSupport.h from CryptoPkg. But uid, euid, gid, egid and sa_family_t sre not used in RedfishPkg. So remove them. Also take "true" and "false" definition from MdePkg's LibFdtSupport.h header file, that also seems based on a he

[edk2-devel] [PATCH v3 04/16] RedfishPkg: RedfishPlatformConfigDxe: reduce memory allocations

2023-12-24 Thread Mike Maslenkin
It's unclear why the new string is allocated as copy of the original string if its pointer is stored in an array and the original string is released immediately after the copy is created. All data allocated in the same pool. Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin Rev

[edk2-devel] [PATCH v3 05/16] RedfishPkg: RedfishDiscoverDxe: fix memory leak on error path.

2023-12-24 Thread Mike Maslenkin
Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c b/RedfishPkg/Redfis

[edk2-devel] [PATCH v3 06/16] RedfishPkg: add Component Name protocols to RedfishConfigHandler driver

2023-12-24 Thread Mike Maslenkin
Currently there is no description for RedfishConfigHandler driver. This leads to in the "DRIVER NAME" column of a `drivers` command for example. Cc: Abner Chang Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin --- .../RedfishConfigHandler/ComponentName.c | 218

[edk2-devel] [PATCH v3 07/16] RedfishPkg: add proper initialization of IPMI request

2023-12-24 Thread Mike Maslenkin
All fields of IPMI_CHANNEL_INFO_CHANNEL_NUMBER union must be initialized to avoid error condition on BMC side. Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- .../PlatformHostInterfaceBmcUsbNicLib.c | 8 +--- 1 file changed

[edk2-devel] [PATCH v3 08/16] EmulatorPkg: fix typo. PcdRedfishServie -> PcdRedfishService

2023-12-24 Thread Mike Maslenkin
Cc: Nickle Wang Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- EmulatorPkg/EmulatorPkg.dec | 8 .../RedfishPlatformCredentialLib.c | 12 ++-- .../RedfishPlatformCredentialLib.inf | 8 3 files chan

[edk2-devel] [PATCH v3 09/16] EmulatorPkg: RedfishPlatformHostInterfaceLib: get rid of unused variable

2023-12-24 Thread Mike Maslenkin
Cc: Nickle Wang Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- .../RedfishPlatformHostInterfaceLib.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/EmulatorPkg/Library/RedfishPlatformHostInterfaceLib/RedfishPlatformHostInterfaceLib.c b/EmulatorPkg/

[edk2-devel] [PATCH v3 10/16] RedfishDiscoverDxe: introduce InitInformationData helper function

2023-12-24 Thread Mike Maslenkin
Cc: Abner Chang Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin --- .../RedfishDiscoverDxe/RedfishDiscoverDxe.c | 108 -- 1 file changed, 74 insertions(+), 34 deletions(-) diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c b/RedfishPkg/Redfis

[edk2-devel] [PATCH v3 11/16] RedfishDiscoverDxe: refine InitInformationData(), remove unnecessary casts

2023-12-24 Thread Mike Maslenkin
Cc: Nickle Wang Cc: Igor Kulchytskyy Cc: Pedro Falcato Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- .../RedfishDiscoverDxe/RedfishDiscoverDxe.c | 36 +-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/RedfishPkg/RedfishDiscoverDxe/RedfishDiscov

[edk2-devel] [PATCH v3 14/16] RedfishDiscoverDxe: release resources when refreshing information data

2023-12-24 Thread Mike Maslenkin
It is required to free previously allocated strings stored in DiscoveredInstance->Information structure. Cc: Abner Chang Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin --- RedfishPkg/RedfishDiscoverDxe/RedfishDiscoverDxe.c | 2 ++ 1 file changed, 2 insertions(+) diff --gi

[edk2-devel] [PATCH v3 13/16] RedfishDiscoverDxe: add a helper function deallocating string resources.

2023-12-24 Thread Mike Maslenkin
This patch adds a handy helper function deallocating resources from the EFI_REDFISH_DISCOVERED_INFORMATION structure instance. Cc: Abner Chang Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin --- .../RedfishDiscoverDxe/RedfishDiscoverDxe.c | 68 --- 1 file

[edk2-devel] [PATCH v3 16/16] RedfishPkg: RedfishDiscoverDxe: add [] brackets to URI for IPv6 addresses

2023-12-24 Thread Mike Maslenkin
URI is generated based on the RedfishLocation containing an ASCII string representing the IP address. So, in the case of IPv4 the canonical representation of an IPv4 address was inserted into the resulting Unicode string i.e: "http{,s}://X.X.X.X/". In the case of IPv6, to access resources, the IP

[edk2-devel] [PATCH v3 12/16] RedfishDiscoverDxe: refine InitInformationData() function

2023-12-24 Thread Mike Maslenkin
Cache size of ASCII string in local variable. Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin Reviewed-by: Abner Chang --- .../RedfishDiscoverDxe/RedfishDiscoverDxe.c | 34 --- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/RedfishPkg/Redf

[edk2-devel] [PATCH v3 15/16] RedfishDiscoverDxe: handle memory allocation error conditions.

2023-12-24 Thread Mike Maslenkin
REF:https://bugzilla.tianocore.org/show_bug.cgi?id=4625 Cc: Abner Chang Cc: Nickle Wang Cc: Igor Kulchytskyy Signed-off-by: Mike Maslenkin --- .../RedfishDiscoverDxe/RedfishDiscoverDxe.c | 89 --- 1 file changed, 78 insertions(+), 11 deletions(-) diff --git a/RedfishPkg/Red

[edk2-devel] 回复: [Patch V2] BaseTools: VfrCompiler Adds DefaultValueError Feature

2023-12-24 Thread gaoliming via groups.io
Zifeng: VFR_FLAGS can be set in the module INF for this module only. So, you can set --catch_default option in the modules those you want to remove the default opcode for. With this usage, --except_list option is not required. For example, module INF can add below section to enable this option.

回复: 回复: 回复: [edk2-devel] [PATCH v4 09/37] MdePkg: Add a new library named PeiServicesTablePointerLibKs0

2023-12-24 Thread gaoliming via groups.io
Chao: I add my comments below. 发件人: devel@edk2.groups.io 代表 Chao Li 发送时间: 2023年12月21日 19:19 收件人: gaoliming ; devel@edk2.groups.io; 'Michael D Kinney' 抄送: 'Zhiguang Liu' ; 'Laszlo Ersek' 主题: Re: 回复: 回复: [edk2-devel] [PATCH v4 09/37] MdePkg: Add a new library named PeiServicesTablePointer

[edk2-devel] [PATCH V2 0/2] Update keybaord map based on UEFI spec 2.10

2023-12-24 Thread Li, Yi
REF: UEFI SPEC 2.10 34.8.10 EFI_KEY Add EfiKeyIntl0-9. Signed-off-by: Yi Li v2: Remove changes in HiiFramework.h because it should follow HII spec. Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen Yi Li (2): MdePkg: Upda

[edk2-devel] [PATCH V2 1/2] MdePkg: Update keybaord map based on UEFI spec 2.10

2023-12-24 Thread Li, Yi
REF: UEFI SPEC 2.10 34.8.10 EFI_KEY Add EfiKeyIntl0-9. Signed-off-by: Yi Li Cc: Michael D Kinney Cc: Liming Gao Cc: Zhiguang Liu --- MdePkg/Include/Uefi/UefiInternalFormRepresentation.h | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/MdePkg/Include/Uefi/Uefi

[edk2-devel] [PATCH V2 2/2] BaseTools: Update keybaord map based on UEFI spec 2.10

2023-12-24 Thread Li, Yi
REF: UEFI SPEC 2.10 34.8.10 EFI_KEY Add EfiKeyIntl0-9. Signed-off-by: Yi Li Cc: Rebecca Cran Cc: Liming Gao Cc: Bob Feng Cc: Yuwei Chen --- .../Include/Common/UefiInternalFormRepresentation.h | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/BaseTools/Source

Re: [edk2-devel] [PATCH 2/2] BaseTools: Update keybaord map based on UEFI spec 2.10

2023-12-24 Thread Li, Yi
Thanks for your review, patch updated. Regards, Yi -Original Message- From: gaoliming Sent: Saturday, December 23, 2023 10:16 AM To: Li, Yi1 ; devel@edk2.groups.io Cc: 'Rebecca Cran' ; Feng, Bob C ; Chen, Christine Subject: 回复: [PATCH 2/2] BaseTools: Update keybaord map based on UEFI

Re: [edk2-devel] [Patch V2] BaseTools: VfrCompiler Adds DefaultValueError Feature

2023-12-24 Thread Yuting Yang
Hi Liming, Thank you for adding the account for me. I have created a Bugzilla https://bugzilla.tianocore.org/show_bug.cgi?id=4629 for this feature ~ Best Regards,, Yuting -Original Message- From: gaoliming Sent: Monday, December 25, 2023 9:14 AM To: Yang, Yuting2 Subject: 回复: [Patch