Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-04-09 Thread Jan Beulich
>>> On 30.03.18 at 12:32, wrote: > On 03/30/2018 11:14 AM, Razvan Cojocaru wrote: >> On 03/30/2018 01:03 PM, George Dunlap wrote: >>> In response to v8, I said: "FWIW the approach looks good to me here. >>> You mainly need an x86 maintainer's ack and a toolstack maintainer's >>> ack (of which I a

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-30 Thread Razvan Cojocaru
On 03/30/2018 02:16 PM, George Dunlap wrote: > On Wed, Dec 13, 2017 at 2:22 PM, Petre Pircalabu > wrote: >> From: Razvan Cojocaru >> >> For the default EPT view we have xc_set_mem_access_multi(), which >> is able to set an array of pages to an array of access rights with >> a single hypercall. Ho

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-30 Thread George Dunlap
On Wed, Dec 13, 2017 at 2:22 PM, Petre Pircalabu wrote: > From: Razvan Cojocaru > > For the default EPT view we have xc_set_mem_access_multi(), which > is able to set an array of pages to an array of access rights with > a single hypercall. However, this functionality was lacking for the > altp2m

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-30 Thread George Dunlap
On 03/30/2018 11:14 AM, Razvan Cojocaru wrote: > On 03/30/2018 01:03 PM, George Dunlap wrote: >> On 03/30/2018 07:34 AM, Razvan Cojocaru wrote: >>> On 03/13/2018 07:44 PM, Wei Liu wrote: On Wed, Dec 13, 2017 at 04:22:20PM +0200, Petre Pircalabu wrote: [...] Tabs here. With

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-30 Thread Razvan Cojocaru
On 03/30/2018 01:03 PM, George Dunlap wrote: > On 03/30/2018 07:34 AM, Razvan Cojocaru wrote: >> On 03/13/2018 07:44 PM, Wei Liu wrote: >>> On Wed, Dec 13, 2017 at 04:22:20PM +0200, Petre Pircalabu wrote: >>> [...] >>> Tabs here. >>> >>> With this fixed, libxc bits: >>> >>> Acked-by: Wei Liu >> >>

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-30 Thread George Dunlap
On 03/30/2018 07:34 AM, Razvan Cojocaru wrote: > On 03/13/2018 07:44 PM, Wei Liu wrote: >> On Wed, Dec 13, 2017 at 04:22:20PM +0200, Petre Pircalabu wrote: >> [...] >> Tabs here. >> >> With this fixed, libxc bits: >> >> Acked-by: Wei Liu > > We seem to have (again) arrived at an impasse with this

[Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-29 Thread Razvan Cojocaru
On 03/13/2018 07:44 PM, Wei Liu wrote: > On Wed, Dec 13, 2017 at 04:22:20PM +0200, Petre Pircalabu wrote: > [...] > Tabs here. > > With this fixed, libxc bits: > > Acked-by: Wei Liu We seem to have (again) arrived at an impasse with this patch. To summarize, the original designers of altp2m ha

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-13 Thread Wei Liu
On Wed, Dec 13, 2017 at 04:22:20PM +0200, Petre Pircalabu wrote: > /** > * Mem paging operations. > diff --git a/tools/libxc/xc_altp2m.c b/tools/libxc/xc_altp2m.c > index 07fcd18..0f792b5 100644 > --- a/tools/libxc/xc_altp2m.c > +++ b/tools/libxc/xc_altp2m.c > @@ -213,3 +213,44 @@ int xc_altp2m

Re: [Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2018-03-13 Thread Jan Beulich
>>> On 13.12.17 at 15:22, wrote: > From: Razvan Cojocaru > > For the default EPT view we have xc_set_mem_access_multi(), which > is able to set an array of pages to an array of access rights with > a single hypercall. However, this functionality was lacking for the > altp2m subsystem, which coul

[Xen-devel] [PATCH v10] x86/altp2m: support for setting restrictions for an array of pages

2017-12-13 Thread Petre Pircalabu
From: Razvan Cojocaru For the default EPT view we have xc_set_mem_access_multi(), which is able to set an array of pages to an array of access rights with a single hypercall. However, this functionality was lacking for the altp2m subsystem, which could only set page restrictions for one page at a