9p: rename uid and gid parameters

Change the names of 'uid' and 'gid' parameters to the more appropriate
'dfltuid' and 'dfltgid'.  This also sets the default uid/gid to -2
(aka nfsnobody)

Signed-off-by: Latchesar Ionkov <lucho@ionkov.net>
Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>

3 files changed, 16 insertions, 12 deletions

diff --git a/fs/9p/v9fs.c b/fs/9p/v9fs.c
index 8ac246700b70..68f82be3bf37 100644
--- a/fs/9p/v9fs.c
+++ b/fs/9p/v9fs.c
@@ -84,7 +84,7 @@ static struct p9_trans_module *v9fs_match_trans(const substring_t *name)
 
 enum {
         /* Options that take integer arguments */
-        Opt_debug, Opt_msize, Opt_uid, Opt_gid, Opt_afid,
+        Opt_debug, Opt_msize, Opt_dfltuid, Opt_dfltgid, Opt_afid,
         /* String options */
         Opt_uname, Opt_remotename, Opt_trans,
         /* Options that take no arguments */
@@ -98,8 +98,8 @@ enum {
 static match_table_t tokens = {
         {Opt_debug, "debug=%x"},
         {Opt_msize, "msize=%u"},
-        {Opt_uid, "uid=%u"},
-        {Opt_gid, "gid=%u"},
+        {Opt_dfltuid, "dfltuid=%u"},
+        {Opt_dfltgid, "dfltgid=%u"},
         {Opt_afid, "afid=%u"},
         {Opt_uname, "uname=%s"},
         {Opt_remotename, "aname=%s"},
@@ -159,11 +159,11 @@ static void v9fs_parse_options(struct v9fs_session_info *v9ses)
                 case Opt_msize:
                         v9ses->maxdata = option;
                         break;
-                case Opt_uid:
-                        v9ses->uid = option;
+                case Opt_dfltuid:
+                        v9ses->dfltuid = option;
                         break;
-                case Opt_gid:
-                        v9ses->gid = option;
+                case Opt_dfltgid:
+                        v9ses->dfltgid = option;
                         break;
                 case Opt_afid:
                         v9ses->afid = option;
@@ -219,6 +219,8 @@ struct p9_fid *v9fs_session_init(struct v9fs_session_info *v9ses,
 
         strcpy(v9ses->name, V9FS_DEFUSER);
         strcpy(v9ses->remotename, V9FS_DEFANAME);
+        v9ses->dfltuid = V9FS_DEFUID;
+        v9ses->dfltgid = V9FS_DEFGID;
 
         v9ses->options = kstrdup(data, GFP_KERNEL);
         v9fs_parse_options(v9ses);
diff --git a/fs/9p/v9fs.h b/fs/9p/v9fs.h
index 804b3ef8feab..8e0999b3d0cc 100644
--- a/fs/9p/v9fs.h
+++ b/fs/9p/v9fs.h
@@ -38,8 +38,8 @@ struct v9fs_session_info {
         char *options;          /* copy of mount options */
         char *name;             /* user name to mount as */
         char *remotename;       /* name of remote hierarchy being mounted */
-        unsigned int uid;       /* default uid/muid for legacy support */
-        unsigned int gid;       /* default gid for legacy support */
+        unsigned int dfltuid;   /* default uid/muid for legacy support */
+        unsigned int dfltgid;   /* default gid for legacy support */
         struct p9_trans_module *trans; /* 9p transport */
         struct p9_client *clnt; /* 9p client */
         struct dentry *debugfs_dir;
@@ -67,9 +67,11 @@ void v9fs_session_cancel(struct v9fs_session_info *v9ses);
 #define V9FS_MAGIC 0x01021997
 
 /* other default globals */
-#define V9FS_PORT               564
+#define V9FS_PORT       564
 #define V9FS_DEFUSER    "nobody"
 #define V9FS_DEFANAME   ""
+#define V9FS_DEFUID     (-2)
+#define V9FS_DEFGID     (-2)
 
 static inline struct v9fs_session_info *v9fs_inode2v9ses(struct inode *inode)
 {
diff --git a/fs/9p/vfs_inode.c b/fs/9p/vfs_inode.c
index 2270d06ccd56..f08a35d2973a 100644
--- a/fs/9p/vfs_inode.c
+++ b/fs/9p/vfs_inode.c
@@ -805,8 +805,8 @@ v9fs_stat2inode(struct p9_stat *stat, struct inode *inode,
         inode->i_mtime.tv_sec = stat->mtime;
         inode->i_ctime.tv_sec = stat->mtime;
 
-        inode->i_uid = v9ses->uid;
-        inode->i_gid = v9ses->gid;
+        inode->i_uid = v9ses->dfltuid;
+        inode->i_gid = v9ses->dfltgid;
 
         if (v9fs_extended(v9ses)) {
                 inode->i_uid = stat->n_uid;