From: Dave Jones <davej@codemonkey.org.uk>
To: Linus Torvalds <torvalds@transmeta.com>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>,
dri-devel@lists.sourceforge.net
Subject: Re: Split AGP GART device lists.
Date: Mon, 2 Dec 2002 17:23:31 +0000 [thread overview]
Message-ID: <20021202172331.GA30694@suse.de> (raw)
In-Reply-To: <Pine.LNX.4.44.0212020850530.13962-100000@home.transmeta.com>
[taking this to the lists, to keep anyone who cares about this in the loop]
This conversation evolved out of my split-agpgart-device-list patch.
Linus' proposal is to take this a step further and split each of
the chipsets into a seperate module. I'm about to tackle this,
but in case there's some hidden gotchas that myself and Linus have
overlooked, I figured I'd give a 'heads up'.
Any comments?
Dave.
On Mon, Dec 02, 2002 at 08:55:24AM -0800, Linus Torvalds wrote:
> > I'm worrying about breaking existing behaviour.
> > X loads /dev/agpgart, which pulls in agpgart.o, but what pulls
> > in via.o, amd.o etc.. ?
>
> Done right, the regular PCI driver detection should load the thing
> automatically without X needing to do anything at all. With the AGP
> drivers showing up with the PCI entries they can drive, all the normal
> auto-loading should just work _without_ having any special cases.
>
> I really think this is worth doing _right_, without stupid (and incorrect)
> module dependencies. Even if it breaks something, it's worth doing:
> people who compile their own kernels can just compile the AGP driver
> statically, the way all sane people - me - do, and people who don't
> compile their own kernels obviously get them from distributions that can
> trivially make modprobe do the right thing.
>
> We get "eth0" behaviour right without having to have some "eth0" driver
> that knows about all the devices that might be networking devices.
> Similarly, we should get agp behaviour right without having to have some
> silly central thing.
--
| Dave Jones. http://www.codemonkey.org.uk
| SuSE Labs
parent reply other threads:[~2002-12-02 17:18 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <Pine.LNX.4.44.0212020850530.13962-100000@home.transmeta.com>]
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=20021202172331.GA30694@suse.de \
--to=davej@codemonkey.org.uk \
--cc=dri-devel@lists.sourceforge.net \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@transmeta.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.