RE: [PATCH 1/1] perf,tools: error out unsupported group leader immediately for perf stat

[Liang, Kan]

> 
> Em Thu, Jun 11, 2015 at 02:32:40AM -0400, kan.liang@xxxxxxxxx escreveu:
> > perf stat ignores the unsupported event and continue to count
> > supported event. But if the unsupported event is group leader, perf
> > tool will crash. After applying this patch, the unsupported group
> > leader will error out immediately.
> 
> > Without this patch:
> > $ perf stat -x, -e '{node-prefetch-refs,cycles}' --  sleep 1
> > perf: util/evsel.c:1009: get_group_fd: Assertion `!(fd == -1)' failed.
> > Aborted (core dumped)
> 
> > With this patch:
> > $ perf stat -x, -e '{node-prefetch-refs,cycles}' --  sleep 1
> > Error:
> > The node-prefetch-refs event is not supported.
> 
> I get something different, maybe I am missing a patch?
> 
> With this patch:
> 
>   [acme@zoo linux]$ perf stat -x, -e '{node-prefetch-refs,cycles}' -- sleep 1
>   Error:
>   The sys_perf_event_open() syscall returned with 22 (Invalid argument)
>   for event (node-prefetch-refs).
>   /bin/dmesg may provide additional information.
>   No CONFIG_PERF_EVENTS=y kernel support configured?
>   [acme@zoo linux]$
>
I don't have other patches for this issue.
It looks they have different error code, ENOENT VS. EINVAL. 
I have no idea why. 

> Applying it anyway, as it avoids a core dump and the message, albeit not
> optimal, points to a problem with that event.
> 
> Have you tried this with other tools, like perf record?

Yes. Perf record and perf top  will error out immediately if any unsupported event found.
So they don't have such issue.

Thanks,
Kan

> 
> - Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/