Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Chao Peng
On Wed, Jan 07, 2015 at 09:43:44PM +, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to > get CMT L3 event mask"): > > Other culprits are xc_get_max_nodes(), xc_get_max_cpus(), 4 instances in > > xc_psr.c and most

Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask"): > Other culprits are xc_get_max_nodes(), xc_get_max_cpus(), 4 instances in > xc_psr.c and most things in xc_offline_page.c which appears to have > static structures for d

Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Andrew Cooper
On 07/01/15 16:37, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to > get CMT L3 event mask"): >> On 07/01/15 11:12, Chao Peng wrote: >>> +int xc_psr_cmt_get_l3_event_mask(xc_interface *xch, uint32_t *event_mask) >

Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Ian Campbell
On Wed, 2015-01-07 at 16:37 +, Ian Jackson wrote: > > As the result of the hypercall is going to be the same, the > > worse that a race could achieve is a wasted hypercall. > > This kind of analysis is unfounded in the presence of modern compilers > with aggressive optimisations. At the very

Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Wei Liu
On Wed, Jan 07, 2015 at 04:37:40PM +, Ian Jackson wrote: > Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to > get CMT L3 event mask"): > > On 07/01/15 11:12, Chao Peng wrote: > > > +int xc_psr_cmt_get_l3_event_mask(xc_inter

Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Ian Jackson
Andrew Cooper writes ("Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask"): > On 07/01/15 11:12, Chao Peng wrote: > > +int xc_psr_cmt_get_l3_event_mask(xc_interface *xch, uint32_t *event_mask) > > +{ > > +static int val = 0; > >

Re: [Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Andrew Cooper
On 07/01/15 11:12, Chao Peng wrote: > This is the tools side wrapper for XEN_SYSCTL_PSR_CMT_get_l3_event_mask > of XEN_SYSCTL_psr_cmt_op. > > Signed-off-by: Chao Peng > --- > tools/libxc/include/xenctrl.h |1 + > tools/libxc/xc_psr.c | 24 > tools/libxl/lib

[Xen-devel] [PATCH v2 2/5] tools: add routine to get CMT L3 event mask

2015-01-07 Thread Chao Peng
This is the tools side wrapper for XEN_SYSCTL_PSR_CMT_get_l3_event_mask of XEN_SYSCTL_psr_cmt_op. Signed-off-by: Chao Peng --- tools/libxc/include/xenctrl.h |1 + tools/libxc/xc_psr.c | 24 tools/libxl/libxl.h |1 + tools/libxl/libxl_psr.c