[Intel-gfx] [RFC 1/3] fs: Introduce drmfs pseudo filesystem interfaces

sourab gupta sourab.gupta at intel.com
Sat Dec 3 00:53:20 UTC 2016


On Fri, 2016-12-02 at 14:40 -0800, Roper, Matthew D wrote:
> On Thu, Dec 01, 2016 at 01:44:15PM +0530, swati.dhingra at intel.com wrote:
> > From: Swati Dhingra <swati.dhingra at intel.com>
> > 
> > The patch introduces a new pseudo filesystem type, named 'drmfs' which is
> > intended to house the files for the data generated by drm subsystem that
> > cannot be accommodated by any of the existing filesystems.
> > The filesystem is modelled on the lines of existing pseudo-filesystems such
> > as debugfs/tracefs, and borrows ideas from their implementation.
> > This filesystem will be appearing at sys/kernel/drm.
> > 
> > A new config 'CONFIG_DRMFS' is introduced to enable/disable the filesystem,
> > which is dependent on CONFIG_DRM.
> > The filesystem will not be registered standalone during kernel init time,
> > instead it is intended to be initialized/registered during drm initialization.
> > 
> > The intent for introduction of the filesystem is to act as a location to hold
> > various kinds of data output from Linux DRM subsystems, which can't really fit
> > anywhere else into the existing filesystems such as debugfs/sysfs etc. All these
> > filesystems have their own constraints and are intended to output a particular
> > type of data such as attributes and small debug parameter data. Due to these
> > constraints, there is a need for a new pseudo filesytem, customizable to DRM
> > specific requirements and catering to the needs to DRM subsystem components
> 
> Am I correct in assuming the data made available via drmfs is not
> considered ABI?  If we're not going to guarantee a stable ABI and
> backward compatibility forever for the items exposed here, then it's
> important to state that very explicitly up front so that no userspace
> software gets written with the wrong assumption.  I'd suggest making an
> explicit note one way or the other in the commit message here and
> probably in the Kconfig help text as well.
> 
> 
> Matt
> 

We've intended for drmfs to be ABI as Chris mentioned here:
https://lists.freedesktop.org/archives/intel-gfx/2016-December/113245.html
The intent is for drmfs to be a stable ABI for the files it's holding.
This can be ensured moresoever since it'll be under sole control of drm.
Chris, can you correct me if i'm wrong.

