NFSv4.1: Use more sensible names for 'initialize_mountpoint'
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Thu, 21 Oct 2010 20:56:48 +0000 (16:56 -0400)
committerTrond Myklebust <Trond.Myklebust@netapp.com>
Sun, 24 Oct 2010 22:07:11 +0000 (18:07 -0400)
The initialize_mountpoint/uninitialise_mountpoint functions are really about
setting or clearing the layout driver to be used on this filesystem. Change
the names to the more descriptive 'set_layoutdriver/clear_layoutdriver'.

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
fs/nfs/nfs4filelayout.c
fs/nfs/pnfs.c
fs/nfs/pnfs.h

index 9e82b51cb515ccda778088f921df87b520cd6200..2e92f0d8d654eb4ad6de028ab38f8349f9483f5f 100644 (file)
@@ -40,8 +40,8 @@ MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Dean Hildebrand <dhildebz@umich.edu>");
 MODULE_DESCRIPTION("The NFSv4 file layout driver");
 
-int
-filelayout_initialize_mountpoint(struct nfs_server *nfss)
+static int
+filelayout_set_layoutdriver(struct nfs_server *nfss)
 {
        int status = pnfs_alloc_init_deviceid_cache(nfss->nfs_client,
                                                nfs4_fl_free_deviceid_callback);
@@ -55,9 +55,9 @@ filelayout_initialize_mountpoint(struct nfs_server *nfss)
        return 0;
 }
 
-/* Uninitialize a mountpoint by destroying its device list */
-int
-filelayout_uninitialize_mountpoint(struct nfs_server *nfss)
+/* Clear out the layout by destroying its device list */
+static int
+filelayout_clear_layoutdriver(struct nfs_server *nfss)
 {
        dprintk("--> %s\n", __func__);
 
@@ -256,8 +256,8 @@ static struct pnfs_layoutdriver_type filelayout_type = {
        .id = LAYOUT_NFSV4_1_FILES,
        .name = "LAYOUT_NFSV4_1_FILES",
        .owner = THIS_MODULE,
-       .initialize_mountpoint   = filelayout_initialize_mountpoint,
-       .uninitialize_mountpoint = filelayout_uninitialize_mountpoint,
+       .set_layoutdriver = filelayout_set_layoutdriver,
+       .clear_layoutdriver = filelayout_clear_layoutdriver,
        .alloc_lseg              = filelayout_alloc_lseg,
        .free_lseg               = filelayout_free_lseg,
 };
index d1ad7df3479eed8a0c4a2ed660701e4f09f4308e..db773428f95f28e03a9631d316f2eec711b2aa41 100644 (file)
@@ -75,7 +75,7 @@ void
 unset_pnfs_layoutdriver(struct nfs_server *nfss)
 {
        if (nfss->pnfs_curr_ld) {
-               nfss->pnfs_curr_ld->uninitialize_mountpoint(nfss);
+               nfss->pnfs_curr_ld->clear_layoutdriver(nfss);
                module_put(nfss->pnfs_curr_ld->owner);
        }
        nfss->pnfs_curr_ld = NULL;
@@ -115,7 +115,7 @@ set_pnfs_layoutdriver(struct nfs_server *server, u32 id)
                goto out_no_driver;
        }
        server->pnfs_curr_ld = ld_type;
-       if (ld_type->initialize_mountpoint(server)) {
+       if (ld_type->set_layoutdriver(server)) {
                printk(KERN_ERR
                       "%s: Error initializing mount point for layout driver %u.\n",
                       __func__, id);
index cbba28cb02a71595b9c584236f07b7629ecdd988..e12367d504894ae15e9066d66446487fd69fa81e 100644 (file)
@@ -53,8 +53,8 @@ struct pnfs_layoutdriver_type {
        const u32 id;
        const char *name;
        struct module *owner;
-       int (*initialize_mountpoint) (struct nfs_server *);
-       int (*uninitialize_mountpoint) (struct nfs_server *);
+       int (*set_layoutdriver) (struct nfs_server *);
+       int (*clear_layoutdriver) (struct nfs_server *);
        struct pnfs_layout_segment * (*alloc_lseg) (struct pnfs_layout_hdr *layoutid, struct nfs4_layoutget_res *lgr);
        void (*free_lseg) (struct pnfs_layout_segment *lseg);
 };