From: Christian Brauner <christian.brauner@canonical.com>
To: David Howells <dhowells@redhat.com>
Cc: viro@zeniv.linux.org.uk,
"Eric W. Biederman" <ebiederm@xmission.com>,
linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC PATCH 64/68] vfs: Convert devpts to use the new mount API
Date: Fri, 5 Apr 2019 14:52:25 +0200 [thread overview]
Message-ID: <20190405125224.vvsus7disjwgskeq@gmail.com> (raw)
In-Reply-To: <155373052127.7602.17325161039679063335.stgit@warthog.procyon.org.uk>
On Wed, Mar 27, 2019 at 11:48:41PM +0000, David Howells wrote:
> Convert the devpts filesystem to the new internal mount API as the old
> one will be obsoleted and removed. This allows greater flexibility in
> communication of mount parameters between userspace, the VFS and the
> filesystem.
>
> See Documentation/filesystems/mount_api.txt for more information.
>
> Signed-off-by: David Howells <dhowells@redhat.com>
> cc: "Eric W. Biederman" <ebiederm@xmission.com>
> cc: Christian Brauner <christian.brauner@ubuntu.com>
One nit below otherwise:
Acked-by: Christian Brauner <christian@brauner.io>
> ---
>
> fs/devpts/inode.c | 265 +++++++++++++++++++++++++----------------------------
> 1 file changed, 124 insertions(+), 141 deletions(-)
>
> diff --git a/fs/devpts/inode.c b/fs/devpts/inode.c
> index 553a3f3300ae..ed4a49012570 100644
> --- a/fs/devpts/inode.c
> +++ b/fs/devpts/inode.c
> @@ -15,6 +15,8 @@
> #include <linux/module.h>
> #include <linux/init.h>
> #include <linux/fs.h>
> +#include <linux/fs_context.h>
> +#include <linux/fs_parser.h>
> #include <linux/sched.h>
> #include <linux/namei.h>
> #include <linux/slab.h>
> @@ -24,7 +26,6 @@
> #include <linux/magic.h>
> #include <linux/idr.h>
> #include <linux/devpts_fs.h>
> -#include <linux/parser.h>
> #include <linux/fsnotify.h>
> #include <linux/seq_file.h>
>
> @@ -109,14 +110,19 @@ enum {
> Opt_err
> };
>
> -static const match_table_t tokens = {
> - {Opt_uid, "uid=%u"},
> - {Opt_gid, "gid=%u"},
> - {Opt_mode, "mode=%o"},
> - {Opt_ptmxmode, "ptmxmode=%o"},
> - {Opt_newinstance, "newinstance"},
> - {Opt_max, "max=%d"},
> - {Opt_err, NULL}
> +static const struct fs_parameter_spec devpts_param_specs[] = {
> + fsparam_u32 ("gid", Opt_gid),
> + fsparam_s32 ("max", Opt_max),
> + fsparam_u32oct ("mode", Opt_mode),
> + fsparam_flag ("newinstance", Opt_newinstance),
> + fsparam_u32oct ("ptmxmode", Opt_ptmxmode),
> + fsparam_u32 ("uid", Opt_uid),
> + {}
> +};
> +
> +static const struct fs_parameter_description devpts_fs_parameters = {
> + .name = "devpts",
> + .specs = devpts_param_specs,
> };
>
> struct pts_fs_info {
> @@ -236,93 +242,56 @@ void devpts_release(struct pts_fs_info *fsi)
> deactivate_super(fsi->sb);
> }
>
> -#define PARSE_MOUNT 0
> -#define PARSE_REMOUNT 1
> -
> /*
> - * parse_mount_options():
> - * Set @opts to mount options specified in @data. If an option is not
> - * specified in @data, set it to its default value.
> - *
> - * Note: @data may be NULL (in which case all options are set to default).
> + * devpts_parse_param - Parse mount parameters
> */
> -static int parse_mount_options(char *data, int op, struct pts_mount_opts *opts)
> +static int devpts_parse_param(struct fs_context *fc, struct fs_parameter *param)
> {
> - char *p;
> + struct pts_fs_info *fsi = fc->s_fs_info;
> + struct pts_mount_opts *opts = &fsi->mount_opts;
> + struct fs_parse_result result;
> kuid_t uid;
> kgid_t gid;
> -
> - opts->setuid = 0;
> - opts->setgid = 0;
> - opts->uid = GLOBAL_ROOT_UID;
> - opts->gid = GLOBAL_ROOT_GID;
> - opts->mode = DEVPTS_DEFAULT_MODE;
> - opts->ptmxmode = DEVPTS_DEFAULT_PTMX_MODE;
> - opts->max = NR_UNIX98_PTY_MAX;
> -
> - /* Only allow instances mounted from the initial mount
> - * namespace to tap the reserve pool of ptys.
> - */
> - if (op == PARSE_MOUNT)
> - opts->reserve =
> - (current->nsproxy->mnt_ns == init_task.nsproxy->mnt_ns);
> -
> - while ((p = strsep(&data, ",")) != NULL) {
> - substring_t args[MAX_OPT_ARGS];
> - int token;
> - int option;
> -
> - 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;
> - opts->setuid = 1;
> - 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;
> - opts->setgid = 1;
> - break;
> - case Opt_mode:
> - if (match_octal(&args[0], &option))
> - return -EINVAL;
> - opts->mode = option & S_IALLUGO;
> - break;
> - case Opt_ptmxmode:
> - if (match_octal(&args[0], &option))
> - return -EINVAL;
> - opts->ptmxmode = option & S_IALLUGO;
> - break;
> - case Opt_newinstance:
> - break;
> - case Opt_max:
> - if (match_int(&args[0], &option) ||
> - option < 0 || option > NR_UNIX98_PTY_MAX)
> - return -EINVAL;
> - opts->max = option;
> - break;
> - default:
> - pr_err("called with bogus options\n");
> - return -EINVAL;
> - }
> + int opt;
> +
> + opt = fs_parse(fc, &devpts_fs_parameters, param, &result);
> + if (opt < 0)
> + return opt;
> +
> + switch (opt) {
> + case Opt_uid:
> + uid = make_kuid(current_user_ns(), result.uint_32);
> + if (!uid_valid(uid))
> + return invalf(fc, "Unknown uid");
> + opts->uid = uid;
> + opts->setuid = 1;
> + break;
> + case Opt_gid:
> + gid = make_kgid(current_user_ns(), result.uint_32);
> + if (!gid_valid(gid))
> + return invalf(fc, "Unknown gid");
> + opts->gid = gid;
> + opts->setgid = 1;
> + break;
> + case Opt_mode:
> + opts->mode = result.uint_32 & S_IALLUGO;
> + break;
> + case Opt_ptmxmode:
> + opts->ptmxmode = result.uint_32 & S_IALLUGO;
> + break;
> + case Opt_newinstance:
> + break;
> + case Opt_max:
> + if (result.uint_32 > NR_UNIX98_PTY_MAX)
> + return invalf(fc, "max out of range");
> + opts->max = result.uint_32;
> + break;
> }
>
> return 0;
> }
>
> -static int mknod_ptmx(struct super_block *sb)
> +static int mknod_ptmx(struct super_block *sb, struct fs_context *fc)
> {
> int mode;
> int rc = -ENOMEM;
> @@ -344,7 +313,7 @@ static int mknod_ptmx(struct super_block *sb)
>
> dentry = d_alloc_name(root, "ptmx");
> if (!dentry) {
> - pr_err("Unable to alloc dentry for ptmx node\n");
> + errorf(fc, "Unable to alloc dentry for ptmx node\n");
> goto out;
> }
>
> @@ -353,7 +322,7 @@ static int mknod_ptmx(struct super_block *sb)
> */
> inode = new_inode(sb);
> if (!inode) {
> - pr_err("Unable to alloc inode for ptmx node\n");
> + errorf(fc, "Unable to alloc inode for ptmx node\n");
> dput(dentry);
> goto out;
> }
> @@ -384,13 +353,23 @@ static void update_ptmx_mode(struct pts_fs_info *fsi)
> }
> }
>
> -static int devpts_remount(struct super_block *sb, int *flags, char *data)
> +static int devpts_reconfigure(struct fs_context *fc)
> {
> - int err;
> - struct pts_fs_info *fsi = DEVPTS_SB(sb);
> - struct pts_mount_opts *opts = &fsi->mount_opts;
> + struct pts_fs_info *fsi = DEVPTS_SB(fc->root->d_sb);
> + struct pts_fs_info *new = fc->s_fs_info;
>
> - err = parse_mount_options(data, PARSE_REMOUNT, opts);
> + /* Apply the revised options. We don't want to change ->reserve.
> + * Ideally, we'd update each option conditionally on it having being
s/being/been/ ?
> + * explicitly changed, but the default is to reset everything so that
> + * would break UAPI...
> + */
> + fsi->mount_opts.setuid = new->mount_opts.setuid;
> + fsi->mount_opts.setgid = new->mount_opts.setgid;
> + fsi->mount_opts.uid = new->mount_opts.uid;
> + fsi->mount_opts.gid = new->mount_opts.gid;
> + fsi->mount_opts.mode = new->mount_opts.mode;
> + fsi->mount_opts.ptmxmode = new->mount_opts.ptmxmode;
> + fsi->mount_opts.max = new->mount_opts.max;
>
> /*
> * parse_mount_options() restores options to default values
> @@ -400,7 +379,7 @@ static int devpts_remount(struct super_block *sb, int *flags, char *data)
> */
> update_ptmx_mode(fsi);
>
> - return err;
> + return 0;
> }
>
> static int devpts_show_options(struct seq_file *seq, struct dentry *root)
> @@ -424,31 +403,13 @@ static int devpts_show_options(struct seq_file *seq, struct dentry *root)
>
> static const struct super_operations devpts_sops = {
> .statfs = simple_statfs,
> - .remount_fs = devpts_remount,
> .show_options = devpts_show_options,
> };
>
> -static void *new_pts_fs_info(struct super_block *sb)
> -{
> - struct pts_fs_info *fsi;
> -
> - fsi = kzalloc(sizeof(struct pts_fs_info), GFP_KERNEL);
> - if (!fsi)
> - return NULL;
> -
> - ida_init(&fsi->allocated_ptys);
> - fsi->mount_opts.mode = DEVPTS_DEFAULT_MODE;
> - fsi->mount_opts.ptmxmode = DEVPTS_DEFAULT_PTMX_MODE;
> - fsi->sb = sb;
> -
> - return fsi;
> -}
> -
> -static int
> -devpts_fill_super(struct super_block *s, void *data, int silent)
> +static int devpts_fill_super(struct super_block *s, struct fs_context *fc)
> {
> + struct pts_fs_info *fsi = DEVPTS_SB(s);
> struct inode *inode;
> - int error;
>
> s->s_iflags &= ~SB_I_NODEV;
> s->s_blocksize = 1024;
> @@ -457,20 +418,12 @@ devpts_fill_super(struct super_block *s, void *data, int silent)
> s->s_op = &devpts_sops;
> s->s_d_op = &simple_dentry_operations;
> s->s_time_gran = 1;
> + fsi->sb = s;
>
> - error = -ENOMEM;
> - s->s_fs_info = new_pts_fs_info(s);
> - if (!s->s_fs_info)
> - goto fail;
> -
> - error = parse_mount_options(data, PARSE_MOUNT, &DEVPTS_SB(s)->mount_opts);
> - if (error)
> - goto fail;
> -
> - error = -ENOMEM;
> inode = new_inode(s);
> if (!inode)
> - goto fail;
> + return -ENOMEM;
> +
> inode->i_ino = 1;
> inode->i_mtime = inode->i_atime = inode->i_ctime = current_time(inode);
> inode->i_mode = S_IFDIR | S_IRUGO | S_IXUGO | S_IWUSR;
> @@ -480,20 +433,11 @@ devpts_fill_super(struct super_block *s, void *data, int silent)
>
> s->s_root = d_make_root(inode);
> if (!s->s_root) {
> - pr_err("get root dentry failed\n");
> - goto fail;
> + errorf(fc, "get root dentry failed");
> + return -ENOMEM;
> }
>
> - error = mknod_ptmx(s);
> - if (error)
> - goto fail_dput;
> -
> - return 0;
> -fail_dput:
> - dput(s->s_root);
> - s->s_root = NULL;
> -fail:
> - return error;
> + return mknod_ptmx(s, fc);
> }
>
> /*
> @@ -502,10 +446,48 @@ devpts_fill_super(struct super_block *s, void *data, int silent)
> * Mount a new (private) instance of devpts. PTYs created in this
> * instance are independent of the PTYs in other devpts instances.
> */
> -static struct dentry *devpts_mount(struct file_system_type *fs_type,
> - int flags, const char *dev_name, void *data)
> +static int devpts_get_tree(struct fs_context *fc)
> +{
> + return vfs_get_super(fc, vfs_get_independent_super, devpts_fill_super);
> +}
> +
> +static void devpts_free_fc(struct fs_context *fc)
> +{
> + kfree(fc->s_fs_info);
> +}
> +
> +static const struct fs_context_operations devpts_context_ops = {
> + .free = devpts_free_fc,
> + .parse_param = devpts_parse_param,
> + .get_tree = devpts_get_tree,
> + .reconfigure = devpts_reconfigure,
> +};
> +
> +/*
> + * Set up the filesystem mount context.
> + */
> +static int devpts_init_fs_context(struct fs_context *fc)
> {
> - return mount_nodev(fs_type, flags, data, devpts_fill_super);
> + struct pts_fs_info *fsi;
> +
> + fsi = kzalloc(sizeof(struct pts_fs_info), GFP_KERNEL);
> + if (!fsi)
> + return -ENOMEM;
> +
> + ida_init(&fsi->allocated_ptys);
> + fsi->mount_opts.uid = GLOBAL_ROOT_UID;
> + fsi->mount_opts.gid = GLOBAL_ROOT_GID;
> + fsi->mount_opts.mode = DEVPTS_DEFAULT_MODE;
> + fsi->mount_opts.ptmxmode = DEVPTS_DEFAULT_PTMX_MODE;
> + fsi->mount_opts.max = NR_UNIX98_PTY_MAX;
> +
> + if (fc->purpose == FS_CONTEXT_FOR_MOUNT &&
> + current->nsproxy->mnt_ns == init_task.nsproxy->mnt_ns)
> + fsi->mount_opts.reserve = true;
> +
> + fc->s_fs_info = fsi;
> + fc->ops = &devpts_context_ops;
> + return 0;
> }
>
> static void devpts_kill_sb(struct super_block *sb)
> @@ -520,7 +502,8 @@ static void devpts_kill_sb(struct super_block *sb)
>
> static struct file_system_type devpts_fs_type = {
> .name = "devpts",
> - .mount = devpts_mount,
> + .init_fs_context = devpts_init_fs_context,
> + .parameters = &devpts_fs_parameters,
> .kill_sb = devpts_kill_sb,
> .fs_flags = FS_USERNS_MOUNT,
> };
>
next prev parent reply other threads:[~2019-04-05 12:52 UTC|newest]
Thread overview: 87+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <155372999953.7602.13784796495137723805.stgit@warthog.procyon.org.uk>
2019-03-27 23:40 ` [RFC PATCH 01/68] kbuild: skip sub-make for in-tree build with GNU Make 4.x David Howells
2019-03-28 0:53 ` Masahiro Yamada
2019-03-30 12:11 ` Masahiro Yamada
2019-04-10 9:54 ` David Howells
2019-03-27 23:40 ` [RFC PATCH 02/68] vfs: Update mount API docs David Howells
2019-03-27 23:40 ` [RFC PATCH 03/68] vfs: Fix refcounting of filenames in fs_parser David Howells
2019-03-27 23:41 ` [RFC PATCH 04/68] vfs: Provide sb->s_iflags settings in fs_context struct David Howells
2019-03-27 23:41 ` [RFC PATCH 05/68] vfs: Provide a mount_pseudo-replacement for the new mount API David Howells
2019-03-27 23:41 ` [RFC PATCH 06/68] vfs: Convert aio to use " David Howells
2019-03-27 23:41 ` [RFC PATCH 07/68] vfs: Convert anon_inodes " David Howells
2019-03-27 23:41 ` [RFC PATCH 08/68] vfs: Convert bdev " David Howells
2019-03-27 23:41 ` [RFC PATCH 09/68] vfs: Convert nsfs " David Howells
2019-03-27 23:41 ` [RFC PATCH 10/68] vfs: Convert pipe " David Howells
2019-03-27 23:41 ` [RFC PATCH 11/68] vfs: Convert zsmalloc " David Howells
2019-03-27 23:42 ` [RFC PATCH 12/68] vfs: Convert sockfs " David Howells
2019-03-27 23:42 ` [RFC PATCH 13/68] vfs: Convert dax " David Howells
2019-03-27 23:42 ` [RFC PATCH 14/68] vfs: Convert drm " David Howells
2019-03-28 7:47 ` Daniel Vetter
2019-04-10 9:59 ` David Howells
2019-03-27 23:42 ` [RFC PATCH 15/68] vfs: Convert ia64 perfmon " David Howells
2019-03-27 23:42 ` [RFC PATCH 16/68] vfs: Convert cxl " David Howells
2019-03-27 23:42 ` [RFC PATCH 17/68] vfs: Convert ocxlflash " David Howells
2019-03-27 23:42 ` [RFC PATCH 18/68] vfs: Convert virtio_balloon " David Howells
2019-03-27 23:42 ` [RFC PATCH 19/68] vfs: Convert btrfs_test " David Howells
2019-03-27 23:43 ` [RFC PATCH 20/68] vfs: Kill off mount_pseudo() and mount_pseudo_xattr() David Howells
2019-03-27 23:43 ` [RFC PATCH 21/68] vfs: Use sget_fc() for pseudo-filesystems David Howells
2019-03-27 23:43 ` [RFC PATCH 22/68] vfs: Convert binderfs to use the new mount API David Howells
2019-03-28 14:45 ` Dan Carpenter
2019-04-10 10:01 ` David Howells
2019-03-27 23:43 ` [RFC PATCH 23/68] vfs: Convert nfsctl " David Howells
2019-03-27 23:43 ` [RFC PATCH 24/68] vfs: Convert rpc_pipefs " David Howells
2019-03-27 23:43 ` [RFC PATCH 25/68] vfs: Kill mount_ns() David Howells
2019-03-27 23:43 ` [RFC PATCH 26/68] vfs: Kill sget_userns() David Howells
2019-03-27 23:43 ` [RFC PATCH 27/68] vfs: Convert binfmt_misc to use the new mount API David Howells
2019-03-27 23:44 ` [RFC PATCH 28/68] vfs: Convert configfs " David Howells
2019-03-27 23:44 ` [RFC PATCH 29/68] vfs: Convert efivarfs " David Howells
2019-03-27 23:44 ` [RFC PATCH 30/68] vfs: Convert fusectl " David Howells
2019-04-24 13:49 ` Miklos Szeredi
2019-04-24 15:16 ` David Howells
2019-03-27 23:44 ` [RFC PATCH 31/68] vfs: Convert qib_fs/ipathfs " David Howells
2019-04-09 17:14 ` Dennis Dalessandro
2019-03-27 23:44 ` [RFC PATCH 32/68] vfs: Convert ibmasmfs " David Howells
2019-03-27 23:44 ` [RFC PATCH 33/68] vfs: Convert oprofilefs " David Howells
2019-03-27 23:44 ` [RFC PATCH 34/68] vfs: Convert gadgetfs " David Howells
2019-03-27 23:44 ` [RFC PATCH 35/68] vfs: Convert xenfs " David Howells
2019-04-02 7:38 ` Juergen Gross
2019-03-27 23:45 ` [RFC PATCH 36/68] vfs: Convert openpromfs " David Howells
2019-03-27 23:45 ` [RFC PATCH 37/68] vfs: Convert apparmorfs " David Howells
2019-03-27 23:45 ` [RFC PATCH 38/68] vfs: Convert securityfs " David Howells
2019-03-27 23:45 ` [RFC PATCH 39/68] vfs: Convert selinuxfs " David Howells
2019-03-27 23:45 ` [RFC PATCH 40/68] vfs: Convert smackfs " David Howells
2019-03-27 23:45 ` [RFC PATCH 41/68] vfs: Convert ramfs, shmem, tmpfs, devtmpfs, rootfs " David Howells
2019-03-27 23:45 ` [RFC PATCH 42/68] vfs: Create fs_context-aware mount_bdev() replacement David Howells
2019-03-27 23:45 ` [RFC PATCH 43/68] vfs: Make fs_parse() handle fs_param_is_fd-type params better David Howells
2019-03-27 23:46 ` [RFC PATCH 44/68] vfs: Convert fuse to use the new mount API David Howells
2019-04-24 13:53 ` Miklos Szeredi
2019-04-24 15:22 ` David Howells
2019-04-24 17:28 ` Miklos Szeredi
2019-03-27 23:46 ` [RFC PATCH 45/68] vfs: Move the subtype parameter into fuse David Howells
2019-03-27 23:46 ` [RFC PATCH 46/68] mtd: Provide fs_context-aware mount_mtd() replacement David Howells
2019-03-27 23:46 ` [RFC PATCH 47/68] vfs: Convert romfs to use the new mount API David Howells
2019-03-27 23:46 ` [RFC PATCH 48/68] vfs: Convert cramfs " David Howells
2019-04-01 15:25 ` Nicolas Pitre
2019-03-27 23:46 ` [RFC PATCH 49/68] vfs: Convert jffs2 " David Howells
2019-03-27 23:46 ` [RFC PATCH 50/68] mtd: Kill mount_mtd() David Howells
2019-03-27 23:47 ` [RFC PATCH 51/68] vfs: Convert squashfs to use the new mount API David Howells
2019-03-27 23:47 ` [RFC PATCH 52/68] vfs: Convert ceph " David Howells
2019-04-27 12:27 ` Jeff Layton
2019-03-27 23:47 ` [RFC PATCH 53/68] vfs: Convert functionfs " David Howells
2019-03-28 10:43 ` Michał Nazarewicz
2019-03-27 23:47 ` [RFC PATCH 54/68] vfs: Add a single-or-reconfig keying to vfs_get_super() David Howells
2019-03-27 23:47 ` [RFC PATCH 55/68] vfs: Convert debugfs to use the new mount API David Howells
2019-03-27 23:47 ` [RFC PATCH 56/68] vfs: Convert tracefs " David Howells
2019-03-27 23:47 ` [RFC PATCH 57/68] vfs: Convert pstore " David Howells
2019-03-28 0:24 ` Kees Cook
2019-03-27 23:47 ` [RFC PATCH 58/68] hypfs: Fix error number left in struct pointer member David Howells
2019-03-27 23:48 ` [RFC PATCH 59/68] vfs: Convert hypfs to use the new mount API David Howells
2019-03-27 23:48 ` [RFC PATCH 60/68] vfs: Convert spufs " David Howells
2019-03-27 23:48 ` [RFC PATCH 61/68] vfs: Kill mount_single() David Howells
2019-03-27 23:48 ` [RFC PATCH 62/68] vfs: Convert coda to use the new mount API David Howells
2019-03-27 23:48 ` [RFC PATCH 63/68] vfs: Convert autofs " David Howells
2019-03-27 23:48 ` [RFC PATCH 64/68] vfs: Convert devpts " David Howells
2019-04-05 12:52 ` Christian Brauner [this message]
2019-03-27 23:48 ` [RFC PATCH 65/68] vfs: Convert bpf " David Howells
2019-03-27 23:48 ` [RFC PATCH 66/68] vfs: Convert ubifs " David Howells
2019-03-27 23:49 ` [RFC PATCH 67/68] vfs: Convert orangefs " David Howells
2019-03-27 23:49 ` [RFC PATCH 68/68] gfs2: Convert gfs2 to fs_context David Howells
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=20190405125224.vvsus7disjwgskeq@gmail.com \
--to=christian.brauner@canonical.com \
--cc=dhowells@redhat.com \
--cc=ebiederm@xmission.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=viro@zeniv.linux.org.uk \
/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).