Re: [PATCH 1/4] perf, x86: Add Intel Nehalem/Westmere uncore pmu

From: Ingo Molnar
Date: Wed Jul 06 2011 - 05:35:22 EST



* Andi Kleen <andi@xxxxxxxxxxxxxx> wrote:

>
> >
> >> But it's not strictly required I would say,
> >> 44(?) bits are probably enough for near all use cases.
> >
> > 44bits is in the hours range for pure cycle counts, which is
> > so-so. I bet you're going to be very annoyed when you find your
> > counters are wrecked after your 5 hour test run finishes.
>
> For these kinds of long running measurements you usually care about
> ratios, not absolutes. Some infrequent wrapping should not
> substantially change. those.

That's just empty blathering - those who *do* take a look at
long-running absolute values obviously care.

There's absolutely no reason to be sloppy here - this needs to be
implemented correctly.

Thanks,

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