linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bjorn Helgaas <bhelgaas@google.com>
To: Martin Mokrejs <mmokrejs@fold.natur.cuni.cz>
Cc: Yinghai Lu <yinghai@kernel.org>,
	"Rafael J. Wysocki" <rjw@sisk.pl>,
	"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
	Sarah Sharp <sarah.a.sharp@linux.intel.com>,
	Alan Stern <stern@rowland.harvard.edu>
Subject: Re: 3.8.2: stale pci device info for a previously inserted express card
Date: Fri, 10 May 2013 11:47:31 -0600	[thread overview]
Message-ID: <CAErSpo5594KNtZz9Gpdw_j5NZX_sGpzKB9CT2-SCB2bCASfrSQ@mail.gmail.com> (raw)
In-Reply-To: <513E002C.40805@fold.natur.cuni.cz>

[+cc linux-pci, Sarah, Alan]

On Mon, Mar 11, 2013 at 10:02 AM, Martin Mokrejs
<mmokrejs@fold.natur.cuni.cz> wrote:
> [re-sending to you all three directly, looks the original email did not make it into linux-pci
> through some filters]
>
>   I use for my daily work acpiphp to manage express cards in Dell Vostro 3550.
> I have never seen something like this before and believe this is some new regression
> in 3.8 series. I had in teh a USB3 card and ejected it. Then I inserted a
> SATA Sil3132 card but it is not detected and dmesg still ends with last lines
> added when the USB card was being removed. The funny thing is that lspci reports
> a mixture of USB-card properties with NEC chips along with Silicon Image eSATA card.

I don't know anything about the kmemleaks mentioned elsewhere in this
thread, but the idea of "stale PCI device info" seems possibly related
to some acpiphp issues we've been working on recently.

Starting with v3.9, we don't handle ACPI Bus Check notifications to
host bridges correctly, and the result is that when we're using
acpiphp, we don't notice when PCI devices are added or removed. There
are more details in https://bugzilla.kernel.org/show_bug.cgi?id=57961

It looks like your Vostro 3550 uses acpiphp, so maybe there's a
connection.  Can you try a current Linus upstream kernel (v3.10-rc1
should be released in the next day or two) and apply the v3.10 patch
from the bugzilla, and collect a complete dmesg log after you try a
couple hotplug events?

Bjorn

       reply	other threads:[~2013-05-10 17:47 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <513E002C.40805@fold.natur.cuni.cz>
2013-05-10 17:47 ` Bjorn Helgaas [this message]
2013-06-07 11:20   ` 3.8.2: stale pci device info for a previously inserted express card Martin Mokrejs
2013-06-07 11:27     ` Martin Mokrejs
2013-06-07 15:55       ` Martin Mokrejs
     [not found] <513E02B7.4080900@fold.natur.cuni.cz>
2013-03-11 16:35 ` Alan Stern
     [not found] <513DD059.5030604@fold.natur.cuni.cz>
2013-03-11 15:26 ` Alan Stern
2013-03-11 15:48   ` Martin Mokrejs
2013-03-11 16:05     ` Alan Stern
2013-03-11 23:11   ` Sarah Sharp
2013-03-12  2:25     ` Alan Stern

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=CAErSpo5594KNtZz9Gpdw_j5NZX_sGpzKB9CT2-SCB2bCASfrSQ@mail.gmail.com \
    --to=bhelgaas@google.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=mmokrejs@fold.natur.cuni.cz \
    --cc=rjw@sisk.pl \
    --cc=sarah.a.sharp@linux.intel.com \
    --cc=stern@rowland.harvard.edu \
    --cc=yinghai@kernel.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;
as well as URLs for NNTP newsgroup(s).