Re: [RFC PATCH 0/2] Add ARM support to operf

[Will D. <wil...@ar...>]

Hi Maynard,

On Mon, Jun 18, 2012 at 01:25:42PM +0100, Maynard Johnson wrote:
> On 06/18/2012 06:15 AM, Will Deacon wrote:
> > Well, I started looking at adding support for A5, A7 and A15 and noticed
> > that our common ARMv7 events are somewhat out-of-date (new reserved events
> > have been added to the architecture). Furthermore, ARM now provides
> > recommended mnemonics and descriptions for these architected events.
> > 
> > Would changing the event names be acceptable or is that considered an ABI
> > break? The change is slightly more than cosmetic in that it would guarantee
> > uniqueness between the micro-architectural events and those defined by the
> > architecture which is required for what we want.
> Will,
> Since event name changes affect only the users of the affected architecture, I feel that the decision make such changes is best left in the hands of the architecture maintainers.  *Richard* is technically the ARM sub-maintainer for the oprofile project, so I'd like his opinion.  But as the main ARM contributor, your opinion will count heavily. Changes such as these could be documented in the release notes.

Sure, let's see what Richard says. I'll hold fire on the new cores until
we've sorted out the event mnemonics.

Cheers,

Will