public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Stefan Assmann <sassmann@redhat.com>
To: Bjorn Helgaas <bjorn.helgaas@hp.com>
Cc: Ingo Molnar <mingo@elte.hu>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Jan Beulich <JBeulich@novell.com>, Takashi Iwai <tiwai@suse.de>,
	Junio C Hamano <gitster@pobox.com>,
	linux-acpi@vger.kernel.org, linux-pci@vger.kernel.org,
	Len Brown <lenb@kernel.org>, Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Olaf Dabrunz <Olaf.Dabrunz@gmx.net>
Subject: Re: lost parts of "pci, acpi: reroute PCI interrupt to legacy boot interrupt equivalent" during merge
Date: Wed, 20 Oct 2010 21:54:44 +0200	[thread overview]
Message-ID: <4CBF4904.8050503@redhat.com> (raw)
In-Reply-To: <201010201214.03363.bjorn.helgaas@hp.com>

On 20.10.2010 20:14, Bjorn Helgaas wrote:
> On Wednesday, October 20, 2010 10:28:06 am Stefan Assmann wrote:
>> Let me take a look at the current situation and see if I can come up
>> with a solution. Sorry if things got messy.
> 
> When you redo this, can you update the printks to use dev_info()
> and "[%04x:%04x]" for vendor/device, like the rest of PCI?

Noted.

> 
> Actually, the bridge_has_boot_interrupt_variant() printk looks
> superfluous to me.

That's a leftover from debugging and will be removed.

> 
> Do you know how Windows handles these machines?  I'm just wondering
> if there's some ACPI or other information from the BIOS that we're
> not handling quite correctly, and if we fixed that maybe we wouldn't
> need a quirk.

I have no knowledge about the Windows internals. It might be that
Windows does not mask the interrupt line on the IO-APIC while handling
the interrupt and shuts off the interrupt on the device itself.
Thus no boot interrupt would be generated.
Remember, this problem was first discovered when the RT kernel masked
the interrupt line until the threaded interrupt handler has done its
work.

For your second question, let me point you to
http://lkml.org/lkml/2009/10/19/74
which I posted a while ago, trying to summarize the boot interrupt
problem and how chipset and BIOS developers may avoid it in the future.

In short, yes it can be avoided.However there are already broken
chipsets out there where you simply cannot disable the generation of
boot interrupts if a (non-primary IO-APIC) interrupt line is masked.

> 
> ISTR a paper or some kind of writeup you did, but the commit
> (e1d3a90846) doesn't mention it.  Am I mis-remembering that?

Please see the URL above, it contains references to the writeups we did
at that time. There's also a paper we were working on, but we got
side-tracked and it never reached a state that was publishable.
Apologies for that, we might be able release a stripped down version of
it, but I will have to coordinate this as I'm not the sole author.

Ccing Olaf Dabrunz, as he was very involved in the whole writing and
fixing as well.

> 
> It'd be kind of nice for archaeologists like me if there were a
> kernel bugzilla with before/after dmesg logs and stuff.

Sorry I'm not aware of any.

  Stefan
--
Stefan Assmann         | Red Hat GmbH
Software Engineer      | Otto-Hahn-Strasse 20, 85609 Dornach
                       | HR: Amtsgericht Muenchen HRB 153243
                       | GF: Brendan Lane, Charlie Peters,
sassmann at redhat.com |     Michael Cunningham, Charles Cachera

  reply	other threads:[~2010-10-20 19:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-10-20 10:03 lost parts of "pci, acpi: reroute PCI interrupt to legacy boot interrupt equivalent" during merge Jan Beulich
2010-10-20 13:48 ` Ingo Molnar
2010-10-20 15:15   ` Linus Torvalds
2010-10-20 15:24     ` Ingo Molnar
2010-10-20 16:28       ` Stefan Assmann
2010-10-20 18:14         ` Bjorn Helgaas
2010-10-20 19:54           ` Stefan Assmann [this message]
2010-10-20 21:31             ` Bjorn Helgaas
2010-10-21 18:20             ` Olaf Dabrunz

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=4CBF4904.8050503@redhat.com \
    --to=sassmann@redhat.com \
    --cc=JBeulich@novell.com \
    --cc=Olaf.Dabrunz@gmx.net \
    --cc=bjorn.helgaas@hp.com \
    --cc=gitster@pobox.com \
    --cc=hpa@zytor.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    --cc=tiwai@suse.de \
    --cc=torvalds@linux-foundation.org \
    /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