Re: [v7][PATCH 10/16] tools: introduce some new parameters to set rdm policy

[Wei Liu <wei.liu2@citrix.com>]

On Thu, Jul 09, 2015 at 01:34:01PM +0800, Tiejun Chen wrote:
> This patch introduces user configurable parameters to specify RDM
> resource and according policies,
> 
> Global RDM parameter:
>     rdm = "strategy=host,policy=strict/relaxed"
> Per-device RDM parameter:
>     pci = [ 'sbdf, rdm_policy=strict/relaxed' ]
> 
> Global RDM parameter, "strategy", allows user to specify reserved regions
> explicitly, Currently, 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. By default this isn't set so we don't
> check all rdms. Instead, we just check rdm specific to a given device if
> you're assigning this kind of device. Note this option is not recommended
> unless you can make sure any conflict does exist.
> 
> '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 can't keep running, while 'relaxed' allows moving forward with a
> warning message thrown out.
> 
> Default per-device RDM policy is same as default global RDM policy as being
> 'relaxed'. And the per-device policy would override the global policy like
> others.
> 
> CC: Ian Jackson <ian.jackson@eu.citrix.com>
> CC: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> CC: Ian Campbell <ian.campbell@citrix.com>
> CC: Wei Liu <wei.liu2@citrix.com>
> Acked-by: Wei Liu <wei.liu2@citrix.com>
> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
> Acked-by: Ian Campbell <ian.campbell@citrix.com>
> Signed-off-by: Tiejun Chen <tiejun.chen@intel.com>

FWIW you shouldn't put Ian and Ian's acks here because they have not
explicitly done so.

No need to resend just for this (Ian and Ian, speak up if you disagree).
They may delete these tags when they commit this patch. But if you do
resend, please delete those two acks.

Wei.