Re: [PATCH v3] perf stat: Start tool events when enable_on_exec is used

[TAKANO Mitsuhiro <takano32@gmail.com>]

A ping on this one: three weeks since v3, and the only reply is the
automated review.

Nothing has changed on my side, and v3 is still the version I would
stand behind.  Both rounds of that review are addressed in it: the tool
event is started through evsel__tool_pmu_enable() rather than
evsel__enable(), so the members of a mixed group are left to the kernel,
and an event that failed to open is skipped instead of being started and
then read through a NULL core.fd.

The limitation stands: this machine does not allow perf_event_open for
the events involved, so the wrong output was read from the code - a
zeroed duration_time turning CPUs_utilized into a division by zero - and
not produced here.  A reviewer with the hardware can see it in one run.

If it is not wanted, say so and I will drop it.  If it wants a different
shape - a selftest, a smaller change, a different call site - tell me
which and I will send a v4.

Written with Hermes Agent, an LLM assistant; the author reviewed it.