回复: [edk2-devel][PATCH] BaseTools: Increase the DevicePath length for support more PCD value.


gaoliming
 

Xiaolu:
Can you calculate the required memory, then allocate it instead of use hard code memory length?

Thanks
Liming

-----邮件原件-----
发件人: Jiang, Xiaolu <xiaolu.jiang@intel.com>
发送时间: 2021年11月8日 10:26
收件人: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io
抄送: Feng, Bob C <bob.c.feng@intel.com>; Chen, Christine
<yuwei.chen@intel.com>; Fu, Siyuan <siyuan.fu@intel.com>
主题: RE: [edk2-devel][PATCH] BaseTools: Increase the DevicePath length for
support more PCD value.

Hi Liming,

I have checked with Team member , Totally We need support 50+ Driver GUID
in DevicePath PCD value ,So the length need to increase again, I will change
the Patch and re-send again.

Thanks!

-----Original Message-----
From: Jiang, Xiaolu
Sent: Monday, November 8, 2021 9:24 AM
To: gaoliming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Chen, Christine
<Yuwei.Chen@intel.com>
Subject: RE: [edk2-devel][PATCH] BaseTools: Increase the DevicePath length
for support more PCD value.

1. Currently the PCD value length is More than 1024, less than 2048, 2. Now
we have 14 members, Also need to add 4~6 part to the PCD.
So we Increase the length to 4096.

-----Original Message-----
From: gaoliming <gaoliming@byosoft.com.cn>
Sent: Monday, November 8, 2021 9:15 AM
To: Jiang, Xiaolu <xiaolu.jiang@intel.com>; devel@edk2.groups.io
Cc: Feng, Bob C <bob.c.feng@intel.com>; Chen, Christine
<yuwei.chen@intel.com>
Subject: 回复: [edk2-devel][PATCH] BaseTools: Increase the DevicePath
length for support more PCD value.

Xiaolu:
Why choose 4096? Is it enough?

Thanks
Liming
-----邮件原件-----
发件人: Xiaolu.Jiang <xiaolu.jiang@intel.com>
发送时间: 2021年11月6日 11:51
收件人: devel@edk2.groups.io
抄送: Xiaolu.Jiang <xiaolu.jiang@intel.com>; Bob Feng
<bob.c.feng@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Yuwei
Chen <yuwei.chen@intel.com>
主题: [edk2-devel][PATCH] BaseTools: Increase the DevicePath length for
support more PCD value.

Currently the PCD Value only support 13 Guid,When use more 13 pcd will
cause the build tool fail, Need increase the DevicePath length to
support more value.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3718

Cc: Bob Feng <bob.c.feng@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Yuwei Chen <yuwei.chen@intel.com>

Signed-off-by: Xiaolu Jiang <xiaolu.jiang@intel.com>
---
BaseTools/Source/C/DevicePath/DevicePath.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/BaseTools/Source/C/DevicePath/DevicePath.c
b/BaseTools/Source/C/DevicePath/DevicePath.c
index c4d224ed61..ef493f5506 100644
--- a/BaseTools/Source/C/DevicePath/DevicePath.c
+++ b/BaseTools/Source/C/DevicePath/DevicePath.c
@@ -170,7 +170,7 @@ int main(int argc, CHAR8 *argv[])
fprintf(stderr, "Invalid option value, Device Path can't be
NULL");

return STATUS_ERROR;

}

- Str16 = (CHAR16 *)malloc(1024);

+ Str16 = (CHAR16 *)malloc(4096);

if (Str16 == NULL) {

fprintf(stderr, "Resource, memory cannot be allocated");

return STATUS_ERROR;

--
2.30.2.windows.1

Join devel@edk2.groups.io to automatically receive all group messages.