2005-04-17 06:20:36 +08:00
|
|
|
/*
|
|
|
|
* Created: Fri Jan 19 10:48:35 2001 by faith@acm.org
|
|
|
|
*
|
|
|
|
* Copyright 2001 VA Linux Systems, Inc., Sunnyvale, California.
|
|
|
|
* All Rights Reserved.
|
|
|
|
*
|
2014-05-19 19:39:07 +08:00
|
|
|
* Author Rickard E. (Rik) Faith <faith@valinux.com>
|
|
|
|
*
|
2005-04-17 06:20:36 +08:00
|
|
|
* Permission is hereby granted, free of charge, to any person obtaining a
|
|
|
|
* copy of this software and associated documentation files (the "Software"),
|
|
|
|
* to deal in the Software without restriction, including without limitation
|
|
|
|
* the rights to use, copy, modify, merge, publish, distribute, sublicense,
|
|
|
|
* and/or sell copies of the Software, and to permit persons to whom the
|
|
|
|
* Software is furnished to do so, subject to the following conditions:
|
|
|
|
*
|
|
|
|
* The above copyright notice and this permission notice (including the next
|
|
|
|
* paragraph) shall be included in all copies or substantial portions of the
|
|
|
|
* Software.
|
|
|
|
*
|
|
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
|
|
|
|
* PRECISION INSIGHT AND/OR ITS SUPPLIERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
|
|
|
|
* OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
|
|
|
|
* ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
|
|
|
|
* DEALINGS IN THE SOFTWARE.
|
|
|
|
*/
|
|
|
|
|
2014-07-23 18:29:56 +08:00
|
|
|
#include <linux/debugfs.h>
|
2014-01-03 21:09:47 +08:00
|
|
|
#include <linux/fs.h>
|
2005-04-17 06:20:36 +08:00
|
|
|
#include <linux/module.h>
|
|
|
|
#include <linux/moduleparam.h>
|
2014-01-03 21:09:47 +08:00
|
|
|
#include <linux/mount.h>
|
include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h
percpu.h is included by sched.h and module.h and thus ends up being
included when building most .c files. percpu.h includes slab.h which
in turn includes gfp.h making everything defined by the two files
universally available and complicating inclusion dependencies.
percpu.h -> slab.h dependency is about to be removed. Prepare for
this change by updating users of gfp and slab facilities include those
headers directly instead of assuming availability. As this conversion
needs to touch large number of source files, the following script is
used as the basis of conversion.
http://userweb.kernel.org/~tj/misc/slabh-sweep.py
The script does the followings.
* Scan files for gfp and slab usages and update includes such that
only the necessary includes are there. ie. if only gfp is used,
gfp.h, if slab is used, slab.h.
* When the script inserts a new include, it looks at the include
blocks and try to put the new include such that its order conforms
to its surrounding. It's put in the include block which contains
core kernel includes, in the same order that the rest are ordered -
alphabetical, Christmas tree, rev-Xmas-tree or at the end if there
doesn't seem to be any matching order.
* If the script can't find a place to put a new include (mostly
because the file doesn't have fitting include block), it prints out
an error message indicating which .h file needs to be added to the
file.
The conversion was done in the following steps.
1. The initial automatic conversion of all .c files updated slightly
over 4000 files, deleting around 700 includes and adding ~480 gfp.h
and ~3000 slab.h inclusions. The script emitted errors for ~400
files.
2. Each error was manually checked. Some didn't need the inclusion,
some needed manual addition while adding it to implementation .h or
embedding .c file was more appropriate for others. This step added
inclusions to around 150 files.
3. The script was run again and the output was compared to the edits
from #2 to make sure no file was left behind.
4. Several build tests were done and a couple of problems were fixed.
e.g. lib/decompress_*.c used malloc/free() wrappers around slab
APIs requiring slab.h to be added manually.
5. The script was run on all .h files but without automatically
editing them as sprinkling gfp.h and slab.h inclusions around .h
files could easily lead to inclusion dependency hell. Most gfp.h
inclusion directives were ignored as stuff from gfp.h was usually
wildly available and often used in preprocessor macros. Each
slab.h inclusion directive was examined and added manually as
necessary.
6. percpu.h was updated not to include slab.h.
7. Build test were done on the following configurations and failures
were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my
distributed build env didn't work with gcov compiles) and a few
more options had to be turned off depending on archs to make things
build (like ipr on powerpc/64 which failed due to missing writeq).
* x86 and x86_64 UP and SMP allmodconfig and a custom test config.
* powerpc and powerpc64 SMP allmodconfig
* sparc and sparc64 SMP allmodconfig
* ia64 SMP allmodconfig
* s390 SMP allmodconfig
* alpha SMP allmodconfig
* um on x86_64 SMP allmodconfig
8. percpu.h modifications were reverted so that it could be applied as
a separate patch and serve as bisection point.
Given the fact that I had only a couple of failures from tests on step
6, I'm fairly confident about the coverage of this conversion patch.
If there is a breakage, it's likely to be something in one of the arch
headers which should be easily discoverable easily on most builds of
the specific arch.
Signed-off-by: Tejun Heo <tj@kernel.org>
Guess-its-ok-by: Christoph Lameter <cl@linux-foundation.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Lee Schermerhorn <Lee.Schermerhorn@hp.com>
2010-03-24 16:04:11 +08:00
|
|
|
#include <linux/slab.h>
|
2018-03-28 15:38:35 +08:00
|
|
|
#include <linux/srcu.h>
|
2016-11-14 19:58:19 +08:00
|
|
|
|
2018-07-04 00:03:47 +08:00
|
|
|
#include <drm/drm_client.h>
|
2019-05-27 01:35:35 +08:00
|
|
|
#include <drm/drm_color_mgmt.h>
|
2016-11-14 19:58:19 +08:00
|
|
|
#include <drm/drm_drv.h>
|
2019-05-27 01:35:35 +08:00
|
|
|
#include <drm/drm_file.h>
|
|
|
|
#include <drm/drm_mode_object.h>
|
|
|
|
#include <drm/drm_print.h>
|
2016-11-14 19:58:19 +08:00
|
|
|
|
2016-06-21 22:37:09 +08:00
|
|
|
#include "drm_crtc_internal.h"
|
2014-09-10 18:43:53 +08:00
|
|
|
#include "drm_internal.h"
|
2019-05-27 01:35:35 +08:00
|
|
|
#include "drm_legacy.h"
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2016-04-21 00:45:03 +08:00
|
|
|
/*
|
|
|
|
* drm_debug: Enable debug output.
|
2019-05-27 01:35:35 +08:00
|
|
|
* Bitmask of DRM_UT_x. See include/drm/drm_print.h for details.
|
2016-04-21 00:45:03 +08:00
|
|
|
*/
|
|
|
|
unsigned int drm_debug = 0;
|
2005-04-17 06:20:36 +08:00
|
|
|
EXPORT_SYMBOL(drm_debug);
|
|
|
|
|
drm: drop obsolete drm_core.h
The drm_core.h header contains a set of constants meant to be used
throughout DRM. However, as it turns out, they're each used just once and
don't bring any benefit. They're also grossly mis-named and lack
name-spacing. This patch inlines them, or moves them into drm_internal.h
as appropriate:
- CORE_AUTHOR and CORE_DESC are inlined into corresponding MODULE_*()
macros. It's just confusing having to follow 2 pointers when trying to
find the definition of these fields. Grep'ping for MODULE_AUTHOR()
should reveal the full information, if there's no strong reason not to.
- CORE_NAME, CORE_DATE, CORE_MAJOR, CORE_MINOR, and CORE_PATCHLEVEL are
inlined into the sysfs 'version' attribute. They're stripped
everywhere else (which is just some printk() statements). CORE_NAME
just doesn't make *any* sense, as we hard-code it in many places,
anyway. The other constants are outdated and just serve
binary-compatibility purposes. Hence, inline them in 'version' sysfs
attribute (we might even try dropping it..).
- DRM_IF_MAJOR and DRM_IF_MINOR are moved into drm_internal.h as they're
only used by the global ioctl handlers. Furthermore, versioning
interfaces breaks backports and as such is deprecated, anyway. We just
keep them for historic reasons. I doubt anyone will ever modify them
again.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160901124837.680-6-dh.herrmann@gmail.com
2016-09-01 20:48:36 +08:00
|
|
|
MODULE_AUTHOR("Gareth Hughes, Leif Delgass, José Fonseca, Jon Smirl");
|
|
|
|
MODULE_DESCRIPTION("DRM shared core routines");
|
2005-04-17 06:20:36 +08:00
|
|
|
MODULE_LICENSE("GPL and additional rights");
|
2016-04-21 00:45:03 +08:00
|
|
|
MODULE_PARM_DESC(debug, "Enable debug output, where each bit enables a debug category.\n"
|
2018-07-16 23:44:32 +08:00
|
|
|
"\t\tBit 0 (0x01) will enable CORE messages (drm core code)\n"
|
|
|
|
"\t\tBit 1 (0x02) will enable DRIVER messages (drm controller code)\n"
|
|
|
|
"\t\tBit 2 (0x04) will enable KMS messages (modesetting code)\n"
|
|
|
|
"\t\tBit 3 (0x08) will enable PRIME messages (prime code)\n"
|
|
|
|
"\t\tBit 4 (0x10) will enable ATOMIC messages (atomic code)\n"
|
|
|
|
"\t\tBit 5 (0x20) will enable VBL messages (vblank code)\n"
|
|
|
|
"\t\tBit 7 (0x80) will enable LEASE messages (leasing code)\n"
|
|
|
|
"\t\tBit 8 (0x100) will enable DP messages (displayport code)");
|
2005-10-04 03:02:20 +08:00
|
|
|
module_param_named(debug, drm_debug, int, 0600);
|
2005-04-17 06:20:36 +08:00
|
|
|
|
2014-02-24 22:53:25 +08:00
|
|
|
static DEFINE_SPINLOCK(drm_minor_lock);
|
2014-07-23 18:29:56 +08:00
|
|
|
static struct idr drm_minors_idr;
|
2008-04-21 14:47:32 +08:00
|
|
|
|
2017-07-09 05:43:52 +08:00
|
|
|
/*
|
|
|
|
* If the drm core fails to init for whatever reason,
|
|
|
|
* we should prevent any drivers from registering with it.
|
|
|
|
* It's best to check this at drm_dev_init(), as some drivers
|
|
|
|
* prefer to embed struct drm_device into their own device
|
|
|
|
* structure and call drm_dev_init() themselves.
|
|
|
|
*/
|
|
|
|
static bool drm_core_init_complete = false;
|
|
|
|
|
2014-07-23 18:29:56 +08:00
|
|
|
static struct dentry *drm_debugfs_root;
|
2011-04-18 11:35:51 +08:00
|
|
|
|
2018-03-28 15:38:35 +08:00
|
|
|
DEFINE_STATIC_SRCU(drm_unplug_srcu);
|
|
|
|
|
2014-02-24 22:53:25 +08:00
|
|
|
/*
|
|
|
|
* DRM Minors
|
|
|
|
* A DRM device can provide several char-dev interfaces on the DRM-Major. Each
|
|
|
|
* of them is represented by a drm_minor object. Depending on the capabilities
|
|
|
|
* of the device-driver, different interfaces are registered.
|
2005-04-17 06:20:36 +08:00
|
|
|
*
|
2014-02-24 22:53:25 +08:00
|
|
|
* Minors can be accessed via dev->$minor_name. This pointer is either
|
|
|
|
* NULL or a valid drm_minor pointer and stays valid as long as the device is
|
|
|
|
* valid. This means, DRM minors have the same life-time as the underlying
|
|
|
|
* device. However, this doesn't mean that the minor is active. Minors are
|
|
|
|
* registered and unregistered dynamically according to device-state.
|
2005-04-17 06:20:36 +08:00
|
|
|
*/
|
2014-02-24 22:53:25 +08:00
|
|
|
|
2014-01-29 19:43:56 +08:00
|
|
|
static struct drm_minor **drm_minor_get_slot(struct drm_device *dev,
|
|
|
|
unsigned int type)
|
|
|
|
{
|
|
|
|
switch (type) {
|
2016-08-04 02:04:25 +08:00
|
|
|
case DRM_MINOR_PRIMARY:
|
2014-01-29 19:43:56 +08:00
|
|
|
return &dev->primary;
|
|
|
|
case DRM_MINOR_RENDER:
|
|
|
|
return &dev->render;
|
|
|
|
default:
|
2018-02-21 03:11:57 +08:00
|
|
|
BUG();
|
2014-01-29 19:43:56 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static int drm_minor_alloc(struct drm_device *dev, unsigned int type)
|
|
|
|
{
|
|
|
|
struct drm_minor *minor;
|
2014-07-23 16:34:52 +08:00
|
|
|
unsigned long flags;
|
|
|
|
int r;
|
2014-01-29 19:43:56 +08:00
|
|
|
|
|
|
|
minor = kzalloc(sizeof(*minor), GFP_KERNEL);
|
|
|
|
if (!minor)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
minor->type = type;
|
|
|
|
minor->dev = dev;
|
|
|
|
|
2014-07-23 16:34:52 +08:00
|
|
|
idr_preload(GFP_KERNEL);
|
|
|
|
spin_lock_irqsave(&drm_minor_lock, flags);
|
|
|
|
r = idr_alloc(&drm_minors_idr,
|
|
|
|
NULL,
|
|
|
|
64 * type,
|
|
|
|
64 * (type + 1),
|
|
|
|
GFP_NOWAIT);
|
|
|
|
spin_unlock_irqrestore(&drm_minor_lock, flags);
|
|
|
|
idr_preload_end();
|
|
|
|
|
|
|
|
if (r < 0)
|
|
|
|
goto err_free;
|
|
|
|
|
|
|
|
minor->index = r;
|
|
|
|
|
2014-07-23 17:38:38 +08:00
|
|
|
minor->kdev = drm_sysfs_minor_alloc(minor);
|
|
|
|
if (IS_ERR(minor->kdev)) {
|
|
|
|
r = PTR_ERR(minor->kdev);
|
|
|
|
goto err_index;
|
|
|
|
}
|
|
|
|
|
2014-01-29 19:43:56 +08:00
|
|
|
*drm_minor_get_slot(dev, type) = minor;
|
|
|
|
return 0;
|
2014-07-23 16:34:52 +08:00
|
|
|
|
2014-07-23 17:38:38 +08:00
|
|
|
err_index:
|
|
|
|
spin_lock_irqsave(&drm_minor_lock, flags);
|
|
|
|
idr_remove(&drm_minors_idr, minor->index);
|
|
|
|
spin_unlock_irqrestore(&drm_minor_lock, flags);
|
2014-07-23 16:34:52 +08:00
|
|
|
err_free:
|
|
|
|
kfree(minor);
|
|
|
|
return r;
|
2014-01-29 19:43:56 +08:00
|
|
|
}
|
|
|
|
|
2014-01-29 19:55:48 +08:00
|
|
|
static void drm_minor_free(struct drm_device *dev, unsigned int type)
|
|
|
|
{
|
2014-07-23 16:34:52 +08:00
|
|
|
struct drm_minor **slot, *minor;
|
|
|
|
unsigned long flags;
|
2014-01-29 19:55:48 +08:00
|
|
|
|
|
|
|
slot = drm_minor_get_slot(dev, type);
|
2014-07-23 16:34:52 +08:00
|
|
|
minor = *slot;
|
|
|
|
if (!minor)
|
|
|
|
return;
|
|
|
|
|
2014-07-23 17:38:38 +08:00
|
|
|
put_device(minor->kdev);
|
2014-07-23 16:34:52 +08:00
|
|
|
|
|
|
|
spin_lock_irqsave(&drm_minor_lock, flags);
|
|
|
|
idr_remove(&drm_minors_idr, minor->index);
|
|
|
|
spin_unlock_irqrestore(&drm_minor_lock, flags);
|
|
|
|
|
|
|
|
kfree(minor);
|
|
|
|
*slot = NULL;
|
2014-01-29 19:55:48 +08:00
|
|
|
}
|
|
|
|
|
2014-01-29 19:57:05 +08:00
|
|
|
static int drm_minor_register(struct drm_device *dev, unsigned int type)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2014-07-23 16:34:52 +08:00
|
|
|
struct drm_minor *minor;
|
2014-02-24 22:53:25 +08:00
|
|
|
unsigned long flags;
|
2005-04-17 06:20:36 +08:00
|
|
|
int ret;
|
|
|
|
|
|
|
|
DRM_DEBUG("\n");
|
|
|
|
|
2014-07-23 16:34:52 +08:00
|
|
|
minor = *drm_minor_get_slot(dev, type);
|
|
|
|
if (!minor)
|
2014-01-29 19:43:56 +08:00
|
|
|
return 0;
|
|
|
|
|
2014-07-23 16:34:52 +08:00
|
|
|
ret = drm_debugfs_init(minor, minor->index, drm_debugfs_root);
|
2009-02-18 09:08:49 +08:00
|
|
|
if (ret) {
|
2009-06-02 14:01:37 +08:00
|
|
|
DRM_ERROR("DRM: Failed to initialize /sys/kernel/debug/dri.\n");
|
2017-01-27 06:56:04 +08:00
|
|
|
goto err_debugfs;
|
2009-02-18 09:08:49 +08:00
|
|
|
}
|
2008-04-21 14:47:32 +08:00
|
|
|
|
2014-07-23 17:38:38 +08:00
|
|
|
ret = device_add(minor->kdev);
|
|
|
|
if (ret)
|
2013-08-08 21:41:34 +08:00
|
|
|
goto err_debugfs;
|
2008-04-21 14:47:32 +08:00
|
|
|
|
2014-02-24 22:53:25 +08:00
|
|
|
/* replace NULL with @minor so lookups will succeed from now on */
|
|
|
|
spin_lock_irqsave(&drm_minor_lock, flags);
|
2014-07-23 16:34:52 +08:00
|
|
|
idr_replace(&drm_minors_idr, minor, minor->index);
|
2014-02-24 22:53:25 +08:00
|
|
|
spin_unlock_irqrestore(&drm_minor_lock, flags);
|
2008-04-21 14:47:32 +08:00
|
|
|
|
2014-07-23 16:34:52 +08:00
|
|
|
DRM_DEBUG("new minor registered %d\n", minor->index);
|
2008-04-21 14:47:32 +08:00
|
|
|
return 0;
|
|
|
|
|
2013-08-08 21:41:34 +08:00
|
|
|
err_debugfs:
|
2014-07-23 16:34:52 +08:00
|
|
|
drm_debugfs_cleanup(minor);
|
2005-04-17 06:20:36 +08:00
|
|
|
return ret;
|
|
|
|
}
|
2005-09-25 12:28:13 +08:00
|
|
|
|
2014-01-29 19:57:05 +08:00
|
|
|
static void drm_minor_unregister(struct drm_device *dev, unsigned int type)
|
2013-10-21 00:55:40 +08:00
|
|
|
{
|
2014-01-29 19:57:05 +08:00
|
|
|
struct drm_minor *minor;
|
2014-02-24 22:53:25 +08:00
|
|
|
unsigned long flags;
|
2014-01-29 19:57:05 +08:00
|
|
|
|
|
|
|
minor = *drm_minor_get_slot(dev, type);
|
2014-07-23 17:38:38 +08:00
|
|
|
if (!minor || !device_is_registered(minor->kdev))
|
2013-10-21 00:55:40 +08:00
|
|
|
return;
|
|
|
|
|
2014-07-23 16:34:52 +08:00
|
|
|
/* replace @minor with NULL so lookups will fail from now on */
|
2014-02-24 22:53:25 +08:00
|
|
|
spin_lock_irqsave(&drm_minor_lock, flags);
|
2014-07-23 16:34:52 +08:00
|
|
|
idr_replace(&drm_minors_idr, NULL, minor->index);
|
2014-02-24 22:53:25 +08:00
|
|
|
spin_unlock_irqrestore(&drm_minor_lock, flags);
|
2013-10-21 00:55:43 +08:00
|
|
|
|
2014-07-23 17:38:38 +08:00
|
|
|
device_del(minor->kdev);
|
|
|
|
dev_set_drvdata(minor->kdev, NULL); /* safety belt */
|
2013-10-21 00:55:43 +08:00
|
|
|
drm_debugfs_cleanup(minor);
|
2013-10-21 00:55:40 +08:00
|
|
|
}
|
|
|
|
|
2016-11-14 19:58:19 +08:00
|
|
|
/*
|
2014-01-29 17:49:19 +08:00
|
|
|
* Looks up the given minor-ID and returns the respective DRM-minor object. The
|
|
|
|
* refence-count of the underlying device is increased so you must release this
|
|
|
|
* object with drm_minor_release().
|
|
|
|
*
|
|
|
|
* As long as you hold this minor, it is guaranteed that the object and the
|
|
|
|
* minor->dev pointer will stay valid! However, the device may get unplugged and
|
|
|
|
* unregistered while you hold the minor.
|
2005-04-17 06:20:36 +08:00
|
|
|
*/
|
2014-01-29 17:49:19 +08:00
|
|
|
struct drm_minor *drm_minor_acquire(unsigned int minor_id)
|
2005-04-17 06:20:36 +08:00
|
|
|
{
|
2014-01-29 17:49:19 +08:00
|
|
|
struct drm_minor *minor;
|
2014-02-24 22:53:25 +08:00
|
|
|
unsigned long flags;
|
2014-01-29 17:49:19 +08:00
|
|
|
|
2014-02-24 22:53:25 +08:00
|
|
|
spin_lock_irqsave(&drm_minor_lock, flags);
|
2014-01-29 17:49:19 +08:00
|
|
|
minor = idr_find(&drm_minors_idr, minor_id);
|
2014-02-24 22:53:25 +08:00
|
|
|
if (minor)
|
2017-09-26 16:28:49 +08:00
|
|
|
drm_dev_get(minor->dev);
|
2014-02-24 22:53:25 +08:00
|
|
|
spin_unlock_irqrestore(&drm_minor_lock, flags);
|
|
|
|
|
|
|
|
if (!minor) {
|
|
|
|
return ERR_PTR(-ENODEV);
|
2017-08-02 19:56:02 +08:00
|
|
|
} else if (drm_dev_is_unplugged(minor->dev)) {
|
2017-09-26 16:28:49 +08:00
|
|
|
drm_dev_put(minor->dev);
|
2014-01-29 17:49:19 +08:00
|
|
|
return ERR_PTR(-ENODEV);
|
2014-02-24 22:53:25 +08:00
|
|
|
}
|
2008-07-31 03:06:12 +08:00
|
|
|
|
2014-01-29 17:49:19 +08:00
|
|
|
return minor;
|
|
|
|
}
|
2005-09-25 12:28:13 +08:00
|
|
|
|
2014-01-29 17:49:19 +08:00
|
|
|
void drm_minor_release(struct drm_minor *minor)
|
|
|
|
{
|
2017-09-26 16:28:49 +08:00
|
|
|
drm_dev_put(minor->dev);
|
2005-04-17 06:20:36 +08:00
|
|
|
}
|
2009-01-05 05:55:33 +08:00
|
|
|
|
2015-09-29 03:46:35 +08:00
|
|
|
/**
|
|
|
|
* DOC: driver instance overview
|
|
|
|
*
|
2016-12-30 04:48:26 +08:00
|
|
|
* A device instance for a drm driver is represented by &struct drm_device. This
|
2019-01-12 00:40:44 +08:00
|
|
|
* is initialized with drm_dev_init(), usually from bus-specific ->probe()
|
2015-09-29 03:46:35 +08:00
|
|
|
* callbacks implemented by the driver. The driver then needs to initialize all
|
|
|
|
* the various subsystems for the drm device like memory management, vblank
|
|
|
|
* handling, modesetting support and intial output configuration plus obviously
|
2019-01-12 00:40:44 +08:00
|
|
|
* initialize all the corresponding hardware bits. Finally when everything is up
|
|
|
|
* and running and ready for userspace the device instance can be published
|
|
|
|
* using drm_dev_register().
|
2015-09-29 03:46:35 +08:00
|
|
|
*
|
|
|
|
* There is also deprecated support for initalizing device instances using
|
2017-01-25 14:26:47 +08:00
|
|
|
* bus-specific helpers and the &drm_driver.load callback. But due to
|
2015-09-29 03:46:35 +08:00
|
|
|
* backwards-compatibility needs the device instance have to be published too
|
|
|
|
* early, which requires unpretty global locking to make safe and is therefore
|
|
|
|
* only support for existing drivers not yet converted to the new scheme.
|
|
|
|
*
|
|
|
|
* When cleaning up a device instance everything needs to be done in reverse:
|
|
|
|
* First unpublish the device instance with drm_dev_unregister(). Then clean up
|
|
|
|
* any other resources allocated at device initialization and drop the driver's
|
2017-09-26 16:28:49 +08:00
|
|
|
* reference to &drm_device using drm_dev_put().
|
2015-09-29 03:46:35 +08:00
|
|
|
*
|
|
|
|
* Note that the lifetime rules for &drm_device instance has still a lot of
|
|
|
|
* historical baggage. Hence use the reference counting provided by
|
2017-09-26 16:28:49 +08:00
|
|
|
* drm_dev_get() and drm_dev_put() only carefully.
|
2019-02-25 22:42:28 +08:00
|
|
|
*
|
|
|
|
* Display driver example
|
|
|
|
* ~~~~~~~~~~~~~~~~~~~~~~
|
|
|
|
*
|
|
|
|
* The following example shows a typical structure of a DRM display driver.
|
|
|
|
* The example focus on the probe() function and the other functions that is
|
|
|
|
* almost always present and serves as a demonstration of devm_drm_dev_init()
|
|
|
|
* usage with its accompanying drm_driver->release callback.
|
|
|
|
*
|
|
|
|
* .. code-block:: c
|
|
|
|
*
|
|
|
|
* struct driver_device {
|
|
|
|
* struct drm_device drm;
|
|
|
|
* void *userspace_facing;
|
|
|
|
* struct clk *pclk;
|
|
|
|
* };
|
|
|
|
*
|
|
|
|
* static void driver_drm_release(struct drm_device *drm)
|
|
|
|
* {
|
|
|
|
* struct driver_device *priv = container_of(...);
|
|
|
|
*
|
|
|
|
* drm_mode_config_cleanup(drm);
|
|
|
|
* drm_dev_fini(drm);
|
|
|
|
* kfree(priv->userspace_facing);
|
|
|
|
* kfree(priv);
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* static struct drm_driver driver_drm_driver = {
|
|
|
|
* [...]
|
|
|
|
* .release = driver_drm_release,
|
|
|
|
* };
|
|
|
|
*
|
|
|
|
* static int driver_probe(struct platform_device *pdev)
|
|
|
|
* {
|
|
|
|
* struct driver_device *priv;
|
|
|
|
* struct drm_device *drm;
|
|
|
|
* int ret;
|
|
|
|
*
|
|
|
|
* [
|
|
|
|
* devm_kzalloc() can't be used here because the drm_device
|
|
|
|
* lifetime can exceed the device lifetime if driver unbind
|
|
|
|
* happens when userspace still has open file descriptors.
|
|
|
|
* ]
|
|
|
|
* priv = kzalloc(sizeof(*priv), GFP_KERNEL);
|
|
|
|
* if (!priv)
|
|
|
|
* return -ENOMEM;
|
|
|
|
*
|
|
|
|
* drm = &priv->drm;
|
|
|
|
*
|
|
|
|
* ret = devm_drm_dev_init(&pdev->dev, drm, &driver_drm_driver);
|
|
|
|
* if (ret) {
|
|
|
|
* kfree(drm);
|
|
|
|
* return ret;
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* drm_mode_config_init(drm);
|
|
|
|
*
|
|
|
|
* priv->userspace_facing = kzalloc(..., GFP_KERNEL);
|
|
|
|
* if (!priv->userspace_facing)
|
|
|
|
* return -ENOMEM;
|
|
|
|
*
|
|
|
|
* priv->pclk = devm_clk_get(dev, "PCLK");
|
|
|
|
* if (IS_ERR(priv->pclk))
|
|
|
|
* return PTR_ERR(priv->pclk);
|
|
|
|
*
|
|
|
|
* [ Further setup, display pipeline etc ]
|
|
|
|
*
|
|
|
|
* platform_set_drvdata(pdev, drm);
|
|
|
|
*
|
|
|
|
* drm_mode_config_reset(drm);
|
|
|
|
*
|
|
|
|
* ret = drm_dev_register(drm);
|
|
|
|
* if (ret)
|
|
|
|
* return ret;
|
|
|
|
*
|
|
|
|
* drm_fbdev_generic_setup(drm, 32);
|
|
|
|
*
|
|
|
|
* return 0;
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* [ This function is called before the devm_ resources are released ]
|
|
|
|
* static int driver_remove(struct platform_device *pdev)
|
|
|
|
* {
|
|
|
|
* struct drm_device *drm = platform_get_drvdata(pdev);
|
|
|
|
*
|
|
|
|
* drm_dev_unregister(drm);
|
|
|
|
* drm_atomic_helper_shutdown(drm)
|
|
|
|
*
|
|
|
|
* return 0;
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* [ This function is called on kernel restart and shutdown ]
|
|
|
|
* static void driver_shutdown(struct platform_device *pdev)
|
|
|
|
* {
|
|
|
|
* drm_atomic_helper_shutdown(platform_get_drvdata(pdev));
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* static int __maybe_unused driver_pm_suspend(struct device *dev)
|
|
|
|
* {
|
|
|
|
* return drm_mode_config_helper_suspend(dev_get_drvdata(dev));
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* static int __maybe_unused driver_pm_resume(struct device *dev)
|
|
|
|
* {
|
|
|
|
* drm_mode_config_helper_resume(dev_get_drvdata(dev));
|
|
|
|
*
|
|
|
|
* return 0;
|
|
|
|
* }
|
|
|
|
*
|
|
|
|
* static const struct dev_pm_ops driver_pm_ops = {
|
|
|
|
* SET_SYSTEM_SLEEP_PM_OPS(driver_pm_suspend, driver_pm_resume)
|
|
|
|
* };
|
|
|
|
*
|
|
|
|
* static struct platform_driver driver_driver = {
|
|
|
|
* .driver = {
|
|
|
|
* [...]
|
|
|
|
* .pm = &driver_pm_ops,
|
|
|
|
* },
|
|
|
|
* .probe = driver_probe,
|
|
|
|
* .remove = driver_remove,
|
|
|
|
* .shutdown = driver_shutdown,
|
|
|
|
* };
|
|
|
|
* module_platform_driver(driver_driver);
|
|
|
|
*
|
|
|
|
* Drivers that want to support device unplugging (USB, DT overlay unload) should
|
|
|
|
* use drm_dev_unplug() instead of drm_dev_unregister(). The driver must protect
|
|
|
|
* regions that is accessing device resources to prevent use after they're
|
|
|
|
* released. This is done using drm_dev_enter() and drm_dev_exit(). There is one
|
|
|
|
* shortcoming however, drm_dev_unplug() marks the drm_device as unplugged before
|
|
|
|
* drm_atomic_helper_shutdown() is called. This means that if the disable code
|
|
|
|
* paths are protected, they will not run on regular driver module unload,
|
|
|
|
* possibily leaving the hardware enabled.
|
2015-09-29 03:46:35 +08:00
|
|
|
*/
|
|
|
|
|
2009-01-05 05:55:33 +08:00
|
|
|
/**
|
2014-05-19 19:39:07 +08:00
|
|
|
* drm_put_dev - Unregister and release a DRM device
|
|
|
|
* @dev: DRM device
|
2009-01-05 05:55:33 +08:00
|
|
|
*
|
2014-05-19 19:39:07 +08:00
|
|
|
* Called at module unload time or when a PCI device is unplugged.
|
2009-01-05 05:55:33 +08:00
|
|
|
*
|
2014-05-19 19:39:07 +08:00
|
|
|
* Cleans up all DRM device, calling drm_lastclose().
|
2015-09-29 03:46:35 +08:00
|
|
|
*
|
|
|
|
* Note: Use of this function is deprecated. It will eventually go away
|
2017-09-26 16:28:49 +08:00
|
|
|
* completely. Please use drm_dev_unregister() and drm_dev_put() explicitly
|
2015-09-29 03:46:35 +08:00
|
|
|
* instead to make sure that the device isn't userspace accessible any more
|
|
|
|
* while teardown is in progress, ensuring that userspace can't access an
|
|
|
|
* inconsistent state.
|
2009-01-05 05:55:33 +08:00
|
|
|
*/
|
|
|
|
void drm_put_dev(struct drm_device *dev)
|
|
|
|
{
|
|
|
|
DRM_DEBUG("\n");
|
|
|
|
|
|
|
|
if (!dev) {
|
|
|
|
DRM_ERROR("cleanup called no dev\n");
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2013-10-02 17:23:38 +08:00
|
|
|
drm_dev_unregister(dev);
|
2017-09-26 16:28:49 +08:00
|
|
|
drm_dev_put(dev);
|
2009-01-05 05:55:33 +08:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_put_dev);
|
2012-02-20 22:18:07 +08:00
|
|
|
|
2018-03-28 15:38:35 +08:00
|
|
|
/**
|
|
|
|
* drm_dev_enter - Enter device critical section
|
|
|
|
* @dev: DRM device
|
|
|
|
* @idx: Pointer to index that will be passed to the matching drm_dev_exit()
|
|
|
|
*
|
|
|
|
* This function marks and protects the beginning of a section that should not
|
|
|
|
* be entered after the device has been unplugged. The section end is marked
|
|
|
|
* with drm_dev_exit(). Calls to this function can be nested.
|
|
|
|
*
|
|
|
|
* Returns:
|
|
|
|
* True if it is OK to enter the section, false otherwise.
|
|
|
|
*/
|
|
|
|
bool drm_dev_enter(struct drm_device *dev, int *idx)
|
2017-08-02 19:56:02 +08:00
|
|
|
{
|
2018-03-28 15:38:35 +08:00
|
|
|
*idx = srcu_read_lock(&drm_unplug_srcu);
|
|
|
|
|
|
|
|
if (dev->unplugged) {
|
|
|
|
srcu_read_unlock(&drm_unplug_srcu, *idx);
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
return true;
|
2017-08-02 19:56:02 +08:00
|
|
|
}
|
2018-03-28 15:38:35 +08:00
|
|
|
EXPORT_SYMBOL(drm_dev_enter);
|
|
|
|
|
|
|
|
/**
|
|
|
|
* drm_dev_exit - Exit device critical section
|
|
|
|
* @idx: index returned from drm_dev_enter()
|
|
|
|
*
|
|
|
|
* This function marks the end of a section that should not be entered after
|
|
|
|
* the device has been unplugged.
|
|
|
|
*/
|
|
|
|
void drm_dev_exit(int idx)
|
2017-08-02 19:56:02 +08:00
|
|
|
{
|
2018-03-28 15:38:35 +08:00
|
|
|
srcu_read_unlock(&drm_unplug_srcu, idx);
|
2017-08-02 19:56:02 +08:00
|
|
|
}
|
2018-03-28 15:38:35 +08:00
|
|
|
EXPORT_SYMBOL(drm_dev_exit);
|
2017-08-02 19:56:02 +08:00
|
|
|
|
|
|
|
/**
|
|
|
|
* drm_dev_unplug - unplug a DRM device
|
|
|
|
* @dev: DRM device
|
|
|
|
*
|
|
|
|
* This unplugs a hotpluggable DRM device, which makes it inaccessible to
|
2018-03-28 15:38:35 +08:00
|
|
|
* userspace operations. Entry-points can use drm_dev_enter() and
|
|
|
|
* drm_dev_exit() to protect device resources in a race free manner. This
|
2017-08-02 19:56:02 +08:00
|
|
|
* essentially unregisters the device like drm_dev_unregister(), but can be
|
|
|
|
* called while there are still open users of @dev.
|
|
|
|
*/
|
|
|
|
void drm_dev_unplug(struct drm_device *dev)
|
2012-02-20 22:18:07 +08:00
|
|
|
{
|
2018-03-28 15:38:35 +08:00
|
|
|
/*
|
|
|
|
* After synchronizing any critical read section is guaranteed to see
|
|
|
|
* the new value of ->unplugged, and any critical section which might
|
|
|
|
* still have seen the old value of ->unplugged is guaranteed to have
|
|
|
|
* finished.
|
|
|
|
*/
|
|
|
|
dev->unplugged = true;
|
|
|
|
synchronize_srcu(&drm_unplug_srcu);
|
2018-05-22 22:13:04 +08:00
|
|
|
|
|
|
|
drm_dev_unregister(dev);
|
2012-02-20 22:18:07 +08:00
|
|
|
}
|
2017-08-02 19:56:02 +08:00
|
|
|
EXPORT_SYMBOL(drm_dev_unplug);
|
2013-10-02 17:23:34 +08:00
|
|
|
|
2014-01-03 21:09:47 +08:00
|
|
|
/*
|
|
|
|
* DRM internal mount
|
|
|
|
* We want to be able to allocate our own "struct address_space" to control
|
|
|
|
* memory-mappings in VRAM (or stolen RAM, ...). However, core MM does not allow
|
|
|
|
* stand-alone address_space objects, so we need an underlying inode. As there
|
|
|
|
* is no way to allocate an independent inode easily, we need a fake internal
|
|
|
|
* VFS mount-point.
|
|
|
|
*
|
|
|
|
* The drm_fs_inode_new() function allocates a new inode, drm_fs_inode_free()
|
|
|
|
* frees it again. You are allowed to use iget() and iput() to get references to
|
|
|
|
* the inode. But each drm_fs_inode_new() call must be paired with exactly one
|
|
|
|
* drm_fs_inode_free() call (which does not have to be the last iput()).
|
|
|
|
* We use drm_fs_inode_*() to manage our internal VFS mount-point and share it
|
|
|
|
* between multiple inode-users. You could, technically, call
|
|
|
|
* iget() + drm_fs_inode_free() directly after alloc and sometime later do an
|
|
|
|
* iput(), but this way you'd end up with a new vfsmount for each inode.
|
|
|
|
*/
|
|
|
|
|
|
|
|
static int drm_fs_cnt;
|
|
|
|
static struct vfsmount *drm_fs_mnt;
|
|
|
|
|
|
|
|
static const struct dentry_operations drm_fs_dops = {
|
|
|
|
.d_dname = simple_dname,
|
|
|
|
};
|
|
|
|
|
|
|
|
static const struct super_operations drm_fs_sops = {
|
|
|
|
.statfs = simple_statfs,
|
|
|
|
};
|
|
|
|
|
|
|
|
static struct dentry *drm_fs_mount(struct file_system_type *fs_type, int flags,
|
|
|
|
const char *dev_name, void *data)
|
|
|
|
{
|
|
|
|
return mount_pseudo(fs_type,
|
|
|
|
"drm:",
|
|
|
|
&drm_fs_sops,
|
|
|
|
&drm_fs_dops,
|
|
|
|
0x010203ff);
|
|
|
|
}
|
|
|
|
|
|
|
|
static struct file_system_type drm_fs_type = {
|
|
|
|
.name = "drm",
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
.mount = drm_fs_mount,
|
|
|
|
.kill_sb = kill_anon_super,
|
|
|
|
};
|
|
|
|
|
|
|
|
static struct inode *drm_fs_inode_new(void)
|
|
|
|
{
|
|
|
|
struct inode *inode;
|
|
|
|
int r;
|
|
|
|
|
|
|
|
r = simple_pin_fs(&drm_fs_type, &drm_fs_mnt, &drm_fs_cnt);
|
|
|
|
if (r < 0) {
|
|
|
|
DRM_ERROR("Cannot mount pseudo fs: %d\n", r);
|
|
|
|
return ERR_PTR(r);
|
|
|
|
}
|
|
|
|
|
|
|
|
inode = alloc_anon_inode(drm_fs_mnt->mnt_sb);
|
|
|
|
if (IS_ERR(inode))
|
|
|
|
simple_release_fs(&drm_fs_mnt, &drm_fs_cnt);
|
|
|
|
|
|
|
|
return inode;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void drm_fs_inode_free(struct inode *inode)
|
|
|
|
{
|
|
|
|
if (inode) {
|
|
|
|
iput(inode);
|
|
|
|
simple_release_fs(&drm_fs_mnt, &drm_fs_cnt);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2019-02-13 00:46:15 +08:00
|
|
|
/**
|
|
|
|
* DOC: component helper usage recommendations
|
|
|
|
*
|
|
|
|
* DRM drivers that drive hardware where a logical device consists of a pile of
|
|
|
|
* independent hardware blocks are recommended to use the :ref:`component helper
|
|
|
|
* library<component>`. For consistency and better options for code reuse the
|
|
|
|
* following guidelines apply:
|
|
|
|
*
|
|
|
|
* - The entire device initialization procedure should be run from the
|
|
|
|
* &component_master_ops.master_bind callback, starting with drm_dev_init(),
|
|
|
|
* then binding all components with component_bind_all() and finishing with
|
|
|
|
* drm_dev_register().
|
|
|
|
*
|
|
|
|
* - The opaque pointer passed to all components through component_bind_all()
|
|
|
|
* should point at &struct drm_device of the device instance, not some driver
|
|
|
|
* specific private structure.
|
|
|
|
*
|
|
|
|
* - The component helper fills the niche where further standardization of
|
|
|
|
* interfaces is not practical. When there already is, or will be, a
|
|
|
|
* standardized interface like &drm_bridge or &drm_panel, providing its own
|
|
|
|
* functions to find such components at driver load time, like
|
|
|
|
* drm_of_find_panel_or_bridge(), then the component helper should not be
|
|
|
|
* used.
|
|
|
|
*/
|
|
|
|
|
2013-10-02 17:23:34 +08:00
|
|
|
/**
|
2016-06-15 20:17:46 +08:00
|
|
|
* drm_dev_init - Initialise new DRM device
|
|
|
|
* @dev: DRM device
|
|
|
|
* @driver: DRM driver
|
2013-10-02 17:23:34 +08:00
|
|
|
* @parent: Parent device object
|
|
|
|
*
|
2016-06-15 20:17:46 +08:00
|
|
|
* Initialize a new DRM device. No device registration is done.
|
2013-10-02 17:23:35 +08:00
|
|
|
* Call drm_dev_register() to advertice the device to user space and register it
|
2015-09-29 03:46:35 +08:00
|
|
|
* with other core subsystems. This should be done last in the device
|
|
|
|
* initialization sequence to make sure userspace can't access an inconsistent
|
|
|
|
* state.
|
2013-10-02 17:23:34 +08:00
|
|
|
*
|
2017-09-26 16:28:49 +08:00
|
|
|
* The initial ref-count of the object is 1. Use drm_dev_get() and
|
|
|
|
* drm_dev_put() to take and drop further ref-counts.
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
*
|
2019-01-12 00:40:44 +08:00
|
|
|
* It is recommended that drivers embed &struct drm_device into their own device
|
|
|
|
* structure.
|
|
|
|
*
|
2016-06-15 20:17:46 +08:00
|
|
|
* Drivers that do not want to allocate their own device struct
|
2016-12-30 04:48:26 +08:00
|
|
|
* embedding &struct drm_device can call drm_dev_alloc() instead. For drivers
|
|
|
|
* that do embed &struct drm_device it must be placed first in the overall
|
2016-12-08 18:28:47 +08:00
|
|
|
* structure, and the overall structure must be allocated using kmalloc(): The
|
|
|
|
* drm core's release function unconditionally calls kfree() on the @dev pointer
|
2017-02-02 17:36:32 +08:00
|
|
|
* when the final reference is released. To override this behaviour, and so
|
|
|
|
* allow embedding of the drm_device inside the driver's device struct at an
|
|
|
|
* arbitrary offset, you must supply a &drm_driver.release callback and control
|
|
|
|
* the finalization explicitly.
|
2016-06-15 20:17:46 +08:00
|
|
|
*
|
2013-10-02 17:23:34 +08:00
|
|
|
* RETURNS:
|
2016-06-15 20:17:46 +08:00
|
|
|
* 0 on success, or error code on failure.
|
2013-10-02 17:23:34 +08:00
|
|
|
*/
|
2016-06-15 20:17:46 +08:00
|
|
|
int drm_dev_init(struct drm_device *dev,
|
|
|
|
struct drm_driver *driver,
|
|
|
|
struct device *parent)
|
2013-10-02 17:23:34 +08:00
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
2017-07-09 05:43:52 +08:00
|
|
|
if (!drm_core_init_complete) {
|
|
|
|
DRM_ERROR("DRM core is not initialized\n");
|
|
|
|
return -ENODEV;
|
|
|
|
}
|
|
|
|
|
2018-10-24 00:35:49 +08:00
|
|
|
BUG_ON(!parent);
|
|
|
|
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
kref_init(&dev->ref);
|
2019-02-25 22:42:26 +08:00
|
|
|
dev->dev = get_device(parent);
|
2013-10-02 17:23:34 +08:00
|
|
|
dev->driver = driver;
|
|
|
|
|
2018-09-13 21:16:21 +08:00
|
|
|
/* no per-device feature limits by default */
|
|
|
|
dev->driver_features = ~0u;
|
|
|
|
|
2019-04-23 07:47:27 +08:00
|
|
|
drm_legacy_init_members(dev);
|
2013-10-02 17:23:34 +08:00
|
|
|
INIT_LIST_HEAD(&dev->filelist);
|
2018-07-04 00:03:47 +08:00
|
|
|
INIT_LIST_HEAD(&dev->filelist_internal);
|
|
|
|
INIT_LIST_HEAD(&dev->clientlist);
|
2013-10-02 17:23:34 +08:00
|
|
|
INIT_LIST_HEAD(&dev->vblank_event_list);
|
|
|
|
|
|
|
|
spin_lock_init(&dev->event_lock);
|
|
|
|
mutex_init(&dev->struct_mutex);
|
2016-04-27 01:29:41 +08:00
|
|
|
mutex_init(&dev->filelist_mutex);
|
2018-07-04 00:03:47 +08:00
|
|
|
mutex_init(&dev->clientlist_mutex);
|
2014-02-26 02:57:44 +08:00
|
|
|
mutex_init(&dev->master_mutex);
|
2013-10-02 17:23:34 +08:00
|
|
|
|
2014-01-03 21:24:19 +08:00
|
|
|
dev->anon_inode = drm_fs_inode_new();
|
|
|
|
if (IS_ERR(dev->anon_inode)) {
|
|
|
|
ret = PTR_ERR(dev->anon_inode);
|
|
|
|
DRM_ERROR("Cannot allocate anonymous inode: %d\n", ret);
|
2013-10-02 17:23:34 +08:00
|
|
|
goto err_free;
|
2014-01-03 21:24:19 +08:00
|
|
|
}
|
|
|
|
|
2014-03-16 21:38:40 +08:00
|
|
|
if (drm_core_check_feature(dev, DRIVER_RENDER)) {
|
2014-01-29 19:43:56 +08:00
|
|
|
ret = drm_minor_alloc(dev, DRM_MINOR_RENDER);
|
|
|
|
if (ret)
|
|
|
|
goto err_minors;
|
|
|
|
}
|
|
|
|
|
2016-08-04 02:04:25 +08:00
|
|
|
ret = drm_minor_alloc(dev, DRM_MINOR_PRIMARY);
|
2014-01-29 19:43:56 +08:00
|
|
|
if (ret)
|
|
|
|
goto err_minors;
|
|
|
|
|
2019-04-23 07:40:54 +08:00
|
|
|
ret = drm_legacy_create_map_hash(dev);
|
2016-06-15 20:17:46 +08:00
|
|
|
if (ret)
|
2014-01-29 19:43:56 +08:00
|
|
|
goto err_minors;
|
2013-10-02 17:23:34 +08:00
|
|
|
|
2015-06-23 17:22:36 +08:00
|
|
|
drm_legacy_ctxbitmap_init(dev);
|
2013-10-02 17:23:34 +08:00
|
|
|
|
2014-09-30 22:49:56 +08:00
|
|
|
if (drm_core_check_feature(dev, DRIVER_GEM)) {
|
2013-10-02 17:23:34 +08:00
|
|
|
ret = drm_gem_init(dev);
|
|
|
|
if (ret) {
|
|
|
|
DRM_ERROR("Cannot initialize graphics execution manager (GEM)\n");
|
|
|
|
goto err_ctxbitmap;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2018-10-24 00:35:49 +08:00
|
|
|
ret = drm_dev_set_unique(dev, dev_name(parent));
|
2016-06-21 16:54:14 +08:00
|
|
|
if (ret)
|
|
|
|
goto err_setunique;
|
2015-12-11 18:20:28 +08:00
|
|
|
|
2016-06-15 20:17:46 +08:00
|
|
|
return 0;
|
2013-10-02 17:23:34 +08:00
|
|
|
|
2015-12-11 18:20:28 +08:00
|
|
|
err_setunique:
|
|
|
|
if (drm_core_check_feature(dev, DRIVER_GEM))
|
|
|
|
drm_gem_destroy(dev);
|
2013-10-02 17:23:34 +08:00
|
|
|
err_ctxbitmap:
|
2014-07-24 18:10:04 +08:00
|
|
|
drm_legacy_ctxbitmap_cleanup(dev);
|
2019-04-23 07:40:54 +08:00
|
|
|
drm_legacy_remove_map_hash(dev);
|
2014-01-29 19:43:56 +08:00
|
|
|
err_minors:
|
2016-08-04 02:04:25 +08:00
|
|
|
drm_minor_free(dev, DRM_MINOR_PRIMARY);
|
2014-01-29 19:55:48 +08:00
|
|
|
drm_minor_free(dev, DRM_MINOR_RENDER);
|
2014-01-03 21:24:19 +08:00
|
|
|
drm_fs_inode_free(dev->anon_inode);
|
2013-10-02 17:23:34 +08:00
|
|
|
err_free:
|
2019-02-25 22:42:26 +08:00
|
|
|
put_device(dev->dev);
|
2014-02-26 02:57:44 +08:00
|
|
|
mutex_destroy(&dev->master_mutex);
|
2018-07-04 00:03:47 +08:00
|
|
|
mutex_destroy(&dev->clientlist_mutex);
|
2016-11-10 21:50:35 +08:00
|
|
|
mutex_destroy(&dev->filelist_mutex);
|
|
|
|
mutex_destroy(&dev->struct_mutex);
|
2019-04-23 07:47:27 +08:00
|
|
|
drm_legacy_destroy_members(dev);
|
2016-06-15 20:17:46 +08:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_dev_init);
|
|
|
|
|
2019-02-25 22:42:27 +08:00
|
|
|
static void devm_drm_dev_init_release(void *data)
|
|
|
|
{
|
|
|
|
drm_dev_put(data);
|
|
|
|
}
|
|
|
|
|
|
|
|
/**
|
|
|
|
* devm_drm_dev_init - Resource managed drm_dev_init()
|
|
|
|
* @parent: Parent device object
|
|
|
|
* @dev: DRM device
|
|
|
|
* @driver: DRM driver
|
|
|
|
*
|
|
|
|
* Managed drm_dev_init(). The DRM device initialized with this function is
|
|
|
|
* automatically put on driver detach using drm_dev_put(). You must supply a
|
|
|
|
* &drm_driver.release callback to control the finalization explicitly.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
|
|
|
* 0 on success, or error code on failure.
|
|
|
|
*/
|
|
|
|
int devm_drm_dev_init(struct device *parent,
|
|
|
|
struct drm_device *dev,
|
|
|
|
struct drm_driver *driver)
|
|
|
|
{
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (WARN_ON(!parent || !driver->release))
|
|
|
|
return -EINVAL;
|
|
|
|
|
|
|
|
ret = drm_dev_init(dev, driver, parent);
|
|
|
|
if (ret)
|
|
|
|
return ret;
|
|
|
|
|
|
|
|
ret = devm_add_action(parent, devm_drm_dev_init_release, dev);
|
|
|
|
if (ret)
|
|
|
|
devm_drm_dev_init_release(dev);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(devm_drm_dev_init);
|
|
|
|
|
2017-02-02 17:36:32 +08:00
|
|
|
/**
|
|
|
|
* drm_dev_fini - Finalize a dead DRM device
|
|
|
|
* @dev: DRM device
|
|
|
|
*
|
|
|
|
* Finalize a dead DRM device. This is the converse to drm_dev_init() and
|
|
|
|
* frees up all data allocated by it. All driver private data should be
|
|
|
|
* finalized first. Note that this function does not free the @dev, that is
|
|
|
|
* left to the caller.
|
|
|
|
*
|
|
|
|
* The ref-count of @dev must be zero, and drm_dev_fini() should only be called
|
|
|
|
* from a &drm_driver.release callback.
|
|
|
|
*/
|
|
|
|
void drm_dev_fini(struct drm_device *dev)
|
|
|
|
{
|
|
|
|
drm_vblank_cleanup(dev);
|
|
|
|
|
|
|
|
if (drm_core_check_feature(dev, DRIVER_GEM))
|
|
|
|
drm_gem_destroy(dev);
|
|
|
|
|
|
|
|
drm_legacy_ctxbitmap_cleanup(dev);
|
2019-04-23 07:40:54 +08:00
|
|
|
drm_legacy_remove_map_hash(dev);
|
2017-02-02 17:36:32 +08:00
|
|
|
drm_fs_inode_free(dev->anon_inode);
|
|
|
|
|
|
|
|
drm_minor_free(dev, DRM_MINOR_PRIMARY);
|
|
|
|
drm_minor_free(dev, DRM_MINOR_RENDER);
|
|
|
|
|
2019-02-25 22:42:26 +08:00
|
|
|
put_device(dev->dev);
|
|
|
|
|
2017-02-02 17:36:32 +08:00
|
|
|
mutex_destroy(&dev->master_mutex);
|
2018-07-04 00:03:47 +08:00
|
|
|
mutex_destroy(&dev->clientlist_mutex);
|
2017-02-02 17:36:32 +08:00
|
|
|
mutex_destroy(&dev->filelist_mutex);
|
|
|
|
mutex_destroy(&dev->struct_mutex);
|
2019-04-23 07:47:27 +08:00
|
|
|
drm_legacy_destroy_members(dev);
|
2017-02-02 17:36:32 +08:00
|
|
|
kfree(dev->unique);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_dev_fini);
|
|
|
|
|
2016-06-15 20:17:46 +08:00
|
|
|
/**
|
|
|
|
* drm_dev_alloc - Allocate new DRM device
|
|
|
|
* @driver: DRM driver to allocate device for
|
|
|
|
* @parent: Parent device object
|
|
|
|
*
|
|
|
|
* Allocate and initialize a new DRM device. No device registration is done.
|
|
|
|
* Call drm_dev_register() to advertice the device to user space and register it
|
|
|
|
* with other core subsystems. This should be done last in the device
|
|
|
|
* initialization sequence to make sure userspace can't access an inconsistent
|
|
|
|
* state.
|
|
|
|
*
|
2017-09-26 16:28:49 +08:00
|
|
|
* The initial ref-count of the object is 1. Use drm_dev_get() and
|
|
|
|
* drm_dev_put() to take and drop further ref-counts.
|
2016-06-15 20:17:46 +08:00
|
|
|
*
|
|
|
|
* Note that for purely virtual devices @parent can be NULL.
|
|
|
|
*
|
2016-12-30 04:48:26 +08:00
|
|
|
* Drivers that wish to subclass or embed &struct drm_device into their
|
2016-06-15 20:17:46 +08:00
|
|
|
* own struct should look at using drm_dev_init() instead.
|
|
|
|
*
|
|
|
|
* RETURNS:
|
2016-09-21 22:59:19 +08:00
|
|
|
* Pointer to new DRM device, or ERR_PTR on failure.
|
2016-06-15 20:17:46 +08:00
|
|
|
*/
|
|
|
|
struct drm_device *drm_dev_alloc(struct drm_driver *driver,
|
|
|
|
struct device *parent)
|
|
|
|
{
|
|
|
|
struct drm_device *dev;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
dev = kzalloc(sizeof(*dev), GFP_KERNEL);
|
|
|
|
if (!dev)
|
2016-09-21 22:59:19 +08:00
|
|
|
return ERR_PTR(-ENOMEM);
|
2016-06-15 20:17:46 +08:00
|
|
|
|
|
|
|
ret = drm_dev_init(dev, driver, parent);
|
|
|
|
if (ret) {
|
|
|
|
kfree(dev);
|
2016-09-21 22:59:19 +08:00
|
|
|
return ERR_PTR(ret);
|
2016-06-15 20:17:46 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
return dev;
|
2013-10-02 17:23:34 +08:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_dev_alloc);
|
2013-10-02 17:23:35 +08:00
|
|
|
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
static void drm_dev_release(struct kref *ref)
|
2013-10-02 17:23:37 +08:00
|
|
|
{
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
struct drm_device *dev = container_of(ref, struct drm_device, ref);
|
2013-10-21 00:55:45 +08:00
|
|
|
|
2017-02-02 17:36:32 +08:00
|
|
|
if (dev->driver->release) {
|
|
|
|
dev->driver->release(dev);
|
|
|
|
} else {
|
|
|
|
drm_dev_fini(dev);
|
|
|
|
kfree(dev);
|
|
|
|
}
|
2013-10-02 17:23:37 +08:00
|
|
|
}
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
|
|
|
|
/**
|
2017-09-26 16:28:49 +08:00
|
|
|
* drm_dev_get - Take reference of a DRM device
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
* @dev: device to take reference of or NULL
|
|
|
|
*
|
|
|
|
* This increases the ref-count of @dev by one. You *must* already own a
|
2017-09-26 16:28:49 +08:00
|
|
|
* reference when calling this. Use drm_dev_put() to drop this reference
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
* again.
|
|
|
|
*
|
|
|
|
* This function never fails. However, this function does not provide *any*
|
|
|
|
* guarantee whether the device is alive or running. It only provides a
|
|
|
|
* reference to the object and the memory associated with it.
|
|
|
|
*/
|
2017-09-26 16:28:49 +08:00
|
|
|
void drm_dev_get(struct drm_device *dev)
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
{
|
|
|
|
if (dev)
|
|
|
|
kref_get(&dev->ref);
|
|
|
|
}
|
2017-09-26 16:28:49 +08:00
|
|
|
EXPORT_SYMBOL(drm_dev_get);
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
|
|
|
|
/**
|
2017-09-26 16:28:49 +08:00
|
|
|
* drm_dev_put - Drop reference of a DRM device
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
* @dev: device to drop reference of or NULL
|
|
|
|
*
|
|
|
|
* This decreases the ref-count of @dev by one. The device is destroyed if the
|
|
|
|
* ref-count drops to zero.
|
|
|
|
*/
|
2017-09-26 16:28:49 +08:00
|
|
|
void drm_dev_put(struct drm_device *dev)
|
drm: provide device-refcount
Lets not trick ourselves into thinking "drm_device" objects are not
ref-counted. That's just utterly stupid. We manage "drm_minor" objects on
each drm-device and each minor can have an unlimited number of open
handles. Each of these handles has the drm_minor (and thus the drm_device)
as private-data in the file-handle. Therefore, we may not destroy
"drm_device" until all these handles are closed.
It is *not* possible to reset all these pointers atomically and restrict
access to them, and this is *not* how this is done! Instead, we use
ref-counts to make sure the object is valid and not freed.
Note that we currently use "dev->open_count" for that, which is *exactly*
the same as a reference-count, just open coded. So this patch doesn't
change any semantics on DRM devices (well, this patch just introduces the
ref-count, anyway. Follow-up patches will replace open_count by it).
Also note that generic VFS revoke support could allow us to drop this
ref-count again. We could then just synchronously disable any fops->xy()
calls. However, this is not the case, yet, and no such patches are
in sight (and I seriously question the idea of dropping the ref-cnt
again).
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
2014-01-29 17:21:36 +08:00
|
|
|
{
|
|
|
|
if (dev)
|
|
|
|
kref_put(&dev->ref, drm_dev_release);
|
|
|
|
}
|
2017-09-26 16:28:49 +08:00
|
|
|
EXPORT_SYMBOL(drm_dev_put);
|
|
|
|
|
2016-12-09 21:56:56 +08:00
|
|
|
static int create_compat_control_link(struct drm_device *dev)
|
|
|
|
{
|
|
|
|
struct drm_minor *minor;
|
|
|
|
char *name;
|
|
|
|
int ret;
|
|
|
|
|
|
|
|
if (!drm_core_check_feature(dev, DRIVER_MODESET))
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
minor = *drm_minor_get_slot(dev, DRM_MINOR_PRIMARY);
|
|
|
|
if (!minor)
|
|
|
|
return 0;
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Some existing userspace out there uses the existing of the controlD*
|
|
|
|
* sysfs files to figure out whether it's a modeset driver. It only does
|
|
|
|
* readdir, hence a symlink is sufficient (and the least confusing
|
|
|
|
* option). Otherwise controlD* is entirely unused.
|
|
|
|
*
|
|
|
|
* Old controlD chardev have been allocated in the range
|
|
|
|
* 64-127.
|
|
|
|
*/
|
|
|
|
name = kasprintf(GFP_KERNEL, "controlD%d", minor->index + 64);
|
|
|
|
if (!name)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
ret = sysfs_create_link(minor->kdev->kobj.parent,
|
|
|
|
&minor->kdev->kobj,
|
|
|
|
name);
|
|
|
|
|
|
|
|
kfree(name);
|
|
|
|
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void remove_compat_control_link(struct drm_device *dev)
|
|
|
|
{
|
|
|
|
struct drm_minor *minor;
|
|
|
|
char *name;
|
|
|
|
|
|
|
|
if (!drm_core_check_feature(dev, DRIVER_MODESET))
|
|
|
|
return;
|
|
|
|
|
|
|
|
minor = *drm_minor_get_slot(dev, DRM_MINOR_PRIMARY);
|
|
|
|
if (!minor)
|
|
|
|
return;
|
|
|
|
|
2018-05-11 12:15:42 +08:00
|
|
|
name = kasprintf(GFP_KERNEL, "controlD%d", minor->index + 64);
|
2016-12-09 21:56:56 +08:00
|
|
|
if (!name)
|
|
|
|
return;
|
|
|
|
|
|
|
|
sysfs_remove_link(minor->kdev->kobj.parent, name);
|
|
|
|
|
|
|
|
kfree(name);
|
|
|
|
}
|
|
|
|
|
2013-10-02 17:23:35 +08:00
|
|
|
/**
|
|
|
|
* drm_dev_register - Register DRM device
|
|
|
|
* @dev: Device to register
|
2014-05-19 19:39:07 +08:00
|
|
|
* @flags: Flags passed to the driver's .load() function
|
2013-10-02 17:23:35 +08:00
|
|
|
*
|
|
|
|
* Register the DRM device @dev with the system, advertise device to user-space
|
2019-01-12 00:40:44 +08:00
|
|
|
* and start normal device operation. @dev must be initialized via drm_dev_init()
|
2016-06-17 16:25:17 +08:00
|
|
|
* previously.
|
2013-10-02 17:23:35 +08:00
|
|
|
*
|
|
|
|
* Never call this twice on any device!
|
|
|
|
*
|
2015-09-29 03:46:35 +08:00
|
|
|
* NOTE: To ensure backward compatibility with existing drivers method this
|
2017-01-25 14:26:47 +08:00
|
|
|
* function calls the &drm_driver.load method after registering the device
|
|
|
|
* nodes, creating race conditions. Usage of the &drm_driver.load methods is
|
|
|
|
* therefore deprecated, drivers must perform all initialization before calling
|
2015-09-29 03:46:35 +08:00
|
|
|
* drm_dev_register().
|
|
|
|
*
|
2013-10-02 17:23:35 +08:00
|
|
|
* RETURNS:
|
|
|
|
* 0 on success, negative error code on failure.
|
|
|
|
*/
|
|
|
|
int drm_dev_register(struct drm_device *dev, unsigned long flags)
|
|
|
|
{
|
2016-12-28 22:32:11 +08:00
|
|
|
struct drm_driver *driver = dev->driver;
|
2013-10-02 17:23:35 +08:00
|
|
|
int ret;
|
|
|
|
|
|
|
|
mutex_lock(&drm_global_mutex);
|
|
|
|
|
2014-01-29 19:57:05 +08:00
|
|
|
ret = drm_minor_register(dev, DRM_MINOR_RENDER);
|
2014-01-29 19:43:56 +08:00
|
|
|
if (ret)
|
|
|
|
goto err_minors;
|
2013-10-02 17:23:35 +08:00
|
|
|
|
2016-08-04 02:04:25 +08:00
|
|
|
ret = drm_minor_register(dev, DRM_MINOR_PRIMARY);
|
2013-10-02 17:23:35 +08:00
|
|
|
if (ret)
|
2014-01-29 19:43:56 +08:00
|
|
|
goto err_minors;
|
2013-10-02 17:23:35 +08:00
|
|
|
|
2016-12-09 21:56:56 +08:00
|
|
|
ret = create_compat_control_link(dev);
|
|
|
|
if (ret)
|
|
|
|
goto err_minors;
|
|
|
|
|
2017-01-13 00:15:56 +08:00
|
|
|
dev->registered = true;
|
|
|
|
|
2013-10-02 17:23:35 +08:00
|
|
|
if (dev->driver->load) {
|
|
|
|
ret = dev->driver->load(dev, flags);
|
|
|
|
if (ret)
|
2014-01-29 19:43:56 +08:00
|
|
|
goto err_minors;
|
2013-10-02 17:23:35 +08:00
|
|
|
}
|
|
|
|
|
2016-06-18 21:46:41 +08:00
|
|
|
if (drm_core_check_feature(dev, DRIVER_MODESET))
|
2016-06-21 22:37:09 +08:00
|
|
|
drm_modeset_register_all(dev);
|
2016-06-17 16:25:17 +08:00
|
|
|
|
2013-10-02 17:23:35 +08:00
|
|
|
ret = 0;
|
2016-12-28 22:32:11 +08:00
|
|
|
|
|
|
|
DRM_INFO("Initialized %s %d.%d.%d %s for %s on minor %d\n",
|
|
|
|
driver->name, driver->major, driver->minor,
|
2016-12-30 22:16:39 +08:00
|
|
|
driver->patchlevel, driver->date,
|
|
|
|
dev->dev ? dev_name(dev->dev) : "virtual device",
|
2016-12-28 22:32:11 +08:00
|
|
|
dev->primary->index);
|
|
|
|
|
2013-10-02 17:23:35 +08:00
|
|
|
goto out_unlock;
|
|
|
|
|
2014-01-29 19:43:56 +08:00
|
|
|
err_minors:
|
2016-12-09 21:56:56 +08:00
|
|
|
remove_compat_control_link(dev);
|
2016-08-04 02:04:25 +08:00
|
|
|
drm_minor_unregister(dev, DRM_MINOR_PRIMARY);
|
2014-01-29 19:57:05 +08:00
|
|
|
drm_minor_unregister(dev, DRM_MINOR_RENDER);
|
2013-10-02 17:23:35 +08:00
|
|
|
out_unlock:
|
|
|
|
mutex_unlock(&drm_global_mutex);
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_dev_register);
|
2013-10-02 17:23:38 +08:00
|
|
|
|
|
|
|
/**
|
|
|
|
* drm_dev_unregister - Unregister DRM device
|
|
|
|
* @dev: Device to unregister
|
|
|
|
*
|
|
|
|
* Unregister the DRM device from the system. This does the reverse of
|
|
|
|
* drm_dev_register() but does not deallocate the device. The caller must call
|
2017-09-26 16:28:49 +08:00
|
|
|
* drm_dev_put() to drop their final reference.
|
2015-09-29 03:46:35 +08:00
|
|
|
*
|
2017-08-02 19:56:02 +08:00
|
|
|
* A special form of unregistering for hotpluggable devices is drm_dev_unplug(),
|
|
|
|
* which can be called while there are still open users of @dev.
|
|
|
|
*
|
2015-09-29 03:46:35 +08:00
|
|
|
* This should be called first in the device teardown code to make sure
|
|
|
|
* userspace can't access the device instance any more.
|
2013-10-02 17:23:38 +08:00
|
|
|
*/
|
|
|
|
void drm_dev_unregister(struct drm_device *dev)
|
|
|
|
{
|
drm: Only lastclose on unload for legacy drivers
The only thing modern drivers are supposed to do in lastclose is
restore the fb emulation state. Which is entirely optional, and
there's really no reason to do that. So restrict it to legacy drivers
(where the driver cleanup essentially happens in lastclose).
This will also allow us to share the unregister function with
drm_dev_unplug().
Quoting my reply to Alex on dri-devel:
On Thu, Aug 3, 2017 at 1:17 AM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
> On Wed, Aug 2, 2017 at 10:50 PM, Alex Deucher <alexdeucher@gmail.com> wrote:
>> On Wed, Aug 2, 2017 at 7:56 AM, Daniel Vetter <daniel.vetter@ffwll.ch> wrote:
>>> The only thing modern drivers are supposed to do in lastclose is
>>> restore the fb emulation state. Which is entirely optional, and
>>> there's really no reason to do that. So restrict it to legacy drivers
>>> (where the driver cleanup essentially happens in lastclose).
>>
>> vga_switcheroo_process_delayed_switch() gets called in lastclose.
>> Won't that need to get moved elsewhere for this to work?
>
> Hm right, I forgot the lazy way to do runtime pm by keeping the device
> alive as long as anyone has an open fd for it ... This shouldn't be a
> problem, since you need to unregister from vgaswitcheroo anyway on
> unload. Maybe that blows up, I'll check the code and augment the patch
> as needed.
So I think there's 3 cases:
- Trying to unload the module. You can't do that while anyone has the
fd still open, so lastclose is guaranteeed to run.
- Forcefully unbinding the driver through sysfs. Not any better, since
the can_switch stuff checks for the open count, and so will delay the
delayed switch no matter what.
- Actual hotremoval: a) not implemented since none of the drivers
taking part in vgaswitcheroo correctly unplug the drm device and b)
you can't hotremove a chip from a laptop.
v2: Extend commit message with m-l discussion.
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20170802115604.12734-4-daniel.vetter@ffwll.ch
2017-08-02 19:56:03 +08:00
|
|
|
if (drm_core_check_feature(dev, DRIVER_LEGACY))
|
|
|
|
drm_lastclose(dev);
|
2013-10-02 17:23:38 +08:00
|
|
|
|
2017-01-13 00:15:56 +08:00
|
|
|
dev->registered = false;
|
|
|
|
|
2018-07-04 00:03:47 +08:00
|
|
|
drm_client_dev_unregister(dev);
|
|
|
|
|
2016-06-18 21:46:41 +08:00
|
|
|
if (drm_core_check_feature(dev, DRIVER_MODESET))
|
2016-06-21 22:37:09 +08:00
|
|
|
drm_modeset_unregister_all(dev);
|
2016-06-17 16:25:17 +08:00
|
|
|
|
2013-10-02 17:23:38 +08:00
|
|
|
if (dev->driver->unload)
|
|
|
|
dev->driver->unload(dev);
|
|
|
|
|
2013-12-11 18:34:38 +08:00
|
|
|
if (dev->agp)
|
|
|
|
drm_pci_agp_destroy(dev);
|
2013-10-02 17:23:38 +08:00
|
|
|
|
2019-04-23 06:45:12 +08:00
|
|
|
drm_legacy_rmmaps(dev);
|
2013-10-02 17:23:38 +08:00
|
|
|
|
2016-12-09 21:56:56 +08:00
|
|
|
remove_compat_control_link(dev);
|
2016-08-04 02:04:25 +08:00
|
|
|
drm_minor_unregister(dev, DRM_MINOR_PRIMARY);
|
2014-01-29 19:57:05 +08:00
|
|
|
drm_minor_unregister(dev, DRM_MINOR_RENDER);
|
2013-10-02 17:23:38 +08:00
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_dev_unregister);
|
2014-04-11 21:23:00 +08:00
|
|
|
|
2016-10-04 17:09:35 +08:00
|
|
|
/**
|
|
|
|
* drm_dev_set_unique - Set the unique name of a DRM device
|
|
|
|
* @dev: device of which to set the unique name
|
|
|
|
* @name: unique name
|
|
|
|
*
|
2019-01-12 00:40:44 +08:00
|
|
|
* Sets the unique name of a DRM device using the specified string. This is
|
|
|
|
* already done by drm_dev_init(), drivers should only override the default
|
|
|
|
* unique name for backwards compatibility reasons.
|
2016-10-04 17:09:35 +08:00
|
|
|
*
|
|
|
|
* Return: 0 on success or a negative error code on failure.
|
|
|
|
*/
|
|
|
|
int drm_dev_set_unique(struct drm_device *dev, const char *name)
|
|
|
|
{
|
|
|
|
kfree(dev->unique);
|
|
|
|
dev->unique = kstrdup(name, GFP_KERNEL);
|
|
|
|
|
|
|
|
return dev->unique ? 0 : -ENOMEM;
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(drm_dev_set_unique);
|
|
|
|
|
2014-07-23 18:29:56 +08:00
|
|
|
/*
|
|
|
|
* DRM Core
|
|
|
|
* The DRM core module initializes all global DRM objects and makes them
|
|
|
|
* available to drivers. Once setup, drivers can probe their respective
|
|
|
|
* devices.
|
|
|
|
* Currently, core management includes:
|
|
|
|
* - The "DRM-Global" key/value database
|
|
|
|
* - Global ID management for connectors
|
|
|
|
* - DRM major number allocation
|
|
|
|
* - DRM minor management
|
|
|
|
* - DRM sysfs class
|
|
|
|
* - DRM debugfs root
|
|
|
|
*
|
|
|
|
* Furthermore, the DRM core provides dynamic char-dev lookups. For each
|
|
|
|
* interface registered on a DRM device, you can request minor numbers from DRM
|
|
|
|
* core. DRM core takes care of major-number management and char-dev
|
|
|
|
* registration. A stub ->open() callback forwards any open() requests to the
|
|
|
|
* registered minor.
|
|
|
|
*/
|
|
|
|
|
|
|
|
static int drm_stub_open(struct inode *inode, struct file *filp)
|
|
|
|
{
|
|
|
|
const struct file_operations *new_fops;
|
|
|
|
struct drm_minor *minor;
|
|
|
|
int err;
|
|
|
|
|
|
|
|
DRM_DEBUG("\n");
|
|
|
|
|
|
|
|
mutex_lock(&drm_global_mutex);
|
|
|
|
minor = drm_minor_acquire(iminor(inode));
|
|
|
|
if (IS_ERR(minor)) {
|
|
|
|
err = PTR_ERR(minor);
|
|
|
|
goto out_unlock;
|
|
|
|
}
|
|
|
|
|
|
|
|
new_fops = fops_get(minor->dev->driver->fops);
|
|
|
|
if (!new_fops) {
|
|
|
|
err = -ENODEV;
|
|
|
|
goto out_release;
|
|
|
|
}
|
|
|
|
|
|
|
|
replace_fops(filp, new_fops);
|
|
|
|
if (filp->f_op->open)
|
|
|
|
err = filp->f_op->open(inode, filp);
|
|
|
|
else
|
|
|
|
err = 0;
|
|
|
|
|
|
|
|
out_release:
|
|
|
|
drm_minor_release(minor);
|
|
|
|
out_unlock:
|
|
|
|
mutex_unlock(&drm_global_mutex);
|
|
|
|
return err;
|
|
|
|
}
|
|
|
|
|
|
|
|
static const struct file_operations drm_stub_fops = {
|
|
|
|
.owner = THIS_MODULE,
|
|
|
|
.open = drm_stub_open,
|
|
|
|
.llseek = noop_llseek,
|
|
|
|
};
|
|
|
|
|
drm: cleanup drm_core_{init,exit}()
Various cleanups to the DRM core initialization and exit handlers:
- Register chrdev last: Once register_chrdev() returns, open() will
succeed on the given chrdevs. This is usually not an issue, as no
chardevs are registered, yet. However, nodes can be created by
user-space via mknod(2), even though such major/minor combinations are
unknown to the kernel. Avoid calling into drm_stub_open() in those
cases.
Again, drm_stub_open() would just bail out as the inode is unknown,
but it's really non-obvious if you hack on drm_stub_open().
- Unify error-paths into just one label. All the error-path helpers can
be called even though the constructors were not called yet, or failed.
Hence, just call all cleanups unconditionally.
- Call into drm_global_release(). This is a no-op, but provides
debugging helpers in case there're GLOBALS left on module unload. This
function was unused until now.
- Use DRM_ERROR() instead of printk(), and also print the error-code on
failure (even if it is static!).
- Don't throw away error-codes of register_chrdev()!
- Don't hardcode -1 as errno. This is just plain wrong.
- Order exit-handlers in the exact reverse order of initialization
(except if the order actually matters for syncing-reasons, which is
not the case here, though).
v2:
- Call drm_core_exit() directly from the init-error-handler. Requires to
drop __exit annotation, though.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160901124837.680-7-dh.herrmann@gmail.com
2016-09-01 20:48:37 +08:00
|
|
|
static void drm_core_exit(void)
|
|
|
|
{
|
|
|
|
unregister_chrdev(DRM_MAJOR, "drm");
|
|
|
|
debugfs_remove(drm_debugfs_root);
|
|
|
|
drm_sysfs_destroy();
|
|
|
|
idr_destroy(&drm_minors_idr);
|
|
|
|
drm_connector_ida_destroy();
|
|
|
|
}
|
|
|
|
|
2014-07-23 18:29:56 +08:00
|
|
|
static int __init drm_core_init(void)
|
|
|
|
{
|
drm: cleanup drm_core_{init,exit}()
Various cleanups to the DRM core initialization and exit handlers:
- Register chrdev last: Once register_chrdev() returns, open() will
succeed on the given chrdevs. This is usually not an issue, as no
chardevs are registered, yet. However, nodes can be created by
user-space via mknod(2), even though such major/minor combinations are
unknown to the kernel. Avoid calling into drm_stub_open() in those
cases.
Again, drm_stub_open() would just bail out as the inode is unknown,
but it's really non-obvious if you hack on drm_stub_open().
- Unify error-paths into just one label. All the error-path helpers can
be called even though the constructors were not called yet, or failed.
Hence, just call all cleanups unconditionally.
- Call into drm_global_release(). This is a no-op, but provides
debugging helpers in case there're GLOBALS left on module unload. This
function was unused until now.
- Use DRM_ERROR() instead of printk(), and also print the error-code on
failure (even if it is static!).
- Don't throw away error-codes of register_chrdev()!
- Don't hardcode -1 as errno. This is just plain wrong.
- Order exit-handlers in the exact reverse order of initialization
(except if the order actually matters for syncing-reasons, which is
not the case here, though).
v2:
- Call drm_core_exit() directly from the init-error-handler. Requires to
drop __exit annotation, though.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160901124837.680-7-dh.herrmann@gmail.com
2016-09-01 20:48:37 +08:00
|
|
|
int ret;
|
2014-07-23 18:29:56 +08:00
|
|
|
|
|
|
|
drm_connector_ida_init();
|
|
|
|
idr_init(&drm_minors_idr);
|
|
|
|
|
2015-09-09 20:21:30 +08:00
|
|
|
ret = drm_sysfs_init();
|
|
|
|
if (ret < 0) {
|
drm: cleanup drm_core_{init,exit}()
Various cleanups to the DRM core initialization and exit handlers:
- Register chrdev last: Once register_chrdev() returns, open() will
succeed on the given chrdevs. This is usually not an issue, as no
chardevs are registered, yet. However, nodes can be created by
user-space via mknod(2), even though such major/minor combinations are
unknown to the kernel. Avoid calling into drm_stub_open() in those
cases.
Again, drm_stub_open() would just bail out as the inode is unknown,
but it's really non-obvious if you hack on drm_stub_open().
- Unify error-paths into just one label. All the error-path helpers can
be called even though the constructors were not called yet, or failed.
Hence, just call all cleanups unconditionally.
- Call into drm_global_release(). This is a no-op, but provides
debugging helpers in case there're GLOBALS left on module unload. This
function was unused until now.
- Use DRM_ERROR() instead of printk(), and also print the error-code on
failure (even if it is static!).
- Don't throw away error-codes of register_chrdev()!
- Don't hardcode -1 as errno. This is just plain wrong.
- Order exit-handlers in the exact reverse order of initialization
(except if the order actually matters for syncing-reasons, which is
not the case here, though).
v2:
- Call drm_core_exit() directly from the init-error-handler. Requires to
drop __exit annotation, though.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160901124837.680-7-dh.herrmann@gmail.com
2016-09-01 20:48:37 +08:00
|
|
|
DRM_ERROR("Cannot create DRM class: %d\n", ret);
|
|
|
|
goto error;
|
2014-07-23 18:29:56 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
drm_debugfs_root = debugfs_create_dir("dri", NULL);
|
|
|
|
|
drm: cleanup drm_core_{init,exit}()
Various cleanups to the DRM core initialization and exit handlers:
- Register chrdev last: Once register_chrdev() returns, open() will
succeed on the given chrdevs. This is usually not an issue, as no
chardevs are registered, yet. However, nodes can be created by
user-space via mknod(2), even though such major/minor combinations are
unknown to the kernel. Avoid calling into drm_stub_open() in those
cases.
Again, drm_stub_open() would just bail out as the inode is unknown,
but it's really non-obvious if you hack on drm_stub_open().
- Unify error-paths into just one label. All the error-path helpers can
be called even though the constructors were not called yet, or failed.
Hence, just call all cleanups unconditionally.
- Call into drm_global_release(). This is a no-op, but provides
debugging helpers in case there're GLOBALS left on module unload. This
function was unused until now.
- Use DRM_ERROR() instead of printk(), and also print the error-code on
failure (even if it is static!).
- Don't throw away error-codes of register_chrdev()!
- Don't hardcode -1 as errno. This is just plain wrong.
- Order exit-handlers in the exact reverse order of initialization
(except if the order actually matters for syncing-reasons, which is
not the case here, though).
v2:
- Call drm_core_exit() directly from the init-error-handler. Requires to
drop __exit annotation, though.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160901124837.680-7-dh.herrmann@gmail.com
2016-09-01 20:48:37 +08:00
|
|
|
ret = register_chrdev(DRM_MAJOR, "drm", &drm_stub_fops);
|
|
|
|
if (ret < 0)
|
|
|
|
goto error;
|
|
|
|
|
2017-07-09 05:43:52 +08:00
|
|
|
drm_core_init_complete = true;
|
|
|
|
|
2016-12-29 21:37:29 +08:00
|
|
|
DRM_DEBUG("Initialized\n");
|
2014-07-23 18:29:56 +08:00
|
|
|
return 0;
|
|
|
|
|
drm: cleanup drm_core_{init,exit}()
Various cleanups to the DRM core initialization and exit handlers:
- Register chrdev last: Once register_chrdev() returns, open() will
succeed on the given chrdevs. This is usually not an issue, as no
chardevs are registered, yet. However, nodes can be created by
user-space via mknod(2), even though such major/minor combinations are
unknown to the kernel. Avoid calling into drm_stub_open() in those
cases.
Again, drm_stub_open() would just bail out as the inode is unknown,
but it's really non-obvious if you hack on drm_stub_open().
- Unify error-paths into just one label. All the error-path helpers can
be called even though the constructors were not called yet, or failed.
Hence, just call all cleanups unconditionally.
- Call into drm_global_release(). This is a no-op, but provides
debugging helpers in case there're GLOBALS left on module unload. This
function was unused until now.
- Use DRM_ERROR() instead of printk(), and also print the error-code on
failure (even if it is static!).
- Don't throw away error-codes of register_chrdev()!
- Don't hardcode -1 as errno. This is just plain wrong.
- Order exit-handlers in the exact reverse order of initialization
(except if the order actually matters for syncing-reasons, which is
not the case here, though).
v2:
- Call drm_core_exit() directly from the init-error-handler. Requires to
drop __exit annotation, though.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Link: http://patchwork.freedesktop.org/patch/msgid/20160901124837.680-7-dh.herrmann@gmail.com
2016-09-01 20:48:37 +08:00
|
|
|
error:
|
|
|
|
drm_core_exit();
|
2014-07-23 18:29:56 +08:00
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
module_init(drm_core_init);
|
|
|
|
module_exit(drm_core_exit);
|