Re: [Xen-devel] [PATCH v12] introduce XENMEM_reserved_device_memory_map

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Wed, 2015-07-22 at 04:42 -0600, Jan Beulich wrote:
> This is a prerequisite for punching holes into HVM and PVH guests' 
> P2M
> to allow passing through devices that are associated with (on VT-d)
> RMRRs.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
> Signed-off-by: Tiejun Chen <tiejun.chen@xxxxxxxxx>
> Acked-by: Kevin Tian <kevin.tian@xxxxxxxxx>
> ---
> v12: Restore changes as much as possible to my original version, 



Looking through the older comments, it seems like there was a
conclusion to have an XSM check here, which isn't present here?

It looks like Tim and Ian's comments have been addressed (so far as
they were agreed to at the time).

WRT the comments Julien raised: at some point (early on) you said this
was only intended to be used by the toolstack. In which case can it not
be done in one of the unstable interfaces (e.g. sysctl I suppose is the
obvious one)?

If not then perhaps just adding a type field with only one option (PCI)
would introduce sufficient extensibility?

Ian.


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