public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Daniel Hazelton <dhazelton@enter.net>
To: Bernhard Kaindl <bk@suse.de>
Cc: linux-pci@atrey.karlin.mff.cuni.cz,
	"Adrian Bunk" <bunk@stusta.de>,
	"Alois Nešpor" <alois.nespor@gmail.com>,
	linux-kernel@vger.kernel.org, gregkh@suse.de
Subject: Re: Ok, lets kill the 'PCI hidden behind bridge' message (was: pci=assign-busses)
Date: Mon, 30 Jul 2007 16:15:44 -0400	[thread overview]
Message-ID: <200707301615.44992.dhazelton@enter.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0707301932440.25292@jbgna.fhfr.qr>

On Monday 30 July 2007 14:35:13 Bernhard Kaindl wrote:
<snip>
> Ok, lets kill the message. As Alois Nešpor also saw, that's fixed up by
> Yenta, so PCI does not have to warn about it. PCI could still warn about it
> if is_cardbus is 0 in that instance of pci_scan_bridge(), but so far I have
> not seen a report where this would have been the case so I think we can
> spare the kernel of that check (removes ~300 lines of asm) unless debugging
> is done.
>
> History: The whole check was added in the days before we had the fixup
> for this in Yenta and pci=assign-busses was the only way to get CardBus
> cards detected on many (not all) of the machines which give this warning.

The message triggers on systems without Cardbus at all.

> In theory, there could be cases when this warning would be triggered and
> it's not cardbus, then the warning should still apply, but I think this
> should only be the case when working on a completely broken PCI setup,
> but one may have already enabled the debug code in drivers/pci and the
> patched check would then trigger.

I wouldn't say totally broken. The last 5 computers I've owned have had this 
message. On only one could it even *remotely* be related to CardBus (my one 
year old laptop), and on one of them it does create problems for certain 
(NVidia) binary drivers. (and then pci=assign-busses doesn't do jack to clear 
the message)

I'm not saying that binary drivers shouldn't be broken - but...
<snip>


DRH

-- 
Dialup is like pissing through a pipette. Slow and excruciatingly painful.

      parent reply	other threads:[~2007-07-30 20:16 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-07-30  8:29 pci=assign-busses Alois Nešpor
2007-07-30 11:53 ` pci=assign-busses Adrian Bunk
2007-07-30 12:24   ` pci=assign-busses Damien Wyart
2007-07-30 18:35   ` Ok, lets kill the 'PCI hidden behind bridge' message (was: pci=assign-busses) Bernhard Kaindl
2007-07-30 20:11     ` Adrian Bunk
2007-07-30 20:14       ` Greg KH
2007-07-30 20:15     ` Daniel Hazelton [this message]

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=200707301615.44992.dhazelton@enter.net \
    --to=dhazelton@enter.net \
    --cc=alois.nespor@gmail.com \
    --cc=bk@suse.de \
    --cc=bunk@stusta.de \
    --cc=gregkh@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@atrey.karlin.mff.cuni.cz \
    /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