linux-kernel - Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf

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: <20121120121747.GJ2504@rric.localhost>
Date:	Tue, 20 Nov 2012 13:17:47 +0100
From:	Robert Richter <rric@...nel.org>
To:	Will Deacon <will.deacon@....com>
Cc:	Maynard Johnson <maynardj@...ibm.com>,
	"jgq516@...il.com" <jgq516@...il.com>,
	"linux@....linux.org.uk" <linux@....linux.org.uk>,
	"oprofile-list@...ts.sf.net" <oprofile-list@...ts.sf.net>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 1/1] ARM: oprofile: add A5/A7/A15 entries in op_perf_name

Will,

On 05.11.12 11:31:03, Will Deacon wrote:
> > diff --git a/arch/arm/oprofile/common.c b/arch/arm/oprofile/common.c
> > index 99c63d4b..ec10db1 100644
> > --- a/arch/arm/oprofile/common.c
> > +++ b/arch/arm/oprofile/common.c
> > @@ -37,8 +37,11 @@ static struct op_perf_name {
> >  	{ "xscale1",		"arm/xscale2"	},
> >  	{ "v6",			"arm/armv6"	},
> >  	{ "v6mpcore",		"arm/mpcore"	},
> > +	{ "ARMv7 Cortex-A5",	"arm/armv7-ca5"	},
> > +	{ "ARMv7 Cortex-A7",	"arm/armv7-ca7"	},
> >  	{ "ARMv7 Cortex-A8",	"arm/armv7"	},
> >  	{ "ARMv7 Cortex-A9",	"arm/armv7-ca9"	},
> > +	{ "ARMv7 Cortex-A15",	"arm/armv7-ca15" },
> >  };

> I'd rather not go down this route now that we have the operf tool as part of
> oprofile, which can use the perf syscall directly and doesn't need this
> string translation.

since this is just an update of cpu detection I would be willing to
include this into kernel code anyway.

We could further move the cpu detection to userspace if perf_event
exists. We let the kernel enable oprofile with cpu_type="unknown".
User space then could either bind mount the file (user could do this
manually) or we implement to write to cpu_type. Doing so oprofile
could use in-kernel perf_events if it exists always as fallback.

Any thoughts?

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