Re: [tip:perf/urgent] perf/x86/cqm: Fix CQM handling of grouping events into a cache_group
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [tip:perf/urgent] perf/x86/cqm: Fix CQM handling of grouping events into a cache_group
- From: Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx>
- Date: Wed, 23 Mar 2016 20:14:38 +0000
- Cc: hpa@xxxxxxxxx, eranian@xxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, tony.luck@xxxxxxxxx, brgerst@xxxxxxxxx, peterz@xxxxxxxxxxxxx, bp@xxxxxxxxx, tglx@xxxxxxxxxxxxx, dsahern@xxxxxxxxx, dvlasenk@xxxxxxxxxx, jolsa@xxxxxxxxxx, luto@xxxxxxxxxxxxxx, mingo@xxxxxxxxxx, acme@xxxxxxxxxx, torvalds@xxxxxxxxxxxxxxxxxxxx, namhyung@xxxxxxxxxx, vincent.weaver@xxxxxxxxx, alexander.shishkin@xxxxxxxxxxxxxxx, linux-tip-commits@xxxxxxxxxxxxxxx
- In-reply-to: <alpine.DEB.2.10.1603211109210.25124@vshiva-Udesk>
- User-agent: Mutt/1.5.24+41 (02bc14ed1569) (2015-08-30)
On Mon, 21 Mar, at 11:14:37AM, Vikas Shivappa wrote:
>
>
> Before MBM , the below condition was never hit because we had only one event ?
>
> - if (a->hw.target == b->hw.target)
> + if (a->hw.target == b->hw.target) {
> + b->hw.is_group_event = true;
>
> We are trying to address this for cases where different MBM(local or total)
> and cqm events are grouped into one RMID.
I can't test these changes, so I'm only working from memory, but I
seem to recall that this condition is hit if monitoring simultaneously
from two invocations of perf. It's also possible to have pid/tid
groups overlapping, and that needs to be handled.
> Which is the case which led to duplicate values ?
Good question. Try monitoring a multithread process with these changes
and see if you get duplicate values reported.
--
To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[Index of Archives]
[Linux Stable Commits]
[Linux Stable Kernel]
[Linux Kernel]
[Linux USB Devel]
[Linux Video &Media]
[Linux Audio Users]
[Yosemite News]
[Linux SCSI]