ceph: fix, clean up string mount arg parsing
authorSage Weil <sage@newdream.net>
Tue, 27 Oct 2009 17:19:28 +0000 (10:19 -0700)
committerSage Weil <sage@newdream.net>
Tue, 27 Oct 2009 18:17:25 +0000 (11:17 -0700)
Clearly demark int and string argument options, and do not try to convert
string arguments to ints.

Signed-off-by: Sage Weil <sage@newdream.net>
fs/ceph/super.c

index 924e6cad0b66c70903e617b2fb915c74aa910451..b094f5003ef848f0d672159fbd3a5d24b51f468c 100644 (file)
@@ -264,9 +264,11 @@ enum {
        Opt_caps_wanted_delay_min,
        Opt_caps_wanted_delay_max,
        Opt_readdir_max_entries,
+       Opt_last_int,
        /* int args above */
        Opt_snapdirname,
        Opt_secret,
+       Opt_last_string,
        /* string args above */
        Opt_ip,
        Opt_noshare,
@@ -386,14 +388,19 @@ static int parse_mount_args(struct ceph_client *client,
                        pr_err("bad mount option at '%s'\n", c);
                        goto out;
                }
-               if (token < Opt_ip) {
+               if (token < Opt_last_int) {
                        ret = match_int(&argstr[0], &intval);
                        if (ret < 0) {
                                pr_err("bad mount option arg (not int) "
                                       "at '%s'\n", c);
                                continue;
                        }
-                       dout("got token %d intval %d\n", token, intval);
+                       dout("got int token %d val %d\n", token, intval);
+               } else if (token > Opt_last_int && token < Opt_last_string) {
+                       dout("got string token %d val %s\n", token,
+                            argstr[0].from);
+               } else {
+                       dout("got token %d\n", token);
                }
                switch (token) {
                case Opt_fsidmajor: