All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andi Kleen <andi@firstfloor.org>
To: Ingo Molnar <mingo@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>,
	linux-kernel@vger.kernel.org,
	Peter Zijlstra <a.p.zijlstra@chello.nl>,
	Arnaldo Carvalho de Melo <acme@infradead.org>
Subject: Re: Basic perf PMU support for Haswell v14
Date: Wed, 19 Jun 2013 21:38:55 +0200	[thread overview]
Message-ID: <20130619193855.GY6123@two.firstfloor.org> (raw)
In-Reply-To: <20130619124822.GC11209@gmail.com>

> Ok, this series looks good at first sight: you've slightly overdone the 
> checkpatch style changes without checking that the result is actually an 
> improvement - I've fixed those few places.
> 
> I'll give it a bit of testing.

Thanks for testing and merging.

I still have some other patches to enable other features,
on top of this.

I'll repost them in hopefully logical groups. 

If you want them grouped in some particular way or other
requirements please let me know.

https://git.kernel.org/cgit/linux/kernel/git/ak/linux-misc.git/log/?h=hsw/pmu6

FWIW the outstanding patches do:

- Some changes to make CPU event aliases more user friendly:

print aliases in perf list, avoid the need to specify cpu//,
automatically enable PEBS for mem-loads/stores,
Allow aliases to specify "precise"

This is all not Haswell specific, but makes
the TSX changes below a lot more user friendly.

Most of this is actually user mode only, except for the
"precise" alias changes.

- TSX changes: perf stat -T, TSX event aliases, support for 
reporting the abort cause from PEBS, plus the user tools
support for all of this

- Full width counting (generally useful, but also avoids
a TSX checkpoint problem)

- Support TSX filtering in the the LBR, plus a workaround
for the duplicated abort record.

Plus also on top (not in my git tree):

- Zheng's call graph lbr mode patches, to allow user mode -g without
frame pointers or full dwarf. This has a few known problems, but is still
useful enough that it should be available as a option.

-Andi

-- 
ak@linux.intel.com -- Speaking for myself only.

  reply	other threads:[~2013-06-19 19:38 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1371515812-9646-1-git-send-email-andi@firstfloor.org>
2013-06-19  8:20 ` Basic perf PMU support for Haswell v14 Ingo Molnar
2013-06-19 12:48 ` Ingo Molnar
2013-06-19 19:38   ` Andi Kleen [this message]
2013-06-20 10:16     ` Ingo Molnar

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=20130619193855.GY6123@two.firstfloor.org \
    --to=andi@firstfloor.org \
    --cc=a.p.zijlstra@chello.nl \
    --cc=acme@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.