[lustre-devel] [PATCH 617/622] lustre: ptlrpc: use l_wait_event_abortable in ptlrpcd_add_reg()

James Simmons jsimmons at infradead.org
Thu Feb 27 13:18:05 PST 2020


From: Mr NeilBrown <neilb at suse.com>

Using wait_event_idle() will ignore signals which is not what we
want in ptlrpcd_add_req(). Change it to l_wait_event_abortable().

WC-bug-id: https://jira.whamcloud.com/browse/LU-10467
Lustre-commit: ca6c35cab141 ("LU-10467 lustre: convert users of back_to_sleep()")
Signed-off-by: Mr NeilBrown <neilb at suse.com>
Reviewed-on: https://review.whamcloud.com/35980
Reviewed-by: James Simmons <jsimmons at infradead.org>
Reviewed-by: Andreas Dilger <adilger at whamcloud.com>
Reviewed-by: Petros Koutoupis <pkoutoupis at cray.com>
Reviewed-by: Oleg Drokin <green at whamcloud.com>
Signed-off-by: James Simmons <jsimmons at infradead.org>
---
 fs/lustre/ptlrpc/ptlrpcd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/fs/lustre/ptlrpc/ptlrpcd.c b/fs/lustre/ptlrpc/ptlrpcd.c
index 1a1fa05..533f592 100644
--- a/fs/lustre/ptlrpc/ptlrpcd.c
+++ b/fs/lustre/ptlrpc/ptlrpcd.c
@@ -235,8 +235,8 @@ void ptlrpcd_add_req(struct ptlrpc_request *req)
 		if (wait_event_idle_timeout(req->rq_set_waitq,
 					    !req->rq_set,
 					    5 * HZ) == 0)
-			wait_event_idle(req->rq_set_waitq,
-					!req->rq_set);
+			l_wait_event_abortable(req->rq_set_waitq,
+					       !req->rq_set);
 	} else if (req->rq_set) {
 		/*
 		 * If we have a valid "rq_set", just reuse it to avoid double
-- 
1.8.3.1



More information about the lustre-devel mailing list