[PATCH 1/5] eventpoll: cleanup branches around sleeping for events

From: Jens Axboe
Date: Fri Oct 28 2022 - 17:43:45 EST


Rather than have two separate branches here, collapse them into a single
one instead. No functional changes here, just a cleanup in preparation
for changes in this area.

Signed-off-by: Jens Axboe <axboe@xxxxxxxxx>
---
fs/eventpoll.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/fs/eventpoll.c b/fs/eventpoll.c
index 52954d4637b5..3061bdde6cba 100644
--- a/fs/eventpoll.c
+++ b/fs/eventpoll.c
@@ -1869,14 +1869,15 @@ static int ep_poll(struct eventpoll *ep, struct epoll_event __user *events,
* important.
*/
eavail = ep_events_available(ep);
- if (!eavail)
+ if (!eavail) {
__add_wait_queue_exclusive(&ep->wq, &wait);
-
- write_unlock_irq(&ep->lock);
-
- if (!eavail)
+ write_unlock_irq(&ep->lock);
timed_out = !schedule_hrtimeout_range(to, slack,
HRTIMER_MODE_ABS);
+ } else {
+ write_unlock_irq(&ep->lock);
+ }
+
__set_current_state(TASK_RUNNING);

/*
--
2.35.1