LKML Archive on
help / color / mirror / Atom feed
From: (Eric W. Biederman)
To: Andrew Morton <>
Cc: Len Brown <>, <>,
	Thomas Meyer <>,
	Pekka Enberg <>,
	Jeff Garzik <>, <>
Subject: Re: Fw: Re: Bug in pci_restore_msi_state
Date: Thu, 22 Mar 2007 07:24:12 -0600	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <> (Andrew Morton's message of "Wed, 21 Mar 2007 23:53:13 -0800")

I couldn't find the original thread so I am restarting one adding I hope
the appropriate people to the cc list.

Andrew thanks for forwarding this one. I finally see enough of this to
see what is going on.

The question is about the following sequence from libata.


Should this be allowed?

Specifically should pci_disable_device() be allowed when we have
device resources requested and potentially in use.

Specifically should pci_enable_device() be expected to cope with the
case where we are already using some a pci devices resources?


If this sequence is sane pci_enable_device needs to not reset dev->irq
In this case I guess it is acpi_pci_irq_enable doing that.

If this sequence is not sane we can just put outlaw calling
pci_disable_device when msi is enabled and fix it that way.

The suggestion below of setting msi_enabled to 0.  While it will avoid
the immediate problem, leaves us still having requested the msi
irq (which won't work now) and so we won't be in a state where we can
do useful things after resume, plus we would wind up leaking the
msi state.

I think fixing pci_enable_device to cope when a device is already in
use is likely to be the path of least resistance.

If suspend to disk needs to cope with any pci resource assignment
changes then we need to free all irqs and all pci bars before the
suspend and reallocate them after the resume, and we should outlaw
pci_disable_device with pci device resources in use.


Andrew Morton <> writes:

> Begin forwarded message:
> Date: Tue, 20 Mar 2007 22:07:59 +0100
> From: Thomas Meyer <>
> To: Linux Kernel Mailing List <>
> Cc: Pekka Enberg <>
> Subject: Re: Bug in pci_restore_msi_state
> Pekka Enberg schrieb:
>> On 3/17/07, Thomas Meyer <> wrote:
>>> Hello everybody.
>>> I get this bug after suspending to disk twice:
>>> This happens with current git head
>>> cd05a1f818073a623455a58e756c5b419fc98db9.
>> If you know a kernel that works, please consider doing git bisect:
> Ok. The error happens in the call of pcibios_enable_device in the
> function do_pci_enable_device (drivers/pci/pci.c):
> Before the call of pcibios_enable_device:
> do_pci_enable_device: pci_dev= c1a40000
> do_pci_enable_device: irq= 218
> do_pci_enable_device: msi_enabled= 1
> after the call:
> ACPI: PCI Interrupt 0000:00:1f.2[B] -> GSI 19 (level, low) -> IRQ 19
> do_pci_enable_device: pci_dev= c1a40000
> do_pci_enable_device: irq= 19
> do_pci_enable_device: msi_enabled= 1
> So the function acpi_pci_irq_enable in drivers/acpi/pci_irq.c  needs to
> be fixed:
> a.) acpi_pci_irq_enable should care about msi setups and set the flag
> msi_enabled to 0 or
> b.) acpi_pci_irq_enable should care about msi setups and should touch
> the dev->irq field.
> Please comment on this.
> with kind regards
> thomas

           reply	other threads:[~2007-03-22 13:26 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <>]

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:

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

  git send-email \ \ \ \ \ \ \ \ \ \
    --subject='Re: Fw: Re: Bug in pci_restore_msi_state' \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).