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

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

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

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

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

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

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 c...@codeaurora.org ---

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

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 accounting

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 +0100,

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 can't see

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 +0100,

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. >> >>

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

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 c...@codeaurora.org Thanks

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-by:

[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

[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 c...@codeaurora.org --- arch/arm64/kernel/process.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff