[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for 2.7] ppc: parse cpu features once
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH for 2.7] ppc: parse cpu features once |
Date: |
Thu, 11 Aug 2016 17:14:29 +1000 |
User-agent: |
Mutt/1.6.2 (2016-07-01) |
On Wed, Aug 10, 2016 at 09:08:01PM +0200, Cédric Le Goater wrote:
> From: Greg Kurz <address@hidden>
>
> Considering that features are converted to global properties and
> global properties are automatically applied to every new instance
> of created CPU (at object_new() time), there is no point in
> parsing cpu_model string every time a CPU created. So move
> parsing outside CPU creation loop and do it only once.
>
> Parsing also should be done before any CPU is created so that
> features would affect the first CPU a well.
>
> This patch does that for all PowerPC machine types.
>
> It is based on previous work from Bharata:
>
> https://lists.nongnu.org/archive/html/qemu-devel/2016-06/msg07564.html
>
> Signed-off-by: Greg Kurz <address@hidden>
> [clg: only kept the fix for the spapr platform. support for other
> platform will be added in 2.8 ]
> Signed-off-by: Cédric Le Goater <address@hidden>
>
> ---
>
> This is a standalone version of patch 2 from my latest series:
>
> https://lists.nongnu.org/archive/html/qemu-devel/2016-07/msg02433.html
>
> The only changes are trivial conflict fixes caused by the introduction
> of ppc_cpu_init by patch 1.
>
> ---
> hw/ppc/ppc.c | 26 ++++++++++++++++++++++++++
> hw/ppc/spapr.c | 2 ++
> include/hw/ppc/ppc.h | 1 +
> 3 files changed, 29 insertions(+)
Applied to ppc-for-2.7.
>
> Index: qemu-dgibson-for-2.8.git/hw/ppc/ppc.c
> ===================================================================
> --- qemu-dgibson-for-2.8.git.orig/hw/ppc/ppc.c
> +++ qemu-dgibson-for-2.8.git/hw/ppc/ppc.c
> @@ -33,6 +33,7 @@
> #include "hw/timer/m48t59.h"
> #include "qemu/log.h"
> #include "qemu/error-report.h"
> +#include "qapi/error.h"
> #include "hw/loader.h"
> #include "sysemu/kvm.h"
> #include "kvm_ppc.h"
> @@ -1350,3 +1351,28 @@ PowerPCCPU *ppc_get_vcpu_by_dt_id(int cp
>
> return NULL;
> }
> +
> +void ppc_cpu_parse_features(const char *cpu_model)
> +{
> + CPUClass *cc;
> + ObjectClass *oc;
> + const char *typename;
> + gchar **model_pieces;
> +
> + model_pieces = g_strsplit(cpu_model, ",", 2);
> + if (!model_pieces[0]) {
> + error_report("Invalid/empty CPU model name");
> + exit(1);
> + }
> +
> + oc = cpu_class_by_name(TYPE_POWERPC_CPU, model_pieces[0]);
> + if (oc == NULL) {
> + error_report("Unable to find CPU definition: %s", model_pieces[0]);
> + exit(1);
> + }
> +
> + typename = object_class_get_name(oc);
> + cc = CPU_CLASS(oc);
> + cc->parse_features(typename, model_pieces[1], &error_fatal);
> + g_strfreev(model_pieces);
> +}
> Index: qemu-dgibson-for-2.8.git/hw/ppc/spapr.c
> ===================================================================
> --- qemu-dgibson-for-2.8.git.orig/hw/ppc/spapr.c
> +++ qemu-dgibson-for-2.8.git/hw/ppc/spapr.c
> @@ -1769,6 +1769,8 @@ static void ppc_spapr_init(MachineState
> machine->cpu_model = kvm_enabled() ? "host" : "POWER7";
> }
>
> + ppc_cpu_parse_features(machine->cpu_model);
> +
> if (mc->query_hotpluggable_cpus) {
> char *type = spapr_get_cpu_core_type(machine->cpu_model);
>
> Index: qemu-dgibson-for-2.8.git/include/hw/ppc/ppc.h
> ===================================================================
> --- qemu-dgibson-for-2.8.git.orig/include/hw/ppc/ppc.h
> +++ qemu-dgibson-for-2.8.git/include/hw/ppc/ppc.h
> @@ -106,4 +106,5 @@ enum {
> /* ppc_booke.c */
> void ppc_booke_timers_init(PowerPCCPU *cpu, uint32_t freq, uint32_t flags);
>
> +void ppc_cpu_parse_features(const char *cpu_model);
> #endif
>
--
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
signature.asc
Description: PGP signature