Re: [Xen-devel] [v10][PATCH 16/16] tools: parse to enable new rdm policy parameters

2015-07-20 Thread Chen, Tiejun
For clarity: I am not acking this patch, primarily because I am not happy with the code in xlu_rdm_parse which is (a) the result of repeated clone-and-hack and (b) consists of ad-hoc string pointer fiddling. Yes, I knew you mentioned this previously but I also remember our last deal was someth

Re: [Xen-devel] [v10][PATCH 16/16] tools: parse to enable new rdm policy parameters

2015-07-20 Thread Ian Jackson
Tiejun Chen writes ("[v10][PATCH 16/16] tools: parse to enable new rdm policy parameters"): > This patch parses to enable user configurable parameters to specify > RDM resource and according policies which are defined previously, > > Global RDM parameter: > rdm = "strategy=host,policy=strict/

[Xen-devel] [v10][PATCH 16/16] tools: parse to enable new rdm policy parameters

2015-07-19 Thread Tiejun Chen
This patch parses to enable user configurable parameters to specify RDM resource and according policies which are defined previously, Global RDM parameter: rdm = "strategy=host,policy=strict/relaxed" Per-device RDM parameter: pci = [ 'sbdf, rdm_policy=strict/relaxed' ] Default per-device