audit: flatten kauditd_thread wait queue code
authorRichard Guy Briggs <rgb@redhat.com>
Thu, 24 Jan 2013 18:15:11 +0000 (13:15 -0500)
committerEric Paris <eparis@redhat.com>
Mon, 8 Apr 2013 20:19:17 +0000 (16:19 -0400)
The wait queue control code in kauditd_thread() was nested deeper than
necessary.  The function has been flattened for better legibility.

Signed-off-by: Richard Guy Briggs <rbriggs@redhat.com>
Signed-off-by: Eric Paris <eparis@redhat.com>
kernel/audit.c

index 4bf486c3e9e87ea4a05c92bf6f3b495599fc7d71..1531efbd11e2fe645b4a9c14898487caca914675 100644 (file)
@@ -458,10 +458,11 @@ static void flush_hold_queue(void)
 
 static int kauditd_thread(void *dummy)
 {
-       struct sk_buff *skb;
-
        set_freezable();
        while (!kthread_should_stop()) {
+               struct sk_buff *skb;
+               DECLARE_WAITQUEUE(wait, current);
+
                flush_hold_queue();
 
                skb = skb_dequeue(&audit_skb_queue);
@@ -471,19 +472,18 @@ static int kauditd_thread(void *dummy)
                                kauditd_send_skb(skb);
                        else
                                audit_printk_skb(skb);
-               } else {
-                       DECLARE_WAITQUEUE(wait, current);
-                       set_current_state(TASK_INTERRUPTIBLE);
-                       add_wait_queue(&kauditd_wait, &wait);
-
-                       if (!skb_queue_len(&audit_skb_queue)) {
-                               try_to_freeze();
-                               schedule();
-                       }
+                       continue;
+               }
+               set_current_state(TASK_INTERRUPTIBLE);
+               add_wait_queue(&kauditd_wait, &wait);
 
-                       __set_current_state(TASK_RUNNING);
-                       remove_wait_queue(&kauditd_wait, &wait);
+               if (!skb_queue_len(&audit_skb_queue)) {
+                       try_to_freeze();
+                       schedule();
                }
+
+               __set_current_state(TASK_RUNNING);
+               remove_wait_queue(&kauditd_wait, &wait);
        }
        return 0;
 }