Re: [PATCH 3/3] Add timeout feature

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Dave Chinner
Date: Wednesday, July 9, 2008 - 4:49 am

On Wed, Jul 09, 2008 at 07:09:00AM -0400, Theodore Tso wrote:


(e) none of the above.  The kernel compilation will appear to pause
until the filesystem is unfrozen. No other visible effect should
occur. It will get blocked in a write or filesystem transaction
because the fs is frozen.

Look at vfs_check_frozen() - any call to that will block if the
filesystem is frozen or being frozen. The generic hook is in
__generic_file_aio_write_nolock() and various other filesystems have
calls in their specific write paths (fuse, ntfs, ocfs2, xfs, xip) to
do this.

For all other modifications, filesystem specific methods of
blocking transactions are used. XFS uses vfs_check_frozen() in
xfs_trans_alloc(), ext3 (and probably ocfs2) do it via
their ->write_super_lockfs method calling journal_lock_updates(),
ext4 via jbd2_lock_updates() and so on....

When the filesystem is unfrozen the journal is unlocked and
anything sleeping on the vfs_check_frozen() waitqueue is
woken.....

Cheers,

Dave.
-- 
Dave Chinner
david@fromorbit.com
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 3/3] Add timeout feature, Takashi Sato, (Mon Jun 30, 5:24 am)
Re: [PATCH 3/3] Add timeout feature, Christoph Hellwig, (Tue Jul 1, 1:10 am)
Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature, Alasdair G Kergon, (Tue Jul 1, 3:52 am)
Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature, Takashi Sato, (Thu Jul 3, 5:11 am)
Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature, Alasdair G Kergon, (Thu Jul 3, 5:47 am)
Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature, Eric Sandeen, (Thu Jul 3, 7:45 am)
Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Thu Jul 3, 3:11 pm)
Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature, Takashi Sato, (Fri Jul 4, 5:08 am)
Re: [PATCH 3/3] Add timeout feature, Pavel Machek, (Mon Jul 7, 4:07 am)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Tue Jul 8, 4:10 pm)
Re: [PATCH 3/3] Add timeout feature, Pavel Machek, (Tue Jul 8, 4:20 pm)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Tue Jul 8, 5:52 pm)
Re: [PATCH 3/3] Add timeout feature, Theodore Tso, (Tue Jul 8, 6:09 pm)
Re: [PATCH 3/3] Add timeout feature, Brad Boyer, (Tue Jul 8, 9:21 pm)
Re: [PATCH 3/3] Add timeout feature, Miklos Szeredi, (Tue Jul 8, 11:13 pm)
Re: [PATCH 3/3] Add timeout feature, Christoph Hellwig, (Tue Jul 8, 11:16 pm)
Re: [PATCH 3/3] Add timeout feature, Miklos Szeredi, (Tue Jul 8, 11:22 pm)
Re: [PATCH 3/3] Add timeout feature, Arjan van de Ven, (Tue Jul 8, 11:41 pm)
Re: [PATCH 3/3] Add timeout feature, Miklos Szeredi, (Tue Jul 8, 11:48 pm)
Re: [PATCH 3/3] Add timeout feature, Arjan van de Ven, (Tue Jul 8, 11:55 pm)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Tue Jul 8, 11:59 pm)
Re: [PATCH 3/3] Add timeout feature, Miklos Szeredi, (Wed Jul 9, 12:08 am)
Re: [PATCH 3/3] Add timeout feature, Miklos Szeredi, (Wed Jul 9, 12:13 am)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Wed Jul 9, 12:13 am)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Wed Jul 9, 12:33 am)
Re: [PATCH 3/3] Add timeout feature, Miklos Szeredi, (Wed Jul 9, 1:11 am)
Re: [PATCH 3/3] Add timeout feature, Theodore Tso, (Wed Jul 9, 4:09 am)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Wed Jul 9, 4:15 am)
Re: [PATCH 3/3] Add timeout feature, Dave Chinner, (Wed Jul 9, 4:49 am)
Re: [PATCH 3/3] Add timeout feature, Theodore Tso, (Wed Jul 9, 5:24 am)
Re: [PATCH 3/3] Add timeout feature, Olaf , (Wed Jul 9, 5:59 am)
Re: [PATCH 3/3] Add timeout feature, Arjan van de Ven, (Wed Jul 9, 6:53 am)
Re: [PATCH 3/3] Add timeout feature, Arjan van de Ven, (Wed Jul 9, 6:55 am)
Re: [PATCH 3/3] Add timeout feature, Arjan van de Ven, (Wed Jul 9, 6:57 am)
Re: [PATCH 3/3] Add timeout feature, jim owens, (Wed Jul 9, 6:58 am)
Re: [PATCH 3/3] Add timeout feature, jim owens, (Wed Jul 9, 7:13 am)
Re: [PATCH 3/3] Add timeout feature, Pavel Machek, (Wed Jul 9, 1:44 pm)
Re: [PATCH 3/3] Add timeout feature, Pavel Machek, (Wed Jul 9, 1:48 pm)
Re: [PATCH 3/3] Add timeout feature, Pavel Machek, (Sun Jul 13, 5:06 am)
Re: [PATCH 3/3] Add timeout feature, jim owens, (Sun Jul 13, 10:15 am)
Re: [PATCH 3/3] Add timeout feature, Pavel Machek, (Sun Jul 13, 11:36 pm)
Re: [PATCH 3/3] Add timeout feature, Takashi Sato, (Mon Jul 14, 6:12 am)
Re: [PATCH 3/3] Add timeout feature, jim owens, (Mon Jul 14, 6:17 am)
Re: [PATCH 3/3] Add timeout feature, jim owens, (Mon Jul 14, 7:04 am)