[v1] spapr/xive: Activate StoreEOI in P10 compat guests

[PATCH 2/8] linux-headers: Update for KVM_DEV_XIVE_GRP_SOURCE_INFO

Posted by Cédric Le Goater 5 years, 5 months ago

To be sent with the linux-headers update when support is merged.

Signed-off-by: Cédric Le Goater <clg@kaod.org>
---
 linux-headers/asm-powerpc/kvm.h | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/linux-headers/asm-powerpc/kvm.h b/linux-headers/asm-powerpc/kvm.h
index 264e266a85bf..aeb8e8c4633b 100644
--- a/linux-headers/asm-powerpc/kvm.h
+++ b/linux-headers/asm-powerpc/kvm.h
@@ -690,6 +690,7 @@ struct kvm_ppc_cpu_char {
 #define KVM_DEV_XIVE_GRP_SOURCE_CONFIG	3	/* 64-bit source identifier */
 #define KVM_DEV_XIVE_GRP_EQ_CONFIG	4	/* 64-bit EQ identifier */
 #define KVM_DEV_XIVE_GRP_SOURCE_SYNC	5       /* 64-bit source identifier */
+#define KVM_DEV_XIVE_GRP_SOURCE_INFO	6       /* 64-bit source identifier */
 
 /* Layout of 64-bit XIVE source attribute values */
 #define KVM_XIVE_LEVEL_SENSITIVE	(1ULL << 0)
@@ -721,6 +722,13 @@ struct kvm_ppc_xive_eq {
 	__u8  pad[40];
 };
 
+#define KVM_XIVE_SOURCE_FLAG_STORE_EOI	0x00000001
+
+/* Layout of source characteristics (8 bytes) */
+struct kvm_ppc_xive_src {
+	__u64 flags;
+};
+
 #define KVM_XIVE_EQ_ALWAYS_NOTIFY	0x00000001
 
 #define KVM_XIVE_TIMA_PAGE_OFFSET	0
-- 
2.25.4

Re: [PATCH 2/8] linux-headers: Update for KVM_DEV_XIVE_GRP_SOURCE_INFO

Posted by David Gibson 5 years, 5 months ago

On Wed, Aug 19, 2020 at 03:08:37PM +0200, Cédric Le Goater wrote:
> To be sent with the linux-headers update when support is merged.

Ah, so this isn't ready to go just yet.

> 
> Signed-off-by: Cédric Le Goater <clg@kaod.org>
> ---
>  linux-headers/asm-powerpc/kvm.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/linux-headers/asm-powerpc/kvm.h b/linux-headers/asm-powerpc/kvm.h
> index 264e266a85bf..aeb8e8c4633b 100644
> --- a/linux-headers/asm-powerpc/kvm.h
> +++ b/linux-headers/asm-powerpc/kvm.h
> @@ -690,6 +690,7 @@ struct kvm_ppc_cpu_char {
>  #define KVM_DEV_XIVE_GRP_SOURCE_CONFIG	3	/* 64-bit source identifier */
>  #define KVM_DEV_XIVE_GRP_EQ_CONFIG	4	/* 64-bit EQ identifier */
>  #define KVM_DEV_XIVE_GRP_SOURCE_SYNC	5       /* 64-bit source identifier */
> +#define KVM_DEV_XIVE_GRP_SOURCE_INFO	6       /* 64-bit source identifier */
>  
>  /* Layout of 64-bit XIVE source attribute values */
>  #define KVM_XIVE_LEVEL_SENSITIVE	(1ULL << 0)
> @@ -721,6 +722,13 @@ struct kvm_ppc_xive_eq {
>  	__u8  pad[40];
>  };
>  
> +#define KVM_XIVE_SOURCE_FLAG_STORE_EOI	0x00000001
> +
> +/* Layout of source characteristics (8 bytes) */
> +struct kvm_ppc_xive_src {
> +	__u64 flags;
> +};
> +
>  #define KVM_XIVE_EQ_ALWAYS_NOTIFY	0x00000001
>  
>  #define KVM_XIVE_TIMA_PAGE_OFFSET	0

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Re: [PATCH 2/8] linux-headers: Update for KVM_DEV_XIVE_GRP_SOURCE_INFO

Posted by Cédric Le Goater 5 years, 5 months ago

On 8/20/20 2:58 AM, David Gibson wrote:
> On Wed, Aug 19, 2020 at 03:08:37PM +0200, Cédric Le Goater wrote:
>> To be sent with the linux-headers update when support is merged.
> 
> Ah, so this isn't ready to go just yet.

No indeed. We should get the QEMU part correct first, and handle 
safely the source characteristics of the host.

C.  

 
> 
>>
>> Signed-off-by: Cédric Le Goater <clg@kaod.org>
>> ---
>>  linux-headers/asm-powerpc/kvm.h | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/linux-headers/asm-powerpc/kvm.h b/linux-headers/asm-powerpc/kvm.h
>> index 264e266a85bf..aeb8e8c4633b 100644
>> --- a/linux-headers/asm-powerpc/kvm.h
>> +++ b/linux-headers/asm-powerpc/kvm.h
>> @@ -690,6 +690,7 @@ struct kvm_ppc_cpu_char {
>>  #define KVM_DEV_XIVE_GRP_SOURCE_CONFIG	3	/* 64-bit source identifier */
>>  #define KVM_DEV_XIVE_GRP_EQ_CONFIG	4	/* 64-bit EQ identifier */
>>  #define KVM_DEV_XIVE_GRP_SOURCE_SYNC	5       /* 64-bit source identifier */
>> +#define KVM_DEV_XIVE_GRP_SOURCE_INFO	6       /* 64-bit source identifier */
>>  
>>  /* Layout of 64-bit XIVE source attribute values */
>>  #define KVM_XIVE_LEVEL_SENSITIVE	(1ULL << 0)
>> @@ -721,6 +722,13 @@ struct kvm_ppc_xive_eq {
>>  	__u8  pad[40];
>>  };
>>  
>> +#define KVM_XIVE_SOURCE_FLAG_STORE_EOI	0x00000001
>> +
>> +/* Layout of source characteristics (8 bytes) */
>> +struct kvm_ppc_xive_src {
>> +	__u64 flags;
>> +};
>> +
>>  #define KVM_XIVE_EQ_ALWAYS_NOTIFY	0x00000001
>>  
>>  #define KVM_XIVE_TIMA_PAGE_OFFSET	0
>