LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Gabriel C <crazy@frugalware.org>
To: Dave Jones <davej@codemonkey.org.uk>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Gabriel C <nix.or.die@googlemail.com>,
	Dave Airlie <airlied@linux.ie>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Gabriel C <crazy@frugalware.org>,
	Doug Thompson <dougthompson@xmission.com>,
	Greg KH <greg@kroah.com>
Subject: Re: modular intel-agp does not work on my box
Date: Sat, 23 Feb 2008 20:30:31 +0100	[thread overview]
Message-ID: <47C07457.1060606@frugalware.org> (raw)
In-Reply-To: <20080223181414.GA24313@codemonkey.org.uk>

Dave Jones wrote:
> On Sat, Feb 23, 2008 at 12:03:55AM -0800, Andrew Morton wrote:
>  
>  > >  > You are right without EDAC built , agp modular does work fine. I'm on 2.6.25-rc2-00477-g1a4c6be right now.
>  > >  > So it is an EDAC bug ?
>  > >  
>  > > No, it's a failing of the pci driver model. It currently doesn't
>  > > allow more than one driver to be bound to a single PCI device.
>  > > For multi-function devices like bridges, this means we see problems
>  > > like the one you mention.
>  > 
>  > Well that sounds pretty bad.  What will distros do about this?
> 
> either
> *  ship both modular, and let the user decide which one he wants.

If you do so you have to blacklist one by default because udev will magically load both.

> *  build just one of them (typically agp)
> *  build them both, and let the above bug happen

Well the bug was not with both build into the kernel. 
CONFIG_EDAC=Y CONFIG_EDAC_<something>=m, CONFIG_AGP=m CONFIG_AGP_INTEL=m breaks agp already.

> *  build agp into the kernel, and edac modular, breaking edac
>    for a minority.

What does happen if one builds both into the kernel ? ( eg: all the *_AGP* Y , and all the *_EDAC* y ) 

Guessing no one of them will work ?

>  
>  > Is there something short-term-and-sleazy we can do to "fix" it?
> 
> not that I'm aware of.  I think Greg has been working on a long-term fix.
> 
> 	Dave
> 

Gabriel

  reply	other threads:[~2008-02-23 19:30 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-22  1:15 Gabriel C
2008-02-22  2:06 ` Dave Airlie
2008-02-22  2:26   ` Gabriel C
2008-02-22  3:27     ` Gabriel C
2008-02-22  4:19       ` Dave Jones
2008-02-23  8:03         ` Andrew Morton
2008-02-23 18:14           ` Dave Jones
2008-02-23 19:30             ` Gabriel C [this message]
2008-02-23 19:45               ` Dave Jones

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=47C07457.1060606@frugalware.org \
    --to=crazy@frugalware.org \
    --cc=airlied@linux.ie \
    --cc=akpm@linux-foundation.org \
    --cc=davej@codemonkey.org.uk \
    --cc=dougthompson@xmission.com \
    --cc=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nix.or.die@googlemail.com \
    --subject='Re: modular intel-agp does not work on my box' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).