From: "Martin J. Bligh" <Martin.Bligh@us.ibm.com>
To: William Lee Irwin III <wli@holomorphy.com>, Greg KH <greg@kroah.com>
Cc: linux-kernel@vger.kernel.org, colpatch@us.ibm.com
Subject: Re: [BUG] 2.5.33 PCI and/or starfire.c broken
Date: Tue, 03 Sep 2002 22:37:20 -0700 [thread overview]
Message-ID: <102020607.1031092638@[10.10.2.3]> (raw)
In-Reply-To: <20020904051200.GX888@holomorphy.com>
> It's a wee bit less brain-damaged than crackly sound cards with
> only 24 out of 32 address lines wired.
True ;-)
> At any rate, the issue is in all
> likelihood that that support file is being circumvented.
Those functions that need alteration aren't "switched" between
different arches like that. The patch just hardcodes arch specific
crud into general code, and would never be submittable. There was
some clean way to fix it suggested, but I forget what it was at
the moment ... will work it out again with Greg.
> It's obviously
> getting wrong information when it asks to read from a given bus as
> there are no PCI devices off of node 0 (as this hasn't worked in a long
> time), so perhaps a disassembly or other fishing around will reveal
> whose pci config space accessors are actually being called here.
Does it really matter who's registers we're reading when using
pointers we know are corrupted garbage? Far better to fix the
pointers than fuss around working what they ended up pointing
at this spin of the bottle ...
IIRC, it's getting mislead by the PCI-PCI bridge itself, which is
always returning the local bus number of 3 no matter which quad
it's on. It's not the read operations themselves that are borked
(this time), it's the data that the bridge is returning is being
"misinterpreted".
M.
prev parent reply other threads:[~2002-09-04 5:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-09-04 3:56 William Lee Irwin III
2002-09-04 4:49 ` Martin J. Bligh
2002-09-04 5:04 ` Greg KH
2002-09-04 5:10 ` Martin J. Bligh
2002-09-04 5:12 ` William Lee Irwin III
2002-09-04 5:37 ` Martin J. Bligh [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='102020607.1031092638@[10.10.2.3]' \
--to=martin.bligh@us.ibm.com \
--cc=colpatch@us.ibm.com \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=wli@holomorphy.com \
/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
all inboxes | Powered by JetHome®