[PATCH] kjournald exits with nonzero preempt_count

The preempt_count debug check that went into 2.5.8-pre1 already caught a
simple case in kjournald.  Specifically, kjournald does not drop the BKL
when it exits as it knows schedule will do so for it.

For the sake of clarity and exiting with a preempt_count of zero, the
attached patch explicitly calls unlock_kernel when kjournald is exiting.

diff --git a/fs/jbd/journal.c b/fs/jbd/journal.c
index 7f0e69be25ed882920aeb7a6369205d789f49c25..e0ea5fc1de80f1f06d88bbf1da27b6a1c899a4cd 100644 (file)
--- a/fs/jbd/journal.c
+++ b/fs/jbd/journal.c
@@ -267,6 +267,7 @@ int kjournald(void *arg)
        journal->j_task = NULL;
        wake_up(&journal->j_wait_done_commit);
        jbd_debug(1, "Journal thread exiting.\n");
+       unlock_kernel();
        return 0;
 }