staging: lustre: uapi: remove lustre_cfg_free wrapper
authorJames Simmons <jsimmons@infradead.org>
Sun, 20 Aug 2017 02:26:15 +0000 (22:26 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 23 Aug 2017 01:36:48 +0000 (18:36 -0700)
Replace all lustre_cfg_free() calls with direct kfree()
call.

Signed-off-by: James Simmons <uja.ornl@yahoo.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6401
Reviewed-on: https://review.whamcloud.com/26966
Reviewed-by: Quentin Bouget <quentin.bouget@cea.fr>
Reviewed-by: Ben Evans <bevans@cray.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/include/lustre_cfg.h
drivers/staging/lustre/lustre/mgc/mgc_request.c
drivers/staging/lustre/lustre/obdclass/obd_config.c
drivers/staging/lustre/lustre/obdclass/obd_mount.c

index 8eb394e64b25fddda318c8a95e02e9f6944f2d77..3f280b5ae4f02a330cf894c317bdb9c16c7ec50d 100644 (file)
@@ -246,12 +246,6 @@ static inline struct lustre_cfg *lustre_cfg_new(int cmd,
        return lcfg;
 }
 
-static inline void lustre_cfg_free(struct lustre_cfg *lcfg)
-{
-       kfree(lcfg);
-       return;
-}
-
 static inline int lustre_cfg_sanity_check(void *buf, size_t len)
 {
        struct lustre_cfg *lcfg = (struct lustre_cfg *)buf;
index 36c304953b2d45218b8ce189defd8c9c1496f945..192b24d0186ea74e2d335df2019278d25e31f327 100644 (file)
@@ -1293,7 +1293,7 @@ static int mgc_apply_recover_logs(struct obd_device *mgc,
                       prev_version, max_version, obdname, params);
 
                rc = class_process_config(lcfg);
-               lustre_cfg_free(lcfg);
+               kfree(lcfg);
                if (rc)
                        CDEBUG(D_INFO, "process config for %s error %d\n",
                               obdname, rc);
index 783f9e05ed857474f5d56d7ac5d3b1d2c0867eb1..4b5cd7d0be73d4d13764d4bb9beb499b4b98be4d 100644 (file)
@@ -1262,7 +1262,7 @@ int class_config_llog_handler(const struct lu_env *env,
                lcfg_new->lcfg_nal = 0; /* illegal value for obsolete field */
 
                rc = class_process_config(lcfg_new);
-               lustre_cfg_free(lcfg_new);
+               kfree(lcfg_new);
 
                if (inst)
                        kfree(inst_name);
@@ -1442,7 +1442,7 @@ int class_manual_cleanup(struct obd_device *obd)
        if (rc)
                CERROR("detach failed %d: %s\n", rc, obd->obd_name);
 out:
-       lustre_cfg_free(lcfg);
+       kfree(lcfg);
        return rc;
 }
 EXPORT_SYMBOL(class_manual_cleanup);
index 555e67e11a9dc6273cc2a8b42c7ceaa23381dce6..74de1fa4574c8a50e53a75b90fb6c84668ae72b6 100644 (file)
@@ -90,7 +90,7 @@ int lustre_process_log(struct super_block *sb, char *logname,
        lustre_cfg_bufs_set(bufs, 3, &sb, sizeof(sb));
        lcfg = lustre_cfg_new(LCFG_LOG_START, bufs);
        rc = obd_process_config(mgc, sizeof(*lcfg), lcfg);
-       lustre_cfg_free(lcfg);
+       kfree(lcfg);
 
        kfree(bufs);
 
@@ -128,7 +128,7 @@ int lustre_end_log(struct super_block *sb, char *logname,
                lustre_cfg_bufs_set(&bufs, 2, cfg, sizeof(*cfg));
        lcfg = lustre_cfg_new(LCFG_LOG_END, &bufs);
        rc = obd_process_config(mgc, sizeof(*lcfg), lcfg);
-       lustre_cfg_free(lcfg);
+       kfree(lcfg);
        return rc;
 }
 EXPORT_SYMBOL(lustre_end_log);
@@ -161,7 +161,7 @@ static int do_lcfg(char *cfgname, lnet_nid_t nid, int cmd,
        lcfg = lustre_cfg_new(cmd, &bufs);
        lcfg->lcfg_nid = nid;
        rc = class_process_config(lcfg);
-       lustre_cfg_free(lcfg);
+       kfree(lcfg);
        return rc;
 }