]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
staging: lustre: obd: change type of lmv_tgt_desc->ltd_idx to u32
authorDmitry Eremin <dmitry.eremin@intel.com>
Sun, 18 Sep 2016 20:37:41 +0000 (16:37 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 19 Sep 2016 07:40:36 +0000 (09:40 +0200)
ltd_idx is used as unsigned.

Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5577
Reviewed-on: http://review.whamcloud.com/11879
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.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/obd.h
drivers/staging/lustre/lustre/lmv/lmv_intent.c
drivers/staging/lustre/lustre/lmv/lmv_obd.c
drivers/staging/lustre/lustre/lmv/lproc_lmv.c

index 19230a97e37b99733c8c05d7e6380b083458f92e..4b72fb9ebf591a81984f8362041ecc784cd18aaf 100644 (file)
@@ -406,7 +406,7 @@ struct lov_obd {
 struct lmv_tgt_desc {
        struct obd_uuid         ltd_uuid;
        struct obd_export       *ltd_exp;
-       int                     ltd_idx;
+       u32                     ltd_idx;
        struct mutex            ltd_fid_mutex;
        unsigned long           ltd_active:1; /* target up for requests */
 };
index f24848f9df065eeb334ab837841a7b3ca7bb224a..02bf3c3cded3e2b7c359979cbb8a9df6d1b66b2a 100644 (file)
@@ -113,7 +113,7 @@ static int lmv_intent_remote(struct obd_export *exp, struct lookup_intent *it,
        }
 
        op_data->op_bias = MDS_CROSS_REF;
-       CDEBUG(D_INODE, "REMOTE_INTENT with fid="DFID" -> mds #%d\n",
+       CDEBUG(D_INODE, "REMOTE_INTENT with fid=" DFID " -> mds #%u\n",
               PFID(&body->mbo_fid1), tgt->ltd_idx);
 
        rc = md_intent_lock(tgt->ltd_exp, op_data, it, &req, cb_blocking,
@@ -202,7 +202,7 @@ int lmv_revalidate_slaves(struct obd_export *exp, struct mdt_body *mbody,
                        goto cleanup;
                }
 
-               CDEBUG(D_INODE, "Revalidate slave "DFID" -> mds #%d\n",
+               CDEBUG(D_INODE, "Revalidate slave " DFID " -> mds #%u\n",
                       PFID(&fid), tgt->ltd_idx);
 
                if (req) {
@@ -349,7 +349,7 @@ static int lmv_intent_open(struct obd_export *exp, struct md_op_data *op_data,
                        return rc;
        }
 
-       CDEBUG(D_INODE, "OPEN_INTENT with fid1=" DFID ", fid2=" DFID ", name='%s' -> mds #%d\n",
+       CDEBUG(D_INODE, "OPEN_INTENT with fid1=" DFID ", fid2=" DFID ", name='%s' -> mds #%u\n",
               PFID(&op_data->op_fid1),
               PFID(&op_data->op_fid2), op_data->op_name, tgt->ltd_idx);
 
@@ -427,7 +427,7 @@ static int lmv_intent_lookup(struct obd_export *exp,
        if (!fid_is_sane(&op_data->op_fid2))
                fid_zero(&op_data->op_fid2);
 
-       CDEBUG(D_INODE, "LOOKUP_INTENT with fid1="DFID", fid2="DFID", name='%s' -> mds #%d lsm=%p lsm_magic=%x\n",
+       CDEBUG(D_INODE, "LOOKUP_INTENT with fid1=" DFID ", fid2=" DFID ", name='%s' -> mds #%u lsm=%p lsm_magic=%x\n",
               PFID(&op_data->op_fid1), PFID(&op_data->op_fid2),
               op_data->op_name ? op_data->op_name : "<NULL>",
               tgt->ltd_idx, lsm, !lsm ? -1 : lsm->lsm_md_magic);
index 9be479efc135094a5c7c0d58e445abea8613466e..e1428cbf9d7a97b45fb43e35c58b9314fb159249 100644 (file)
@@ -1834,7 +1834,7 @@ lmv_enqueue(struct obd_export *exp, struct ldlm_enqueue_info *einfo,
        if (IS_ERR(tgt))
                return PTR_ERR(tgt);
 
-       CDEBUG(D_INODE, "ENQUEUE '%s' on "DFID" -> mds #%d\n",
+       CDEBUG(D_INODE, "ENQUEUE '%s' on " DFID " -> mds #%u\n",
               LL_IT2STR(it), PFID(&op_data->op_fid1), tgt->ltd_idx);
 
        rc = md_enqueue(tgt->ltd_exp, einfo, policy, it, op_data, lockh,
@@ -1862,7 +1862,7 @@ lmv_getattr_name(struct obd_export *exp, struct md_op_data *op_data,
        if (IS_ERR(tgt))
                return PTR_ERR(tgt);
 
-       CDEBUG(D_INODE, "GETATTR_NAME for %*s on "DFID" -> mds #%d\n",
+       CDEBUG(D_INODE, "GETATTR_NAME for %*s on " DFID " -> mds #%u\n",
               op_data->op_namelen, op_data->op_name, PFID(&op_data->op_fid1),
               tgt->ltd_idx);
 
@@ -2607,7 +2607,7 @@ try_next_stripe:
        if (rc != 0)
                return rc;
 
-       CDEBUG(D_INODE, "unlink with fid="DFID"/"DFID" -> mds #%d\n",
+       CDEBUG(D_INODE, "unlink with fid=" DFID "/" DFID " -> mds #%u\n",
               PFID(&op_data->op_fid1), PFID(&op_data->op_fid2), tgt->ltd_idx);
 
        rc = md_unlink(tgt->ltd_exp, op_data, request);
index d2316c0a96514904c0f09ac0afc265b9ec523707..20bbdfc21d15d4ddbb217d8b5d1de4648272ad1c 100644 (file)
@@ -169,7 +169,7 @@ static int lmv_tgt_seq_show(struct seq_file *p, void *v)
 
        if (!tgt)
                return 0;
-       seq_printf(p, "%d: %s %sACTIVE\n",
+       seq_printf(p, "%u: %s %sACTIVE\n",
                   tgt->ltd_idx, tgt->ltd_uuid.uuid,
                   tgt->ltd_active ? "" : "IN");
        return 0;