Re: architecture-specific data in struct kvm_run ?

[Avi Kivity <avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org>]

Zhang, Xiantao wrote:
> Hollis Blanchard wrote:
>   
>> There are a bunch of x86-specific fields in struct kvm_run:
>> 	/* in */
>> 	__u8 request_interrupt_window;
>> 	__u8 padding1[7];
>>
>> 	/* out */
>> 	__u8 if_flag;
>> 	__u8 padding2[2];
>>
>> 	/* in (pre_kvm_run), out (post_kvm_run) */
>> 	__u64 cr8;
>> 	__u64 apic_base;
>>
>> Since this structure is shared with all architectures, can we find a
>> more appropriate place to keep this data? Or should we feel free to
>> add arch-specific data here?
>>     
> Hollis,
>
> Maybe #ifdef is enough if we other archs doesn't need to add new more.
> For IA64, we don't require new fields.
>
>   

I agree, ifdefs are best here.  Maybe wrap them in a structure.


-- 
error compiling committee.c: too many arguments to function


-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace