public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: jamie.iles@picochip.com (Jamie Iles)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH/RFC v1 0/2] Human readable performance event description in sysfs
Date: Wed, 20 Jan 2010 15:03:03 +0000	[thread overview]
Message-ID: <20100120150303.GG4089@wear.picochip.com> (raw)
In-Reply-To: <20100120144140.GB22897@n2100.arm.linux.org.uk>

On Wed, Jan 20, 2010 at 02:41:40PM +0000, Russell King - ARM Linux wrote:
> Given that history has shown that identification schemes on ARM change
> in extremely annoying ways, I don't think decoding these registers to
> some kind of textual representation for /proc/cpuinfo is the right
> approach.  It might instead make more sense to just export the entire
> set of CPU ID registers to userspace, and let userspace grapple with
> the complexities of decoding the information it wants from them.
Yes, this would probably be the best generic solution, but in the specific
case of ARM perfevents, the kernel code already has to decode some of the CPU
ID registers to work out what set of events to use. Why make userspace do all
of this decoding again? The x86 code sets up the x86_pmu depending on CPU type
so this is doing a similar thing (although it is easier for x86).

Having perf do all of this decoding for all of the supported CPU types when
the kernel has already done it once and maintaining 2 sets of event lists
seems a bit fiddly compared to simply exporting the supported events from the
kernel...

Jamie

  reply	other threads:[~2010-01-20 15:03 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-01-20  9:11 [PATCH/RFC v1 0/2] Human readable performance event description in sysfs Tomasz Fujak
2010-01-20  9:11 ` [PATCH v1 1/2] perfevent: Add performance event structure definition and 'extevents' sysfs entry Tomasz Fujak
2010-01-20  9:11 ` [PATCH v1 2/2] [ARM] perfevent: Event description list for ARMv6, Cortex-A8 and Cortex-A9 exported Tomasz Fujak
2010-01-20  9:16 ` [PATCH/RFC v1 0/2] Human readable performance event description in sysfs Peter Zijlstra
2010-01-20  9:46   ` Tomasz Fujak
2010-01-20  9:57   ` Michał Nazarewicz
2010-01-20 13:31     ` Jamie Iles
2010-01-20 13:39       ` Peter Zijlstra
2010-01-20 13:55         ` Russell King - ARM Linux
2010-01-20 14:01           ` Peter Zijlstra
2010-01-20 14:09             ` Michał Nazarewicz
2010-01-20 14:16               ` Peter Zijlstra
2010-01-20 14:26                 ` Peter Zijlstra
2010-01-20 14:45                   ` Russell King - ARM Linux
2010-01-20 14:54                 ` Michał Nazarewicz
2010-01-20 14:41             ` Russell King - ARM Linux
2010-01-20 15:03               ` Jamie Iles [this message]
2010-01-20 15:42                 ` Russell King - ARM Linux
2010-01-20 16:18                   ` Jamie Iles
2010-01-20 16:26               ` Jamie Lokier
2010-01-20 16:35                 ` Russell King - ARM Linux
2010-01-20  9:57 ` Russell King - ARM Linux
2010-01-20 10:21   ` Tomasz Fujak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100120150303.GG4089@wear.picochip.com \
    --to=jamie.iles@picochip.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox