Re: [PATCH] arm64: setup return path for el1_undef

2014-12-23 Thread Min-Hua Chen
On Tue, Dec 23, 2014 at 11:57 PM, Catalin Marinas wrote: > On Tue, Dec 23, 2014 at 03:15:10PM +, Min-Hua Chen wrote: >> Setup return path for el1_undef since el1_undef may >> be handled by handlers. > > Did you find a real issue or it was just code inspection. Thanks for your reply. It was

Re: [PATCH] arm64: setup return path for el1_undef

2014-12-23 Thread Catalin Marinas
On Tue, Dec 23, 2014 at 03:15:10PM +, Min-Hua Chen wrote: > Setup return path for el1_undef since el1_undef may > be handled by handlers. Did you find a real issue or it was just code inspection. > asmlinkage void __exception do_undefinstr(struct pt_regs *regs) > { > siginfo_t info; >

[PATCH] arm64: setup return path for el1_undef

2014-12-23 Thread Min-Hua Chen
Setup return path for el1_undef since el1_undef may be handled by handlers. asmlinkage void __exception do_undefinstr(struct pt_regs *regs) { siginfo_t info; void __user *pc = (void __user *)instruction_pointer(regs); /* check for AArch32 breakpoint instructions */ if

[PATCH] arm64: setup return path for el1_undef

2014-12-23 Thread Min-Hua Chen
Setup return path for el1_undef since el1_undef may be handled by handlers. asmlinkage void __exception do_undefinstr(struct pt_regs *regs) { siginfo_t info; void __user *pc = (void __user *)instruction_pointer(regs); /* check for AArch32 breakpoint instructions */ if

Re: [PATCH] arm64: setup return path for el1_undef

2014-12-23 Thread Catalin Marinas
On Tue, Dec 23, 2014 at 03:15:10PM +, Min-Hua Chen wrote: Setup return path for el1_undef since el1_undef may be handled by handlers. Did you find a real issue or it was just code inspection. asmlinkage void __exception do_undefinstr(struct pt_regs *regs) { siginfo_t info; void

Re: [PATCH] arm64: setup return path for el1_undef

2014-12-23 Thread Min-Hua Chen
On Tue, Dec 23, 2014 at 11:57 PM, Catalin Marinas catalin.mari...@arm.com wrote: On Tue, Dec 23, 2014 at 03:15:10PM +, Min-Hua Chen wrote: Setup return path for el1_undef since el1_undef may be handled by handlers. Did you find a real issue or it was just code inspection. Thanks for your