Messages in this thread | | | Subject | Re: [RFC] arm:remove clear_thread_flag(TIF_UPROBE) | From | Joe Perches <> | Date | Fri, 31 Oct 2014 00:58:09 -0700 |
| |
On Fri, 2014-10-31 at 15:51 +0800, Wang, Yalin wrote: > > From: Joe Perches [mailto:joe@perches.com] > > > diff --git a/arch/arm/kernel/signal.c b/arch/arm/kernel/signal.c > > [] > > > @@ -591,10 +591,9 @@ do_work_pending(struct pt_regs *regs, unsigned int > > thread_flags, int syscall) > > > return restart; > > > } > > > syscall = 0; > > > - } else if (thread_flags & _TIF_UPROBE) { > > > - clear_thread_flag(TIF_UPROBE); > > > + } else if (thread_flags & _TIF_UPROBE) > > > uprobe_notify_resume(regs); > > > - } else { > > > + else { > > > clear_thread_flag(TIF_NOTIFY_RESUME); > > > tracehook_notify_resume(regs); > > > } > > > > Please keep the braces. > > mm.. could I know the reason ? :)
Try read Documentation/CodingStyle
Chapter 3: Placing Braces and Spaces
use braces in both branches:
if (condition) { do_this(); do_that(); } else { otherwise(); }
| |