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
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
