xref: /linux-6.15/fs/jbd2/commit.c (revision 67c457a8)
1470decc6SDave Kleikamp /*
2f7f4bccbSMingming Cao  * linux/fs/jbd2/commit.c
3470decc6SDave Kleikamp  *
4470decc6SDave Kleikamp  * Written by Stephen C. Tweedie <[email protected]>, 1998
5470decc6SDave Kleikamp  *
6470decc6SDave Kleikamp  * Copyright 1998 Red Hat corp --- All Rights Reserved
7470decc6SDave Kleikamp  *
8470decc6SDave Kleikamp  * This file is part of the Linux kernel and is made available under
9470decc6SDave Kleikamp  * the terms of the GNU General Public License, version 2, or at your
10470decc6SDave Kleikamp  * option, any later version, incorporated herein by reference.
11470decc6SDave Kleikamp  *
12470decc6SDave Kleikamp  * Journal commit routines for the generic filesystem journaling code;
13470decc6SDave Kleikamp  * part of the ext2fs journaling system.
14470decc6SDave Kleikamp  */
15470decc6SDave Kleikamp 
16470decc6SDave Kleikamp #include <linux/time.h>
17470decc6SDave Kleikamp #include <linux/fs.h>
18f7f4bccbSMingming Cao #include <linux/jbd2.h>
19ede86cc4STheodore Ts'o #include <linux/marker.h>
20470decc6SDave Kleikamp #include <linux/errno.h>
21470decc6SDave Kleikamp #include <linux/slab.h>
22470decc6SDave Kleikamp #include <linux/mm.h>
23470decc6SDave Kleikamp #include <linux/pagemap.h>
248e85fb3fSJohann Lombardi #include <linux/jiffies.h>
25818d276cSGirish Shilamkar #include <linux/crc32.h>
26cd1aac32SAneesh Kumar K.V #include <linux/writeback.h>
27cd1aac32SAneesh Kumar K.V #include <linux/backing-dev.h>
28fd98496fSTheodore Ts'o #include <linux/bio.h>
29470decc6SDave Kleikamp 
30470decc6SDave Kleikamp /*
31470decc6SDave Kleikamp  * Default IO end handler for temporary BJ_IO buffer_heads.
32470decc6SDave Kleikamp  */
33470decc6SDave Kleikamp static void journal_end_buffer_io_sync(struct buffer_head *bh, int uptodate)
34470decc6SDave Kleikamp {
35470decc6SDave Kleikamp 	BUFFER_TRACE(bh, "");
36470decc6SDave Kleikamp 	if (uptodate)
37470decc6SDave Kleikamp 		set_buffer_uptodate(bh);
38470decc6SDave Kleikamp 	else
39470decc6SDave Kleikamp 		clear_buffer_uptodate(bh);
40470decc6SDave Kleikamp 	unlock_buffer(bh);
41470decc6SDave Kleikamp }
42470decc6SDave Kleikamp 
43470decc6SDave Kleikamp /*
4487c89c23SJan Kara  * When an ext4 file is truncated, it is possible that some pages are not
4587c89c23SJan Kara  * successfully freed, because they are attached to a committing transaction.
46470decc6SDave Kleikamp  * After the transaction commits, these pages are left on the LRU, with no
47470decc6SDave Kleikamp  * ->mapping, and with attached buffers.  These pages are trivially reclaimable
48470decc6SDave Kleikamp  * by the VM, but their apparent absence upsets the VM accounting, and it makes
49470decc6SDave Kleikamp  * the numbers in /proc/meminfo look odd.
50470decc6SDave Kleikamp  *
51470decc6SDave Kleikamp  * So here, we have a buffer which has just come off the forget list.  Look to
52470decc6SDave Kleikamp  * see if we can strip all buffers from the backing page.
53470decc6SDave Kleikamp  *
54470decc6SDave Kleikamp  * Called under lock_journal(), and possibly under journal_datalist_lock.  The
55470decc6SDave Kleikamp  * caller provided us with a ref against the buffer, and we drop that here.
56470decc6SDave Kleikamp  */
57470decc6SDave Kleikamp static void release_buffer_page(struct buffer_head *bh)
58470decc6SDave Kleikamp {
59470decc6SDave Kleikamp 	struct page *page;
60470decc6SDave Kleikamp 
61470decc6SDave Kleikamp 	if (buffer_dirty(bh))
62470decc6SDave Kleikamp 		goto nope;
63470decc6SDave Kleikamp 	if (atomic_read(&bh->b_count) != 1)
64470decc6SDave Kleikamp 		goto nope;
65470decc6SDave Kleikamp 	page = bh->b_page;
66470decc6SDave Kleikamp 	if (!page)
67470decc6SDave Kleikamp 		goto nope;
68470decc6SDave Kleikamp 	if (page->mapping)
69470decc6SDave Kleikamp 		goto nope;
70470decc6SDave Kleikamp 
71470decc6SDave Kleikamp 	/* OK, it's a truncated page */
72529ae9aaSNick Piggin 	if (!trylock_page(page))
73470decc6SDave Kleikamp 		goto nope;
74470decc6SDave Kleikamp 
75470decc6SDave Kleikamp 	page_cache_get(page);
76470decc6SDave Kleikamp 	__brelse(bh);
77470decc6SDave Kleikamp 	try_to_free_buffers(page);
78470decc6SDave Kleikamp 	unlock_page(page);
79470decc6SDave Kleikamp 	page_cache_release(page);
80470decc6SDave Kleikamp 	return;
81470decc6SDave Kleikamp 
82470decc6SDave Kleikamp nope:
83470decc6SDave Kleikamp 	__brelse(bh);
84470decc6SDave Kleikamp }
85470decc6SDave Kleikamp 
86470decc6SDave Kleikamp /*
87818d276cSGirish Shilamkar  * Done it all: now submit the commit record.  We should have
88470decc6SDave Kleikamp  * cleaned up our previous buffers by now, so if we are in abort
89470decc6SDave Kleikamp  * mode we can now just skip the rest of the journal write
90470decc6SDave Kleikamp  * entirely.
91470decc6SDave Kleikamp  *
92470decc6SDave Kleikamp  * Returns 1 if the journal needs to be aborted or 0 on success
93470decc6SDave Kleikamp  */
94818d276cSGirish Shilamkar static int journal_submit_commit_record(journal_t *journal,
95818d276cSGirish Shilamkar 					transaction_t *commit_transaction,
96818d276cSGirish Shilamkar 					struct buffer_head **cbh,
97818d276cSGirish Shilamkar 					__u32 crc32_sum)
98470decc6SDave Kleikamp {
99470decc6SDave Kleikamp 	struct journal_head *descriptor;
100818d276cSGirish Shilamkar 	struct commit_header *tmp;
101470decc6SDave Kleikamp 	struct buffer_head *bh;
102818d276cSGirish Shilamkar 	int ret;
103470decc6SDave Kleikamp 	int barrier_done = 0;
104736603abSTheodore Ts'o 	struct timespec now = current_kernel_time();
105470decc6SDave Kleikamp 
106470decc6SDave Kleikamp 	if (is_journal_aborted(journal))
107470decc6SDave Kleikamp 		return 0;
108470decc6SDave Kleikamp 
109f7f4bccbSMingming Cao 	descriptor = jbd2_journal_get_descriptor_buffer(journal);
110470decc6SDave Kleikamp 	if (!descriptor)
111470decc6SDave Kleikamp 		return 1;
112470decc6SDave Kleikamp 
113470decc6SDave Kleikamp 	bh = jh2bh(descriptor);
114470decc6SDave Kleikamp 
115818d276cSGirish Shilamkar 	tmp = (struct commit_header *)bh->b_data;
116f7f4bccbSMingming Cao 	tmp->h_magic = cpu_to_be32(JBD2_MAGIC_NUMBER);
117f7f4bccbSMingming Cao 	tmp->h_blocktype = cpu_to_be32(JBD2_COMMIT_BLOCK);
118470decc6SDave Kleikamp 	tmp->h_sequence = cpu_to_be32(commit_transaction->t_tid);
119736603abSTheodore Ts'o 	tmp->h_commit_sec = cpu_to_be64(now.tv_sec);
120736603abSTheodore Ts'o 	tmp->h_commit_nsec = cpu_to_be32(now.tv_nsec);
121818d276cSGirish Shilamkar 
122818d276cSGirish Shilamkar 	if (JBD2_HAS_COMPAT_FEATURE(journal,
123818d276cSGirish Shilamkar 				    JBD2_FEATURE_COMPAT_CHECKSUM)) {
124818d276cSGirish Shilamkar 		tmp->h_chksum_type 	= JBD2_CRC32_CHKSUM;
125818d276cSGirish Shilamkar 		tmp->h_chksum_size 	= JBD2_CRC32_CHKSUM_SIZE;
126818d276cSGirish Shilamkar 		tmp->h_chksum[0] 	= cpu_to_be32(crc32_sum);
127470decc6SDave Kleikamp 	}
128470decc6SDave Kleikamp 
129818d276cSGirish Shilamkar 	JBUFFER_TRACE(descriptor, "submit commit block");
130818d276cSGirish Shilamkar 	lock_buffer(bh);
13145a90bfdSTheodore Ts'o 	clear_buffer_dirty(bh);
132818d276cSGirish Shilamkar 	set_buffer_uptodate(bh);
133818d276cSGirish Shilamkar 	bh->b_end_io = journal_end_buffer_io_sync;
134818d276cSGirish Shilamkar 
135818d276cSGirish Shilamkar 	if (journal->j_flags & JBD2_BARRIER &&
1364d605179SAneesh Kumar K.V 		!JBD2_HAS_INCOMPAT_FEATURE(journal,
137818d276cSGirish Shilamkar 					 JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) {
138470decc6SDave Kleikamp 		set_buffer_ordered(bh);
139470decc6SDave Kleikamp 		barrier_done = 1;
140470decc6SDave Kleikamp 	}
1414194b1eaSJens Axboe 	ret = submit_bh(WRITE_SYNC_PLUG, bh);
142c4e35e07SDave Kleikamp 	if (barrier_done)
143c4e35e07SDave Kleikamp 		clear_buffer_ordered(bh);
144818d276cSGirish Shilamkar 
145470decc6SDave Kleikamp 	/* is it possible for another commit to fail at roughly
146470decc6SDave Kleikamp 	 * the same time as this one?  If so, we don't want to
147470decc6SDave Kleikamp 	 * trust the barrier flag in the super, but instead want
148470decc6SDave Kleikamp 	 * to remember if we sent a barrier request
149470decc6SDave Kleikamp 	 */
150470decc6SDave Kleikamp 	if (ret == -EOPNOTSUPP && barrier_done) {
151470decc6SDave Kleikamp 		printk(KERN_WARNING
152470decc6SDave Kleikamp 		       "JBD: barrier-based sync failed on %s - "
15305496769STheodore Ts'o 		       "disabling barriers\n", journal->j_devname);
154470decc6SDave Kleikamp 		spin_lock(&journal->j_state_lock);
155f7f4bccbSMingming Cao 		journal->j_flags &= ~JBD2_BARRIER;
156470decc6SDave Kleikamp 		spin_unlock(&journal->j_state_lock);
157470decc6SDave Kleikamp 
158470decc6SDave Kleikamp 		/* And try again, without the barrier */
159034772b0STheodore Ts'o 		lock_buffer(bh);
160470decc6SDave Kleikamp 		set_buffer_uptodate(bh);
16145a90bfdSTheodore Ts'o 		clear_buffer_dirty(bh);
1624194b1eaSJens Axboe 		ret = submit_bh(WRITE_SYNC_PLUG, bh);
163470decc6SDave Kleikamp 	}
164818d276cSGirish Shilamkar 	*cbh = bh;
165818d276cSGirish Shilamkar 	return ret;
166818d276cSGirish Shilamkar }
167470decc6SDave Kleikamp 
168818d276cSGirish Shilamkar /*
169818d276cSGirish Shilamkar  * This function along with journal_submit_commit_record
170818d276cSGirish Shilamkar  * allows to write the commit record asynchronously.
171818d276cSGirish Shilamkar  */
172fd98496fSTheodore Ts'o static int journal_wait_on_commit_record(journal_t *journal,
173fd98496fSTheodore Ts'o 					 struct buffer_head *bh)
174818d276cSGirish Shilamkar {
175818d276cSGirish Shilamkar 	int ret = 0;
176818d276cSGirish Shilamkar 
177fd98496fSTheodore Ts'o retry:
178818d276cSGirish Shilamkar 	clear_buffer_dirty(bh);
179818d276cSGirish Shilamkar 	wait_on_buffer(bh);
180fd98496fSTheodore Ts'o 	if (buffer_eopnotsupp(bh) && (journal->j_flags & JBD2_BARRIER)) {
181fd98496fSTheodore Ts'o 		printk(KERN_WARNING
182fd98496fSTheodore Ts'o 		       "JBD2: wait_on_commit_record: sync failed on %s - "
183fd98496fSTheodore Ts'o 		       "disabling barriers\n", journal->j_devname);
184fd98496fSTheodore Ts'o 		spin_lock(&journal->j_state_lock);
185fd98496fSTheodore Ts'o 		journal->j_flags &= ~JBD2_BARRIER;
186fd98496fSTheodore Ts'o 		spin_unlock(&journal->j_state_lock);
187fd98496fSTheodore Ts'o 
188fd98496fSTheodore Ts'o 		lock_buffer(bh);
189fd98496fSTheodore Ts'o 		clear_buffer_dirty(bh);
190fd98496fSTheodore Ts'o 		set_buffer_uptodate(bh);
191fd98496fSTheodore Ts'o 		bh->b_end_io = journal_end_buffer_io_sync;
192fd98496fSTheodore Ts'o 
1934194b1eaSJens Axboe 		ret = submit_bh(WRITE_SYNC_PLUG, bh);
194fd98496fSTheodore Ts'o 		if (ret) {
195fd98496fSTheodore Ts'o 			unlock_buffer(bh);
196fd98496fSTheodore Ts'o 			return ret;
197fd98496fSTheodore Ts'o 		}
198fd98496fSTheodore Ts'o 		goto retry;
199fd98496fSTheodore Ts'o 	}
200818d276cSGirish Shilamkar 
201818d276cSGirish Shilamkar 	if (unlikely(!buffer_uptodate(bh)))
202818d276cSGirish Shilamkar 		ret = -EIO;
203818d276cSGirish Shilamkar 	put_bh(bh);            /* One for getblk() */
204818d276cSGirish Shilamkar 	jbd2_journal_put_journal_head(bh2jh(bh));
205818d276cSGirish Shilamkar 
206818d276cSGirish Shilamkar 	return ret;
207818d276cSGirish Shilamkar }
208818d276cSGirish Shilamkar 
209818d276cSGirish Shilamkar /*
210cd1aac32SAneesh Kumar K.V  * write the filemap data using writepage() address_space_operations.
211cd1aac32SAneesh Kumar K.V  * We don't do block allocation here even for delalloc. We don't
212cd1aac32SAneesh Kumar K.V  * use writepages() because with dealyed allocation we may be doing
213cd1aac32SAneesh Kumar K.V  * block allocation in writepages().
214cd1aac32SAneesh Kumar K.V  */
215cd1aac32SAneesh Kumar K.V static int journal_submit_inode_data_buffers(struct address_space *mapping)
216cd1aac32SAneesh Kumar K.V {
217cd1aac32SAneesh Kumar K.V 	int ret;
218cd1aac32SAneesh Kumar K.V 	struct writeback_control wbc = {
219cd1aac32SAneesh Kumar K.V 		.sync_mode =  WB_SYNC_ALL,
220cd1aac32SAneesh Kumar K.V 		.nr_to_write = mapping->nrpages * 2,
221cd1aac32SAneesh Kumar K.V 		.range_start = 0,
222cd1aac32SAneesh Kumar K.V 		.range_end = i_size_read(mapping->host),
223cd1aac32SAneesh Kumar K.V 		.for_writepages = 1,
224cd1aac32SAneesh Kumar K.V 	};
225cd1aac32SAneesh Kumar K.V 
226cd1aac32SAneesh Kumar K.V 	ret = generic_writepages(mapping, &wbc);
227cd1aac32SAneesh Kumar K.V 	return ret;
228cd1aac32SAneesh Kumar K.V }
229cd1aac32SAneesh Kumar K.V 
230cd1aac32SAneesh Kumar K.V /*
231c851ed54SJan Kara  * Submit all the data buffers of inode associated with the transaction to
232c851ed54SJan Kara  * disk.
233c851ed54SJan Kara  *
234c851ed54SJan Kara  * We are in a committing transaction. Therefore no new inode can be added to
235c851ed54SJan Kara  * our inode list. We use JI_COMMIT_RUNNING flag to protect inode we currently
236c851ed54SJan Kara  * operate on from being released while we write out pages.
237c851ed54SJan Kara  */
238cd1aac32SAneesh Kumar K.V static int journal_submit_data_buffers(journal_t *journal,
239c851ed54SJan Kara 		transaction_t *commit_transaction)
240c851ed54SJan Kara {
241c851ed54SJan Kara 	struct jbd2_inode *jinode;
242c851ed54SJan Kara 	int err, ret = 0;
243c851ed54SJan Kara 	struct address_space *mapping;
244c851ed54SJan Kara 
245c851ed54SJan Kara 	spin_lock(&journal->j_list_lock);
246c851ed54SJan Kara 	list_for_each_entry(jinode, &commit_transaction->t_inode_list, i_list) {
247c851ed54SJan Kara 		mapping = jinode->i_vfs_inode->i_mapping;
248c851ed54SJan Kara 		jinode->i_flags |= JI_COMMIT_RUNNING;
249c851ed54SJan Kara 		spin_unlock(&journal->j_list_lock);
250cd1aac32SAneesh Kumar K.V 		/*
251cd1aac32SAneesh Kumar K.V 		 * submit the inode data buffers. We use writepage
252cd1aac32SAneesh Kumar K.V 		 * instead of writepages. Because writepages can do
253cd1aac32SAneesh Kumar K.V 		 * block allocation  with delalloc. We need to write
254cd1aac32SAneesh Kumar K.V 		 * only allocated blocks here.
255cd1aac32SAneesh Kumar K.V 		 */
256cd1aac32SAneesh Kumar K.V 		err = journal_submit_inode_data_buffers(mapping);
257c851ed54SJan Kara 		if (!ret)
258c851ed54SJan Kara 			ret = err;
259c851ed54SJan Kara 		spin_lock(&journal->j_list_lock);
260c851ed54SJan Kara 		J_ASSERT(jinode->i_transaction == commit_transaction);
261c851ed54SJan Kara 		jinode->i_flags &= ~JI_COMMIT_RUNNING;
262c851ed54SJan Kara 		wake_up_bit(&jinode->i_flags, __JI_COMMIT_RUNNING);
263c851ed54SJan Kara 	}
264c851ed54SJan Kara 	spin_unlock(&journal->j_list_lock);
265c851ed54SJan Kara 	return ret;
266c851ed54SJan Kara }
267c851ed54SJan Kara 
268c851ed54SJan Kara /*
269c851ed54SJan Kara  * Wait for data submitted for writeout, refile inodes to proper
270c851ed54SJan Kara  * transaction if needed.
271c851ed54SJan Kara  *
272c851ed54SJan Kara  */
273c851ed54SJan Kara static int journal_finish_inode_data_buffers(journal_t *journal,
274c851ed54SJan Kara 		transaction_t *commit_transaction)
275c851ed54SJan Kara {
276c851ed54SJan Kara 	struct jbd2_inode *jinode, *next_i;
277c851ed54SJan Kara 	int err, ret = 0;
278c851ed54SJan Kara 
279cd1aac32SAneesh Kumar K.V 	/* For locking, see the comment in journal_submit_data_buffers() */
280c851ed54SJan Kara 	spin_lock(&journal->j_list_lock);
281c851ed54SJan Kara 	list_for_each_entry(jinode, &commit_transaction->t_inode_list, i_list) {
282c851ed54SJan Kara 		jinode->i_flags |= JI_COMMIT_RUNNING;
283c851ed54SJan Kara 		spin_unlock(&journal->j_list_lock);
284c851ed54SJan Kara 		err = filemap_fdatawait(jinode->i_vfs_inode->i_mapping);
285e9e34f4eSHidehiro Kawai 		if (err) {
286e9e34f4eSHidehiro Kawai 			/*
287e9e34f4eSHidehiro Kawai 			 * Because AS_EIO is cleared by
288e9e34f4eSHidehiro Kawai 			 * wait_on_page_writeback_range(), set it again so
289e9e34f4eSHidehiro Kawai 			 * that user process can get -EIO from fsync().
290e9e34f4eSHidehiro Kawai 			 */
291e9e34f4eSHidehiro Kawai 			set_bit(AS_EIO,
292e9e34f4eSHidehiro Kawai 				&jinode->i_vfs_inode->i_mapping->flags);
293e9e34f4eSHidehiro Kawai 
294c851ed54SJan Kara 			if (!ret)
295c851ed54SJan Kara 				ret = err;
296e9e34f4eSHidehiro Kawai 		}
297c851ed54SJan Kara 		spin_lock(&journal->j_list_lock);
298c851ed54SJan Kara 		jinode->i_flags &= ~JI_COMMIT_RUNNING;
299c851ed54SJan Kara 		wake_up_bit(&jinode->i_flags, __JI_COMMIT_RUNNING);
300c851ed54SJan Kara 	}
301c851ed54SJan Kara 
302c851ed54SJan Kara 	/* Now refile inode to proper lists */
303c851ed54SJan Kara 	list_for_each_entry_safe(jinode, next_i,
304c851ed54SJan Kara 				 &commit_transaction->t_inode_list, i_list) {
305c851ed54SJan Kara 		list_del(&jinode->i_list);
306c851ed54SJan Kara 		if (jinode->i_next_transaction) {
307c851ed54SJan Kara 			jinode->i_transaction = jinode->i_next_transaction;
308c851ed54SJan Kara 			jinode->i_next_transaction = NULL;
309c851ed54SJan Kara 			list_add(&jinode->i_list,
310c851ed54SJan Kara 				&jinode->i_transaction->t_inode_list);
311c851ed54SJan Kara 		} else {
312c851ed54SJan Kara 			jinode->i_transaction = NULL;
313c851ed54SJan Kara 		}
314c851ed54SJan Kara 	}
315c851ed54SJan Kara 	spin_unlock(&journal->j_list_lock);
316c851ed54SJan Kara 
317c851ed54SJan Kara 	return ret;
318c851ed54SJan Kara }
319c851ed54SJan Kara 
320818d276cSGirish Shilamkar static __u32 jbd2_checksum_data(__u32 crc32_sum, struct buffer_head *bh)
321818d276cSGirish Shilamkar {
322818d276cSGirish Shilamkar 	struct page *page = bh->b_page;
323818d276cSGirish Shilamkar 	char *addr;
324818d276cSGirish Shilamkar 	__u32 checksum;
325818d276cSGirish Shilamkar 
326818d276cSGirish Shilamkar 	addr = kmap_atomic(page, KM_USER0);
327818d276cSGirish Shilamkar 	checksum = crc32_be(crc32_sum,
328818d276cSGirish Shilamkar 		(void *)(addr + offset_in_page(bh->b_data)), bh->b_size);
329818d276cSGirish Shilamkar 	kunmap_atomic(addr, KM_USER0);
330818d276cSGirish Shilamkar 
331818d276cSGirish Shilamkar 	return checksum;
332818d276cSGirish Shilamkar }
333818d276cSGirish Shilamkar 
334818d276cSGirish Shilamkar static void write_tag_block(int tag_bytes, journal_block_tag_t *tag,
33518eba7aaSMingming Cao 				   unsigned long long block)
336b517bea1SZach Brown {
337b517bea1SZach Brown 	tag->t_blocknr = cpu_to_be32(block & (u32)~0);
338cd02ff0bSMingming Cao 	if (tag_bytes > JBD2_TAG_SIZE32)
339b517bea1SZach Brown 		tag->t_blocknr_high = cpu_to_be32((block >> 31) >> 1);
340b517bea1SZach Brown }
341b517bea1SZach Brown 
342470decc6SDave Kleikamp /*
343f7f4bccbSMingming Cao  * jbd2_journal_commit_transaction
344470decc6SDave Kleikamp  *
345470decc6SDave Kleikamp  * The primary function for committing a transaction to the log.  This
346470decc6SDave Kleikamp  * function is called by the journal thread to begin a complete commit.
347470decc6SDave Kleikamp  */
348f7f4bccbSMingming Cao void jbd2_journal_commit_transaction(journal_t *journal)
349470decc6SDave Kleikamp {
3508e85fb3fSJohann Lombardi 	struct transaction_stats_s stats;
351470decc6SDave Kleikamp 	transaction_t *commit_transaction;
352470decc6SDave Kleikamp 	struct journal_head *jh, *new_jh, *descriptor;
353470decc6SDave Kleikamp 	struct buffer_head **wbuf = journal->j_wbuf;
354470decc6SDave Kleikamp 	int bufs;
355470decc6SDave Kleikamp 	int flags;
356470decc6SDave Kleikamp 	int err;
35718eba7aaSMingming Cao 	unsigned long long blocknr;
358e07f7183SJosef Bacik 	ktime_t start_time;
359e07f7183SJosef Bacik 	u64 commit_time;
360470decc6SDave Kleikamp 	char *tagp = NULL;
361470decc6SDave Kleikamp 	journal_header_t *header;
362470decc6SDave Kleikamp 	journal_block_tag_t *tag = NULL;
363470decc6SDave Kleikamp 	int space_left = 0;
364470decc6SDave Kleikamp 	int first_tag = 0;
365470decc6SDave Kleikamp 	int tag_flag;
366fb68407bSAneesh Kumar K.V 	int i, to_free = 0;
367b517bea1SZach Brown 	int tag_bytes = journal_tag_bytes(journal);
368818d276cSGirish Shilamkar 	struct buffer_head *cbh = NULL; /* For transactional checksums */
369818d276cSGirish Shilamkar 	__u32 crc32_sum = ~0;
3707058548cSTheodore Ts'o 	int write_op = WRITE;
371470decc6SDave Kleikamp 
372470decc6SDave Kleikamp 	/*
373470decc6SDave Kleikamp 	 * First job: lock down the current transaction and wait for
374470decc6SDave Kleikamp 	 * all outstanding updates to complete.
375470decc6SDave Kleikamp 	 */
376470decc6SDave Kleikamp 
377470decc6SDave Kleikamp #ifdef COMMIT_STATS
378470decc6SDave Kleikamp 	spin_lock(&journal->j_list_lock);
379470decc6SDave Kleikamp 	summarise_journal_usage(journal);
380470decc6SDave Kleikamp 	spin_unlock(&journal->j_list_lock);
381470decc6SDave Kleikamp #endif
382470decc6SDave Kleikamp 
383f7f4bccbSMingming Cao 	/* Do we need to erase the effects of a prior jbd2_journal_flush? */
384f7f4bccbSMingming Cao 	if (journal->j_flags & JBD2_FLUSHED) {
385470decc6SDave Kleikamp 		jbd_debug(3, "super block updated\n");
386f7f4bccbSMingming Cao 		jbd2_journal_update_superblock(journal, 1);
387470decc6SDave Kleikamp 	} else {
388470decc6SDave Kleikamp 		jbd_debug(3, "superblock not updated\n");
389470decc6SDave Kleikamp 	}
390470decc6SDave Kleikamp 
391470decc6SDave Kleikamp 	J_ASSERT(journal->j_running_transaction != NULL);
392470decc6SDave Kleikamp 	J_ASSERT(journal->j_committing_transaction == NULL);
393470decc6SDave Kleikamp 
394470decc6SDave Kleikamp 	commit_transaction = journal->j_running_transaction;
395470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_state == T_RUNNING);
396470decc6SDave Kleikamp 
397ede86cc4STheodore Ts'o 	trace_mark(jbd2_start_commit, "dev %s transaction %d",
398ede86cc4STheodore Ts'o 		   journal->j_devname, commit_transaction->t_tid);
399470decc6SDave Kleikamp 	jbd_debug(1, "JBD: starting commit of transaction %d\n",
400470decc6SDave Kleikamp 			commit_transaction->t_tid);
401470decc6SDave Kleikamp 
402470decc6SDave Kleikamp 	spin_lock(&journal->j_state_lock);
403470decc6SDave Kleikamp 	commit_transaction->t_state = T_LOCKED;
404470decc6SDave Kleikamp 
4054194b1eaSJens Axboe 	/*
4064194b1eaSJens Axboe 	 * Use plugged writes here, since we want to submit several before
4074194b1eaSJens Axboe 	 * we unplug the device. We don't do explicit unplugging in here,
4084194b1eaSJens Axboe 	 * instead we rely on sync_buffer() doing the unplug for us.
4094194b1eaSJens Axboe 	 */
4107058548cSTheodore Ts'o 	if (commit_transaction->t_synchronous_commit)
4114194b1eaSJens Axboe 		write_op = WRITE_SYNC_PLUG;
4128e85fb3fSJohann Lombardi 	stats.u.run.rs_wait = commit_transaction->t_max_wait;
4138e85fb3fSJohann Lombardi 	stats.u.run.rs_locked = jiffies;
4148e85fb3fSJohann Lombardi 	stats.u.run.rs_running = jbd2_time_diff(commit_transaction->t_start,
4158e85fb3fSJohann Lombardi 						stats.u.run.rs_locked);
4168e85fb3fSJohann Lombardi 
417470decc6SDave Kleikamp 	spin_lock(&commit_transaction->t_handle_lock);
418470decc6SDave Kleikamp 	while (commit_transaction->t_updates) {
419470decc6SDave Kleikamp 		DEFINE_WAIT(wait);
420470decc6SDave Kleikamp 
421470decc6SDave Kleikamp 		prepare_to_wait(&journal->j_wait_updates, &wait,
422470decc6SDave Kleikamp 					TASK_UNINTERRUPTIBLE);
423470decc6SDave Kleikamp 		if (commit_transaction->t_updates) {
424470decc6SDave Kleikamp 			spin_unlock(&commit_transaction->t_handle_lock);
425470decc6SDave Kleikamp 			spin_unlock(&journal->j_state_lock);
426470decc6SDave Kleikamp 			schedule();
427470decc6SDave Kleikamp 			spin_lock(&journal->j_state_lock);
428470decc6SDave Kleikamp 			spin_lock(&commit_transaction->t_handle_lock);
429470decc6SDave Kleikamp 		}
430470decc6SDave Kleikamp 		finish_wait(&journal->j_wait_updates, &wait);
431470decc6SDave Kleikamp 	}
432470decc6SDave Kleikamp 	spin_unlock(&commit_transaction->t_handle_lock);
433470decc6SDave Kleikamp 
434470decc6SDave Kleikamp 	J_ASSERT (commit_transaction->t_outstanding_credits <=
435470decc6SDave Kleikamp 			journal->j_max_transaction_buffers);
436470decc6SDave Kleikamp 
437470decc6SDave Kleikamp 	/*
438470decc6SDave Kleikamp 	 * First thing we are allowed to do is to discard any remaining
439470decc6SDave Kleikamp 	 * BJ_Reserved buffers.  Note, it is _not_ permissible to assume
440470decc6SDave Kleikamp 	 * that there are no such buffers: if a large filesystem
441470decc6SDave Kleikamp 	 * operation like a truncate needs to split itself over multiple
442f7f4bccbSMingming Cao 	 * transactions, then it may try to do a jbd2_journal_restart() while
443470decc6SDave Kleikamp 	 * there are still BJ_Reserved buffers outstanding.  These must
444470decc6SDave Kleikamp 	 * be released cleanly from the current transaction.
445470decc6SDave Kleikamp 	 *
446470decc6SDave Kleikamp 	 * In this case, the filesystem must still reserve write access
447470decc6SDave Kleikamp 	 * again before modifying the buffer in the new transaction, but
448470decc6SDave Kleikamp 	 * we do not require it to remember exactly which old buffers it
449470decc6SDave Kleikamp 	 * has reserved.  This is consistent with the existing behaviour
450f7f4bccbSMingming Cao 	 * that multiple jbd2_journal_get_write_access() calls to the same
451470decc6SDave Kleikamp 	 * buffer are perfectly permissable.
452470decc6SDave Kleikamp 	 */
453470decc6SDave Kleikamp 	while (commit_transaction->t_reserved_list) {
454470decc6SDave Kleikamp 		jh = commit_transaction->t_reserved_list;
455470decc6SDave Kleikamp 		JBUFFER_TRACE(jh, "reserved, unused: refile");
456470decc6SDave Kleikamp 		/*
457f7f4bccbSMingming Cao 		 * A jbd2_journal_get_undo_access()+jbd2_journal_release_buffer() may
458470decc6SDave Kleikamp 		 * leave undo-committed data.
459470decc6SDave Kleikamp 		 */
460470decc6SDave Kleikamp 		if (jh->b_committed_data) {
461470decc6SDave Kleikamp 			struct buffer_head *bh = jh2bh(jh);
462470decc6SDave Kleikamp 
463470decc6SDave Kleikamp 			jbd_lock_bh_state(bh);
464af1e76d6SMingming Cao 			jbd2_free(jh->b_committed_data, bh->b_size);
465470decc6SDave Kleikamp 			jh->b_committed_data = NULL;
466470decc6SDave Kleikamp 			jbd_unlock_bh_state(bh);
467470decc6SDave Kleikamp 		}
468f7f4bccbSMingming Cao 		jbd2_journal_refile_buffer(journal, jh);
469470decc6SDave Kleikamp 	}
470470decc6SDave Kleikamp 
471470decc6SDave Kleikamp 	/*
472470decc6SDave Kleikamp 	 * Now try to drop any written-back buffers from the journal's
473470decc6SDave Kleikamp 	 * checkpoint lists.  We do this *before* commit because it potentially
474470decc6SDave Kleikamp 	 * frees some memory
475470decc6SDave Kleikamp 	 */
476470decc6SDave Kleikamp 	spin_lock(&journal->j_list_lock);
477f7f4bccbSMingming Cao 	__jbd2_journal_clean_checkpoint_list(journal);
478470decc6SDave Kleikamp 	spin_unlock(&journal->j_list_lock);
479470decc6SDave Kleikamp 
480470decc6SDave Kleikamp 	jbd_debug (3, "JBD: commit phase 1\n");
481470decc6SDave Kleikamp 
482470decc6SDave Kleikamp 	/*
483470decc6SDave Kleikamp 	 * Switch to a new revoke table.
484470decc6SDave Kleikamp 	 */
485f7f4bccbSMingming Cao 	jbd2_journal_switch_revoke_table(journal);
486470decc6SDave Kleikamp 
4878e85fb3fSJohann Lombardi 	stats.u.run.rs_flushing = jiffies;
4888e85fb3fSJohann Lombardi 	stats.u.run.rs_locked = jbd2_time_diff(stats.u.run.rs_locked,
4898e85fb3fSJohann Lombardi 					       stats.u.run.rs_flushing);
4908e85fb3fSJohann Lombardi 
491470decc6SDave Kleikamp 	commit_transaction->t_state = T_FLUSH;
492470decc6SDave Kleikamp 	journal->j_committing_transaction = commit_transaction;
493470decc6SDave Kleikamp 	journal->j_running_transaction = NULL;
494e07f7183SJosef Bacik 	start_time = ktime_get();
495470decc6SDave Kleikamp 	commit_transaction->t_log_start = journal->j_head;
496470decc6SDave Kleikamp 	wake_up(&journal->j_wait_transaction_locked);
497470decc6SDave Kleikamp 	spin_unlock(&journal->j_state_lock);
498470decc6SDave Kleikamp 
499470decc6SDave Kleikamp 	jbd_debug (3, "JBD: commit phase 2\n");
500470decc6SDave Kleikamp 
501470decc6SDave Kleikamp 	/*
502470decc6SDave Kleikamp 	 * Now start flushing things to disk, in the order they appear
503470decc6SDave Kleikamp 	 * on the transaction lists.  Data blocks go first.
504470decc6SDave Kleikamp 	 */
505cd1aac32SAneesh Kumar K.V 	err = journal_submit_data_buffers(journal, commit_transaction);
506c851ed54SJan Kara 	if (err)
507c851ed54SJan Kara 		jbd2_journal_abort(journal, err);
508470decc6SDave Kleikamp 
509*67c457a8STheodore Ts'o 	jbd2_journal_write_revoke_records(journal, commit_transaction,
510*67c457a8STheodore Ts'o 					  write_op);
511470decc6SDave Kleikamp 
512470decc6SDave Kleikamp 	jbd_debug(3, "JBD: commit phase 2\n");
513470decc6SDave Kleikamp 
514470decc6SDave Kleikamp 	/*
515470decc6SDave Kleikamp 	 * Way to go: we have now written out all of the data for a
516470decc6SDave Kleikamp 	 * transaction!  Now comes the tricky part: we need to write out
517470decc6SDave Kleikamp 	 * metadata.  Loop over the transaction's entire buffer list:
518470decc6SDave Kleikamp 	 */
51902c471cbSMingming Cao 	spin_lock(&journal->j_state_lock);
520470decc6SDave Kleikamp 	commit_transaction->t_state = T_COMMIT;
52102c471cbSMingming Cao 	spin_unlock(&journal->j_state_lock);
522470decc6SDave Kleikamp 
5238e85fb3fSJohann Lombardi 	stats.u.run.rs_logging = jiffies;
5248e85fb3fSJohann Lombardi 	stats.u.run.rs_flushing = jbd2_time_diff(stats.u.run.rs_flushing,
5258e85fb3fSJohann Lombardi 						 stats.u.run.rs_logging);
5268e85fb3fSJohann Lombardi 	stats.u.run.rs_blocks = commit_transaction->t_outstanding_credits;
5278e85fb3fSJohann Lombardi 	stats.u.run.rs_blocks_logged = 0;
5288e85fb3fSJohann Lombardi 
5291dfc3220SJosef Bacik 	J_ASSERT(commit_transaction->t_nr_buffers <=
5301dfc3220SJosef Bacik 		 commit_transaction->t_outstanding_credits);
5311dfc3220SJosef Bacik 
53287c89c23SJan Kara 	err = 0;
533470decc6SDave Kleikamp 	descriptor = NULL;
534470decc6SDave Kleikamp 	bufs = 0;
535470decc6SDave Kleikamp 	while (commit_transaction->t_buffers) {
536470decc6SDave Kleikamp 
537470decc6SDave Kleikamp 		/* Find the next buffer to be journaled... */
538470decc6SDave Kleikamp 
539470decc6SDave Kleikamp 		jh = commit_transaction->t_buffers;
540470decc6SDave Kleikamp 
541470decc6SDave Kleikamp 		/* If we're in abort mode, we just un-journal the buffer and
5427ad7445fSHidehiro Kawai 		   release it. */
543470decc6SDave Kleikamp 
544470decc6SDave Kleikamp 		if (is_journal_aborted(journal)) {
5457ad7445fSHidehiro Kawai 			clear_buffer_jbddirty(jh2bh(jh));
546470decc6SDave Kleikamp 			JBUFFER_TRACE(jh, "journal is aborting: refile");
547e06c8227SJoel Becker 			jbd2_buffer_abort_trigger(jh,
548e06c8227SJoel Becker 						  jh->b_frozen_data ?
549e06c8227SJoel Becker 						  jh->b_frozen_triggers :
550e06c8227SJoel Becker 						  jh->b_triggers);
551f7f4bccbSMingming Cao 			jbd2_journal_refile_buffer(journal, jh);
552470decc6SDave Kleikamp 			/* If that was the last one, we need to clean up
553470decc6SDave Kleikamp 			 * any descriptor buffers which may have been
554470decc6SDave Kleikamp 			 * already allocated, even if we are now
555470decc6SDave Kleikamp 			 * aborting. */
556470decc6SDave Kleikamp 			if (!commit_transaction->t_buffers)
557470decc6SDave Kleikamp 				goto start_journal_io;
558470decc6SDave Kleikamp 			continue;
559470decc6SDave Kleikamp 		}
560470decc6SDave Kleikamp 
561470decc6SDave Kleikamp 		/* Make sure we have a descriptor block in which to
562470decc6SDave Kleikamp 		   record the metadata buffer. */
563470decc6SDave Kleikamp 
564470decc6SDave Kleikamp 		if (!descriptor) {
565470decc6SDave Kleikamp 			struct buffer_head *bh;
566470decc6SDave Kleikamp 
567470decc6SDave Kleikamp 			J_ASSERT (bufs == 0);
568470decc6SDave Kleikamp 
569470decc6SDave Kleikamp 			jbd_debug(4, "JBD: get descriptor\n");
570470decc6SDave Kleikamp 
571f7f4bccbSMingming Cao 			descriptor = jbd2_journal_get_descriptor_buffer(journal);
572470decc6SDave Kleikamp 			if (!descriptor) {
573a7fa2bafSJan Kara 				jbd2_journal_abort(journal, -EIO);
574470decc6SDave Kleikamp 				continue;
575470decc6SDave Kleikamp 			}
576470decc6SDave Kleikamp 
577470decc6SDave Kleikamp 			bh = jh2bh(descriptor);
578470decc6SDave Kleikamp 			jbd_debug(4, "JBD: got buffer %llu (%p)\n",
579470decc6SDave Kleikamp 				(unsigned long long)bh->b_blocknr, bh->b_data);
580470decc6SDave Kleikamp 			header = (journal_header_t *)&bh->b_data[0];
581f7f4bccbSMingming Cao 			header->h_magic     = cpu_to_be32(JBD2_MAGIC_NUMBER);
582f7f4bccbSMingming Cao 			header->h_blocktype = cpu_to_be32(JBD2_DESCRIPTOR_BLOCK);
583470decc6SDave Kleikamp 			header->h_sequence  = cpu_to_be32(commit_transaction->t_tid);
584470decc6SDave Kleikamp 
585470decc6SDave Kleikamp 			tagp = &bh->b_data[sizeof(journal_header_t)];
586470decc6SDave Kleikamp 			space_left = bh->b_size - sizeof(journal_header_t);
587470decc6SDave Kleikamp 			first_tag = 1;
588470decc6SDave Kleikamp 			set_buffer_jwrite(bh);
589470decc6SDave Kleikamp 			set_buffer_dirty(bh);
590470decc6SDave Kleikamp 			wbuf[bufs++] = bh;
591470decc6SDave Kleikamp 
592470decc6SDave Kleikamp 			/* Record it so that we can wait for IO
593470decc6SDave Kleikamp                            completion later */
594470decc6SDave Kleikamp 			BUFFER_TRACE(bh, "ph3: file as descriptor");
595f7f4bccbSMingming Cao 			jbd2_journal_file_buffer(descriptor, commit_transaction,
596470decc6SDave Kleikamp 					BJ_LogCtl);
597470decc6SDave Kleikamp 		}
598470decc6SDave Kleikamp 
599470decc6SDave Kleikamp 		/* Where is the buffer to be written? */
600470decc6SDave Kleikamp 
601f7f4bccbSMingming Cao 		err = jbd2_journal_next_log_block(journal, &blocknr);
602470decc6SDave Kleikamp 		/* If the block mapping failed, just abandon the buffer
603470decc6SDave Kleikamp 		   and repeat this loop: we'll fall into the
604470decc6SDave Kleikamp 		   refile-on-abort condition above. */
605470decc6SDave Kleikamp 		if (err) {
606a7fa2bafSJan Kara 			jbd2_journal_abort(journal, err);
607470decc6SDave Kleikamp 			continue;
608470decc6SDave Kleikamp 		}
609470decc6SDave Kleikamp 
610470decc6SDave Kleikamp 		/*
611470decc6SDave Kleikamp 		 * start_this_handle() uses t_outstanding_credits to determine
612470decc6SDave Kleikamp 		 * the free space in the log, but this counter is changed
613f7f4bccbSMingming Cao 		 * by jbd2_journal_next_log_block() also.
614470decc6SDave Kleikamp 		 */
615470decc6SDave Kleikamp 		commit_transaction->t_outstanding_credits--;
616470decc6SDave Kleikamp 
617470decc6SDave Kleikamp 		/* Bump b_count to prevent truncate from stumbling over
618470decc6SDave Kleikamp                    the shadowed buffer!  @@@ This can go if we ever get
619470decc6SDave Kleikamp                    rid of the BJ_IO/BJ_Shadow pairing of buffers. */
620470decc6SDave Kleikamp 		atomic_inc(&jh2bh(jh)->b_count);
621470decc6SDave Kleikamp 
622470decc6SDave Kleikamp 		/* Make a temporary IO buffer with which to write it out
623470decc6SDave Kleikamp                    (this will requeue both the metadata buffer and the
624470decc6SDave Kleikamp                    temporary IO buffer). new_bh goes on BJ_IO*/
625470decc6SDave Kleikamp 
626470decc6SDave Kleikamp 		set_bit(BH_JWrite, &jh2bh(jh)->b_state);
627470decc6SDave Kleikamp 		/*
628f7f4bccbSMingming Cao 		 * akpm: jbd2_journal_write_metadata_buffer() sets
629470decc6SDave Kleikamp 		 * new_bh->b_transaction to commit_transaction.
630470decc6SDave Kleikamp 		 * We need to clean this up before we release new_bh
631470decc6SDave Kleikamp 		 * (which is of type BJ_IO)
632470decc6SDave Kleikamp 		 */
633470decc6SDave Kleikamp 		JBUFFER_TRACE(jh, "ph3: write metadata");
634f7f4bccbSMingming Cao 		flags = jbd2_journal_write_metadata_buffer(commit_transaction,
635470decc6SDave Kleikamp 						      jh, &new_jh, blocknr);
636470decc6SDave Kleikamp 		set_bit(BH_JWrite, &jh2bh(new_jh)->b_state);
637470decc6SDave Kleikamp 		wbuf[bufs++] = jh2bh(new_jh);
638470decc6SDave Kleikamp 
639470decc6SDave Kleikamp 		/* Record the new block's tag in the current descriptor
640470decc6SDave Kleikamp                    buffer */
641470decc6SDave Kleikamp 
642470decc6SDave Kleikamp 		tag_flag = 0;
643470decc6SDave Kleikamp 		if (flags & 1)
644f7f4bccbSMingming Cao 			tag_flag |= JBD2_FLAG_ESCAPE;
645470decc6SDave Kleikamp 		if (!first_tag)
646f7f4bccbSMingming Cao 			tag_flag |= JBD2_FLAG_SAME_UUID;
647470decc6SDave Kleikamp 
648470decc6SDave Kleikamp 		tag = (journal_block_tag_t *) tagp;
649b517bea1SZach Brown 		write_tag_block(tag_bytes, tag, jh2bh(jh)->b_blocknr);
650470decc6SDave Kleikamp 		tag->t_flags = cpu_to_be32(tag_flag);
651b517bea1SZach Brown 		tagp += tag_bytes;
652b517bea1SZach Brown 		space_left -= tag_bytes;
653470decc6SDave Kleikamp 
654470decc6SDave Kleikamp 		if (first_tag) {
655470decc6SDave Kleikamp 			memcpy (tagp, journal->j_uuid, 16);
656470decc6SDave Kleikamp 			tagp += 16;
657470decc6SDave Kleikamp 			space_left -= 16;
658470decc6SDave Kleikamp 			first_tag = 0;
659470decc6SDave Kleikamp 		}
660470decc6SDave Kleikamp 
661470decc6SDave Kleikamp 		/* If there's no more to do, or if the descriptor is full,
662470decc6SDave Kleikamp 		   let the IO rip! */
663470decc6SDave Kleikamp 
664470decc6SDave Kleikamp 		if (bufs == journal->j_wbufsize ||
665470decc6SDave Kleikamp 		    commit_transaction->t_buffers == NULL ||
666b517bea1SZach Brown 		    space_left < tag_bytes + 16) {
667470decc6SDave Kleikamp 
668470decc6SDave Kleikamp 			jbd_debug(4, "JBD: Submit %d IOs\n", bufs);
669470decc6SDave Kleikamp 
670470decc6SDave Kleikamp 			/* Write an end-of-descriptor marker before
671470decc6SDave Kleikamp                            submitting the IOs.  "tag" still points to
672470decc6SDave Kleikamp                            the last tag we set up. */
673470decc6SDave Kleikamp 
674f7f4bccbSMingming Cao 			tag->t_flags |= cpu_to_be32(JBD2_FLAG_LAST_TAG);
675470decc6SDave Kleikamp 
676470decc6SDave Kleikamp start_journal_io:
677470decc6SDave Kleikamp 			for (i = 0; i < bufs; i++) {
678470decc6SDave Kleikamp 				struct buffer_head *bh = wbuf[i];
679818d276cSGirish Shilamkar 				/*
680818d276cSGirish Shilamkar 				 * Compute checksum.
681818d276cSGirish Shilamkar 				 */
682818d276cSGirish Shilamkar 				if (JBD2_HAS_COMPAT_FEATURE(journal,
683818d276cSGirish Shilamkar 					JBD2_FEATURE_COMPAT_CHECKSUM)) {
684818d276cSGirish Shilamkar 					crc32_sum =
685818d276cSGirish Shilamkar 					    jbd2_checksum_data(crc32_sum, bh);
686818d276cSGirish Shilamkar 				}
687818d276cSGirish Shilamkar 
688470decc6SDave Kleikamp 				lock_buffer(bh);
689470decc6SDave Kleikamp 				clear_buffer_dirty(bh);
690470decc6SDave Kleikamp 				set_buffer_uptodate(bh);
691470decc6SDave Kleikamp 				bh->b_end_io = journal_end_buffer_io_sync;
6927058548cSTheodore Ts'o 				submit_bh(write_op, bh);
693470decc6SDave Kleikamp 			}
694470decc6SDave Kleikamp 			cond_resched();
6958e85fb3fSJohann Lombardi 			stats.u.run.rs_blocks_logged += bufs;
696470decc6SDave Kleikamp 
697470decc6SDave Kleikamp 			/* Force a new descriptor to be generated next
698470decc6SDave Kleikamp                            time round the loop. */
699470decc6SDave Kleikamp 			descriptor = NULL;
700470decc6SDave Kleikamp 			bufs = 0;
701470decc6SDave Kleikamp 		}
702470decc6SDave Kleikamp 	}
703470decc6SDave Kleikamp 
704818d276cSGirish Shilamkar 	/* Done it all: now write the commit record asynchronously. */
705818d276cSGirish Shilamkar 
706818d276cSGirish Shilamkar 	if (JBD2_HAS_INCOMPAT_FEATURE(journal,
707818d276cSGirish Shilamkar 		JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) {
708818d276cSGirish Shilamkar 		err = journal_submit_commit_record(journal, commit_transaction,
709818d276cSGirish Shilamkar 						 &cbh, crc32_sum);
710818d276cSGirish Shilamkar 		if (err)
711818d276cSGirish Shilamkar 			__jbd2_journal_abort_hard(journal);
712818d276cSGirish Shilamkar 	}
713818d276cSGirish Shilamkar 
714c851ed54SJan Kara 	/*
715c851ed54SJan Kara 	 * This is the right place to wait for data buffers both for ASYNC
716c851ed54SJan Kara 	 * and !ASYNC commit. If commit is ASYNC, we need to wait only after
717c851ed54SJan Kara 	 * the commit block went to disk (which happens above). If commit is
718c851ed54SJan Kara 	 * SYNC, we need to wait for data buffers before we start writing
719c851ed54SJan Kara 	 * commit block, which happens below in such setting.
720c851ed54SJan Kara 	 */
721c851ed54SJan Kara 	err = journal_finish_inode_data_buffers(journal, commit_transaction);
722e9e34f4eSHidehiro Kawai 	if (err) {
723e9e34f4eSHidehiro Kawai 		printk(KERN_WARNING
724e9e34f4eSHidehiro Kawai 			"JBD2: Detected IO errors while flushing file data "
72505496769STheodore Ts'o 		       "on %s\n", journal->j_devname);
7265bf5683aSHidehiro Kawai 		if (journal->j_flags & JBD2_ABORT_ON_SYNCDATA_ERR)
7275bf5683aSHidehiro Kawai 			jbd2_journal_abort(journal, err);
728e9e34f4eSHidehiro Kawai 		err = 0;
729e9e34f4eSHidehiro Kawai 	}
730c851ed54SJan Kara 
731470decc6SDave Kleikamp 	/* Lo and behold: we have just managed to send a transaction to
732470decc6SDave Kleikamp            the log.  Before we can commit it, wait for the IO so far to
733470decc6SDave Kleikamp            complete.  Control buffers being written are on the
734470decc6SDave Kleikamp            transaction's t_log_list queue, and metadata buffers are on
735470decc6SDave Kleikamp            the t_iobuf_list queue.
736470decc6SDave Kleikamp 
737470decc6SDave Kleikamp 	   Wait for the buffers in reverse order.  That way we are
738470decc6SDave Kleikamp 	   less likely to be woken up until all IOs have completed, and
739470decc6SDave Kleikamp 	   so we incur less scheduling load.
740470decc6SDave Kleikamp 	*/
741470decc6SDave Kleikamp 
74287c89c23SJan Kara 	jbd_debug(3, "JBD: commit phase 3\n");
743470decc6SDave Kleikamp 
744470decc6SDave Kleikamp 	/*
745470decc6SDave Kleikamp 	 * akpm: these are BJ_IO, and j_list_lock is not needed.
746470decc6SDave Kleikamp 	 * See __journal_try_to_free_buffer.
747470decc6SDave Kleikamp 	 */
748470decc6SDave Kleikamp wait_for_iobuf:
749470decc6SDave Kleikamp 	while (commit_transaction->t_iobuf_list != NULL) {
750470decc6SDave Kleikamp 		struct buffer_head *bh;
751470decc6SDave Kleikamp 
752470decc6SDave Kleikamp 		jh = commit_transaction->t_iobuf_list->b_tprev;
753470decc6SDave Kleikamp 		bh = jh2bh(jh);
754470decc6SDave Kleikamp 		if (buffer_locked(bh)) {
755470decc6SDave Kleikamp 			wait_on_buffer(bh);
756470decc6SDave Kleikamp 			goto wait_for_iobuf;
757470decc6SDave Kleikamp 		}
758470decc6SDave Kleikamp 		if (cond_resched())
759470decc6SDave Kleikamp 			goto wait_for_iobuf;
760470decc6SDave Kleikamp 
761470decc6SDave Kleikamp 		if (unlikely(!buffer_uptodate(bh)))
762470decc6SDave Kleikamp 			err = -EIO;
763470decc6SDave Kleikamp 
764470decc6SDave Kleikamp 		clear_buffer_jwrite(bh);
765470decc6SDave Kleikamp 
766470decc6SDave Kleikamp 		JBUFFER_TRACE(jh, "ph4: unfile after journal write");
767f7f4bccbSMingming Cao 		jbd2_journal_unfile_buffer(journal, jh);
768470decc6SDave Kleikamp 
769470decc6SDave Kleikamp 		/*
770470decc6SDave Kleikamp 		 * ->t_iobuf_list should contain only dummy buffer_heads
771f7f4bccbSMingming Cao 		 * which were created by jbd2_journal_write_metadata_buffer().
772470decc6SDave Kleikamp 		 */
773470decc6SDave Kleikamp 		BUFFER_TRACE(bh, "dumping temporary bh");
774f7f4bccbSMingming Cao 		jbd2_journal_put_journal_head(jh);
775470decc6SDave Kleikamp 		__brelse(bh);
776470decc6SDave Kleikamp 		J_ASSERT_BH(bh, atomic_read(&bh->b_count) == 0);
777470decc6SDave Kleikamp 		free_buffer_head(bh);
778470decc6SDave Kleikamp 
779470decc6SDave Kleikamp 		/* We also have to unlock and free the corresponding
780470decc6SDave Kleikamp                    shadowed buffer */
781470decc6SDave Kleikamp 		jh = commit_transaction->t_shadow_list->b_tprev;
782470decc6SDave Kleikamp 		bh = jh2bh(jh);
783470decc6SDave Kleikamp 		clear_bit(BH_JWrite, &bh->b_state);
784470decc6SDave Kleikamp 		J_ASSERT_BH(bh, buffer_jbddirty(bh));
785470decc6SDave Kleikamp 
786470decc6SDave Kleikamp 		/* The metadata is now released for reuse, but we need
787470decc6SDave Kleikamp                    to remember it against this transaction so that when
788470decc6SDave Kleikamp                    we finally commit, we can do any checkpointing
789470decc6SDave Kleikamp                    required. */
790470decc6SDave Kleikamp 		JBUFFER_TRACE(jh, "file as BJ_Forget");
791f7f4bccbSMingming Cao 		jbd2_journal_file_buffer(jh, commit_transaction, BJ_Forget);
792470decc6SDave Kleikamp 		/* Wake up any transactions which were waiting for this
793470decc6SDave Kleikamp 		   IO to complete */
794470decc6SDave Kleikamp 		wake_up_bit(&bh->b_state, BH_Unshadow);
795470decc6SDave Kleikamp 		JBUFFER_TRACE(jh, "brelse shadowed buffer");
796470decc6SDave Kleikamp 		__brelse(bh);
797470decc6SDave Kleikamp 	}
798470decc6SDave Kleikamp 
799470decc6SDave Kleikamp 	J_ASSERT (commit_transaction->t_shadow_list == NULL);
800470decc6SDave Kleikamp 
80187c89c23SJan Kara 	jbd_debug(3, "JBD: commit phase 4\n");
802470decc6SDave Kleikamp 
803470decc6SDave Kleikamp 	/* Here we wait for the revoke record and descriptor record buffers */
804470decc6SDave Kleikamp  wait_for_ctlbuf:
805470decc6SDave Kleikamp 	while (commit_transaction->t_log_list != NULL) {
806470decc6SDave Kleikamp 		struct buffer_head *bh;
807470decc6SDave Kleikamp 
808470decc6SDave Kleikamp 		jh = commit_transaction->t_log_list->b_tprev;
809470decc6SDave Kleikamp 		bh = jh2bh(jh);
810470decc6SDave Kleikamp 		if (buffer_locked(bh)) {
811470decc6SDave Kleikamp 			wait_on_buffer(bh);
812470decc6SDave Kleikamp 			goto wait_for_ctlbuf;
813470decc6SDave Kleikamp 		}
814470decc6SDave Kleikamp 		if (cond_resched())
815470decc6SDave Kleikamp 			goto wait_for_ctlbuf;
816470decc6SDave Kleikamp 
817470decc6SDave Kleikamp 		if (unlikely(!buffer_uptodate(bh)))
818470decc6SDave Kleikamp 			err = -EIO;
819470decc6SDave Kleikamp 
820470decc6SDave Kleikamp 		BUFFER_TRACE(bh, "ph5: control buffer writeout done: unfile");
821470decc6SDave Kleikamp 		clear_buffer_jwrite(bh);
822f7f4bccbSMingming Cao 		jbd2_journal_unfile_buffer(journal, jh);
823f7f4bccbSMingming Cao 		jbd2_journal_put_journal_head(jh);
824470decc6SDave Kleikamp 		__brelse(bh);		/* One for getblk */
825470decc6SDave Kleikamp 		/* AKPM: bforget here */
826470decc6SDave Kleikamp 	}
827470decc6SDave Kleikamp 
82877e841deSHidehiro Kawai 	if (err)
82977e841deSHidehiro Kawai 		jbd2_journal_abort(journal, err);
83077e841deSHidehiro Kawai 
83187c89c23SJan Kara 	jbd_debug(3, "JBD: commit phase 5\n");
832470decc6SDave Kleikamp 
833818d276cSGirish Shilamkar 	if (!JBD2_HAS_INCOMPAT_FEATURE(journal,
834818d276cSGirish Shilamkar 		JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) {
835818d276cSGirish Shilamkar 		err = journal_submit_commit_record(journal, commit_transaction,
836818d276cSGirish Shilamkar 						&cbh, crc32_sum);
837818d276cSGirish Shilamkar 		if (err)
838818d276cSGirish Shilamkar 			__jbd2_journal_abort_hard(journal);
839818d276cSGirish Shilamkar 	}
840b048d846SMingming Cao 	if (!err && !is_journal_aborted(journal))
841fd98496fSTheodore Ts'o 		err = journal_wait_on_commit_record(journal, cbh);
842470decc6SDave Kleikamp 
843470decc6SDave Kleikamp 	if (err)
844a7fa2bafSJan Kara 		jbd2_journal_abort(journal, err);
845470decc6SDave Kleikamp 
846470decc6SDave Kleikamp 	/* End of a transaction!  Finally, we can do checkpoint
847470decc6SDave Kleikamp            processing: any buffers committed as a result of this
848470decc6SDave Kleikamp            transaction can be removed from any checkpoint list it was on
849470decc6SDave Kleikamp            before. */
850470decc6SDave Kleikamp 
85187c89c23SJan Kara 	jbd_debug(3, "JBD: commit phase 6\n");
852470decc6SDave Kleikamp 
853c851ed54SJan Kara 	J_ASSERT(list_empty(&commit_transaction->t_inode_list));
854470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_buffers == NULL);
855470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_checkpoint_list == NULL);
856470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_iobuf_list == NULL);
857470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_shadow_list == NULL);
858470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_log_list == NULL);
859470decc6SDave Kleikamp 
860470decc6SDave Kleikamp restart_loop:
861470decc6SDave Kleikamp 	/*
862470decc6SDave Kleikamp 	 * As there are other places (journal_unmap_buffer()) adding buffers
863470decc6SDave Kleikamp 	 * to this list we have to be careful and hold the j_list_lock.
864470decc6SDave Kleikamp 	 */
865470decc6SDave Kleikamp 	spin_lock(&journal->j_list_lock);
866470decc6SDave Kleikamp 	while (commit_transaction->t_forget) {
867470decc6SDave Kleikamp 		transaction_t *cp_transaction;
868470decc6SDave Kleikamp 		struct buffer_head *bh;
869470decc6SDave Kleikamp 
870470decc6SDave Kleikamp 		jh = commit_transaction->t_forget;
871470decc6SDave Kleikamp 		spin_unlock(&journal->j_list_lock);
872470decc6SDave Kleikamp 		bh = jh2bh(jh);
873470decc6SDave Kleikamp 		jbd_lock_bh_state(bh);
874470decc6SDave Kleikamp 		J_ASSERT_JH(jh,	jh->b_transaction == commit_transaction ||
875470decc6SDave Kleikamp 			jh->b_transaction == journal->j_running_transaction);
876470decc6SDave Kleikamp 
877470decc6SDave Kleikamp 		/*
878470decc6SDave Kleikamp 		 * If there is undo-protected committed data against
879470decc6SDave Kleikamp 		 * this buffer, then we can remove it now.  If it is a
880470decc6SDave Kleikamp 		 * buffer needing such protection, the old frozen_data
881470decc6SDave Kleikamp 		 * field now points to a committed version of the
882470decc6SDave Kleikamp 		 * buffer, so rotate that field to the new committed
883470decc6SDave Kleikamp 		 * data.
884470decc6SDave Kleikamp 		 *
885470decc6SDave Kleikamp 		 * Otherwise, we can just throw away the frozen data now.
886e06c8227SJoel Becker 		 *
887e06c8227SJoel Becker 		 * We also know that the frozen data has already fired
888e06c8227SJoel Becker 		 * its triggers if they exist, so we can clear that too.
889470decc6SDave Kleikamp 		 */
890470decc6SDave Kleikamp 		if (jh->b_committed_data) {
891af1e76d6SMingming Cao 			jbd2_free(jh->b_committed_data, bh->b_size);
892470decc6SDave Kleikamp 			jh->b_committed_data = NULL;
893470decc6SDave Kleikamp 			if (jh->b_frozen_data) {
894470decc6SDave Kleikamp 				jh->b_committed_data = jh->b_frozen_data;
895470decc6SDave Kleikamp 				jh->b_frozen_data = NULL;
896e06c8227SJoel Becker 				jh->b_frozen_triggers = NULL;
897470decc6SDave Kleikamp 			}
898470decc6SDave Kleikamp 		} else if (jh->b_frozen_data) {
899af1e76d6SMingming Cao 			jbd2_free(jh->b_frozen_data, bh->b_size);
900470decc6SDave Kleikamp 			jh->b_frozen_data = NULL;
901e06c8227SJoel Becker 			jh->b_frozen_triggers = NULL;
902470decc6SDave Kleikamp 		}
903470decc6SDave Kleikamp 
904470decc6SDave Kleikamp 		spin_lock(&journal->j_list_lock);
905470decc6SDave Kleikamp 		cp_transaction = jh->b_cp_transaction;
906470decc6SDave Kleikamp 		if (cp_transaction) {
907470decc6SDave Kleikamp 			JBUFFER_TRACE(jh, "remove from old cp transaction");
9088e85fb3fSJohann Lombardi 			cp_transaction->t_chp_stats.cs_dropped++;
909f7f4bccbSMingming Cao 			__jbd2_journal_remove_checkpoint(jh);
910470decc6SDave Kleikamp 		}
911470decc6SDave Kleikamp 
912470decc6SDave Kleikamp 		/* Only re-checkpoint the buffer_head if it is marked
913470decc6SDave Kleikamp 		 * dirty.  If the buffer was added to the BJ_Forget list
914f7f4bccbSMingming Cao 		 * by jbd2_journal_forget, it may no longer be dirty and
915470decc6SDave Kleikamp 		 * there's no point in keeping a checkpoint record for
916470decc6SDave Kleikamp 		 * it. */
917470decc6SDave Kleikamp 
918470decc6SDave Kleikamp 		/* A buffer which has been freed while still being
919470decc6SDave Kleikamp 		 * journaled by a previous transaction may end up still
920470decc6SDave Kleikamp 		 * being dirty here, but we want to avoid writing back
921470decc6SDave Kleikamp 		 * that buffer in the future now that the last use has
922470decc6SDave Kleikamp 		 * been committed.  That's not only a performance gain,
923470decc6SDave Kleikamp 		 * it also stops aliasing problems if the buffer is left
924470decc6SDave Kleikamp 		 * behind for writeback and gets reallocated for another
925470decc6SDave Kleikamp 		 * use in a different page. */
926470decc6SDave Kleikamp 		if (buffer_freed(bh)) {
927470decc6SDave Kleikamp 			clear_buffer_freed(bh);
928470decc6SDave Kleikamp 			clear_buffer_jbddirty(bh);
929470decc6SDave Kleikamp 		}
930470decc6SDave Kleikamp 
931470decc6SDave Kleikamp 		if (buffer_jbddirty(bh)) {
932470decc6SDave Kleikamp 			JBUFFER_TRACE(jh, "add to new checkpointing trans");
933f7f4bccbSMingming Cao 			__jbd2_journal_insert_checkpoint(jh, commit_transaction);
9347ad7445fSHidehiro Kawai 			if (is_journal_aborted(journal))
9357ad7445fSHidehiro Kawai 				clear_buffer_jbddirty(bh);
936470decc6SDave Kleikamp 			JBUFFER_TRACE(jh, "refile for checkpoint writeback");
937f7f4bccbSMingming Cao 			__jbd2_journal_refile_buffer(jh);
938470decc6SDave Kleikamp 			jbd_unlock_bh_state(bh);
939470decc6SDave Kleikamp 		} else {
940470decc6SDave Kleikamp 			J_ASSERT_BH(bh, !buffer_dirty(bh));
941470decc6SDave Kleikamp 			/* The buffer on BJ_Forget list and not jbddirty means
942470decc6SDave Kleikamp 			 * it has been freed by this transaction and hence it
943470decc6SDave Kleikamp 			 * could not have been reallocated until this
944470decc6SDave Kleikamp 			 * transaction has committed. *BUT* it could be
945470decc6SDave Kleikamp 			 * reallocated once we have written all the data to
946470decc6SDave Kleikamp 			 * disk and before we process the buffer on BJ_Forget
947470decc6SDave Kleikamp 			 * list. */
948470decc6SDave Kleikamp 			JBUFFER_TRACE(jh, "refile or unfile freed buffer");
949f7f4bccbSMingming Cao 			__jbd2_journal_refile_buffer(jh);
950470decc6SDave Kleikamp 			if (!jh->b_transaction) {
951470decc6SDave Kleikamp 				jbd_unlock_bh_state(bh);
952470decc6SDave Kleikamp 				 /* needs a brelse */
953f7f4bccbSMingming Cao 				jbd2_journal_remove_journal_head(bh);
954470decc6SDave Kleikamp 				release_buffer_page(bh);
955470decc6SDave Kleikamp 			} else
956470decc6SDave Kleikamp 				jbd_unlock_bh_state(bh);
957470decc6SDave Kleikamp 		}
958470decc6SDave Kleikamp 		cond_resched_lock(&journal->j_list_lock);
959470decc6SDave Kleikamp 	}
960470decc6SDave Kleikamp 	spin_unlock(&journal->j_list_lock);
961470decc6SDave Kleikamp 	/*
962f5a7a6b0SJan Kara 	 * This is a bit sleazy.  We use j_list_lock to protect transition
963f5a7a6b0SJan Kara 	 * of a transaction into T_FINISHED state and calling
964f5a7a6b0SJan Kara 	 * __jbd2_journal_drop_transaction(). Otherwise we could race with
965f5a7a6b0SJan Kara 	 * other checkpointing code processing the transaction...
966470decc6SDave Kleikamp 	 */
967470decc6SDave Kleikamp 	spin_lock(&journal->j_state_lock);
968470decc6SDave Kleikamp 	spin_lock(&journal->j_list_lock);
969470decc6SDave Kleikamp 	/*
970470decc6SDave Kleikamp 	 * Now recheck if some buffers did not get attached to the transaction
971470decc6SDave Kleikamp 	 * while the lock was dropped...
972470decc6SDave Kleikamp 	 */
973470decc6SDave Kleikamp 	if (commit_transaction->t_forget) {
974470decc6SDave Kleikamp 		spin_unlock(&journal->j_list_lock);
975470decc6SDave Kleikamp 		spin_unlock(&journal->j_state_lock);
976470decc6SDave Kleikamp 		goto restart_loop;
977470decc6SDave Kleikamp 	}
978470decc6SDave Kleikamp 
979470decc6SDave Kleikamp 	/* Done with this transaction! */
980470decc6SDave Kleikamp 
98187c89c23SJan Kara 	jbd_debug(3, "JBD: commit phase 7\n");
982470decc6SDave Kleikamp 
983470decc6SDave Kleikamp 	J_ASSERT(commit_transaction->t_state == T_COMMIT);
984470decc6SDave Kleikamp 
9858e85fb3fSJohann Lombardi 	commit_transaction->t_start = jiffies;
9868e85fb3fSJohann Lombardi 	stats.u.run.rs_logging = jbd2_time_diff(stats.u.run.rs_logging,
9878e85fb3fSJohann Lombardi 						commit_transaction->t_start);
9888e85fb3fSJohann Lombardi 
9898e85fb3fSJohann Lombardi 	/*
9908e85fb3fSJohann Lombardi 	 * File the transaction for history
9918e85fb3fSJohann Lombardi 	 */
9928e85fb3fSJohann Lombardi 	stats.ts_type = JBD2_STATS_RUN;
9938e85fb3fSJohann Lombardi 	stats.ts_tid = commit_transaction->t_tid;
9948e85fb3fSJohann Lombardi 	stats.u.run.rs_handle_count = commit_transaction->t_handle_count;
9958e85fb3fSJohann Lombardi 	spin_lock(&journal->j_history_lock);
9968e85fb3fSJohann Lombardi 	memcpy(journal->j_history + journal->j_history_cur, &stats,
9978e85fb3fSJohann Lombardi 			sizeof(stats));
9988e85fb3fSJohann Lombardi 	if (++journal->j_history_cur == journal->j_history_max)
9998e85fb3fSJohann Lombardi 		journal->j_history_cur = 0;
10008e85fb3fSJohann Lombardi 
10018e85fb3fSJohann Lombardi 	/*
10028e85fb3fSJohann Lombardi 	 * Calculate overall stats
10038e85fb3fSJohann Lombardi 	 */
10048e85fb3fSJohann Lombardi 	journal->j_stats.ts_tid++;
10058e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_wait += stats.u.run.rs_wait;
10068e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_running += stats.u.run.rs_running;
10078e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_locked += stats.u.run.rs_locked;
10088e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_flushing += stats.u.run.rs_flushing;
10098e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_logging += stats.u.run.rs_logging;
10108e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_handle_count += stats.u.run.rs_handle_count;
10118e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_blocks += stats.u.run.rs_blocks;
10128e85fb3fSJohann Lombardi 	journal->j_stats.u.run.rs_blocks_logged += stats.u.run.rs_blocks_logged;
10138e85fb3fSJohann Lombardi 	spin_unlock(&journal->j_history_lock);
10148e85fb3fSJohann Lombardi 
1015470decc6SDave Kleikamp 	commit_transaction->t_state = T_FINISHED;
1016470decc6SDave Kleikamp 	J_ASSERT(commit_transaction == journal->j_committing_transaction);
1017470decc6SDave Kleikamp 	journal->j_commit_sequence = commit_transaction->t_tid;
1018470decc6SDave Kleikamp 	journal->j_committing_transaction = NULL;
1019e07f7183SJosef Bacik 	commit_time = ktime_to_ns(ktime_sub(ktime_get(), start_time));
1020470decc6SDave Kleikamp 
1021e07f7183SJosef Bacik 	/*
1022e07f7183SJosef Bacik 	 * weight the commit time higher than the average time so we don't
1023e07f7183SJosef Bacik 	 * react too strongly to vast changes in the commit time
1024e07f7183SJosef Bacik 	 */
1025e07f7183SJosef Bacik 	if (likely(journal->j_average_commit_time))
1026e07f7183SJosef Bacik 		journal->j_average_commit_time = (commit_time +
1027e07f7183SJosef Bacik 				journal->j_average_commit_time*3) / 4;
1028e07f7183SJosef Bacik 	else
1029e07f7183SJosef Bacik 		journal->j_average_commit_time = commit_time;
1030470decc6SDave Kleikamp 	spin_unlock(&journal->j_state_lock);
10316c20ec85STheodore Ts'o 
1032f89b7795SJan Kara 	if (commit_transaction->t_checkpoint_list == NULL &&
1033f89b7795SJan Kara 	    commit_transaction->t_checkpoint_io_list == NULL) {
1034f7f4bccbSMingming Cao 		__jbd2_journal_drop_transaction(journal, commit_transaction);
1035fb68407bSAneesh Kumar K.V 		to_free = 1;
1036470decc6SDave Kleikamp 	} else {
1037470decc6SDave Kleikamp 		if (journal->j_checkpoint_transactions == NULL) {
1038470decc6SDave Kleikamp 			journal->j_checkpoint_transactions = commit_transaction;
1039470decc6SDave Kleikamp 			commit_transaction->t_cpnext = commit_transaction;
1040470decc6SDave Kleikamp 			commit_transaction->t_cpprev = commit_transaction;
1041470decc6SDave Kleikamp 		} else {
1042470decc6SDave Kleikamp 			commit_transaction->t_cpnext =
1043470decc6SDave Kleikamp 				journal->j_checkpoint_transactions;
1044470decc6SDave Kleikamp 			commit_transaction->t_cpprev =
1045470decc6SDave Kleikamp 				commit_transaction->t_cpnext->t_cpprev;
1046470decc6SDave Kleikamp 			commit_transaction->t_cpnext->t_cpprev =
1047470decc6SDave Kleikamp 				commit_transaction;
1048470decc6SDave Kleikamp 			commit_transaction->t_cpprev->t_cpnext =
1049470decc6SDave Kleikamp 				commit_transaction;
1050470decc6SDave Kleikamp 		}
1051470decc6SDave Kleikamp 	}
1052470decc6SDave Kleikamp 	spin_unlock(&journal->j_list_lock);
1053470decc6SDave Kleikamp 
1054fb68407bSAneesh Kumar K.V 	if (journal->j_commit_callback)
1055fb68407bSAneesh Kumar K.V 		journal->j_commit_callback(journal, commit_transaction);
1056fb68407bSAneesh Kumar K.V 
1057ede86cc4STheodore Ts'o 	trace_mark(jbd2_end_commit, "dev %s transaction %d head %d",
1058fb68407bSAneesh Kumar K.V 		   journal->j_devname, commit_transaction->t_tid,
1059ede86cc4STheodore Ts'o 		   journal->j_tail_sequence);
1060470decc6SDave Kleikamp 	jbd_debug(1, "JBD: commit %d complete, head %d\n",
1061470decc6SDave Kleikamp 		  journal->j_commit_sequence, journal->j_tail_sequence);
1062fb68407bSAneesh Kumar K.V 	if (to_free)
1063fb68407bSAneesh Kumar K.V 		kfree(commit_transaction);
1064470decc6SDave Kleikamp 
1065470decc6SDave Kleikamp 	wake_up(&journal->j_wait_done_commit);
1066470decc6SDave Kleikamp }
1067