Yes, we can create a host test specific lib if no better ways. Regards, Yi
From: Yao, Jiewen <[email protected]> Sent: Friday, June 14, 2024 3:20 PM To: Li, Yi1 <[email protected]>; [email protected] Cc: Hou, Wenxing <[email protected]>; Kinney, Michael D <[email protected]>; Pedro Falcato <[email protected]>; Ard Biesheuvel <[email protected]> Subject: RE: CryptoPkg host test broken due to smoketest for RDRAND Can we use a host test specific RngLib? From: Li, Yi1 <[email protected]<mailto:[email protected]>> Sent: Friday, June 14, 2024 3:08 PM To: [email protected]<mailto:[email protected]> Cc: Hou, Wenxing <[email protected]<mailto:[email protected]>>; Yao, Jiewen <[email protected]<mailto:[email protected]>>; Kinney, Michael D <[email protected]<mailto:[email protected]>>; Pedro Falcato <[email protected]<mailto:[email protected]>>; Ard Biesheuvel <[email protected]<mailto:[email protected]>> Subject: CryptoPkg host test broken due to smoketest for RDRAND All crypto host tests which consumed randlib broken due to: https://github.com/tianocore/edk2/pull/5714 Not sure why this issue not reported by CI when merge this PR. The reason is that the ```BaseRngLibConstructor``` of rnglib is not called in host test, so ```mRdRandSupported``` is not enabled. Then the Crypto API calls ```GetRandomNumber*``` will fail. GetRandomNumber64 ( OUT UINT64 *Rand ) { ...... if (!ArchIsRngSupported ()) { return FALSE; } Is there a way to let unit test host to call the constructors correctly? Regards, Yi -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119569): https://edk2.groups.io/g/devel/message/119569 Mute This Topic: https://groups.io/mt/106666288/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
