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

[Andrew Cooper <andrew.cooper3@xxxxxxxxxx>]

On 08/06/17 16:19, Jan Beulich wrote:
> Correct respective comments.
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

> ---
> 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 allowed for MMIO pages not happening to have
> an associated struct page_info.

Nothing actually uses the clear/copy subops, as far as I can tell,
although they could actually be used for a minor performance
optimisation for HVM domain builder domains, to avoid HAP mutations. 
I'd agree with restricting them to RAM only.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel