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