Linux PCI subsystem development
 help / color / mirror / Atom feed
From: Clemens Ladisch <clemens@ladisch.de>
To: Stefan Richter <stefanr@s5r6.in-berlin.de>,
	Ram Pai <linuxram@us.ibm.com>, Yinghai Lu <yinghai@kernel.org>
Cc: linux1394-devel@lists.sourceforge.net,
	Rob Allen <ra-kernel@hotmail.co.uk>,
	bugzilla-daemon@bugzilla.kernel.org, linux-pci@vger.kernel.org
Subject: Re: [Bug 43244] New: firewire_ohci prevents boot
Date: Mon, 14 May 2012 08:32:22 +0200	[thread overview]
Message-ID: <4FB0A6F6.9050504@ladisch.de> (raw)
In-Reply-To: <20120513221357.7574ae79@stein>

Stefan Richter wrote:
> On May 13 bugzilla-daemon@bugzilla.kernel.org wrote:
>> https://bugzilla.kernel.org/show_bug.cgi?id=43244
>>
>> firewire_ohci: Failed to set Link Power Status
>>
>> Afterwards, the machine is completely dead (ignoring Magic SysRq), requiring
>> holding the power button to get anywhere.
>>
>> Blacklisting firewire_ohci allows the kernel to boot okay.
>>
>> Having experimented with Ubuntu's mainline kernel builds, it seem to have been
>> introduced between 3.1.0-rc2 (working) and 3.1.0-rc3.
>
> 0a:09.0 CardBus bridge: O2 Micro, Inc. OZ711MP1/MS1 MemoryCardBus Controller (rev 21)
> 0a:09.1 CardBus bridge: O2 Micro, Inc. OZ711MP1/MS1 MemoryCardBus Controller (rev 21)
> 0a:09.4 FireWire (IEEE 1394): O2 Micro, Inc. Firewire (IEEE 1394) (rev 02) (prog-if 10 [OHCI])
>
> $ git shortlog v3.1-rc2..v3.1-rc3 drivers/firewire/
>
> Stefan Richter (3):
>       firewire: cdev: fix 32 bit userland on 64 bit kernel compat corner cases
>       firewire: ohci: fix DMA unmapping in an error path
>       firewire: core: handle ack_busy when fetching the Config ROM
>
> None of that code is in the path that is taken up to and after "failed to
> set Link Power Status".  Hence the bug was most certainly not introduced
> by these but
>   - either outside v3.1-rc2..v3.1-rc3, and the bisection went wrong
>     due to a false negative,
>   - or inside v3.1-rc2..v3.1-rc3 but due to some changes to the kernel
>     or to the kernel configuration that are not directly related to
>     firewire.

These integrated chips tend to have incestuous relationships between their
PCI functions; it's possible that some CardBus change broke things.

And indeed:

$ git shortlog v3.1-rc2..v3.1-rc3 drivers/pci/
...
Ram Pai (3):
      PCI : ability to relocate assigned pci-resources
      PCI: make cardbus-bridge resources optional
      PCI: code and comments cleanup

Yinghai Lu (2):
      PCI: honor child buses add_size in hot plug configuration
      PCI: make SRIOV resources optional


I'm not sure how resource-related changes can result in something that looks
like a power management issue, but maybe the chip doesn't allow register
accesses after its resources are moved, or something like that.  Any ideas?


Regards,
Clemens

       reply	other threads:[~2012-05-14  6:28 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <bug-43244-4803@https.bugzilla.kernel.org/>
     [not found] ` <20120513221357.7574ae79@stein>
2012-05-14  6:32   ` Clemens Ladisch [this message]
2012-05-14  6:39     ` [Bug 43244] New: firewire_ohci prevents boot Yinghai Lu
2012-05-16 13:12       ` Rob Allen
2012-05-16 13:53         ` Clemens Ladisch
2012-05-21 19:33           ` Rob Allen
2012-05-21 20:08             ` Stefan Richter
2012-05-21 20:20             ` Clemens Ladisch
2012-05-24 18:26               ` Stefan Richter
2012-05-24 18:43                 ` Rob Allen
2012-06-04 23:59                   ` Yinghai Lu
     [not found]                 ` <4FBE8146.1030803@hotmail.co.uk>
2012-06-04 11:00                   ` Rob Allen
2012-06-05  2:54                     ` Ram Pai
2012-06-19 21:11                       ` Rob Allen
     [not found]                         ` <BAY153-W382DF41517C8EA770817E4B7EB0@phx.gbl>
2012-07-01 20:56                           ` Yinghai Lu
2012-07-05 19:53                             ` Rob Allen
     [not found]                             ` <4FF5F09C.5020507@hotmail.co.uk>
2012-07-21 11:31                               ` Rob Allen

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=4FB0A6F6.9050504@ladisch.de \
    --to=clemens@ladisch.de \
    --cc=bugzilla-daemon@bugzilla.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux1394-devel@lists.sourceforge.net \
    --cc=linuxram@us.ibm.com \
    --cc=ra-kernel@hotmail.co.uk \
    --cc=stefanr@s5r6.in-berlin.de \
    --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