Re: smp_send_stop() and disable_local_APIC()

From: Eric W. Biederman (ebiederm@xmission.com)
Date: Fri May 04 2001 - 10:58:04 EST


"Matt D. Robinson" <yakker@alacritech.com> writes:

> It looks like around 2.3.30 or so, someone added the call
> disable_local_APIC() to smp_send_stop(). I'm not sure what the
> intention was, but I'm getting some strange behavior as a result
> based on some code I'm writing.
>
> Basically, I'm doing the following ...
>
> panic()
> {
> /* do whatever you want, notifier list, etc. */
> smp_send_stop();
> write_system_memory();
> /* then do whatever */
> }
>
> write_system_memory() does a write of all system memory pages to some
> block device. It uses kiobufs as the way to get the pages to disk,
> doing brw_kiovec() on those pages (using either the IDE or SCSI
> driver to write the data).

IDE being less likely to hang than SCSI as it tends to use legacy isa
interrupt lines.
 
> The wierd behavior I see is that sometimes, smp_send_stop()
> being called causes the system to hang up (not every time).

Doing event driver i/o after disabling the interrupt controller
hmm, I wonder why...

> If we don't call smp_send_stop() on those systems, everything works fine.
> This looks to be directly caused by the disabling of the APIC, which
> we may need to dump pages to local disk. This only applies to some
> people's systems -- not everyone displays the same behavior.
>
> I'm sure it's good to disable the APIC, but there's no clean way to
> wait on disabling the APIC until after I'm done writing pages out.
>
> My questions are:
>
> 1) Why was disable_local_APIC() added to stop_this_cpu()
> and smp_send_stop()? Completeness?
>
> 2) Is there a better way around this to disable all the
> other CPUs without disabling the APIC?
>

I don't know what a good way is, since there is a kernel panic it
should only be something truly fatal. Given that reusing anything
that hasn't been designed to run in that situation is playing with
fire.

Eric
-
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/



This archive was generated by hypermail 2b29 : Mon May 07 2001 - 21:00:20 EST