Re: [Xen-devel] [PATCH v7 2/2] docs/designs: Add a design document for migration of xenstore data

2020-03-23 Thread Paul Durrant
> -Original Message- > From: Julien Grall > Sent: 23 March 2020 11:34 > To: Paul Durrant ; xen-devel@lists.xenproject.org > Cc: Paul Durrant ; Andrew Cooper > ; George Dunlap > ; Ian Jackson ; Jan > Beulich > ; Konrad Rzeszutek Wilk ; Stefano > Stabell

Re: [Xen-devel] [PATCH v7 1/2] docs/designs: Add a design document for non-cooperative live migration

2020-03-23 Thread Paul Durrant
> -Original Message- > From: Julien Grall > Sent: 23 March 2020 10:47 > To: Paul Durrant ; xen-devel@lists.xenproject.org > Cc: Paul Durrant ; Andrew Cooper > ; George Dunlap > ; Ian Jackson ; Jan > Beulich > ; Konrad Rzeszutek Wilk ; Stefano > Stabell

Re: [Xen-devel] [PATCH 2/2] xen/mm: Introduce PGC_state_uninitialised

2020-03-23 Thread Paul Durrant
> -Original Message- > > > diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c > > > index 62507ca651..5f0581c072 100644 > > > --- a/xen/arch/x86/mm.c > > > +++ b/xen/arch/x86/mm.c > > > @@ -491,7 +491,8 @@ void share_xen_page_with_guest(struct page_info > > > *page, struct domain *d, > >

Re: [Xen-devel] [PATCH 17/17] xen: Switch parameter in get_page_from_gfn to use typesafe gfn

2020-03-23 Thread Paul Durrant
> ; Wei Liu ; Roger Pau Monné > ; Paul Durrant > ; Jun Nakajima ; Kevin Tian > ; Tim Deegan > > Subject: [PATCH 17/17] xen: Switch parameter in get_page_from_gfn to use > typesafe gfn > > From: Julien Grall > > No functional change intended. > > Onl

Re: [Xen-devel] [PATCH 2/2] xen/mm: Introduce PGC_state_uninitialised

2020-03-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of David > Woodhouse > Sent: 19 March 2020 21:22 > To: xen-devel@lists.xenproject.org > Cc: Stefano Stabellini ; Julien Grall > ; Wei Liu ; > Andrew Cooper ; Ian Jackson > ; George Dunlap > ; hongy...@amazon.com; Jan Beulich > ;

Re: [Xen-devel] [PATCH 1/2] xen/mm: fold PGC_broken into PGC_state bits

2020-03-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of David > Woodhouse > Sent: 19 March 2020 21:22 > To: xen-devel@lists.xenproject.org > Cc: Stefano Stabellini ; Julien Grall > ; Wei Liu ; > Andrew Cooper ; Ian Jackson > ; George Dunlap > ; hongy...@amazon.com; Jan Beulich > ;

Re: [Xen-devel] [PATCH 2/2] tools/xenstore: Accumulate errors in xenstore-ls and exit appropriately

2020-03-20 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of David > Woodhouse > Sent: 19 March 2020 20:40 > To: xen-devel@lists.xenproject.org > Cc: Juergen Gross ; Ian Jackson ; > Wei Liu > Subject: [Xen-devel] [PATCH 2/2] tools/xenstore: Accumulate errors in > xenstore-ls and exit >

[Xen-devel] [PATCH v4 0/2] PV driver compatibility fixes

2020-03-19 Thread Paul Durrant
Paul Durrant (2): libxl: create domain 'error' node in xenstore libxl: make creation of xenstore 'suspend event channel' node optional... docs/man/xl.cfg.5.pod.in| 13 + docs/misc/xenstore-paths.pandoc | 17 + tools/libxl/libxl.h | 11

[Xen-devel] [PATCH v4 1/2] libxl: create domain 'error' node in xenstore

2020-03-19 Thread Paul Durrant
From: Paul Durrant Several PV drivers (both historically and currently [1]) report errors by writing text into /local/domain/$DOMID/error. This patch creates the node in libxl and makes it writable by the domain, and also adds some text into xenstore-paths.pandoc to state what the node

[Xen-devel] [PATCH v4 2/2] libxl: make creation of xenstore 'suspend event channel' node optional...

2020-03-19 Thread Paul Durrant
From: Paul Durrant ... and, if it is not created, make the top level 'device' node in xenstore writable by the guest instead. The purpose and semantics of the suspend event channel node are explained in xenstore-paths.pandoc [1]. It was originally introduced in xend by commit 17636f47a474

[Xen-devel] [PATCH v4 1/3] mm: keep PGC_extra pages on a separate list

2020-03-18 Thread Paul Durrant
-by: Paul Durrant Reviewed-by: Jan Beulich --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Julien Grall Cc: Stefano Stabellini Cc: Wei Liu Cc: "Roger Pau Monné" v7: - Cosmetic changes v6: - New in v6 --- xen/arch/x86/domain.c| 9 + xen/common/domain.c

[Xen-devel] [PATCH v4 2/3] x86 / ioreq: use a MEMF_no_refcount allocation for server pages...

2020-03-18 Thread Paul Durrant
From: Paul Durrant ... now that it is safe to assign them. This avoids relying on libxl (or whatever toolstack is in use) setting max_pages up with sufficient 'slop' to allow all necessary ioreq server pages to be allocated. Signed-off-by: Paul Durrant Reviewed-by: Jan Beulich --- Cc: Andrew

[Xen-devel] [PATCH v4 3/3] mm: add 'is_special_page' inline function...

2020-03-18 Thread Paul Durrant
From: Paul Durrant ... to cover xenheap and PGC_extra pages. PGC_extra pages are intended to hold data structures that are associated with a domain and may be mapped by that domain. They should not be treated as 'normal' guest pages (i.e. RAM or page tables). Hence, in many cases where code

[Xen-devel] [PATCH v4 0/3] make sure PGC_extra pages are dealt with properly

2020-03-18 Thread Paul Durrant
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 /

[Xen-devel] [PATCH v7 1/2] docs/designs: Add a design document for non-cooperative live migration

2020-03-18 Thread Paul Durrant
From: Paul Durrant It has become apparent to some large cloud providers that the current model of cooperative migration of guests under Xen is not usable as it relies on software running inside the guest, which is likely beyond the provider's control. This patch introduces a proposal for non

[Xen-devel] [PATCH v7 2/2] docs/designs: Add a design document for migration of xenstore data

2020-03-18 Thread Paul Durrant
From: Paul Durrant This patch details proposes extra migration data and xenstore protocol extensions to support non-cooperative live migration of guests. NOTE: doc/misc/xenstore.txt is also amened to replace the term for the INTRODUCE operation with the , since this is what

[Xen-devel] [PATCH v7 0/2] docs: Migration design documents

2020-03-18 Thread Paul Durrant
Paul Durrant (2): docs/designs: Add a design document for non-cooperative live migration docs/designs: Add a design document for migration of xenstore data docs/designs/non-cooperative-migration.md | 280 ++ docs/designs/xenstore-migration.md| 256

Re: [Xen-devel] [PATCH 2/2] xen/mm: Introduce PG_state_uninitialised

2020-03-18 Thread Paul Durrant
; ian.jack...@eu.citrix.com; george.dun...@citrix.com; > jeff.kubas...@dornerworks.com; 'Xia, Hongyan' > ; stewart.hildebr...@dornerworks.com; > xen-devel@lists.xenproject.org > Subject: Re: [PATCH 2/2] xen/mm: Introduce PG_state_uninitialised > > On 18.03.2020 09:53, Paul Durrant wro

Re: [Xen-devel] [PATCH 2/2] xen/mm: Introduce PG_state_uninitialised

2020-03-18 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of David > Woodhouse > Sent: 17 March 2020 22:15 > To: Jan Beulich > Cc: sstabell...@kernel.org; jul...@xen.org; w...@xen.org; > konrad.w...@oracle.com; > george.dun...@eu.citrix.com; andrew.coop...@citrix.com; >

Re: [Xen-devel] [PATCH v2 2/2] libxl: make creation of xenstore 'suspend event channel' node optional...

2020-03-17 Thread Paul Durrant
> -Original Message- > From: Ian Jackson > Sent: 17 March 2020 16:51 > To: Paul Durrant > Cc: xen-devel@lists.xenproject.org; Wei Liu ; Andrew Cooper > ; > George Dunlap ; Jan Beulich ; > Julien Grall > ; Stefano Stabellini ; Anthony Perard > > Subjec

Re: [Xen-devel] [PATCH v4] x86: irq: Do not BUG_ON multiple unbind calls for shared pirqs

2020-03-17 Thread Paul Durrant
UG_ON multiple unbind calls for > shared pirqs > > On 09.03.2020 18:47, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 09 March 2020 16:29 > >> To: p...@xen.org > >> Cc: xen-devel@lists.xenproject.org; Varad Gauta

Re: [Xen-devel] [EXTERNAL] [PATCH v6 4/5] mm: add 'is_special_page' inline function...

2020-03-17 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 17 March 2020 13:07 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Durrant, Paul ; > Tamas K Lengyel > ; Andrew Cooper ; Wei Liu > ; Roger Pau > Monné ; George Dunlap ; Ian > Jackson > ; Julien Grall ; Konrad Rzeszutek > Wilk

Re: [Xen-devel] [PATCH v6 2/5] mm: keep PGC_extra pages on a separate list

2020-03-17 Thread Paul Durrant
Monné' > Subject: Re: [PATCH v6 2/5] mm: keep PGC_extra pages on a separate list > > On 16.03.2020 19:11, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 16 March 2020 16:53 > >> > >> On 10.03.202

Re: [Xen-devel] [PATCH v6 2/5] mm: keep PGC_extra pages on a separate list

2020-03-16 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 16 March 2020 16:53 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Andrew Cooper > ; George Dunlap > ; Ian Jackson ; Julien > Grall ; > Stefano Stabellini ; Wei Liu ; Roger > Pau Monné > > Subject: Re: [PATCH v6 2/5] mm: keep

[Xen-devel] xenoprof

2020-03-13 Thread Paul Durrant
Hi, I'm trying to determine the status of HYPERVISOR_xenoprof_op. The code behind it appears to be unmaintained and I cannot find any support statement for it. Googling around finds some mentions of Xen and oprofile but it's not clear whether it works and most references I find are quite old.

Re: [Xen-devel] [PATCH] tools/helpers: xen-init-dom0: Mark clear_domid_history() static

2020-03-13 Thread Paul Durrant
tic > > From: Julien Grall > > xen-init-dom0 is a standalone binary, so all the functions but the > main() should be static. > > Signed-off-by: Julien Grall > Cc: p...@xen.org Reviewed-by: Paul Durrant ___ Xen-deve

Re: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of quarantined devices optional

2020-03-13 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 13 March 2020 08:10 > To: Tian, Kevin > Cc: xen-devel@lists.xenproject.org; Andrew Cooper > ; Paul Durrant > > Subject: Re: [PATCH v3] IOMMU: make DMA containment of quarantined devices > optional > >

Re: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of quarantined devices optional

2020-03-13 Thread Paul Durrant
> -Original Message- > From: Tian, Kevin > Sent: 13 March 2020 03:23 > To: p...@xen.org; 'Jan Beulich' > Cc: xen-devel@lists.xenproject.org; 'Andrew Cooper' > > Subject: RE: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of > quarantined devices optional &

Re: [Xen-devel] [EXTERNAL][PATCH v5 6/6] domain: use PGC_extra domheap page for shared_info

2020-03-12 Thread Paul Durrant
ou can confirm the sender and know the content is safe. > > > > On 11.03.2020 16:28, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 11 March 2020 09:17 > >> To: p...@xen.org > >> Cc: xen-devel@lis

Re: [Xen-devel] [PATCH v5 6/6] domain: use PGC_extra domheap page for shared_info

2020-03-11 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 11 March 2020 09:17 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; 'Paul Durrant' ; > 'Stefano Stabellini' > ; 'Julien Grall' ; 'Volodymyr Babchuk' > ; 'Andrew Cooper' ; > 'George Dunlap' > ; 'Ian J

Re: [Xen-devel] [PATCH v5 6/6] domain: use PGC_extra domheap page for shared_info

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 15:56 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Paul Durrant ; > Stefano Stabellini > ; Julien Grall ; Volodymyr Babchuk > ; Andrew Cooper ; > George Dunlap > ; Ian Jackson ; Ko

Re: [Xen-devel] [PATCH v5 5/6] mm: add 'is_special_page' inline function...

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 13:28 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Paul Durrant ; Tamas > K Lengyel > ; Andrew Cooper ; Wei Liu > ; Roger Pau > Monné ; George Dunlap ; Ian > Jackson > ; Julien

Re: [Xen-devel] [PATCH v2 8/9] x86/HVM: reduce hvm.h include dependencies

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 15:52 > To: xen-devel@lists.xenproject.org > Cc: Andrew Cooper ; Wei Liu ; Roger > Pau Monné > ; Paul Durrant ; Kevin Tian > ; Jun Nakajima > > Subject: [PATCH v2 8/9] x86/HVM: reduce hvm.h

Re: [Xen-devel] [PATCH v2 1/9] x86/HVM: reduce domain.h include dependencies

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 15:48 > To: xen-devel@lists.xenproject.org > Cc: Andrew Cooper ; Wei Liu ; Roger > Pau Monné > ; Paul Durrant ; Daniel de Graaf > ; Tamas K > Lengyel ; Petre Pircalabu ; > Alexandru Isaila &g

Re: [Xen-devel] [PATCH v2 2/9] x86/HVM: reduce vcpu.h include dependencies

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 15:49 > To: xen-devel@lists.xenproject.org > Cc: Andrew Cooper ; Wei Liu ; Roger > Pau Monné > ; Paul Durrant > Subject: [PATCH v2 2/9] x86/HVM: reduce vcpu.h include dependencies > &

Re: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of quarantined devices optional

2020-03-10 Thread Paul Durrant
al > > On 10.03.2020 16:13, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 10 March 2020 15:05 > >> To: p...@xen.org > >> Cc: 'Tian, Kevin' ; xen-devel@lists.xenproject.org; > >> 'Andrew Cooper

Re: [Xen-devel] [PATCH v5 3/6] x86 / pv: do not treat PGC_extra pages as RAM

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 15:33 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; 'Paul Durrant' ; > 'Andrew Cooper' > ; 'Wei Liu' ; 'Roger Pau Monné' > > Subject: Re: [PATCH v5 3/6] x86 / pv: do not t

Re: [Xen-devel] [PATCH v5 3/6] x86 / pv: do not treat PGC_extra pages as RAM

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 15:13 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; 'Paul Durrant' ; > 'Andrew Cooper' > ; 'Wei Liu' ; 'Roger Pau Monné' > > Subject: Re: [PATCH v5 3/6] x86 / pv: do not t

Re: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of quarantined devices optional

2020-03-10 Thread Paul Durrant
; > On 10.03.2020 13:30, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 10 March 2020 10:27 > >> To: Tian, Kevin ; Paul Durrant > >> Cc: xen-devel@lists.xenproject.org; Andrew Cooper > >> > &g

Re: [Xen-devel] [PATCH v5 3/6] x86 / pv: do not treat PGC_extra pages as RAM

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 14:59 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; 'Paul Durrant' ; > 'Andrew Cooper' > ; 'Wei Liu' ; 'Roger Pau Monné' > > Subject: Re: [PATCH v5 3/6] x86 / pv: do not t

Re: [Xen-devel] [PATCH v5] x86: irq: Do not BUG_ON multiple unbind calls for shared pirqs

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 13:57 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Varad Gautam ; Julien > Grall ; Roger > Pau Monné ; Andrew Cooper > Subject: Re: [PATCH v5] x86: irq: Do not BUG_ON multiple unbind calls for > shared pirqs >

Re: [Xen-devel] [PATCH v5 3/6] x86 / pv: do not treat PGC_extra pages as RAM

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 13:04 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Paul Durrant ; > Andrew Cooper > ; Wei Liu ; Roger Pau Monné > > Subject: Re: [PATCH v5 3/6] x86 / pv: do not treat PGC_extra pages a

Re: [Xen-devel] [PATCH v4] x86: irq: Do not BUG_ON multiple unbind calls for shared pirqs

2020-03-10 Thread Paul Durrant
UG_ON multiple unbind calls for > shared pirqs > > On 09.03.2020 18:47, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 09 March 2020 16:29 > >> > >> On 06.03.2020 17:02, p...@xen.org wrote: > >>>

Re: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of quarantined devices optional

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 10 March 2020 10:27 > To: Tian, Kevin ; Paul Durrant > Cc: xen-devel@lists.xenproject.org; Andrew Cooper > Subject: Re: [PATCH v3] IOMMU: make DMA containment of quarantined devices > optional > > On 10.03.20

Re: [Xen-devel] [PATCH v2 2/5] IOMMU: iommu_intpost is x86/HVM-only

2020-03-10 Thread Paul Durrant
t; ; 'Ian Jackson' > Subject: Re: [PATCH v2 2/5] IOMMU: iommu_intpost is x86/HVM-only > > On 10.03.2020 11:54, Paul Durrant wrote: > >> -Original Message- > >> From: Jan Beulich > >> Sent: 09 March 2020 10:43 > >> > >> @@ -486,8 +4

Re: [Xen-devel] [PATCH v2 2/5] IOMMU: iommu_intpost is x86/HVM-only

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 10:43 > To: xen-devel@lists.xenproject.org > Cc: Kevin Tian ; Stefano Stabellini > ; Julien Grall > ; Wei Liu ; Konrad Wilk > ; George Dunlap > ; Andrew Cooper ; Ian > Jackson > ; Paul

Re: [Xen-devel] [PATCH v2 5/5] IOMMU: iommu_snoop is x86-only

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 10:45 > To: xen-devel@lists.xenproject.org > Cc: Kevin Tian ; Stefano Stabellini > ; Julien Grall > ; Wei Liu ; Konrad Wilk > ; George Dunlap > ; Andrew Cooper ; Ian > Jackson > ; Paul

Re: [Xen-devel] [PATCH v2 4/5] IOMMU: iommu_qinval is x86-only

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 10:44 > To: xen-devel@lists.xenproject.org > Cc: Kevin Tian ; Stefano Stabellini > ; Julien Grall > ; Wei Liu ; Konrad Wilk > ; George Dunlap > ; Andrew Cooper ; Ian > Jackson > ; Paul

Re: [Xen-devel] [PATCH v2 3/5] IOMMU: iommu_igfx is x86-only

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 10:44 > To: xen-devel@lists.xenproject.org > Cc: Kevin Tian ; Stefano Stabellini > ; Julien Grall > ; Wei Liu ; Konrad Wilk > ; George Dunlap > ; Andrew Cooper ; Ian > Jackson > ; Paul

Re: [Xen-devel] [PATCH v2 1/5] IOMMU: iommu_intremap is x86-only

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 10:43 > To: xen-devel@lists.xenproject.org > Cc: Kevin Tian ; Stefano Stabellini > ; Julien Grall > ; Wei Liu ; Konrad Wilk > ; George Dunlap > ; Andrew Cooper ; Ian > Jackson > ; Paul

Re: [Xen-devel] [PATCH v3] IOMMU: make DMA containment of quarantined devices optional

2020-03-10 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 11:09 > To: xen-devel@lists.xenproject.org > Cc: Andrew Cooper ; Paul Durrant ; > Kevin Tian > > Subject: [PATCH v3] IOMMU: make DMA containment of quarantined devices > optional > &g

Re: [Xen-devel] [PATCH v4] x86: irq: Do not BUG_ON multiple unbind calls for shared pirqs

2020-03-09 Thread Paul Durrant
a shared pirq has already been unbound from this guest. The > > PIRQ will be cleaned up from the domain's pirq_tree during the destruction > > in complete_domain_destroy anyway. > > > > Signed-off-by: Varad Gautam > > [taking over from Varad at v4] > > Signed-off-b

Re: [Xen-devel] [PATCH v4 2/6] x86 / p2m: remove page_list check in p2m_alloc_table...

2020-03-09 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 09 March 2020 09:59 > To: p...@xen.org > Cc: xen-devel@lists.xenproject.org; Paul Durrant ; > Andrew Cooper > ; George Dunlap ; Wei > Liu ; Roger Pau > Monné > Subject: Re: [PATCH v4 2/6] x86

Re: [Xen-devel] [PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-03-09 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 06 March 2020 17:17 > To: Paul Durrant > Cc: sstabell...@kernel.org; jul...@xen.org; volodymyr_babc...@epam.com; > w...@xen.org; > konrad.w...@oracle.com; andrew.coop...@citrix.com; ian.jack...@eu.citrix.com; >

Re: [Xen-devel] [PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 06 March 2020 13:46 > To: Paul Durrant > Cc: sstabell...@kernel.org; jul...@xen.org; volodymyr_babc...@epam.com; > w...@xen.org; > konrad.w...@oracle.com; andrew.coop...@citrix.com; ian.jack...@eu.citrix.com; >

Re: [Xen-devel] [PATCH v3 5/6] mm: add 'is_special_page' macro...

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 06 March 2020 13:52 > To: Paul Durrant > Cc: pdurr...@amzn.com; 'Stefano Stabellini' ; 'Julien > Grall' ; > 'Wei Liu' ; 'Konrad Rzeszutek Wilk' ; > 'Andrew Cooper' > ; 'Ian Jackson' ; > 'George Dunla

Re: [Xen-devel] [PATCH v3 5/6] mm: add 'is_special_page' macro...

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 06 March 2020 13:44 > To: Paul Durrant > Cc: pdurr...@amzn.com; 'Stefano Stabellini' ; 'Julien > Grall' ; > 'Wei Liu' ; 'Konrad Rzeszutek Wilk' ; > 'Andrew Cooper' > ; Durrant, Paul ; 'Ian > Jackson' > ;

Re: [Xen-devel] [PATCH v3 3/6] x86 / pv: do not treat PGC_extra pages as RAM when constructing dom0

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of Jan > Beulich > Sent: 06 March 2020 13:39 > To: Durrant, Paul > Cc: xen-devel@lists.xenproject.org; Andrew Cooper > ; Wei Liu ; > pdurr...@amzn.com; Roger Pau Monné > Subject: Re: [Xen-devel] [PATCH v3 3/6] x86 / pv: do not treat

Re: [Xen-devel] [PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 06 March 2020 13:36 > To: Paul Durrant > Cc: sstabell...@kernel.org; jul...@xen.org; volodymyr_babc...@epam.com; > w...@xen.org; > konrad.w...@oracle.com; andrew.coop...@citrix.com; ian.jack...@eu.citrix.com; >

Re: [Xen-devel] [PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of Jan > Beulich > Sent: 06 March 2020 13:24 > To: Paul Durrant > Cc: sstabell...@kernel.org; jul...@xen.org; volodymyr_babc...@epam.com; > w...@xen.org; > konrad.w...@oracle.com; andrew.coop...@citrix.com; i

Re: [Xen-devel] [EXTERNAL][PATCH v3 2/6] x86 / p2m: remove page_list check in p2m_alloc_table

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Jan Beulich > Sent: 06 March 2020 13:19 > To: Paul Durrant > Cc: pdurr...@amzn.com; xen-devel@lists.xenproject.org; 'Andrew Cooper' > ; > 'George Dunlap' ; 'Wei Liu' ; 'Roger > Pau Monné' > > Subject: Re: [EXTERNAL][P

Re: [Xen-devel] [EXTERNAL][PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-03-06 Thread Paul Durrant
> -Original Message- > From: David Woodhouse > Sent: 06 March 2020 13:16 > To: Jan Beulich ; Durrant, Paul > Cc: jul...@xen.org; andrew.coop...@citrix.com; sstabell...@kernel.org; > konrad.w...@oracle.com; > volodymyr_babc...@epam.com; ian.jack...@eu.citrix.com; w...@xen.org; >

Re: [Xen-devel] [PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of Jan > Beulich > Sent: 06 March 2020 13:10 > To: David Woodhouse ; Durrant, Paul > > Cc: sstabell...@kernel.org; jul...@xen.org; w...@xen.org; > konrad.w...@oracle.com; > andrew.coop...@citrix.com; ian.jack...@eu.citrix.com; >

Re: [Xen-devel] [EXTERNAL][PATCH v3 2/6] x86 / p2m: remove page_list check in p2m_alloc_table

2020-03-06 Thread Paul Durrant
> >>>> Sent: 06 March 2020 11:46 > >>>> To: pdurr...@amzn.com > >>>> Cc: xen-devel@lists.xenproject.org; Durrant, Paul > >>>> ; Andrew Cooper > >>>> ; George Dunlap ; > >>>> Wei Liu ; > Roger &

Re: [Xen-devel] [PATCH v3 5/6] mm: add 'is_special_page' macro...

2020-03-06 Thread Paul Durrant
> -Original Message- > From: Xen-devel On Behalf Of Jan > Beulich > Sent: 06 March 2020 12:20 > To: pdurr...@amzn.com > Cc: Stefano Stabellini ; Julien Grall > ; Wei Liu ; > Konrad Rzeszutek Wilk ; Andrew Cooper > ; Paul > Durrant ; Ian Jackson ; > Geo

Re: [Xen-devel] [EXTERNAL][PATCH v6 2/2] docs/designs: Add a design document for migration of xenstore data

2020-03-06 Thread Paul Durrant
ender and know the content is safe. > > > > Hi Paul, > > On 05/03/2020 17:30, pdurr...@amzn.com wrote: > > From: Paul Durrant > > > > This patch details proposes extra migration data and xenstore protocol > > extensions to support non-cooperative l

Re: [Xen-devel] [PATCH v8 10/10] xen: introduce ERRP_AUTO_PROPAGATE

2020-03-06 Thread Paul Durrant
--in-place --no-show-diff --max-width 80 > > Reported-by: Kevin Wolf > Reported-by: Greg Kurz > Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Paul Durrant ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH v8 01/10] error: auto propagated local_err

2020-03-06 Thread Paul Durrant
of this macro at the start of functions with either use > error_prepend/error_append_hint (solving 1) or which use > local_err+error_propagate to check errors, switching those > functions to use *errp instead (solving 2 and 3). > > Signed-off-by: Vladimir Sementsov-Ogievskiy Reviewed-by: Pa

[Xen-devel] [PATCH 2/6] x86 / p2m: remove page_list check in p2m_alloc_table

2020-02-28 Thread Paul Durrant
There does not seem to be any justification for refusing to create the domain's p2m table simply because it may have assigned pages. Particularly it prevents the prior allocation of PGC_extra pages. Signed-off-by: Paul Durrant --- Cc: Jan Beulich Cc: Andrew Cooper Cc: George Dunlap Cc: Wei

[Xen-devel] [PATCH 6/6] domain: use PGC_extra domheap page for shared_info

2020-02-28 Thread Paul Durrant
and called in the error path for arch_domain_create(). [1] See https://lists.xenproject.org/archives/html/xen-devel/2020-02/msg02018.html Signed-off-by: Paul Durrant --- Cc: Stefano Stabellini Cc: Julien Grall Cc: Volodymyr Babchuk Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan

[Xen-devel] [PATCH v2 0/6] remove one more shared xenheap page: shared_info

2020-02-28 Thread Paul Durrant
Patches #2 and #3 have been split out of the previous version of patch #6 (which was patch #2 of the previous series). Patch #4 is not entirely related but is useful to have in place before patch #5. Patch #5 is also new. Paul Durrant (6): domain: introduce alloc/free_shared_info() helpers

[Xen-devel] [PATCH 5/6] mm: add 'is_special_page' macro...

2020-02-28 Thread Paul Durrant
is_xen_heap_page() it should also check for the PGC_extra bit in 'count_info'. This patch therefore defines is_special_page() to cover both cases and converts tests if is_xen_heap_page() to is_special_page() where appropriate. Signed-off-by: Paul Durrant --- Cc: Jan Beulich Cc: Andrew Cooper Cc: Wei Liu

[Xen-devel] [PATCH 4/6] x86 / ioreq: use a MEMF_no_refcount allocation for server pages...

2020-02-28 Thread Paul Durrant
... now that it is safe to assign them. This avoids relying on libxl (or whatever toolstack is in use) setting max_pages up with sufficient 'slop' to allow all necessary ioreq server pages to be allocated. Signed-off-by: Paul Durrant --- Cc: Paul Durrant Cc: Jan Beulich Cc: Andrew Cooper Cc

[Xen-devel] [PATCH 3/6] x86 / pv: do not treat PGC_extra pages as RAM when constructing dom0

2020-02-28 Thread Paul Durrant
The walk of page_list in dom0_construct_pv() should ignore PGC_extra pages as they are only created for special purposes and, if mapped, will be mapped explicitly for whatever purpose is relevant. Signed-off-by: Paul Durrant --- Cc: Jan Beulich Cc: Andrew Cooper Cc: Wei Liu Cc: "Roge

[Xen-devel] [PATCH 1/6] domain: introduce alloc/free_shared_info() helpers...

2020-02-28 Thread Paul Durrant
this. Hence the need to save the MFN. NOTE: Whist defining the new helpers, virt_to_mfn() in common/domain.c is made type safe. The definition of nmi_reason() in asm-x86/shared.h is also re- flowed to avoid overly long lines. Signed-off-by: Paul Durrant Reviewed-by: Julien Grall --- Cc

[Xen-devel] [PATCH 1/3] libxl: create domain 'error' node in xenstore

2020-02-26 Thread Paul Durrant
://xenbits.xen.org/gitweb/?p=pvdrivers/win/xenvif.git;a=blob;f=src/xenvif/frontend.c;hb=HEAD#l459 Signed-off-by: Paul Durrant --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Julien Grall Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Wei Liu Cc: Anthony PERARD

[Xen-devel] [PATCH 3/3] libxl: make the top level 'device' node in xenstore writable...

2020-02-26 Thread Paul Durrant
Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc: Wei Liu Cc: Anthony PERARD --- tools/libxl/libxl_create.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/libxl/libxl_create.c b/tools/libxl/libxl_create.c index 7119e95412..bc8e525821 100644 --- a/tools/libxl

[Xen-devel] [PATCH 0/3] PV driver compatibility fixes

2020-02-26 Thread Paul Durrant
Paul Durrant (3): libxl: create domain 'error' node in xenstore libxl: make creation of xenstore suspend event channel node optional libxl: make the top level 'device' node in xenstore writable... docs/man/xl.cfg.5.pod.in| 7 +++ docs/misc/xenstore-paths.pandoc | 5

[Xen-devel] [PATCH 2/3] libxl: make creation of xenstore suspend event channel node optional

2020-02-26 Thread Paul Durrant
rroneously implies ibxl_domain_create_info is a function. Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc: Wei Liu Cc: Anthony PERARD --- docs/man/xl.cfg.5.pod.in| 7 +++ tools/libxl/libxl.h | 13 - tools/libxl/libxl_create.c | 12 +--- tools/libxl/libxl_type

[Xen-devel] [PATCH] libxl: add initializers for libxl__domid_history

2020-02-26 Thread Paul Durrant
This patch fixes Coverity issue CID 1459006 (Insecure data handling (INTEGER_OVERFLOW)). The problem is that the error paths for libxl__mark_domid_recent() and libxl__is_domid_recent() check the 'f' field in struct libxl__domid_history when it may not have been initialized. Signed-off-by: Paul

[Xen-devel] [ANNOUNCE] Xen 4.14 Development Update

2020-02-26 Thread Paul Durrant
ode is available at https://xenbits.xen.org/gitweb/?p=people/dwmw2/xen.git;a=shortlog;h=refs/heads/lu-master - Project wiki page at https://wiki.xenproject.org/wiki/Live-Updating_Xen * Non-Cooperative Live Migration - Paul Durrant * Secret Hiding (v5) - Hongyan Xia * Hypervisor f

[Xen-devel] [PATCH] libxl: fix build with older glibc

2020-02-25 Thread Paul Durrant
rsion, and the libxl Makefile was not updated to add '-lrt' so the build will fail in that environment. This patch simply adds '-lrt' to LIBXL_LIBS unconditionally, as it does no harm in newer environments. Signed-off-by: Paul Durrant Fixes: 2b02882ebbbc ("libxl: add infrastructure to track a

[Xen-devel] [PATCH 2/2] domain: use PGC_extra domheap page for shared_info

2020-02-25 Thread Paul Durrant
. A fix is also needed in dom0_construct_pv() to avoid automatically adding PGC_extra pages to the physmap. Signed-off-by: Paul Durrant --- Cc: Stefano Stabellini Cc: Julien Grall Cc: Volodymyr Babchuk Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Konrad Rzeszutek

[Xen-devel] [PATCH 1/2] domain: introduce alloc/free_shared_info() helpers...

2020-02-25 Thread Paul Durrant
this. Hence the need to save the MFN. NOTE: Whist defining the new helpers, virt_to_mfn() in common/domain.c is made type safe. The definition of nmi_reason() in asm-x86/shared.h is also re- flowed to avoid overly long lines. Signed-off-by: Paul Durrant --- Cc: Stefano Stabellini Cc

[Xen-devel] [PATCH 0/2] remove one more shared xenheap page: shared_info

2020-02-25 Thread Paul Durrant
Paul Durrant (2): domain: introduce alloc/free_shared_info() helpers... domain: use PGC_extra domheap page for shared_info xen/arch/arm/domain.c| 10 - xen/arch/arm/mm.c| 2 +- xen/arch/x86/domain.c| 12 +- xen/arch/x86/mm.c| 2

[Xen-devel] [PATCH v7 6/6] xl: allow domid to be preserved on save/restore or migrate

2020-02-21 Thread Paul Durrant
no record in which to transfer domid and remote domain creation occurs before the migration stream is parsed. Hence this patch modifies xl rather then libxl. Signed-off-by: Paul Durrant Acked-by: Ian Jackson --- Cc: Wei Liu Cc: Anthony PERARD v5: - Expand the commit comment

[Xen-devel] [PATCH v7 2/6] libxl: modify libxl__logv() to only log valid domid values

2020-02-21 Thread Paul Durrant
Some code-paths use values other than INVALID_DOMID to indicate an invalid domain id. Specifically, xl will pass a value of 0 when creating/restoring a domain. Therefore modify libxl__logv() to use libxl_domid_valid_guest() as a validity test. Signed-off-by: Paul Durrant Acked-by: Ian Jackson

[Xen-devel] [PATCH v7 1/6] libxl: add infrastructure to track and query 'recent' domids

2020-02-21 Thread Paul Durrant
file is purged on boot to it is safe to use CLOCK_MONOTONIC as a time source. Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc: Wei Liu Cc: Anthony PERARD v7: - Addressed further comments from Ian - Introduced 'struct libxl__domid_history' to hold context v6: _ Addressed further

[Xen-devel] [PATCH v7 3/6] public/xen.h: add a definition for a 'valid domid' mask

2020-02-21 Thread Paul Durrant
rather than in, say, a libxenctrl header since it relates to the validity of a value passed to XEN_DOMCTL_createdomain. This new definition is placed in xen.h rather than domctl.h only to co-locate it with other domid-related defitions. Signed-off-by: Paul Durrant Reviewed-by: Ian

[Xen-devel] [PATCH v7 5/6] xl.conf: introduce 'domid_policy'

2020-02-21 Thread Paul Durrant
behaviour of having Xen choose the domid value during domain_create(). "random" will cause the special RANDOM_DOMID value to be passed to do_domain_create() such that libxl__domain_make() will select a random domid value. Signed-off-by: Paul Durrant Acked-by: Ian Jackson --- Cc: Wei Liu

[Xen-devel] [PATCH v7 0/6] xl/libxl: domid allocation/preservation changes

2020-02-21 Thread Paul Durrant
Paul Durrant (6): libxl: add infrastructure to track and query 'recent' domids libxl: modify libxl__logv() to only log valid domid values public/xen.h: add a definition for a 'valid domid' mask libxl: allow creation of domains with a specified or random domid xl.conf: introduce

[Xen-devel] [PATCH v7 4/6] libxl: allow creation of domains with a specified or random domid

2020-02-21 Thread Paul Durrant
will be destroyed. If the domid used in creation was specified to libxl__domain_make() then it will fail at this point, otherwise the create operation will be re-tried with either a new random or Xen-selected domid value. Signed-off-by: Paul Durrant Acked-by: Ian Jackson --- Cc: Wei Liu Cc: Anthony PERARD

[Xen-devel] [PATCH v6 4/6] libxl: allow creation of domains with a specified or random domid

2020-02-19 Thread Paul Durrant
will be destroyed. If the domid used in creation was specified to libxl__domain_make() then it will fail at this point, otherwise the create operation will be re-tried with either a new random or Xen-selected domid value. Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc: Wei Liu Cc: Anthony PERARD Cc

[Xen-devel] [PATCH v6 5/6] xl.conf: introduce 'domid_policy'

2020-02-19 Thread Paul Durrant
behaviour of having Xen choose the domid value during domain_create(). "random" will cause the special RANDOM_DOMID value to be passed to do_domain_create() such that libxl__domain_make() will select a random domid value. Signed-off-by: Paul Durrant Acked-by: Ian Jackson --- Cc: Wei Liu

[Xen-devel] [PATCH v6 6/6] xl: allow domid to be preserved on save/restore or migrate

2020-02-19 Thread Paul Durrant
no record in which to transfer domid and remote domain creation occurs before the migration stream is parsed. Hence this patch modifies xl rather then libxl. Signed-off-by: Paul Durrant Acked-by: Ian Jackson --- Cc: Wei Liu Cc: Anthony PERARD v5: - Expand the commit comment

[Xen-devel] [PATCH v6 0/6] xl/libxl: domid allocation/preservation changes

2020-02-19 Thread Paul Durrant
Paul Durrant (6): libxl: add infrastructure to track and query 'recent' domids libxl: modify libxl__logv() to only log valid domid values public/xen.h: add a definition for a 'valid domid' mask libxl: allow creation of domains with a specified or random domid xl.conf: introduce

[Xen-devel] [PATCH v6 2/6] libxl: modify libxl__logv() to only log valid domid values

2020-02-19 Thread Paul Durrant
Some code-paths use values other than INVALID_DOMID to indicate an invalid domain id. Specifically, xl will pass a value of 0 when creating/restoring a domain. Therefore modify libxl__logv() to use libxl_domid_valid_guest() as a validity test. Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc

[Xen-devel] [PATCH v6 3/6] public/xen.h: add a definition for a 'valid domid' mask

2020-02-19 Thread Paul Durrant
A subsequent patch will modify libxl to allow selection of a random domid value when creating domains. Valid values are limited to a width of 15 bits, so add an appropriate mask definition to the public header. Signed-off-by: Paul Durrant --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson

[Xen-devel] [PATCH v6 1/6] libxl: add infrastructure to track and query 'recent' domids

2020-02-19 Thread Paul Durrant
file is purged on boot to it is safe to use CLOCK_MONOTONIC as a time source. Signed-off-by: Paul Durrant --- Cc: Ian Jackson Cc: Wei Liu Cc: Anthony PERARD v6: _ Addressed further comments from Ian v5: - Re-work file manipulation some more - Add more error checks v4: - Use new

Re: [Xen-devel] [PATCH v2 fixed 13/16] numa: Teach ram block notifiers about resizable ram blocks

2020-02-13 Thread Paul Durrant
> use to fallback to legacy handling if a registered notifier (esp., SEV and > HAX) does not support actual resizes. > > Cc: Richard Henderson > Cc: Paolo Bonzini > Cc: "Dr. David Alan Gilbert" > Cc: Eduardo Habkost > Cc: Marcel Apfelbaum > Cc: Stefano Stabel

<    8   9   10   11   12   13   14   15   16   17   >