Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 28 April 2016 13:31 > To: Paul Durrant > Cc: Andrew Cooper; xen-devel > Subject: RE: [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS > > >>> On 28.04.16 at 13:58, wrote: > >> -Original Message- > >> From: Jan

Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Jan Beulich
>>> On 28.04.16 at 13:58, wrote: >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 28 April 2016 12:44 >> To: Paul Durrant >> Cc: Andrew Cooper; xen-devel >> Subject: RE: [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS >> >> >>> On 28.04.16 at 13:17, wrote: >> >

Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 28 April 2016 12:44 > To: Paul Durrant > Cc: Andrew Cooper; xen-devel > Subject: RE: [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS > > >>> On 28.04.16 at 13:17, wrote: > >> From: Jan Beulich [mailto:jbeul...@suse.co

Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Jan Beulich
>>> On 28.04.16 at 13:17, wrote: >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 28 April 2016 10:50 >> @@ -366,7 +367,22 @@ static int msixtbl_range(struct vcpu *v, >> ((addr & (PCI_MSIX_ENTRY_SIZE - 1)) == >>PCI_MSIX_ENTRY_VECTOR_CTRL_OFFSET) && >>

Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 28 April 2016 10:50 > To: xen-devel > Cc: Andrew Cooper; Paul Durrant > Subject: [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS > > ... as at least certain versions of Windows use such to update the > MSI-X table. How

Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Jan Beulich
>>> On 28.04.16 at 12:34, wrote: > On 28/04/16 10:50, Jan Beulich wrote: >> ... as at least certain versions of Windows use such to update the >> MSI-X table. However, to not overly complicate the logic for now >> - only EFLAGS.DF=0 is being handled, >> - only updates not crossing MSI-X table entr

Re: [Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Andrew Cooper
On 28/04/16 10:50, Jan Beulich wrote: > ... as at least certain versions of Windows use such to update the > MSI-X table. However, to not overly complicate the logic for now > - only EFLAGS.DF=0 is being handled, > - only updates not crossing MSI-X table entry boundaries are handled. > > Signed-off

[Xen-devel] [PATCH 3/3] x86/vMSI-X: also snoop REP MOVS

2016-04-28 Thread Jan Beulich
... as at least certain versions of Windows use such to update the MSI-X table. However, to not overly complicate the logic for now - only EFLAGS.DF=0 is being handled, - only updates not crossing MSI-X table entry boundaries are handled. Signed-off-by: Jan Beulich --- a/xen/arch/x86/hvm/vmsi.c