Re: [Xen-devel] [PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings

2015-07-03 Thread Dario Faggioli
On Fri, 2015-07-03 at 12:09 +0100, Ian Jackson wrote: > Ian Campbell writes ("Re: [PATCH 1/2] libxl: turn NUMA placement misconfigs > into warnings"): > > On Wed, 2015-07-01 at 15:38 +0100, Ian Jackson wrote: > > > But, be aware that you are making a semantic change to the API which > > > cannot

Re: [Xen-devel] [PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings

2015-07-03 Thread Ian Jackson
Ian Campbell writes ("Re: [PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings"): > On Wed, 2015-07-01 at 15:38 +0100, Ian Jackson wrote: > > Acked-by: Ian Jackson > > Applied both patches with your ack. Right. > > But, be aware that you are making a semantic change to the API which

Re: [Xen-devel] [PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings

2015-07-03 Thread Ian Campbell
On Wed, 2015-07-01 at 15:38 +0100, Ian Jackson wrote: > Dario Faggioli writes ("[PATCH 1/2] libxl: turn NUMA placement misconfigs > into warnings"): > > instead than errors. More specifically, in libxl, > > b_info->numa_autoplacement is meant as a way to > > disable automatic NUMA placement, if on

Re: [Xen-devel] [PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings

2015-07-01 Thread Ian Jackson
Dario Faggioli writes ("[PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings"): > instead than errors. More specifically, in libxl, > b_info->numa_autoplacement is meant as a way to > disable automatic NUMA placement, if one does not > want it to happen. It is, however, useful for > con

[Xen-devel] [PATCH 1/2] libxl: turn NUMA placement misconfigs into warnings

2015-07-01 Thread Dario Faggioli
instead than errors. More specifically, in libxl, b_info->numa_autoplacement is meant as a way to disable automatic NUMA placement, if one does not want it to happen. It is, however, useful for consistency checking as well, i.e., to ensure that the user provided configuration (such as, for instance