[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CABPqkBRncSybcdFiEOTSw_xfmbgO_jvcW0QX4H9XsrOk7aFy+g@mail.gmail.com>
Date: Wed, 31 Aug 2011 14:21:29 +0200
From: Stephane Eranian <eranian@...gle.com>
To: Lin Ming <ming.m.lin@...el.com>
Cc: Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Ingo Molnar <mingo@...e.hu>, Andi Kleen <andi@...stfloor.org>,
Arnaldo Carvalho de Melo <acme@...stprotocols.net>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 0/6] perf: Intel uncore pmu counting support
Hi,
So I finally got a bit of time to test the Ming's uncore PMU patch series
on Intel NHM and SNB. I found several problems which I fixed and I also
made some small improvements. With those fixes + improvements, the
NHM/WSM patches look okay.
OTOH, the SNB patch needs more work. The SNB uncore PMU is more
complicated than what the code supports today based on the info in Vol3b.
For instance, SNB uncore events have constraints, therefore you need a
proper scheduling function.
I will post my patch in separate email thread.
On Fri, Jul 15, 2011 at 4:34 PM, Lin Ming <ming.m.lin@...el.com> wrote:
> Hi, all
>
> This is the v2 patches to support uncore pmu counting.
> The most important change is to use hrtimer to pull uncore counters
> to handle overflow, because uncore pmu interrupt does not work.
>
> v2 -> v1:
> - Use hrtimer to pull counters
> - Add interface to add general events to sysfs
> - Move MSRs to msr-index.h
> - Add uncore fixed counter
> - Allow system-wide 'perf stat' without 'command'
>
> Comments are appreciated.
>
> Thanks,
> Lin Ming
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists