linux-kernel - Re: [PATCH] perf, p4: Add format attributes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20120327220343.GG16537@moon>
Date:	Wed, 28 Mar 2012 02:03:43 +0400
From:	Cyrill Gorcunov <gorcunov@...nvz.org>
To:	Peter Zijlstra <peterz@...radead.org>
Cc:	Ingo Molnar <mingo@...e.hu>, Jiri Olsa <jolsa@...hat.com>,
	Lin Ming <ming.m.lin@...el.com>, rostedt <rostedt@...dmis.org>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	Stephane Eranian <eranian@...gle.com>
Subject: Re: [PATCH] perf, p4: Add format attributes

On Tue, Mar 27, 2012 at 07:12:50PM +0400, Cyrill Gorcunov wrote:
> 
> Ah, I see thanks for the reference, Peter!
> 
> > I've still not actually read the P4-PMU specs, so I don't know if
> > there's anything more convenient we can do on P4. If there's a
> > better/more useful format representation possible, don't hesitate to
> > send a patch! :-)
> 
> Sure, I'll try to find some time tonight to read this brand
> new stuff. And if I find something -- will send out. At moment
> your patch should be more than enough ;)

Hi, Peter, sorry for delay,

after looking some more I think ht bit might be dropped from
the format, it's used internally by the kernel in attmept to
eliminate redundant msr poking if new event runs on the same
thread as previous one, and it's always cleared before real
write to msr happens, which means user-space might set or
clear it, but it never be considered by the kernel.

(still my earlier reviewed-by is valid because this ht bit
 is harmless ;)

	Cyrill
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/