Re: [PATCH] stopmachine: add stopmachine_timeout

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Rusty Russell
Date: Sunday, July 20, 2008 - 2:45 am

On Wednesday 16 July 2008 14:05:31 Hidetoshi Seto wrote:

No.  You aim for perfection, but there is no "right" answer other than "don't 
get your system into this mess".  Whatever we do is going to be an educated 
guess.  And guessing that there'll be no race is a very good guess indeed.

The scenario we are addressing is a stuck CPU and module load.  If we fail 
stop machine, module load fails.

That is why we should continue if we can.  It is also why the default timeout 
cannot be 0.  You can't turn this on once you notice there's a problem: it's 
too late.

If we don't want to handle this case, let's not apply any patch at all.
Rusty.

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] stopmachine: add stopmachine_timeout, Hidetoshi Seto, (Mon Jul 14, 12:52 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Hidetoshi Seto, (Mon Jul 14, 1:19 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Rusty Russell, (Mon Jul 14, 3:43 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Christian Borntraeger, (Mon Jul 14, 4:51 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Rusty Russell, (Mon Jul 14, 5:34 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Jeremy Fitzhardinge, (Mon Jul 14, 11:56 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Heiko Carstens, (Mon Jul 14, 2:20 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Hidetoshi Seto, (Mon Jul 14, 6:11 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Rusty Russell, (Mon Jul 14, 6:14 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Hidetoshi Seto, (Mon Jul 14, 7:24 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Max Krasnyansky, (Mon Jul 14, 7:24 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Max Krasnyansky, (Mon Jul 14, 7:37 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Heiko Carstens, (Mon Jul 14, 11:09 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Rusty Russell, (Tue Jul 15, 12:50 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Rusty Russell, (Tue Jul 15, 1:09 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Heiko Carstens, (Tue Jul 15, 1:39 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Max Krasnyansky, (Tue Jul 15, 1:51 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Hidetoshi Seto, (Tue Jul 15, 9:05 pm)
[PATCH] stopmachine: add stopmachine_timeout v2, Hidetoshi Seto, (Tue Jul 15, 9:27 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout v2, Max Krasnyansky, (Tue Jul 15, 11:23 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout v2, Hidetoshi Seto, (Tue Jul 15, 11:35 pm)
[PATCH] stopmachine: add stopmachine_timeout v3, Hidetoshi Seto, (Tue Jul 15, 11:51 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout v3, Peter Zijlstra, (Wed Jul 16, 12:33 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v3, Hidetoshi Seto, (Wed Jul 16, 1:12 am)
Re: [PATCH] stopmachine: add stopmachine_timeout, Christian Borntraeger, (Wed Jul 16, 2:15 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2, Jeremy Fitzhardinge, (Wed Jul 16, 3:11 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2, Hidetoshi Seto, (Wed Jul 16, 8:40 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout v2, Jeremy Fitzhardinge, (Wed Jul 16, 10:37 pm)
[PATCH] stopmachine: add stopmachine_timeout v4, Hidetoshi Seto, (Wed Jul 16, 11:12 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout v4, Max Krasnyansky, (Thu Jul 17, 12:09 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2, Rusty Russell, (Thu Jul 17, 9:18 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout, Rusty Russell, (Sun Jul 20, 2:45 am)
[PATCH] stopmachine: allow force progress on timeout, Hidetoshi Seto, (Mon Jul 21, 8:28 pm)