From: Jan Kara <jack@suse.cz>
To: Dmitry Monakhov <dmonakhov@openvz.org>
Cc: linux-ext4@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH 1/5] Add additional owner identifier
Date: Fri, 22 Jun 2012 02:03:53 +0200 [thread overview]
Message-ID: <20120622000353.GF11645@quack.suse.cz> (raw)
In-Reply-To: <1340269733-25922-2-git-send-email-dmonakhov@openvz.org>
On Thu 21-06-12 13:08:49, Dmitry Monakhov wrote:
> This patch add project inode identifier. Project ID may be used as
> auxiliary owner specifier in addition to standard uid/gid.
I would just join patches 1 and 2. One without the other looks a bit
dumb... Otherwise I'm OK with the changes. I can take these two patches
through my tree when filesystem maintainers agree they want this feature and
are going to merge corresponding fs patches...
Honza
> Signed-off-by: Dmitry Monakhov <dmonakhov@openvz.org>
> ---
> fs/Kconfig | 7 +++++++
> include/linux/fs.h | 1 +
> include/linux/xattr.h | 3 +++
> 3 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/fs/Kconfig b/fs/Kconfig
> index c8554b5..1932458 100644
> --- a/fs/Kconfig
> +++ b/fs/Kconfig
> @@ -60,6 +60,13 @@ config FILE_LOCKING
> This option enables standard file locking support, required
> for filesystems like NFS and for the flock() system
> call. Disabling this option saves about 11k.
> +config PROJECT_ID
> + bool "Enable project inode identifier"
> + default y
> + help
> + This option enables project inode identifier. Project ID
> + may be used as auxiliary owner specifier in addition to
> + standard uid/gid.
>
> source "fs/notify/Kconfig"
>
> diff --git a/include/linux/fs.h b/include/linux/fs.h
> index b0a6d44..6779fc2 100644
> --- a/include/linux/fs.h
> +++ b/include/linux/fs.h
> @@ -1739,6 +1739,7 @@ struct super_operations {
> int (*bdev_try_to_free_page)(struct super_block*, struct page*, gfp_t);
> int (*nr_cached_objects)(struct super_block *);
> void (*free_cached_objects)(struct super_block *, int);
> + u32 (*get_prjid)(const struct inode *inode);
> };
>
> /*
> diff --git a/include/linux/xattr.h b/include/linux/xattr.h
> index e5d1220..f8a83c3 100644
> --- a/include/linux/xattr.h
> +++ b/include/linux/xattr.h
> @@ -61,6 +61,9 @@
>
> #include <linux/types.h>
>
> +#define XATTR_PRJID "system.project_id"
> +#define XATTR_PRJID_LEN (sizeof (XATTR_PRJID))
> +
> struct inode;
> struct dentry;
>
> --
> 1.7.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Jan Kara <jack@suse.cz>
SUSE Labs, CR
next prev parent reply other threads:[~2012-06-22 0:03 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-06-21 9:08 [PATCH 0/5] RFC: introduce extended inode owner identifier v9 Dmitry Monakhov
2012-06-21 9:08 ` [PATCH 1/5] Add additional owner identifier Dmitry Monakhov
2012-06-22 0:03 ` Jan Kara [this message]
2012-07-03 18:42 ` Dmitry Monakhov
2012-06-21 9:08 ` [PATCH 2/5] Implement project id support for generic quota Dmitry Monakhov
2012-06-22 0:01 ` Jan Kara
2012-06-21 9:08 ` [PATCH 3/5] ext4: Implement project ID support for ext4 filesystem Dmitry Monakhov
2012-06-21 23:51 ` Jan Kara
2012-07-03 18:46 ` Dmitry Monakhov
2012-06-22 3:07 ` Dave Chinner
2012-06-28 10:16 ` Jan Kara
2012-07-03 19:11 ` Dmitry Monakhov
2012-06-21 9:08 ` [PATCH 4/5] ext4: add project quota support Dmitry Monakhov
2012-06-21 23:56 ` Jan Kara
2012-06-21 9:08 ` [PATCH 5/5] XFS: prjquota and grpqouta now may coexist Dmitry Monakhov
2012-06-22 2:36 ` Dave Chinner
2012-07-03 19:14 ` Dmitry Monakhov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120622000353.GF11645@quack.suse.cz \
--to=jack@suse.cz \
--cc=dmonakhov@openvz.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).