[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 05/10] x86: disable rng seeding via setup_data
From: |
Michael S. Tsirkin |
Subject: |
Re: [PULL 05/10] x86: disable rng seeding via setup_data |
Date: |
Thu, 18 Aug 2022 05:54:40 -0400 |
On Thu, Aug 18, 2022 at 11:27:30AM +0200, Paolo Bonzini wrote:
> On 8/17/22 18:14, Michael S. Tsirkin wrote:
> > @@ -332,7 +332,7 @@ static void microvm_memory_init(MicrovmMachineState
> > *mms)
> > rom_set_fw(fw_cfg);
> > if (machine->kernel_filename != NULL) {
> > - x86_load_linux(x86ms, fw_cfg, 0, true, false);
> > + x86_load_linux(x86ms, fw_cfg, 0, true, true);
> > }
> > if (mms->option_roms) {
> > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
> > index a5c65c1c35..20962c34e7 100644
> > --- a/hw/i386/pc_piix.c
> > +++ b/hw/i386/pc_piix.c
> > @@ -439,6 +439,7 @@ static void pc_i440fx_7_1_machine_options(MachineClass
> > *m)
> > m->alias = "pc";
> > m->is_default = true;
> > pcmc->default_cpu_version = 1;
> > + pcmc->legacy_no_rng_seed = true;
> > }
> > DEFINE_I440FX_MACHINE(v7_1, "pc-i440fx-7.1", NULL,
> > @@ -450,7 +451,6 @@ static void pc_i440fx_7_0_machine_options(MachineClass
> > *m)
> > pc_i440fx_7_1_machine_options(m);
> > m->alias = NULL;
> > m->is_default = false;
> > - pcmc->legacy_no_rng_seed = true;
> > pcmc->enforce_amd_1tb_hole = false;
> > compat_props_add(m->compat_props, hw_compat_7_0, hw_compat_7_0_len);
> > compat_props_add(m->compat_props, pc_compat_7_0, pc_compat_7_0_len);
> > diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c
> > index 3a35193ff7..2e5dae9a89 100644
> > --- a/hw/i386/pc_q35.c
> > +++ b/hw/i386/pc_q35.c
> > @@ -376,6 +376,7 @@ static void pc_q35_7_1_machine_options(MachineClass *m)
> > pc_q35_machine_options(m);
> > m->alias = "q35";
> > pcmc->default_cpu_version = 1;
> > + pcmc->legacy_no_rng_seed = true;
> > }
> > DEFINE_Q35_MACHINE(v7_1, "pc-q35-7.1", NULL,
> > @@ -386,7 +387,6 @@ static void pc_q35_7_0_machine_options(MachineClass *m)
> > PCMachineClass *pcmc = PC_MACHINE_CLASS(m);
> > pc_q35_7_1_machine_options(m);
> > m->alias = NULL;
> > - pcmc->legacy_no_rng_seed = true;
> > pcmc->enforce_amd_1tb_hole = false;
> > compat_props_add(m->compat_props, hw_compat_7_0, hw_compat_7_0_len);
> > compat_props_add(m->compat_props, pc_compat_7_0, pc_compat_7_0_len);
>
> Why not just revert the whole patch?
>
> Paolo
At this point I was looking for a minimally intrusive change.
--
MST
- [PULL 00/10] pc,virtio: fixes, Michael S. Tsirkin, 2022/08/17
- [PULL 01/10] virtio-scsi: fix race in virtio_scsi_dataplane_start(), Michael S. Tsirkin, 2022/08/17
- [PULL 03/10] hw/virtio: handle un-configured shutdown in virtio-pci, Michael S. Tsirkin, 2022/08/17
- [PULL 02/10] hw/virtio: gracefully handle unset vhost_dev vdev, Michael S. Tsirkin, 2022/08/17
- [PULL 04/10] hw/virtio: fix vhost_user_read tracepoint, Michael S. Tsirkin, 2022/08/17
- [PULL 10/10] virtio-pci: don't touch pci on virtio reset, Michael S. Tsirkin, 2022/08/17
- [PULL 06/10] hw/cxl: Add stub write function for RO MemoryRegionOps entries., Michael S. Tsirkin, 2022/08/17
- [PULL 05/10] x86: disable rng seeding via setup_data, Michael S. Tsirkin, 2022/08/17
- [PULL 07/10] hw/cxl: Fix Get LSA input payload size which should be 8 bytes., Michael S. Tsirkin, 2022/08/17
- [PULL 08/10] hw/cxl: Correctly handle variable sized mailbox input payloads., Michael S. Tsirkin, 2022/08/17
- [PULL 09/10] tests: acpi: silence applesmc warning about invalid key, Michael S. Tsirkin, 2022/08/17