Re: [Xen-devel] [PATCH] x86: correct vCPU dirty CPU handling

2018-05-22 Thread Jan Beulich
>>> On 22.05.18 at 13:02, wrote: > I still had to apply the patch below with 4.11rc4 for NetBSD. > Any chance to get it in the 4.11 branch before release ? See the thread "Pending patches for 4.11?" IOW - I very much hope so. Jan ___ Xen-devel maili

Re: [Xen-devel] [PATCH] x86: correct vCPU dirty CPU handling

2018-05-22 Thread Manuel Bouyer
Hello, I still had to apply the patch below with 4.11rc4 for NetBSD. Any chance to get it in the 4.11 branch before release ? On Thu, Apr 26, 2018 at 03:41:20AM -0600, Jan Beulich wrote: > Commit df8234fd2c ("replace vCPU's dirty CPU mask by numeric ID") was > too lax in two respects: First of all

Re: [Xen-devel] [PATCH] x86: correct vCPU dirty CPU handling

2018-04-26 Thread Manuel Bouyer
On Thu, Apr 26, 2018 at 03:41:20AM -0600, Jan Beulich wrote: > Commit df8234fd2c ("replace vCPU's dirty CPU mask by numeric ID") was > too lax in two respects: First of all it didn't consider the case of a > vCPU not having a valid dirty CPU in the descriptor table TLB flush > case. This is the iss

Re: [Xen-devel] [PATCH] x86: correct vCPU dirty CPU handling

2018-04-26 Thread Jan Beulich
>>> On 26.04.18 at 11:51, wrote: > On 26/04/18 10:41, Jan Beulich wrote: >> --- a/xen/arch/x86/mm.c >> +++ b/xen/arch/x86/mm.c >> @@ -1202,11 +1202,23 @@ void put_page_from_l1e(l1_pgentry_t l1e, >> unlikely(((page->u.inuse.type_info & PGT_count_mask) != 0)) && >> (l1e_o

Re: [Xen-devel] [PATCH] x86: correct vCPU dirty CPU handling

2018-04-26 Thread Andrew Cooper
On 26/04/18 10:41, Jan Beulich wrote: > --- a/xen/arch/x86/mm.c > +++ b/xen/arch/x86/mm.c > @@ -1202,11 +1202,23 @@ void put_page_from_l1e(l1_pgentry_t l1e, > unlikely(((page->u.inuse.type_info & PGT_count_mask) != 0)) && > (l1e_owner == pg_owner) ) > { > +

[Xen-devel] [PATCH] x86: correct vCPU dirty CPU handling

2018-04-26 Thread Jan Beulich
Commit df8234fd2c ("replace vCPU's dirty CPU mask by numeric ID") was too lax in two respects: First of all it didn't consider the case of a vCPU not having a valid dirty CPU in the descriptor table TLB flush case. This is the issue Manual has run into with NetBSD. Additionally reads of ->dirty_cp