LU-880 mds: Fix mds_open counter.

Remove duplicate mds_open counter.

Signed-off-by: Wang Di <di.wang@whamcloud.com>
Change-Id: I229929e9c1635704a668364f72cf17428ed70ef1
Reviewed-on: http://review.whamcloud.com/1747
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Johann Lombardi <johann@whamcloud.com>

diff --git a/lustre/mds/handler.c b/lustre/mds/handler.c
index 5e2be7f..44f8b99 100644 (file)
--- a/lustre/mds/handler.c
+++ b/lustre/mds/handler.c
@@ -2422,7 +2422,6 @@ static int mds_intent_policy(struct ldlm_namespace *ns,
         switch ((long)it->opc) {
         case IT_OPEN:
         case IT_CREAT|IT_OPEN:
-                mds_counter_incr(req->rq_export, LPROC_MDS_OPEN);
                 fixup_handle_for_resent_req(req, DLM_LOCKREQ_OFF, lock, NULL,
                                             &lockh);
                 /* XXX swab here to assert that an mds_open reint

diff --git a/lustre/tests/sanity.sh b/lustre/tests/sanity.sh
index 4b20f67..d9faae2 100644 (file)
--- a/lustre/tests/sanity.sh
+++ b/lustre/tests/sanity.sh
@@ -6133,7 +6133,7 @@ test_133a() {
        check_stats_facet mds "mkdir" 1
        touch ${testdir}/${tfile} || "touch failed"
        # LPROC_MDS_OPEN is incremented by 2 - in mds_open() and mds_intent_policy()
-       check_stats_facet mds "open" 2
+       check_stats_facet mds "open" 1
        check_stats_facet mds "close" 1
        mknod ${testdir}/${tfile}-pipe p || "mknod failed"
        check_stats_facet mds "mknod" 1