Re: [Xen-devel] [PATCH for Xen 4.6 2/5] tools/libxl: fix socket display error for CMT

2015-09-28 Thread Chao Peng
On Mon, Sep 28, 2015 at 04:36:54PM +0100, Wei Liu wrote: > On Mon, Sep 28, 2015 at 05:06:31PM +0200, Dario Faggioli wrote: > > On Mon, 2015-09-28 at 19:54 +0800, Chao Peng wrote: > > > > > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > > > index 2706759..c72d3df 100644 > > > >

Re: [Xen-devel] [PATCH for Xen 4.6 2/5] tools/libxl: fix socket display error for CMT

2015-09-28 Thread Wei Liu
On Mon, Sep 28, 2015 at 05:06:31PM +0200, Dario Faggioli wrote: > On Mon, 2015-09-28 at 19:54 +0800, Chao Peng wrote: > > > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > > index 2706759..c72d3df 100644 > > > @@ -8228,9 +8228,9 @@ static void > > psr_cmt_print_domain_info(libx

Re: [Xen-devel] [PATCH for Xen 4.6 2/5] tools/libxl: fix socket display error for CMT

2015-09-28 Thread Dario Faggioli
On Mon, 2015-09-28 at 19:54 +0800, Chao Peng wrote: > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > index 2706759..c72d3df 100644 > @@ -8228,9 +8228,9 @@ static void > psr_cmt_print_domain_info(libxl_dominfo *dominfo, > > static int psr_cmt_show(libxl_psr_cmt_type type, ui

Re: [Xen-devel] [PATCH for Xen 4.6 2/5] tools/libxl: fix socket display error for CMT

2015-09-28 Thread Wei Liu
On Mon, Sep 28, 2015 at 07:54:50PM +0800, Chao Peng wrote: > When displaying the CMT information for all the sockets, we assume socket > number is continuous. This is not true in the hotplug case. For instance, > when the 3rd socket is plugged out on a 4-socket system, the available > sockets numbe

[Xen-devel] [PATCH for Xen 4.6 2/5] tools/libxl: fix socket display error for CMT

2015-09-28 Thread Chao Peng
When displaying the CMT information for all the sockets, we assume socket number is continuous. This is not true in the hotplug case. For instance, when the 3rd socket is plugged out on a 4-socket system, the available sockets numbers are 1,2,4 but current we will display the CMT information for so