* [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models @ 2019-07-22 18:23 Cédric Le Goater 2019-07-23 1:38 ` David Gibson 0 siblings, 1 reply; 6+ messages in thread From: Cédric Le Goater @ 2019-07-22 18:23 UTC (permalink / raw) To: David Gibson; +Cc: Cédric Le Goater, qemu-ppc, Greg Kurz, qemu-devel Make the current "powernv" machine an abstract type and derive from it new machines with specific CPU models: power8, power8e, power8nvl, power9. The "powernv" machine is now an alias on the "powernv9" machine. Signed-off-by: Cédric Le Goater <clg@kaod.org> --- hw/ppc/pnv.c | 97 +++++++++++++++++++++++++++++++++++++++++++++------- 1 file changed, 85 insertions(+), 12 deletions(-) diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c index 8cef6d17f111..36f57479a1f5 100644 --- a/hw/ppc/pnv.c +++ b/hw/ppc/pnv.c @@ -600,6 +600,7 @@ static void pnv_chip_power9_pic_print_info(PnvChip *chip, Monitor *mon) static void pnv_init(MachineState *machine) { PnvMachineState *pnv = PNV_MACHINE(machine); + MachineClass *mc = MACHINE_GET_CLASS(machine); MemoryRegion *ram; char *fw_filename; long fw_size; @@ -659,16 +660,25 @@ static void pnv_init(MachineState *machine) } } - /* Create the processor chips */ - i = strlen(machine->cpu_type) - strlen(POWERPC_CPU_TYPE_SUFFIX); - chip_typename = g_strdup_printf(PNV_CHIP_TYPE_NAME("%.*s"), - i, machine->cpu_type); - if (!object_class_by_name(chip_typename)) { - error_report("invalid CPU model '%.*s' for %s machine", - i, machine->cpu_type, MACHINE_GET_CLASS(machine)->name); + /* + * Do a sanity check on the specified CPU to check compatibility + * with the machine default. In the future, we might want to + * create the PnvChip with a compatible CPU model but for now, we + * use the machine default. + */ + if (object_class_by_name(mc->default_cpu_type) != + object_class_by_name(machine->cpu_type)) { + error_report("invalid CPU model '%s' for %s machine", + machine->cpu_type, mc->name); exit(1); } + /* Create the processor chips */ + i = strlen(mc->default_cpu_type) - strlen(POWERPC_CPU_TYPE_SUFFIX); + chip_typename = g_strdup_printf(PNV_CHIP_TYPE_NAME("%.*s"), + i, mc->default_cpu_type); + assert(object_class_by_name(chip_typename)); + pnv->chips = g_new0(PnvChip *, pnv->num_chips); for (i = 0; i < pnv->num_chips; i++) { char chip_name[32]; @@ -1343,25 +1353,69 @@ static void pnv_machine_class_props_init(ObjectClass *oc) NULL); } -static void pnv_machine_class_init(ObjectClass *oc, void *data) +static void pnv_machine_power8e_class_init(ObjectClass *oc, void *data) +{ + MachineClass *mc = MACHINE_CLASS(oc); + XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); + + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8E"; + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8e_v2.1"); + + xic->icp_get = pnv_icp_get; + xic->ics_get = pnv_ics_get; + xic->ics_resend = pnv_ics_resend; +} + +static void pnv_machine_power8_class_init(ObjectClass *oc, void *data) { MachineClass *mc = MACHINE_CLASS(oc); XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); + + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8"; + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0"); + + xic->icp_get = pnv_icp_get; + xic->ics_get = pnv_ics_get; + xic->ics_resend = pnv_ics_resend; +} + +static void pnv_machine_power8nvl_class_init(ObjectClass *oc, void *data) +{ + MachineClass *mc = MACHINE_CLASS(oc); + XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); + + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8NVL"; + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8nvl_v1.0"); + + xic->icp_get = pnv_icp_get; + xic->ics_get = pnv_ics_get; + xic->ics_resend = pnv_ics_resend; +} + +static void pnv_machine_power9_class_init(ObjectClass *oc, void *data) +{ + MachineClass *mc = MACHINE_CLASS(oc); + + mc->desc = "IBM PowerNV (Non-Virtualized) POWER9"; + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power9_v2.0"); + + mc->alias = "powernv"; +} + +static void pnv_machine_class_init(ObjectClass *oc, void *data) +{ + MachineClass *mc = MACHINE_CLASS(oc); InterruptStatsProviderClass *ispc = INTERRUPT_STATS_PROVIDER_CLASS(oc); mc->desc = "IBM PowerNV (Non-Virtualized)"; mc->init = pnv_init; mc->reset = pnv_reset; mc->max_cpus = MAX_CPUS; - mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0"); mc->block_default_type = IF_IDE; /* Pnv provides a AHCI device for * storage */ mc->no_parallel = 1; mc->default_boot_order = NULL; mc->default_ram_size = 2 * GiB; - xic->icp_get = pnv_icp_get; - xic->ics_get = pnv_ics_get; - xic->ics_resend = pnv_ics_resend; ispc->print_info = pnv_pic_print_info; pnv_machine_class_props_init(oc); @@ -1381,10 +1435,29 @@ static void pnv_machine_class_init(ObjectClass *oc, void *data) .parent = TYPE_PNV9_CHIP, \ } +#define DEFINE_PNV_MACHINE_TYPE(cpu, class_initfn) \ + { \ + .name = MACHINE_TYPE_NAME(cpu), \ + .parent = TYPE_PNV_MACHINE, \ + .instance_size = sizeof(PnvMachineState), \ + .instance_init = pnv_machine_instance_init, \ + .class_init = class_initfn, \ + .interfaces = (InterfaceInfo[]) { \ + { TYPE_XICS_FABRIC }, \ + { TYPE_INTERRUPT_STATS_PROVIDER }, \ + { }, \ + }, \ + } + static const TypeInfo types[] = { + DEFINE_PNV_MACHINE_TYPE("powernv8e", pnv_machine_power8e_class_init), + DEFINE_PNV_MACHINE_TYPE("powernv8", pnv_machine_power8_class_init), + DEFINE_PNV_MACHINE_TYPE("powernv8nvl", pnv_machine_power8nvl_class_init), + DEFINE_PNV_MACHINE_TYPE("powernv9", pnv_machine_power9_class_init), { .name = TYPE_PNV_MACHINE, .parent = TYPE_MACHINE, + .abstract = true, .instance_size = sizeof(PnvMachineState), .instance_init = pnv_machine_instance_init, .class_init = pnv_machine_class_init, -- 2.21.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models 2019-07-22 18:23 [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models Cédric Le Goater @ 2019-07-23 1:38 ` David Gibson 2019-07-23 6:00 ` Cédric Le Goater 0 siblings, 1 reply; 6+ messages in thread From: David Gibson @ 2019-07-23 1:38 UTC (permalink / raw) To: Cédric Le Goater; +Cc: qemu-ppc, Greg Kurz, qemu-devel [-- Attachment #1: Type: text/plain, Size: 7353 bytes --] On Mon, Jul 22, 2019 at 08:23:47PM +0200, Cédric Le Goater wrote: > Make the current "powernv" machine an abstract type and derive from it > new machines with specific CPU models: power8, power8e, power8nvl, > power9. > > The "powernv" machine is now an alias on the "powernv9" machine. > > Signed-off-by: Cédric Le Goater <clg@kaod.org> Ah, sorry, I wasn't clear here. I don't think we need a different machine type for every cpu model, I just think we should have powernv8 and powernv9. POWER8E and POWER8NVL don't significantly change the system design (IIUC) so they can still be done with "-machine powernv8 -cpu POWER8E" or whatever. I expect the same will be true for POWER9' when that comes along > --- > hw/ppc/pnv.c | 97 +++++++++++++++++++++++++++++++++++++++++++++------- > 1 file changed, 85 insertions(+), 12 deletions(-) > > diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c > index 8cef6d17f111..36f57479a1f5 100644 > --- a/hw/ppc/pnv.c > +++ b/hw/ppc/pnv.c > @@ -600,6 +600,7 @@ static void pnv_chip_power9_pic_print_info(PnvChip *chip, Monitor *mon) > static void pnv_init(MachineState *machine) > { > PnvMachineState *pnv = PNV_MACHINE(machine); > + MachineClass *mc = MACHINE_GET_CLASS(machine); > MemoryRegion *ram; > char *fw_filename; > long fw_size; > @@ -659,16 +660,25 @@ static void pnv_init(MachineState *machine) > } > } > > - /* Create the processor chips */ > - i = strlen(machine->cpu_type) - strlen(POWERPC_CPU_TYPE_SUFFIX); > - chip_typename = g_strdup_printf(PNV_CHIP_TYPE_NAME("%.*s"), > - i, machine->cpu_type); > - if (!object_class_by_name(chip_typename)) { > - error_report("invalid CPU model '%.*s' for %s machine", > - i, machine->cpu_type, MACHINE_GET_CLASS(machine)->name); > + /* > + * Do a sanity check on the specified CPU to check compatibility > + * with the machine default. In the future, we might want to > + * create the PnvChip with a compatible CPU model but for now, we > + * use the machine default. > + */ > + if (object_class_by_name(mc->default_cpu_type) != > + object_class_by_name(machine->cpu_type)) { > + error_report("invalid CPU model '%s' for %s machine", > + machine->cpu_type, mc->name); > exit(1); > } > > + /* Create the processor chips */ > + i = strlen(mc->default_cpu_type) - strlen(POWERPC_CPU_TYPE_SUFFIX); > + chip_typename = g_strdup_printf(PNV_CHIP_TYPE_NAME("%.*s"), > + i, mc->default_cpu_type); > + assert(object_class_by_name(chip_typename)); > + > pnv->chips = g_new0(PnvChip *, pnv->num_chips); > for (i = 0; i < pnv->num_chips; i++) { > char chip_name[32]; > @@ -1343,25 +1353,69 @@ static void pnv_machine_class_props_init(ObjectClass *oc) > NULL); > } > > -static void pnv_machine_class_init(ObjectClass *oc, void *data) > +static void pnv_machine_power8e_class_init(ObjectClass *oc, void *data) > +{ > + MachineClass *mc = MACHINE_CLASS(oc); > + XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); > + > + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8E"; > + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8e_v2.1"); > + > + xic->icp_get = pnv_icp_get; > + xic->ics_get = pnv_ics_get; > + xic->ics_resend = pnv_ics_resend; > +} > + > +static void pnv_machine_power8_class_init(ObjectClass *oc, void *data) > { > MachineClass *mc = MACHINE_CLASS(oc); > XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); > + > + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8"; > + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0"); > + > + xic->icp_get = pnv_icp_get; > + xic->ics_get = pnv_ics_get; > + xic->ics_resend = pnv_ics_resend; > +} > + > +static void pnv_machine_power8nvl_class_init(ObjectClass *oc, void *data) > +{ > + MachineClass *mc = MACHINE_CLASS(oc); > + XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); > + > + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8NVL"; > + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8nvl_v1.0"); > + > + xic->icp_get = pnv_icp_get; > + xic->ics_get = pnv_ics_get; > + xic->ics_resend = pnv_ics_resend; > +} > + > +static void pnv_machine_power9_class_init(ObjectClass *oc, void *data) > +{ > + MachineClass *mc = MACHINE_CLASS(oc); > + > + mc->desc = "IBM PowerNV (Non-Virtualized) POWER9"; > + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power9_v2.0"); > + > + mc->alias = "powernv"; > +} > + > +static void pnv_machine_class_init(ObjectClass *oc, void *data) > +{ > + MachineClass *mc = MACHINE_CLASS(oc); > InterruptStatsProviderClass *ispc = INTERRUPT_STATS_PROVIDER_CLASS(oc); > > mc->desc = "IBM PowerNV (Non-Virtualized)"; > mc->init = pnv_init; > mc->reset = pnv_reset; > mc->max_cpus = MAX_CPUS; > - mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0"); > mc->block_default_type = IF_IDE; /* Pnv provides a AHCI device for > * storage */ > mc->no_parallel = 1; > mc->default_boot_order = NULL; > mc->default_ram_size = 2 * GiB; > - xic->icp_get = pnv_icp_get; > - xic->ics_get = pnv_ics_get; > - xic->ics_resend = pnv_ics_resend; > ispc->print_info = pnv_pic_print_info; > > pnv_machine_class_props_init(oc); > @@ -1381,10 +1435,29 @@ static void pnv_machine_class_init(ObjectClass *oc, void *data) > .parent = TYPE_PNV9_CHIP, \ > } > > +#define DEFINE_PNV_MACHINE_TYPE(cpu, class_initfn) \ > + { \ > + .name = MACHINE_TYPE_NAME(cpu), \ > + .parent = TYPE_PNV_MACHINE, \ > + .instance_size = sizeof(PnvMachineState), \ > + .instance_init = pnv_machine_instance_init, \ > + .class_init = class_initfn, \ > + .interfaces = (InterfaceInfo[]) { \ > + { TYPE_XICS_FABRIC }, \ > + { TYPE_INTERRUPT_STATS_PROVIDER }, \ > + { }, \ > + }, \ > + } > + > static const TypeInfo types[] = { > + DEFINE_PNV_MACHINE_TYPE("powernv8e", pnv_machine_power8e_class_init), > + DEFINE_PNV_MACHINE_TYPE("powernv8", pnv_machine_power8_class_init), > + DEFINE_PNV_MACHINE_TYPE("powernv8nvl", pnv_machine_power8nvl_class_init), > + DEFINE_PNV_MACHINE_TYPE("powernv9", pnv_machine_power9_class_init), > { > .name = TYPE_PNV_MACHINE, > .parent = TYPE_MACHINE, > + .abstract = true, > .instance_size = sizeof(PnvMachineState), > .instance_init = pnv_machine_instance_init, > .class_init = pnv_machine_class_init, -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models 2019-07-23 1:38 ` David Gibson @ 2019-07-23 6:00 ` Cédric Le Goater 2019-07-23 6:37 ` David Gibson 0 siblings, 1 reply; 6+ messages in thread From: Cédric Le Goater @ 2019-07-23 6:00 UTC (permalink / raw) To: David Gibson; +Cc: qemu-ppc, Greg Kurz, qemu-devel On 23/07/2019 03:38, David Gibson wrote: > On Mon, Jul 22, 2019 at 08:23:47PM +0200, Cédric Le Goater wrote: >> Make the current "powernv" machine an abstract type and derive from it >> new machines with specific CPU models: power8, power8e, power8nvl, >> power9. >> >> The "powernv" machine is now an alias on the "powernv9" machine. >> >> Signed-off-by: Cédric Le Goater <clg@kaod.org> > > Ah, sorry, I wasn't clear here. I don't think we need a different > machine type for every cpu model, I just think we should have powernv8 > and powernv9. POWER8E and POWER8NVL don't significantly change the > system design (IIUC) so they can still be done with "-machine powernv8 > -cpu POWER8E" or whatever. I expect the same will be true for POWER9' > when that comes along I understand but I am afraid we will to have one machine per CPU family because POWER8E and POWER8NVL already have their own PnvChip : DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8, pnv_chip_power8_class_init), DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8E, pnv_chip_power8e_class_init), DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8NVL, pnv_chip_power8nvl_class_init), C. >> --- >> hw/ppc/pnv.c | 97 +++++++++++++++++++++++++++++++++++++++++++++------- >> 1 file changed, 85 insertions(+), 12 deletions(-) >> >> diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c >> index 8cef6d17f111..36f57479a1f5 100644 >> --- a/hw/ppc/pnv.c >> +++ b/hw/ppc/pnv.c >> @@ -600,6 +600,7 @@ static void pnv_chip_power9_pic_print_info(PnvChip *chip, Monitor *mon) >> static void pnv_init(MachineState *machine) >> { >> PnvMachineState *pnv = PNV_MACHINE(machine); >> + MachineClass *mc = MACHINE_GET_CLASS(machine); >> MemoryRegion *ram; >> char *fw_filename; >> long fw_size; >> @@ -659,16 +660,25 @@ static void pnv_init(MachineState *machine) >> } >> } >> >> - /* Create the processor chips */ >> - i = strlen(machine->cpu_type) - strlen(POWERPC_CPU_TYPE_SUFFIX); >> - chip_typename = g_strdup_printf(PNV_CHIP_TYPE_NAME("%.*s"), >> - i, machine->cpu_type); >> - if (!object_class_by_name(chip_typename)) { >> - error_report("invalid CPU model '%.*s' for %s machine", >> - i, machine->cpu_type, MACHINE_GET_CLASS(machine)->name); >> + /* >> + * Do a sanity check on the specified CPU to check compatibility >> + * with the machine default. In the future, we might want to >> + * create the PnvChip with a compatible CPU model but for now, we >> + * use the machine default. >> + */ >> + if (object_class_by_name(mc->default_cpu_type) != >> + object_class_by_name(machine->cpu_type)) { >> + error_report("invalid CPU model '%s' for %s machine", >> + machine->cpu_type, mc->name); >> exit(1); >> } >> >> + /* Create the processor chips */ >> + i = strlen(mc->default_cpu_type) - strlen(POWERPC_CPU_TYPE_SUFFIX); >> + chip_typename = g_strdup_printf(PNV_CHIP_TYPE_NAME("%.*s"), >> + i, mc->default_cpu_type); >> + assert(object_class_by_name(chip_typename)); >> + >> pnv->chips = g_new0(PnvChip *, pnv->num_chips); >> for (i = 0; i < pnv->num_chips; i++) { >> char chip_name[32]; >> @@ -1343,25 +1353,69 @@ static void pnv_machine_class_props_init(ObjectClass *oc) >> NULL); >> } >> >> -static void pnv_machine_class_init(ObjectClass *oc, void *data) >> +static void pnv_machine_power8e_class_init(ObjectClass *oc, void *data) >> +{ >> + MachineClass *mc = MACHINE_CLASS(oc); >> + XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); >> + >> + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8E"; >> + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8e_v2.1"); >> + >> + xic->icp_get = pnv_icp_get; >> + xic->ics_get = pnv_ics_get; >> + xic->ics_resend = pnv_ics_resend; >> +} >> + >> +static void pnv_machine_power8_class_init(ObjectClass *oc, void *data) >> { >> MachineClass *mc = MACHINE_CLASS(oc); >> XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); >> + >> + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8"; >> + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0"); >> + >> + xic->icp_get = pnv_icp_get; >> + xic->ics_get = pnv_ics_get; >> + xic->ics_resend = pnv_ics_resend; >> +} >> + >> +static void pnv_machine_power8nvl_class_init(ObjectClass *oc, void *data) >> +{ >> + MachineClass *mc = MACHINE_CLASS(oc); >> + XICSFabricClass *xic = XICS_FABRIC_CLASS(oc); >> + >> + mc->desc = "IBM PowerNV (Non-Virtualized) POWER8NVL"; >> + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8nvl_v1.0"); >> + >> + xic->icp_get = pnv_icp_get; >> + xic->ics_get = pnv_ics_get; >> + xic->ics_resend = pnv_ics_resend; >> +} >> + >> +static void pnv_machine_power9_class_init(ObjectClass *oc, void *data) >> +{ >> + MachineClass *mc = MACHINE_CLASS(oc); >> + >> + mc->desc = "IBM PowerNV (Non-Virtualized) POWER9"; >> + mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power9_v2.0"); >> + >> + mc->alias = "powernv"; >> +} >> + >> +static void pnv_machine_class_init(ObjectClass *oc, void *data) >> +{ >> + MachineClass *mc = MACHINE_CLASS(oc); >> InterruptStatsProviderClass *ispc = INTERRUPT_STATS_PROVIDER_CLASS(oc); >> >> mc->desc = "IBM PowerNV (Non-Virtualized)"; >> mc->init = pnv_init; >> mc->reset = pnv_reset; >> mc->max_cpus = MAX_CPUS; >> - mc->default_cpu_type = POWERPC_CPU_TYPE_NAME("power8_v2.0"); >> mc->block_default_type = IF_IDE; /* Pnv provides a AHCI device for >> * storage */ >> mc->no_parallel = 1; >> mc->default_boot_order = NULL; >> mc->default_ram_size = 2 * GiB; >> - xic->icp_get = pnv_icp_get; >> - xic->ics_get = pnv_ics_get; >> - xic->ics_resend = pnv_ics_resend; >> ispc->print_info = pnv_pic_print_info; >> >> pnv_machine_class_props_init(oc); >> @@ -1381,10 +1435,29 @@ static void pnv_machine_class_init(ObjectClass *oc, void *data) >> .parent = TYPE_PNV9_CHIP, \ >> } >> >> +#define DEFINE_PNV_MACHINE_TYPE(cpu, class_initfn) \ >> + { \ >> + .name = MACHINE_TYPE_NAME(cpu), \ >> + .parent = TYPE_PNV_MACHINE, \ >> + .instance_size = sizeof(PnvMachineState), \ >> + .instance_init = pnv_machine_instance_init, \ >> + .class_init = class_initfn, \ >> + .interfaces = (InterfaceInfo[]) { \ >> + { TYPE_XICS_FABRIC }, \ >> + { TYPE_INTERRUPT_STATS_PROVIDER }, \ >> + { }, \ >> + }, \ >> + } >> + >> static const TypeInfo types[] = { >> + DEFINE_PNV_MACHINE_TYPE("powernv8e", pnv_machine_power8e_class_init), >> + DEFINE_PNV_MACHINE_TYPE("powernv8", pnv_machine_power8_class_init), >> + DEFINE_PNV_MACHINE_TYPE("powernv8nvl", pnv_machine_power8nvl_class_init), >> + DEFINE_PNV_MACHINE_TYPE("powernv9", pnv_machine_power9_class_init), >> { >> .name = TYPE_PNV_MACHINE, >> .parent = TYPE_MACHINE, >> + .abstract = true, >> .instance_size = sizeof(PnvMachineState), >> .instance_init = pnv_machine_instance_init, >> .class_init = pnv_machine_class_init, > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models 2019-07-23 6:00 ` Cédric Le Goater @ 2019-07-23 6:37 ` David Gibson 2019-07-24 9:32 ` Cédric Le Goater 0 siblings, 1 reply; 6+ messages in thread From: David Gibson @ 2019-07-23 6:37 UTC (permalink / raw) To: Cédric Le Goater; +Cc: qemu-ppc, Greg Kurz, qemu-devel [-- Attachment #1: Type: text/plain, Size: 1688 bytes --] On Tue, Jul 23, 2019 at 08:00:27AM +0200, Cédric Le Goater wrote: > On 23/07/2019 03:38, David Gibson wrote: > > On Mon, Jul 22, 2019 at 08:23:47PM +0200, Cédric Le Goater wrote: > >> Make the current "powernv" machine an abstract type and derive from it > >> new machines with specific CPU models: power8, power8e, power8nvl, > >> power9. > >> > >> The "powernv" machine is now an alias on the "powernv9" machine. > >> > >> Signed-off-by: Cédric Le Goater <clg@kaod.org> > > > > Ah, sorry, I wasn't clear here. I don't think we need a different > > machine type for every cpu model, I just think we should have powernv8 > > and powernv9. POWER8E and POWER8NVL don't significantly change the > > system design (IIUC) so they can still be done with "-machine powernv8 > > -cpu POWER8E" or whatever. I expect the same will be true for POWER9' > > when that comes along > > I understand but I am afraid we will to have one machine per CPU family > because POWER8E and POWER8NVL already have their own PnvChip : > > DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8, pnv_chip_power8_class_init), > DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8E, pnv_chip_power8e_class_init), > DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8NVL, > pnv_chip_power8nvl_class_init), Hrm. Is there an actual reason we need different chip classes for these? Even if there is, I don't see an inherent reason that implies separate machine classes as well. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models 2019-07-23 6:37 ` David Gibson @ 2019-07-24 9:32 ` Cédric Le Goater 2019-07-25 3:27 ` David Gibson 0 siblings, 1 reply; 6+ messages in thread From: Cédric Le Goater @ 2019-07-24 9:32 UTC (permalink / raw) To: David Gibson; +Cc: qemu-ppc, Greg Kurz, qemu-devel On 23/07/2019 08:37, David Gibson wrote: > On Tue, Jul 23, 2019 at 08:00:27AM +0200, Cédric Le Goater wrote: >> On 23/07/2019 03:38, David Gibson wrote: >>> On Mon, Jul 22, 2019 at 08:23:47PM +0200, Cédric Le Goater wrote: >>>> Make the current "powernv" machine an abstract type and derive from it >>>> new machines with specific CPU models: power8, power8e, power8nvl, >>>> power9. >>>> >>>> The "powernv" machine is now an alias on the "powernv9" machine. >>>> >>>> Signed-off-by: Cédric Le Goater <clg@kaod.org> >>> >>> Ah, sorry, I wasn't clear here. I don't think we need a different >>> machine type for every cpu model, I just think we should have powernv8 >>> and powernv9. POWER8E and POWER8NVL don't significantly change the >>> system design (IIUC) so they can still be done with "-machine powernv8 >>> -cpu POWER8E" or whatever. I expect the same will be true for POWER9' >>> when that comes along >> >> I understand but I am afraid we will to have one machine per CPU family >> because POWER8E and POWER8NVL already have their own PnvChip : >> >> DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8, pnv_chip_power8_class_init), >> DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8E, pnv_chip_power8e_class_init), >> DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8NVL, >> pnv_chip_power8nvl_class_init), > > Hrm. Is there an actual reason we need different chip classes for > these? The POWER8 (8E, 8, 8NVL) chips differ by the core : target/ppc/cpu-models.h: CPU_POWERPC_POWER8E_BASE = 0x004B0000, target/ppc/cpu-models.h: CPU_POWERPC_POWER8_BASE = 0x004D0000, target/ppc/cpu-models.h: CPU_POWERPC_POWER8NVL_BASE = 0x004C0000, The chip model is different : k->chip_cfam_id = 0x221ef04980000000ull; /* P8 Murano DD2.1 */ k->chip_cfam_id = 0x220ea04980000000ull; /* P8 Venice DD2.0 */ k->chip_cfam_id = 0x120d304980000000ull; /* P8 Naples DD1.0 */ The chiplets are different also. NVL has a NPU and a different LPC bus which is modeled. There are different number of PHBs. > Even if there is, I don't see an inherent reason that implies separate > machine classes as well. I will see if we can have a single powernv8 machine. C. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models 2019-07-24 9:32 ` Cédric Le Goater @ 2019-07-25 3:27 ` David Gibson 0 siblings, 0 replies; 6+ messages in thread From: David Gibson @ 2019-07-25 3:27 UTC (permalink / raw) To: Cédric Le Goater; +Cc: qemu-ppc, Greg Kurz, qemu-devel [-- Attachment #1: Type: text/plain, Size: 2690 bytes --] On Wed, Jul 24, 2019 at 11:32:00AM +0200, Cédric Le Goater wrote: > On 23/07/2019 08:37, David Gibson wrote: > > On Tue, Jul 23, 2019 at 08:00:27AM +0200, Cédric Le Goater wrote: > >> On 23/07/2019 03:38, David Gibson wrote: > >>> On Mon, Jul 22, 2019 at 08:23:47PM +0200, Cédric Le Goater wrote: > >>>> Make the current "powernv" machine an abstract type and derive from it > >>>> new machines with specific CPU models: power8, power8e, power8nvl, > >>>> power9. > >>>> > >>>> The "powernv" machine is now an alias on the "powernv9" machine. > >>>> > >>>> Signed-off-by: Cédric Le Goater <clg@kaod.org> > >>> > >>> Ah, sorry, I wasn't clear here. I don't think we need a different > >>> machine type for every cpu model, I just think we should have powernv8 > >>> and powernv9. POWER8E and POWER8NVL don't significantly change the > >>> system design (IIUC) so they can still be done with "-machine powernv8 > >>> -cpu POWER8E" or whatever. I expect the same will be true for POWER9' > >>> when that comes along > >> > >> I understand but I am afraid we will to have one machine per CPU family > >> because POWER8E and POWER8NVL already have their own PnvChip : > >> > >> DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8, pnv_chip_power8_class_init), > >> DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8E, pnv_chip_power8e_class_init), > >> DEFINE_PNV8_CHIP_TYPE(TYPE_PNV_CHIP_POWER8NVL, > >> pnv_chip_power8nvl_class_init), > > > > Hrm. Is there an actual reason we need different chip classes for > > these? > The POWER8 (8E, 8, 8NVL) chips differ by the core : > > target/ppc/cpu-models.h: CPU_POWERPC_POWER8E_BASE = 0x004B0000, > target/ppc/cpu-models.h: CPU_POWERPC_POWER8_BASE = 0x004D0000, > target/ppc/cpu-models.h: CPU_POWERPC_POWER8NVL_BASE = 0x004C0000, > > The chip model is different : > > k->chip_cfam_id = 0x221ef04980000000ull; /* P8 Murano DD2.1 */ > k->chip_cfam_id = 0x220ea04980000000ull; /* P8 Venice DD2.0 */ > k->chip_cfam_id = 0x120d304980000000ull; /* P8 Naples DD1.0 */ > > The chiplets are different also. NVL has a NPU and a different LPC bus which > is modeled. There are different number of PHBs. Ah, ok. That seems like enough reason. > > > Even if there is, I don't see an inherent reason that implies separate > > machine classes as well. > > I will see if we can have a single powernv8 machine. > > C. > -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-07-26 7:30 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-07-22 18:23 [Qemu-devel] [PATCH] ppc/pnv: Introduce PowerNV machines with fixed CPU models Cédric Le Goater 2019-07-23 1:38 ` David Gibson 2019-07-23 6:00 ` Cédric Le Goater 2019-07-23 6:37 ` David Gibson 2019-07-24 9:32 ` Cédric Le Goater 2019-07-25 3:27 ` David Gibson
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.