public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: "Maciej W. Rozycki" <macro@linux-mips.org>
Cc: Ingo Molnar <mingo@elte.hu>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	linux-next@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
	Len Brown <lenb@kernel.org>
Subject: Re: linux-next: Tree for June 13: IO APIC breakage on HP nx6325
Date: Thu, 19 Jun 2008 02:25:50 +0200	[thread overview]
Message-ID: <200806190225.51088.rjw@sisk.pl> (raw)
In-Reply-To: <Pine.LNX.4.55.0806182345380.7802@cliff.in.clinika.pl>

On Thursday, 19 of June 2008, Maciej W. Rozycki wrote:
> On Thu, 19 Jun 2008, Rafael J. Wysocki wrote:
> 
> > >  With such a configuration the "x86: I/O APIC: timer through 8259A
> > > second-chance" patch should not matter, because the only change it
> > > introduces is an attempt to try the same I/O APIC pin again, but with the
> > > IRQ0 line of the master 8259A enabled.  That's not a terribly unusual 
> > > configuration and nothing should get confused in the system.
> > 
> > But it _does_ get confused, really.
> 
>  Something certainly gets confused, but so far I am not sure which bit 
> exactly it is, are you?

No, I'm not.

> > >  Barring the unlikely possibility of the 8259A actually being wired to 
> > > INTIN2 of the I/O APIC I can see two possible explanations:
> > > 
> > > 1. The 8259A interrupt actually escapes to the CPU somehow and is handled
> > >    as an ExtINTA interrupt.  This would make the code in check_timer()  
> > >    decide it has found a working configuration, while actually it has been
> > >    fooled.
> [...]
> > Here you go:
> > 
> > [    0.108006] ..TIMER: vector=0x30 apic1=0 pin1=2 apic2=-1 pin2=-1
> > [    0.108006] ..MP-BIOS bug: 8254 timer not connected to IO-APIC
> > [    0.108006] ...trying to set up timer (IRQ0) through the 8259A ... <3>
> > [    0.108006] ..... (found apic 0 pin 2) ...<3> works.
> > 
> > The full dmesg is at: http://www.sisk.pl/kernel/debug/20080618/dmesg-1.log
> 
>  Thanks.  In this case I suspect the case #1 quoted above happens, that is
> the 8259A manages to deliver its interrupt somehow.  Note at this stage it
> is meant to be in the AEOI mode, so it can happily resubmit the interrupt
> indefinitely with no additional handling as long as it receives INTA
> cycles.
> 
>  Can you please try the patch below on top of "x86: I/O APIC: timer
> through 8259A second-chance" to see whether my hypothesis is true?  It
> modifies the through-8259A setup path so that the APIC input gets masked,
> but the 8259A has the timer interrupt still enabled.  Let me know how the
> timer interrupt is routed in this case.

That helped a lot, the system seems to work normally now.

Here's the relevant snippet from dmesg:

[    0.108006] ..TIMER: vector=0x30 apic1=0 pin1=2 apic2=-1 pin2=-1
[    0.108006] ..MP-BIOS bug: 8254 timer not connected to IO-APIC
[    0.108006] ...trying to set up timer (IRQ0) through the 8259A ... <3>
[    0.108006] ..... (found apic 0 pin 2) ...<3> failed.
[    0.108006] ...trying to set up timer as Virtual Wire IRQ...<3> works.

and the whole thing is at: http://www.sisk.pl/kernel/debug/20080618/dmesg-2.log
 
>  BTW, do we have any piece of technical information about the chipset
> used?

