Re: [U-BOOT TEST HOOKS][PATCH 1/1] qemu-riscv: enable virtio-rng-pci

2023-07-31 Thread Heinrich Schuchardt




On 7/31/23 10:48, Bin Meng wrote:

On Mon, Jul 31, 2023 at 4:00 PM Leo Liang  wrote:


On Mon, Jul 31, 2023 at 09:25:07AM +0200, Heinrich Schuchardt wrote:

Linux' KASLR uses the EFI_RNG_PROTOCOL as entropy source. We should
enable CONFIG_DM_RNG in U-Boot. For the EFI unit test for the protocol to
succeed a virtio-rng-pci device has to be provided when invoking QEMU.

Reported-by: Leo Liang 
Signed-off-by: Heinrich Schuchardt 
---
See related patch
[PATCH 1/1] riscv: qemu: imply CONFIG_DM_RNG
https://lists.denx.de/pipermail/u-boot/2023-July/525293.html

The non-spl defconfigs still don't scan virtio devices automatically. This
also needs to be addressed.
---
  bin/travis-ci/conf.qemu-riscv32_na | 2 +-
  bin/travis-ci/conf.qemu-riscv32_spl_na | 2 +-
  bin/travis-ci/conf.qemu-riscv64_na | 2 +-
  bin/travis-ci/conf.qemu-riscv64_spl_na | 2 +-
  4 files changed, 4 insertions(+), 4 deletions(-)


LGTM!
Reviewed-by: Leo Yu-Chi Liang 



NAK!

It should be "virtio-rng-device"


Thanks. That resolves the issue with virtio scanning.

Best regards

Heinrich


Re: [U-BOOT TEST HOOKS][PATCH 1/1] qemu-riscv: enable virtio-rng-pci

2023-07-31 Thread Bin Meng
On Mon, Jul 31, 2023 at 4:00 PM Leo Liang  wrote:
>
> On Mon, Jul 31, 2023 at 09:25:07AM +0200, Heinrich Schuchardt wrote:
> > Linux' KASLR uses the EFI_RNG_PROTOCOL as entropy source. We should
> > enable CONFIG_DM_RNG in U-Boot. For the EFI unit test for the protocol to
> > succeed a virtio-rng-pci device has to be provided when invoking QEMU.
> >
> > Reported-by: Leo Liang 
> > Signed-off-by: Heinrich Schuchardt 
> > ---
> > See related patch
> > [PATCH 1/1] riscv: qemu: imply CONFIG_DM_RNG
> > https://lists.denx.de/pipermail/u-boot/2023-July/525293.html
> >
> > The non-spl defconfigs still don't scan virtio devices automatically. This
> > also needs to be addressed.
> > ---
> >  bin/travis-ci/conf.qemu-riscv32_na | 2 +-
> >  bin/travis-ci/conf.qemu-riscv32_spl_na | 2 +-
> >  bin/travis-ci/conf.qemu-riscv64_na | 2 +-
> >  bin/travis-ci/conf.qemu-riscv64_spl_na | 2 +-
> >  4 files changed, 4 insertions(+), 4 deletions(-)
>
> LGTM!
> Reviewed-by: Leo Yu-Chi Liang 
>

NAK!

It should be "virtio-rng-device"

Regards,
Bin


Re: [U-BOOT TEST HOOKS][PATCH 1/1] qemu-riscv: enable virtio-rng-pci

2023-07-31 Thread Leo Liang
On Mon, Jul 31, 2023 at 09:25:07AM +0200, Heinrich Schuchardt wrote:
> Linux' KASLR uses the EFI_RNG_PROTOCOL as entropy source. We should
> enable CONFIG_DM_RNG in U-Boot. For the EFI unit test for the protocol to
> succeed a virtio-rng-pci device has to be provided when invoking QEMU.
> 
> Reported-by: Leo Liang 
> Signed-off-by: Heinrich Schuchardt 
> ---
> See related patch
> [PATCH 1/1] riscv: qemu: imply CONFIG_DM_RNG
> https://lists.denx.de/pipermail/u-boot/2023-July/525293.html
> 
> The non-spl defconfigs still don't scan virtio devices automatically. This
> also needs to be addressed.
> ---
>  bin/travis-ci/conf.qemu-riscv32_na | 2 +-
>  bin/travis-ci/conf.qemu-riscv32_spl_na | 2 +-
>  bin/travis-ci/conf.qemu-riscv64_na | 2 +-
>  bin/travis-ci/conf.qemu-riscv64_spl_na | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)

LGTM!
Reviewed-by: Leo Yu-Chi Liang 

Best regards,
Leo