Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1 | /* |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 2 | * linux/fs/jbd2/commit.c |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 3 | * |
| 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 Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 18 | #include <linux/jbd2.h> |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 19 | #include <linux/errno.h> |
| 20 | #include <linux/slab.h> |
| 21 | #include <linux/mm.h> |
| 22 | #include <linux/pagemap.h> |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 23 | #include <linux/jiffies.h> |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 24 | #include <linux/crc32.h> |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 25 | #include <linux/writeback.h> |
| 26 | #include <linux/backing-dev.h> |
Theodore Ts'o | fd98496 | 2009-01-05 21:34:13 -0500 | [diff] [blame] | 27 | #include <linux/bio.h> |
Theodore Ts'o | 0e3d2a6 | 2009-09-11 09:30:12 -0400 | [diff] [blame] | 28 | #include <linux/blkdev.h> |
Brian King | 39e3ac2 | 2010-10-27 21:25:12 -0400 | [diff] [blame] | 29 | #include <linux/bitops.h> |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 30 | #include <trace/events/jbd2.h> |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 31 | |
| 32 | /* |
| 33 | * Default IO end handler for temporary BJ_IO buffer_heads. |
| 34 | */ |
| 35 | static void journal_end_buffer_io_sync(struct buffer_head *bh, int uptodate) |
| 36 | { |
| 37 | BUFFER_TRACE(bh, ""); |
| 38 | if (uptodate) |
| 39 | set_buffer_uptodate(bh); |
| 40 | else |
| 41 | clear_buffer_uptodate(bh); |
| 42 | unlock_buffer(bh); |
| 43 | } |
| 44 | |
| 45 | /* |
Jan Kara | 87c89c2 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 46 | * When an ext4 file is truncated, it is possible that some pages are not |
| 47 | * successfully freed, because they are attached to a committing transaction. |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 48 | * After the transaction commits, these pages are left on the LRU, with no |
| 49 | * ->mapping, and with attached buffers. These pages are trivially reclaimable |
| 50 | * by the VM, but their apparent absence upsets the VM accounting, and it makes |
| 51 | * the numbers in /proc/meminfo look odd. |
| 52 | * |
| 53 | * So here, we have a buffer which has just come off the forget list. Look to |
| 54 | * see if we can strip all buffers from the backing page. |
| 55 | * |
| 56 | * Called under lock_journal(), and possibly under journal_datalist_lock. The |
| 57 | * caller provided us with a ref against the buffer, and we drop that here. |
| 58 | */ |
| 59 | static void release_buffer_page(struct buffer_head *bh) |
| 60 | { |
| 61 | struct page *page; |
| 62 | |
| 63 | if (buffer_dirty(bh)) |
| 64 | goto nope; |
| 65 | if (atomic_read(&bh->b_count) != 1) |
| 66 | goto nope; |
| 67 | page = bh->b_page; |
| 68 | if (!page) |
| 69 | goto nope; |
| 70 | if (page->mapping) |
| 71 | goto nope; |
| 72 | |
| 73 | /* OK, it's a truncated page */ |
Nick Piggin | 529ae9a | 2008-08-02 12:01:03 +0200 | [diff] [blame] | 74 | if (!trylock_page(page)) |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 75 | goto nope; |
| 76 | |
| 77 | page_cache_get(page); |
| 78 | __brelse(bh); |
| 79 | try_to_free_buffers(page); |
| 80 | unlock_page(page); |
| 81 | page_cache_release(page); |
| 82 | return; |
| 83 | |
| 84 | nope: |
| 85 | __brelse(bh); |
| 86 | } |
| 87 | |
Darrick J. Wong | 1f56c58 | 2012-05-27 08:10:25 -0400 | [diff] [blame] | 88 | static void jbd2_commit_block_csum_set(journal_t *j, |
| 89 | struct journal_head *descriptor) |
| 90 | { |
| 91 | struct commit_header *h; |
| 92 | __u32 csum; |
| 93 | |
| 94 | if (!JBD2_HAS_INCOMPAT_FEATURE(j, JBD2_FEATURE_INCOMPAT_CSUM_V2)) |
| 95 | return; |
| 96 | |
| 97 | h = (struct commit_header *)(jh2bh(descriptor)->b_data); |
| 98 | h->h_chksum_type = 0; |
| 99 | h->h_chksum_size = 0; |
| 100 | h->h_chksum[0] = 0; |
| 101 | csum = jbd2_chksum(j, j->j_csum_seed, jh2bh(descriptor)->b_data, |
| 102 | j->j_blocksize); |
| 103 | h->h_chksum[0] = cpu_to_be32(csum); |
| 104 | } |
| 105 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 106 | /* |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 107 | * Done it all: now submit the commit record. We should have |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 108 | * cleaned up our previous buffers by now, so if we are in abort |
| 109 | * mode we can now just skip the rest of the journal write |
| 110 | * entirely. |
| 111 | * |
| 112 | * Returns 1 if the journal needs to be aborted or 0 on success |
| 113 | */ |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 114 | static int journal_submit_commit_record(journal_t *journal, |
| 115 | transaction_t *commit_transaction, |
| 116 | struct buffer_head **cbh, |
| 117 | __u32 crc32_sum) |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 118 | { |
| 119 | struct journal_head *descriptor; |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 120 | struct commit_header *tmp; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 121 | struct buffer_head *bh; |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 122 | int ret; |
Theodore Ts'o | 736603a | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 123 | struct timespec now = current_kernel_time(); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 124 | |
Zhang Huan | 6cba611 | 2011-04-05 19:16:20 -0400 | [diff] [blame] | 125 | *cbh = NULL; |
| 126 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 127 | if (is_journal_aborted(journal)) |
| 128 | return 0; |
| 129 | |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 130 | descriptor = jbd2_journal_get_descriptor_buffer(journal); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 131 | if (!descriptor) |
| 132 | return 1; |
| 133 | |
| 134 | bh = jh2bh(descriptor); |
| 135 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 136 | tmp = (struct commit_header *)bh->b_data; |
| 137 | tmp->h_magic = cpu_to_be32(JBD2_MAGIC_NUMBER); |
| 138 | tmp->h_blocktype = cpu_to_be32(JBD2_COMMIT_BLOCK); |
| 139 | tmp->h_sequence = cpu_to_be32(commit_transaction->t_tid); |
Theodore Ts'o | 736603a | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 140 | tmp->h_commit_sec = cpu_to_be64(now.tv_sec); |
| 141 | tmp->h_commit_nsec = cpu_to_be32(now.tv_nsec); |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 142 | |
| 143 | if (JBD2_HAS_COMPAT_FEATURE(journal, |
| 144 | JBD2_FEATURE_COMPAT_CHECKSUM)) { |
| 145 | tmp->h_chksum_type = JBD2_CRC32_CHKSUM; |
| 146 | tmp->h_chksum_size = JBD2_CRC32_CHKSUM_SIZE; |
| 147 | tmp->h_chksum[0] = cpu_to_be32(crc32_sum); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 148 | } |
Darrick J. Wong | 1f56c58 | 2012-05-27 08:10:25 -0400 | [diff] [blame] | 149 | jbd2_commit_block_csum_set(journal, descriptor); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 150 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 151 | JBUFFER_TRACE(descriptor, "submit commit block"); |
| 152 | lock_buffer(bh); |
Theodore Ts'o | 45a90bf | 2008-10-06 12:04:02 -0400 | [diff] [blame] | 153 | clear_buffer_dirty(bh); |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 154 | set_buffer_uptodate(bh); |
| 155 | bh->b_end_io = journal_end_buffer_io_sync; |
| 156 | |
| 157 | if (journal->j_flags & JBD2_BARRIER && |
Theodore Ts'o | 0e3d2a6 | 2009-09-11 09:30:12 -0400 | [diff] [blame] | 158 | !JBD2_HAS_INCOMPAT_FEATURE(journal, |
Christoph Hellwig | 9c35575 | 2010-08-18 05:29:17 -0400 | [diff] [blame] | 159 | JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) |
Jens Axboe | 721a960 | 2011-03-09 11:56:30 +0100 | [diff] [blame] | 160 | ret = submit_bh(WRITE_SYNC | WRITE_FLUSH_FUA, bh); |
Christoph Hellwig | 9c35575 | 2010-08-18 05:29:17 -0400 | [diff] [blame] | 161 | else |
Jens Axboe | 721a960 | 2011-03-09 11:56:30 +0100 | [diff] [blame] | 162 | ret = submit_bh(WRITE_SYNC, bh); |
Christoph Hellwig | 9c35575 | 2010-08-18 05:29:17 -0400 | [diff] [blame] | 163 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 164 | *cbh = bh; |
| 165 | return ret; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 166 | } |
| 167 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 168 | /* |
| 169 | * This function along with journal_submit_commit_record |
| 170 | * allows to write the commit record asynchronously. |
| 171 | */ |
Theodore Ts'o | fd98496 | 2009-01-05 21:34:13 -0500 | [diff] [blame] | 172 | static int journal_wait_on_commit_record(journal_t *journal, |
| 173 | struct buffer_head *bh) |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 174 | { |
| 175 | int ret = 0; |
| 176 | |
| 177 | clear_buffer_dirty(bh); |
| 178 | wait_on_buffer(bh); |
| 179 | |
| 180 | if (unlikely(!buffer_uptodate(bh))) |
| 181 | ret = -EIO; |
| 182 | put_bh(bh); /* One for getblk() */ |
| 183 | jbd2_journal_put_journal_head(bh2jh(bh)); |
| 184 | |
| 185 | return ret; |
| 186 | } |
| 187 | |
| 188 | /* |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 189 | * write the filemap data using writepage() address_space_operations. |
| 190 | * We don't do block allocation here even for delalloc. We don't |
| 191 | * use writepages() because with dealyed allocation we may be doing |
| 192 | * block allocation in writepages(). |
| 193 | */ |
| 194 | static int journal_submit_inode_data_buffers(struct address_space *mapping) |
| 195 | { |
| 196 | int ret; |
| 197 | struct writeback_control wbc = { |
| 198 | .sync_mode = WB_SYNC_ALL, |
| 199 | .nr_to_write = mapping->nrpages * 2, |
| 200 | .range_start = 0, |
| 201 | .range_end = i_size_read(mapping->host), |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 202 | }; |
| 203 | |
| 204 | ret = generic_writepages(mapping, &wbc); |
| 205 | return ret; |
| 206 | } |
| 207 | |
| 208 | /* |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 209 | * Submit all the data buffers of inode associated with the transaction to |
| 210 | * disk. |
| 211 | * |
| 212 | * We are in a committing transaction. Therefore no new inode can be added to |
| 213 | * our inode list. We use JI_COMMIT_RUNNING flag to protect inode we currently |
| 214 | * operate on from being released while we write out pages. |
| 215 | */ |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 216 | static int journal_submit_data_buffers(journal_t *journal, |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 217 | transaction_t *commit_transaction) |
| 218 | { |
| 219 | struct jbd2_inode *jinode; |
| 220 | int err, ret = 0; |
| 221 | struct address_space *mapping; |
| 222 | |
| 223 | spin_lock(&journal->j_list_lock); |
| 224 | list_for_each_entry(jinode, &commit_transaction->t_inode_list, i_list) { |
| 225 | mapping = jinode->i_vfs_inode->i_mapping; |
Brian King | 39e3ac2 | 2010-10-27 21:25:12 -0400 | [diff] [blame] | 226 | set_bit(__JI_COMMIT_RUNNING, &jinode->i_flags); |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 227 | spin_unlock(&journal->j_list_lock); |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 228 | /* |
| 229 | * submit the inode data buffers. We use writepage |
| 230 | * instead of writepages. Because writepages can do |
| 231 | * block allocation with delalloc. We need to write |
| 232 | * only allocated blocks here. |
| 233 | */ |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 234 | trace_jbd2_submit_inode_data(jinode->i_vfs_inode); |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 235 | err = journal_submit_inode_data_buffers(mapping); |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 236 | if (!ret) |
| 237 | ret = err; |
| 238 | spin_lock(&journal->j_list_lock); |
| 239 | J_ASSERT(jinode->i_transaction == commit_transaction); |
Brian King | 39e3ac2 | 2010-10-27 21:25:12 -0400 | [diff] [blame] | 240 | clear_bit(__JI_COMMIT_RUNNING, &jinode->i_flags); |
| 241 | smp_mb__after_clear_bit(); |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 242 | wake_up_bit(&jinode->i_flags, __JI_COMMIT_RUNNING); |
| 243 | } |
| 244 | spin_unlock(&journal->j_list_lock); |
| 245 | return ret; |
| 246 | } |
| 247 | |
| 248 | /* |
| 249 | * Wait for data submitted for writeout, refile inodes to proper |
| 250 | * transaction if needed. |
| 251 | * |
| 252 | */ |
| 253 | static int journal_finish_inode_data_buffers(journal_t *journal, |
| 254 | transaction_t *commit_transaction) |
| 255 | { |
| 256 | struct jbd2_inode *jinode, *next_i; |
| 257 | int err, ret = 0; |
| 258 | |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 259 | /* For locking, see the comment in journal_submit_data_buffers() */ |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 260 | spin_lock(&journal->j_list_lock); |
| 261 | list_for_each_entry(jinode, &commit_transaction->t_inode_list, i_list) { |
Brian King | 39e3ac2 | 2010-10-27 21:25:12 -0400 | [diff] [blame] | 262 | set_bit(__JI_COMMIT_RUNNING, &jinode->i_flags); |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 263 | spin_unlock(&journal->j_list_lock); |
| 264 | err = filemap_fdatawait(jinode->i_vfs_inode->i_mapping); |
Hidehiro Kawai | e9e34f4 | 2008-07-31 22:26:04 -0400 | [diff] [blame] | 265 | if (err) { |
| 266 | /* |
| 267 | * Because AS_EIO is cleared by |
Christoph Hellwig | 94004ed | 2009-09-30 22:16:33 +0200 | [diff] [blame] | 268 | * filemap_fdatawait_range(), set it again so |
Hidehiro Kawai | e9e34f4 | 2008-07-31 22:26:04 -0400 | [diff] [blame] | 269 | * that user process can get -EIO from fsync(). |
| 270 | */ |
| 271 | set_bit(AS_EIO, |
| 272 | &jinode->i_vfs_inode->i_mapping->flags); |
| 273 | |
| 274 | if (!ret) |
| 275 | ret = err; |
| 276 | } |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 277 | spin_lock(&journal->j_list_lock); |
Brian King | 39e3ac2 | 2010-10-27 21:25:12 -0400 | [diff] [blame] | 278 | clear_bit(__JI_COMMIT_RUNNING, &jinode->i_flags); |
| 279 | smp_mb__after_clear_bit(); |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 280 | wake_up_bit(&jinode->i_flags, __JI_COMMIT_RUNNING); |
| 281 | } |
| 282 | |
| 283 | /* Now refile inode to proper lists */ |
| 284 | list_for_each_entry_safe(jinode, next_i, |
| 285 | &commit_transaction->t_inode_list, i_list) { |
| 286 | list_del(&jinode->i_list); |
| 287 | if (jinode->i_next_transaction) { |
| 288 | jinode->i_transaction = jinode->i_next_transaction; |
| 289 | jinode->i_next_transaction = NULL; |
| 290 | list_add(&jinode->i_list, |
| 291 | &jinode->i_transaction->t_inode_list); |
| 292 | } else { |
| 293 | jinode->i_transaction = NULL; |
| 294 | } |
| 295 | } |
| 296 | spin_unlock(&journal->j_list_lock); |
| 297 | |
| 298 | return ret; |
| 299 | } |
| 300 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 301 | static __u32 jbd2_checksum_data(__u32 crc32_sum, struct buffer_head *bh) |
| 302 | { |
| 303 | struct page *page = bh->b_page; |
| 304 | char *addr; |
| 305 | __u32 checksum; |
| 306 | |
Cong Wang | 303a8f2 | 2011-11-25 23:14:31 +0800 | [diff] [blame] | 307 | addr = kmap_atomic(page); |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 308 | checksum = crc32_be(crc32_sum, |
| 309 | (void *)(addr + offset_in_page(bh->b_data)), bh->b_size); |
Cong Wang | 303a8f2 | 2011-11-25 23:14:31 +0800 | [diff] [blame] | 310 | kunmap_atomic(addr); |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 311 | |
| 312 | return checksum; |
| 313 | } |
| 314 | |
| 315 | static void write_tag_block(int tag_bytes, journal_block_tag_t *tag, |
Mingming Cao | 18eba7a | 2006-10-11 01:21:13 -0700 | [diff] [blame] | 316 | unsigned long long block) |
Zach Brown | b517bea | 2006-10-11 01:21:08 -0700 | [diff] [blame] | 317 | { |
| 318 | tag->t_blocknr = cpu_to_be32(block & (u32)~0); |
Mingming Cao | cd02ff0 | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 319 | if (tag_bytes > JBD2_TAG_SIZE32) |
Zach Brown | b517bea | 2006-10-11 01:21:08 -0700 | [diff] [blame] | 320 | tag->t_blocknr_high = cpu_to_be32((block >> 31) >> 1); |
| 321 | } |
| 322 | |
Darrick J. Wong | 3caa487 | 2012-05-27 08:10:22 -0400 | [diff] [blame] | 323 | static void jbd2_descr_block_csum_set(journal_t *j, |
| 324 | struct journal_head *descriptor) |
| 325 | { |
| 326 | struct jbd2_journal_block_tail *tail; |
| 327 | __u32 csum; |
| 328 | |
| 329 | if (!JBD2_HAS_INCOMPAT_FEATURE(j, JBD2_FEATURE_INCOMPAT_CSUM_V2)) |
| 330 | return; |
| 331 | |
| 332 | tail = (struct jbd2_journal_block_tail *) |
| 333 | (jh2bh(descriptor)->b_data + j->j_blocksize - |
| 334 | sizeof(struct jbd2_journal_block_tail)); |
| 335 | tail->t_checksum = 0; |
| 336 | csum = jbd2_chksum(j, j->j_csum_seed, jh2bh(descriptor)->b_data, |
| 337 | j->j_blocksize); |
| 338 | tail->t_checksum = cpu_to_be32(csum); |
| 339 | } |
| 340 | |
Darrick J. Wong | c390087 | 2012-05-27 08:12:12 -0400 | [diff] [blame] | 341 | static void jbd2_block_tag_csum_set(journal_t *j, journal_block_tag_t *tag, |
| 342 | struct buffer_head *bh, __u32 sequence) |
| 343 | { |
| 344 | struct page *page = bh->b_page; |
| 345 | __u8 *addr; |
Darrick J. Wong | eee06c5 | 2013-05-28 07:31:59 -0400 | [diff] [blame] | 346 | __u32 csum32; |
Darrick J. Wong | c390087 | 2012-05-27 08:12:12 -0400 | [diff] [blame] | 347 | |
| 348 | if (!JBD2_HAS_INCOMPAT_FEATURE(j, JBD2_FEATURE_INCOMPAT_CSUM_V2)) |
| 349 | return; |
| 350 | |
| 351 | sequence = cpu_to_be32(sequence); |
Cong Wang | 906adea | 2012-06-23 11:24:48 +0800 | [diff] [blame] | 352 | addr = kmap_atomic(page); |
Darrick J. Wong | eee06c5 | 2013-05-28 07:31:59 -0400 | [diff] [blame] | 353 | csum32 = jbd2_chksum(j, j->j_csum_seed, (__u8 *)&sequence, |
| 354 | sizeof(sequence)); |
| 355 | csum32 = jbd2_chksum(j, csum32, addr + offset_in_page(bh->b_data), |
| 356 | bh->b_size); |
Cong Wang | 906adea | 2012-06-23 11:24:48 +0800 | [diff] [blame] | 357 | kunmap_atomic(addr); |
Darrick J. Wong | c390087 | 2012-05-27 08:12:12 -0400 | [diff] [blame] | 358 | |
Darrick J. Wong | eee06c5 | 2013-05-28 07:31:59 -0400 | [diff] [blame] | 359 | /* We only have space to store the lower 16 bits of the crc32c. */ |
| 360 | tag->t_checksum = cpu_to_be16(csum32); |
Darrick J. Wong | c390087 | 2012-05-27 08:12:12 -0400 | [diff] [blame] | 361 | } |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 362 | /* |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 363 | * jbd2_journal_commit_transaction |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 364 | * |
| 365 | * The primary function for committing a transaction to the log. This |
| 366 | * function is called by the journal thread to begin a complete commit. |
| 367 | */ |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 368 | void jbd2_journal_commit_transaction(journal_t *journal) |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 369 | { |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 370 | struct transaction_stats_s stats; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 371 | transaction_t *commit_transaction; |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 372 | struct journal_head *jh, *descriptor; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 373 | struct buffer_head **wbuf = journal->j_wbuf; |
| 374 | int bufs; |
| 375 | int flags; |
| 376 | int err; |
Mingming Cao | 18eba7a | 2006-10-11 01:21:13 -0700 | [diff] [blame] | 377 | unsigned long long blocknr; |
Josef Bacik | e07f718 | 2008-11-26 01:14:26 -0500 | [diff] [blame] | 378 | ktime_t start_time; |
| 379 | u64 commit_time; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 380 | 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 Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 386 | int i; |
Zach Brown | b517bea | 2006-10-11 01:21:08 -0700 | [diff] [blame] | 387 | int tag_bytes = journal_tag_bytes(journal); |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 388 | struct buffer_head *cbh = NULL; /* For transactional checksums */ |
| 389 | __u32 crc32_sum = ~0; |
Jens Axboe | 82f04ab | 2011-03-17 11:01:52 +0100 | [diff] [blame] | 390 | struct blk_plug plug; |
Jan Kara | 3339578 | 2012-03-13 22:45:38 -0400 | [diff] [blame] | 391 | /* Tail of the journal */ |
| 392 | unsigned long first_block; |
| 393 | tid_t first_tid; |
| 394 | int update_tail; |
Darrick J. Wong | 3caa487 | 2012-05-27 08:10:22 -0400 | [diff] [blame] | 395 | int csum_size = 0; |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 396 | LIST_HEAD(io_bufs); |
Darrick J. Wong | 3caa487 | 2012-05-27 08:10:22 -0400 | [diff] [blame] | 397 | |
| 398 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, JBD2_FEATURE_INCOMPAT_CSUM_V2)) |
| 399 | csum_size = sizeof(struct jbd2_journal_block_tail); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 400 | |
| 401 | /* |
| 402 | * First job: lock down the current transaction and wait for |
| 403 | * all outstanding updates to complete. |
| 404 | */ |
| 405 | |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 406 | /* Do we need to erase the effects of a prior jbd2_journal_flush? */ |
| 407 | if (journal->j_flags & JBD2_FLUSHED) { |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 408 | jbd_debug(3, "super block updated\n"); |
Jan Kara | a78bb11 | 2012-03-13 15:43:04 -0400 | [diff] [blame] | 409 | mutex_lock(&journal->j_checkpoint_mutex); |
Jan Kara | 79feb52 | 2012-03-13 22:22:54 -0400 | [diff] [blame] | 410 | /* |
| 411 | * We hold j_checkpoint_mutex so tail cannot change under us. |
| 412 | * We don't need any special data guarantees for writing sb |
| 413 | * since journal is empty and it is ok for write to be |
| 414 | * flushed only with transaction commit. |
| 415 | */ |
| 416 | jbd2_journal_update_sb_log_tail(journal, |
| 417 | journal->j_tail_sequence, |
| 418 | journal->j_tail, |
| 419 | WRITE_SYNC); |
Jan Kara | a78bb11 | 2012-03-13 15:43:04 -0400 | [diff] [blame] | 420 | mutex_unlock(&journal->j_checkpoint_mutex); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 421 | } else { |
| 422 | jbd_debug(3, "superblock not updated\n"); |
| 423 | } |
| 424 | |
| 425 | J_ASSERT(journal->j_running_transaction != NULL); |
| 426 | J_ASSERT(journal->j_committing_transaction == NULL); |
| 427 | |
| 428 | commit_transaction = journal->j_running_transaction; |
| 429 | J_ASSERT(commit_transaction->t_state == T_RUNNING); |
| 430 | |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 431 | trace_jbd2_start_commit(journal, commit_transaction); |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 432 | jbd_debug(1, "JBD2: starting commit of transaction %d\n", |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 433 | commit_transaction->t_tid); |
| 434 | |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 435 | write_lock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 436 | commit_transaction->t_state = T_LOCKED; |
| 437 | |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 438 | trace_jbd2_commit_locking(journal, commit_transaction); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 439 | stats.run.rs_wait = commit_transaction->t_max_wait; |
Theodore Ts'o | 9fff24a | 2013-02-06 22:30:23 -0500 | [diff] [blame] | 440 | stats.run.rs_request_delay = 0; |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 441 | stats.run.rs_locked = jiffies; |
Theodore Ts'o | 9fff24a | 2013-02-06 22:30:23 -0500 | [diff] [blame] | 442 | if (commit_transaction->t_requested) |
| 443 | stats.run.rs_request_delay = |
| 444 | jbd2_time_diff(commit_transaction->t_requested, |
| 445 | stats.run.rs_locked); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 446 | stats.run.rs_running = jbd2_time_diff(commit_transaction->t_start, |
| 447 | stats.run.rs_locked); |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 448 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 449 | spin_lock(&commit_transaction->t_handle_lock); |
Theodore Ts'o | a51dca9 | 2010-08-02 08:43:25 -0400 | [diff] [blame] | 450 | while (atomic_read(&commit_transaction->t_updates)) { |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 451 | DEFINE_WAIT(wait); |
| 452 | |
| 453 | prepare_to_wait(&journal->j_wait_updates, &wait, |
| 454 | TASK_UNINTERRUPTIBLE); |
Theodore Ts'o | a51dca9 | 2010-08-02 08:43:25 -0400 | [diff] [blame] | 455 | if (atomic_read(&commit_transaction->t_updates)) { |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 456 | spin_unlock(&commit_transaction->t_handle_lock); |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 457 | write_unlock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 458 | schedule(); |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 459 | write_lock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 460 | spin_lock(&commit_transaction->t_handle_lock); |
| 461 | } |
| 462 | finish_wait(&journal->j_wait_updates, &wait); |
| 463 | } |
| 464 | spin_unlock(&commit_transaction->t_handle_lock); |
| 465 | |
Theodore Ts'o | a51dca9 | 2010-08-02 08:43:25 -0400 | [diff] [blame] | 466 | J_ASSERT (atomic_read(&commit_transaction->t_outstanding_credits) <= |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 467 | journal->j_max_transaction_buffers); |
| 468 | |
| 469 | /* |
| 470 | * First thing we are allowed to do is to discard any remaining |
| 471 | * BJ_Reserved buffers. Note, it is _not_ permissible to assume |
| 472 | * that there are no such buffers: if a large filesystem |
| 473 | * operation like a truncate needs to split itself over multiple |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 474 | * transactions, then it may try to do a jbd2_journal_restart() while |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 475 | * there are still BJ_Reserved buffers outstanding. These must |
| 476 | * be released cleanly from the current transaction. |
| 477 | * |
| 478 | * In this case, the filesystem must still reserve write access |
| 479 | * again before modifying the buffer in the new transaction, but |
| 480 | * we do not require it to remember exactly which old buffers it |
| 481 | * has reserved. This is consistent with the existing behaviour |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 482 | * that multiple jbd2_journal_get_write_access() calls to the same |
Lucas De Marchi | 25985ed | 2011-03-30 22:57:33 -0300 | [diff] [blame] | 483 | * buffer are perfectly permissible. |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 484 | */ |
| 485 | while (commit_transaction->t_reserved_list) { |
| 486 | jh = commit_transaction->t_reserved_list; |
| 487 | JBUFFER_TRACE(jh, "reserved, unused: refile"); |
| 488 | /* |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 489 | * A jbd2_journal_get_undo_access()+jbd2_journal_release_buffer() may |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 490 | * leave undo-committed data. |
| 491 | */ |
| 492 | if (jh->b_committed_data) { |
| 493 | struct buffer_head *bh = jh2bh(jh); |
| 494 | |
| 495 | jbd_lock_bh_state(bh); |
Mingming Cao | af1e76d | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 496 | jbd2_free(jh->b_committed_data, bh->b_size); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 497 | jh->b_committed_data = NULL; |
| 498 | jbd_unlock_bh_state(bh); |
| 499 | } |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 500 | jbd2_journal_refile_buffer(journal, jh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 501 | } |
| 502 | |
| 503 | /* |
| 504 | * Now try to drop any written-back buffers from the journal's |
| 505 | * checkpoint lists. We do this *before* commit because it potentially |
| 506 | * frees some memory |
| 507 | */ |
| 508 | spin_lock(&journal->j_list_lock); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 509 | __jbd2_journal_clean_checkpoint_list(journal); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 510 | spin_unlock(&journal->j_list_lock); |
| 511 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 512 | jbd_debug(3, "JBD2: commit phase 1\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 513 | |
| 514 | /* |
Yongqiang Yang | 1ba3726 | 2011-12-28 17:46:46 -0500 | [diff] [blame] | 515 | * Clear revoked flag to reflect there is no revoked buffers |
| 516 | * in the next transaction which is going to be started. |
| 517 | */ |
| 518 | jbd2_clear_buffer_revoked_flags(journal); |
| 519 | |
| 520 | /* |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 521 | * Switch to a new revoke table. |
| 522 | */ |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 523 | jbd2_journal_switch_revoke_table(journal); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 524 | |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 525 | trace_jbd2_commit_flushing(journal, commit_transaction); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 526 | stats.run.rs_flushing = jiffies; |
| 527 | stats.run.rs_locked = jbd2_time_diff(stats.run.rs_locked, |
| 528 | stats.run.rs_flushing); |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 529 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 530 | commit_transaction->t_state = T_FLUSH; |
| 531 | journal->j_committing_transaction = commit_transaction; |
| 532 | journal->j_running_transaction = NULL; |
Josef Bacik | e07f718 | 2008-11-26 01:14:26 -0500 | [diff] [blame] | 533 | start_time = ktime_get(); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 534 | commit_transaction->t_log_start = journal->j_head; |
| 535 | wake_up(&journal->j_wait_transaction_locked); |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 536 | write_unlock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 537 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 538 | jbd_debug(3, "JBD2: commit phase 2\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 539 | |
| 540 | /* |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 541 | * Now start flushing things to disk, in the order they appear |
| 542 | * on the transaction lists. Data blocks go first. |
| 543 | */ |
Aneesh Kumar K.V | cd1aac3 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 544 | err = journal_submit_data_buffers(journal, commit_transaction); |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 545 | if (err) |
| 546 | jbd2_journal_abort(journal, err); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 547 | |
Jens Axboe | 82f04ab | 2011-03-17 11:01:52 +0100 | [diff] [blame] | 548 | blk_start_plug(&plug); |
Theodore Ts'o | 67c457a | 2009-04-14 07:50:56 -0400 | [diff] [blame] | 549 | jbd2_journal_write_revoke_records(journal, commit_transaction, |
Jens Axboe | 82f04ab | 2011-03-17 11:01:52 +0100 | [diff] [blame] | 550 | WRITE_SYNC); |
| 551 | blk_finish_plug(&plug); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 552 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 553 | jbd_debug(3, "JBD2: commit phase 2\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 554 | |
| 555 | /* |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 556 | * Way to go: we have now written out all of the data for a |
| 557 | * transaction! Now comes the tricky part: we need to write out |
| 558 | * metadata. Loop over the transaction's entire buffer list: |
| 559 | */ |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 560 | write_lock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 561 | commit_transaction->t_state = T_COMMIT; |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 562 | write_unlock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 563 | |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 564 | trace_jbd2_commit_logging(journal, commit_transaction); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 565 | stats.run.rs_logging = jiffies; |
| 566 | stats.run.rs_flushing = jbd2_time_diff(stats.run.rs_flushing, |
| 567 | stats.run.rs_logging); |
Theodore Ts'o | a51dca9 | 2010-08-02 08:43:25 -0400 | [diff] [blame] | 568 | stats.run.rs_blocks = |
| 569 | atomic_read(&commit_transaction->t_outstanding_credits); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 570 | stats.run.rs_blocks_logged = 0; |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 571 | |
Josef Bacik | 1dfc322 | 2008-04-17 10:38:59 -0400 | [diff] [blame] | 572 | J_ASSERT(commit_transaction->t_nr_buffers <= |
Theodore Ts'o | a51dca9 | 2010-08-02 08:43:25 -0400 | [diff] [blame] | 573 | atomic_read(&commit_transaction->t_outstanding_credits)); |
Josef Bacik | 1dfc322 | 2008-04-17 10:38:59 -0400 | [diff] [blame] | 574 | |
Jan Kara | 87c89c2 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 575 | err = 0; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 576 | descriptor = NULL; |
| 577 | bufs = 0; |
Jens Axboe | 82f04ab | 2011-03-17 11:01:52 +0100 | [diff] [blame] | 578 | blk_start_plug(&plug); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 579 | while (commit_transaction->t_buffers) { |
| 580 | |
| 581 | /* Find the next buffer to be journaled... */ |
| 582 | |
| 583 | jh = commit_transaction->t_buffers; |
| 584 | |
| 585 | /* If we're in abort mode, we just un-journal the buffer and |
Hidehiro Kawai | 7ad7445 | 2008-10-10 20:29:31 -0400 | [diff] [blame] | 586 | release it. */ |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 587 | |
| 588 | if (is_journal_aborted(journal)) { |
Hidehiro Kawai | 7ad7445 | 2008-10-10 20:29:31 -0400 | [diff] [blame] | 589 | clear_buffer_jbddirty(jh2bh(jh)); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 590 | JBUFFER_TRACE(jh, "journal is aborting: refile"); |
Joel Becker | e06c822 | 2008-09-11 15:35:47 -0700 | [diff] [blame] | 591 | jbd2_buffer_abort_trigger(jh, |
| 592 | jh->b_frozen_data ? |
| 593 | jh->b_frozen_triggers : |
| 594 | jh->b_triggers); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 595 | jbd2_journal_refile_buffer(journal, jh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 596 | /* If that was the last one, we need to clean up |
| 597 | * any descriptor buffers which may have been |
| 598 | * already allocated, even if we are now |
| 599 | * aborting. */ |
| 600 | if (!commit_transaction->t_buffers) |
| 601 | goto start_journal_io; |
| 602 | continue; |
| 603 | } |
| 604 | |
| 605 | /* Make sure we have a descriptor block in which to |
| 606 | record the metadata buffer. */ |
| 607 | |
| 608 | if (!descriptor) { |
| 609 | struct buffer_head *bh; |
| 610 | |
| 611 | J_ASSERT (bufs == 0); |
| 612 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 613 | jbd_debug(4, "JBD2: get descriptor\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 614 | |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 615 | descriptor = jbd2_journal_get_descriptor_buffer(journal); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 616 | if (!descriptor) { |
Jan Kara | a7fa2ba | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 617 | jbd2_journal_abort(journal, -EIO); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 618 | continue; |
| 619 | } |
| 620 | |
| 621 | bh = jh2bh(descriptor); |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 622 | jbd_debug(4, "JBD2: got buffer %llu (%p)\n", |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 623 | (unsigned long long)bh->b_blocknr, bh->b_data); |
| 624 | header = (journal_header_t *)&bh->b_data[0]; |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 625 | header->h_magic = cpu_to_be32(JBD2_MAGIC_NUMBER); |
| 626 | header->h_blocktype = cpu_to_be32(JBD2_DESCRIPTOR_BLOCK); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 627 | header->h_sequence = cpu_to_be32(commit_transaction->t_tid); |
| 628 | |
| 629 | tagp = &bh->b_data[sizeof(journal_header_t)]; |
| 630 | space_left = bh->b_size - sizeof(journal_header_t); |
| 631 | first_tag = 1; |
| 632 | set_buffer_jwrite(bh); |
| 633 | set_buffer_dirty(bh); |
| 634 | wbuf[bufs++] = bh; |
| 635 | |
| 636 | /* Record it so that we can wait for IO |
| 637 | completion later */ |
| 638 | BUFFER_TRACE(bh, "ph3: file as descriptor"); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 639 | jbd2_journal_file_buffer(descriptor, commit_transaction, |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 640 | BJ_LogCtl); |
| 641 | } |
| 642 | |
| 643 | /* Where is the buffer to be written? */ |
| 644 | |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 645 | err = jbd2_journal_next_log_block(journal, &blocknr); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 646 | /* If the block mapping failed, just abandon the buffer |
| 647 | and repeat this loop: we'll fall into the |
| 648 | refile-on-abort condition above. */ |
| 649 | if (err) { |
Jan Kara | a7fa2ba | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 650 | jbd2_journal_abort(journal, err); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 651 | continue; |
| 652 | } |
| 653 | |
| 654 | /* |
| 655 | * start_this_handle() uses t_outstanding_credits to determine |
| 656 | * the free space in the log, but this counter is changed |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 657 | * by jbd2_journal_next_log_block() also. |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 658 | */ |
Theodore Ts'o | a51dca9 | 2010-08-02 08:43:25 -0400 | [diff] [blame] | 659 | atomic_dec(&commit_transaction->t_outstanding_credits); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 660 | |
| 661 | /* Bump b_count to prevent truncate from stumbling over |
| 662 | the shadowed buffer! @@@ This can go if we ever get |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 663 | rid of the shadow pairing of buffers. */ |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 664 | atomic_inc(&jh2bh(jh)->b_count); |
| 665 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 666 | /* |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 667 | * Make a temporary IO buffer with which to write it out |
| 668 | * (this will requeue the metadata buffer to BJ_Shadow). |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 669 | */ |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 670 | set_bit(BH_JWrite, &jh2bh(jh)->b_state); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 671 | JBUFFER_TRACE(jh, "ph3: write metadata"); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 672 | flags = jbd2_journal_write_metadata_buffer(commit_transaction, |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 673 | jh, &wbuf[bufs], blocknr); |
Theodore Ts'o | e6ec116 | 2009-12-01 09:04:42 -0500 | [diff] [blame] | 674 | if (flags < 0) { |
| 675 | jbd2_journal_abort(journal, flags); |
| 676 | continue; |
| 677 | } |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 678 | jbd2_file_log_bh(&io_bufs, wbuf[bufs]); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 679 | |
| 680 | /* Record the new block's tag in the current descriptor |
| 681 | buffer */ |
| 682 | |
| 683 | tag_flag = 0; |
| 684 | if (flags & 1) |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 685 | tag_flag |= JBD2_FLAG_ESCAPE; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 686 | if (!first_tag) |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 687 | tag_flag |= JBD2_FLAG_SAME_UUID; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 688 | |
| 689 | tag = (journal_block_tag_t *) tagp; |
Zach Brown | b517bea | 2006-10-11 01:21:08 -0700 | [diff] [blame] | 690 | write_tag_block(tag_bytes, tag, jh2bh(jh)->b_blocknr); |
Darrick J. Wong | 8f888ef | 2012-05-22 22:43:41 -0400 | [diff] [blame] | 691 | tag->t_flags = cpu_to_be16(tag_flag); |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 692 | jbd2_block_tag_csum_set(journal, tag, wbuf[bufs], |
Darrick J. Wong | c390087 | 2012-05-27 08:12:12 -0400 | [diff] [blame] | 693 | commit_transaction->t_tid); |
Zach Brown | b517bea | 2006-10-11 01:21:08 -0700 | [diff] [blame] | 694 | tagp += tag_bytes; |
| 695 | space_left -= tag_bytes; |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 696 | bufs++; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 697 | |
| 698 | if (first_tag) { |
| 699 | memcpy (tagp, journal->j_uuid, 16); |
| 700 | tagp += 16; |
| 701 | space_left -= 16; |
| 702 | first_tag = 0; |
| 703 | } |
| 704 | |
| 705 | /* If there's no more to do, or if the descriptor is full, |
| 706 | let the IO rip! */ |
| 707 | |
| 708 | if (bufs == journal->j_wbufsize || |
| 709 | commit_transaction->t_buffers == NULL || |
Darrick J. Wong | 3caa487 | 2012-05-27 08:10:22 -0400 | [diff] [blame] | 710 | space_left < tag_bytes + 16 + csum_size) { |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 711 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 712 | jbd_debug(4, "JBD2: Submit %d IOs\n", bufs); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 713 | |
| 714 | /* Write an end-of-descriptor marker before |
| 715 | submitting the IOs. "tag" still points to |
| 716 | the last tag we set up. */ |
| 717 | |
Darrick J. Wong | 8f888ef | 2012-05-22 22:43:41 -0400 | [diff] [blame] | 718 | tag->t_flags |= cpu_to_be16(JBD2_FLAG_LAST_TAG); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 719 | |
Darrick J. Wong | 3caa487 | 2012-05-27 08:10:22 -0400 | [diff] [blame] | 720 | jbd2_descr_block_csum_set(journal, descriptor); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 721 | start_journal_io: |
| 722 | for (i = 0; i < bufs; i++) { |
| 723 | struct buffer_head *bh = wbuf[i]; |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 724 | /* |
| 725 | * Compute checksum. |
| 726 | */ |
| 727 | if (JBD2_HAS_COMPAT_FEATURE(journal, |
| 728 | JBD2_FEATURE_COMPAT_CHECKSUM)) { |
| 729 | crc32_sum = |
| 730 | jbd2_checksum_data(crc32_sum, bh); |
| 731 | } |
| 732 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 733 | lock_buffer(bh); |
| 734 | clear_buffer_dirty(bh); |
| 735 | set_buffer_uptodate(bh); |
| 736 | bh->b_end_io = journal_end_buffer_io_sync; |
Jens Axboe | 82f04ab | 2011-03-17 11:01:52 +0100 | [diff] [blame] | 737 | submit_bh(WRITE_SYNC, bh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 738 | } |
| 739 | cond_resched(); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 740 | stats.run.rs_blocks_logged += bufs; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 741 | |
| 742 | /* Force a new descriptor to be generated next |
| 743 | time round the loop. */ |
| 744 | descriptor = NULL; |
| 745 | bufs = 0; |
| 746 | } |
| 747 | } |
| 748 | |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 749 | err = journal_finish_inode_data_buffers(journal, commit_transaction); |
Hidehiro Kawai | e9e34f4 | 2008-07-31 22:26:04 -0400 | [diff] [blame] | 750 | if (err) { |
Hidehiro Kawai | e9e34f4 | 2008-07-31 22:26:04 -0400 | [diff] [blame] | 751 | printk(KERN_WARNING |
| 752 | "JBD2: Detected IO errors while flushing file data " |
Theodore Ts'o | 0549676 | 2008-09-16 14:36:17 -0400 | [diff] [blame] | 753 | "on %s\n", journal->j_devname); |
Hidehiro Kawai | 5bf5683 | 2008-10-10 22:12:43 -0400 | [diff] [blame] | 754 | if (journal->j_flags & JBD2_ABORT_ON_SYNCDATA_ERR) |
| 755 | jbd2_journal_abort(journal, err); |
Hidehiro Kawai | e9e34f4 | 2008-07-31 22:26:04 -0400 | [diff] [blame] | 756 | err = 0; |
| 757 | } |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 758 | |
Jan Kara | 3339578 | 2012-03-13 22:45:38 -0400 | [diff] [blame] | 759 | /* |
| 760 | * Get current oldest transaction in the log before we issue flush |
| 761 | * to the filesystem device. After the flush we can be sure that |
| 762 | * blocks of all older transactions are checkpointed to persistent |
| 763 | * storage and we will be safe to update journal start in the |
| 764 | * superblock with the numbers we get here. |
| 765 | */ |
| 766 | update_tail = |
| 767 | jbd2_journal_get_log_tail(journal, &first_tid, &first_block); |
| 768 | |
Jan Kara | bbd2be3 | 2011-05-24 11:59:18 -0400 | [diff] [blame] | 769 | write_lock(&journal->j_state_lock); |
Jan Kara | 3339578 | 2012-03-13 22:45:38 -0400 | [diff] [blame] | 770 | if (update_tail) { |
| 771 | long freed = first_block - journal->j_tail; |
| 772 | |
| 773 | if (first_block < journal->j_tail) |
| 774 | freed += journal->j_last - journal->j_first; |
| 775 | /* Update tail only if we free significant amount of space */ |
| 776 | if (freed < journal->j_maxlen / 4) |
| 777 | update_tail = 0; |
| 778 | } |
Jan Kara | bbd2be3 | 2011-05-24 11:59:18 -0400 | [diff] [blame] | 779 | J_ASSERT(commit_transaction->t_state == T_COMMIT); |
| 780 | commit_transaction->t_state = T_COMMIT_DFLUSH; |
| 781 | write_unlock(&journal->j_state_lock); |
Jan Kara | 3339578 | 2012-03-13 22:45:38 -0400 | [diff] [blame] | 782 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 783 | /* |
| 784 | * If the journal is not located on the file system device, |
| 785 | * then we must flush the file system device before we issue |
| 786 | * the commit record |
| 787 | */ |
Jan Kara | 81be12c | 2011-05-24 11:52:40 -0400 | [diff] [blame] | 788 | if (commit_transaction->t_need_data_flush && |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 789 | (journal->j_fs_dev != journal->j_dev) && |
| 790 | (journal->j_flags & JBD2_BARRIER)) |
Shaohua Li | 99aa784 | 2012-04-13 10:27:35 +0800 | [diff] [blame] | 791 | blkdev_issue_flush(journal->j_fs_dev, GFP_NOFS, NULL); |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 792 | |
| 793 | /* Done it all: now write the commit record asynchronously. */ |
| 794 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, |
| 795 | JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) { |
| 796 | err = journal_submit_commit_record(journal, commit_transaction, |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 797 | &cbh, crc32_sum); |
| 798 | if (err) |
| 799 | __jbd2_journal_abort_hard(journal); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 800 | } |
| 801 | |
Jens Axboe | 82f04ab | 2011-03-17 11:01:52 +0100 | [diff] [blame] | 802 | blk_finish_plug(&plug); |
| 803 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 804 | /* Lo and behold: we have just managed to send a transaction to |
| 805 | the log. Before we can commit it, wait for the IO so far to |
| 806 | complete. Control buffers being written are on the |
| 807 | transaction's t_log_list queue, and metadata buffers are on |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 808 | the io_bufs list. |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 809 | |
| 810 | Wait for the buffers in reverse order. That way we are |
| 811 | less likely to be woken up until all IOs have completed, and |
| 812 | so we incur less scheduling load. |
| 813 | */ |
| 814 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 815 | jbd_debug(3, "JBD2: commit phase 3\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 816 | |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 817 | while (!list_empty(&io_bufs)) { |
| 818 | struct buffer_head *bh = list_entry(io_bufs.prev, |
| 819 | struct buffer_head, |
| 820 | b_assoc_buffers); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 821 | |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 822 | wait_on_buffer(bh); |
| 823 | cond_resched(); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 824 | |
| 825 | if (unlikely(!buffer_uptodate(bh))) |
| 826 | err = -EIO; |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 827 | jbd2_unfile_log_bh(bh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 828 | |
| 829 | /* |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 830 | * The list contains temporary buffer heads created by |
| 831 | * jbd2_journal_write_metadata_buffer(). |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 832 | */ |
| 833 | BUFFER_TRACE(bh, "dumping temporary bh"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 834 | __brelse(bh); |
| 835 | J_ASSERT_BH(bh, atomic_read(&bh->b_count) == 0); |
| 836 | free_buffer_head(bh); |
| 837 | |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 838 | /* We also have to refile the corresponding shadowed buffer */ |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 839 | jh = commit_transaction->t_shadow_list->b_tprev; |
| 840 | bh = jh2bh(jh); |
Jan Kara | f5113ef | 2013-06-04 12:01:45 -0400 | [diff] [blame^] | 841 | clear_buffer_jwrite(bh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 842 | J_ASSERT_BH(bh, buffer_jbddirty(bh)); |
| 843 | |
| 844 | /* The metadata is now released for reuse, but we need |
| 845 | to remember it against this transaction so that when |
| 846 | we finally commit, we can do any checkpointing |
| 847 | required. */ |
| 848 | JBUFFER_TRACE(jh, "file as BJ_Forget"); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 849 | jbd2_journal_file_buffer(jh, commit_transaction, BJ_Forget); |
Jan Kara | 229309c | 2011-05-08 19:09:53 -0400 | [diff] [blame] | 850 | /* |
| 851 | * Wake up any transactions which were waiting for this IO to |
| 852 | * complete. The barrier must be here so that changes by |
| 853 | * jbd2_journal_file_buffer() take effect before wake_up_bit() |
| 854 | * does the waitqueue check. |
| 855 | */ |
| 856 | smp_mb(); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 857 | wake_up_bit(&bh->b_state, BH_Unshadow); |
| 858 | JBUFFER_TRACE(jh, "brelse shadowed buffer"); |
| 859 | __brelse(bh); |
| 860 | } |
| 861 | |
| 862 | J_ASSERT (commit_transaction->t_shadow_list == NULL); |
| 863 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 864 | jbd_debug(3, "JBD2: commit phase 4\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 865 | |
| 866 | /* Here we wait for the revoke record and descriptor record buffers */ |
| 867 | wait_for_ctlbuf: |
| 868 | while (commit_transaction->t_log_list != NULL) { |
| 869 | struct buffer_head *bh; |
| 870 | |
| 871 | jh = commit_transaction->t_log_list->b_tprev; |
| 872 | bh = jh2bh(jh); |
| 873 | if (buffer_locked(bh)) { |
| 874 | wait_on_buffer(bh); |
| 875 | goto wait_for_ctlbuf; |
| 876 | } |
| 877 | if (cond_resched()) |
| 878 | goto wait_for_ctlbuf; |
| 879 | |
| 880 | if (unlikely(!buffer_uptodate(bh))) |
| 881 | err = -EIO; |
| 882 | |
| 883 | BUFFER_TRACE(bh, "ph5: control buffer writeout done: unfile"); |
| 884 | clear_buffer_jwrite(bh); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 885 | jbd2_journal_unfile_buffer(journal, jh); |
| 886 | jbd2_journal_put_journal_head(jh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 887 | __brelse(bh); /* One for getblk */ |
| 888 | /* AKPM: bforget here */ |
| 889 | } |
| 890 | |
Hidehiro Kawai | 77e841d | 2008-10-12 16:39:16 -0400 | [diff] [blame] | 891 | if (err) |
| 892 | jbd2_journal_abort(journal, err); |
| 893 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 894 | jbd_debug(3, "JBD2: commit phase 5\n"); |
Jan Kara | bbd2be3 | 2011-05-24 11:59:18 -0400 | [diff] [blame] | 895 | write_lock(&journal->j_state_lock); |
| 896 | J_ASSERT(commit_transaction->t_state == T_COMMIT_DFLUSH); |
| 897 | commit_transaction->t_state = T_COMMIT_JFLUSH; |
| 898 | write_unlock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 899 | |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 900 | if (!JBD2_HAS_INCOMPAT_FEATURE(journal, |
Theodore Ts'o | 0e3d2a6 | 2009-09-11 09:30:12 -0400 | [diff] [blame] | 901 | JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT)) { |
Girish Shilamkar | 818d276 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 902 | err = journal_submit_commit_record(journal, commit_transaction, |
| 903 | &cbh, crc32_sum); |
| 904 | if (err) |
| 905 | __jbd2_journal_abort_hard(journal); |
| 906 | } |
Zhang Huan | 6cba611 | 2011-04-05 19:16:20 -0400 | [diff] [blame] | 907 | if (cbh) |
Theodore Ts'o | fd98496 | 2009-01-05 21:34:13 -0500 | [diff] [blame] | 908 | err = journal_wait_on_commit_record(journal, cbh); |
Jan Kara | f73bee4 | 2010-08-18 15:56:56 +0200 | [diff] [blame] | 909 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, |
| 910 | JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT) && |
| 911 | journal->j_flags & JBD2_BARRIER) { |
Shaohua Li | 99aa784 | 2012-04-13 10:27:35 +0800 | [diff] [blame] | 912 | blkdev_issue_flush(journal->j_dev, GFP_NOFS, NULL); |
Jan Kara | f73bee4 | 2010-08-18 15:56:56 +0200 | [diff] [blame] | 913 | } |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 914 | |
| 915 | if (err) |
Jan Kara | a7fa2ba | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 916 | jbd2_journal_abort(journal, err); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 917 | |
Jan Kara | 3339578 | 2012-03-13 22:45:38 -0400 | [diff] [blame] | 918 | /* |
| 919 | * Now disk caches for filesystem device are flushed so we are safe to |
| 920 | * erase checkpointed transactions from the log by updating journal |
| 921 | * superblock. |
| 922 | */ |
| 923 | if (update_tail) |
| 924 | jbd2_update_log_tail(journal, first_tid, first_block); |
| 925 | |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 926 | /* End of a transaction! Finally, we can do checkpoint |
| 927 | processing: any buffers committed as a result of this |
| 928 | transaction can be removed from any checkpoint list it was on |
| 929 | before. */ |
| 930 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 931 | jbd_debug(3, "JBD2: commit phase 6\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 932 | |
Jan Kara | c851ed5 | 2008-07-11 19:27:31 -0400 | [diff] [blame] | 933 | J_ASSERT(list_empty(&commit_transaction->t_inode_list)); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 934 | J_ASSERT(commit_transaction->t_buffers == NULL); |
| 935 | J_ASSERT(commit_transaction->t_checkpoint_list == NULL); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 936 | J_ASSERT(commit_transaction->t_shadow_list == NULL); |
| 937 | J_ASSERT(commit_transaction->t_log_list == NULL); |
| 938 | |
| 939 | restart_loop: |
| 940 | /* |
| 941 | * As there are other places (journal_unmap_buffer()) adding buffers |
| 942 | * to this list we have to be careful and hold the j_list_lock. |
| 943 | */ |
| 944 | spin_lock(&journal->j_list_lock); |
| 945 | while (commit_transaction->t_forget) { |
| 946 | transaction_t *cp_transaction; |
| 947 | struct buffer_head *bh; |
Jan Kara | de1b794 | 2011-06-13 15:38:22 -0400 | [diff] [blame] | 948 | int try_to_free = 0; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 949 | |
| 950 | jh = commit_transaction->t_forget; |
| 951 | spin_unlock(&journal->j_list_lock); |
| 952 | bh = jh2bh(jh); |
Jan Kara | de1b794 | 2011-06-13 15:38:22 -0400 | [diff] [blame] | 953 | /* |
| 954 | * Get a reference so that bh cannot be freed before we are |
| 955 | * done with it. |
| 956 | */ |
| 957 | get_bh(bh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 958 | jbd_lock_bh_state(bh); |
dingdinghua | 23e2af3 | 2010-02-24 12:11:20 -0500 | [diff] [blame] | 959 | J_ASSERT_JH(jh, jh->b_transaction == commit_transaction); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 960 | |
| 961 | /* |
| 962 | * If there is undo-protected committed data against |
| 963 | * this buffer, then we can remove it now. If it is a |
| 964 | * buffer needing such protection, the old frozen_data |
| 965 | * field now points to a committed version of the |
| 966 | * buffer, so rotate that field to the new committed |
| 967 | * data. |
| 968 | * |
| 969 | * Otherwise, we can just throw away the frozen data now. |
Joel Becker | e06c822 | 2008-09-11 15:35:47 -0700 | [diff] [blame] | 970 | * |
| 971 | * We also know that the frozen data has already fired |
| 972 | * its triggers if they exist, so we can clear that too. |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 973 | */ |
| 974 | if (jh->b_committed_data) { |
Mingming Cao | af1e76d | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 975 | jbd2_free(jh->b_committed_data, bh->b_size); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 976 | jh->b_committed_data = NULL; |
| 977 | if (jh->b_frozen_data) { |
| 978 | jh->b_committed_data = jh->b_frozen_data; |
| 979 | jh->b_frozen_data = NULL; |
Joel Becker | e06c822 | 2008-09-11 15:35:47 -0700 | [diff] [blame] | 980 | jh->b_frozen_triggers = NULL; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 981 | } |
| 982 | } else if (jh->b_frozen_data) { |
Mingming Cao | af1e76d | 2007-10-16 18:38:25 -0400 | [diff] [blame] | 983 | jbd2_free(jh->b_frozen_data, bh->b_size); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 984 | jh->b_frozen_data = NULL; |
Joel Becker | e06c822 | 2008-09-11 15:35:47 -0700 | [diff] [blame] | 985 | jh->b_frozen_triggers = NULL; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 986 | } |
| 987 | |
| 988 | spin_lock(&journal->j_list_lock); |
| 989 | cp_transaction = jh->b_cp_transaction; |
| 990 | if (cp_transaction) { |
| 991 | JBUFFER_TRACE(jh, "remove from old cp transaction"); |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 992 | cp_transaction->t_chp_stats.cs_dropped++; |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 993 | __jbd2_journal_remove_checkpoint(jh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 994 | } |
| 995 | |
| 996 | /* Only re-checkpoint the buffer_head if it is marked |
| 997 | * dirty. If the buffer was added to the BJ_Forget list |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 998 | * by jbd2_journal_forget, it may no longer be dirty and |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 999 | * there's no point in keeping a checkpoint record for |
| 1000 | * it. */ |
| 1001 | |
Jan Kara | b794e7a | 2012-09-26 23:11:13 -0400 | [diff] [blame] | 1002 | /* |
| 1003 | * A buffer which has been freed while still being journaled by |
| 1004 | * a previous transaction. |
| 1005 | */ |
| 1006 | if (buffer_freed(bh)) { |
| 1007 | /* |
| 1008 | * If the running transaction is the one containing |
| 1009 | * "add to orphan" operation (b_next_transaction != |
| 1010 | * NULL), we have to wait for that transaction to |
| 1011 | * commit before we can really get rid of the buffer. |
| 1012 | * So just clear b_modified to not confuse transaction |
| 1013 | * credit accounting and refile the buffer to |
| 1014 | * BJ_Forget of the running transaction. If the just |
| 1015 | * committed transaction contains "add to orphan" |
| 1016 | * operation, we can completely invalidate the buffer |
| 1017 | * now. We are rather through in that since the |
| 1018 | * buffer may be still accessible when blocksize < |
| 1019 | * pagesize and it is attached to the last partial |
| 1020 | * page. |
| 1021 | */ |
| 1022 | jh->b_modified = 0; |
| 1023 | if (!jh->b_next_transaction) { |
| 1024 | clear_buffer_freed(bh); |
| 1025 | clear_buffer_jbddirty(bh); |
| 1026 | clear_buffer_mapped(bh); |
| 1027 | clear_buffer_new(bh); |
| 1028 | clear_buffer_req(bh); |
| 1029 | bh->b_bdev = NULL; |
| 1030 | } |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1031 | } |
| 1032 | |
| 1033 | if (buffer_jbddirty(bh)) { |
| 1034 | JBUFFER_TRACE(jh, "add to new checkpointing trans"); |
Mingming Cao | f7f4bcc | 2006-10-11 01:20:59 -0700 | [diff] [blame] | 1035 | __jbd2_journal_insert_checkpoint(jh, commit_transaction); |
Hidehiro Kawai | 7ad7445 | 2008-10-10 20:29:31 -0400 | [diff] [blame] | 1036 | if (is_journal_aborted(journal)) |
| 1037 | clear_buffer_jbddirty(bh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1038 | } else { |
| 1039 | J_ASSERT_BH(bh, !buffer_dirty(bh)); |
Jan Kara | de1b794 | 2011-06-13 15:38:22 -0400 | [diff] [blame] | 1040 | /* |
| 1041 | * The buffer on BJ_Forget list and not jbddirty means |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1042 | * it has been freed by this transaction and hence it |
| 1043 | * could not have been reallocated until this |
| 1044 | * transaction has committed. *BUT* it could be |
| 1045 | * reallocated once we have written all the data to |
| 1046 | * disk and before we process the buffer on BJ_Forget |
Jan Kara | de1b794 | 2011-06-13 15:38:22 -0400 | [diff] [blame] | 1047 | * list. |
| 1048 | */ |
| 1049 | if (!jh->b_next_transaction) |
| 1050 | try_to_free = 1; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1051 | } |
Jan Kara | de1b794 | 2011-06-13 15:38:22 -0400 | [diff] [blame] | 1052 | JBUFFER_TRACE(jh, "refile or unfile buffer"); |
| 1053 | __jbd2_journal_refile_buffer(jh); |
| 1054 | jbd_unlock_bh_state(bh); |
| 1055 | if (try_to_free) |
| 1056 | release_buffer_page(bh); /* Drops bh reference */ |
| 1057 | else |
| 1058 | __brelse(bh); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1059 | cond_resched_lock(&journal->j_list_lock); |
| 1060 | } |
| 1061 | spin_unlock(&journal->j_list_lock); |
| 1062 | /* |
Jan Kara | f5a7a6b | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1063 | * This is a bit sleazy. We use j_list_lock to protect transition |
| 1064 | * of a transaction into T_FINISHED state and calling |
| 1065 | * __jbd2_journal_drop_transaction(). Otherwise we could race with |
| 1066 | * other checkpointing code processing the transaction... |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1067 | */ |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 1068 | write_lock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1069 | spin_lock(&journal->j_list_lock); |
| 1070 | /* |
| 1071 | * Now recheck if some buffers did not get attached to the transaction |
| 1072 | * while the lock was dropped... |
| 1073 | */ |
| 1074 | if (commit_transaction->t_forget) { |
| 1075 | spin_unlock(&journal->j_list_lock); |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 1076 | write_unlock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1077 | goto restart_loop; |
| 1078 | } |
| 1079 | |
| 1080 | /* Done with this transaction! */ |
| 1081 | |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 1082 | jbd_debug(3, "JBD2: commit phase 7\n"); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1083 | |
Jan Kara | bbd2be3 | 2011-05-24 11:59:18 -0400 | [diff] [blame] | 1084 | J_ASSERT(commit_transaction->t_state == T_COMMIT_JFLUSH); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1085 | |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1086 | commit_transaction->t_start = jiffies; |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 1087 | stats.run.rs_logging = jbd2_time_diff(stats.run.rs_logging, |
| 1088 | commit_transaction->t_start); |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1089 | |
| 1090 | /* |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 1091 | * File the transaction statistics |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1092 | */ |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1093 | stats.ts_tid = commit_transaction->t_tid; |
Theodore Ts'o | 8dd4204 | 2010-08-03 21:38:29 -0400 | [diff] [blame] | 1094 | stats.run.rs_handle_count = |
| 1095 | atomic_read(&commit_transaction->t_handle_count); |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 1096 | trace_jbd2_run_stats(journal->j_fs_dev->bd_dev, |
| 1097 | commit_transaction->t_tid, &stats.run); |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1098 | |
| 1099 | /* |
| 1100 | * Calculate overall stats |
| 1101 | */ |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 1102 | spin_lock(&journal->j_history_lock); |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1103 | journal->j_stats.ts_tid++; |
Theodore Ts'o | 9fff24a | 2013-02-06 22:30:23 -0500 | [diff] [blame] | 1104 | if (commit_transaction->t_requested) |
| 1105 | journal->j_stats.ts_requested++; |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 1106 | journal->j_stats.run.rs_wait += stats.run.rs_wait; |
Theodore Ts'o | 9fff24a | 2013-02-06 22:30:23 -0500 | [diff] [blame] | 1107 | journal->j_stats.run.rs_request_delay += stats.run.rs_request_delay; |
Theodore Ts'o | bf69932 | 2009-09-30 00:32:06 -0400 | [diff] [blame] | 1108 | journal->j_stats.run.rs_running += stats.run.rs_running; |
| 1109 | journal->j_stats.run.rs_locked += stats.run.rs_locked; |
| 1110 | journal->j_stats.run.rs_flushing += stats.run.rs_flushing; |
| 1111 | journal->j_stats.run.rs_logging += stats.run.rs_logging; |
| 1112 | journal->j_stats.run.rs_handle_count += stats.run.rs_handle_count; |
| 1113 | journal->j_stats.run.rs_blocks += stats.run.rs_blocks; |
| 1114 | journal->j_stats.run.rs_blocks_logged += stats.run.rs_blocks_logged; |
Johann Lombardi | 8e85fb3 | 2008-01-28 23:58:27 -0500 | [diff] [blame] | 1115 | spin_unlock(&journal->j_history_lock); |
| 1116 | |
Dmitry Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 1117 | commit_transaction->t_state = T_COMMIT_CALLBACK; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1118 | J_ASSERT(commit_transaction == journal->j_committing_transaction); |
| 1119 | journal->j_commit_sequence = commit_transaction->t_tid; |
| 1120 | journal->j_committing_transaction = NULL; |
Josef Bacik | e07f718 | 2008-11-26 01:14:26 -0500 | [diff] [blame] | 1121 | commit_time = ktime_to_ns(ktime_sub(ktime_get(), start_time)); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1122 | |
Josef Bacik | e07f718 | 2008-11-26 01:14:26 -0500 | [diff] [blame] | 1123 | /* |
| 1124 | * weight the commit time higher than the average time so we don't |
| 1125 | * react too strongly to vast changes in the commit time |
| 1126 | */ |
| 1127 | if (likely(journal->j_average_commit_time)) |
| 1128 | journal->j_average_commit_time = (commit_time + |
| 1129 | journal->j_average_commit_time*3) / 4; |
| 1130 | else |
| 1131 | journal->j_average_commit_time = commit_time; |
Dmitry Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 1132 | |
Theodore Ts'o | a931da6 | 2010-08-03 21:35:12 -0400 | [diff] [blame] | 1133 | write_unlock(&journal->j_state_lock); |
Theodore Ts'o | 6c20ec8 | 2008-10-28 21:08:20 -0400 | [diff] [blame] | 1134 | |
Dmitry Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 1135 | if (journal->j_checkpoint_transactions == NULL) { |
| 1136 | journal->j_checkpoint_transactions = commit_transaction; |
| 1137 | commit_transaction->t_cpnext = commit_transaction; |
| 1138 | commit_transaction->t_cpprev = commit_transaction; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1139 | } else { |
Dmitry Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 1140 | commit_transaction->t_cpnext = |
| 1141 | journal->j_checkpoint_transactions; |
| 1142 | commit_transaction->t_cpprev = |
| 1143 | commit_transaction->t_cpnext->t_cpprev; |
| 1144 | commit_transaction->t_cpnext->t_cpprev = |
| 1145 | commit_transaction; |
| 1146 | commit_transaction->t_cpprev->t_cpnext = |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1147 | commit_transaction; |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1148 | } |
| 1149 | spin_unlock(&journal->j_list_lock); |
Dmitry Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 1150 | /* Drop all spin_locks because commit_callback may be block. |
| 1151 | * __journal_remove_checkpoint() can not destroy transaction |
| 1152 | * under us because it is not marked as T_FINISHED yet */ |
Aneesh Kumar K.V | fb68407 | 2008-11-06 17:50:21 -0500 | [diff] [blame] | 1153 | if (journal->j_commit_callback) |
| 1154 | journal->j_commit_callback(journal, commit_transaction); |
| 1155 | |
Theodore Ts'o | 879c5e6 | 2009-06-17 11:47:48 -0400 | [diff] [blame] | 1156 | trace_jbd2_end_commit(journal, commit_transaction); |
Eryu Guan | f2a4452 | 2011-11-01 19:09:18 -0400 | [diff] [blame] | 1157 | jbd_debug(1, "JBD2: commit %d complete, head %d\n", |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1158 | journal->j_commit_sequence, journal->j_tail_sequence); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1159 | |
Dmitry Monakhov | 794446c | 2013-04-03 22:06:52 -0400 | [diff] [blame] | 1160 | write_lock(&journal->j_state_lock); |
| 1161 | spin_lock(&journal->j_list_lock); |
| 1162 | commit_transaction->t_state = T_FINISHED; |
| 1163 | /* Recheck checkpoint lists after j_list_lock was dropped */ |
| 1164 | if (commit_transaction->t_checkpoint_list == NULL && |
| 1165 | commit_transaction->t_checkpoint_io_list == NULL) { |
| 1166 | __jbd2_journal_drop_transaction(journal, commit_transaction); |
| 1167 | jbd2_journal_free_transaction(commit_transaction); |
| 1168 | } |
| 1169 | spin_unlock(&journal->j_list_lock); |
| 1170 | write_unlock(&journal->j_state_lock); |
Dave Kleikamp | 470decc | 2006-10-11 01:20:57 -0700 | [diff] [blame] | 1171 | wake_up(&journal->j_wait_done_commit); |
| 1172 | } |