On Sun, Apr 23, 2023 at 11:39 PM Yong Li <yong...@intel.com> wrote: > > The firmware may be specified with -bios > and the plfash0 device with option -drive if=pflash. > If both options are applied, it is intented that the pflash0 will > store the secure variable and the firmware be the one specified > by -bios. Explicitly specify "-bios none" if choose to boot from > pflash0
This seems like the right approach. Can you update the docs/system/riscv/virt.rst docs to include this information? Alistair > > Signed-off-by: Yong Li <yong...@intel.com> > Cc: "Zhiwei Liu" <zhiwei_...@linux.alibaba.com> > --- > hw/riscv/virt.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/hw/riscv/virt.c b/hw/riscv/virt.c > index 4e3efbee16..b38b41e685 100644 > --- a/hw/riscv/virt.c > +++ b/hw/riscv/virt.c > @@ -1296,10 +1296,12 @@ static void virt_machine_done(Notifier *notifier, > void *data) > kernel_entry = 0; > } > > - if (drive_get(IF_PFLASH, 0, 0)) { > + if (drive_get(IF_PFLASH, 0, 0) && > + !strcmp(machine->firmware, "none")) { > /* > - * Pflash was supplied, let's overwrite the address we jump to after > - * reset to the base of the flash. > + * If pflash (unit 0) was supplied and at the same time the -bois > + * is not specified, then let's overwrite the address we jump to > + * after reset to the base of the flash. > */ > start_addr = virt_memmap[VIRT_FLASH].base; > } > -- > 2.25.1 > >