I, personally, don't have any and AMD only has SB600 documentation on its
web page (it's still marked as "AMD confidential" ;-)).

> The southbridge used is an ATI SB400, which is where I would 
> normally expect two 8259A and an I/O APIC core to be placed.

There is an interrupt controller in there, but I'm not sure if there's any
8259A.  The northbridge is on the CPU, actually.

Thanks,
Rafael

  reply	other threads:[~2008-06-19  0:25 UTC|newest]

Thread overview: 73+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080613232214.394fd6fd.sfr@canb.auug.org.au>
     [not found] ` <200806161539.05524.rjw@sisk.pl>
     [not found]   ` <Pine.LNX.4.55.0806161636510.20218@cliff.in.clinika.pl>
2008-06-16 22:38     ` linux-next: Tree for June 13: IO APIC breakage on HP nx6325 Rafael J. Wysocki
2008-06-16 23:05       ` Rafael J. Wysocki
2008-06-17  7:12         ` Thomas Gleixner
2008-06-17 20:44           ` Rafael J. Wysocki
2008-06-17 22:19             ` Rafael J. Wysocki
2008-06-17 22:25               ` Rafael J. Wysocki
2008-06-18  8:02                 ` Thomas Gleixner
2008-06-18 12:41                   ` Thomas Gleixner
2008-06-18 14:37                     ` Rafael J. Wysocki
2008-06-18 14:40                     ` Rafael J. Wysocki
2008-06-18 15:29                       ` Thomas Gleixner
2008-06-21 22:47                         ` Rafael J. Wysocki
2008-06-18 13:15                 ` Ingo Molnar
2008-06-18 13:14               ` Ingo Molnar
2008-06-17 20:59       ` Rafael J. Wysocki
2008-06-17 21:19         ` Maciej W. Rozycki
2008-06-17 21:38           ` Rafael J. Wysocki
2008-06-17 22:53             ` Rafael J. Wysocki
2008-06-18  4:02               ` Maciej W. Rozycki
2008-06-18 19:06                 ` Cyrill Gorcunov
2008-06-18 22:36                   ` Maciej W. Rozycki
2008-06-20 18:59                     ` Cyrill Gorcunov
2008-06-20 20:44                       ` Maciej W. Rozycki
2008-06-18 22:11                 ` Rafael J. Wysocki
2008-06-18 23:39                   ` Maciej W. Rozycki
2008-06-19  0:25                     ` Rafael J. Wysocki [this message]
2008-06-20  0:35                       ` Maciej W. Rozycki
2008-06-20 11:53                         ` Rafael J. Wysocki
2008-06-20 11:57                           ` Matthew Garrett
2008-06-20 12:22                             ` Rafael J. Wysocki
2008-06-20 12:27                               ` Matthew Garrett
2008-06-21  1:09                                 ` Maciej W. Rozycki
2008-06-21  1:40                                   ` Matthew Garrett
2008-06-21  2:41                                     ` Maciej W. Rozycki
2008-06-21 12:38                                       ` Matthew Garrett
2008-06-26 19:52                                     ` Rafael J. Wysocki
2008-06-27  0:06                                       ` Maciej W. Rozycki
2008-06-29 14:00                                         ` Rafael J. Wysocki
2008-06-29 19:05                                           ` Maciej W. Rozycki
2008-06-29 19:23                                             ` Rafael J. Wysocki
2008-06-29 19:56                                               ` Maciej W. Rozycki
2008-06-29 20:02                                                 ` Ingo Molnar
2008-06-29 20:14                                                   ` Maciej W. Rozycki
2008-06-29 23:06                                                     ` Rafael J. Wysocki
2008-06-30  0:45                                                       ` Andi Kleen
2008-06-30  0:47                                                         ` Matthew Garrett
2008-06-30  1:39                                                       ` Maciej W. Rozycki
2008-06-30  9:24                                                         ` Andi Kleen
2008-07-02  1:19                                                           ` Maciej W. Rozycki
2008-06-30 10:41                                                         ` Rafael J. Wysocki
2008-07-02  1:48                                                           ` Maciej W. Rozycki
2008-07-02  9:35                                                             ` Andi Kleen
2008-06-29 22:59                                                   ` Rafael J. Wysocki
2008-06-29 22:56                                                 ` Rafael J. Wysocki
2008-06-30  1:00                                                   ` Maciej W. Rozycki
2008-06-30  9:06                                                     ` Matthew Garrett
2008-06-30 15:29                                                       ` Maciej W. Rozycki
2008-06-30 15:35                                                         ` Matthew Garrett
2008-06-29 19:23                                             ` Matthew Garrett
2008-06-29 19:31                                               ` Rafael J. Wysocki
2008-06-29 20:03                                               ` Maciej W. Rozycki
2008-06-29 20:07                                                 ` Matthew Garrett
2008-06-29 20:16                                                   ` Maciej W. Rozycki
2008-06-24  9:15                               ` Pavel Machek
2008-06-26  8:37                                 ` Rafael J. Wysocki
2008-06-27  1:53                                 ` Maciej W. Rozycki
2008-07-08 12:48                                   ` Pavel Machek
2008-06-21  1:49                           ` Maciej W. Rozycki
2008-06-19  9:35                     ` Ingo Molnar
2008-06-19 18:17                       ` Maciej W. Rozycki
2008-06-20 10:44                         ` Ingo Molnar
2008-06-20 13:11                         ` Thomas Gleixner
2008-06-20 20:56                           ` Maciej W. Rozycki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200806190225.51088.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=macro@linux-mips.org \
    --cc=mingo@elte.hu \
    --cc=sfr@canb.auug.org.au \
    --cc=tglx@linutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox