From 39c090bdb9beacc0837cf921d87a451308364131 Mon Sep 17 00:00:00 2001 From: Vitaly Fertman Date: Tue, 20 Jun 2017 22:52:19 -0400 Subject: [PATCH] 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 Signed-off-by: Andreas Dilger Reviewed-by: James Simmons Reviewed-by: Fan Yong Reviewed-on: https://review.whamcloud.com/28181 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: John L. Hammond --- lustre/ptlrpc/client.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) 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); } } -- 1.8.3.1