X.Org 7.3 final: Patch to ignore requests for loading obsolete drm module?

[Stefan Dirsch]

On Sun, Dec 17, 2006 at 10:31:36AM +1100, Dave Airlie wrote:
> On 12/17/06, Michel Dänzer <michel at tungstengraphics.com> wrote:
> >On Sat, 2006-12-16 at 08:36 +1100, Dave Airlie wrote:
> >> On 12/15/06, Stefan Dirsch <sndirsch at suse.de> wrote:
> >> > On Fri, Dec 15, 2006 at 06:39:05PM +1100, Dave Airlie wrote:
> >> > >
> >> > > There is a module blacklist in xf86Config.c for blacklisting the
> >> > > modules from xorg.conf, perhaps you could somehow re-use or unify the
> >> > > two lists?
> >> >
> >> > Hmm ... not sure how this is related to the config file. The fglrx
> >> > driver uses explicitely xf86LoadSubModule() to check if the module can
> >> > be loaded successfully.
> >>
> >> I'm saying perhaps you could merge the blacklists, so we don't have to
> >> maintain two lists of modules that we shouldn't load going forward?
> >
> >There's a difference between modules that can no longer be loaded (such
> >as drm) and modules that should not be loaded from the config file, but
> >still can and need to be loaded on demand (such as GLcore).
> >
> 
> I suppose so, I just wondered, the idea of maintaining two blacklists
> doesn't impress me, and it just seems like putting an fglrx specific
> hack into the X server with no benefit to open source drivers is just
> adding maintainance load going forward... 

My intention was to avoid support load for X.Org developers and Linux
distributors, which ship X.Org 7.2. 

> I've punted the patch to the 7.2 RM (blocker on 6666) 

Thanks.

> I don't think it needs to go into master as
> fglrx won't work with master for lots of other reasons...

I agree.

Best regards,
Stefan

Public Key available
------------------------------------------------------
Stefan Dirsch (Res. & Dev.)   SUSE LINUX Products GmbH
Tel: 0911-740 53 0            Maxfeldstraße 5
FAX: 0911-740 53 479          D-90409 Nürnberg
http://www.suse.de            Germany 
------------------------------------------------------