lkml.org 
[lkml]   [2004]   [Aug]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 3/14] kexec: apic-virtwire-on-shutdown.i386.patch
From
Date
"Maciej W. Rozycki" <macro@linux-mips.org> writes:

> On Mon, 23 Aug 2004, Eric W. Biederman wrote:
>
> > The local apic still needs to be put into virtual wire mode in that
> > case.
>
> Well, depending on actual wiring you may need to mask LINT0 in this case
> to avoid duplicate interrupts. I think the safest approach would be
> remembering the initial values of LVT0 and LVT1 registers of the BSP --
> they are just four bytes each, so it would not be a terrible memory waste.

If I was seeing problems I guess I would worry about it. As I have
tested on both Opteron's which require the ioapic to be in
virtual wire mode, and on Xeons which require the local apic to be in
virtual wire mode and both work I am not too concerned.

I don't think the configuration you are worrying about where
both the ioapic and the local apic can both be put into virtual
wire mode is valid according to the mp specification. Although that
is probably a grey area in the specification.

It is more code to save off and then restore the registers,
then simply hard coding a value into them.

It is nice to have hard coded values when you can as then
you don't have to guess how something was configured on this occasion.

As my code is simpler than your suggestion and it works I'm not
a fan of changing it until I see a case where it breaks.

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/

\
 
 \ /
  Last update: 2005-03-22 14:05    [W:0.690 / U:0.016 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site