mbox series

[v4,0/3] make sure PGC_extra pages are dealt with properly

Message ID 20200318173243.29183-1-paul@xen.org (mailing list archive)
Headers show
Series make sure PGC_extra pages are dealt with properly | expand

Message

Paul Durrant March 18, 2020, 5:32 p.m. UTC 
This series was formerly called "remove one more shared xenheap page:
shared_info" but I have dropped the patches actually changing shared_info
and just left the PGC_extra clean-up that was previously intertwined.

Paul Durrant (3):
  mm: keep PGC_extra pages on a separate list
  x86 / ioreq: use a MEMF_no_refcount allocation for server pages...
  mm: add 'is_special_page' inline function...

 xen/arch/x86/domain.c           |  9 +++++++++
 xen/arch/x86/domctl.c           |  2 +-
 xen/arch/x86/hvm/ioreq.c        |  2 +-
 xen/arch/x86/mm.c               | 13 ++++++-------
 xen/arch/x86/mm/altp2m.c        |  2 +-
 xen/arch/x86/mm/mem_sharing.c   |  3 +--
 xen/arch/x86/mm/p2m-pod.c       | 12 +++++++-----
 xen/arch/x86/mm/p2m.c           |  4 ++--
 xen/arch/x86/mm/shadow/common.c | 13 ++++++++-----
 xen/arch/x86/mm/shadow/multi.c  |  3 ++-
 xen/arch/x86/tboot.c            |  4 ++--
 xen/common/domain.c             |  1 +
 xen/common/page_alloc.c         |  2 +-
 xen/include/asm-x86/mm.h        |  6 ++----
 xen/include/xen/mm.h            | 10 +++++++---
 xen/include/xen/sched.h         | 13 +++++++++++++
 16 files changed, 64 insertions(+), 35 deletions(-)
---
Cc: Andrew Cooper <andrew.cooper3@citrix.com>
Cc: George Dunlap <george.dunlap@citrix.com>
Cc: Ian Jackson <ian.jackson@eu.citrix.com>
Cc: Jan Beulich <jbeulich@suse.com>
Cc: Julien Grall <julien@xen.org>
Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Cc: "Roger Pau Monné" <roger.pau@citrix.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>
Cc: Tim Deegan <tim@xen.org>
Cc: Wei Liu <wl@xen.org>

Comments

Jan Beulich March 24, 2020, 9:39 a.m. UTC | #1 
On 18.03.2020 18:32, Paul Durrant wrote:
> This series was formerly called "remove one more shared xenheap page:
> shared_info" but I have dropped the patches actually changing shared_info
> and just left the PGC_extra clean-up that was previously intertwined.
> 
> Paul Durrant (3):
>   mm: keep PGC_extra pages on a separate list
>   x86 / ioreq: use a MEMF_no_refcount allocation for server pages...
>   mm: add 'is_special_page' inline function...

So I'm confused - I had just replied twice to v6 patch 5/5. This
series calls itself v4 and consists of the middle three patches
of what v6 was. What's the deal? Is this really v7, and the two
patches have been dropped / postponed?

Jan
Paul Durrant March 24, 2020, 10:26 a.m. UTC | #2 
> -----Original Message-----
> From: Jan Beulich <jbeulich@suse.com>
> Sent: 24 March 2020 09:39
> To: Paul Durrant <paul@xen.org>
> Cc: xen-devel@lists.xenproject.org; Andrew Cooper <andrew.cooper3@citrix.com>; George Dunlap
> <george.dunlap@citrix.com>; Ian Jackson <ian.jackson@eu.citrix.com>; Julien Grall <julien@xen.org>;
> Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>; Roger Pau Monné <roger.pau@citrix.com>; Stefano
> Stabellini <sstabellini@kernel.org>; Tim Deegan <tim@xen.org>; Wei Liu <wl@xen.org>
> Subject: Re: [PATCH v4 0/3] make sure PGC_extra pages are dealt with properly
> 
> On 18.03.2020 18:32, Paul Durrant wrote:
> > This series was formerly called "remove one more shared xenheap page:
> > shared_info" but I have dropped the patches actually changing shared_info
> > and just left the PGC_extra clean-up that was previously intertwined.
> >
> > Paul Durrant (3):
> >   mm: keep PGC_extra pages on a separate list
> >   x86 / ioreq: use a MEMF_no_refcount allocation for server pages...
> >   mm: add 'is_special_page' inline function...
> 
> So I'm confused - I had just replied twice to v6 patch 5/5. This
> series calls itself v4 and consists of the middle three patches
> of what v6 was. What's the deal? Is this really v7, and the two
> patches have been dropped / postponed?

Sorry, I clearly got confused with numbering against one of my other series. Yes, this should be v7.

I wanted to send the patches that clear up use of PGC_extra, separating from the change to shared_info since I'm pressed for time to complete all the other conversions from xenheap pages such that I can send them as a single series.

  Paul

> 
> Jan