On Mon, Dec 07, 2015 at 05:24:27PM +0200, Marcel Apfelbaum wrote: > On 12/02/2015 03:46 AM, Eduardo Habkost wrote: > >Lots of PcGuestInfo fields are duplicates of PCMachineClass or > >PCMachineState fields. Pass PCMachineState as argument to > >acpi_setup(), so we can simply let the ACPI code use those fields > >directly. > > I completely agree with removing duplicated fields and using PCMachine > fields directly, but this not what this patch does. > It only extracts PcGuestInfo info from the machine.
I should have appended "later" to the commit message. "So we are able to simply let the ACPI code use those fields later (in another commit)". The goal of this commit is to just change the function signature to allow us to move the fields later. Maybe I will squash some of those changes together in a new version of the series. -- Eduardo