-sourab
> > 
> > Signed-off-by: Sourab Gupta <sourab.gupta at intel.com>
> > Signed-off-by: Swati Dhingra <swati.dhingra at intel.com>
> > ---
> >  drivers/gpu/drm/drm_drv.c  |   1 +
> >  fs/Kconfig                 |   9 +
> >  fs/Makefile                |   1 +
> >  fs/drmfs/Makefile          |   3 +
> >  fs/drmfs/inode.c           | 561 +++++++++++++++++++++++++++++++++++++++++++++
> >  include/linux/drmfs.h      |  56 +++++
> >  include/uapi/linux/magic.h |   3 +
> >  7 files changed, 634 insertions(+)
> >  create mode 100644 fs/drmfs/Makefile
> >  create mode 100644 fs/drmfs/inode.c
> >  create mode 100644 include/linux/drmfs.h
> > 
> > diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
> > index 6dbb986..84fcfcb 100644
> > --- a/drivers/gpu/drm/drm_drv.c
> > +++ b/drivers/gpu/drm/drm_drv.c
> > @@ -27,6 +27,7 @@
> >   */
> >  
> >  #include <linux/debugfs.h>
> > +#include <linux/drmfs.h>
> >  #include <linux/fs.h>
> >  #include <linux/module.h>
> >  #include <linux/moduleparam.h>
> > diff --git a/fs/Kconfig b/fs/Kconfig
> > index 4bd03a2..7d0ac20 100644
> > --- a/fs/Kconfig
> > +++ b/fs/Kconfig
> > @@ -200,6 +200,15 @@ config HUGETLBFS
> >  config HUGETLB_PAGE
> >  	def_bool HUGETLBFS
> >  
> > +config DRMFS
> > +	bool "Drmfs file system support"
> > +	depends on DRM
> > +	help
> > +	  Drmfs is a pseudo file system for drm subsystem output data.
> > +
> > +	  drmfs is a filesystem to hold miscellaneous output data from drm
> > +	  subsystems.
> > +
> >  config ARCH_HAS_GIGANTIC_PAGE
> >  	bool
> >  
> > diff --git a/fs/Makefile b/fs/Makefile
> > index ed2b632..b34a96e 100644
> > --- a/fs/Makefile
> > +++ b/fs/Makefile
> > @@ -120,6 +120,7 @@ obj-$(CONFIG_BEFS_FS)		+= befs/
> >  obj-$(CONFIG_HOSTFS)		+= hostfs/
> >  obj-$(CONFIG_CACHEFILES)	+= cachefiles/
> >  obj-$(CONFIG_DEBUG_FS)		+= debugfs/
> > +obj-$(CONFIG_DRMFS)		+= drmfs/
> >  obj-$(CONFIG_TRACING)		+= tracefs/
> >  obj-$(CONFIG_OCFS2_FS)		+= ocfs2/
> >  obj-$(CONFIG_BTRFS_FS)		+= btrfs/
> > diff --git a/fs/drmfs/Makefile b/fs/drmfs/Makefile
> > new file mode 100644
> > index 0000000..ac87e497
> > --- /dev/null
> > +++ b/fs/drmfs/Makefile
> > @@ -0,0 +1,3 @@
> > +drmfs-objs	:= inode.o
> > +
> > +obj-$(CONFIG_DRMFS)	+= drmfs.o
> > diff --git a/fs/drmfs/inode.c b/fs/drmfs/inode.c
> > new file mode 100644
> > index 0000000..9220705
> > --- /dev/null
> > +++ b/fs/drmfs/inode.c
> > @@ -0,0 +1,561 @@
> > +/*
> > + * Copyright © 2014 Intel Corporation
> > + *
> > + * 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
> > + * THE AUTHORS OR COPYRIGHT HOLDERS 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.
> > + *
> > + * Authors:
> > + *	Swati Dhingra <swati.dhingra at intel.com>
> > + *	Sourab Gupta <sourab.gupta at intel.com>
> > + *	Akash Goel <akash.goel at intel.com>
> > + */
> > +
> > +/*
> > + * drmfs is the filesystem used for output of drm subsystem data
> > + */
> > +
> > +#include <linux/module.h>
> > +#include <linux/fs.h>
> > +#include <linux/mount.h>
> > +#include <linux/kobject.h>
> > +#include <linux/namei.h>
> > +#include <linux/drmfs.h>
> > +#include <linux/fsnotify.h>
> > +#include <linux/seq_file.h>
> > +#include <linux/parser.h>
> > +#include <linux/magic.h>
> > +#include <linux/slab.h>
> > +
> > +#define DRMFS_DEFAULT_MODE	0700
> > +
> > +static struct vfsmount *drmfs_mount;
> > +static int drmfs_mount_count;
> > +static bool drmfs_registered;
> > +
> > +static ssize_t default_read_file(struct file *file, char __user *buf,
> > +				size_t count, loff_t *ppos)
> > +{
> > +	return 0;
> > +}
> > +
> > +static ssize_t default_write_file(struct file *file, const char __user *buf,
> > +				size_t count, loff_t *ppos)
> > +{
> > +	return count;
> > +}
> > +
> > +static const struct file_operations drmfs_default_file_operations = {
> > +	.read =		default_read_file,
> > +	.write =	default_write_file,
> > +	.open =		simple_open,
> > +	.llseek =	noop_llseek,
> > +};
> > +
> > +static struct inode *drmfs_get_inode(struct super_block *sb)
> > +{
> > +	struct inode *inode = new_inode(sb);
> > +
> > +	if (inode) {
> > +		inode->i_ino = get_next_ino();
> > +		inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME;
> > +	}
> > +	return inode;
> > +}
> > +
> > +struct drmfs_mount_opts {
> > +	kuid_t uid;
> > +	kgid_t gid;
> > +	umode_t mode;
> > +};
> > +
> > +enum {
> > +	Opt_uid,
> > +	Opt_gid,
> > +	Opt_mode,
> > +	Opt_err
> > +};
> > +
> > +static const match_table_t tokens = {
> > +	{Opt_uid, "uid=%u"},
> > +	{Opt_gid, "gid=%u"},
> > +	{Opt_mode, "mode=%o"},
> > +	{Opt_err, NULL}
> > +};
> > +
> > +struct drmfs_fs_info {
> > +	struct drmfs_mount_opts mount_opts;
> > +};
> > +
> > +static int drmfs_parse_options(char *data, struct drmfs_mount_opts *opts)
> > +{
> > +	substring_t args[MAX_OPT_ARGS];
> > +	int option;
> > +	int token;
> > +	kuid_t uid;
> > +	kgid_t gid;
> > +	char *p;
> > +
> > +	opts->mode = DRMFS_DEFAULT_MODE;
> > +
> > +	while ((p = strsep(&data, ",")) != NULL) {
> > +		if (!*p)
> > +			continue;
> > +
> > +		token = match_token(p, tokens, args);
> > +		switch (token) {
> > +		case Opt_uid:
> > +			if (match_int(&args[0], &option))
> > +				return -EINVAL;
> > +			uid = make_kuid(current_user_ns(), option);
> > +			if (!uid_valid(uid))
> > +				return -EINVAL;
> > +			opts->uid = uid;
> > +			break;
> > +		case Opt_gid:
> > +			if (match_int(&args[0], &option))
> > +				return -EINVAL;
> > +			gid = make_kgid(current_user_ns(), option);
> > +			if (!gid_valid(gid))
> > +				return -EINVAL;
> > +			opts->gid = gid;
> > +			break;
> > +		case Opt_mode:
> > +			if (match_octal(&args[0], &option))
> > +				return -EINVAL;
> > +			opts->mode = option & S_IALLUGO;
> > +			break;
> > +		/*
> > +		 * We might like to report bad mount options here
> > +		 */
> > +		}
> > +	}
> > +
> > +	return 0;
> > +}
> > +
> > +static int drmfs_apply_options(struct super_block *sb)
> > +{
> > +	struct drmfs_fs_info *fsi = sb->s_fs_info;
> > +	struct inode *inode = sb->s_root->d_inode;
> > +	struct drmfs_mount_opts *opts = &fsi->mount_opts;
> > +
> > +	inode->i_mode &= ~S_IALLUGO;
> > +	inode->i_mode |= opts->mode;
> > +
> > +	inode->i_uid = opts->uid;
> > +	inode->i_gid = opts->gid;
> > +
> > +	return 0;
> > +}
> > +
> > +static int drmfs_remount(struct super_block *sb, int *flags, char *data)
> > +{
> > +	int err;
> > +	struct drmfs_fs_info *fsi = sb->s_fs_info;
> > +
> > +	sync_filesystem(sb);
> > +	err = drmfs_parse_options(data, &fsi->mount_opts);
> > +	if (err)
> > +		goto fail;
> > +
> > +	drmfs_apply_options(sb);
> > +
> > +fail:
> > +	return err;
> > +}
> > +
> > +static int drmfs_show_options(struct seq_file *m, struct dentry *root)
> > +{
> > +	struct drmfs_fs_info *fsi = root->d_sb->s_fs_info;
> > +	struct drmfs_mount_opts *opts = &fsi->mount_opts;
> > +
> > +	if (!uid_eq(opts->uid, GLOBAL_ROOT_UID))
> > +		seq_printf(m, ",uid=%u",
> > +			   from_kuid_munged(&init_user_ns, opts->uid));
> > +	if (!gid_eq(opts->gid, GLOBAL_ROOT_GID))
> > +		seq_printf(m, ",gid=%u",
> > +			   from_kgid_munged(&init_user_ns, opts->gid));
> > +	if (opts->mode != DRMFS_DEFAULT_MODE)
> > +		seq_printf(m, ",mode=%o", opts->mode);
> > +
> > +	return 0;
> > +}
> > +
> > +static const struct super_operations drmfs_super_operations = {
> > +	.statfs		= simple_statfs,
> > +	.remount_fs	= drmfs_remount,
> > +	.show_options	= drmfs_show_options,
> > +};
> > +
> > +static int drm_fill_super(struct super_block *sb, void *data, int silent)
> > +{
> > +	static struct tree_descr drm_files[] = { {""} };
> > +	struct drmfs_fs_info *fsi;
> > +	int err;
> > +
> > +	save_mount_options(sb, data);
> > +
> > +	fsi = kzalloc(sizeof(struct drmfs_fs_info), GFP_KERNEL);
> > +	sb->s_fs_info = fsi;
> > +	if (!fsi) {
> > +		err = -ENOMEM;
> > +		goto fail;
> > +	}
> > +
> > +	err = drmfs_parse_options(data, &fsi->mount_opts);
> > +	if (err)
> > +		goto fail;
> > +
> > +	err  =  simple_fill_super(sb, DRMFS_MAGIC, drm_files);
> > +	if (err)
> > +		goto fail;
> > +
> > +	sb->s_op = &drmfs_super_operations;
> > +
> > +	drmfs_apply_options(sb);
> > +
> > +	return 0;
> > +
> > +fail:
> > +	kfree(fsi);
> > +	sb->s_fs_info = NULL;
> > +	return err;
> > +}
> > +
> > +static struct dentry *drm_mount(struct file_system_type *fs_type,
> > +			int flags, const char *dev_name,
> > +			void *data)
> > +{
> > +	return mount_single(fs_type, flags, data, drm_fill_super);
> > +}
> > +
> > +static struct file_system_type drm_fs_type = {
> > +	.owner =	THIS_MODULE,
> > +	.name =		"drmfs",
> > +	.mount =	drm_mount,
> > +	.kill_sb =	kill_litter_super,
> > +};
> > +MODULE_ALIAS_FS("drmfs");
> > +
> > +static struct dentry *start_creating(const char *name, struct dentry *parent)
> > +{
> > +	struct dentry *dentry;
> > +	int error;
> > +
> > +	pr_debug("drmfs: creating file '%s'\n", name);
> > +
> > +	error = simple_pin_fs(&drm_fs_type, &drmfs_mount,
> > +			      &drmfs_mount_count);
> > +	if (error)
> > +		return ERR_PTR(error);
> > +
> > +	/* If the parent is not specified, we create it in the root.
> > +	 * We need the root dentry to do this, which is in the super
> > +	 * block. A pointer to that is in the struct vfsmount that we
> > +	 * have around.
> > +	 */
> > +	if (!parent)
> > +		parent = drmfs_mount->mnt_root;
> > +
> > +	inode_lock(parent->d_inode);
> > +	dentry = lookup_one_len(name, parent, strlen(name));
> > +	if (!IS_ERR(dentry) && dentry->d_inode) {
> > +		dput(dentry);
> > +		dentry = ERR_PTR(-EEXIST);
> > +	}
> > +
> > +	if (IS_ERR(dentry)) {
> > +		inode_unlock(parent->d_inode);
> > +		simple_release_fs(&drmfs_mount, &drmfs_mount_count);
> > +	}
> > +
> > +	return dentry;
> > +}
> > +
> > +static struct dentry *failed_creating(struct dentry *dentry)
> > +{
> > +	inode_unlock(dentry->d_parent->d_inode);
> > +	dput(dentry);
> > +	simple_release_fs(&drmfs_mount, &drmfs_mount_count);
> > +	return NULL;
> > +}
> > +
> > +static struct dentry *end_creating(struct dentry *dentry)
> > +{
> > +	inode_unlock(dentry->d_parent->d_inode);
> > +	return dentry;
> > +}
> > +
> > +/**
> > + * drmfs_create_file - create a file in the drmfs filesystem
> > + * @name: a pointer to a string containing the name of the file to create.
> > + * @mode: the permission that the file should have.
> > + * @parent: a pointer to the parent dentry for this file.  This should be a
> > + *          directory dentry if set.  If this parameter is NULL, then the
> > + *          file will be created in the root of the drmfs filesystem.
> > + * @data: a pointer to something that the caller will want to get to later
> > + *        on.  The inode.i_private pointer will point to this value on
> > + *        the open() call.
> > + * @fops: a pointer to a struct file_operations that should be used for
> > + *        this file.
> > + *
> > + * This is the basic "create a file" function for drmfs.  It allows for a
> > + * wide range of flexibility in creating a file, or a directory (if you want
> > + * to create a directory, the drmfs_create_dir() function is
> > + * recommended to be used instead.)
> > + *
> > + * This function will return a pointer to a dentry if it succeeds.  This
> > + * pointer must be passed to the drmfs_remove() function when the file is
> > + * to be removed (no automatic cleanup happens if your module is unloaded,
> > + * you are responsible here.)  If an error occurs, %NULL will be returned.
> > + *
> > + */
> > +struct dentry *drmfs_create_file(const char *name, umode_t mode,
> > +				   struct dentry *parent, void *data,
> > +				   const struct file_operations *fops)
> > +{
> > +	struct dentry *dentry;
> > +	struct inode *inode;
> > +
> > +	if (!(mode & S_IFMT))
> > +		mode |= S_IFREG;
> > +
> > +	if (WARN_ON(!S_ISREG(mode)))
> > +		return NULL;
> > +
> > +	dentry = start_creating(name, parent);
> > +
> > +	if (IS_ERR(dentry))
> > +		return NULL;
> > +
> > +	inode = drmfs_get_inode(dentry->d_sb);
> > +	if (unlikely(!inode))
> > +		return failed_creating(dentry);
> > +
> > +	inode->i_mode = mode;
> > +	inode->i_fop = fops ? fops : &drmfs_default_file_operations;
> > +	inode->i_private = data;
> > +	d_instantiate(dentry, inode);
> > +	fsnotify_create(dentry->d_parent->d_inode, dentry);
> > +	return end_creating(dentry);
> > +}
> > +EXPORT_SYMBOL(drmfs_create_file);
> > +
> > +static struct dentry *__create_dir(const char *name, struct dentry *parent,
> > +				   const struct inode_operations *ops)
> > +{
> > +	struct dentry *dentry = start_creating(name, parent);
> > +	struct inode *inode;
> > +
> > +	if (IS_ERR(dentry))
> > +		return NULL;
> > +
> > +	inode = drmfs_get_inode(dentry->d_sb);
> > +	if (unlikely(!inode))
> > +		return failed_creating(dentry);
> > +
> > +	inode->i_mode = S_IFDIR | S_IRWXU | S_IRUGO | S_IXUGO;
> > +	inode->i_op = ops;
> > +	inode->i_fop = &simple_dir_operations;
> > +
> > +	/* directory inodes start off with i_nlink == 2 (for "." entry) */
> > +	inc_nlink(inode);
> > +	d_instantiate(dentry, inode);
> > +	inc_nlink(dentry->d_parent->d_inode);
> > +	fsnotify_mkdir(dentry->d_parent->d_inode, dentry);
> > +	return end_creating(dentry);
> > +}
> > +
> > +/**
> > + * drmfs_create_dir - create a directory in the drmfs filesystem
> > + * @name: a pointer to a string containing the name of the directory to
> > + *        create.
> > + * @parent: a pointer to the parent dentry for this file.  This should be a
> > + *          directory dentry if set.  If this parameter is NULL, then the
> > + *          directory will be created in the root of the drmfs filesystem.
> > + *
> > + * This function creates a directory in drmfs with the given name.
> > + *
> > + * This function will return a pointer to a dentry if it succeeds.  This
> > + * pointer must be passed to the drmfs_remove() function when the file is
> > + * to be removed. If an error occurs, %NULL will be returned.
> > + *
> > + */
> > +struct dentry *drmfs_create_dir(const char *name, struct dentry *parent)
> > +{
> > +	return __create_dir(name, parent, &simple_dir_inode_operations);
> > +}
> > +EXPORT_SYMBOL(drmfs_create_dir);
> > +
> > +static int __drmfs_remove(struct dentry *dentry, struct dentry *parent)
> > +{
> > +	int ret = 0;
> > +
> > +	if (simple_positive(dentry)) {
> > +		if (dentry->d_inode) {
> > +			dget(dentry);
> > +			switch (dentry->d_inode->i_mode & S_IFMT) {
> > +			case S_IFDIR:
> > +				ret = simple_rmdir(parent->d_inode, dentry);
> > +				break;
> > +			default:
> > +				simple_unlink(parent->d_inode, dentry);
> > +				break;
> > +			}
> > +			if (!ret)
> > +				d_delete(dentry);
> > +			dput(dentry);
> > +		}
> > +	}
> > +	return ret;
> > +}
> > +
> > +
> > +/**
> > + * drmfs_remove - removes a file or directory from the drmfs filesystem
> > + * @dentry: a pointer to a the dentry of the file or directory to be
> > + *          removed.
> > + *
> > + * This function removes a file or directory in drmfs that was previously
> > + * created with a call to another drmfs function (like
> > + * drmfs_create_file() or variants thereof.)
> > + */
> > +void drmfs_remove(struct dentry *dentry)
> > +{
> > +	struct dentry *parent;
> > +	int ret;
> > +
> > +	if (IS_ERR_OR_NULL(dentry))
> > +		return;
> > +
> > +	parent = dentry->d_parent;
> > +	inode_lock(parent->d_inode);
> > +	ret = __drmfs_remove(dentry, parent);
> > +	inode_unlock(parent->d_inode);
> > +	if (!ret)
> > +		simple_release_fs(&drmfs_mount, &drmfs_mount_count);
> > +}
> > +EXPORT_SYMBOL(drmfs_remove);
> > +
> > +/**
> > + * drmfs_remove_recursive - recursively removes a directory
> > + * @dentry: a pointer to a the dentry of the directory to be removed.
> > + *
> > + * This function recursively removes a directory tree in drmfs that
> > + * was previously created with a call to another drmfs function
> > + * (like drmfs_create_file() or variants thereof.)
> > + */
> > +void drmfs_remove_recursive(struct dentry *dentry)
> > +{
> > +	struct dentry *child, *parent;
> > +
> > +	if (IS_ERR_OR_NULL(dentry))
> > +		return;
> > +
> > +	parent = dentry;
> > + down:
> > +	inode_lock(parent->d_inode);
> > + loop:
> > +	/*
> > +	 * The parent->d_subdirs is protected by the d_lock. Outside that
> > +	 * lock, the child can be unlinked and set to be freed which can
> > +	 * use the d_u.d_child as the rcu head and corrupt this list.
> > +	 */
> > +	spin_lock(&parent->d_lock);
> > +	list_for_each_entry(child, &parent->d_subdirs, d_child) {
> > +		if (!simple_positive(child))
> > +			continue;
> > +
> > +		/* perhaps simple_empty(child) makes more sense */
> > +		if (!list_empty(&child->d_subdirs)) {
> > +			spin_unlock(&parent->d_lock);
> > +			inode_unlock(parent->d_inode);
> > +			parent = child;
> > +			goto down;
> > +		}
> > +
> > +		spin_unlock(&parent->d_lock);
> > +
> > +		if (!__drmfs_remove(child, parent))
> > +			simple_release_fs(&drmfs_mount, &drmfs_mount_count);
> > +
> > +		/*
> > +		 * The parent->d_lock protects against child from unlinking
> > +		 * from d_subdirs. When releasing the parent->d_lock we can
> > +		 * no longer trust that the next pointer is valid.
> > +		 * Restart the loop. We'll skip this one with the
> > +		 * simple_positive() check.
> > +		 */
> > +		goto loop;
> > +	}
> > +	spin_unlock(&parent->d_lock);
> > +
> > +	inode_unlock(parent->d_inode);
> > +	child = parent;
> > +	parent = parent->d_parent;
> > +	inode_lock(parent->d_inode);
> > +
> > +	if (child != dentry)
> > +		/* go up */
> > +		goto loop;
> > +
> > +	if (!__drmfs_remove(child, parent))
> > +		simple_release_fs(&drmfs_mount, &drmfs_mount_count);
> > +	inode_unlock(parent->d_inode);
> > +}
> > +EXPORT_SYMBOL(drmfs_remove_recursive);
> > +
> > +/**
> > + * drmfs_initialized - Tells whether drmfs has been registered
> > + */
> > +bool drmfs_initialized(void)
> > +{
> > +	return drmfs_registered;
> > +}
> > +EXPORT_SYMBOL(drmfs_initialized);
> > +
> > +int drmfs_init(void)
> > +{
> > +	int retval;
> > +
> > +	retval = sysfs_create_mount_point(kernel_kobj, "drm");
> > +	if (retval)
> > +		return -EINVAL;
> > +
> > +	retval = register_filesystem(&drm_fs_type);
> > +	if (!retval)
> > +		drmfs_registered = true;
> > +
> > +	return retval;
> > +}
> > +EXPORT_SYMBOL(drmfs_init);
> > +
> > +int drmfs_fini(void)
> > +{
> > +	int retval;
> > +
> > +	retval = unregister_filesystem(&drm_fs_type);
> > +	if (retval)
> > +		return retval;
> > +
> > +	drmfs_registered = false;
> > +
> > +	sysfs_remove_mount_point(kernel_kobj, "drm");
> > +}
> > +EXPORT_SYMBOL(drmfs_fini);
> > diff --git a/include/linux/drmfs.h b/include/linux/drmfs.h
> > new file mode 100644
> > index 0000000..3091480
> > --- /dev/null
> > +++ b/include/linux/drmfs.h
> > @@ -0,0 +1,56 @@
> > +/*
> > + * Copyright © 2014 Intel Corporation
> > + *
> > + * 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
> > + * THE AUTHORS OR COPYRIGHT HOLDERS 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.
> > + *
> > + * Authors:
> > + *	Swati Dhingra <swati.dhingra at intel.com>
> > + *	Sourab Gupta <sourab.gupta at intel.com>
> > + *	Akash Goel <akash.goel at intel.com>
> > + */
> > +
> > +#ifndef _DRMFS_H_
> > +#define _DRMFS_H_
> > +
> > +#include <linux/fs.h>
> > +#include <linux/seq_file.h>
> > +
> > +#include <linux/types.h>
> > +
> > +struct file_operations;
> > +
> > +#ifdef CONFIG_DRMFS
> > +
> > +struct dentry *drmfs_create_file(const char *name, umode_t mode,
> > +				   struct dentry *parent, void *data,
> > +				   const struct file_operations *fops);
> > +
> > +struct dentry *drmfs_create_dir(const char *name, struct dentry *parent);
> > +
> > +void drmfs_remove(struct dentry *dentry);
> > +void drmfs_remove_recursive(struct dentry *dentry);
> > +
> > +bool drmfs_initialized(void);
> > +int drmfs_init(void);
> > +int drmfs_fini(void);
> > +
> > +#endif /* CONFIG_DRMFS */
> > +
> > +#endif
> > diff --git a/include/uapi/linux/magic.h b/include/uapi/linux/magic.h
> > index 9bd5594..ef01eb7 100644
> > --- a/include/uapi/linux/magic.h
> > +++ b/include/uapi/linux/magic.h
> > @@ -61,6 +61,9 @@
> >  #define STACK_END_MAGIC		0x57AC6E9D
> >  
> >  #define TRACEFS_MAGIC          0x74726163
> > +#define DRMFS_MAGIC	       0x84724143      /* some random number.
> > +						* Is there a better way?
> > +						*/
> >  
> >  #define V9FS_MAGIC		0x01021997
> >  
> > -- 
> > 2.7.4
> > 
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> 




More information about the Intel-gfx mailing list