Re: io_apic & timer_ack fix

From: Jon Grimm
Date: Fri Apr 09 2004 - 10:28:43 EST


Philippe Elie wrote:

On Fri, 09 Apr 2004 at 08:39 +0000, Jon Grimm wrote:



Hmmm....

I see that the following patch got pulled in by Andrew:
http://linux.bkbits.net:8080/linux-2.5/diffs/arch/i386/kernel/io_apic.c@xxxx?nav=index.html|src/|src/arch|src/arch/i386|src/arch/i386/kernel|hist/arch/i386/kernel/io_apic.c

The patch had a couple bugs:
http://seclists.org/lists/linux-kernel/2004/Mar/4152.html

But the patch was pulled out entirely by Linus:
http://linux.bkbits.net:8080/linux-2.5/diffs/arch/i386/kernel/io_apic.c@xxxx?nav=index.html|src/|src/arch|src/arch/i386|src/arch/i386/kernel|hist/arch/i386/kernel/io_apic.c

Was it determined that the fix was bogus? damaging? fixable?



http://marc.theaimsgroup.com/?l=linux-kernel&m=107840458123059&w=2

what's the right fix ? This patch fix timer_ack in three place, the
two last look like typo (spurious ';' after if ()), the first chunk
apparently cause higher temp on some mobo.



I have the spurious ';' removed in my testing.

I ask as I see behavior identical for which this patch seems to have been originally carved up for (buggy SMM BIOS at fault, but this was a workaround in the OS).

http://marc.theaimsgroup.com/?l=linux-kernel&m=101604672921823&w=2
http://www.ussg.iu.edu/hypermail/linux/kernel/0203.2/0698.html

Its a fair answer to force the BIOS vendor to fix, but in the meantime, I'm trying to figure out how safe/unsafe the workaround patch is ? I've ran on it overnight (with the semi-colon's fixed) and it hasn't exhibited the troubling behavior (where timer interrupts seem stuck or in some cases just extremely slow.... and the 8259 IMR is mucked up when Linux isn't even touching anymore).



I agree but actually it cause trouble for non bugged mobo, can this fixed ?




I hope. It looks like a real problem on a box I have.

regards,
Philippe Elie


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