.. | .. |
---|
39 | 39 | * Write the MMP block using REQ_SYNC to try to get the block on-disk |
---|
40 | 40 | * faster. |
---|
41 | 41 | */ |
---|
42 | | -static int write_mmp_block(struct super_block *sb, struct buffer_head *bh) |
---|
| 42 | +static int write_mmp_block_thawed(struct super_block *sb, |
---|
| 43 | + struct buffer_head *bh) |
---|
43 | 44 | { |
---|
44 | 45 | struct mmp_struct *mmp = (struct mmp_struct *)(bh->b_data); |
---|
45 | 46 | |
---|
46 | | - /* |
---|
47 | | - * We protect against freezing so that we don't create dirty buffers |
---|
48 | | - * on frozen filesystem. |
---|
49 | | - */ |
---|
50 | | - sb_start_write(sb); |
---|
51 | 47 | ext4_mmp_csum_set(sb, mmp); |
---|
52 | 48 | lock_buffer(bh); |
---|
53 | 49 | bh->b_end_io = end_buffer_write_sync; |
---|
54 | 50 | get_bh(bh); |
---|
55 | 51 | submit_bh(REQ_OP_WRITE, REQ_SYNC | REQ_META | REQ_PRIO, bh); |
---|
56 | 52 | wait_on_buffer(bh); |
---|
57 | | - sb_end_write(sb); |
---|
58 | 53 | if (unlikely(!buffer_uptodate(bh))) |
---|
59 | | - return 1; |
---|
60 | | - |
---|
| 54 | + return -EIO; |
---|
61 | 55 | return 0; |
---|
| 56 | +} |
---|
| 57 | + |
---|
| 58 | +static int write_mmp_block(struct super_block *sb, struct buffer_head *bh) |
---|
| 59 | +{ |
---|
| 60 | + int err; |
---|
| 61 | + |
---|
| 62 | + /* |
---|
| 63 | + * We protect against freezing so that we don't create dirty buffers |
---|
| 64 | + * on frozen filesystem. |
---|
| 65 | + */ |
---|
| 66 | + sb_start_write(sb); |
---|
| 67 | + err = write_mmp_block_thawed(sb, bh); |
---|
| 68 | + sb_end_write(sb); |
---|
| 69 | + return err; |
---|
62 | 70 | } |
---|
63 | 71 | |
---|
64 | 72 | /* |
---|
.. | .. |
---|
85 | 93 | } |
---|
86 | 94 | } |
---|
87 | 95 | |
---|
88 | | - get_bh(*bh); |
---|
89 | 96 | lock_buffer(*bh); |
---|
90 | | - (*bh)->b_end_io = end_buffer_read_sync; |
---|
91 | | - submit_bh(REQ_OP_READ, REQ_META | REQ_PRIO, *bh); |
---|
92 | | - wait_on_buffer(*bh); |
---|
93 | | - if (!buffer_uptodate(*bh)) { |
---|
94 | | - ret = -EIO; |
---|
| 97 | + ret = ext4_read_bh(*bh, REQ_META | REQ_PRIO, NULL); |
---|
| 98 | + if (ret) |
---|
95 | 99 | goto warn_exit; |
---|
96 | | - } |
---|
| 100 | + |
---|
97 | 101 | mmp = (struct mmp_struct *)((*bh)->b_data); |
---|
98 | 102 | if (le32_to_cpu(mmp->mmp_magic) != EXT4_MMP_MAGIC) { |
---|
99 | 103 | ret = -EFSCORRUPTED; |
---|
.. | .. |
---|
131 | 135 | */ |
---|
132 | 136 | static int kmmpd(void *data) |
---|
133 | 137 | { |
---|
134 | | - struct super_block *sb = ((struct mmpd_data *) data)->sb; |
---|
135 | | - struct buffer_head *bh = ((struct mmpd_data *) data)->bh; |
---|
| 138 | + struct super_block *sb = (struct super_block *) data; |
---|
136 | 139 | struct ext4_super_block *es = EXT4_SB(sb)->s_es; |
---|
| 140 | + struct buffer_head *bh = EXT4_SB(sb)->s_mmp_bh; |
---|
137 | 141 | struct mmp_struct *mmp; |
---|
138 | 142 | ext4_fsblk_t mmp_block; |
---|
139 | 143 | u32 seq = 0; |
---|
.. | .. |
---|
160 | 164 | memcpy(mmp->mmp_nodename, init_utsname()->nodename, |
---|
161 | 165 | sizeof(mmp->mmp_nodename)); |
---|
162 | 166 | |
---|
163 | | - while (!kthread_should_stop()) { |
---|
| 167 | + while (!kthread_should_stop() && !sb_rdonly(sb)) { |
---|
| 168 | + if (!ext4_has_feature_mmp(sb)) { |
---|
| 169 | + ext4_warning(sb, "kmmpd being stopped since MMP feature" |
---|
| 170 | + " has been disabled."); |
---|
| 171 | + goto wait_to_exit; |
---|
| 172 | + } |
---|
164 | 173 | if (++seq > EXT4_MMP_SEQ_MAX) |
---|
165 | 174 | seq = 1; |
---|
166 | 175 | |
---|
.. | .. |
---|
174 | 183 | * (s_mmp_update_interval * 60) seconds. |
---|
175 | 184 | */ |
---|
176 | 185 | if (retval) { |
---|
177 | | - if ((failed_writes % 60) == 0) |
---|
178 | | - ext4_error(sb, "Error writing to MMP block"); |
---|
| 186 | + if ((failed_writes % 60) == 0) { |
---|
| 187 | + ext4_error_err(sb, -retval, |
---|
| 188 | + "Error writing to MMP block"); |
---|
| 189 | + } |
---|
179 | 190 | failed_writes++; |
---|
180 | 191 | } |
---|
181 | | - |
---|
182 | | - if (!(le32_to_cpu(es->s_feature_incompat) & |
---|
183 | | - EXT4_FEATURE_INCOMPAT_MMP)) { |
---|
184 | | - ext4_warning(sb, "kmmpd being stopped since MMP feature" |
---|
185 | | - " has been disabled."); |
---|
186 | | - goto exit_thread; |
---|
187 | | - } |
---|
188 | | - |
---|
189 | | - if (sb_rdonly(sb)) |
---|
190 | | - break; |
---|
191 | 192 | |
---|
192 | 193 | diff = jiffies - last_update_time; |
---|
193 | 194 | if (diff < mmp_update_interval * HZ) |
---|
.. | .. |
---|
206 | 207 | |
---|
207 | 208 | retval = read_mmp_block(sb, &bh_check, mmp_block); |
---|
208 | 209 | if (retval) { |
---|
209 | | - ext4_error(sb, "error reading MMP data: %d", |
---|
210 | | - retval); |
---|
211 | | - goto exit_thread; |
---|
| 210 | + ext4_error_err(sb, -retval, |
---|
| 211 | + "error reading MMP data: %d", |
---|
| 212 | + retval); |
---|
| 213 | + goto wait_to_exit; |
---|
212 | 214 | } |
---|
213 | 215 | |
---|
214 | 216 | mmp_check = (struct mmp_struct *)(bh_check->b_data); |
---|
.. | .. |
---|
219 | 221 | "Error while updating MMP info. " |
---|
220 | 222 | "The filesystem seems to have been" |
---|
221 | 223 | " multiply mounted."); |
---|
222 | | - ext4_error(sb, "abort"); |
---|
| 224 | + ext4_error_err(sb, EBUSY, "abort"); |
---|
223 | 225 | put_bh(bh_check); |
---|
224 | 226 | retval = -EBUSY; |
---|
225 | | - goto exit_thread; |
---|
| 227 | + goto wait_to_exit; |
---|
226 | 228 | } |
---|
227 | 229 | put_bh(bh_check); |
---|
228 | 230 | } |
---|
.. | .. |
---|
245 | 247 | |
---|
246 | 248 | retval = write_mmp_block(sb, bh); |
---|
247 | 249 | |
---|
248 | | -exit_thread: |
---|
249 | | - EXT4_SB(sb)->s_mmp_tsk = NULL; |
---|
250 | | - kfree(data); |
---|
251 | | - brelse(bh); |
---|
| 250 | +wait_to_exit: |
---|
| 251 | + while (!kthread_should_stop()) { |
---|
| 252 | + set_current_state(TASK_INTERRUPTIBLE); |
---|
| 253 | + if (!kthread_should_stop()) |
---|
| 254 | + schedule(); |
---|
| 255 | + } |
---|
| 256 | + set_current_state(TASK_RUNNING); |
---|
252 | 257 | return retval; |
---|
| 258 | +} |
---|
| 259 | + |
---|
| 260 | +void ext4_stop_mmpd(struct ext4_sb_info *sbi) |
---|
| 261 | +{ |
---|
| 262 | + if (sbi->s_mmp_tsk) { |
---|
| 263 | + kthread_stop(sbi->s_mmp_tsk); |
---|
| 264 | + brelse(sbi->s_mmp_bh); |
---|
| 265 | + sbi->s_mmp_tsk = NULL; |
---|
| 266 | + } |
---|
253 | 267 | } |
---|
254 | 268 | |
---|
255 | 269 | /* |
---|
.. | .. |
---|
276 | 290 | struct ext4_super_block *es = EXT4_SB(sb)->s_es; |
---|
277 | 291 | struct buffer_head *bh = NULL; |
---|
278 | 292 | struct mmp_struct *mmp = NULL; |
---|
279 | | - struct mmpd_data *mmpd_data; |
---|
280 | 293 | u32 seq; |
---|
281 | 294 | unsigned int mmp_check_interval = le16_to_cpu(es->s_mmp_update_interval); |
---|
282 | 295 | unsigned int wait_time = 0; |
---|
.. | .. |
---|
285 | 298 | if (mmp_block < le32_to_cpu(es->s_first_data_block) || |
---|
286 | 299 | mmp_block >= ext4_blocks_count(es)) { |
---|
287 | 300 | ext4_warning(sb, "Invalid MMP block in superblock"); |
---|
| 301 | + retval = -EINVAL; |
---|
288 | 302 | goto failed; |
---|
289 | 303 | } |
---|
290 | 304 | |
---|
.. | .. |
---|
310 | 324 | |
---|
311 | 325 | if (seq == EXT4_MMP_SEQ_FSCK) { |
---|
312 | 326 | dump_mmp_msg(sb, mmp, "fsck is running on the filesystem"); |
---|
| 327 | + retval = -EBUSY; |
---|
313 | 328 | goto failed; |
---|
314 | 329 | } |
---|
315 | 330 | |
---|
.. | .. |
---|
323 | 338 | |
---|
324 | 339 | if (schedule_timeout_interruptible(HZ * wait_time) != 0) { |
---|
325 | 340 | ext4_warning(sb, "MMP startup interrupted, failing mount\n"); |
---|
| 341 | + retval = -ETIMEDOUT; |
---|
326 | 342 | goto failed; |
---|
327 | 343 | } |
---|
328 | 344 | |
---|
.. | .. |
---|
333 | 349 | if (seq != le32_to_cpu(mmp->mmp_seq)) { |
---|
334 | 350 | dump_mmp_msg(sb, mmp, |
---|
335 | 351 | "Device is already active on another node."); |
---|
| 352 | + retval = -EBUSY; |
---|
336 | 353 | goto failed; |
---|
337 | 354 | } |
---|
338 | 355 | |
---|
.. | .. |
---|
343 | 360 | seq = mmp_new_seq(); |
---|
344 | 361 | mmp->mmp_seq = cpu_to_le32(seq); |
---|
345 | 362 | |
---|
346 | | - retval = write_mmp_block(sb, bh); |
---|
| 363 | + /* |
---|
| 364 | + * On mount / remount we are protected against fs freezing (by s_umount |
---|
| 365 | + * semaphore) and grabbing freeze protection upsets lockdep |
---|
| 366 | + */ |
---|
| 367 | + retval = write_mmp_block_thawed(sb, bh); |
---|
347 | 368 | if (retval) |
---|
348 | 369 | goto failed; |
---|
349 | 370 | |
---|
.. | .. |
---|
352 | 373 | */ |
---|
353 | 374 | if (schedule_timeout_interruptible(HZ * wait_time) != 0) { |
---|
354 | 375 | ext4_warning(sb, "MMP startup interrupted, failing mount"); |
---|
| 376 | + retval = -ETIMEDOUT; |
---|
355 | 377 | goto failed; |
---|
356 | 378 | } |
---|
357 | 379 | |
---|
.. | .. |
---|
362 | 384 | if (seq != le32_to_cpu(mmp->mmp_seq)) { |
---|
363 | 385 | dump_mmp_msg(sb, mmp, |
---|
364 | 386 | "Device is already active on another node."); |
---|
| 387 | + retval = -EBUSY; |
---|
365 | 388 | goto failed; |
---|
366 | 389 | } |
---|
367 | 390 | |
---|
368 | | - mmpd_data = kmalloc(sizeof(*mmpd_data), GFP_KERNEL); |
---|
369 | | - if (!mmpd_data) { |
---|
370 | | - ext4_warning(sb, "not enough memory for mmpd_data"); |
---|
371 | | - goto failed; |
---|
372 | | - } |
---|
373 | | - mmpd_data->sb = sb; |
---|
374 | | - mmpd_data->bh = bh; |
---|
| 391 | + EXT4_SB(sb)->s_mmp_bh = bh; |
---|
375 | 392 | |
---|
376 | 393 | /* |
---|
377 | 394 | * Start a kernel thread to update the MMP block periodically. |
---|
378 | 395 | */ |
---|
379 | | - EXT4_SB(sb)->s_mmp_tsk = kthread_run(kmmpd, mmpd_data, "kmmpd-%.*s", |
---|
| 396 | + EXT4_SB(sb)->s_mmp_tsk = kthread_run(kmmpd, sb, "kmmpd-%.*s", |
---|
380 | 397 | (int)sizeof(mmp->mmp_bdevname), |
---|
381 | 398 | bdevname(bh->b_bdev, |
---|
382 | 399 | mmp->mmp_bdevname)); |
---|
383 | 400 | if (IS_ERR(EXT4_SB(sb)->s_mmp_tsk)) { |
---|
384 | 401 | EXT4_SB(sb)->s_mmp_tsk = NULL; |
---|
385 | | - kfree(mmpd_data); |
---|
386 | 402 | ext4_warning(sb, "Unable to create kmmpd thread for %s.", |
---|
387 | 403 | sb->s_id); |
---|
| 404 | + retval = -ENOMEM; |
---|
388 | 405 | goto failed; |
---|
389 | 406 | } |
---|
390 | 407 | |
---|
.. | .. |
---|
392 | 409 | |
---|
393 | 410 | failed: |
---|
394 | 411 | brelse(bh); |
---|
395 | | - return 1; |
---|
| 412 | + return retval; |
---|
396 | 413 | } |
---|
397 | | - |
---|
398 | | - |
---|