xref: /linux/drivers/gpu/drm/drm_lease.c (revision a0c1af46e9d71fdae7df904b91542ce2518382a1)
12ed077e4SKeith Packard /*
22ed077e4SKeith Packard  * Copyright © 2017 Keith Packard <keithp@keithp.com>
32ed077e4SKeith Packard  *
42ed077e4SKeith Packard  * This program is free software; you can redistribute it and/or modify
52ed077e4SKeith Packard  * it under the terms of the GNU General Public License as published by
62ed077e4SKeith Packard  * the Free Software Foundation, either version 2 of the License, or
72ed077e4SKeith Packard  * (at your option) any later version.
82ed077e4SKeith Packard  *
92ed077e4SKeith Packard  * This program is distributed in the hope that it will be useful, but
102ed077e4SKeith Packard  * WITHOUT ANY WARRANTY; without even the implied warranty of
112ed077e4SKeith Packard  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
122ed077e4SKeith Packard  * General Public License for more details.
132ed077e4SKeith Packard  */
142ed077e4SKeith Packard 
152ed077e4SKeith Packard #include <drm/drmP.h>
162ed077e4SKeith Packard #include "drm_internal.h"
172ed077e4SKeith Packard #include "drm_legacy.h"
182ed077e4SKeith Packard #include "drm_crtc_internal.h"
192ed077e4SKeith Packard #include <drm/drm_lease.h>
202ed077e4SKeith Packard #include <drm/drm_auth.h>
212ed077e4SKeith Packard #include <drm/drm_crtc_helper.h>
222ed077e4SKeith Packard 
232ed077e4SKeith Packard #define drm_for_each_lessee(lessee, lessor) \
242ed077e4SKeith Packard 	list_for_each_entry((lessee), &(lessor)->lessees, lessee_list)
252ed077e4SKeith Packard 
2662884cd3SKeith Packard static uint64_t drm_lease_idr_object;
2762884cd3SKeith Packard 
282ed077e4SKeith Packard /**
292ed077e4SKeith Packard  * drm_lease_owner - return ancestor owner drm_master
302ed077e4SKeith Packard  * @master: drm_master somewhere within tree of lessees and lessors
312ed077e4SKeith Packard  *
322ed077e4SKeith Packard  * RETURN:
332ed077e4SKeith Packard  *
342ed077e4SKeith Packard  * drm_master at the top of the tree (i.e, with lessor NULL
352ed077e4SKeith Packard  */
362ed077e4SKeith Packard struct drm_master *drm_lease_owner(struct drm_master *master)
372ed077e4SKeith Packard {
382ed077e4SKeith Packard 	while (master->lessor != NULL)
392ed077e4SKeith Packard 		master = master->lessor;
402ed077e4SKeith Packard 	return master;
412ed077e4SKeith Packard }
422ed077e4SKeith Packard 
432ed077e4SKeith Packard /**
442ed077e4SKeith Packard  * _drm_find_lessee - find lessee by id (idr_mutex held)
452ed077e4SKeith Packard  * @master: drm_master of lessor
467591844cSChris Wilson  * @lessee_id: id
472ed077e4SKeith Packard  *
482ed077e4SKeith Packard  * RETURN:
492ed077e4SKeith Packard  *
502ed077e4SKeith Packard  * drm_master of the lessee if valid, NULL otherwise
512ed077e4SKeith Packard  */
522ed077e4SKeith Packard 
532ed077e4SKeith Packard static struct drm_master*
542ed077e4SKeith Packard _drm_find_lessee(struct drm_master *master, int lessee_id)
552ed077e4SKeith Packard {
562ed077e4SKeith Packard 	lockdep_assert_held(&master->dev->mode_config.idr_mutex);
572ed077e4SKeith Packard 	return idr_find(&drm_lease_owner(master)->lessee_idr, lessee_id);
582ed077e4SKeith Packard }
592ed077e4SKeith Packard 
602ed077e4SKeith Packard /**
612ed077e4SKeith Packard  * _drm_lease_held_master - check to see if an object is leased (or owned) by master (idr_mutex held)
622ed077e4SKeith Packard  * @master: the master to check the lease status of
632ed077e4SKeith Packard  * @id: the id to check
642ed077e4SKeith Packard  *
652ed077e4SKeith Packard  * Checks if the specified master holds a lease on the object. Return
662ed077e4SKeith Packard  * value:
672ed077e4SKeith Packard  *
682ed077e4SKeith Packard  *	true		'master' holds a lease on (or owns) the object
692ed077e4SKeith Packard  *	false		'master' does not hold a lease.
702ed077e4SKeith Packard  */
712ed077e4SKeith Packard static int _drm_lease_held_master(struct drm_master *master, int id)
722ed077e4SKeith Packard {
732ed077e4SKeith Packard 	lockdep_assert_held(&master->dev->mode_config.idr_mutex);
742ed077e4SKeith Packard 	if (master->lessor)
752ed077e4SKeith Packard 		return idr_find(&master->leases, id) != NULL;
762ed077e4SKeith Packard 	return true;
772ed077e4SKeith Packard }
782ed077e4SKeith Packard 
792ed077e4SKeith Packard /**
802ed077e4SKeith Packard  * _drm_has_leased - check to see if an object has been leased (idr_mutex held)
812ed077e4SKeith Packard  * @master: the master to check the lease status of
822ed077e4SKeith Packard  * @id: the id to check
832ed077e4SKeith Packard  *
842ed077e4SKeith Packard  * Checks if any lessee of 'master' holds a lease on 'id'. Return
852ed077e4SKeith Packard  * value:
862ed077e4SKeith Packard  *
872ed077e4SKeith Packard  *	true		Some lessee holds a lease on the object.
882ed077e4SKeith Packard  *	false		No lessee has a lease on the object.
892ed077e4SKeith Packard  */
902ed077e4SKeith Packard static bool _drm_has_leased(struct drm_master *master, int id)
912ed077e4SKeith Packard {
922ed077e4SKeith Packard 	struct drm_master *lessee;
932ed077e4SKeith Packard 
942ed077e4SKeith Packard 	lockdep_assert_held(&master->dev->mode_config.idr_mutex);
952ed077e4SKeith Packard 	drm_for_each_lessee(lessee, master)
962ed077e4SKeith Packard 		if (_drm_lease_held_master(lessee, id))
972ed077e4SKeith Packard 			return true;
982ed077e4SKeith Packard 	return false;
992ed077e4SKeith Packard }
1002ed077e4SKeith Packard 
1012ed077e4SKeith Packard /**
1022ed077e4SKeith Packard  * _drm_lease_held - check drm_mode_object lease status (idr_mutex held)
1037591844cSChris Wilson  * @file_priv: the master drm_file
1042ed077e4SKeith Packard  * @id: the object id
1052ed077e4SKeith Packard  *
1062ed077e4SKeith Packard  * Checks if the specified master holds a lease on the object. Return
1072ed077e4SKeith Packard  * value:
1082ed077e4SKeith Packard  *
1092ed077e4SKeith Packard  *	true		'master' holds a lease on (or owns) the object
1102ed077e4SKeith Packard  *	false		'master' does not hold a lease.
1112ed077e4SKeith Packard  */
1122ed077e4SKeith Packard bool _drm_lease_held(struct drm_file *file_priv, int id)
1132ed077e4SKeith Packard {
1142ed077e4SKeith Packard 	if (file_priv == NULL || file_priv->master == NULL)
1152ed077e4SKeith Packard 		return true;
1162ed077e4SKeith Packard 
1172ed077e4SKeith Packard 	return _drm_lease_held_master(file_priv->master, id);
1182ed077e4SKeith Packard }
1192ed077e4SKeith Packard 
1202ed077e4SKeith Packard /**
1212ed077e4SKeith Packard  * drm_lease_held - check drm_mode_object lease status (idr_mutex not held)
1227591844cSChris Wilson  * @file_priv: the master drm_file
1232ed077e4SKeith Packard  * @id: the object id
1242ed077e4SKeith Packard  *
1252ed077e4SKeith Packard  * Checks if the specified master holds a lease on the object. Return
1262ed077e4SKeith Packard  * value:
1272ed077e4SKeith Packard  *
1282ed077e4SKeith Packard  *	true		'master' holds a lease on (or owns) the object
1292ed077e4SKeith Packard  *	false		'master' does not hold a lease.
1302ed077e4SKeith Packard  */
1312ed077e4SKeith Packard bool drm_lease_held(struct drm_file *file_priv, int id)
1322ed077e4SKeith Packard {
1332ed077e4SKeith Packard 	struct drm_master *master;
1342ed077e4SKeith Packard 	bool ret;
1352ed077e4SKeith Packard 
1362ed077e4SKeith Packard 	if (file_priv == NULL || file_priv->master == NULL)
1372ed077e4SKeith Packard 		return true;
1382ed077e4SKeith Packard 
1392ed077e4SKeith Packard 	master = file_priv->master;
1402ed077e4SKeith Packard 	mutex_lock(&master->dev->mode_config.idr_mutex);
1412ed077e4SKeith Packard 	ret = _drm_lease_held_master(master, id);
1422ed077e4SKeith Packard 	mutex_unlock(&master->dev->mode_config.idr_mutex);
1432ed077e4SKeith Packard 	return ret;
1442ed077e4SKeith Packard }
1452ed077e4SKeith Packard 
1462ed077e4SKeith Packard /**
1472ed077e4SKeith Packard  * drm_lease_filter_crtcs - restricted crtc set to leased values (idr_mutex not held)
1482ed077e4SKeith Packard  * @file_priv: requestor file
1497591844cSChris Wilson  * @crtcs_in: bitmask of crtcs to check
1502ed077e4SKeith Packard  *
1512ed077e4SKeith Packard  * Reconstructs a crtc mask based on the crtcs which are visible
1522ed077e4SKeith Packard  * through the specified file.
1532ed077e4SKeith Packard  */
1542ed077e4SKeith Packard uint32_t drm_lease_filter_crtcs(struct drm_file *file_priv, uint32_t crtcs_in)
1552ed077e4SKeith Packard {
1562ed077e4SKeith Packard 	struct drm_master *master;
1572ed077e4SKeith Packard 	struct drm_device *dev;
1582ed077e4SKeith Packard 	struct drm_crtc *crtc;
1592ed077e4SKeith Packard 	int count_in, count_out;
1602ed077e4SKeith Packard 	uint32_t crtcs_out = 0;
1612ed077e4SKeith Packard 
1622ed077e4SKeith Packard 	if (file_priv == NULL || file_priv->master == NULL)
1632ed077e4SKeith Packard 		return crtcs_in;
1642ed077e4SKeith Packard 
1652ed077e4SKeith Packard 	master = file_priv->master;
1662ed077e4SKeith Packard 	dev = master->dev;
1672ed077e4SKeith Packard 
1682ed077e4SKeith Packard 	count_in = count_out = 0;
1692ed077e4SKeith Packard 	mutex_lock(&master->dev->mode_config.idr_mutex);
1702ed077e4SKeith Packard 	list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
1712ed077e4SKeith Packard 		if (_drm_lease_held_master(master, crtc->base.id)) {
1722ed077e4SKeith Packard 			uint32_t mask_in = 1ul << count_in;
1732ed077e4SKeith Packard 			if ((crtcs_in & mask_in) != 0) {
1742ed077e4SKeith Packard 				uint32_t mask_out = 1ul << count_out;
1752ed077e4SKeith Packard 				crtcs_out |= mask_out;
1762ed077e4SKeith Packard 			}
1772ed077e4SKeith Packard 			count_out++;
1782ed077e4SKeith Packard 		}
1792ed077e4SKeith Packard 		count_in++;
1802ed077e4SKeith Packard 	}
1812ed077e4SKeith Packard 	mutex_unlock(&master->dev->mode_config.idr_mutex);
1822ed077e4SKeith Packard 	return crtcs_out;
1832ed077e4SKeith Packard }
1842ed077e4SKeith Packard 
1852ed077e4SKeith Packard /*
1862ed077e4SKeith Packard  * drm_lease_create - create a new drm_master with leased objects (idr_mutex not held)
1872ed077e4SKeith Packard  * @lessor: lease holder (or owner) of objects
1882ed077e4SKeith Packard  * @leases: objects to lease to the new drm_master
1892ed077e4SKeith Packard  *
1902ed077e4SKeith Packard  * Uses drm_master_create to allocate a new drm_master, then checks to
1912ed077e4SKeith Packard  * make sure all of the desired objects can be leased, atomically
1922ed077e4SKeith Packard  * leasing them to the new drmmaster.
1932ed077e4SKeith Packard  *
194918d89bbSColin Ian King  * 	ERR_PTR(-EACCES)	some other master holds the title to any object
1952ed077e4SKeith Packard  * 	ERR_PTR(-ENOENT)	some object is not a valid DRM object for this device
1962ed077e4SKeith Packard  * 	ERR_PTR(-EBUSY)		some other lessee holds title to this object
1972ed077e4SKeith Packard  *	ERR_PTR(-EEXIST)	same object specified more than once in the provided list
1982ed077e4SKeith Packard  *	ERR_PTR(-ENOMEM)	allocation failed
1992ed077e4SKeith Packard  */
2002ed077e4SKeith Packard static struct drm_master *drm_lease_create(struct drm_master *lessor, struct idr *leases)
2012ed077e4SKeith Packard {
2022ed077e4SKeith Packard 	struct drm_device *dev = lessor->dev;
2032ed077e4SKeith Packard 	int error;
2042ed077e4SKeith Packard 	struct drm_master *lessee;
2052ed077e4SKeith Packard 	int object;
2062ed077e4SKeith Packard 	int id;
2072ed077e4SKeith Packard 	void *entry;
2082ed077e4SKeith Packard 
2092ed077e4SKeith Packard 	DRM_DEBUG_LEASE("lessor %d\n", lessor->lessee_id);
2102ed077e4SKeith Packard 
2112ed077e4SKeith Packard 	lessee = drm_master_create(lessor->dev);
2122ed077e4SKeith Packard 	if (!lessee) {
2132ed077e4SKeith Packard 		DRM_DEBUG_LEASE("drm_master_create failed\n");
2142ed077e4SKeith Packard 		return ERR_PTR(-ENOMEM);
2152ed077e4SKeith Packard 	}
2162ed077e4SKeith Packard 
2172ed077e4SKeith Packard 	mutex_lock(&dev->mode_config.idr_mutex);
2182ed077e4SKeith Packard 
2192ed077e4SKeith Packard 	idr_for_each_entry(leases, entry, object) {
2202ed077e4SKeith Packard 		error = 0;
2212ed077e4SKeith Packard 		if (!idr_find(&dev->mode_config.crtc_idr, object))
2222ed077e4SKeith Packard 			error = -ENOENT;
2232ed077e4SKeith Packard 		else if (!_drm_lease_held_master(lessor, object))
2242ed077e4SKeith Packard 			error = -EACCES;
2252ed077e4SKeith Packard 		else if (_drm_has_leased(lessor, object))
2262ed077e4SKeith Packard 			error = -EBUSY;
2272ed077e4SKeith Packard 
2282ed077e4SKeith Packard 		if (error != 0) {
2292ed077e4SKeith Packard 			DRM_DEBUG_LEASE("object %d failed %d\n", object, error);
2302ed077e4SKeith Packard 			goto out_lessee;
2312ed077e4SKeith Packard 		}
2322ed077e4SKeith Packard 	}
2332ed077e4SKeith Packard 
234d2a48e52SKeith Packard 	/* Insert the new lessee into the tree */
235d2a48e52SKeith Packard 	id = idr_alloc(&(drm_lease_owner(lessor)->lessee_idr), lessee, 1, 0, GFP_KERNEL);
236d2a48e52SKeith Packard 	if (id < 0) {
237d2a48e52SKeith Packard 		error = id;
238d2a48e52SKeith Packard 		goto out_lessee;
239d2a48e52SKeith Packard 	}
240d2a48e52SKeith Packard 
241d2a48e52SKeith Packard 	lessee->lessee_id = id;
242d2a48e52SKeith Packard 	lessee->lessor = drm_master_get(lessor);
243d2a48e52SKeith Packard 	list_add_tail(&lessee->lessee_list, &lessor->lessees);
244d2a48e52SKeith Packard 
2452ed077e4SKeith Packard 	/* Move the leases over */
2462ed077e4SKeith Packard 	lessee->leases = *leases;
2472ed077e4SKeith Packard 	DRM_DEBUG_LEASE("new lessee %d %p, lessor %d %p\n", lessee->lessee_id, lessee, lessor->lessee_id, lessor);
2482ed077e4SKeith Packard 
2492ed077e4SKeith Packard 	mutex_unlock(&dev->mode_config.idr_mutex);
2502ed077e4SKeith Packard 	return lessee;
2512ed077e4SKeith Packard 
2522ed077e4SKeith Packard out_lessee:
2532ed077e4SKeith Packard 	mutex_unlock(&dev->mode_config.idr_mutex);
2542ed077e4SKeith Packard 
255bd36d3baSMarius Vlad 	drm_master_put(&lessee);
256bd36d3baSMarius Vlad 
2572ed077e4SKeith Packard 	return ERR_PTR(error);
2582ed077e4SKeith Packard }
2592ed077e4SKeith Packard 
2602ed077e4SKeith Packard /**
2612ed077e4SKeith Packard  * drm_lease_destroy - a master is going away (idr_mutex not held)
2622ed077e4SKeith Packard  * @master: the drm_master being destroyed
2632ed077e4SKeith Packard  *
2642ed077e4SKeith Packard  * All lessees will have been destroyed as they
2652ed077e4SKeith Packard  * hold a reference on their lessor. Notify any
2662ed077e4SKeith Packard  * lessor for this master so that it can check
2672ed077e4SKeith Packard  * the list of lessees.
2682ed077e4SKeith Packard  */
2692ed077e4SKeith Packard void drm_lease_destroy(struct drm_master *master)
2702ed077e4SKeith Packard {
2712ed077e4SKeith Packard 	struct drm_device *dev = master->dev;
2722ed077e4SKeith Packard 
2732ed077e4SKeith Packard 	mutex_lock(&dev->mode_config.idr_mutex);
2742ed077e4SKeith Packard 
2752ed077e4SKeith Packard 	DRM_DEBUG_LEASE("drm_lease_destroy %d\n", master->lessee_id);
2762ed077e4SKeith Packard 
2772ed077e4SKeith Packard 	/* This master is referenced by all lessees, hence it cannot be destroyed
2782ed077e4SKeith Packard 	 * until all of them have been
2792ed077e4SKeith Packard 	 */
2802ed077e4SKeith Packard 	WARN_ON(!list_empty(&master->lessees));
2812ed077e4SKeith Packard 
2822ed077e4SKeith Packard 	/* Remove this master from the lessee idr in the owner */
2832ed077e4SKeith Packard 	if (master->lessee_id != 0) {
2842ed077e4SKeith Packard 		DRM_DEBUG_LEASE("remove master %d from device list of lessees\n", master->lessee_id);
2852ed077e4SKeith Packard 		idr_remove(&(drm_lease_owner(master)->lessee_idr), master->lessee_id);
2862ed077e4SKeith Packard 	}
2872ed077e4SKeith Packard 
2882ed077e4SKeith Packard 	/* Remove this master from any lessee list it may be on */
2892ed077e4SKeith Packard 	list_del(&master->lessee_list);
2902ed077e4SKeith Packard 
2912ed077e4SKeith Packard 	mutex_unlock(&dev->mode_config.idr_mutex);
2922ed077e4SKeith Packard 
2932ed077e4SKeith Packard 	if (master->lessor) {
2942ed077e4SKeith Packard 		/* Tell the master to check the lessee list */
2952ed077e4SKeith Packard 		drm_sysfs_hotplug_event(dev);
2962ed077e4SKeith Packard 		drm_master_put(&master->lessor);
2972ed077e4SKeith Packard 	}
2982ed077e4SKeith Packard 
2992ed077e4SKeith Packard 	DRM_DEBUG_LEASE("drm_lease_destroy done %d\n", master->lessee_id);
3002ed077e4SKeith Packard }
3012ed077e4SKeith Packard 
3022ed077e4SKeith Packard /**
3032ed077e4SKeith Packard  * _drm_lease_revoke - revoke access to all leased objects (idr_mutex held)
3047591844cSChris Wilson  * @top: the master losing its lease
3052ed077e4SKeith Packard  */
3062ed077e4SKeith Packard static void _drm_lease_revoke(struct drm_master *top)
3072ed077e4SKeith Packard {
3082ed077e4SKeith Packard 	int object;
3092ed077e4SKeith Packard 	void *entry;
3102ed077e4SKeith Packard 	struct drm_master *master = top;
3112ed077e4SKeith Packard 
3122ed077e4SKeith Packard 	lockdep_assert_held(&top->dev->mode_config.idr_mutex);
3132ed077e4SKeith Packard 
3142ed077e4SKeith Packard 	/*
3152ed077e4SKeith Packard 	 * Walk the tree starting at 'top' emptying all leases. Because
3162ed077e4SKeith Packard 	 * the tree is fully connected, we can do this without recursing
3172ed077e4SKeith Packard 	 */
3182ed077e4SKeith Packard 	for (;;) {
3192ed077e4SKeith Packard 		DRM_DEBUG_LEASE("revoke leases for %p %d\n", master, master->lessee_id);
3202ed077e4SKeith Packard 
3212ed077e4SKeith Packard 		/* Evacuate the lease */
3222ed077e4SKeith Packard 		idr_for_each_entry(&master->leases, entry, object)
3232ed077e4SKeith Packard 			idr_remove(&master->leases, object);
3242ed077e4SKeith Packard 
3252ed077e4SKeith Packard 		/* Depth-first list walk */
3262ed077e4SKeith Packard 
3272ed077e4SKeith Packard 		/* Down */
3282ed077e4SKeith Packard 		if (!list_empty(&master->lessees)) {
3292ed077e4SKeith Packard 			master = list_first_entry(&master->lessees, struct drm_master, lessee_list);
3302ed077e4SKeith Packard 		} else {
3312ed077e4SKeith Packard 			/* Up */
3322ed077e4SKeith Packard 			while (master != top && master == list_last_entry(&master->lessor->lessees, struct drm_master, lessee_list))
3332ed077e4SKeith Packard 				master = master->lessor;
3342ed077e4SKeith Packard 
3352ed077e4SKeith Packard 			if (master == top)
3362ed077e4SKeith Packard 				break;
3372ed077e4SKeith Packard 
3382ed077e4SKeith Packard 			/* Over */
339aec06c76SArushi Singhal 			master = list_next_entry(master, lessee_list);
3402ed077e4SKeith Packard 		}
3412ed077e4SKeith Packard 	}
3422ed077e4SKeith Packard }
3432ed077e4SKeith Packard 
3442ed077e4SKeith Packard /**
3452ed077e4SKeith Packard  * drm_lease_revoke - revoke access to all leased objects (idr_mutex not held)
3462ed077e4SKeith Packard  * @top: the master losing its lease
3472ed077e4SKeith Packard  */
3482ed077e4SKeith Packard void drm_lease_revoke(struct drm_master *top)
3492ed077e4SKeith Packard {
3502ed077e4SKeith Packard 	mutex_lock(&top->dev->mode_config.idr_mutex);
3512ed077e4SKeith Packard 	_drm_lease_revoke(top);
3522ed077e4SKeith Packard 	mutex_unlock(&top->dev->mode_config.idr_mutex);
3532ed077e4SKeith Packard }
35462884cd3SKeith Packard 
35562884cd3SKeith Packard static int validate_lease(struct drm_device *dev,
35662884cd3SKeith Packard 			  struct drm_file *lessor_priv,
35762884cd3SKeith Packard 			  int object_count,
35862884cd3SKeith Packard 			  struct drm_mode_object **objects)
35962884cd3SKeith Packard {
36062884cd3SKeith Packard 	int o;
36162884cd3SKeith Packard 	int has_crtc = -1;
36262884cd3SKeith Packard 	int has_connector = -1;
36362884cd3SKeith Packard 	int has_plane = -1;
36462884cd3SKeith Packard 
36562884cd3SKeith Packard 	/* we want to confirm that there is at least one crtc, plane
36662884cd3SKeith Packard 	   connector object. */
36762884cd3SKeith Packard 
36862884cd3SKeith Packard 	for (o = 0; o < object_count; o++) {
36962884cd3SKeith Packard 		if (objects[o]->type == DRM_MODE_OBJECT_CRTC && has_crtc == -1) {
37062884cd3SKeith Packard 			has_crtc = o;
37162884cd3SKeith Packard 		}
37262884cd3SKeith Packard 		if (objects[o]->type == DRM_MODE_OBJECT_CONNECTOR && has_connector == -1)
37362884cd3SKeith Packard 			has_connector = o;
37462884cd3SKeith Packard 
37562884cd3SKeith Packard 		if (lessor_priv->universal_planes) {
37662884cd3SKeith Packard 			if (objects[o]->type == DRM_MODE_OBJECT_PLANE && has_plane == -1)
37762884cd3SKeith Packard 				has_plane = o;
37862884cd3SKeith Packard 		}
37962884cd3SKeith Packard 	}
38062884cd3SKeith Packard 	if (has_crtc == -1 || has_connector == -1)
38162884cd3SKeith Packard 		return -EINVAL;
38262884cd3SKeith Packard 	if (lessor_priv->universal_planes && has_plane == -1)
38362884cd3SKeith Packard 		return -EINVAL;
38462884cd3SKeith Packard 	return 0;
38562884cd3SKeith Packard }
38662884cd3SKeith Packard 
38762884cd3SKeith Packard static int fill_object_idr(struct drm_device *dev,
38862884cd3SKeith Packard 			   struct drm_file *lessor_priv,
38962884cd3SKeith Packard 			   struct idr *leases,
39062884cd3SKeith Packard 			   int object_count,
39162884cd3SKeith Packard 			   u32 *object_ids)
39262884cd3SKeith Packard {
39362884cd3SKeith Packard 	struct drm_mode_object **objects;
39462884cd3SKeith Packard 	u32 o;
39562884cd3SKeith Packard 	int ret;
39662884cd3SKeith Packard 	objects = kcalloc(object_count, sizeof(struct drm_mode_object *),
39762884cd3SKeith Packard 			  GFP_KERNEL);
39862884cd3SKeith Packard 	if (!objects)
39962884cd3SKeith Packard 		return -ENOMEM;
40062884cd3SKeith Packard 
40162884cd3SKeith Packard 	/* step one - get references to all the mode objects
40262884cd3SKeith Packard 	   and check for validity. */
40362884cd3SKeith Packard 	for (o = 0; o < object_count; o++) {
40462884cd3SKeith Packard 		if ((int) object_ids[o] < 0) {
40562884cd3SKeith Packard 			ret = -EINVAL;
40662884cd3SKeith Packard 			goto out_free_objects;
40762884cd3SKeith Packard 		}
40862884cd3SKeith Packard 
40962884cd3SKeith Packard 		objects[o] = drm_mode_object_find(dev, lessor_priv,
41062884cd3SKeith Packard 						  object_ids[o],
41162884cd3SKeith Packard 						  DRM_MODE_OBJECT_ANY);
41262884cd3SKeith Packard 		if (!objects[o]) {
41362884cd3SKeith Packard 			ret = -ENOENT;
41462884cd3SKeith Packard 			goto out_free_objects;
41562884cd3SKeith Packard 		}
41662884cd3SKeith Packard 
41762884cd3SKeith Packard 		if (!drm_mode_object_lease_required(objects[o]->type)) {
418*a0c1af46SDaniel Vetter 			DRM_DEBUG_KMS("invalid object for lease\n");
41962884cd3SKeith Packard 			ret = -EINVAL;
42062884cd3SKeith Packard 			goto out_free_objects;
42162884cd3SKeith Packard 		}
42262884cd3SKeith Packard 	}
42362884cd3SKeith Packard 
42462884cd3SKeith Packard 	ret = validate_lease(dev, lessor_priv, object_count, objects);
425*a0c1af46SDaniel Vetter 	if (ret) {
426*a0c1af46SDaniel Vetter 		DRM_DEBUG_LEASE("lease validation failed\n");
42762884cd3SKeith Packard 		goto out_free_objects;
428*a0c1af46SDaniel Vetter 	}
42962884cd3SKeith Packard 
43062884cd3SKeith Packard 	/* add their IDs to the lease request - taking into account
43162884cd3SKeith Packard 	   universal planes */
43262884cd3SKeith Packard 	for (o = 0; o < object_count; o++) {
43362884cd3SKeith Packard 		struct drm_mode_object *obj = objects[o];
43462884cd3SKeith Packard 		u32 object_id = objects[o]->id;
43562884cd3SKeith Packard 		DRM_DEBUG_LEASE("Adding object %d to lease\n", object_id);
43662884cd3SKeith Packard 
43762884cd3SKeith Packard 		/*
43862884cd3SKeith Packard 		 * We're using an IDR to hold the set of leased
43962884cd3SKeith Packard 		 * objects, but we don't need to point at the object's
44062884cd3SKeith Packard 		 * data structure from the lease as the main crtc_idr
44162884cd3SKeith Packard 		 * will be used to actually find that. Instead, all we
44262884cd3SKeith Packard 		 * really want is a 'leased/not-leased' result, for
44362884cd3SKeith Packard 		 * which any non-NULL pointer will work fine.
44462884cd3SKeith Packard 		 */
44562884cd3SKeith Packard 		ret = idr_alloc(leases, &drm_lease_idr_object , object_id, object_id + 1, GFP_KERNEL);
44662884cd3SKeith Packard 		if (ret < 0) {
44762884cd3SKeith Packard 			DRM_DEBUG_LEASE("Object %d cannot be inserted into leases (%d)\n",
44862884cd3SKeith Packard 					object_id, ret);
44962884cd3SKeith Packard 			goto out_free_objects;
45062884cd3SKeith Packard 		}
45162884cd3SKeith Packard 		if (obj->type == DRM_MODE_OBJECT_CRTC && !lessor_priv->universal_planes) {
45262884cd3SKeith Packard 			struct drm_crtc *crtc = obj_to_crtc(obj);
45362884cd3SKeith Packard 			ret = idr_alloc(leases, &drm_lease_idr_object, crtc->primary->base.id, crtc->primary->base.id + 1, GFP_KERNEL);
45462884cd3SKeith Packard 			if (ret < 0) {
45562884cd3SKeith Packard 				DRM_DEBUG_LEASE("Object primary plane %d cannot be inserted into leases (%d)\n",
45662884cd3SKeith Packard 						object_id, ret);
45762884cd3SKeith Packard 				goto out_free_objects;
45862884cd3SKeith Packard 			}
45962884cd3SKeith Packard 			if (crtc->cursor) {
46062884cd3SKeith Packard 				ret = idr_alloc(leases, &drm_lease_idr_object, crtc->cursor->base.id, crtc->cursor->base.id + 1, GFP_KERNEL);
46162884cd3SKeith Packard 				if (ret < 0) {
46262884cd3SKeith Packard 					DRM_DEBUG_LEASE("Object cursor plane %d cannot be inserted into leases (%d)\n",
46362884cd3SKeith Packard 							object_id, ret);
46462884cd3SKeith Packard 					goto out_free_objects;
46562884cd3SKeith Packard 				}
46662884cd3SKeith Packard 			}
46762884cd3SKeith Packard 		}
46862884cd3SKeith Packard 	}
46962884cd3SKeith Packard 
47062884cd3SKeith Packard 	ret = 0;
47162884cd3SKeith Packard out_free_objects:
47262884cd3SKeith Packard 	for (o = 0; o < object_count; o++) {
47362884cd3SKeith Packard 		if (objects[o])
47462884cd3SKeith Packard 			drm_mode_object_put(objects[o]);
47562884cd3SKeith Packard 	}
47662884cd3SKeith Packard 	kfree(objects);
47762884cd3SKeith Packard 	return ret;
47862884cd3SKeith Packard }
47962884cd3SKeith Packard 
48062884cd3SKeith Packard /**
48162884cd3SKeith Packard  * drm_mode_create_lease_ioctl - create a new lease
48262884cd3SKeith Packard  * @dev: the drm device
48362884cd3SKeith Packard  * @data: pointer to struct drm_mode_create_lease
4847591844cSChris Wilson  * @lessor_priv: the file being manipulated
48562884cd3SKeith Packard  *
48662884cd3SKeith Packard  * The master associated with the specified file will have a lease
48762884cd3SKeith Packard  * created containing the objects specified in the ioctl structure.
48862884cd3SKeith Packard  * A file descriptor will be allocated for that and returned to the
48962884cd3SKeith Packard  * application.
49062884cd3SKeith Packard  */
49162884cd3SKeith Packard int drm_mode_create_lease_ioctl(struct drm_device *dev,
49262884cd3SKeith Packard 				void *data, struct drm_file *lessor_priv)
49362884cd3SKeith Packard {
49462884cd3SKeith Packard 	struct drm_mode_create_lease *cl = data;
49562884cd3SKeith Packard 	size_t object_count;
49662884cd3SKeith Packard 	int ret = 0;
49762884cd3SKeith Packard 	struct idr leases;
49862884cd3SKeith Packard 	struct drm_master *lessor = lessor_priv->master;
49962884cd3SKeith Packard 	struct drm_master *lessee = NULL;
50062884cd3SKeith Packard 	struct file *lessee_file = NULL;
50162884cd3SKeith Packard 	struct file *lessor_file = lessor_priv->filp;
50262884cd3SKeith Packard 	struct drm_file *lessee_priv;
50362884cd3SKeith Packard 	int fd = -1;
50462884cd3SKeith Packard 	uint32_t *object_ids;
50562884cd3SKeith Packard 
50662884cd3SKeith Packard 	/* Can't lease without MODESET */
50762884cd3SKeith Packard 	if (!drm_core_check_feature(dev, DRIVER_MODESET))
50869fdf420SChris Wilson 		return -EOPNOTSUPP;
50962884cd3SKeith Packard 
51062884cd3SKeith Packard 	/* Do not allow sub-leases */
511*a0c1af46SDaniel Vetter 	if (lessor->lessor) {
512*a0c1af46SDaniel Vetter 		DRM_DEBUG_LEASE("recursive leasing not allowed\n");
51362884cd3SKeith Packard 		return -EINVAL;
514*a0c1af46SDaniel Vetter 	}
51562884cd3SKeith Packard 
51662884cd3SKeith Packard 	/* need some objects */
517*a0c1af46SDaniel Vetter 	if (cl->object_count == 0) {
518*a0c1af46SDaniel Vetter 		DRM_DEBUG_LEASE("no objects in lease\n");
51962884cd3SKeith Packard 		return -EINVAL;
520*a0c1af46SDaniel Vetter 	}
52162884cd3SKeith Packard 
522*a0c1af46SDaniel Vetter 	if (cl->flags && (cl->flags & ~(O_CLOEXEC | O_NONBLOCK))) {
523*a0c1af46SDaniel Vetter 		DRM_DEBUG_LEASE("invalid flags\n");
52462884cd3SKeith Packard 		return -EINVAL;
525*a0c1af46SDaniel Vetter 	}
52662884cd3SKeith Packard 
52762884cd3SKeith Packard 	object_count = cl->object_count;
52862884cd3SKeith Packard 
52962884cd3SKeith Packard 	object_ids = memdup_user(u64_to_user_ptr(cl->object_ids), object_count * sizeof(__u32));
53062884cd3SKeith Packard 	if (IS_ERR(object_ids))
53162884cd3SKeith Packard 		return PTR_ERR(object_ids);
53262884cd3SKeith Packard 
53362884cd3SKeith Packard 	idr_init(&leases);
53462884cd3SKeith Packard 
53562884cd3SKeith Packard 	/* fill and validate the object idr */
53662884cd3SKeith Packard 	ret = fill_object_idr(dev, lessor_priv, &leases,
53762884cd3SKeith Packard 			      object_count, object_ids);
53862884cd3SKeith Packard 	kfree(object_ids);
53962884cd3SKeith Packard 	if (ret) {
540*a0c1af46SDaniel Vetter 		DRM_DEBUG_LEASE("lease object lookup failed: %i\n", ret);
54162884cd3SKeith Packard 		idr_destroy(&leases);
54262884cd3SKeith Packard 		return ret;
54362884cd3SKeith Packard 	}
54462884cd3SKeith Packard 
54562884cd3SKeith Packard 	/* Allocate a file descriptor for the lease */
54662884cd3SKeith Packard 	fd = get_unused_fd_flags(cl->flags & (O_CLOEXEC | O_NONBLOCK));
54762884cd3SKeith Packard 	if (fd < 0) {
54862884cd3SKeith Packard 		idr_destroy(&leases);
54962884cd3SKeith Packard 		return fd;
55062884cd3SKeith Packard 	}
55162884cd3SKeith Packard 
55262884cd3SKeith Packard 	DRM_DEBUG_LEASE("Creating lease\n");
55362884cd3SKeith Packard 	lessee = drm_lease_create(lessor, &leases);
55462884cd3SKeith Packard 
55562884cd3SKeith Packard 	if (IS_ERR(lessee)) {
55662884cd3SKeith Packard 		ret = PTR_ERR(lessee);
55762884cd3SKeith Packard 		goto out_leases;
55862884cd3SKeith Packard 	}
55962884cd3SKeith Packard 
56062884cd3SKeith Packard 	/* Clone the lessor file to create a new file for us */
56162884cd3SKeith Packard 	DRM_DEBUG_LEASE("Allocating lease file\n");
56219f391ebSAl Viro 	lessee_file = file_clone_open(lessor_file);
56362884cd3SKeith Packard 	if (IS_ERR(lessee_file)) {
56462884cd3SKeith Packard 		ret = PTR_ERR(lessee_file);
56562884cd3SKeith Packard 		goto out_lessee;
56662884cd3SKeith Packard 	}
56762884cd3SKeith Packard 
56862884cd3SKeith Packard 	lessee_priv = lessee_file->private_data;
56962884cd3SKeith Packard 	/* Change the file to a master one */
57062884cd3SKeith Packard 	drm_master_put(&lessee_priv->master);
57162884cd3SKeith Packard 	lessee_priv->master = lessee;
57262884cd3SKeith Packard 	lessee_priv->is_master = 1;
57362884cd3SKeith Packard 	lessee_priv->authenticated = 1;
57462884cd3SKeith Packard 
57562884cd3SKeith Packard 	/* Hook up the fd */
57662884cd3SKeith Packard 	fd_install(fd, lessee_file);
57762884cd3SKeith Packard 
57862884cd3SKeith Packard 	/* Pass fd back to userspace */
57962884cd3SKeith Packard 	DRM_DEBUG_LEASE("Returning fd %d id %d\n", fd, lessee->lessee_id);
58062884cd3SKeith Packard 	cl->fd = fd;
58162884cd3SKeith Packard 	cl->lessee_id = lessee->lessee_id;
58262884cd3SKeith Packard 
58362884cd3SKeith Packard 	DRM_DEBUG_LEASE("drm_mode_create_lease_ioctl succeeded\n");
58462884cd3SKeith Packard 	return 0;
58562884cd3SKeith Packard 
58662884cd3SKeith Packard out_lessee:
58762884cd3SKeith Packard 	drm_master_put(&lessee);
58862884cd3SKeith Packard 
58962884cd3SKeith Packard out_leases:
59062884cd3SKeith Packard 	put_unused_fd(fd);
59162884cd3SKeith Packard 	idr_destroy(&leases);
59262884cd3SKeith Packard 
59362884cd3SKeith Packard 	DRM_DEBUG_LEASE("drm_mode_create_lease_ioctl failed: %d\n", ret);
59462884cd3SKeith Packard 	return ret;
59562884cd3SKeith Packard }
59662884cd3SKeith Packard 
59762884cd3SKeith Packard /**
59862884cd3SKeith Packard  * drm_mode_list_lessees_ioctl - list lessee ids
59962884cd3SKeith Packard  * @dev: the drm device
60062884cd3SKeith Packard  * @data: pointer to struct drm_mode_list_lessees
60162884cd3SKeith Packard  * @lessor_priv: the file being manipulated
60262884cd3SKeith Packard  *
60362884cd3SKeith Packard  * Starting from the master associated with the specified file,
60462884cd3SKeith Packard  * the master with the provided lessee_id is found, and then
60562884cd3SKeith Packard  * an array of lessee ids associated with leases from that master
60662884cd3SKeith Packard  * are returned.
60762884cd3SKeith Packard  */
60862884cd3SKeith Packard 
60962884cd3SKeith Packard int drm_mode_list_lessees_ioctl(struct drm_device *dev,
61062884cd3SKeith Packard 			       void *data, struct drm_file *lessor_priv)
61162884cd3SKeith Packard {
61262884cd3SKeith Packard 	struct drm_mode_list_lessees *arg = data;
61362884cd3SKeith Packard 	__u32 __user *lessee_ids = (__u32 __user *) (uintptr_t) (arg->lessees_ptr);
61462884cd3SKeith Packard 	__u32 count_lessees = arg->count_lessees;
61562884cd3SKeith Packard 	struct drm_master *lessor = lessor_priv->master, *lessee;
61662884cd3SKeith Packard 	int count;
61762884cd3SKeith Packard 	int ret = 0;
61862884cd3SKeith Packard 
61962884cd3SKeith Packard 	if (arg->pad)
62062884cd3SKeith Packard 		return -EINVAL;
62162884cd3SKeith Packard 
62262884cd3SKeith Packard 	/* Can't lease without MODESET */
62362884cd3SKeith Packard 	if (!drm_core_check_feature(dev, DRIVER_MODESET))
62469fdf420SChris Wilson 		return -EOPNOTSUPP;
62562884cd3SKeith Packard 
62662884cd3SKeith Packard 	DRM_DEBUG_LEASE("List lessees for %d\n", lessor->lessee_id);
62762884cd3SKeith Packard 
62862884cd3SKeith Packard 	mutex_lock(&dev->mode_config.idr_mutex);
62962884cd3SKeith Packard 
63062884cd3SKeith Packard 	count = 0;
63162884cd3SKeith Packard 	drm_for_each_lessee(lessee, lessor) {
63262884cd3SKeith Packard 		/* Only list un-revoked leases */
63362884cd3SKeith Packard 		if (!idr_is_empty(&lessee->leases)) {
63462884cd3SKeith Packard 			if (count_lessees > count) {
63562884cd3SKeith Packard 				DRM_DEBUG_LEASE("Add lessee %d\n", lessee->lessee_id);
63662884cd3SKeith Packard 				ret = put_user(lessee->lessee_id, lessee_ids + count);
63762884cd3SKeith Packard 				if (ret)
63862884cd3SKeith Packard 					break;
63962884cd3SKeith Packard 			}
64062884cd3SKeith Packard 			count++;
64162884cd3SKeith Packard 		}
64262884cd3SKeith Packard 	}
64362884cd3SKeith Packard 
64462884cd3SKeith Packard 	DRM_DEBUG_LEASE("Lessor leases to %d\n", count);
64562884cd3SKeith Packard 	if (ret == 0)
64662884cd3SKeith Packard 		arg->count_lessees = count;
64762884cd3SKeith Packard 
64862884cd3SKeith Packard 	mutex_unlock(&dev->mode_config.idr_mutex);
64962884cd3SKeith Packard 
65062884cd3SKeith Packard 	return ret;
65162884cd3SKeith Packard }
65262884cd3SKeith Packard 
65362884cd3SKeith Packard /**
65462884cd3SKeith Packard  * drm_mode_get_lease_ioctl - list leased objects
65562884cd3SKeith Packard  * @dev: the drm device
65662884cd3SKeith Packard  * @data: pointer to struct drm_mode_get_lease
6577591844cSChris Wilson  * @lessee_priv: the file being manipulated
65862884cd3SKeith Packard  *
65962884cd3SKeith Packard  * Return the list of leased objects for the specified lessee
66062884cd3SKeith Packard  */
66162884cd3SKeith Packard 
66262884cd3SKeith Packard int drm_mode_get_lease_ioctl(struct drm_device *dev,
66362884cd3SKeith Packard 			     void *data, struct drm_file *lessee_priv)
66462884cd3SKeith Packard {
66562884cd3SKeith Packard 	struct drm_mode_get_lease *arg = data;
66662884cd3SKeith Packard 	__u32 __user *object_ids = (__u32 __user *) (uintptr_t) (arg->objects_ptr);
66762884cd3SKeith Packard 	__u32 count_objects = arg->count_objects;
66862884cd3SKeith Packard 	struct drm_master *lessee = lessee_priv->master;
66962884cd3SKeith Packard 	struct idr *object_idr;
67062884cd3SKeith Packard 	int count;
67162884cd3SKeith Packard 	void *entry;
67262884cd3SKeith Packard 	int object;
67362884cd3SKeith Packard 	int ret = 0;
67462884cd3SKeith Packard 
67562884cd3SKeith Packard 	if (arg->pad)
67662884cd3SKeith Packard 		return -EINVAL;
67762884cd3SKeith Packard 
67862884cd3SKeith Packard 	/* Can't lease without MODESET */
67962884cd3SKeith Packard 	if (!drm_core_check_feature(dev, DRIVER_MODESET))
68069fdf420SChris Wilson 		return -EOPNOTSUPP;
68162884cd3SKeith Packard 
68262884cd3SKeith Packard 	DRM_DEBUG_LEASE("get lease for %d\n", lessee->lessee_id);
68362884cd3SKeith Packard 
68462884cd3SKeith Packard 	mutex_lock(&dev->mode_config.idr_mutex);
68562884cd3SKeith Packard 
68662884cd3SKeith Packard 	if (lessee->lessor == NULL)
68762884cd3SKeith Packard 		/* owner can use all objects */
68862884cd3SKeith Packard 		object_idr = &lessee->dev->mode_config.crtc_idr;
68962884cd3SKeith Packard 	else
69062884cd3SKeith Packard 		/* lessee can only use allowed object */
69162884cd3SKeith Packard 		object_idr = &lessee->leases;
69262884cd3SKeith Packard 
69362884cd3SKeith Packard 	count = 0;
69462884cd3SKeith Packard 	idr_for_each_entry(object_idr, entry, object) {
69562884cd3SKeith Packard 		if (count_objects > count) {
69662884cd3SKeith Packard 			DRM_DEBUG_LEASE("adding object %d\n", object);
69762884cd3SKeith Packard 			ret = put_user(object, object_ids + count);
69862884cd3SKeith Packard 			if (ret)
69962884cd3SKeith Packard 				break;
70062884cd3SKeith Packard 		}
70162884cd3SKeith Packard 		count++;
70262884cd3SKeith Packard 	}
70362884cd3SKeith Packard 
70462884cd3SKeith Packard 	DRM_DEBUG("lease holds %d objects\n", count);
70562884cd3SKeith Packard 	if (ret == 0)
70662884cd3SKeith Packard 		arg->count_objects = count;
70762884cd3SKeith Packard 
70862884cd3SKeith Packard 	mutex_unlock(&dev->mode_config.idr_mutex);
70962884cd3SKeith Packard 
71062884cd3SKeith Packard 	return ret;
71162884cd3SKeith Packard }
71262884cd3SKeith Packard 
71362884cd3SKeith Packard /**
71462884cd3SKeith Packard  * drm_mode_revoke_lease_ioctl - revoke lease
71562884cd3SKeith Packard  * @dev: the drm device
71662884cd3SKeith Packard  * @data: pointer to struct drm_mode_revoke_lease
7177591844cSChris Wilson  * @lessor_priv: the file being manipulated
71862884cd3SKeith Packard  *
71962884cd3SKeith Packard  * This removes all of the objects from the lease without
72062884cd3SKeith Packard  * actually getting rid of the lease itself; that way all
72162884cd3SKeith Packard  * references to it still work correctly
72262884cd3SKeith Packard  */
72362884cd3SKeith Packard int drm_mode_revoke_lease_ioctl(struct drm_device *dev,
72462884cd3SKeith Packard 				void *data, struct drm_file *lessor_priv)
72562884cd3SKeith Packard {
72662884cd3SKeith Packard 	struct drm_mode_revoke_lease *arg = data;
72762884cd3SKeith Packard 	struct drm_master *lessor = lessor_priv->master;
72862884cd3SKeith Packard 	struct drm_master *lessee;
72962884cd3SKeith Packard 	int ret = 0;
73062884cd3SKeith Packard 
73162884cd3SKeith Packard 	DRM_DEBUG_LEASE("revoke lease for %d\n", arg->lessee_id);
73262884cd3SKeith Packard 
73362884cd3SKeith Packard 	/* Can't lease without MODESET */
73462884cd3SKeith Packard 	if (!drm_core_check_feature(dev, DRIVER_MODESET))
73569fdf420SChris Wilson 		return -EOPNOTSUPP;
73662884cd3SKeith Packard 
73762884cd3SKeith Packard 	mutex_lock(&dev->mode_config.idr_mutex);
73862884cd3SKeith Packard 
73962884cd3SKeith Packard 	lessee = _drm_find_lessee(lessor, arg->lessee_id);
74062884cd3SKeith Packard 
74162884cd3SKeith Packard 	/* No such lessee */
74262884cd3SKeith Packard 	if (!lessee) {
74362884cd3SKeith Packard 		ret = -ENOENT;
74462884cd3SKeith Packard 		goto fail;
74562884cd3SKeith Packard 	}
74662884cd3SKeith Packard 
74762884cd3SKeith Packard 	/* Lease is not held by lessor */
74862884cd3SKeith Packard 	if (lessee->lessor != lessor) {
74962884cd3SKeith Packard 		ret = -EACCES;
75062884cd3SKeith Packard 		goto fail;
75162884cd3SKeith Packard 	}
75262884cd3SKeith Packard 
75362884cd3SKeith Packard 	_drm_lease_revoke(lessee);
75462884cd3SKeith Packard 
75562884cd3SKeith Packard fail:
75662884cd3SKeith Packard 	mutex_unlock(&dev->mode_config.idr_mutex);
75762884cd3SKeith Packard 
75862884cd3SKeith Packard 	return ret;
75962884cd3SKeith Packard }
760