mirror of
https://gitee.com/bianbu-linux/linux-6.6
synced 2025-04-26 14:17:26 -04:00
fs/epoll: avoid barrier after an epoll_wait(2) timeout
Upon timeout, we can just exit out of the loop, without the cost of the changing the task's state with an smp_store_mb call. Just exit out of the loop and be done - setting the task state afterwards will be, of course, redundant. [dave@stgolabs.net: forgotten fixlets] Link: http://lkml.kernel.org/r/20181109155258.jxcr4t2pnz6zqct3@linux-r8p5 Link: http://lkml.kernel.org/r/20181108051006.18751-7-dave@stgolabs.net Signed-off-by: Davidlohr Bueso <dave@stgolabs.net> Reviewed-by: Andrew Morton <akpm@linux-foundation.org> Cc: Al Viro <viro@zeniv.linux.org.uk> Cc: Davidlohr Bueso <dbueso@suse.de> Cc: Jason Baron <jbaron@akamai.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
c5a282e963
commit
abc610e01c
1 changed files with 6 additions and 2 deletions
|
@ -1820,15 +1820,19 @@ fetch_events:
|
||||||
res = -EINTR;
|
res = -EINTR;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
if (ep_events_available(ep) || timed_out)
|
|
||||||
|
eavail = ep_events_available(ep);
|
||||||
|
if (eavail)
|
||||||
break;
|
break;
|
||||||
if (signal_pending(current)) {
|
if (signal_pending(current)) {
|
||||||
res = -EINTR;
|
res = -EINTR;
|
||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!schedule_hrtimeout_range(to, slack, HRTIMER_MODE_ABS))
|
if (!schedule_hrtimeout_range(to, slack, HRTIMER_MODE_ABS)) {
|
||||||
timed_out = 1;
|
timed_out = 1;
|
||||||
|
break;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
__set_current_state(TASK_RUNNING);
|
__set_current_state(TASK_RUNNING);
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue