Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-13 Thread Like Xu
On 2020/8/13 15:08, Jin, Yao wrote: On 8/13/2020 2:57 PM, Like Xu wrote: Hi Yao, On 2020/8/13 11:11, Jin, Yao wrote: Hi Like, On 8/12/2020 9:02 PM, Like Xu wrote: On 2020/8/12 20:15, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-13 Thread Jin, Yao
On 8/13/2020 2:57 PM, Like Xu wrote: Hi Yao, On 2020/8/13 11:11, Jin, Yao wrote: Hi Like, On 8/12/2020 9:02 PM, Like Xu wrote: On 2020/8/12 20:15, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u',

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-13 Thread Like Xu
Hi Yao, On 2020/8/13 11:11, Jin, Yao wrote: Hi Like, On 8/12/2020 9:02 PM, Like Xu wrote: On 2020/8/12 20:15, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Jin, Yao
Hi Arnaldo, On 8/12/2020 8:55 PM, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 09:15:04AM -0300, Arnaldo Carvalho de Melo escreveu: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Jin, Yao
Hi Like, On 8/12/2020 9:02 PM, Like Xu wrote: On 2020/8/12 20:15, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make sense that we request for user-space counting

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Like Xu
On 2020/8/12 20:15, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make sense that we request for user-space counting but we also get the guest report. Please hold

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Arnaldo Carvalho de Melo
Em Wed, Aug 12, 2020 at 09:15:04AM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: > > Currently if we run 'perf record -e cycles:u', exclude_guest is 0. > > > > But it doesn't make sense that we request for user-space counting > > but we

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Arnaldo Carvalho de Melo
Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: > Currently if we run 'perf record -e cycles:u', exclude_guest is 0. > > But it doesn't make sense that we request for user-space counting > but we also get the guest report. > > To keep perf semantics consistent and clear, this patch

[PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Jin Yao
Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make sense that we request for user-space counting but we also get the guest report. To keep perf semantics consistent and clear, this patch sets exclude_guest for user-space counting. Signed-off-by: Jin Yao ---