diff options
author | Sebastian Andrzej Siewior <bigeasy@linutronix.de> | 2021-02-16 19:27:14 +0100 |
---|---|---|
committer | Miguel Ojeda <ojeda@kernel.org> | 2021-03-16 16:32:40 +0100 |
commit | 701454bce906241ba7f50e2773881560d6404d29 (patch) | |
tree | c37ca8b84e8ca8f626dfea29d07831af9b18d571 /sound | |
parent | 1e28eed17697bcf343c6743f0028cc3b5dd88bf0 (diff) | |
download | lwn-701454bce906241ba7f50e2773881560d6404d29.tar.gz lwn-701454bce906241ba7f50e2773881560d6404d29.zip |
auxdisplay: Remove in_interrupt() usage.
charlcd_write() is invoked as a VFS->write() callback and as such it is
always invoked from preemptible context and may sleep.
charlcd_puts() is invoked from register/unregister callback which is
preemptible. The reboot notifier callback is also invoked from
preemptible context.
Therefore there is no need to use in_interrupt() to figure out if it
is safe to sleep because it always is. in_interrupt() and related
context checks are being removed from non-core code.
Using schedule() to schedule (and be friendly to others) is
discouraged and cond_resched() should be used instead.
Remove in_interrupt() and use cond_resched() to schedule every 32
iterations if needed.
Link: https://lkml.kernel.org/r/20200914204209.256266093@linutronix.de
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
[mo: fixed a couple typos in comment and commit message]
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
Diffstat (limited to 'sound')
0 files changed, 0 insertions, 0 deletions