Re: [PATCH] ARM : unwinder : Prevent data abort due to stack overflow

2013-12-09 Thread Dave Martin
On Sat, Dec 07, 2013 at 01:43:21PM +0530, Anurag Aggarwal wrote: > >> > >> + /* we are just starting, initialize last register set as 0 */ > >> + ctrl.last_register_set = 0; > >> + > >> while (ctrl.entries > 0) { > >> - int urc = unwind_exec_insn(&ctrl); > >> +

Re: [PATCH] ARM : unwinder : Prevent data abort due to stack overflow

2013-12-07 Thread Anurag Aggarwal
>> >> + /* we are just starting, initialize last register set as 0 */ >> + ctrl.last_register_set = 0; >> + >> while (ctrl.entries > 0) { >> - int urc = unwind_exec_insn(&ctrl); >> + int urc; >> + if ((ctrl.sp_high - ctrl.vrs[SP]) < TOTAL_REGISTERS)

Re: [PATCH] ARM : unwinder : Prevent data abort due to stack overflow

2013-12-06 Thread Dave Martin
On Fri, Dec 06, 2013 at 06:09:53AM +, Anurag Aggarwal wrote: > While unwinding backtrace, stack overflow is possible. This stack > overflow can sometimes lead to data abort in system if the area after > stack is not mapped to physical memory. > > To prevent this problem from happening, execute

[PATCH] ARM : unwinder : Prevent data abort due to stack overflow

2013-12-05 Thread Anurag Aggarwal
While unwinding backtrace, stack overflow is possible. This stack overflow can sometimes lead to data abort in system if the area after stack is not mapped to physical memory. To prevent this problem from happening, execute the instructions that can cause a data abort in separate helper functions,

Re: [PATCH] ARM : unwinder : Prevent data abort due to stack overflow in unwind_exec_insn Signed-off-by: Anurag Aggarwal

2013-11-28 Thread Anurag Aggarwal
+05:30) Title : Re: [PATCH] ARM : unwinder : Prevent data abort due to stack overflow in unwind_exec_insn Signed-off-by: Anurag Aggarwal On Thu, Nov 28, 2013 at 03:57:19PM +0530, Anurag Aggarwal wrote: > While executing some unwind instructions stack overflow can cause a data abort > when

Re: [PATCH] ARM : unwinder : Prevent data abort due to stack overflow in unwind_exec_insn Signed-off-by: Anurag Aggarwal

2013-11-28 Thread Dave Martin
On Thu, Nov 28, 2013 at 03:57:19PM +0530, Anurag Aggarwal wrote: > While executing some unwind instructions stack overflow can cause a data abort > when area beyond stack is not mapped to physical memory. > > To prevent the data abort check whether it is possible to execute > these instructions be

[PATCH] ARM : unwinder : Prevent data abort due to stack overflow in unwind_exec_insn Signed-off-by: Anurag Aggarwal

2013-11-28 Thread Anurag Aggarwal
While executing some unwind instructions stack overflow can cause a data abort when area beyond stack is not mapped to physical memory. To prevent the data abort check whether it is possible to execute these instructions before unwinding the stack --- arch/arm/kernel/unwind.c | 59 +