> 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
>>> 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.
>
> 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/
>>> 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(
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);
>>
>>> 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_
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_