Re: [Xen-devel] [PATCH V3] x86/HVM: Introduce struct hvm_pi_ops

2017-01-16 Thread Jan Beulich
>>> On 16.01.17 at 14:38, wrote: > So, what if I generalize and simplify as following: > > +/* > + * This structure defines function hooks to support hardware-assisted > + * virtual interrupt delivery to guest. (e.g. VMX PI and SVM AVIC). > + * > + * These hooks are defined by the underlying arch

Re: [Xen-devel] [PATCH V3] x86/HVM: Introduce struct hvm_pi_ops

2017-01-16 Thread Suravee Suthikulpanit
Kevin, On 1/16/17 09:13, Tian, Kevin wrote: diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c index 7b2c50c..0854e17 100644 --- a/xen/arch/x86/hvm/vmx/vmx.c +++ b/xen/arch/x86/hvm/vmx/vmx.c @@ -103,6 +103,47 @@ void vmx_pi_per_cpu_init(unsigned int cpu) spin_lock_init(&pe

Re: [Xen-devel] [PATCH V3] x86/HVM: Introduce struct hvm_pi_ops

2017-01-16 Thread Suravee Suthikulpanit
Jan, I would like to updated the following to be more correct. On 1/12/17 19:37, Jan Beulich wrote: On 12.01.17 at 05:47, wrote: --- a/xen/include/asm-x86/hvm/domain.h +++ b/xen/include/asm-x86/hvm/domain.h @@ -72,6 +72,38 @@ struct hvm_ioreq_server { bool_t bufioreq_atom

Re: [Xen-devel] [PATCH V3] x86/HVM: Introduce struct hvm_pi_ops

2017-01-15 Thread Tian, Kevin
> From: Suravee Suthikulpanit [mailto:suravee.suthikulpa...@amd.com] > Sent: Thursday, January 12, 2017 12:47 PM > > The current function pointers in struct vmx_domain for managing hvm > posted interrupt can be used also by SVM AVIC. Therefore, this patch > introduces the struct hvm_pi_ops in the

Re: [Xen-devel] [PATCH V3] x86/HVM: Introduce struct hvm_pi_ops

2017-01-12 Thread Jan Beulich
>>> On 12.01.17 at 05:47, wrote: > --- a/xen/include/asm-x86/hvm/domain.h > +++ b/xen/include/asm-x86/hvm/domain.h > @@ -72,6 +72,38 @@ struct hvm_ioreq_server { > bool_t bufioreq_atomic; > }; > > +/* > + * This structure defines function hooks to support hardware-assisted

[Xen-devel] [PATCH V3] x86/HVM: Introduce struct hvm_pi_ops

2017-01-11 Thread Suravee Suthikulpanit
The current function pointers in struct vmx_domain for managing hvm posted interrupt can be used also by SVM AVIC. Therefore, this patch introduces the struct hvm_pi_ops in the struct hvm_domain to hold them. Signed-off-by: Suravee Suthikulpanit Cc: Andrew Cooper Cc: Konrad Rzeszutek Wilk Cc: J