Re: [Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-03-03 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Friday, March 03, 2017 6:49 PM > > >>> On 03.03.17 at 09:29, wrote: > >> From: Jan Beulich [mailto:jbeul...@suse.com] > >> Sent: Wednesday, March 01, 2017 3:42 PM > >> Sounds good to me (read: Reviewed-by: Jan Beulich ). > >> If Kevin would

Re: [Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-03-03 Thread Jan Beulich
>>> On 03.03.17 at 09:29, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: Wednesday, March 01, 2017 3:42 PM >> Sounds good to me (read: Reviewed-by: Jan Beulich ). >> If Kevin would give his ack, I could replace the comment while committing, >> so you wouldn't need to re-send. >

Re: [Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-03-03 Thread Tian, Kevin
> From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: Wednesday, March 01, 2017 3:42 PM > > >>> On 01.03.17 at 01:01, wrote: > > On Tue, Feb 28, 2017 at 09:43:09AM -0700, Jan Beulich wrote: > > On 27.02.17 at 02:45, wrote: > >>> --- a/xen/arch/x86/hvm/vmx/vmx.c > >>> +++ b/xen/arch/x86/hvm/

Re: [Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-02-28 Thread Jan Beulich
>>> On 01.03.17 at 01:01, wrote: > On Tue, Feb 28, 2017 at 09:43:09AM -0700, Jan Beulich wrote: > On 27.02.17 at 02:45, wrote: >>> --- a/xen/arch/x86/hvm/vmx/vmx.c >>> +++ b/xen/arch/x86/hvm/vmx/vmx.c >>> @@ -260,9 +260,15 @@ void vmx_pi_hooks_deassign(struct domain *d) >>> >>> ASSERT(

Re: [Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-02-28 Thread Chao Gao
On Tue, Feb 28, 2017 at 09:43:09AM -0700, Jan Beulich wrote: On 27.02.17 at 02:45, wrote: >> --- a/xen/arch/x86/hvm/vmx/vmx.c >> +++ b/xen/arch/x86/hvm/vmx/vmx.c >> @@ -260,9 +260,15 @@ void vmx_pi_hooks_deassign(struct domain *d) >> >> ASSERT(d->arch.hvm_domain.pi_ops.vcpu_block); >>

Re: [Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-02-28 Thread Jan Beulich
>>> On 27.02.17 at 02:45, wrote: > --- a/xen/arch/x86/hvm/vmx/vmx.c > +++ b/xen/arch/x86/hvm/vmx/vmx.c > @@ -260,9 +260,15 @@ void vmx_pi_hooks_deassign(struct domain *d) > > ASSERT(d->arch.hvm_domain.pi_ops.vcpu_block); > > +/* > + * Note that we don't set 'd->arch.hvm_domain.pi_

[Xen-devel] [PATCH v9 1/8] VMX: Permanently assign PI hook vmx_pi_switch_to()

2017-02-27 Thread Chao Gao
From: Feng Wu PI hook vmx_pi_switch_to() is needed even after any previously assigned device is detached from the domain. Since 'SN' bit is also used to control the CPU side PI and we change the state of SN bit in vmx_pi_switch_to() and vmx_pi_switch_from(), then evaluate this bit in vmx_deliver_