Re: [PATCH v3] Added perf functionality to mmdc driver

From: Mark Rutland
Date: Wed Aug 31 2016 - 06:30:54 EST


On Tue, Aug 30, 2016 at 08:01:18PM +0000, Zhengyu Shen wrote:
> Hi, I've done some testing with the fuzzer.

Great! Many thanks for doing this.

> Mmdc was only responsible for one crash which I fixed (had to remove
> the event from the pmu properly).

Ok. I take it that there will be a v4 appearing shortly with that fix?

> Other drivers also cause crashes and the program reports that events are
> Throttling. Is this normal?

Throttling is normal, and warnings about throttling can (generally) be
safely ignored.

Crashes are very bad. Do you have any logs from those crashes in other
drivers that you can share?

It is possible that a bug in one driver causes failures in another (e.g.
if events are erroneously grouped together), so it's not alwyas clear if
this is an unrelated bug (though if you can trigger the issue without
your driver present, it clearly is).

Thanks,
Mark.