Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-21 Thread Will Deacon
On Mon, Aug 20, 2012 at 09:10:59PM +0100, Arnd Bergmann wrote: > On Monday 20 August 2012, Will Deacon wrote: > > On Mon, Aug 20, 2012 at 10:07:54AM +0100, Will Deacon wrote: > > > On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: > > > > Sorry for the dumb question, but why do you

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-21 Thread Will Deacon
On Mon, Aug 20, 2012 at 09:10:59PM +0100, Arnd Bergmann wrote: On Monday 20 August 2012, Will Deacon wrote: On Mon, Aug 20, 2012 at 10:07:54AM +0100, Will Deacon wrote: On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: Sorry for the dumb question, but why do you even need

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-20 Thread Arnd Bergmann
On Monday 20 August 2012, Will Deacon wrote: > On Mon, Aug 20, 2012 at 10:07:54AM +0100, Will Deacon wrote: > > On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: > > > Sorry for the dumb question, but why do you even need > > > PTRACE_GET_THREAD_AREA > > > for 64 bit tasks? I thought

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-20 Thread Will Deacon
On Mon, Aug 20, 2012 at 10:07:54AM +0100, Will Deacon wrote: > On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: > > Sorry for the dumb question, but why do you even need PTRACE_GET_THREAD_AREA > > for 64 bit tasks? I thought the thread pointer is a GPR, or is this just > > for compat

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-20 Thread Will Deacon
On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: > Sorry for the dumb question, but why do you even need PTRACE_GET_THREAD_AREA > for 64 bit tasks? I thought the thread pointer is a GPR, or is this just > for compat tasks? The TLS is stored in a co-processor register which is

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-20 Thread Will Deacon
On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: Sorry for the dumb question, but why do you even need PTRACE_GET_THREAD_AREA for 64 bit tasks? I thought the thread pointer is a GPR, or is this just for compat tasks? The TLS is stored in a co-processor register which is read-only

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-20 Thread Will Deacon
On Mon, Aug 20, 2012 at 10:07:54AM +0100, Will Deacon wrote: On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: Sorry for the dumb question, but why do you even need PTRACE_GET_THREAD_AREA for 64 bit tasks? I thought the thread pointer is a GPR, or is this just for compat tasks?

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-20 Thread Arnd Bergmann
On Monday 20 August 2012, Will Deacon wrote: On Mon, Aug 20, 2012 at 10:07:54AM +0100, Will Deacon wrote: On Fri, Aug 17, 2012 at 08:06:07AM +0100, Arnd Bergmann wrote: Sorry for the dumb question, but why do you even need PTRACE_GET_THREAD_AREA for 64 bit tasks? I thought the thread

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-17 Thread Arnd Bergmann
On Thursday 16 August 2012, Arnd Bergmann wrote: > On Thursday 16 August 2012, Will Deacon wrote: > > On Wed, Aug 15, 2012 at 04:07:36PM +0100, Arnd Bergmann wrote: > > > On Tuesday 14 August 2012, Catalin Marinas wrote: > > > > From what I can tell, there is no support for 32 bit processes

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-17 Thread Arnd Bergmann
On Thursday 16 August 2012, Arnd Bergmann wrote: On Thursday 16 August 2012, Will Deacon wrote: On Wed, Aug 15, 2012 at 04:07:36PM +0100, Arnd Bergmann wrote: On Tuesday 14 August 2012, Catalin Marinas wrote: From what I can tell, there is no support for 32 bit processes debugging 64

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-16 Thread Arnd Bergmann
On Thursday 16 August 2012, Will Deacon wrote: > On Wed, Aug 15, 2012 at 04:07:36PM +0100, Arnd Bergmann wrote: > > On Tuesday 14 August 2012, Catalin Marinas wrote: > > From what I can tell, there is no support for 32 bit processes debugging > > 64 bit ones. Is that something you plan to add in

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-16 Thread Will Deacon
On Wed, Aug 15, 2012 at 04:07:36PM +0100, Arnd Bergmann wrote: > On Tuesday 14 August 2012, Catalin Marinas wrote: > > > +const struct user_regset_view *task_user_regset_view(struct task_struct > > *task) > > +{ > > +#ifdef CONFIG_AARCH32_EMULATION > > + if (test_tsk_thread_flag(task,

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-16 Thread Will Deacon
On Wed, Aug 15, 2012 at 04:07:36PM +0100, Arnd Bergmann wrote: On Tuesday 14 August 2012, Catalin Marinas wrote: +const struct user_regset_view *task_user_regset_view(struct task_struct *task) +{ +#ifdef CONFIG_AARCH32_EMULATION + if (test_tsk_thread_flag(task, TIF_32BIT)) +

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-16 Thread Arnd Bergmann
On Thursday 16 August 2012, Will Deacon wrote: On Wed, Aug 15, 2012 at 04:07:36PM +0100, Arnd Bergmann wrote: On Tuesday 14 August 2012, Catalin Marinas wrote: From what I can tell, there is no support for 32 bit processes debugging 64 bit ones. Is that something you plan to add in the

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-15 Thread Arnd Bergmann
On Tuesday 14 August 2012, Catalin Marinas wrote: > +const struct user_regset_view *task_user_regset_view(struct task_struct > *task) > +{ > +#ifdef CONFIG_AARCH32_EMULATION > + if (test_tsk_thread_flag(task, TIF_32BIT)) > + return _aarch32_view; > +#endif > + return

Re: [PATCH v2 23/31] arm64: Debugging support

2012-08-15 Thread Arnd Bergmann
On Tuesday 14 August 2012, Catalin Marinas wrote: +const struct user_regset_view *task_user_regset_view(struct task_struct *task) +{ +#ifdef CONFIG_AARCH32_EMULATION + if (test_tsk_thread_flag(task, TIF_32BIT)) + return user_aarch32_view; +#endif + return