From: Vitaly Fertman <vitaly.fertman@seagate.com>
Date: Wed, 21 Jun 2017 02:52:19 +0000 (-0400)
Subject: LU-8062 ptlrpc: increase sleep time in ptlrpc_request_bufs_pack()
X-Git-Tag: 2.10.1-RC1~77
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=39c090bdb9beacc0837cf921d87a451308364131;p=fs%2Flustre-release.git

LU-8062 ptlrpc: increase sleep time in ptlrpc_request_bufs_pack()

schedule_timeout() does not necessarily expire. Increase the sleeping
time in ptlrpc_request_bufs_pack() as 2 seconds is too short, given
the 1 second sleep used for recovery-small test_115_write().

Test-Parameters: envdefinitions=ONLY=115 testlist=recovery-small,recovery-small,recovery-small,recovery-small
Test-Parameters: envdefinitions=ONLY=115 testlist=recovery-small,recovery-small,recovery-small,recovery-small
Test-Parameters: envdefinitions=ONLY=115 testlist=recovery-small,recovery-small,recovery-small,recovery-small
Test-Parameters: envdefinitions=ONLY=115 testlist=recovery-small,recovery-small,recovery-small,recovery-small

Lustre-commit: e9e744ea7352ea0d1a5d9b2bd05e0e7c19f08596
Lustre-change: https://review.whamcloud.com/26815

Change-Id: Ia1b1f096b500f67d26e5fa7c7fbd42642992f327
Signed-off-by: Vitaly Fertman <vitaly.fertman@seagate.com>
Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: James Simmons <uja.ornl@yahoo.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Reviewed-on: https://review.whamcloud.com/28181
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
---

diff --git a/lustre/ptlrpc/client.c b/lustre/ptlrpc/client.c
index e31010c..ca4c2cd 100644
--- a/lustre/ptlrpc/client.c
+++ b/lustre/ptlrpc/client.c
@@ -763,11 +763,11 @@ int ptlrpc_request_bufs_pack(struct ptlrpc_request *request,
 				*fail2_t = ktime_get_real_seconds() +
 					   LONG_UNLINK;
 
-			/* The RPC is infected, let the test to change the
-			 * fail_loc */
-			set_current_state(TASK_UNINTERRUPTIBLE);
-			schedule_timeout(cfs_time_seconds(2));
-			set_current_state(TASK_RUNNING);
+			/*
+			 * The RPC is infected, let the test to change the
+			 * fail_loc
+			 */
+			msleep(4 * MSEC_PER_SEC);
 		}
 	}