login
Login
/
Register
Search
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2008
»
July
»
15
Re: [PATCH] stopmachine: add stopmachine_timeout
view
thread
!MAILaRCHIVE_VOTE_RePLACE
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From:
Max Krasnyansky <maxk@...>
To: Rusty Russell <rusty@...>
Cc: Heiko Carstens <heiko.carstens@...>, Jeremy Fitzhardinge <jeremy@...>, Christian Borntraeger <borntraeger@...>, Hidetoshi Seto <seto.hidetoshi@...>, <linux-kernel@...>, <virtualization@...>, Zachary Amsden <zach@...>
Subject:
Re: [PATCH] stopmachine: add stopmachine_timeout
Date: Tuesday, July 15, 2008 - 4:51 am
Rusty Russell wrote:
quoted text
> On Tuesday 15 July 2008 12:24:54 Max Krasnyansky wrote: >> Heiko Carstens wrote: >>> On Mon, Jul 14, 2008 at 11:56:18AM -0700, Jeremy Fitzhardinge wrote: >>> This is asking for trouble... a config option to disable this would be >>> nice. But as I don't know which problem this patch originally addresses >>> it might be that this is needed anyway. So lets see why we need it first. >> How about this. We'll make this a sysctl, as Rusty already did, and set the >> default to 0 which means "never timeout". That way crazy people like me who >> care about this scenario can enable this feature. > > Indeed, this was my thought too. s390 can initialize it to zero somewhere in > their boot code. > >> btw Rusty, I just had this "why didn't I think of that" moments. This is >> actually another way of handling my workload. I mean it certainly does not >> fix the root case of the problems and we still need other things that we >> talked about (non-blocking module delete, lock-free module insertion, etc) >> but at least in the mean time it avoids wedging the machines for good. >> btw I'd like that timeout in milliseconds. I think 5 seconds is way tooooo >> long :). > > We can make it ms, sure. 200ms should be plenty of time: worst I ever saw was > 150ms, and that was some weird Power box doing crazy stuff. I wouldn't be > surprised if you'd never see 1ms on your hardware.
Sounds good.
quoted text
> The ipi idea would handle your case a little more nicely, too, but that's > probably not going to hit this merge window.
Which reminds me that I wanted to submit a bunch of kthread and workqueue related things in this window :). Max --
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
[PATCH] stopmachine: add stopmachine_timeout
, Hidetoshi Seto
, (Mon Jul 14, 3:52 am)
[PATCH] stopmachine: add stopmachine_timeout v4
, Hidetoshi Seto
, (Thu Jul 17, 2:12 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v4
, Max Krasnyansky
, (Thu Jul 17, 3:09 am)
[PATCH] stopmachine: add stopmachine_timeout v2
, Hidetoshi Seto
, (Wed Jul 16, 12:27 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2
, Jeremy Fitzhardinge
, (Wed Jul 16, 6:11 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2
, Hidetoshi Seto
, (Wed Jul 16, 11:40 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout v2
, Rusty Russell
, (Fri Jul 18, 12:18 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2
, Jeremy Fitzhardinge
, (Thu Jul 17, 1:37 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2
, Max Krasnyansky
, (Wed Jul 16, 2:23 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v2
, Hidetoshi Seto
, (Wed Jul 16, 2:35 am)
[PATCH] stopmachine: add stopmachine_timeout v3
, Hidetoshi Seto
, (Wed Jul 16, 2:51 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v3
, Peter Zijlstra
, (Wed Jul 16, 3:33 am)
Re: [PATCH] stopmachine: add stopmachine_timeout v3
, Hidetoshi Seto
, (Wed Jul 16, 4:12 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Christian Borntraeger
, (Mon Jul 14, 7:51 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Rusty Russell
, (Mon Jul 14, 8:34 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Jeremy Fitzhardinge
, (Mon Jul 14, 2:56 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Heiko Carstens
, (Mon Jul 14, 5:20 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Hidetoshi Seto
, (Mon Jul 14, 10:24 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Max Krasnyansky
, (Mon Jul 14, 10:37 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Max Krasnyansky
, (Mon Jul 14, 10:24 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Rusty Russell
, (Tue Jul 15, 4:09 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Christian Borntraeger
, (Wed Jul 16, 5:15 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Max Krasnyansky
, (Tue Jul 15, 4:51 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Heiko Carstens
, (Tue Jul 15, 4:39 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Heiko Carstens
, (Tue Jul 15, 2:09 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Rusty Russell
, (Mon Jul 14, 9:14 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Rusty Russell
, (Mon Jul 14, 6:43 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Hidetoshi Seto
, (Mon Jul 14, 9:11 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Rusty Russell
, (Tue Jul 15, 3:50 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Hidetoshi Seto
, (Wed Jul 16, 12:05 am)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Rusty Russell
, (Sun Jul 20, 5:45 am)
[PATCH] stopmachine: allow force progress on timeout
, Hidetoshi Seto
, (Mon Jul 21, 11:28 pm)
Re: [PATCH] stopmachine: add stopmachine_timeout
, Hidetoshi Seto
, (Mon Jul 14, 4:19 am)
Navigation
Create content
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Paul Jackson
Re: cpuset-remove-sched-domain-hooks-from-cpusets
James Bottomley
Re: Announce: Linux-next (Or Andrew's dream :-))
David Miller
Slow DOWN, please!!!
Masami Hiramatsu
Re: [RFC PATCH v4] Unified trace buffer
git
:
linux-netdev
:
Gerrit Renker
[PATCH 27/37] dccp: Integration of dynamic feature activation - part 2 (server side)
Parag Warudkar
Re: 2.6.29-rc3: tg3 dead after resume
Jarek Poplawski
[PATCH] pkt_sched: Destroy gen estimators under rtnl_lock().
openbsd-misc
:
Colocation donated by:
Who's online
There are currently
9 users
and
1034 guests
online.
Online users
zeekec
trabajornet
aluminiumwindo
teethwhitening
shoppingonline
alabamajailsvna
prosperityrisi
privateinveswnh
tshirtprinting
Syndicate