Re: [PATCH] stopmachine: add stopmachine_timeout
- From: Hidetoshi Seto <seto.hidetoshi@xxxxxxxxxxxxxx>
- Date: Tue, 15 Jul 2008 11:24:52 +0900
Heiko Carstens wrote:
Hmm.. probably a stupid question: but what could happen that a real cpu
(not virtual) becomes unresponsive so that it won't schedule a MAX_RT_PRIO-1
prioritized task for 5 seconds?
The original problem (once I heard and easily reproduced) was there was an
another MAX_RT_PRIO-1 task and the task was spinning in itself by a bug.
(Now this would not be a problem since RLIMIT_RTTIME will work for it, but
I cannot deny that there are some situations which cannot set the limit.)
However there would be more possible problem in the world, ex. assume that
a routine work with interrupt (and also preemption) disabled have an issue
of scalability so it takes long time on huge machine then stop_machine will
stop whole system such long time. You can assume a driver's bug. Now the
stop_machine is good tool to escalate a partial problem to global suddenly.
So I think monotonic wallclock time actually makes the most sense here.
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.
I'm not good at VM etc., but I think user doesn't care who holds a cpu,
whether other guest or actual buggy software or space alien or so.
The important thing here is return control to user if timeout.
Thanks,
H.Seto
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
- Follow-Ups:
- Re: [PATCH] stopmachine: add stopmachine_timeout
- From: Max Krasnyansky
- Re: [PATCH] stopmachine: add stopmachine_timeout
- References:
- [PATCH] stopmachine: add stopmachine_timeout
- From: Hidetoshi Seto
- Re: [PATCH] stopmachine: add stopmachine_timeout
- From: Christian Borntraeger
- Re: [PATCH] stopmachine: add stopmachine_timeout
- From: Rusty Russell
- Re: [PATCH] stopmachine: add stopmachine_timeout
- From: Jeremy Fitzhardinge
- Re: [PATCH] stopmachine: add stopmachine_timeout
- From: Heiko Carstens
- [PATCH] stopmachine: add stopmachine_timeout
- Prev by Date: Re: [PATCH] stopmachine: add stopmachine_timeout
- Next by Date: Re: [stable] Linux 2.6.25.10
- Previous by thread: Re: [PATCH] stopmachine: add stopmachine_timeout
- Next by thread: Re: [PATCH] stopmachine: add stopmachine_timeout
- Index(es):
Relevant Pages
|
Loading