diff options
| -rw-r--r-- | fs/jbd2/checkpoint.c | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/fs/jbd2/checkpoint.c b/fs/jbd2/checkpoint.c index 91389c8aee8a..af4651bf3570 100644 --- a/fs/jbd2/checkpoint.c +++ b/fs/jbd2/checkpoint.c | |||
| @@ -126,14 +126,29 @@ void __jbd2_log_wait_for_space(journal_t *journal) | |||
| 126 | 126 | ||
| 127 | /* | 127 | /* |
| 128 | * Test again, another process may have checkpointed while we | 128 | * Test again, another process may have checkpointed while we |
| 129 | * were waiting for the checkpoint lock | 129 | * were waiting for the checkpoint lock. If there are no |
| 130 | * outstanding transactions there is nothing to checkpoint and | ||
| 131 | * we can't make progress. Abort the journal in this case. | ||
| 130 | */ | 132 | */ |
| 131 | spin_lock(&journal->j_state_lock); | 133 | spin_lock(&journal->j_state_lock); |
| 134 | spin_lock(&journal->j_list_lock); | ||
| 132 | nblocks = jbd_space_needed(journal); | 135 | nblocks = jbd_space_needed(journal); |
| 133 | if (__jbd2_log_space_left(journal) < nblocks) { | 136 | if (__jbd2_log_space_left(journal) < nblocks) { |
| 137 | int chkpt = journal->j_checkpoint_transactions != NULL; | ||
| 138 | |||
| 139 | spin_unlock(&journal->j_list_lock); | ||
| 134 | spin_unlock(&journal->j_state_lock); | 140 | spin_unlock(&journal->j_state_lock); |
| 135 | jbd2_log_do_checkpoint(journal); | 141 | if (chkpt) { |
| 142 | jbd2_log_do_checkpoint(journal); | ||
| 143 | } else { | ||
| 144 | printk(KERN_ERR "%s: no transactions\n", | ||
| 145 | __func__); | ||
| 146 | jbd2_journal_abort(journal, 0); | ||
| 147 | } | ||
| 148 | |||
| 136 | spin_lock(&journal->j_state_lock); | 149 | spin_lock(&journal->j_state_lock); |
| 150 | } else { | ||
| 151 | spin_unlock(&journal->j_list_lock); | ||
| 137 | } | 152 | } |
| 138 | mutex_unlock(&journal->j_checkpoint_mutex); | 153 | mutex_unlock(&journal->j_checkpoint_mutex); |
| 139 | } | 154 | } |
