Re: [PATCH -tip, final] perf, x86: Add hw_watchdog_set_attr() in asake of nmi-watchdog on P4

From: Cyrill Gorcunov
Date: Wed Jul 06 2011 - 03:01:13 EST


On Tue, Jul 05, 2011 at 07:25:15PM +0400, Cyrill Gorcunov wrote:
...
>
> i don't think it changes much, Ingo, if I change it to bus cycles I still
> will have to setup nmi-watchdog event separately (but simply with bus
> event).
>
> so an only option is the aliases, i'll try to deal with it but no milestones
>
> Cyrill

Seems I recall one of the problem with aliasing. Look, for example one
cpu-cycles (as native non-halted ticks) is served as nmi-watchdog, then
say perf top is started and it uses non-sleeping ticks (via execution unit
events) for counting cycles. Note the end user doesn't know about it since
this all is transparent. But what if some other user needs execution unit
events via RAW interface and because this event is already borrowed as alias
it never be granted, and what is worse is that a user has no idea why. Is it
acceptable tradeoff?

Cyrill
--
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/