On Mon, Mar 01, 2010 at 08:19:02AM -0800, Eric Anholt wrote: > On Mon, 1 Mar 2010 18:33:14 +1000, Dave Airlie <airl...@gmail.com> wrote: > > On Sun, Feb 28, 2010 at 1:30 PM, Ben Hutchings <b...@decadent.org.uk> wrote: > > > This addresses <http://bugzilla.kernel.org/show_bug.cgi?id=15021>. > > > > > > DRM drivers may fail to probe their devices when the associated AGP > > > GART does not yet have a driver, so we reprobe unbound VGA devices > > > when a GART device is added. > > > > > > Signed-off-by: Ben Hutchings <b...@decadent.org.uk> > > > --- > > > This is intended to address the dependency problem highlighted in the > > > above bug report. That specific bug can be fixed by adding a module > > > dependency from i915 to intel_agp, but in general there is no fixed > > > relationship betweem DRM and GART drivers. > > > > > > There is a narrow race between the check for a current driver binding > > > and the call to device_reprobe(). This could be closed by adding a > > > variant on device_attach() and device_reprobe() that is a no-op for > > > bound devices. > > > > > > > This isn't useful, generally there is no AGP binding, and most drivers > > if they can't find an AGP backend will still run fine without it. i.e. > > radeon, > > mga etc.
I see, only the Intel GPU drivers set DRIVER_REQUIRE_AGP. > > Intel is a special case and I think we've already merged an explicit > > depend. > > > > Just build agp drivers into the kernel, I'm tempted to make them all > > non-modular. > > That seems easier. Easier, yes, but it's a fair amount of bloat for i386 kernels (less so for x86-64) since there are many different GART drivers. I was hoping to avoid that. Ben. -- Ben Hutchings Life is like a sewer: what you get out of it depends on what you put into it. ------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
