No changes in the compiled code. Just make the code parsable by humans.

Signed-off-by: Oleg Nesterov <o...@redhat.com>
---

 kernel/utrace.c |   16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

--- kstub/kernel/utrace.c~1_use_cleanup_exit_state_checks       2010-08-14 
04:28:58.000000000 +0200
+++ kstub/kernel/utrace.c       2010-08-14 04:30:28.000000000 +0200
@@ -539,13 +539,15 @@ int utrace_set_events(struct task_struct
        old_utrace_flags = target->utrace_flags;
        old_flags = engine->flags & ~ENGINE_STOP;
 
-       if (target->exit_state &&
-           (((events & ~old_flags) & _UTRACE_DEATH_EVENTS) ||
-            (utrace->death &&
-             ((old_flags & ~events) & _UTRACE_DEATH_EVENTS)) ||
-            (utrace->reap && ((old_flags & ~events) & UTRACE_EVENT(REAP))))) {
-               spin_unlock(&utrace->lock);
-               return -EALREADY;
+       if (target->exit_state) {
+               unsigned long cleared = (old_flags & ~events);
+
+               if (((events & ~old_flags) & _UTRACE_DEATH_EVENTS) ||
+                   (utrace->death && (cleared & _UTRACE_DEATH_EVENTS)) ||
+                   (utrace->reap  && (cleared &  UTRACE_EVENT(REAP)))) {
+                       spin_unlock(&utrace->lock);
+                       return -EALREADY;
+               }
        }
 
        /*

Reply via email to