diff options
| author | Pavel Emelianov <xemul@sw.ru> | 2007-05-08 00:30:42 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-08 11:15:13 -0700 |
| commit | 97f067846786d255888ccad14e2f38a1f63d8e9b (patch) | |
| tree | ebff291172a51b25a0a0c69c85463f0c789bb2ce | |
| parent | ee6f958291e2a768fd727e7a67badfff0b67711a (diff) | |
| download | kernel_samsung_espresso10-97f067846786d255888ccad14e2f38a1f63d8e9b.tar.gz kernel_samsung_espresso10-97f067846786d255888ccad14e2f38a1f63d8e9b.tar.bz2 kernel_samsung_espresso10-97f067846786d255888ccad14e2f38a1f63d8e9b.zip | |
jbd: check for error returned by kthread_create on creating journal thread
If the thread failed to create the subsequent wait_event will hang forever.
This is likely to happen if kernel hits max_threads limit.
Will be critical for virtualization systems that limit the number of tasks
and kernel memory usage within the container.
(akpm: JBD should be converted fully to the kthread API: kthread_should_stop()
and kthread_stop()).
Cc: <linux-ext4@vger.kernel.org>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | fs/jbd/journal.c | 13 | ||||
| -rw-r--r-- | fs/jbd2/journal.c | 13 |
2 files changed, 18 insertions, 8 deletions
diff --git a/fs/jbd/journal.c b/fs/jbd/journal.c index e1eb7e999ae..46fe7439fb9 100644 --- a/fs/jbd/journal.c +++ b/fs/jbd/journal.c @@ -210,10 +210,16 @@ end_loop: return 0; } -static void journal_start_thread(journal_t *journal) +static int journal_start_thread(journal_t *journal) { - kthread_run(kjournald, journal, "kjournald"); + struct task_struct *t; + + t = kthread_run(kjournald, journal, "kjournald"); + if (IS_ERR(t)) + return PTR_ERR(t); + wait_event(journal->j_wait_done_commit, journal->j_task != 0); + return 0; } static void journal_kill_thread(journal_t *journal) @@ -839,8 +845,7 @@ static int journal_reset(journal_t *journal) /* Add the dynamic fields and write it to disk. */ journal_update_superblock(journal, 1); - journal_start_thread(journal); - return 0; + return journal_start_thread(journal); } /** diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c index 6b3158ec9e2..78d63b818f0 100644 --- a/fs/jbd2/journal.c +++ b/fs/jbd2/journal.c @@ -210,10 +210,16 @@ end_loop: return 0; } -static void jbd2_journal_start_thread(journal_t *journal) +static int jbd2_journal_start_thread(journal_t *journal) { - kthread_run(kjournald2, journal, "kjournald2"); + struct task_struct *t; + + t = kthread_run(kjournald2, journal, "kjournald2"); + if (IS_ERR(t)) + return PTR_ERR(t); + wait_event(journal->j_wait_done_commit, journal->j_task != 0); + return 0; } static void journal_kill_thread(journal_t *journal) @@ -839,8 +845,7 @@ static int journal_reset(journal_t *journal) /* Add the dynamic fields and write it to disk. */ jbd2_journal_update_superblock(journal, 1); - jbd2_journal_start_thread(journal); - return 0; + return jbd2_journal_start_thread(journal); } /** |