]> git.decadent.org.uk Git - nfs-utils.git/commitdiff
Added wrappers around the setting of default values
authorSteve Dickson <steved@redhat.com>
Tue, 27 Oct 2009 19:47:27 +0000 (15:47 -0400)
committerSteve Dickson <steved@redhat.com>
Tue, 27 Oct 2009 19:47:27 +0000 (15:47 -0400)
from the config file which will be compiled out
when the config file is not enabled.

Signed-off-by: Steve Dickson <steved@redhat.com>
support/include/conffile.h
utils/mount/network.c
utils/mount/stropts.c

index fe23ec27daf319f57c1d66048ea38a333c8053ad..ce7aa218a56527d9bcc1d77f275bab6a10c4e2fc 100644 (file)
@@ -76,10 +76,5 @@ static inline void upper2lower(char *str)
                *str++ = c;
 }
 
-/*
- * Default Mount options
- */
-extern unsigned long config_default_vers;
-extern unsigned long config_default_proto;
 
 #endif                         /* _CONFFILE_H_ */
index e6511676b0a58cf895c18306464ece1c3792cf54..d4ecbc947c47e99911ba1dd3c39fec64b0a99954 100644 (file)
@@ -172,6 +172,27 @@ static const unsigned long probe_mnt3_first[] = {
        0,
 };
 
+inline const unsigned int *set_default_proto(void);
+#ifdef MOUNT_CONFIG
+inline const unsigned int *set_default_proto()
+{
+       extern unsigned long config_default_proto;
+       /*
+        * If the default proto has been set and 
+        * its not TCP, start with UDP
+        */
+       if (config_default_proto && config_default_proto != IPPROTO_TCP)
+               return probe_udp_first;
+
+       return probe_tcp_first; 
+}
+#else
+inline const unsigned int *set_default_proto() 
+{
+       return probe_tcp_first; 
+}
+#endif /* MOUNT_CONFIG */
+
 static int nfs_lookup(const char *hostname, const sa_family_t family,
                      struct sockaddr *sap, socklen_t *salen)
 {
@@ -590,7 +611,6 @@ out_ok:
        nfs_clear_rpc_createerr();
        return 1;
 }
-
 /*
  * Probe a server's NFS service to determine which versions and
  * transport protocols are supported.
@@ -611,14 +631,9 @@ static int nfs_probe_nfsport(const struct sockaddr *sap, const socklen_t salen,
                return 1;
 
        if (nfs_mount_data_version >= 4) {
-               const unsigned int *probe_proto = probe_tcp_first;
+               const unsigned int *probe_proto;
 
-               /*
-                * If the default proto has been set and 
-                * its not TCP, start with UDP
-                */
-               if (config_default_proto && config_default_proto != IPPROTO_TCP)
-                       probe_proto =  probe_udp_first;
+               probe_proto = set_default_proto();
 
                return nfs_probe_port(sap, salen, pmap,
                                        probe_nfs3_first, probe_proto);
index ceefdb0fc24cecb5a5215db809fbe01fece76251..7b63b4572128909dbe6bf61fd6084caef1a8e2b6 100644 (file)
@@ -93,6 +93,24 @@ struct nfsmount_info {
                                child;          /* forked bg child? */
 };
 
+inline void set_default_version(struct nfsmount_info *mi);
+#ifdef MOUNT_CONFIG
+inline void set_default_version(struct nfsmount_info *mi)
+{
+       extern unsigned long config_default_vers;
+       /*
+        * Use the default value set in the config file when
+        * the version has not been explicitly set.
+        */
+       if (mi->version == 0 && config_default_vers) {
+               if (config_default_vers < 4)
+                       mi->version = config_default_vers;
+       }
+}
+#else
+inline void set_default_version(struct nfsmount_info *mi) {}
+#endif /* MOUNT_CONFIG */
+
 /*
  * Obtain a retry timeout value based on the value of the "retry=" option.
  *
@@ -258,7 +276,6 @@ static int nfs_append_sloppy_option(struct mount_options *options)
                return 0;
        return 1;
 }
-
 /*
  * Set up mandatory non-version specific NFS mount options.
  *
@@ -284,14 +301,12 @@ static int nfs_validate_options(struct nfsmount_info *mi)
                if (option && strcmp(option, "rdma") == 0)
                        mi->version = 3;
        }
+
        /*
-        * Use the default value set in the config file when
-        * the version has not been explicitly set.
+        * If enabled, see if the default version was
+        * set in the config file
         */
-       if (mi->version == 0 && config_default_vers) {
-               if (config_default_vers < 4)
-                       mi->version = config_default_vers;
-       }
+       set_default_version(mi);
 
        if (!nfs_append_sloppy_option(mi->options))
                return 0;