Messages in this thread | ![/](/images/icornerl.gif) | | Date | Tue, 5 Apr 2005 19:25:10 +1000 | From | Dave Airlie <> | Subject | Re: 2.6.12-rc2-mm1 |
| |
> > > Paulus these look like your patches care to update them with the "new" > > method of doing stuff.. > > What are we going to do about the DRM CVS? Change it to the new way > and break everyone running 2.6.10 or earlier, or leave it at the old > way that will work for people with distro kernels, and have a > divergence between it and what's in the kernel?
Yet more backwards compatibility is more than likely going to be needed, but if it is a biggie I'd be happy to drop the older 32/64-bit stuff from CVS and make it contingent on having 2.6.11 or greater as it is a new feature anyways and hasn't seen a release yet...
> > Also, the compat_ioctl method is called without the BKL held, unlike > the ioctl method. What impact will that have? Do we need to take the > BKL in the compat_ioctl method?
I don't think so as the DRM has its own locking that handles most of the issues at a higher level... I've been thinking of switching DRM to ioctl_unlocked but I'd really want someone with an SMP system to beat on it .. (not that the DRI has a great record on SMP anyways..)
Dave.
Dave. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| ![\](/images/icornerr.gif) |