Re: [edk2-devel] Cannot build due to recent change

2020-10-02 Thread Laszlo Ersek 
on a tangent:

On 10/01/20 12:43, Leif Lindholm wrote:

> if the platform port produces EFI_RNG_PROTOCOL
> from some onboard hw, you can use the approach from edk2-platforms
> commit 1e09147a01ae ("Platforms/RaspberryPi: Fix RPi4 RngLib build error").

Emphasis on "onboard hw".

I find it incredibly attractive and good system design that

  Silicon/Broadcom/Bcm283x/Drivers/Bcm2838RngDxe

can *exist*, because the underlying *platform hardware* also exists. I
wish we had something similar for virt, i.e. randomness available from
just a handful of fixed location MMIO registers. Randomness is needed
way earlier than when various bus enumerations occur, such as PCI, or
even virtio-mmio.

See also: why virtio-serial is unusable as a real debug output device.

Yes, I know, on Intel there's RDRAND and friends; they didn't have good
press, and it's still unclear to me how much one can count on RDRAND's
presence on the various QEMU/KVM CPU models.

/end tangent

Laszlo



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65832): https://edk2.groups.io/g/devel/message/65832
Mute This Topic: https://groups.io/mt/77236499/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] Cannot build due to recent change

2020-10-01 Thread Leif Lindholm 
On Thu, Oct 01, 2020 at 12:06:07 +0100, Andrew Goodbody wrote:
> > Anyway, yes - commit b5701a4c7a0f ("CryptoPkg: OpensslLib: Use RngLib
> > to generate entropy in rand_pool"), merged through
> > https://github.com/tianocore/edk2/pull/938 broke EmulatorPkg, and most
> > platforms outside the main edk2 tree.
> > 
> > This should have been handled better.
> > 
> > But the fix is simple - if the platform port produces EFI_RNG_PROTOCOL
> > from some onboard hw, you can use the approach from edk2-platforms
> > commit 1e09147a01ae ("Platforms/RaspberryPi: Fix RPi4 RngLib build error").
> > Otherwise, you can use the approach used in edk2 commit 416813753726
> > ("ArmVirtPkg: Add RngLib based on TimerLib for CryptoPkg").
> 
> Just to be sure I understand this - most platforms outside of the main edk2
> tree are going to need updating to take account of this change.

Well, most non-x86 platforms. There is an RdRand implementation in
tree, which I expect most platforms that could use already did.

/
Leif


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65796): https://edk2.groups.io/g/devel/message/65796
Mute This Topic: https://groups.io/mt/77236499/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] Cannot build due to recent change

2020-10-01 Thread Sami Mujawar 
Hi All,

I am looking into this and will provide a fix shortly.

Regards,

Sami Mujawar

-Original Message-
From: devel@edk2.groups.io  On Behalf Of Leif Lindholm 
via groups.io
Sent: 01 October 2020 11:44 AM
To: devel@edk2.groups.io; andrew.goodb...@linaro.org
Cc: Ard Biesheuvel ; Thomas Abraham 
; Liming Gao 
Subject: Re: [edk2-devel] Cannot build due to recent change

Hi Andrew,

Welcome to the list.

First of all, it helps to cc maintainers of the bits you're having
issues with.

The GetMaintainer.py script can look these up for you automatically
for a given path:
python ../edk2/BaseTools/Scripts/GetMaintainer.py -l 
Platform/ARM/JunoPkg/ArmJuno.dsc
Platform/ARM/JunoPkg/ArmJuno.dsc
  Leif Lindholm 
  Ard Biesheuvel 
  Thomas Abraham 

Anyway, yes - commit b5701a4c7a0f ("CryptoPkg: OpensslLib: Use RngLib
to generate entropy in rand_pool"), merged through
https://github.com/tianocore/edk2/pull/938 broke EmulatorPkg, and most
platforms outside the main edk2 tree.

This should have been handled better.

But the fix is simple - if the platform port produces EFI_RNG_PROTOCOL
from some onboard hw, you can use the approach from edk2-platforms
commit 1e09147a01ae ("Platforms/RaspberryPi: Fix RPi4 RngLib build error").
Otherwise, you can use the approach used in edk2 commit 416813753726
("ArmVirtPkg: Add RngLib based on TimerLib for CryptoPkg").

If you're just trying to build something, you can rewind your edk2
HEAD to the edk2-stable202008 tag.

Best Regards,

Leif

On Wed, Sep 30, 2020 at 13:16:22 +0100, Andrew Goodbody wrote:
> Looks like a recent change is not working for AARCH64
>
> https://github.com/tianocore/edk2/commit/b5701a4c7a0fb185e0c5b9db9525939c78664bfd
>
> edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc(...): error 4000: Instance
> of library class [RngLib] is not found
> in [/home/andrew/src/edk2/CryptoPkg/Library/OpensslLib/OpensslLib.inf]
> [AARCH64]
> consumed by module [/home/andrew/src/edk2/NetworkPkg/IScsiDxe/IScsiDxe.inf]
>
>
> - Failed -
> Build end time: 12:56:54, Sep.30 2020
> Build total time: 00:00:01
>
> Sorry but I have not found my way around here yet. Is there anyone that can
> see what the best fix is for this please?
>
> Thanks,
> Andrew
>
>
>
>
>





IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65795): https://edk2.groups.io/g/devel/message/65795
Mute This Topic: https://groups.io/mt/77236499/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] Cannot build due to recent change

