linux-kernel - Re: [PATCH 12/14] 9p: Implement show

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20170711223435.x5tysl756jkrcrrt@yury-thinkpad>
Date:   Wed, 12 Jul 2017 01:34:35 +0300
From:   Yury Norov <ynorov@...iumnetworks.com>
To:     David Howells <dhowells@...hat.com>
Cc:     viro@...iv.linux.org.uk, Latchesar Ionkov <lucho@...kov.net>,
        Eric Van Hensbergen <ericvh@...il.com>,
        linux-kernel@...r.kernel.org, v9fs-developer@...ts.sourceforge.net,
        linux-fsdevel@...r.kernel.org, Ron Minnich <rminnich@...dia.gov>
Subject: Re: [PATCH 12/14] 9p: Implement show_options

On Wed, Jul 05, 2017 at 04:25:37PM +0100, David Howells wrote:
> Implement the show_options superblock op for 9p as part of a bid to get
> rid of s_options and generic_show_options() to make it easier to implement
> a context-based mount where the mount options can be passed individually
> over a file descriptor.
> 
> Signed-off-by: David Howells <dhowells@...hat.com>
> cc: Eric Van Hensbergen <ericvh@...il.com>
> cc: Ron Minnich <rminnich@...dia.gov>
> cc: Latchesar Ionkov <lucho@...kov.net>
> cc: v9fs-developer@...ts.sourceforge.net
> ---

[...]

> +/*
> + * Display the mount options in /proc/mounts.
> + */
> +int v9fs_show_options(struct seq_file *m, struct dentry *root)
> +{
> +	struct v9fs_session_info *v9ses = root->d_sb->s_fs_info;
> +
> +	if (v9ses->debug)
> +		seq_printf(m, ",debug=%x", v9ses->debug);
> +	if (!uid_eq(v9ses->dfltuid, V9FS_DEFUID))
> +		seq_printf(m, ",dfltuid=%u",
> +			   from_kuid_munged(&init_user_ns, v9ses->dfltuid));
> +	if (!gid_eq(v9ses->dfltgid, V9FS_DEFGID))
> +		seq_printf(m, ",dfltgid=%u",
> +			   from_kgid_munged(&init_user_ns, v9ses->dfltgid));
> +	if (v9ses->afid != ~0)
> +		seq_printf(m, ",afid=%u", v9ses->afid);
> +	if (strcmp(v9ses->uname, V9FS_DEFUSER) != 0)
> +		seq_printf(m, ",uname=%s", v9ses->uname);
> +	if (strcmp(v9ses->aname, V9FS_DEFANAME) != 0)
> +		seq_printf(m, ",aname=%s", v9ses->aname);
> +	if (v9ses->nodev)
> +		seq_puts(m, ",nodevmap");
> +	if (v9ses->cache)
> +		seq_printf(m, ",%s", v9fs_cache_modes[v9ses->cache]);
> +	if (v9ses->cachetag && v9ses->cache == CACHE_FSCACHE)
> +		seq_printf(m, ",cachetag=%s", v9ses->cachetag);

Hi David, 

I just found that this patch breaks the next-20170711 build for me:
fs/9p/v9fs.c: In function ‘v9fs_show_options’:
fs/9p/v9fs.c:140:13: error: ‘struct v9fs_session_info’ has no member
named ‘cachetag’; did you mean ‘cache’?
  if (v9ses->cachetag && v9ses->cache == CACHE_FSCACHE)
               ^~~~~~~~
               cache
fs/9p/v9fs.c:141:40: error: ‘struct v9fs_session_info’ has no member named ‘cachetag’; did you mean ‘cache’?
   seq_printf(m, ",cachetag=%s", v9ses->cachetag);
                                        ^~~~~~~~
                                        cache
It seems you forgot to wrap it with #ifdef CONFIG_9P_FSCACHE

Yury