qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 1/4] i8254: Factor out base class for KVM reuse


From: Anthony Liguori
Subject: Re: [Qemu-devel] [PATCH 1/4] i8254: Factor out base class for KVM reuse
Date: Wed, 01 Feb 2012 07:19:19 -0600
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.23) Gecko/20110922 Lightning/1.0b2 Thunderbird/3.1.15

On 02/01/2012 06:43 AM, Jan Kiszka wrote:
On 2012-02-01 13:23, Paolo Bonzini wrote:
On 01/31/2012 10:49 PM, Jan Kiszka wrote:
  Just make the methods that you want to override virtual with the default
  implementation and then make a KVMPIT that inherits from the PIT and
  then overrides whatever virtual functions it needs to.

That doesn't sound like the proper design for this purpose. Rather, we
have an abstract base class that both implementations are derived from.

Absolutely!

If I'm not using QOM properly to achieve this, please tell me.

It looks good, besides the need to rebase to Anthony's "part 3" changes.

Given that this part is not upstream and that the relevant conversion is
scripted, I would leave this to Anthony so far. Or what is the schedule?

I haven't run through regression testing yet so YMMV but the latest series is 
at:

https://github.com/aliguori/qemu/tree/qom-upstream.13

Regards,

Anthony Liguori


Jan





reply via email to

[Prev in Thread] Current Thread [Next in Thread]