Re: [PATCH 1/1] perf tools: Add missing user space support forconfig1/config2

[Vince Weaver]

On Wed, 27 Apr 2011, Ingo Molnar wrote:

> 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'.

by your logic I should be able to delete a file by saying
  echo "delete /tmp/tempfile" > /dev/sdc1
because using unlink() is too low of an abstraction and confusing to the 
user.

> 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.

Well we disagree.  I think one of perf_events biggest failings (among 
many) is that these generalized event definitions are shoved into the 
kernel.  At least it bloats the kernel in an option commonly turned on by 
vendors.  At worst it gives users a full sense of security in thinking 
these counters are A). Portable across architectures and B). Actually 
measure what they say they do.

I know it is fun to reinvent the wheel, but you ignored decades of 
experience in dealing with perf-counters when you ran off and invented 
perf_events.  It will bite you eventually.

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