Whamcloud - gitweb
LU-6356 ptlrpc: dont take unwrap in req_waittime calculation 04/14404/2
authorSebastien Buisson <sebastien.buisson@bull.net>
Wed, 8 Apr 2015 14:05:34 +0000 (16:05 +0200)
committerOleg Drokin <oleg.drokin@intel.com>
Tue, 28 Apr 2015 05:21:31 +0000 (05:21 +0000)
Do not take unwrap time in req_waittime calculation on client part
as decryption is not related to request service time.

Signed-off-by: Sebastien Buisson <sebastien.buisson@bull.net>
Change-Id: I17986905f4d67323cd7e0644ab25f73dc00f23fb
Reviewed-on: http://review.whamcloud.com/14404
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Dmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: Jeremy Filizetti <jeremy.filizetti@gmail.com>
Reviewed-by: Andrew Perepechko <andrew.perepechko@seagate.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ptlrpc/client.c

index d8773ca..091cbcb 100644 (file)
@@ -1247,6 +1247,9 @@ static int after_reply(struct ptlrpc_request *req)
                 RETURN(0);
         }
 
                 RETURN(0);
         }
 
+       do_gettimeofday(&work_start);
+       timediff = cfs_timeval_sub(&work_start, &req->rq_sent_tv, NULL);
+
         /*
          * NB Until this point, the whole of the incoming message,
          * including buflens, status etc is in the sender's byte order.
         /*
          * NB Until this point, the whole of the incoming message,
          * including buflens, status etc is in the sender's byte order.
@@ -1301,8 +1304,6 @@ static int after_reply(struct ptlrpc_request *req)
                RETURN(0);
        }
 
                RETURN(0);
        }
 
-       do_gettimeofday(&work_start);
-       timediff = cfs_timeval_sub(&work_start, &req->rq_sent_tv, NULL);
        if (obd->obd_svc_stats != NULL) {
                lprocfs_counter_add(obd->obd_svc_stats, PTLRPC_REQWAIT_CNTR,
                                    timediff);
        if (obd->obd_svc_stats != NULL) {
                lprocfs_counter_add(obd->obd_svc_stats, PTLRPC_REQWAIT_CNTR,
                                    timediff);