Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Catalin Marinas
On Mon, Apr 15, 2013 at 02:09:20PM +0100, Christopher Covington wrote: > On 04/15/2013 07:43 AM, Catalin Marinas wrote: > > On Mon, Apr 15, 2013 at 11:58:40AM +0100, Catalin Marinas wrote: > >> On Mon, Apr 15, 2013 at 11:45:42AM +0100, Will Deacon wrote: > >>> On Mon, Apr 15, 2013 at 11:11:59AM +01

Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Will Deacon
On Mon, Apr 15, 2013 at 12:43:07PM +0100, Catalin Marinas wrote: > On Mon, Apr 15, 2013 at 11:58:40AM +0100, Catalin Marinas wrote: > > On Mon, Apr 15, 2013 at 11:45:42AM +0100, Will Deacon wrote: > > > Really? If prev is NULL in context_switch(...), the scheduler will > > > implode, > > > and I c

Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Christopher Covington
Hi Catalin, On 04/15/2013 07:43 AM, Catalin Marinas wrote: > On Mon, Apr 15, 2013 at 11:58:40AM +0100, Catalin Marinas wrote: >> On Mon, Apr 15, 2013 at 11:45:42AM +0100, Will Deacon wrote: >>> On Mon, Apr 15, 2013 at 11:11:59AM +0100, Catalin Marinas wrote: On Tue, Apr 09, 2013 at 01:33:34PM

Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Catalin Marinas
On Mon, Apr 15, 2013 at 11:58:40AM +0100, Catalin Marinas wrote: > On Mon, Apr 15, 2013 at 11:45:42AM +0100, Will Deacon wrote: > > On Mon, Apr 15, 2013 at 11:11:59AM +0100, Catalin Marinas wrote: > > > On Tue, Apr 09, 2013 at 01:33:34PM +0100, Christopher Covington wrote: > > > > For accurate acco

Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Catalin Marinas
On Mon, Apr 15, 2013 at 11:45:42AM +0100, Will Deacon wrote: > On Mon, Apr 15, 2013 at 11:11:59AM +0100, Catalin Marinas wrote: > > On Tue, Apr 09, 2013 at 01:33:34PM +0100, Christopher Covington wrote: > > > For accurate accounting pass contextidr_thread_switch the prev > > > task pointer, since c

Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Will Deacon
On Mon, Apr 15, 2013 at 11:11:59AM +0100, Catalin Marinas wrote: > On Tue, Apr 09, 2013 at 01:33:34PM +0100, Christopher Covington wrote: > > For accurate accounting pass contextidr_thread_switch the prev > > task pointer, since cpu_switch_to has at that point changed the > > the stack pointer. > >

Re: [PATCH v2] arm64: Fix task tracing

2013-04-15 Thread Catalin Marinas
On Tue, Apr 09, 2013 at 01:33:34PM +0100, Christopher Covington wrote: > For accurate accounting pass contextidr_thread_switch the prev > task pointer, since cpu_switch_to has at that point changed the > the stack pointer. > > Signed-off-by: Christopher Covington > --- > arch/arm64/kernel/proces

Re: [PATCH v2] arm64: Fix task tracing

2013-04-10 Thread Christopher Covington
Hi Will, On 04/10/2013 07:41 AM, Will Deacon wrote: > On Tue, Apr 09, 2013 at 01:33:34PM +0100, Christopher Covington wrote: >> For accurate accounting pass contextidr_thread_switch the prev >> task pointer, since cpu_switch_to has at that point changed the >> the stack pointer. >> >> Signed-off-b

Re: [PATCH v2] arm64: Fix task tracing

2013-04-10 Thread Will Deacon
On Tue, Apr 09, 2013 at 01:33:34PM +0100, Christopher Covington wrote: > For accurate accounting pass contextidr_thread_switch the prev > task pointer, since cpu_switch_to has at that point changed the > the stack pointer. > > Signed-off-by: Christopher Covington Thanks Christopher -- I assume t

[PATCH v2] arm64: Fix task tracing

2013-04-09 Thread Christopher Covington
For accurate accounting pass contextidr_thread_switch the prev task pointer, since cpu_switch_to has at that point changed the the stack pointer. Signed-off-by: Christopher Covington --- arch/arm64/kernel/process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/ke