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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 6 Mar 2014 11:33:15 +0000 From: Will Deacon <will.deacon@....com> To: Jean Pihet <jean.pihet@...aro.org> Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "linaro-kernel@...ts.linaro.org" <linaro-kernel@...ts.linaro.org>, "linux-arm-kernel@...ts.infradead.org" <linux-arm-kernel@...ts.infradead.org>, Arnaldo <acme@...stprotocols.net>, Ingo Molnar <mingo@...nel.org>, Jiri Olsa <jolsa@...hat.com>, "steve.capper@...aro.org" <steve.capper@...aro.org>, "patches@...aro.org" <patches@...aro.org>, Corey Ashford <cjashfor@...ux.vnet.ibm.com>, Frederic Weisbecker <fweisbec@...il.com>, Namhyung Kim <namhyung@...nel.org>, Paul Mackerras <paulus@...ba.org>, Peter Zijlstra <a.p.zijlstra@...llo.nl>, David Ahern <dsahern@...il.com> Subject: Re: [PATCH 1/3] perf tests: Introduce perf_regs_load function on ARM On Wed, Mar 05, 2014 at 02:17:00AM +0000, Jean Pihet wrote: > On 4 March 2014 12:00, Will Deacon <will.deacon@....com> wrote: > > On Mon, Mar 03, 2014 at 09:53:21AM +0000, Jean Pihet wrote: > >> + str lr, [r0, #PC] @ Save caller PC > > > > This isn't necessarily the `caller PC' (depending on how you define it). > > It's the return address, which is probably (but not always) the instruction > > following the branch to this function. > Agreed. However the perf test code expects a registers buffer filled > in with the caller's values. > I can change the comment here, is that needed? It depends what the perf test code really expects. At the moment, you're not providing it with anything consistent which doesn't sound correct. What does it use the caller PC for? Will -- 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/
Powered by blists - more mailing lists