multipath-tools: cleanup for all unused-but-set-variable variables in mpathpersist
authorChauhan, Vijay <Vijay.Chauhan@netapp.com>
Tue, 6 Mar 2012 15:11:38 +0000 (15:11 +0000)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Wed, 7 Mar 2012 21:31:13 +0000 (22:31 +0100)
This patch is a cleanup for all unused-but-set-variable variables
in mpathpersist.

Signed-off-by: Vijay Chauhan <vijay.chauhan@netapp.com>
libmpathpersist/mpath_persist.c
libmpathpersist/mpath_pr_ioctl.c
mpathpersist/main.c

index e23fe4a..302dd3f 100644 (file)
@@ -394,14 +394,8 @@ void * mpath_prin_pthread_fn (void *p)
 int mpath_send_prin_activepath (char * dev, int rq_servact, struct prin_resp * resp, int noisy)
 {
 
-       struct prin_param param;
        int rc;
 
-       param.rq_servact = rq_servact; 
-       param.resp = resp;
-       param.noisy = noisy;
-       param.status = MPATH_PR_OTHER;
-
        rc = prin_do_scsi_ioctl(dev, rq_servact, resp,  noisy);
        
        return (rc);
@@ -550,7 +544,6 @@ int mpath_prout_common(struct multipath *mpp,int rq_servact, int rq_scope,
        int i,j, ret;
        struct pathgroup *pgp = NULL;
        struct path *pp = NULL;
-       struct path *pptemp = NULL;
 
        vector_foreach_slot (mpp->pg, pgp, j){
                vector_foreach_slot (pgp->paths, pp, i){
@@ -562,7 +555,6 @@ int mpath_prout_common(struct multipath *mpp,int rq_servact, int rq_scope,
                        condlog (3, "%s: sending pr out command to %s", mpp->wwid, pp->dev);
                        ret = send_prout_activepath(pp->dev, rq_servact, rq_scope, rq_type, 
                                        paramp, noisy); 
-                       pptemp = pp;
                        return ret ;
                }
        }
index 018e8de..2d4d968 100644 (file)
@@ -516,7 +516,6 @@ void
 dumpHex(const char* str, int len, int log)
 {
        const char * p = str;
-       const char * formatstr;
        unsigned char c;
        char buff[82];
        const int bpstart = 5;
@@ -525,7 +524,6 @@ dumpHex(const char* str, int len, int log)
 
        if (len <= 0)
                return;
-       formatstr = (0 == log) ? "%.76s\n" : "%.56s\n";
        memset(buff, ' ', 80);
        buff[80] = '\0';
        for (k = 0; k < len; k++) {
index 079080c..33dad90 100644 (file)
@@ -400,9 +400,7 @@ int main (int argc, char * argv[])
        else if (prout)
        {
                int j; 
-               int t_arr_len=0;
                struct prout_param_descriptor *paramp;
-               t_arr_len = MPATH_MX_TID_LEN * num_transport;
 
                paramp= malloc(sizeof(struct prout_param_descriptor) + (sizeof(struct transportid *)*(MPATH_MX_TIDS )));