On Wed, 2003-10-15 at 19:12, Jon Smirl wrote:
> The attached patch adds a new DRM IOCTL: DRM_IOCTL_GET_SUGGESTED_UNIQUE. It
> works by hooking into the OS PCI device detection at module load time. Each DRM
> driver now contains the PCI IDs of all the cards that it supports. The OS then
> matches these to the hardware in the system.  Current Xfree binaries will work
> unmodifed with this change.
> 
> The new call is needed to support standalone Mesa. Standalone Mesa does not
> have the PCI probing code that is in Xfree.

How does standalone Mesa handle video modes etc.? I'm not sure the DRM
is the right place for this, but I certainly think we shouldn't rush to
allocate a precious driver independent ioctl until we're absolutely sure
it's the best solution.


> The patch is against kernel 2.6.0-test7. 

Does it rely on any 2.6 specific APIs? The DRM in DRI CVS must build and
work with 2.4 kernels.

> I've been told that the same code should would with BSD but I don't run it.

I doubt it does in this form (seems to use Linux specific APIs
directly), but it might be easy to fix that.


> @@ -160,14 +176,10 @@
>  #endif
>  
>  /**
> - * The default number of instances (minor numbers) to initialize.
> + * Maximum number of devices supported
>   */
> -#ifndef DRIVER_NUM_CARDS
> -#define DRIVER_NUM_CARDS 1
> -#endif

Does the patch fix all problems with multiple instances in all DRMs?
That was the reason only to allow a single instance by default.


Also, at least the first patch contained some gratuitous whitespace
changes, haven't looked at the second version in detail.


-- 
Earthling Michel Dänzer   \  Debian (powerpc), XFree86 and DRI developer
Software libre enthusiast  \     http://svcs.affero.net/rm.php?r=daenzer



-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
SourceForge.net hosts over 70,000 Open Source Projects.
See the people who have HELPED US provide better services:
Click here: http://sourceforge.net/supporters.php
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to