Re: [Qemu-devel] [uq/master PATCH 7/7 v8] target-i386: CPU model subclasses

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux