drm/auth: Drop master_create/destroy hooks

vmwgfx stopped using them.

With the drm device model that we've slowly evolved over the past few
years master status essentially controls access to display resources,
and nothing else. Since that's a pure access permission check drivers
should have no need at all to track additional state on a per file
basis.

Aside: For cleanup and restoring kernel-internal clients the grand
plan is to move everyone over to drm_client and
drm_master_internal_acquire/release, like the generic fbdev code
already does. That should get rid of most ->lastclose implementations,
and I think also subsumes any processing vmwgfx does in
master_set/drop.

Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Thomas Hellstrom <thellstrom@vmware.com>
Cc: "Thomas Hellström (VMware)" <thomas_os@shipmail.org>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200127100203.1299322-1-daniel.vetter@ffwll.ch

diff --git a/drivers/gpu/drm/drm_auth.c b/drivers/gpu/drm/drm_auth.c
index cc9acd986c68f68c79add713e4b7dfd66a967b35..531b876d0ed83c7ce9e4419c309b446ada63f9d7 100644 (file)
--- a/drivers/gpu/drm/drm_auth.c
+++ b/drivers/gpu/drm/drm_auth.c
@@ -153,11 +153,6 @@ static int drm_new_set_master(struct drm_device *dev, struct drm_file *fpriv)
                return -ENOMEM;
        }
 
-       if (dev->driver->master_create) {
-               ret = dev->driver->master_create(dev, fpriv->master);
-               if (ret)
-                       goto out_err;
-       }
        fpriv->is_master = 1;
        fpriv->authenticated = 1;
 
@@ -332,9 +327,6 @@ static void drm_master_destroy(struct kref *kref)
        if (drm_core_check_feature(dev, DRIVER_MODESET))
                drm_lease_destroy(master);
 
-       if (dev->driver->master_destroy)
-               dev->driver->master_destroy(dev, master);
-
        drm_legacy_master_rmmaps(dev, master);
 
        idr_destroy(&master->magic_map);

diff --git a/include/drm/drm_drv.h b/include/drm/drm_drv.h
index 23b636691fb449b48917469a3501186cd35f37e3..77685ed7aa653a2225df014a213a498e053109bf 100644 (file)
--- a/include/drm/drm_drv.h
+++ b/include/drm/drm_drv.h
@@ -457,20 +457,6 @@ struct drm_driver {
         */
        void (*irq_uninstall) (struct drm_device *dev);
 
-       /**
-        * @master_create:
-        *
-        * Called whenever a new master is created. Only used by vmwgfx.
-        */
-       int (*master_create)(struct drm_device *dev, struct drm_master *master);
-
-       /**
-        * @master_destroy:
-        *
-        * Called whenever a master is destroyed. Only used by vmwgfx.
-        */
-       void (*master_destroy)(struct drm_device *dev, struct drm_master *master);
-
        /**
         * @master_set:
         *