2020-10-01 Thread Andrew Goodbody 

On 01/10/2020 11:43, Leif Lindholm wrote:

Hi Andrew,

Welcome to the list.


Thank you. I did BIOS work and some UEFI in a previous role but have 
been out of the loop for some years.



First of all, it helps to cc maintainers of the bits you're having
issues with.

The GetMaintainer.py script can look these up for you automatically
for a given path:
python ../edk2/BaseTools/Scripts/GetMaintainer.py -l 
Platform/ARM/JunoPkg/ArmJuno.dsc
Platform/ARM/JunoPkg/ArmJuno.dsc
   Leif Lindholm 
   Ard Biesheuvel 
   Thomas Abraham 


Thanks for the pointer, will do in future.


Anyway, yes - commit b5701a4c7a0f ("CryptoPkg: OpensslLib: Use RngLib
to generate entropy in rand_pool"), merged through
https://github.com/tianocore/edk2/pull/938 broke EmulatorPkg, and most
platforms outside the main edk2 tree.

This should have been handled better.

But the fix is simple - if the platform port produces EFI_RNG_PROTOCOL
from some onboard hw, you can use the approach from edk2-platforms
commit 1e09147a01ae ("Platforms/RaspberryPi: Fix RPi4 RngLib build error").
Otherwise, you can use the approach used in edk2 commit 416813753726
("ArmVirtPkg: Add RngLib based on TimerLib for CryptoPkg").


Just to be sure I understand this - most platforms outside of the main 
edk2 tree are going to need updating to take account of this change.



If you're just trying to build something, you can rewind your edk2
HEAD to the edk2-stable202008 tag.


Thanks, good to know.

Andrew


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65794): https://edk2.groups.io/g/devel/message/65794
Mute This Topic: https://groups.io/mt/77236499/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] Cannot build due to recent change

2020-10-01 Thread Leif Lindholm 
Hi Andrew,

Welcome to the list.

First of all, it helps to cc maintainers of the bits you're having
issues with.

The GetMaintainer.py script can look these up for you automatically
for a given path:
python ../edk2/BaseTools/Scripts/GetMaintainer.py -l 
Platform/ARM/JunoPkg/ArmJuno.dsc
Platform/ARM/JunoPkg/ArmJuno.dsc
  Leif Lindholm 
  Ard Biesheuvel 
  Thomas Abraham 

Anyway, yes - commit b5701a4c7a0f ("CryptoPkg: OpensslLib: Use RngLib
to generate entropy in rand_pool"), merged through
https://github.com/tianocore/edk2/pull/938 broke EmulatorPkg, and most
platforms outside the main edk2 tree.

This should have been handled better.

But the fix is simple - if the platform port produces EFI_RNG_PROTOCOL
from some onboard hw, you can use the approach from edk2-platforms
commit 1e09147a01ae ("Platforms/RaspberryPi: Fix RPi4 RngLib build error").
Otherwise, you can use the approach used in edk2 commit 416813753726
("ArmVirtPkg: Add RngLib based on TimerLib for CryptoPkg").

If you're just trying to build something, you can rewind your edk2
HEAD to the edk2-stable202008 tag.

Best Regards,

Leif

On Wed, Sep 30, 2020 at 13:16:22 +0100, Andrew Goodbody wrote:
> Looks like a recent change is not working for AARCH64
> 
> https://github.com/tianocore/edk2/commit/b5701a4c7a0fb185e0c5b9db9525939c78664bfd
> 
> edk2-platforms/Platform/ARM/JunoPkg/ArmJuno.dsc(...): error 4000: Instance
> of library class [RngLib] is not found
>   in [/home/andrew/src/edk2/CryptoPkg/Library/OpensslLib/OpensslLib.inf]
> [AARCH64]
>   consumed by module 
> [/home/andrew/src/edk2/NetworkPkg/IScsiDxe/IScsiDxe.inf]
> 
> 
> - Failed -
> Build end time: 12:56:54, Sep.30 2020
> Build total time: 00:00:01
> 
> Sorry but I have not found my way around here yet. Is there anyone that can
> see what the best fix is for this please?
> 
> Thanks,
> Andrew
> 
> 
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#65791): https://edk2.groups.io/g/devel/message/65791
Mute This Topic: https://groups.io/mt/77236499/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-