On Mon, Feb 10, 2014 at 01:23:37AM +0100, Andreas Färber wrote: [...]
/** * X86CPUClass: * @parent_realize: The parent class' realize handler. @@ -49,6 +52,16 @@ typedef struct X86CPUClass { CPUClass parent_class; /*< public >*/
+ /* CPU model definition + * Should be eventually replaced by subclass-specific property defaults + */ + X86CPUDefinition *cpu_def; + /* CPU model requires KVM to be enabled */ + bool kvm_required; + /* Optional description of CPU model. + * If unavailable, cpu_def->model_id is used */ + const char *model_description;
Here I wondered why you needed this? For PowerPCCPU subclasses we have reused DeviceClass::desc.
I was not aware of DeviceClass::desc. We can use it instead. Do you prefer a respin, or an additional patch? -- Eduardo