This patch is a cleanup for all unused-but-set-variable variables in mpathpersist. Signed-off-by: Vijay Chauhan <vijay.chauhan@xxxxxxxxxx> --- diff -uprN multipath-tools.orig/libmpathpersist/mpath_persist.c multipath-tools/libmpathpersist/mpath_persist.c --- multipath-tools.orig/libmpathpersist/mpath_persist.c 2012-03-04 02:16:55.000000000 -0500 +++ multipath-tools/libmpathpersist/mpath_persist.c 2012-03-04 02:52:24.000000000 -0500 @@ -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 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 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 ; } } diff -uprN multipath-tools.orig/libmpathpersist/mpath_pr_ioctl.c multipath-tools/libmpathpersist/mpath_pr_ioctl.c --- multipath-tools.orig/libmpathpersist/mpath_pr_ioctl.c 2012-03-04 02:16:55.000000000 -0500 +++ multipath-tools/libmpathpersist/mpath_pr_ioctl.c 2012-03-04 02:36:43.000000000 -0500 @@ -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 lo if (len <= 0) return; - formatstr = (0 == log) ? "%.76s\n" : "%.56s\n"; memset(buff, ' ', 80); buff[80] = '\0'; for (k = 0; k < len; k++) { diff -uprN multipath-tools.orig/mpathpersist/main.c multipath-tools/mpathpersist/main.c --- multipath-tools.orig/mpathpersist/main.c 2012-03-04 02:16:55.000000000 -0500 +++ multipath-tools/mpathpersist/main.c 2012-03-04 02:56:37.000000000 -0500 @@ -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 ))); -- -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel