Thread (14 messages) 14 messages, 6 authors, 2017-08-07

Re: [PATCH 6/7] fcntl: Don't use ambiguous SIG_POLL si_codes

From: Oleg Nesterov <hidden>
Date: 2017-07-20 16:16:13
Also in: linux-arch, lkml

Possibly related (same subject, not in this thread)

On 07/18, Eric W. Biederman wrote:
-			BUG_ON((reason & __SI_MASK) != __SI_POLL);
+			BUG_ON((reason < POLL_IN) || (reason > NSIGPOLL));
                                                      ^^^^^^^^^^^^^^^^^
looks obviously wrong? Say, POLL_IN is obviously > NSIGPOLL == 6.

Probably you meant

			BUG_ON((reason < POLL_IN) || (reason - POLL_IN > NSIGPOLL)

?

but this contradicts with the next line:
 			if (reason - POLL_IN >= NSIGPOLL)
 				si.si_band  = ~0L;
confused...

Oleg.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help