From e81847bd06519dd9847c31244abe6da978bcd016 Mon Sep 17 00:00:00 2001 From: Alex Zhuravlev Date: Tue, 29 Aug 2017 15:08:42 +0300 Subject: [PATCH] LU-9660 ptlrpc: do not wakeup every second Even if there are no RPC requests on the set, there is no need to wake up every second. The thread is woken up when a request is added to the set or when the STOP bit is set, so it is sufficient to only wake up when there are requests on the set to worry about. Change-Id: Iac01d8c46e8645ecb6303ce72e0f6c59f16dcd5d Signed-off-by: Alex Zhuravlev Reviewed-on: https://review.whamcloud.com/28776 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Andreas Dilger Reviewed-by: Patrick Farrell Reviewed-by: Oleg Drokin --- lustre/ptlrpc/ptlrpcd.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lustre/ptlrpc/ptlrpcd.c b/lustre/ptlrpc/ptlrpcd.c index 7566459..dc9f948 100644 --- a/lustre/ptlrpc/ptlrpcd.c +++ b/lustre/ptlrpc/ptlrpcd.c @@ -413,7 +413,7 @@ static int ptlrpcd_check(struct lu_env *env, struct ptlrpcd_ctl *pc) } } - RETURN(rc); + RETURN(rc || test_bit(LIOD_STOP, &pc->pc_flags)); } /** @@ -479,8 +479,8 @@ static int ptlrpcd(void *arg) time64_t timeout; timeout = ptlrpc_set_next_timeout(set); - lwi = LWI_TIMEOUT(cfs_time_seconds(timeout ? timeout : 1), - ptlrpc_expired_set, set); + lwi = LWI_TIMEOUT(cfs_time_seconds(timeout), + ptlrpc_expired_set, set); lu_context_enter(&env.le_ctx); lu_context_enter(env.le_ses); -- 1.8.3.1