kvm-devel

[kvm-devel] architecture-specific data in struct kvm_run ?

[Hollis B. <ho...@us...>]

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 Blanchard
IBM Linux Technology Center

Re: [kvm-devel] architecture-specific data in struct kvm_run ?

[Zhang, X. <xia...@in...>]

Hollis Blanchard wrote:
> There are a bunch of x86-specific fields in struct kvm_run:
> 	/* in */
> 	__u8 request_interrupt_window;
> 	__u8 padding1[7];
>=20
> 	/* out */
> 	__u8 if_flag;
> 	__u8 padding2[2];
>=20
> 	/* in (pre_kvm_run), out (post_kvm_run) */
> 	__u64 cr8;
> 	__u64 apic_base;
>=20
> 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.

Xiantao

Re: [kvm-devel] architecture-specific data in struct kvm_run ?

[Avi K. <av...@qu...>]

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

Re: [kvm-devel] architecture-specific data in struct kvm_run ?

[Carsten O. <co...@de...>]

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?
Afaics, we'll need an arch extension to kvm_run. On s390, we'll need 
different extensions then x86 has defined here. And we don't need any 
of above fields.

Carsten