Hello Liming, Jiewen,
On 7/10/23 03:26, gaoliming wrote:
Pierre:
Another option is to define two PCD for Rng algorithm in MdePkg. One PCD
value is ArmRndr GUID, another is UnSafe GUID. This way can also resolve the
package dependency.
So there would be a Pcd for the unsafe algorithm so that the platform can
choose a custom GUID to advertise the unsafe algorithm ? If so, I think
it would be better to have only one common GUID,
Thanks
Liming
-----邮件原件-----
发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Yao, Jiewen
发送时间: 2023年7月7日 22:34
收件人: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com>; Pierre
Gondois <pierre.gond...@arm.com>
抄送: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming
<gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; Wang,
Jian J <jian.j.w...@intel.com>; Ard Biesheuvel <ardb+tianoc...@kernel.org>;
Sami Mujawar <sami.muja...@arm.com>; Jose Marinho
<jose.mari...@arm.com>; Kun Qin <kuqi...@gmail.com>
主题: Re: [edk2-devel] [PATCH v3 3/6] MdePkg/Rng: Add GUIDs to describe
Rng algorithms
I think a better way is to define a new library instance in other package with
the new ARM APIs.
The old one can be kept as is.
That will limit the impact to existing platform.
Yes right,
However this would mean having 2 identical implementations of the
BaseRngTimerLib,
I ll try to move the library instead, if it's ok
Regards,
Pierre
-----Original Message-----
From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Yao,
Jiewen
Sent: Friday, July 7, 2023 10:28 PM
To: Pierre Gondois <pierre.gond...@arm.com>; devel@edk2.groups.io
Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming
<gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>;
Wang,
Jian J <jian.j.w...@intel.com>; Ard Biesheuvel
<ardb+tianoc...@kernel.org>;
Sami Mujawar <sami.muja...@arm.com>; Jose Marinho
<jose.mari...@arm.com>; Kun Qin <kuqi...@gmail.com>
Subject: Re: [edk2-devel] [PATCH v3 3/6] MdePkg/Rng: Add GUIDs to
describe
Rng algorithms
Thanks Pierre.
Yes, I agree to move it to other package to resolve dependency issue.
-----Original Message-----
From: Pierre Gondois <pierre.gond...@arm.com>
Sent: Friday, July 7, 2023 10:25 PM
To: Yao, Jiewen <jiewen....@intel.com>; devel@edk2.groups.io
Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Gao, Liming
<gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>;
Wang,
Jian J <jian.j.w...@intel.com>; Ard Biesheuvel
<ardb+tianoc...@kernel.org>;
Sami Mujawar <sami.muja...@arm.com>; Jose Marinho
<jose.mari...@arm.com>; Kun Qin <kuqi...@gmail.com>
Subject: Re: [edk2-devel] [PATCH v3 3/6] MdePkg/Rng: Add GUIDs to
describe
Rng algorithms
Hello Jiewen,
We have the following dependency issue:
- the BaseRngTimerLib is in the MdePkg
- we need a GUID to describe the BaseRngTimerLib algorithm
- we cannot add the gEdkiiRngAlgorithmUnSafe in the MdePkg, and the
gZeroGuid is also not in the MdePkg
- the MdePkg should not have dependencies over other packages
As the BaseRngTimerLib is not really standard and should not be used in
production builds,
would you agree if it was moved to the MdeModulePkg or to the
SecurityPkg
(with the gEdkiiRngAlgorithmUnSafe definition) ?
Regards,
Pierre
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106791): https://edk2.groups.io/g/devel/message/106791
Mute This Topic: https://groups.io/mt/100050641/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
