On Wed, 3 Jun 2020 11:37:45 +0200 Andrew Jones <drjo...@redhat.com> wrote:
> On Mon, Jun 01, 2020 at 03:04:33PM +0000, Salil Mehta wrote: > > Hello, > > I could see below within function fdt_add_pmu_nodes() part of > > hw/arm/virt.c during virt machine initialization time: ... > > > Q4. This function fdt_* looks to be wrongly named. The info > > being initialized here shall be used even when ACPI is > > being used. Initialization part and FDT info looked > > mixed up here if I am right? > > Agreed. The function has the wrong name. mach-virt has many functions that > mix the initialization and fdt building together, but those functions are > named something like create_foo(). Patches welcome. that was where I gave up on cpu hotplug arm/virt the last time. Ideally we should split out from create_foo() all firmware generation code (fdt) and move it to virt_machine_done time + make sure that it could be regenerated at reset time so guest would get updated FDT on reset. > > Thanks, > drew