IB/iser: Get rid of un-maintained counters
authorSagi Grimberg <sagig@mellanox.com>
Thu, 6 Aug 2015 15:32:49 +0000 (18:32 +0300)
committerDoug Ledford <dledford@redhat.com>
Sun, 30 Aug 2015 22:12:27 +0000 (18:12 -0400)
We don't update those anywhere in the code and they
seem pretty useless (no one seem to care about those).

qp_tx_queue_full: We never should get this
fmr_map_not_avail: We can never get to this
eh_abort_cnt: We don't monitor aborts

Go ahead and remove them.

Signed-off-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/ulp/iser/iscsi_iser.c

index e112cbef9bfba373118287998c9230dd93ca1d24..9cc7319887d163493cda14119bfbd9d4d3d8b9a7 100644 (file)
@@ -750,15 +750,9 @@ iscsi_iser_conn_get_stats(struct iscsi_cls_conn *cls_conn, struct iscsi_stats *s
        stats->r2t_pdus = conn->r2t_pdus_cnt; /* always 0 */
        stats->tmfcmd_pdus = conn->tmfcmd_pdus_cnt;
        stats->tmfrsp_pdus = conn->tmfrsp_pdus_cnt;
-       stats->custom_length = 4;
-       strcpy(stats->custom[0].desc, "qp_tx_queue_full");
-       stats->custom[0].value = 0; /* TB iser_conn->qp_tx_queue_full; */
-       strcpy(stats->custom[1].desc, "fmr_map_not_avail");
-       stats->custom[1].value = 0; /* TB iser_conn->fmr_map_not_avail */;
-       strcpy(stats->custom[2].desc, "eh_abort_cnt");
-       stats->custom[2].value = conn->eh_abort_cnt;
-       strcpy(stats->custom[3].desc, "fmr_unalign_cnt");
-       stats->custom[3].value = conn->fmr_unalign_cnt;
+       stats->custom_length = 1;
+       strcpy(stats->custom[0].desc, "fmr_unalign_cnt");
+       stats->custom[0].value = conn->fmr_unalign_cnt;
 }
 
 static int iscsi_iser_get_ep_param(struct iscsi_endpoint *ep,