Re: [PATCH 0/2] perf/x86: Add ability to sample TSC

From: Ingo Molnar
Date: Thu Feb 19 2015 - 14:13:30 EST

* Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote:

> > I suspect you're talking about something else entirely;
> > your changelogs are inadequate for they tell ntohing of
> > your usecase and have me guessing. Don't do that.
> Sorry. I did mention Intel PT in patch 2, but I basically
> assumed the need to synchronize events with other time
> sources was understood.

Written down, slightly repetitive walk-throughs from first
concepts to implementation that don't assume too much
context are very useful to us slightly overworked
maintainers with an attention span of a slightly retarded
golden retriever. (Make the story interesting and mix in a
few jokes and you've got our attention for sure!)

So please do explain verbosely, especially if a review
discussion turns somewhat discordant, it's a useful tool.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at