All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alex Chiang <achiang@hp.com>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: Pierre Ossman <drzeus-list@drzeus.cx>,
	Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>,
	LKML <linux-kernel@vger.kernel.org>,
	linux-pci@vger.kernel.org
Subject: Re: post 2.6.26 requires pciehp_slot_with_bus
Date: Thu, 24 Jul 2008 16:29:14 -0600	[thread overview]
Message-ID: <20080724222914.GG5307@ldl.fc.hp.com> (raw)
In-Reply-To: <200807241506.58973.jbarnes@virtuousgeek.org>

* Jesse Barnes <jbarnes@virtuousgeek.org>:
> On Thursday, July 24, 2008 2:51 pm Pierre Ossman wrote:
> > On Thu, 24 Jul 2008 14:07:18 -0700
> >
> > Jesse Barnes <jbarnes@virtuousgeek.org> wrote:
> > > On Thursday, July 24, 2008 1:39 pm Pierre Ossman wrote:
> > > > This is a laptop with a single expressport that worked
> > > > fine up until this merge window. What changed and why is
> > > > it no longer possible to support this hardware without a
> > > > kernel parameter?
> > >
> > > Yeah we're being a bit more careful about registering
> > > hotplug slots these days.  The fact that you got a conflict
> > > message indicates that more than one driver is trying to
> > > bind to that PCIe port and handle hotplug for it.  I guess
> > > acpiphp must already be loaded?
> >
> > Nope. Only pciehp is builtin, and acpiphp is not loaded.
> > Anything non-hotplug related that can block stuff?
> 
> Theoretically only one of the other hotplug drivers could claim
> the slot (e.g.  acpi, cpci, cpqhp, fakephp etc.).  You could
> put a dump_stack() or something into
> drivers/pci/hotplug/pci_hotplug_core.c:pci_hp_register to see
> if we're getting multiple callers on the same slot for some
> reason...  Maybe the pciehp driver itself is trying to register
> the same slot twice?

This is a good idea.

Any chance you could get that debug info for us, Pierre?

Thanks.

/ac


  reply	other threads:[~2008-07-24 22:29 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-24 11:47 post 2.6.26 requires pciehp_slot_with_bus Pierre Ossman
2008-07-24 12:38 ` Kenji Kaneshige
2008-07-24 20:39   ` Pierre Ossman
2008-07-24 21:07     ` Jesse Barnes
2008-07-24 21:51       ` Pierre Ossman
2008-07-24 22:06         ` Jesse Barnes
2008-07-24 22:29           ` Alex Chiang [this message]
2008-07-24 22:49             ` Pierre Ossman
2008-07-24 23:08               ` Alex Chiang
2008-07-24 23:29                 ` Pierre Ossman
2008-07-25  3:29                   ` Matthew Wilcox
2008-07-25  4:42                     ` Alex Chiang
2008-07-25  5:38                       ` Kenji Kaneshige
2008-07-25 11:18                         ` Matthew Wilcox
2008-07-28 18:05                       ` Greg KH
2008-07-25  4:57                     ` Kenji Kaneshige
2008-07-30  2:38                       ` Alex Chiang
2008-07-30  2:42                         ` [PATCH 1/2] pciehp: Rename duplicate slot name N as N-1, N-2, N-M Alex Chiang
2008-07-31 10:32                           ` Kenji Kaneshige
2008-07-30  2:44                         ` [PATCH 2/2] shpchp: " Alex Chiang
2008-07-31 10:32                           ` Kenji Kaneshige
2008-07-31 10:31                         ` post 2.6.26 requires pciehp_slot_with_bus Kenji Kaneshige
2008-07-31 15:47                           ` Alex Chiang
2008-08-01  8:43                             ` Kenji Kaneshige
2008-07-25  8:53                     ` Kenji Kaneshige
2008-07-25 11:40                       ` Matthew Wilcox
2008-07-28  7:21                         ` Kenji Kaneshige
2008-07-25  4:50                   ` Kenji Kaneshige
2008-07-25 22:18                     ` Jesse Barnes
2008-07-26  1:16                       ` Matthew Wilcox
2008-07-28  8:58                         ` Kenji Kaneshige
2008-07-28  8:44                       ` Kenji Kaneshige
2008-07-28 16:16                         ` Jesse Barnes
2008-07-29  2:43                           ` Kenji Kaneshige
2008-07-29 15:14                             ` Jesse Barnes
2008-07-30  2:44                               ` Kenji Kaneshige
2008-07-28 16:57                         ` Matthew Wilcox

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=20080724222914.GG5307@ldl.fc.hp.com \
    --to=achiang@hp.com \
    --cc=drzeus-list@drzeus.cx \
    --cc=jbarnes@virtuousgeek.org \
    --cc=kaneshige.kenji@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.