Re: [dm-devel] Re: [PATCH 3/3] Add timeout feature

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Alasdair G Kergon
Date: Thursday, July 3, 2008 - 5:47 am

On Thu, Jul 03, 2008 at 09:11:05PM +0900, Takashi Sato wrote:

But you could also say that if the 'freezer' process accesses the frozen
filesystem and deadlocks then that's just a bug and that userspace code
should be fixed and there's no need to introduce the complexity of a
timeout parameter.

 
The point I'm trying to make here is:
  Under what real-world circumstances might multiple concurrent freezing
  attempts occur, and which of A, B or C (or other variations) would be
  the most appropriate way of handling such situations?

A common example is people running xfs_freeze followed by an lvm command
which also attempts to freeze the filesystem.

I can see a case for B or C, but personally I prefer A:


Alasdair
-- 
agk@redhat.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)