Re: [Xen-devel] [PATCH] public: there's no MMUEXT_SET_FOREIGNDOM

2017-06-09 Thread Julien Grall
On 09/06/17 14:20, Julien Grall wrote: Hi Jan, On 08/06/17 16:19, Jan Beulich wrote: Correct respective comments. Signed-off-by: Jan Beulich --- MMUEXT_{CLEAR,COPY}_PAGE in fact also allow to be invoked on DOMID_IO owned pages at present. I've intentionally not added this to the text, as I'

Re: [Xen-devel] [PATCH] public: there's no MMUEXT_SET_FOREIGNDOM

2017-06-09 Thread Julien Grall
Hi Jan, On 08/06/17 16:19, Jan Beulich wrote: Correct respective comments. Signed-off-by: Jan Beulich --- MMUEXT_{CLEAR,COPY}_PAGE in fact also allow to be invoked on DOMID_IO owned pages at present. I've intentionally not added this to the text, as I'm not sure we really mean to allow this. I

Re: [Xen-devel] [PATCH] public: there's no MMUEXT_SET_FOREIGNDOM

2017-06-09 Thread Jan Beulich
>>> On 09.06.17 at 14:29, wrote: > On 08/06/17 16:19, Jan Beulich wrote: >> Correct respective comments. >> >> Signed-off-by: Jan Beulich > > Acked-by: Andrew Cooper > >> --- >> MMUEXT_{CLEAR,COPY}_PAGE in fact also allow to be invoked on DOMID_IO >> owned pages at present. I've intentionally

Re: [Xen-devel] [PATCH] public: there's no MMUEXT_SET_FOREIGNDOM

2017-06-09 Thread Andrew Cooper
On 08/06/17 16:19, Jan Beulich wrote: > Correct respective comments. > > Signed-off-by: Jan Beulich Acked-by: Andrew Cooper > --- > MMUEXT_{CLEAR,COPY}_PAGE in fact also allow to be invoked on DOMID_IO > owned pages at present. I've intentionally not added this to the text, > as I'm not sure we

[Xen-devel] [PATCH] public: there's no MMUEXT_SET_FOREIGNDOM

2017-06-08 Thread Jan Beulich
Correct respective comments. Signed-off-by: Jan Beulich --- MMUEXT_{CLEAR,COPY}_PAGE in fact also allow to be invoked on DOMID_IO owned pages at present. I've intentionally not added this to the text, as I'm not sure we really mean to allow this. If we do, I think the operation should also be all