Messages in this thread | | | Date | Mon, 19 Sep 2005 09:28:42 +0200 | From | Ingo Molnar <> | Subject | Re: PATCH: Fix race in cpu_down (hotplug cpu) |
| |
* Nick Piggin <nickpiggin@yahoo.com.au> wrote:
> On Mon, 2005-09-19 at 15:12 +0800, Shaohua Li wrote: > > On Mon, 2005-09-19 at 16:36 +1000, Nick Piggin wrote: > > > > Ah, actually I have a patch which makes all CPU idle threads > > > run with preempt disabled and only enable preempt when scheduling. > > > Would that help? > > It should solve the issue to me. Should we take care of the latency? > > acpi_processor_idle might execute for a long time. > > > > Oh really? I think yes, the latency should be taken care of because we > want to be able to provide good latency even for !preempt kernels. If > a solution can be found for acpi_processor_idle, that would be ideal.
the ACPI idle code runs with irqs disabled anyway, so there's no issue here. If something takes long there, we can do little about it. (but in practice ACPI sleep latencies are pretty ok - the only latencies i found in the past were due to need_resched bugs in the ACPI idle routine)
> IMO it always felt kind of hackish to run the idle threads with > preempt on.
Yes, idle threads can have preemption disabled. There's not any big difference in terms of latencies, the execution paths are all very short.
Ingo - 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/
| |