FreeBSD: There is no compelling reason to hold the lock here.

When I was lock profiling, this was high up on the list and I
see no reason to do it.

diff --git a/bsd-core/drm_drv.c b/bsd-core/drm_drv.c
index afcad19..fc8e814 100644 (file)
--- a/bsd-core/drm_drv.c
+++ b/bsd-core/drm_drv.c
@@ -651,9 +651,7 @@ int drm_ioctl(struct cdev *kdev, u_long cmd, caddr_t data, int flags,
        int is_driver_ioctl = 0;
        struct drm_file *file_priv;
 
-       DRM_LOCK();
        retcode = devfs_get_cdevpriv((void **)&file_priv);
-       DRM_UNLOCK();
        if (retcode != 0) {
                DRM_ERROR("can't find authenticator\n");
                return EINVAL;