lkml.org 
[lkml]   [2011]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/1] perf tools: Add missing user space support for config1/config2

* Vince Weaver <vweaver1@eecs.utk.edu> wrote:

> On Tue, 26 Apr 2011, Ingo Molnar wrote:
>
> > The kernel development process is in essence an abstraction engine, and if
> > you expect something else you'll probably be facing a lot of frustrating
> > episodes in the future as well where others try to abstract out meaningful
> > generalizations.
>
> yes, but you are taking abstraction to the extreme.

Firstly, that claim is a far cry from your original claim:

' How do you "generalize" a functionality like writing a value to an auxiliary
MSR register? '

... so i guess you conceded the point at least partially, without actually
openly and honestly conceding the point?

Secondly, you are still quite wrong even with your revised opinion. Being able
to type '-e cycles' and '-e instructions' in perf and get ... cycles and
instructions counts/events, and the kernel helping that kind of approach is not
'abstraction to the extreme', it's called 'common sense'.

The fact that perfmon and oprofile works via magic vendor-specific event string
incantations is one of the many design failures of those projects - not a
virtue.

Thanks,

Ingo


\
 
 \ /
  Last update: 2011-04-27 08:55    [W:0.106 / U:1.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site