[PATCH -mm 3/2] do_wait: factor out "retval != 0" checks

[Oleg Nesterov]

Every branch if the main "if" statement does the same code at the end. Move it
down. Also, fix the indentation.

Signed-off-by: Oleg Nesterov <[EMAIL PROTECTED]>

--- PT/kernel/exit.c~3_result   2007-11-22 18:06:01.000000000 +0300
+++ PT/kernel/exit.c    2007-11-22 19:08:43.000000000 +0300
@@ -1508,6 +1508,7 @@ repeat:
                        }
                        allowed = 1;
 
+                       retval = 0;
                        if (is_task_stopped_or_traced(p)) {
                                /*
                                 * It's stopped now, so it might later
@@ -1521,8 +1522,6 @@ repeat:
                                retval = wait_task_stopped(p, ret == 2,
                                                (options & WNOWAIT), infop,
                                                stat_addr, ru);
-                               if (retval != 0) /* He released the lock.  */
-                                       goto end;
                        } else if (p->exit_state == EXIT_ZOMBIE) {
                                /*
                                 * Eligible but we cannot release it yet:
@@ -1534,9 +1533,6 @@ repeat:
                                retval = wait_task_zombie(p,
                                                (options & WNOWAIT), infop,
                                                stat_addr, ru);
-                               /* He released the lock.  */
-                               if (retval != 0)
-                                       goto end;
                        } else if (p->exit_state != EXIT_DEAD) {
 check_continued:
                                /*
@@ -1549,9 +1545,9 @@ check_continued:
                                retval = wait_task_continued(p,
                                                (options & WNOWAIT), infop,
                                                stat_addr, ru);
-                               if (retval != 0) /* He released the lock.  */
-                                       goto end;
                        }
+                       if (retval != 0) /* tasklist_lock released */
+                               goto end;
                }
                if (!flag) {
                        list_for_each_entry(p, &tsk->ptrace_children,
@@ -1587,7 +1583,7 @@ end:
        remove_wait_queue(&current->signal->wait_chldexit,&wait);
        if (infop) {
                if (retval > 0)
-               retval = 0;
+                       retval = 0;
                else {
                        /*
                         * For a WNOHANG return, clear out all the fields

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/