blob: 559bec1a37b429bed7f639866fbadb6c148b10a0 [file] [log] [blame]
Dave Kleikamp470decc2006-10-11 01:20:57 -07001/*
Mingming Caof7f4bcc2006-10-11 01:20:59 -07002 * linux/fs/jbd2/commit.c
Dave Kleikamp470decc2006-10-11 01:20:57 -07003 *
4 * Written by Stephen C. Tweedie <sct@redhat.com>, 1998
5 *
6 * Copyright 1998 Red Hat corp --- All Rights Reserved
7 *
8 * This file is part of the Linux kernel and is made available under
9 * the terms of the GNU General Public License, version 2, or at your
10 * option, any later version, incorporated herein by reference.
11 *
12 * Journal commit routines for the generic filesystem journaling code;
13 * part of the ext2fs journaling system.
14 */
15
16#include <linux/time.h>
17#include <linux/fs.h>
Mingming Caof7f4bcc2006-10-11 01:20:59 -070018#include <linux/jbd2.h>
Dave Kleikamp470decc2006-10-11 01:20:57 -070019#include <linux/errno.h>
20#include <linux/slab.h>
21#include <linux/mm.h>
22#include <linux/pagemap.h>
Johann Lombardi8e85fb32008-01-28 23:58:27 -050023#include <linux/jiffies.h>
Girish Shilamkar818d2762008-01-28 23:58:27 -050024#include <linux/crc32.h>
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -040025#include <linux/writeback.h>
26#include <linux/backing-dev.h>
Theodore Ts'ofd984962009-01-05 21:34:13 -050027#include <linux/bio.h>
Theodore Ts'o0e3d2a62009-09-11 09:30:12 -040028#include <linux/blkdev.h>
Brian King39e3ac22010-10-27 21:25:12 -040029#include <linux/bitops.h>
Theodore Ts'o879c5e62009-06-17 11:47:48 -040030#include <trace/events/jbd2.h>
Dave Kleikamp470decc2006-10-11 01:20:57 -070031
32/*
Jan Karab34090e2013-06-04 12:08:56 -040033 * IO end handler for temporary buffer_heads handling writes to the journal.
Dave Kleikamp470decc2006-10-11 01:20:57 -070034 */
35static void journal_end_buffer_io_sync(struct buffer_head *bh, int uptodate)
36{
Jan Karab34090e2013-06-04 12:08:56 -040037 struct buffer_head *orig_bh = bh->b_private;
38
Dave Kleikamp470decc2006-10-11 01:20:57 -070039 BUFFER_TRACE(bh, "");
40 if (uptodate)
41 set_buffer_uptodate(bh);
42 else
43 clear_buffer_uptodate(bh);
Jan Karab34090e2013-06-04 12:08:56 -040044 if (orig_bh) {
45 clear_bit_unlock(BH_Shadow, &orig_bh->b_state);
46 smp_mb__after_clear_bit();
47 wake_up_bit(&orig_bh->b_state, BH_Shadow);
48 }
Dave Kleikamp470decc2006-10-11 01:20:57 -070049 unlock_buffer(bh);
50}
51
52/*
Jan Kara87c89c22008-07-11 19:27:31 -040053 * When an ext4 file is truncated, it is possible that some pages are not
54 * successfully freed, because they are attached to a committing transaction.
Dave Kleikamp470decc2006-10-11 01:20:57 -070055 * After the transaction commits, these pages are left on the LRU, with no
56 * ->mapping, and with attached buffers. These pages are trivially reclaimable
57 * by the VM, but their apparent absence upsets the VM accounting, and it makes
58 * the numbers in /proc/meminfo look odd.
59 *
60 * So here, we have a buffer which has just come off the forget list. Look to
61 * see if we can strip all buffers from the backing page.
62 *
63 * Called under lock_journal(), and possibly under journal_datalist_lock. The
64 * caller provided us with a ref against the buffer, and we drop that here.
65 */
66static void release_buffer_page(struct buffer_head *bh)
67{
68 struct page *page;
69
70 if (buffer_dirty(bh))
71 goto nope;
72 if (atomic_read(&bh->b_count) != 1)
73 goto nope;
74 page = bh->b_page;
75 if (!page)
76 goto nope;
77 if (page->mapping)
78 goto nope;
79
80 /* OK, it's a truncated page */
Nick Piggin529ae9a2008-08-02 12:01:03 +020081 if (!trylock_page(page))
Dave Kleikamp470decc2006-10-11 01:20:57 -070082 goto nope;
83
84 page_cache_get(page);
85 __brelse(bh);
86 try_to_free_buffers(page);
87 unlock_page(page);
88 page_cache_release(page);
89 return;
90
91nope:
92 __brelse(bh);
93}
94
Jan Karae5a120a2013-06-04 12:06:01 -040095static void jbd2_commit_block_csum_set(journal_t *j, struct buffer_head *bh)
Darrick J. Wong1f56c582012-05-27 08:10:25 -040096{
97 struct commit_header *h;
98 __u32 csum;
99
100 if (!JBD2_HAS_INCOMPAT_FEATURE(j, JBD2_FEATURE_INCOMPAT_CSUM_V2))
101 return;
102
Jan Karae5a120a2013-06-04 12:06:01 -0400103 h = (struct commit_header *)(bh->b_data);
Darrick J. Wong1f56c582012-05-27 08:10:25 -0400104 h->h_chksum_type = 0;
105 h->h_chksum_size = 0;
106 h->h_chksum[0] = 0;
Jan Karae5a120a2013-06-04 12:06:01 -0400107 csum = jbd2_chksum(j, j->j_csum_seed, bh->b_data, j->j_blocksize);
Darrick J. Wong1f56c582012-05-27 08:10:25 -0400108 h->h_chksum[0] = cpu_to_be32(csum);
109}
110
Dave Kleikamp470decc2006-10-11 01:20:57 -0700111/*
Girish Shilamkar818d2762008-01-28 23:58:27 -0500112 * Done it all: now submit the commit record. We should have
Dave Kleikamp470decc2006-10-11 01:20:57 -0700113 * cleaned up our previous buffers by now, so if we are in abort
114 * mode we can now just skip the rest of the journal write
115 * entirely.
116 *
117 * Returns 1 if the journal needs to be aborted or 0 on success
118 */
Girish Shilamkar818d2762008-01-28 23:58:27 -0500119static int journal_submit_commit_record(journal_t *journal,
120 transaction_t *commit_transaction,
121 struct buffer_head **cbh,
122 __u32 crc32_sum)
Dave Kleikamp470decc2006-10-11 01:20:57 -0700123{
Girish Shilamkar818d2762008-01-28 23:58:27 -0500124 struct commit_header *tmp;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700125 struct buffer_head *bh;
Girish Shilamkar818d2762008-01-28 23:58:27 -0500126 int ret;
Theodore Ts'o736603a2008-07-11 19:27:31 -0400127 struct timespec now = current_kernel_time();
Dave Kleikamp470decc2006-10-11 01:20:57 -0700128
Zhang Huan6cba6112011-04-05 19:16:20 -0400129 *cbh = NULL;
130
Dave Kleikamp470decc2006-10-11 01:20:57 -0700131 if (is_journal_aborted(journal))
132 return 0;
133
Jan Karae5a120a2013-06-04 12:06:01 -0400134 bh = jbd2_journal_get_descriptor_buffer(journal);
135 if (!bh)
Dave Kleikamp470decc2006-10-11 01:20:57 -0700136 return 1;
137
Girish Shilamkar818d2762008-01-28 23:58:27 -0500138 tmp = (struct commit_header *)bh->b_data;
139 tmp->h_magic = cpu_to_be32(JBD2_MAGIC_NUMBER);
140 tmp->h_blocktype = cpu_to_be32(JBD2_COMMIT_BLOCK);
141 tmp->h_sequence = cpu_to_be32(commit_transaction->t_tid);
Theodore Ts'o736603a2008-07-11 19:27:31 -0400142 tmp->h_commit_sec = cpu_to_be64(now.tv_sec);
143 tmp->h_commit_nsec = cpu_to_be32(now.tv_nsec);
Girish Shilamkar818d2762008-01-28 23:58:27 -0500144
145 if (JBD2_HAS_COMPAT_FEATURE(journal,
146 JBD2_FEATURE_COMPAT_CHECKSUM)) {
147 tmp->h_chksum_type = JBD2_CRC32_CHKSUM;
148 tmp->h_chksum_size = JBD2_CRC32_CHKSUM_SIZE;
149 tmp->h_chksum[0] = cpu_to_be32(crc32_sum);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700150 }
Jan Karae5a120a2013-06-04 12:06:01 -0400151 jbd2_commit_block_csum_set(journal, bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700152
Jan Karae5a120a2013-06-04 12:06:01 -0400153 BUFFER_TRACE(bh, "submit commit block");
Girish Shilamkar818d2762008-01-28 23:58:27 -0500154 lock_buffer(bh);
Theodore Ts'o45a90bf2008-10-06 12:04:02 -0400155 clear_buffer_dirty(bh);
Girish Shilamkar818d2762008-01-28 23:58:27 -0500156 set_buffer_uptodate(bh);
157 bh->b_end_io = journal_end_buffer_io_sync;
158
159 if (journal->j_flags & JBD2_BARRIER &&
Theodore Ts'o0e3d2a62009-09-11 09:30:12 -0400160 !JBD2_HAS_INCOMPAT_FEATURE(journal,
Christoph Hellwig9c355752010-08-18 05:29:17 -0400161 JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT))
Jens Axboe721a9602011-03-09 11:56:30 +0100162 ret = submit_bh(WRITE_SYNC | WRITE_FLUSH_FUA, bh);
Christoph Hellwig9c355752010-08-18 05:29:17 -0400163 else
Jens Axboe721a9602011-03-09 11:56:30 +0100164 ret = submit_bh(WRITE_SYNC, bh);
Christoph Hellwig9c355752010-08-18 05:29:17 -0400165
Girish Shilamkar818d2762008-01-28 23:58:27 -0500166 *cbh = bh;
167 return ret;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700168}
169
Girish Shilamkar818d2762008-01-28 23:58:27 -0500170/*
171 * This function along with journal_submit_commit_record
172 * allows to write the commit record asynchronously.
173 */
Theodore Ts'ofd984962009-01-05 21:34:13 -0500174static int journal_wait_on_commit_record(journal_t *journal,
175 struct buffer_head *bh)
Girish Shilamkar818d2762008-01-28 23:58:27 -0500176{
177 int ret = 0;
178
179 clear_buffer_dirty(bh);
180 wait_on_buffer(bh);
181
182 if (unlikely(!buffer_uptodate(bh)))
183 ret = -EIO;
184 put_bh(bh); /* One for getblk() */
Girish Shilamkar818d2762008-01-28 23:58:27 -0500185
186 return ret;
187}
188
189/*
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400190 * write the filemap data using writepage() address_space_operations.
191 * We don't do block allocation here even for delalloc. We don't
192 * use writepages() because with dealyed allocation we may be doing
193 * block allocation in writepages().
194 */
195static int journal_submit_inode_data_buffers(struct address_space *mapping)
196{
197 int ret;
198 struct writeback_control wbc = {
199 .sync_mode = WB_SYNC_ALL,
200 .nr_to_write = mapping->nrpages * 2,
201 .range_start = 0,
202 .range_end = i_size_read(mapping->host),
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400203 };
204
205 ret = generic_writepages(mapping, &wbc);
206 return ret;
207}
208
209/*
Jan Karac851ed52008-07-11 19:27:31 -0400210 * Submit all the data buffers of inode associated with the transaction to
211 * disk.
212 *
213 * We are in a committing transaction. Therefore no new inode can be added to
214 * our inode list. We use JI_COMMIT_RUNNING flag to protect inode we currently
215 * operate on from being released while we write out pages.
216 */
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400217static int journal_submit_data_buffers(journal_t *journal,
Jan Karac851ed52008-07-11 19:27:31 -0400218 transaction_t *commit_transaction)
219{
220 struct jbd2_inode *jinode;
221 int err, ret = 0;
222 struct address_space *mapping;
223
224 spin_lock(&journal->j_list_lock);
225 list_for_each_entry(jinode, &commit_transaction->t_inode_list, i_list) {
226 mapping = jinode->i_vfs_inode->i_mapping;
Brian King39e3ac22010-10-27 21:25:12 -0400227 set_bit(__JI_COMMIT_RUNNING, &jinode->i_flags);
Jan Karac851ed52008-07-11 19:27:31 -0400228 spin_unlock(&journal->j_list_lock);
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400229 /*
230 * submit the inode data buffers. We use writepage
231 * instead of writepages. Because writepages can do
232 * block allocation with delalloc. We need to write
233 * only allocated blocks here.
234 */
Theodore Ts'o879c5e62009-06-17 11:47:48 -0400235 trace_jbd2_submit_inode_data(jinode->i_vfs_inode);
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400236 err = journal_submit_inode_data_buffers(mapping);
Jan Karac851ed52008-07-11 19:27:31 -0400237 if (!ret)
238 ret = err;
239 spin_lock(&journal->j_list_lock);
240 J_ASSERT(jinode->i_transaction == commit_transaction);
Brian King39e3ac22010-10-27 21:25:12 -0400241 clear_bit(__JI_COMMIT_RUNNING, &jinode->i_flags);
242 smp_mb__after_clear_bit();
Jan Karac851ed52008-07-11 19:27:31 -0400243 wake_up_bit(&jinode->i_flags, __JI_COMMIT_RUNNING);
244 }
245 spin_unlock(&journal->j_list_lock);
246 return ret;
247}
248
249/*
250 * Wait for data submitted for writeout, refile inodes to proper
251 * transaction if needed.
252 *
253 */
254static int journal_finish_inode_data_buffers(journal_t *journal,
255 transaction_t *commit_transaction)
256{
257 struct jbd2_inode *jinode, *next_i;
258 int err, ret = 0;
259
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400260 /* For locking, see the comment in journal_submit_data_buffers() */
Jan Karac851ed52008-07-11 19:27:31 -0400261 spin_lock(&journal->j_list_lock);
262 list_for_each_entry(jinode, &commit_transaction->t_inode_list, i_list) {
Brian King39e3ac22010-10-27 21:25:12 -0400263 set_bit(__JI_COMMIT_RUNNING, &jinode->i_flags);
Jan Karac851ed52008-07-11 19:27:31 -0400264 spin_unlock(&journal->j_list_lock);
265 err = filemap_fdatawait(jinode->i_vfs_inode->i_mapping);
Hidehiro Kawaie9e34f42008-07-31 22:26:04 -0400266 if (err) {
267 /*
268 * Because AS_EIO is cleared by
Christoph Hellwig94004ed2009-09-30 22:16:33 +0200269 * filemap_fdatawait_range(), set it again so
Hidehiro Kawaie9e34f42008-07-31 22:26:04 -0400270 * that user process can get -EIO from fsync().
271 */
272 set_bit(AS_EIO,
273 &jinode->i_vfs_inode->i_mapping->flags);
274
275 if (!ret)
276 ret = err;
277 }
Jan Karac851ed52008-07-11 19:27:31 -0400278 spin_lock(&journal->j_list_lock);
Brian King39e3ac22010-10-27 21:25:12 -0400279 clear_bit(__JI_COMMIT_RUNNING, &jinode->i_flags);
280 smp_mb__after_clear_bit();
Jan Karac851ed52008-07-11 19:27:31 -0400281 wake_up_bit(&jinode->i_flags, __JI_COMMIT_RUNNING);
282 }
283
284 /* Now refile inode to proper lists */
285 list_for_each_entry_safe(jinode, next_i,
286 &commit_transaction->t_inode_list, i_list) {
287 list_del(&jinode->i_list);
288 if (jinode->i_next_transaction) {
289 jinode->i_transaction = jinode->i_next_transaction;
290 jinode->i_next_transaction = NULL;
291 list_add(&jinode->i_list,
292 &jinode->i_transaction->t_inode_list);
293 } else {
294 jinode->i_transaction = NULL;
295 }
296 }
297 spin_unlock(&journal->j_list_lock);
298
299 return ret;
300}
301
Girish Shilamkar818d2762008-01-28 23:58:27 -0500302static __u32 jbd2_checksum_data(__u32 crc32_sum, struct buffer_head *bh)
303{
304 struct page *page = bh->b_page;
305 char *addr;
306 __u32 checksum;
307
Cong Wang303a8f22011-11-25 23:14:31 +0800308 addr = kmap_atomic(page);
Girish Shilamkar818d2762008-01-28 23:58:27 -0500309 checksum = crc32_be(crc32_sum,
310 (void *)(addr + offset_in_page(bh->b_data)), bh->b_size);
Cong Wang303a8f22011-11-25 23:14:31 +0800311 kunmap_atomic(addr);
Girish Shilamkar818d2762008-01-28 23:58:27 -0500312
313 return checksum;
314}
315
316static void write_tag_block(int tag_bytes, journal_block_tag_t *tag,
Mingming Cao18eba7a2006-10-11 01:21:13 -0700317 unsigned long long block)
Zach Brownb517bea2006-10-11 01:21:08 -0700318{
319 tag->t_blocknr = cpu_to_be32(block & (u32)~0);
Mingming Caocd02ff02007-10-16 18:38:25 -0400320 if (tag_bytes > JBD2_TAG_SIZE32)
Zach Brownb517bea2006-10-11 01:21:08 -0700321 tag->t_blocknr_high = cpu_to_be32((block >> 31) >> 1);
322}
323
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400324static void jbd2_descr_block_csum_set(journal_t *j,
Jan Karae5a120a2013-06-04 12:06:01 -0400325 struct buffer_head *bh)
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400326{
327 struct jbd2_journal_block_tail *tail;
328 __u32 csum;
329
330 if (!JBD2_HAS_INCOMPAT_FEATURE(j, JBD2_FEATURE_INCOMPAT_CSUM_V2))
331 return;
332
Jan Karae5a120a2013-06-04 12:06:01 -0400333 tail = (struct jbd2_journal_block_tail *)(bh->b_data + j->j_blocksize -
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400334 sizeof(struct jbd2_journal_block_tail));
335 tail->t_checksum = 0;
Jan Karae5a120a2013-06-04 12:06:01 -0400336 csum = jbd2_chksum(j, j->j_csum_seed, bh->b_data, j->j_blocksize);
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400337 tail->t_checksum = cpu_to_be32(csum);
338}
339
Darrick J. Wongc3900872012-05-27 08:12:12 -0400340static void jbd2_block_tag_csum_set(journal_t *j, journal_block_tag_t *tag,
341 struct buffer_head *bh, __u32 sequence)
342{
343 struct page *page = bh->b_page;
344 __u8 *addr;
Darrick J. Wongeee06c52013-05-28 07:31:59 -0400345 __u32 csum32;
Darrick J. Wongc3900872012-05-27 08:12:12 -0400346
347 if (!JBD2_HAS_INCOMPAT_FEATURE(j, JBD2_FEATURE_INCOMPAT_CSUM_V2))
348 return;
349
350 sequence = cpu_to_be32(sequence);
Cong Wang906adea2012-06-23 11:24:48 +0800351 addr = kmap_atomic(page);
Darrick J. Wongeee06c52013-05-28 07:31:59 -0400352 csum32 = jbd2_chksum(j, j->j_csum_seed, (__u8 *)&sequence,
353 sizeof(sequence));
354 csum32 = jbd2_chksum(j, csum32, addr + offset_in_page(bh->b_data),
355 bh->b_size);
Cong Wang906adea2012-06-23 11:24:48 +0800356 kunmap_atomic(addr);
Darrick J. Wongc3900872012-05-27 08:12:12 -0400357
Darrick J. Wongeee06c52013-05-28 07:31:59 -0400358 /* We only have space to store the lower 16 bits of the crc32c. */
359 tag->t_checksum = cpu_to_be16(csum32);
Darrick J. Wongc3900872012-05-27 08:12:12 -0400360}
Dave Kleikamp470decc2006-10-11 01:20:57 -0700361/*
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700362 * jbd2_journal_commit_transaction
Dave Kleikamp470decc2006-10-11 01:20:57 -0700363 *
364 * The primary function for committing a transaction to the log. This
365 * function is called by the journal thread to begin a complete commit.
366 */
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700367void jbd2_journal_commit_transaction(journal_t *journal)
Dave Kleikamp470decc2006-10-11 01:20:57 -0700368{
Johann Lombardi8e85fb32008-01-28 23:58:27 -0500369 struct transaction_stats_s stats;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700370 transaction_t *commit_transaction;
Jan Karae5a120a2013-06-04 12:06:01 -0400371 struct journal_head *jh;
372 struct buffer_head *descriptor;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700373 struct buffer_head **wbuf = journal->j_wbuf;
374 int bufs;
375 int flags;
376 int err;
Mingming Cao18eba7a2006-10-11 01:21:13 -0700377 unsigned long long blocknr;
Josef Bacike07f7182008-11-26 01:14:26 -0500378 ktime_t start_time;
379 u64 commit_time;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700380 char *tagp = NULL;
381 journal_header_t *header;
382 journal_block_tag_t *tag = NULL;
383 int space_left = 0;
384 int first_tag = 0;
385 int tag_flag;
Dmitry Monakhov794446c2013-04-03 22:06:52 -0400386 int i;
Zach Brownb517bea2006-10-11 01:21:08 -0700387 int tag_bytes = journal_tag_bytes(journal);
Girish Shilamkar818d2762008-01-28 23:58:27 -0500388 struct buffer_head *cbh = NULL; /* For transactional checksums */
389 __u32 crc32_sum = ~0;
Jens Axboe82f04ab2011-03-17 11:01:52 +0100390 struct blk_plug plug;
Jan Kara33395782012-03-13 22:45:38 -0400391 /* Tail of the journal */
392 unsigned long first_block;
393 tid_t first_tid;
394 int update_tail;
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400395 int csum_size = 0;
Jan Karaf5113ef2013-06-04 12:01:45 -0400396 LIST_HEAD(io_bufs);
Jan Karae5a120a2013-06-04 12:06:01 -0400397 LIST_HEAD(log_bufs);
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400398
399 if (JBD2_HAS_INCOMPAT_FEATURE(journal, JBD2_FEATURE_INCOMPAT_CSUM_V2))
400 csum_size = sizeof(struct jbd2_journal_block_tail);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700401
402 /*
403 * First job: lock down the current transaction and wait for
404 * all outstanding updates to complete.
405 */
406
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700407 /* Do we need to erase the effects of a prior jbd2_journal_flush? */
408 if (journal->j_flags & JBD2_FLUSHED) {
Dave Kleikamp470decc2006-10-11 01:20:57 -0700409 jbd_debug(3, "super block updated\n");
Jan Karaa78bb112012-03-13 15:43:04 -0400410 mutex_lock(&journal->j_checkpoint_mutex);
Jan Kara79feb522012-03-13 22:22:54 -0400411 /*
412 * We hold j_checkpoint_mutex so tail cannot change under us.
413 * We don't need any special data guarantees for writing sb
414 * since journal is empty and it is ok for write to be
415 * flushed only with transaction commit.
416 */
417 jbd2_journal_update_sb_log_tail(journal,
418 journal->j_tail_sequence,
419 journal->j_tail,
420 WRITE_SYNC);
Jan Karaa78bb112012-03-13 15:43:04 -0400421 mutex_unlock(&journal->j_checkpoint_mutex);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700422 } else {
423 jbd_debug(3, "superblock not updated\n");
424 }
425
426 J_ASSERT(journal->j_running_transaction != NULL);
427 J_ASSERT(journal->j_committing_transaction == NULL);
428
429 commit_transaction = journal->j_running_transaction;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700430
Theodore Ts'o879c5e62009-06-17 11:47:48 -0400431 trace_jbd2_start_commit(journal, commit_transaction);
Eryu Guanf2a44522011-11-01 19:09:18 -0400432 jbd_debug(1, "JBD2: starting commit of transaction %d\n",
Dave Kleikamp470decc2006-10-11 01:20:57 -0700433 commit_transaction->t_tid);
434
Theodore Ts'oa931da62010-08-03 21:35:12 -0400435 write_lock(&journal->j_state_lock);
Paul Gortmaker3ca841c2013-06-12 22:46:35 -0400436 J_ASSERT(commit_transaction->t_state == T_RUNNING);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700437 commit_transaction->t_state = T_LOCKED;
438
Theodore Ts'o879c5e62009-06-17 11:47:48 -0400439 trace_jbd2_commit_locking(journal, commit_transaction);
Theodore Ts'obf699322009-09-30 00:32:06 -0400440 stats.run.rs_wait = commit_transaction->t_max_wait;
Theodore Ts'o9fff24a2013-02-06 22:30:23 -0500441 stats.run.rs_request_delay = 0;
Theodore Ts'obf699322009-09-30 00:32:06 -0400442 stats.run.rs_locked = jiffies;
Theodore Ts'o9fff24a2013-02-06 22:30:23 -0500443 if (commit_transaction->t_requested)
444 stats.run.rs_request_delay =
445 jbd2_time_diff(commit_transaction->t_requested,
446 stats.run.rs_locked);
Theodore Ts'obf699322009-09-30 00:32:06 -0400447 stats.run.rs_running = jbd2_time_diff(commit_transaction->t_start,
448 stats.run.rs_locked);
Johann Lombardi8e85fb32008-01-28 23:58:27 -0500449
Dave Kleikamp470decc2006-10-11 01:20:57 -0700450 spin_lock(&commit_transaction->t_handle_lock);
Theodore Ts'oa51dca92010-08-02 08:43:25 -0400451 while (atomic_read(&commit_transaction->t_updates)) {
Dave Kleikamp470decc2006-10-11 01:20:57 -0700452 DEFINE_WAIT(wait);
453
454 prepare_to_wait(&journal->j_wait_updates, &wait,
455 TASK_UNINTERRUPTIBLE);
Theodore Ts'oa51dca92010-08-02 08:43:25 -0400456 if (atomic_read(&commit_transaction->t_updates)) {
Dave Kleikamp470decc2006-10-11 01:20:57 -0700457 spin_unlock(&commit_transaction->t_handle_lock);
Theodore Ts'oa931da62010-08-03 21:35:12 -0400458 write_unlock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700459 schedule();
Theodore Ts'oa931da62010-08-03 21:35:12 -0400460 write_lock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700461 spin_lock(&commit_transaction->t_handle_lock);
462 }
463 finish_wait(&journal->j_wait_updates, &wait);
464 }
465 spin_unlock(&commit_transaction->t_handle_lock);
466
Theodore Ts'oa51dca92010-08-02 08:43:25 -0400467 J_ASSERT (atomic_read(&commit_transaction->t_outstanding_credits) <=
Dave Kleikamp470decc2006-10-11 01:20:57 -0700468 journal->j_max_transaction_buffers);
469
470 /*
471 * First thing we are allowed to do is to discard any remaining
472 * BJ_Reserved buffers. Note, it is _not_ permissible to assume
473 * that there are no such buffers: if a large filesystem
474 * operation like a truncate needs to split itself over multiple
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700475 * transactions, then it may try to do a jbd2_journal_restart() while
Dave Kleikamp470decc2006-10-11 01:20:57 -0700476 * there are still BJ_Reserved buffers outstanding. These must
477 * be released cleanly from the current transaction.
478 *
479 * In this case, the filesystem must still reserve write access
480 * again before modifying the buffer in the new transaction, but
481 * we do not require it to remember exactly which old buffers it
482 * has reserved. This is consistent with the existing behaviour
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700483 * that multiple jbd2_journal_get_write_access() calls to the same
Lucas De Marchi25985ed2011-03-30 22:57:33 -0300484 * buffer are perfectly permissible.
Dave Kleikamp470decc2006-10-11 01:20:57 -0700485 */
486 while (commit_transaction->t_reserved_list) {
487 jh = commit_transaction->t_reserved_list;
488 JBUFFER_TRACE(jh, "reserved, unused: refile");
489 /*
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700490 * A jbd2_journal_get_undo_access()+jbd2_journal_release_buffer() may
Dave Kleikamp470decc2006-10-11 01:20:57 -0700491 * leave undo-committed data.
492 */
493 if (jh->b_committed_data) {
494 struct buffer_head *bh = jh2bh(jh);
495
496 jbd_lock_bh_state(bh);
Mingming Caoaf1e76d2007-10-16 18:38:25 -0400497 jbd2_free(jh->b_committed_data, bh->b_size);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700498 jh->b_committed_data = NULL;
499 jbd_unlock_bh_state(bh);
500 }
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700501 jbd2_journal_refile_buffer(journal, jh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700502 }
503
504 /*
505 * Now try to drop any written-back buffers from the journal's
506 * checkpoint lists. We do this *before* commit because it potentially
507 * frees some memory
508 */
509 spin_lock(&journal->j_list_lock);
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700510 __jbd2_journal_clean_checkpoint_list(journal);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700511 spin_unlock(&journal->j_list_lock);
512
Eryu Guanf2a44522011-11-01 19:09:18 -0400513 jbd_debug(3, "JBD2: commit phase 1\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700514
515 /*
Yongqiang Yang1ba37262011-12-28 17:46:46 -0500516 * Clear revoked flag to reflect there is no revoked buffers
517 * in the next transaction which is going to be started.
518 */
519 jbd2_clear_buffer_revoked_flags(journal);
520
521 /*
Dave Kleikamp470decc2006-10-11 01:20:57 -0700522 * Switch to a new revoke table.
523 */
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700524 jbd2_journal_switch_revoke_table(journal);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700525
Jan Kara8f7d89f2013-06-04 12:35:11 -0400526 /*
527 * Reserved credits cannot be claimed anymore, free them
528 */
529 atomic_sub(atomic_read(&journal->j_reserved_credits),
530 &commit_transaction->t_outstanding_credits);
531
Theodore Ts'o879c5e62009-06-17 11:47:48 -0400532 trace_jbd2_commit_flushing(journal, commit_transaction);
Theodore Ts'obf699322009-09-30 00:32:06 -0400533 stats.run.rs_flushing = jiffies;
534 stats.run.rs_locked = jbd2_time_diff(stats.run.rs_locked,
535 stats.run.rs_flushing);
Johann Lombardi8e85fb32008-01-28 23:58:27 -0500536
Dave Kleikamp470decc2006-10-11 01:20:57 -0700537 commit_transaction->t_state = T_FLUSH;
538 journal->j_committing_transaction = commit_transaction;
539 journal->j_running_transaction = NULL;
Josef Bacike07f7182008-11-26 01:14:26 -0500540 start_time = ktime_get();
Dave Kleikamp470decc2006-10-11 01:20:57 -0700541 commit_transaction->t_log_start = journal->j_head;
542 wake_up(&journal->j_wait_transaction_locked);
Theodore Ts'oa931da62010-08-03 21:35:12 -0400543 write_unlock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700544
Paul Gortmakercfc7bc82013-06-12 22:56:35 -0400545 jbd_debug(3, "JBD2: commit phase 2a\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700546
547 /*
Dave Kleikamp470decc2006-10-11 01:20:57 -0700548 * Now start flushing things to disk, in the order they appear
549 * on the transaction lists. Data blocks go first.
550 */
Aneesh Kumar K.Vcd1aac32008-07-11 19:27:31 -0400551 err = journal_submit_data_buffers(journal, commit_transaction);
Jan Karac851ed52008-07-11 19:27:31 -0400552 if (err)
553 jbd2_journal_abort(journal, err);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700554
Jens Axboe82f04ab2011-03-17 11:01:52 +0100555 blk_start_plug(&plug);
Theodore Ts'o67c457a2009-04-14 07:50:56 -0400556 jbd2_journal_write_revoke_records(journal, commit_transaction,
Jan Karae5a120a2013-06-04 12:06:01 -0400557 &log_bufs, WRITE_SYNC);
Jens Axboe82f04ab2011-03-17 11:01:52 +0100558 blk_finish_plug(&plug);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700559
Paul Gortmakercfc7bc82013-06-12 22:56:35 -0400560 jbd_debug(3, "JBD2: commit phase 2b\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700561
562 /*
Dave Kleikamp470decc2006-10-11 01:20:57 -0700563 * Way to go: we have now written out all of the data for a
564 * transaction! Now comes the tricky part: we need to write out
565 * metadata. Loop over the transaction's entire buffer list:
566 */
Theodore Ts'oa931da62010-08-03 21:35:12 -0400567 write_lock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700568 commit_transaction->t_state = T_COMMIT;
Theodore Ts'oa931da62010-08-03 21:35:12 -0400569 write_unlock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700570
Theodore Ts'o879c5e62009-06-17 11:47:48 -0400571 trace_jbd2_commit_logging(journal, commit_transaction);
Theodore Ts'obf699322009-09-30 00:32:06 -0400572 stats.run.rs_logging = jiffies;
573 stats.run.rs_flushing = jbd2_time_diff(stats.run.rs_flushing,
574 stats.run.rs_logging);
Theodore Ts'oa51dca92010-08-02 08:43:25 -0400575 stats.run.rs_blocks =
576 atomic_read(&commit_transaction->t_outstanding_credits);
Theodore Ts'obf699322009-09-30 00:32:06 -0400577 stats.run.rs_blocks_logged = 0;
Johann Lombardi8e85fb32008-01-28 23:58:27 -0500578
Josef Bacik1dfc3222008-04-17 10:38:59 -0400579 J_ASSERT(commit_transaction->t_nr_buffers <=
Theodore Ts'oa51dca92010-08-02 08:43:25 -0400580 atomic_read(&commit_transaction->t_outstanding_credits));
Josef Bacik1dfc3222008-04-17 10:38:59 -0400581
Jan Kara87c89c22008-07-11 19:27:31 -0400582 err = 0;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700583 bufs = 0;
Jan Karae5a120a2013-06-04 12:06:01 -0400584 descriptor = NULL;
Jens Axboe82f04ab2011-03-17 11:01:52 +0100585 blk_start_plug(&plug);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700586 while (commit_transaction->t_buffers) {
587
588 /* Find the next buffer to be journaled... */
589
590 jh = commit_transaction->t_buffers;
591
592 /* If we're in abort mode, we just un-journal the buffer and
Hidehiro Kawai7ad74452008-10-10 20:29:31 -0400593 release it. */
Dave Kleikamp470decc2006-10-11 01:20:57 -0700594
595 if (is_journal_aborted(journal)) {
Hidehiro Kawai7ad74452008-10-10 20:29:31 -0400596 clear_buffer_jbddirty(jh2bh(jh));
Dave Kleikamp470decc2006-10-11 01:20:57 -0700597 JBUFFER_TRACE(jh, "journal is aborting: refile");
Joel Beckere06c8222008-09-11 15:35:47 -0700598 jbd2_buffer_abort_trigger(jh,
599 jh->b_frozen_data ?
600 jh->b_frozen_triggers :
601 jh->b_triggers);
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700602 jbd2_journal_refile_buffer(journal, jh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700603 /* If that was the last one, we need to clean up
604 * any descriptor buffers which may have been
605 * already allocated, even if we are now
606 * aborting. */
607 if (!commit_transaction->t_buffers)
608 goto start_journal_io;
609 continue;
610 }
611
612 /* Make sure we have a descriptor block in which to
613 record the metadata buffer. */
614
615 if (!descriptor) {
Dave Kleikamp470decc2006-10-11 01:20:57 -0700616 J_ASSERT (bufs == 0);
617
Eryu Guanf2a44522011-11-01 19:09:18 -0400618 jbd_debug(4, "JBD2: get descriptor\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700619
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700620 descriptor = jbd2_journal_get_descriptor_buffer(journal);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700621 if (!descriptor) {
Jan Karaa7fa2ba2007-10-16 18:38:25 -0400622 jbd2_journal_abort(journal, -EIO);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700623 continue;
624 }
625
Eryu Guanf2a44522011-11-01 19:09:18 -0400626 jbd_debug(4, "JBD2: got buffer %llu (%p)\n",
Jan Karae5a120a2013-06-04 12:06:01 -0400627 (unsigned long long)descriptor->b_blocknr,
628 descriptor->b_data);
629 header = (journal_header_t *)descriptor->b_data;
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700630 header->h_magic = cpu_to_be32(JBD2_MAGIC_NUMBER);
631 header->h_blocktype = cpu_to_be32(JBD2_DESCRIPTOR_BLOCK);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700632 header->h_sequence = cpu_to_be32(commit_transaction->t_tid);
633
Jan Karae5a120a2013-06-04 12:06:01 -0400634 tagp = &descriptor->b_data[sizeof(journal_header_t)];
635 space_left = descriptor->b_size -
636 sizeof(journal_header_t);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700637 first_tag = 1;
Jan Karae5a120a2013-06-04 12:06:01 -0400638 set_buffer_jwrite(descriptor);
639 set_buffer_dirty(descriptor);
640 wbuf[bufs++] = descriptor;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700641
642 /* Record it so that we can wait for IO
643 completion later */
Jan Karae5a120a2013-06-04 12:06:01 -0400644 BUFFER_TRACE(descriptor, "ph3: file as descriptor");
645 jbd2_file_log_bh(&log_bufs, descriptor);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700646 }
647
648 /* Where is the buffer to be written? */
649
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700650 err = jbd2_journal_next_log_block(journal, &blocknr);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700651 /* If the block mapping failed, just abandon the buffer
652 and repeat this loop: we'll fall into the
653 refile-on-abort condition above. */
654 if (err) {
Jan Karaa7fa2ba2007-10-16 18:38:25 -0400655 jbd2_journal_abort(journal, err);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700656 continue;
657 }
658
659 /*
660 * start_this_handle() uses t_outstanding_credits to determine
661 * the free space in the log, but this counter is changed
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700662 * by jbd2_journal_next_log_block() also.
Dave Kleikamp470decc2006-10-11 01:20:57 -0700663 */
Theodore Ts'oa51dca92010-08-02 08:43:25 -0400664 atomic_dec(&commit_transaction->t_outstanding_credits);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700665
666 /* Bump b_count to prevent truncate from stumbling over
667 the shadowed buffer! @@@ This can go if we ever get
Jan Karaf5113ef2013-06-04 12:01:45 -0400668 rid of the shadow pairing of buffers. */
Dave Kleikamp470decc2006-10-11 01:20:57 -0700669 atomic_inc(&jh2bh(jh)->b_count);
670
Dave Kleikamp470decc2006-10-11 01:20:57 -0700671 /*
Jan Karaf5113ef2013-06-04 12:01:45 -0400672 * Make a temporary IO buffer with which to write it out
673 * (this will requeue the metadata buffer to BJ_Shadow).
Dave Kleikamp470decc2006-10-11 01:20:57 -0700674 */
Jan Karaf5113ef2013-06-04 12:01:45 -0400675 set_bit(BH_JWrite, &jh2bh(jh)->b_state);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700676 JBUFFER_TRACE(jh, "ph3: write metadata");
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700677 flags = jbd2_journal_write_metadata_buffer(commit_transaction,
Jan Karaf5113ef2013-06-04 12:01:45 -0400678 jh, &wbuf[bufs], blocknr);
Theodore Ts'oe6ec1162009-12-01 09:04:42 -0500679 if (flags < 0) {
680 jbd2_journal_abort(journal, flags);
681 continue;
682 }
Jan Karaf5113ef2013-06-04 12:01:45 -0400683 jbd2_file_log_bh(&io_bufs, wbuf[bufs]);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700684
685 /* Record the new block's tag in the current descriptor
686 buffer */
687
688 tag_flag = 0;
689 if (flags & 1)
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700690 tag_flag |= JBD2_FLAG_ESCAPE;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700691 if (!first_tag)
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700692 tag_flag |= JBD2_FLAG_SAME_UUID;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700693
694 tag = (journal_block_tag_t *) tagp;
Zach Brownb517bea2006-10-11 01:21:08 -0700695 write_tag_block(tag_bytes, tag, jh2bh(jh)->b_blocknr);
Darrick J. Wong8f888ef2012-05-22 22:43:41 -0400696 tag->t_flags = cpu_to_be16(tag_flag);
Jan Karaf5113ef2013-06-04 12:01:45 -0400697 jbd2_block_tag_csum_set(journal, tag, wbuf[bufs],
Darrick J. Wongc3900872012-05-27 08:12:12 -0400698 commit_transaction->t_tid);
Zach Brownb517bea2006-10-11 01:21:08 -0700699 tagp += tag_bytes;
700 space_left -= tag_bytes;
Jan Karaf5113ef2013-06-04 12:01:45 -0400701 bufs++;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700702
703 if (first_tag) {
704 memcpy (tagp, journal->j_uuid, 16);
705 tagp += 16;
706 space_left -= 16;
707 first_tag = 0;
708 }
709
710 /* If there's no more to do, or if the descriptor is full,
711 let the IO rip! */
712
713 if (bufs == journal->j_wbufsize ||
714 commit_transaction->t_buffers == NULL ||
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400715 space_left < tag_bytes + 16 + csum_size) {
Dave Kleikamp470decc2006-10-11 01:20:57 -0700716
Eryu Guanf2a44522011-11-01 19:09:18 -0400717 jbd_debug(4, "JBD2: Submit %d IOs\n", bufs);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700718
719 /* Write an end-of-descriptor marker before
720 submitting the IOs. "tag" still points to
721 the last tag we set up. */
722
Darrick J. Wong8f888ef2012-05-22 22:43:41 -0400723 tag->t_flags |= cpu_to_be16(JBD2_FLAG_LAST_TAG);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700724
Darrick J. Wong3caa4872012-05-27 08:10:22 -0400725 jbd2_descr_block_csum_set(journal, descriptor);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700726start_journal_io:
727 for (i = 0; i < bufs; i++) {
728 struct buffer_head *bh = wbuf[i];
Girish Shilamkar818d2762008-01-28 23:58:27 -0500729 /*
730 * Compute checksum.
731 */
732 if (JBD2_HAS_COMPAT_FEATURE(journal,
733 JBD2_FEATURE_COMPAT_CHECKSUM)) {
734 crc32_sum =
735 jbd2_checksum_data(crc32_sum, bh);
736 }
737
Dave Kleikamp470decc2006-10-11 01:20:57 -0700738 lock_buffer(bh);
739 clear_buffer_dirty(bh);
740 set_buffer_uptodate(bh);
741 bh->b_end_io = journal_end_buffer_io_sync;
Jens Axboe82f04ab2011-03-17 11:01:52 +0100742 submit_bh(WRITE_SYNC, bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700743 }
744 cond_resched();
Theodore Ts'obf699322009-09-30 00:32:06 -0400745 stats.run.rs_blocks_logged += bufs;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700746
747 /* Force a new descriptor to be generated next
748 time round the loop. */
749 descriptor = NULL;
750 bufs = 0;
751 }
752 }
753
Jan Karac851ed52008-07-11 19:27:31 -0400754 err = journal_finish_inode_data_buffers(journal, commit_transaction);
Hidehiro Kawaie9e34f42008-07-31 22:26:04 -0400755 if (err) {
Hidehiro Kawaie9e34f42008-07-31 22:26:04 -0400756 printk(KERN_WARNING
757 "JBD2: Detected IO errors while flushing file data "
Theodore Ts'o05496762008-09-16 14:36:17 -0400758 "on %s\n", journal->j_devname);
Hidehiro Kawai5bf56832008-10-10 22:12:43 -0400759 if (journal->j_flags & JBD2_ABORT_ON_SYNCDATA_ERR)
760 jbd2_journal_abort(journal, err);
Hidehiro Kawaie9e34f42008-07-31 22:26:04 -0400761 err = 0;
762 }
Jan Karac851ed52008-07-11 19:27:31 -0400763
Jan Kara33395782012-03-13 22:45:38 -0400764 /*
765 * Get current oldest transaction in the log before we issue flush
766 * to the filesystem device. After the flush we can be sure that
767 * blocks of all older transactions are checkpointed to persistent
768 * storage and we will be safe to update journal start in the
769 * superblock with the numbers we get here.
770 */
771 update_tail =
772 jbd2_journal_get_log_tail(journal, &first_tid, &first_block);
773
Jan Karabbd2be32011-05-24 11:59:18 -0400774 write_lock(&journal->j_state_lock);
Jan Kara33395782012-03-13 22:45:38 -0400775 if (update_tail) {
776 long freed = first_block - journal->j_tail;
777
778 if (first_block < journal->j_tail)
779 freed += journal->j_last - journal->j_first;
780 /* Update tail only if we free significant amount of space */
781 if (freed < journal->j_maxlen / 4)
782 update_tail = 0;
783 }
Jan Karabbd2be32011-05-24 11:59:18 -0400784 J_ASSERT(commit_transaction->t_state == T_COMMIT);
785 commit_transaction->t_state = T_COMMIT_DFLUSH;
786 write_unlock(&journal->j_state_lock);
Jan Kara33395782012-03-13 22:45:38 -0400787
Girish Shilamkar818d2762008-01-28 23:58:27 -0500788 /*
789 * If the journal is not located on the file system device,
790 * then we must flush the file system device before we issue
791 * the commit record
792 */
Jan Kara81be12c2011-05-24 11:52:40 -0400793 if (commit_transaction->t_need_data_flush &&
Girish Shilamkar818d2762008-01-28 23:58:27 -0500794 (journal->j_fs_dev != journal->j_dev) &&
795 (journal->j_flags & JBD2_BARRIER))
Shaohua Li99aa7842012-04-13 10:27:35 +0800796 blkdev_issue_flush(journal->j_fs_dev, GFP_NOFS, NULL);
Girish Shilamkar818d2762008-01-28 23:58:27 -0500797
798 /* Done it all: now write the commit record asynchronously. */
799 if (JBD2_HAS_INCOMPAT_FEATURE(journal,
800 JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) {
801 err = journal_submit_commit_record(journal, commit_transaction,
Dave Kleikamp470decc2006-10-11 01:20:57 -0700802 &cbh, crc32_sum);
803 if (err)
804 __jbd2_journal_abort_hard(journal);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700805 }
806
Jens Axboe82f04ab2011-03-17 11:01:52 +0100807 blk_finish_plug(&plug);
808
Dave Kleikamp470decc2006-10-11 01:20:57 -0700809 /* Lo and behold: we have just managed to send a transaction to
810 the log. Before we can commit it, wait for the IO so far to
811 complete. Control buffers being written are on the
812 transaction's t_log_list queue, and metadata buffers are on
Jan Karaf5113ef2013-06-04 12:01:45 -0400813 the io_bufs list.
Dave Kleikamp470decc2006-10-11 01:20:57 -0700814
815 Wait for the buffers in reverse order. That way we are
816 less likely to be woken up until all IOs have completed, and
817 so we incur less scheduling load.
818 */
819
Eryu Guanf2a44522011-11-01 19:09:18 -0400820 jbd_debug(3, "JBD2: commit phase 3\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700821
Jan Karaf5113ef2013-06-04 12:01:45 -0400822 while (!list_empty(&io_bufs)) {
823 struct buffer_head *bh = list_entry(io_bufs.prev,
824 struct buffer_head,
825 b_assoc_buffers);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700826
Jan Karaf5113ef2013-06-04 12:01:45 -0400827 wait_on_buffer(bh);
828 cond_resched();
Dave Kleikamp470decc2006-10-11 01:20:57 -0700829
830 if (unlikely(!buffer_uptodate(bh)))
831 err = -EIO;
Jan Karaf5113ef2013-06-04 12:01:45 -0400832 jbd2_unfile_log_bh(bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700833
834 /*
Jan Karaf5113ef2013-06-04 12:01:45 -0400835 * The list contains temporary buffer heads created by
836 * jbd2_journal_write_metadata_buffer().
Dave Kleikamp470decc2006-10-11 01:20:57 -0700837 */
838 BUFFER_TRACE(bh, "dumping temporary bh");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700839 __brelse(bh);
840 J_ASSERT_BH(bh, atomic_read(&bh->b_count) == 0);
841 free_buffer_head(bh);
842
Jan Karaf5113ef2013-06-04 12:01:45 -0400843 /* We also have to refile the corresponding shadowed buffer */
Dave Kleikamp470decc2006-10-11 01:20:57 -0700844 jh = commit_transaction->t_shadow_list->b_tprev;
845 bh = jh2bh(jh);
Jan Karaf5113ef2013-06-04 12:01:45 -0400846 clear_buffer_jwrite(bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700847 J_ASSERT_BH(bh, buffer_jbddirty(bh));
Jan Karab34090e2013-06-04 12:08:56 -0400848 J_ASSERT_BH(bh, !buffer_shadow(bh));
Dave Kleikamp470decc2006-10-11 01:20:57 -0700849
850 /* The metadata is now released for reuse, but we need
851 to remember it against this transaction so that when
852 we finally commit, we can do any checkpointing
853 required. */
854 JBUFFER_TRACE(jh, "file as BJ_Forget");
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700855 jbd2_journal_file_buffer(jh, commit_transaction, BJ_Forget);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700856 JBUFFER_TRACE(jh, "brelse shadowed buffer");
857 __brelse(bh);
858 }
859
860 J_ASSERT (commit_transaction->t_shadow_list == NULL);
861
Eryu Guanf2a44522011-11-01 19:09:18 -0400862 jbd_debug(3, "JBD2: commit phase 4\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700863
864 /* Here we wait for the revoke record and descriptor record buffers */
Jan Karae5a120a2013-06-04 12:06:01 -0400865 while (!list_empty(&log_bufs)) {
Dave Kleikamp470decc2006-10-11 01:20:57 -0700866 struct buffer_head *bh;
867
Jan Karae5a120a2013-06-04 12:06:01 -0400868 bh = list_entry(log_bufs.prev, struct buffer_head, b_assoc_buffers);
869 wait_on_buffer(bh);
870 cond_resched();
Dave Kleikamp470decc2006-10-11 01:20:57 -0700871
872 if (unlikely(!buffer_uptodate(bh)))
873 err = -EIO;
874
875 BUFFER_TRACE(bh, "ph5: control buffer writeout done: unfile");
876 clear_buffer_jwrite(bh);
Jan Karae5a120a2013-06-04 12:06:01 -0400877 jbd2_unfile_log_bh(bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700878 __brelse(bh); /* One for getblk */
879 /* AKPM: bforget here */
880 }
881
Hidehiro Kawai77e841d2008-10-12 16:39:16 -0400882 if (err)
883 jbd2_journal_abort(journal, err);
884
Eryu Guanf2a44522011-11-01 19:09:18 -0400885 jbd_debug(3, "JBD2: commit phase 5\n");
Jan Karabbd2be32011-05-24 11:59:18 -0400886 write_lock(&journal->j_state_lock);
887 J_ASSERT(commit_transaction->t_state == T_COMMIT_DFLUSH);
888 commit_transaction->t_state = T_COMMIT_JFLUSH;
889 write_unlock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700890
Girish Shilamkar818d2762008-01-28 23:58:27 -0500891 if (!JBD2_HAS_INCOMPAT_FEATURE(journal,
Theodore Ts'o0e3d2a62009-09-11 09:30:12 -0400892 JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) {
Girish Shilamkar818d2762008-01-28 23:58:27 -0500893 err = journal_submit_commit_record(journal, commit_transaction,
894 &cbh, crc32_sum);
895 if (err)
896 __jbd2_journal_abort_hard(journal);
897 }
Zhang Huan6cba6112011-04-05 19:16:20 -0400898 if (cbh)
Theodore Ts'ofd984962009-01-05 21:34:13 -0500899 err = journal_wait_on_commit_record(journal, cbh);
Jan Karaf73bee42010-08-18 15:56:56 +0200900 if (JBD2_HAS_INCOMPAT_FEATURE(journal,
901 JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT) &&
902 journal->j_flags & JBD2_BARRIER) {
Shaohua Li99aa7842012-04-13 10:27:35 +0800903 blkdev_issue_flush(journal->j_dev, GFP_NOFS, NULL);
Jan Karaf73bee42010-08-18 15:56:56 +0200904 }
Dave Kleikamp470decc2006-10-11 01:20:57 -0700905
906 if (err)
Jan Karaa7fa2ba2007-10-16 18:38:25 -0400907 jbd2_journal_abort(journal, err);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700908
Jan Kara33395782012-03-13 22:45:38 -0400909 /*
910 * Now disk caches for filesystem device are flushed so we are safe to
911 * erase checkpointed transactions from the log by updating journal
912 * superblock.
913 */
914 if (update_tail)
915 jbd2_update_log_tail(journal, first_tid, first_block);
916
Dave Kleikamp470decc2006-10-11 01:20:57 -0700917 /* End of a transaction! Finally, we can do checkpoint
918 processing: any buffers committed as a result of this
919 transaction can be removed from any checkpoint list it was on
920 before. */
921
Eryu Guanf2a44522011-11-01 19:09:18 -0400922 jbd_debug(3, "JBD2: commit phase 6\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -0700923
Jan Karac851ed52008-07-11 19:27:31 -0400924 J_ASSERT(list_empty(&commit_transaction->t_inode_list));
Dave Kleikamp470decc2006-10-11 01:20:57 -0700925 J_ASSERT(commit_transaction->t_buffers == NULL);
926 J_ASSERT(commit_transaction->t_checkpoint_list == NULL);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700927 J_ASSERT(commit_transaction->t_shadow_list == NULL);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700928
929restart_loop:
930 /*
931 * As there are other places (journal_unmap_buffer()) adding buffers
932 * to this list we have to be careful and hold the j_list_lock.
933 */
934 spin_lock(&journal->j_list_lock);
935 while (commit_transaction->t_forget) {
936 transaction_t *cp_transaction;
937 struct buffer_head *bh;
Jan Karade1b7942011-06-13 15:38:22 -0400938 int try_to_free = 0;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700939
940 jh = commit_transaction->t_forget;
941 spin_unlock(&journal->j_list_lock);
942 bh = jh2bh(jh);
Jan Karade1b7942011-06-13 15:38:22 -0400943 /*
944 * Get a reference so that bh cannot be freed before we are
945 * done with it.
946 */
947 get_bh(bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700948 jbd_lock_bh_state(bh);
dingdinghua23e2af32010-02-24 12:11:20 -0500949 J_ASSERT_JH(jh, jh->b_transaction == commit_transaction);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700950
951 /*
952 * If there is undo-protected committed data against
953 * this buffer, then we can remove it now. If it is a
954 * buffer needing such protection, the old frozen_data
955 * field now points to a committed version of the
956 * buffer, so rotate that field to the new committed
957 * data.
958 *
959 * Otherwise, we can just throw away the frozen data now.
Joel Beckere06c8222008-09-11 15:35:47 -0700960 *
961 * We also know that the frozen data has already fired
962 * its triggers if they exist, so we can clear that too.
Dave Kleikamp470decc2006-10-11 01:20:57 -0700963 */
964 if (jh->b_committed_data) {
Mingming Caoaf1e76d2007-10-16 18:38:25 -0400965 jbd2_free(jh->b_committed_data, bh->b_size);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700966 jh->b_committed_data = NULL;
967 if (jh->b_frozen_data) {
968 jh->b_committed_data = jh->b_frozen_data;
969 jh->b_frozen_data = NULL;
Joel Beckere06c8222008-09-11 15:35:47 -0700970 jh->b_frozen_triggers = NULL;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700971 }
972 } else if (jh->b_frozen_data) {
Mingming Caoaf1e76d2007-10-16 18:38:25 -0400973 jbd2_free(jh->b_frozen_data, bh->b_size);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700974 jh->b_frozen_data = NULL;
Joel Beckere06c8222008-09-11 15:35:47 -0700975 jh->b_frozen_triggers = NULL;
Dave Kleikamp470decc2006-10-11 01:20:57 -0700976 }
977
978 spin_lock(&journal->j_list_lock);
979 cp_transaction = jh->b_cp_transaction;
980 if (cp_transaction) {
981 JBUFFER_TRACE(jh, "remove from old cp transaction");
Johann Lombardi8e85fb32008-01-28 23:58:27 -0500982 cp_transaction->t_chp_stats.cs_dropped++;
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700983 __jbd2_journal_remove_checkpoint(jh);
Dave Kleikamp470decc2006-10-11 01:20:57 -0700984 }
985
986 /* Only re-checkpoint the buffer_head if it is marked
987 * dirty. If the buffer was added to the BJ_Forget list
Mingming Caof7f4bcc2006-10-11 01:20:59 -0700988 * by jbd2_journal_forget, it may no longer be dirty and
Dave Kleikamp470decc2006-10-11 01:20:57 -0700989 * there's no point in keeping a checkpoint record for
990 * it. */
991
Jan Karab794e7a2012-09-26 23:11:13 -0400992 /*
993 * A buffer which has been freed while still being journaled by
994 * a previous transaction.
995 */
996 if (buffer_freed(bh)) {
997 /*
998 * If the running transaction is the one containing
999 * "add to orphan" operation (b_next_transaction !=
1000 * NULL), we have to wait for that transaction to
1001 * commit before we can really get rid of the buffer.
1002 * So just clear b_modified to not confuse transaction
1003 * credit accounting and refile the buffer to
1004 * BJ_Forget of the running transaction. If the just
1005 * committed transaction contains "add to orphan"
1006 * operation, we can completely invalidate the buffer
1007 * now. We are rather through in that since the
1008 * buffer may be still accessible when blocksize <
1009 * pagesize and it is attached to the last partial
1010 * page.
1011 */
1012 jh->b_modified = 0;
1013 if (!jh->b_next_transaction) {
1014 clear_buffer_freed(bh);
1015 clear_buffer_jbddirty(bh);
1016 clear_buffer_mapped(bh);
1017 clear_buffer_new(bh);
1018 clear_buffer_req(bh);
1019 bh->b_bdev = NULL;
1020 }
Dave Kleikamp470decc2006-10-11 01:20:57 -07001021 }
1022
1023 if (buffer_jbddirty(bh)) {
1024 JBUFFER_TRACE(jh, "add to new checkpointing trans");
Mingming Caof7f4bcc2006-10-11 01:20:59 -07001025 __jbd2_journal_insert_checkpoint(jh, commit_transaction);
Hidehiro Kawai7ad74452008-10-10 20:29:31 -04001026 if (is_journal_aborted(journal))
1027 clear_buffer_jbddirty(bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001028 } else {
1029 J_ASSERT_BH(bh, !buffer_dirty(bh));
Jan Karade1b7942011-06-13 15:38:22 -04001030 /*
1031 * The buffer on BJ_Forget list and not jbddirty means
Dave Kleikamp470decc2006-10-11 01:20:57 -07001032 * it has been freed by this transaction and hence it
1033 * could not have been reallocated until this
1034 * transaction has committed. *BUT* it could be
1035 * reallocated once we have written all the data to
1036 * disk and before we process the buffer on BJ_Forget
Jan Karade1b7942011-06-13 15:38:22 -04001037 * list.
1038 */
1039 if (!jh->b_next_transaction)
1040 try_to_free = 1;
Dave Kleikamp470decc2006-10-11 01:20:57 -07001041 }
Jan Karade1b7942011-06-13 15:38:22 -04001042 JBUFFER_TRACE(jh, "refile or unfile buffer");
1043 __jbd2_journal_refile_buffer(jh);
1044 jbd_unlock_bh_state(bh);
1045 if (try_to_free)
1046 release_buffer_page(bh); /* Drops bh reference */
1047 else
1048 __brelse(bh);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001049 cond_resched_lock(&journal->j_list_lock);
1050 }
1051 spin_unlock(&journal->j_list_lock);
1052 /*
Jan Karaf5a7a6b2008-01-28 23:58:27 -05001053 * This is a bit sleazy. We use j_list_lock to protect transition
1054 * of a transaction into T_FINISHED state and calling
1055 * __jbd2_journal_drop_transaction(). Otherwise we could race with
1056 * other checkpointing code processing the transaction...
Dave Kleikamp470decc2006-10-11 01:20:57 -07001057 */
Theodore Ts'oa931da62010-08-03 21:35:12 -04001058 write_lock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001059 spin_lock(&journal->j_list_lock);
1060 /*
1061 * Now recheck if some buffers did not get attached to the transaction
1062 * while the lock was dropped...
1063 */
1064 if (commit_transaction->t_forget) {
1065 spin_unlock(&journal->j_list_lock);
Theodore Ts'oa931da62010-08-03 21:35:12 -04001066 write_unlock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001067 goto restart_loop;
1068 }
1069
1070 /* Done with this transaction! */
1071
Eryu Guanf2a44522011-11-01 19:09:18 -04001072 jbd_debug(3, "JBD2: commit phase 7\n");
Dave Kleikamp470decc2006-10-11 01:20:57 -07001073
Jan Karabbd2be32011-05-24 11:59:18 -04001074 J_ASSERT(commit_transaction->t_state == T_COMMIT_JFLUSH);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001075
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001076 commit_transaction->t_start = jiffies;
Theodore Ts'obf699322009-09-30 00:32:06 -04001077 stats.run.rs_logging = jbd2_time_diff(stats.run.rs_logging,
1078 commit_transaction->t_start);
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001079
1080 /*
Theodore Ts'obf699322009-09-30 00:32:06 -04001081 * File the transaction statistics
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001082 */
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001083 stats.ts_tid = commit_transaction->t_tid;
Theodore Ts'o8dd42042010-08-03 21:38:29 -04001084 stats.run.rs_handle_count =
1085 atomic_read(&commit_transaction->t_handle_count);
Theodore Ts'obf699322009-09-30 00:32:06 -04001086 trace_jbd2_run_stats(journal->j_fs_dev->bd_dev,
1087 commit_transaction->t_tid, &stats.run);
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001088
1089 /*
1090 * Calculate overall stats
1091 */
Theodore Ts'obf699322009-09-30 00:32:06 -04001092 spin_lock(&journal->j_history_lock);
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001093 journal->j_stats.ts_tid++;
Theodore Ts'o9fff24a2013-02-06 22:30:23 -05001094 if (commit_transaction->t_requested)
1095 journal->j_stats.ts_requested++;
Theodore Ts'obf699322009-09-30 00:32:06 -04001096 journal->j_stats.run.rs_wait += stats.run.rs_wait;
Theodore Ts'o9fff24a2013-02-06 22:30:23 -05001097 journal->j_stats.run.rs_request_delay += stats.run.rs_request_delay;
Theodore Ts'obf699322009-09-30 00:32:06 -04001098 journal->j_stats.run.rs_running += stats.run.rs_running;
1099 journal->j_stats.run.rs_locked += stats.run.rs_locked;
1100 journal->j_stats.run.rs_flushing += stats.run.rs_flushing;
1101 journal->j_stats.run.rs_logging += stats.run.rs_logging;
1102 journal->j_stats.run.rs_handle_count += stats.run.rs_handle_count;
1103 journal->j_stats.run.rs_blocks += stats.run.rs_blocks;
1104 journal->j_stats.run.rs_blocks_logged += stats.run.rs_blocks_logged;
Johann Lombardi8e85fb32008-01-28 23:58:27 -05001105 spin_unlock(&journal->j_history_lock);
1106
Dmitry Monakhov794446c2013-04-03 22:06:52 -04001107 commit_transaction->t_state = T_COMMIT_CALLBACK;
Dave Kleikamp470decc2006-10-11 01:20:57 -07001108 J_ASSERT(commit_transaction == journal->j_committing_transaction);
1109 journal->j_commit_sequence = commit_transaction->t_tid;
1110 journal->j_committing_transaction = NULL;
Josef Bacike07f7182008-11-26 01:14:26 -05001111 commit_time = ktime_to_ns(ktime_sub(ktime_get(), start_time));
Dave Kleikamp470decc2006-10-11 01:20:57 -07001112
Josef Bacike07f7182008-11-26 01:14:26 -05001113 /*
1114 * weight the commit time higher than the average time so we don't
1115 * react too strongly to vast changes in the commit time
1116 */
1117 if (likely(journal->j_average_commit_time))
1118 journal->j_average_commit_time = (commit_time +
1119 journal->j_average_commit_time*3) / 4;
1120 else
1121 journal->j_average_commit_time = commit_time;
Dmitry Monakhov794446c2013-04-03 22:06:52 -04001122
Theodore Ts'oa931da62010-08-03 21:35:12 -04001123 write_unlock(&journal->j_state_lock);
Theodore Ts'o6c20ec82008-10-28 21:08:20 -04001124
Dmitry Monakhov794446c2013-04-03 22:06:52 -04001125 if (journal->j_checkpoint_transactions == NULL) {
1126 journal->j_checkpoint_transactions = commit_transaction;
1127 commit_transaction->t_cpnext = commit_transaction;
1128 commit_transaction->t_cpprev = commit_transaction;
Dave Kleikamp470decc2006-10-11 01:20:57 -07001129 } else {
Dmitry Monakhov794446c2013-04-03 22:06:52 -04001130 commit_transaction->t_cpnext =
1131 journal->j_checkpoint_transactions;
1132 commit_transaction->t_cpprev =
1133 commit_transaction->t_cpnext->t_cpprev;
1134 commit_transaction->t_cpnext->t_cpprev =
1135 commit_transaction;
1136 commit_transaction->t_cpprev->t_cpnext =
Dave Kleikamp470decc2006-10-11 01:20:57 -07001137 commit_transaction;
Dave Kleikamp470decc2006-10-11 01:20:57 -07001138 }
1139 spin_unlock(&journal->j_list_lock);
Dmitry Monakhov794446c2013-04-03 22:06:52 -04001140 /* Drop all spin_locks because commit_callback may be block.
1141 * __journal_remove_checkpoint() can not destroy transaction
1142 * under us because it is not marked as T_FINISHED yet */
Aneesh Kumar K.Vfb684072008-11-06 17:50:21 -05001143 if (journal->j_commit_callback)
1144 journal->j_commit_callback(journal, commit_transaction);
1145
Theodore Ts'o879c5e62009-06-17 11:47:48 -04001146 trace_jbd2_end_commit(journal, commit_transaction);
Eryu Guanf2a44522011-11-01 19:09:18 -04001147 jbd_debug(1, "JBD2: commit %d complete, head %d\n",
Dave Kleikamp470decc2006-10-11 01:20:57 -07001148 journal->j_commit_sequence, journal->j_tail_sequence);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001149
Dmitry Monakhov794446c2013-04-03 22:06:52 -04001150 write_lock(&journal->j_state_lock);
1151 spin_lock(&journal->j_list_lock);
1152 commit_transaction->t_state = T_FINISHED;
1153 /* Recheck checkpoint lists after j_list_lock was dropped */
1154 if (commit_transaction->t_checkpoint_list == NULL &&
1155 commit_transaction->t_checkpoint_io_list == NULL) {
1156 __jbd2_journal_drop_transaction(journal, commit_transaction);
1157 jbd2_journal_free_transaction(commit_transaction);
1158 }
1159 spin_unlock(&journal->j_list_lock);
1160 write_unlock(&journal->j_state_lock);
Dave Kleikamp470decc2006-10-11 01:20:57 -07001161 wake_up(&journal->j_wait_done_commit);
1162}