On Tue, Jun 23, 2015 at 05:57:22PM +0800, Tiejun Chen wrote: > This patch introduces user configurable parameters to specify RDM > resource and according policies, > > Global RDM parameter: > rdm = "type=none/host,reserve=strict/relaxed" > Per-device RDM parameter: > pci = [ 'sbdf, rdm_reserve=strict/relaxed' ] > > Global RDM parameter, "type", allows user to specify reserved regions > explicitly, e.g. using 'host' to include all reserved regions reported > on this platform which is good to handle hotplug scenario. In the future > this parameter may be further extended to allow specifying random regions, > e.g. even those belonging to another platform as a preparation for live > migration with passthrough devices. Instead, 'none' means we have nothing > to do all reserved regions and ignore all policies, so guest work as before. > > 'strict/relaxed' policy decides how to handle conflict when reserving RDM > regions in pfn space. If conflict exists, 'strict' means an immediate error > so VM will be killed, while 'relaxed' allows moving forward with a warning > message thrown out. > > Default per-device RDM policy is 'strict', while default global RDM policy > is 'relaxed'. When both policies are specified on a given region, 'strict' is > always preferred. > > CC: Ian Jackson <ian.jack...@eu.citrix.com> > CC: Stefano Stabellini <stefano.stabell...@eu.citrix.com> > CC: Ian Campbell <ian.campb...@citrix.com> > CC: Wei Liu <wei.l...@citrix.com> > Signed-off-by: Tiejun Chen <tiejun.c...@intel.com>
The code looks good to me. I will wait for native English speakers to have a look at the docs. Wei. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel