Avi Kivity wrote:
> Hollis Blanchard wrote:
>> On Wed, 2007-11-21 at 11:18 +0200, Avi Kivity wrote:
>>>>
>>> Well, I hate to say it, but the resulting code doesn't look too well
>>> (all the kvm_x86 variables), and it's entirely my fault as I
>>> recommended this approach. Not like it was difficult to predict.
>>>
>>
>> I guess we still have reached no conclusion on this question?
>>
>>
>
> Right. Thanks for re-raising it.
Thanks too. I have almost done the rebase work for IA64 support, maybe
we should work out a solution for that :)
>>> I'm thinking again of
>>>
>>> struct kvm {
>>> struct kvm_arch a;
>>> ...
>>> }
>>>
>>> Where each arch defines its own kvm_arch. Now the changes look
>>> like a bunch of "kvm->blah" to "kvm->a.blah" conversions.
>>>
>>
>>
>
> The nicer one:
>
> struct kvm {
> struct kvm_arch arch;
> // common fields
> }
I prefer this one, seems it is more direct and readable. Same thinking
about kvm_vcpu structure:)
-------------------------------------------------------------------------
SF.Net email is sponsored by: The Future of Linux Business White Paper
from Novell. From the desktop to the data center, Linux is going
mainstream. Let it simplify your IT future.
http://altfarm.mediaplex.com/ad/ck/8857-50307-18918-4
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel
