From: Jun Sun <jsun@mvista.com>
To: "Bradley D. LaRonde" <brad@ltc.com>
Cc: linux-mips@oss.sgi.com, linux-mips-kernel@lists.sourceforge.net
Subject: Re: [Linux-mips-kernel]Re: PATCH: pci_auto bridge support
Date: Mon, 29 Oct 2001 16:19:46 -0800 [thread overview]
Message-ID: <3BDDF222.56105220@mvista.com> (raw)
In-Reply-To: 066201c160d5$eb51ed40$3501010a@ltc.com
"Bradley D. LaRonde" wrote:
>
> ----- Original Message -----
> From: "Jun Sun" <jsun@mvista.com>
> To: "Bradley D. LaRonde" <brad@ltc.com>
> Cc: <linux-mips@oss.sgi.com>; <linux-mips-kernel@lists.sourceforge.net>
> Sent: Monday, October 29, 2001 5:38 PM
> Subject: [Linux-mips-kernel]Re: PATCH: pci_auto bridge support
>
> > "Bradley D. LaRonde" wrote:
> > >
> > > I considered that, but since only this small chuck of run-once surrogate
> > > bios autoconfig code needs to know, I figured better keep it separate.
> > >
> >
> > I would vote to put it inside the hose structure:
> >
> > . It makes a workaround look like a real fix. :-)
> >
> > . In other implementations of pci_auto, hose is the private sys data of a
> pci
> > dev. Having a bus number inside is very useful (e.g., pci_ops can tell
> whether
> > it is type0 of type1 configuration based on the bus number rather than a
> shaky
> > NULL parent bus pointer). In the future, all pci_auto should be combined
> into
> > the pci driver. So that is probably the right direction to go.
> >
> > I think hose may evolve to be the data structure that represents the
> topology
> > of PCI buses. It should have more uses in the future (e.g., the standard
> IRQ
> > routing across PCI-PCI bridges).
>
> Isn't the bus topology already adequately represented in the pci_dev and
> pci_channel structures?
>
Not really. For example, we don't know which bus is the sub-bus of which,
directly, and how their address space translate into each other. Those data
structures are needed when we start to support dynamically mapped and/or
arbitrarily mapped PCI memory spaces.
> I look at the pci autoconfig stuff as a bios replacement. The fact that we
> can use some of the same structures and functions to help us implement it is
> a bonus, but not a mandate to mess with the existing model.
>
That is a right point. I might be too far ahead of myself. :-)
> Isn't Linux already handling PCI-PCI bridges and multiple PCI channles fine
> already, or has our autoconfig code exposed some existing non-arch-specific
> weakness?
>
I think on PCs, P2P still largely depends on BIOS. (Correct me if I am
wrong).
There are some post-scan_bus() mechanism to setup P2P bridges. I have not
looked into it closely.
Jun
prev parent reply other threads:[~2001-10-30 0:19 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-10-27 1:07 PATCH: pci_auto bridge support Bradley D. LaRonde
2001-10-29 19:24 ` Jun Sun
2001-10-29 19:30 ` Bradley D. LaRonde
2001-10-29 19:30 ` Bradley D. LaRonde
2001-10-29 22:38 ` Jun Sun
2001-10-30 0:00 ` [Linux-mips-kernel]Re: " Bradley D. LaRonde
2001-10-30 0:00 ` Bradley D. LaRonde
2001-10-30 0:19 ` Jun Sun [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=3BDDF222.56105220@mvista.com \
--to=jsun@mvista.com \
--cc=brad@ltc.com \
--cc=linux-mips-kernel@lists.sourceforge.net \
--cc=linux-mips@oss.sgi.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 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.