Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-30 Thread Paul Durrant
ts.xenproject.org; > Konrad Rzeszutek Wilk ; Daniel De Graaf > ; Tim (Xen.org) > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > Hi Paul, > > On 27/10/17 16:19, Paul Durrant wrote: > >> -Original Message-

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-30 Thread Julien Grall
...@lists.xenproject.org; Konrad Rzeszutek Wilk ; Daniel De Graaf ; Tim (Xen.org) Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources Hi, On 26/10/17 16:39, Jan Beulich wrote: On 26.10.17 at 17:32, wrote: On 26/10/17 16:26, Jan Beulich wrote: On

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-30 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 26 October 2017 16:27 > To: Paul Durrant > Cc: Julien Grall ; Andrew Cooper > ; Wei Liu ; George > Dunlap ; Ian Jackson ; > Stefano Stabellini ; xen-de...@lists.xenproject.org; > Konrad Rzeszutek Wilk ; Daniel De G

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-27 Thread Paul Durrant
ts.xenproject.org; > Konrad Rzeszutek Wilk ; Daniel De Graaf > ; Tim (Xen.org) > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > Hi, > > On 26/10/17 16:39, Jan Beulich wrote: > >>>> On 26.10.17 at 17:

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-27 Thread Julien Grall
Hi, On 26/10/17 16:39, Jan Beulich wrote: On 26.10.17 at 17:32, wrote: On 26/10/17 16:26, Jan Beulich wrote: On 17.10.17 at 15:24, wrote: +/* IN/OUT - If the tools domain is PV then, upon return, frame_list + * will be populated with the MFNs of the resource. + *

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-26 Thread Jan Beulich
>>> On 26.10.17 at 17:32, wrote: > On 26/10/17 16:26, Jan Beulich wrote: > On 17.10.17 at 15:24, wrote: >>> +/* IN/OUT - If the tools domain is PV then, upon return, frame_list >>> + * will be populated with the MFNs of the resource. >>> + * If the tools domain i

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-26 Thread Julien Grall
On 26/10/17 16:26, Jan Beulich wrote: On 17.10.17 at 15:24, wrote: +/* IN/OUT - If the tools domain is PV then, upon return, frame_list + * will be populated with the MFNs of the resource. + * If the tools domain is HVM then it is expected that, on + *

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-26 Thread Jan Beulich
>>> On 17.10.17 at 15:24, wrote: > @@ -535,6 +588,48 @@ int compat_memory_op(unsigned int cmd, > XEN_GUEST_HANDLE_PARAM(void) compat) > rc = -EFAULT; > break; > > +case XENMEM_acquire_resource: > +{ > +const xen_ulong_t *xen_frame_list =

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-25 Thread Paul Durrant
i Liu ; Stefano > Stabellini ; xen-de...@lists.xenproject.org; Konrad > Rzeszutek Wilk ; Daniel De Graaf > ; Tim (Xen.org) > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > > > On 20/10/17 11:10, Paul Durrant wrote: > >

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-23 Thread Julien Grall
Stabellini ; xen-de...@lists.xenproject.org; Konrad Rzeszutek Wilk ; Daniel De Graaf ; Tim (Xen.org) Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources Hi Paul, On 20/10/17 09:26, Paul Durrant wrote: -Original Message- From: J

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-20 Thread Paul Durrant
i Liu ; Stefano > Stabellini ; xen-de...@lists.xenproject.org; Konrad > Rzeszutek Wilk ; Daniel De Graaf > ; Tim (Xen.org) > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > Hi Paul, > > On 20/10/17 09:26, Paul Durra

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-20 Thread Julien Grall
; xen-de...@lists.xenproject.org; Konrad Rzeszutek Wilk ; Daniel De Graaf ; Tim (Xen.org) Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources On 19.10.17 at 18:21, wrote: Looking a bit more at the resource you can acquire from this hypercall

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-20 Thread Paul Durrant
xen-de...@lists.xenproject.org; Konrad Rzeszutek > Wilk ; Daniel De Graaf ; > Tim (Xen.org) > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > >>> On 19.10.17 at 18:21, wrote: > > Looking a bit more at the

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Jan Beulich
>>> On 19.10.17 at 18:21, wrote: > Looking a bit more at the resource you can acquire from this hypercall. > Some of them are allocated using alloc_xenheap_page() so not assigned to > a domain. > > So I am not sure how you can expect a function set_foreign_p2m_entry to > take reference in that

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Jan Beulich
>>> On 19.10.17 at 18:06, wrote: > On 19/10/17 16:47, Jan Beulich wrote: >> I don't understand: The refcounting is to be done by ARM-specific >> code anyway, i.e. by the implementation of set_foreign_p2m_entry(), >> not its caller. At least that's what I would have expected. > > I thought I said

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Julien Grall
Hi, On 19/10/17 17:06, Julien Grall wrote: On 19/10/17 16:47, Jan Beulich wrote: On 19.10.17 at 17:37, wrote: Hi, On 19/10/17 16:11, Jan Beulich wrote: On 19.10.17 at 16:49, wrote: I'd prefer to make the whole thing x86-only since that's the only platform on which I can test it, and inde

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Julien Grall
Hi, On 19/10/17 16:47, Jan Beulich wrote: On 19.10.17 at 17:37, wrote: Hi, On 19/10/17 16:11, Jan Beulich wrote: On 19.10.17 at 16:49, wrote: I'd prefer to make the whole thing x86-only since that's the only platform on which I can test it, and indeed the code used to be x86-only. Jan obj

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Jan Beulich
>>> On 19.10.17 at 17:37, wrote: > Hi, > > On 19/10/17 16:11, Jan Beulich wrote: > On 19.10.17 at 16:49, wrote: > I'd prefer to make the whole thing x86-only since that's the only platform on which I can test it, and indeed the code used to be x86-only. Jan > objected to this

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Julien Grall
Hi, On 19/10/17 16:11, Jan Beulich wrote: On 19.10.17 at 16:49, wrote: I'd prefer to make the whole thing x86-only since that's the only platform on which I can test it, and indeed the code used to be x86-only. Jan objected to this so all I'm trying to achieve is that it builds for ARM. Please

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Jan Beulich
>>> On 19.10.17 at 16:49, wrote: >> > I'd prefer to make the whole thing x86-only since that's the only platform >> on which I can test it, and indeed the code used to be x86-only. Jan objected >> to this so all I'm trying to achieve is that it builds for ARM. Please can >> you and >> Jan reach a

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Paul Durrant
> -Original Message- [snip] > > > > I'd prefer to make the whole thing x86-only since that's the only platform > on which I can test it, and indeed the code used to be x86-only. Jan objected > to this so all I'm trying to achieve is that it builds for ARM. Please can > you and > Jan reach

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Julien Grall
Jackson ; Tim (Xen.org) ; Julien Grall ; Jan Beulich ; Daniel De Graaf ; Roger Pau Monne Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources Hi, On 10/19/2017 01:57 PM, Paul Durrant wrote: -Original Message- From: Julien Grall

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Paul Durrant
Jackson ; Tim > (Xen.org) ; Julien Grall ; Jan Beulich > ; Daniel De Graaf ; Roger > Pau Monne > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > Hi, > > On 10/19/2017 01:57 PM, Paul Durrant wrote: >

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Julien Grall
; Ian Jackson ; Tim (Xen.org) ; Julien Grall ; Jan Beulich ; Daniel De Graaf ; Roger Pau Monne Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources Hi, On 17/10/17 14:24, Paul Durrant wrote: Certain memory resources associated with a guest are not

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Paul Durrant
Jackson ; Tim > (Xen.org) ; Julien Grall ; Jan Beulich > ; Daniel De Graaf ; Roger > Pau Monne > Subject: Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add > HYPERVISOR_memory_op to acquire guest resources > > Hi, > > On 17/10/17 14:24, Paul Durrant wrote: > > Certain mem

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-19 Thread Julien Grall
Hi, On 17/10/17 14:24, Paul Durrant wrote: Certain memory resources associated with a guest are not necessarily present in the guest P2M. This patch adds the boilerplate for new memory op to allow such a resource to be priv-mapped directly, by either a PV or HVM tools domain. NOTE: Whilst the

Re: [Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-17 Thread Daniel De Graaf
On 10/17/2017 09:24 AM, Paul Durrant wrote: Certain memory resources associated with a guest are not necessarily present in the guest P2M. This patch adds the boilerplate for new memory op to allow such a resource to be priv-mapped directly, by either a PV or HVM tools domain. NOTE: Whilst the

[Xen-devel] [PATCH v12 05/11] x86/mm: add HYPERVISOR_memory_op to acquire guest resources

2017-10-17 Thread Paul Durrant
Certain memory resources associated with a guest are not necessarily present in the guest P2M. This patch adds the boilerplate for new memory op to allow such a resource to be priv-mapped directly, by either a PV or HVM tools domain. NOTE: Whilst the new op is not intrinsicly specific to the x86