On Wed, Jul 10, 2019 at 09:05:21PM -0700, Linus Torvalds wrote: > If wait_event_interruptible() returns -ERESTARTSYS, it means that we > have a signal pending. > > And if we have a signal pending, then you can't go back and call > wait_event_interruptible() in a loop, because the signal will > *continue* to be pending, so now your "wait event" becomes a kernel > busy loop. > > If you don't want to react to signals, then you shouldn't use the > "interruptible()" version of wait-event.
Right, a simple wait_event looks obvious; I'll have the submitters test that before sending that next time around. I'll put together another pull with the two trivial commits. Thanks Dave
