unify {de,}mangle_poll(), get rid of kernel-side POLL...

except, again, POLLFREE and POLL_BUSY_LOOP.

With this, we finally get to the promised end result:

 - POLL{IN,OUT,...} are plain integers and *not* in __poll_t, so any
   stray instances of ->poll() still using those will be caught by
   sparse.

 - eventpoll.c and select.c warning-free wrt __poll_t

 - no more kernel-side definitions of POLL... - userland ones are
   visible through the entire kernel (and used pretty much only for
   mangle/demangle)

 - same behavior as after the first series (i.e. sparc et.al. epoll(2)
   working correctly).

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

diff --git a/arch/blackfin/include/uapi/asm/poll.h b/arch/blackfin/include/uapi/asm/poll.h
index 3b162f2d29703b326ad2583ab04a338a5575a741..cd2f1a78aba5722c7d716ac404694c529957cfc6 100644 (file)
--- a/arch/blackfin/include/uapi/asm/poll.h
+++ b/arch/blackfin/include/uapi/asm/poll.h
@@ -9,25 +9,8 @@
 #ifndef _UAPI__BFIN_POLL_H
 #define _UAPI__BFIN_POLL_H
 
-#ifndef __KERNEL__
 #define POLLWRNORM     POLLOUT
-#define POLLWRBAND     (__force __poll_t)256
-#else
-#define __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       __u16 v = (__force __u16)val;
-       /* bit 9 -> bit 8, bit 8 -> bit 2 */
-       return (v & ~0x300) | ((v & 0x200) >> 1) | ((v & 0x100) >> 6);
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-        /* bit 8 -> bit 9, bit 2 -> bits 2 and 8 */
-       return (__force __poll_t)((v & ~0x100) | ((v & 0x100) << 1) |
-                               ((v & 4) << 6));
-}
-#endif
+#define POLLWRBAND     256
 
 #include <asm-generic/poll.h>
 

diff --git a/arch/frv/include/uapi/asm/poll.h b/arch/frv/include/uapi/asm/poll.h
index a44c8f0ebee7961a1d4d47794e822e1f52d5f3eb..f55b45f475eca8f779443a003e8b4231776d0240 100644 (file)
--- a/arch/frv/include/uapi/asm/poll.h
+++ b/arch/frv/include/uapi/asm/poll.h
@@ -2,25 +2,8 @@
 #ifndef _ASM_POLL_H
 #define _ASM_POLL_H
 
-#ifndef __KERNEL__
 #define POLLWRNORM     POLLOUT
-#define POLLWRBAND     (__force __poll_t)256
-#else
-#define __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       __u16 v = (__force __u16)val;
-       /* bit 9 -> bit 8, bit 8 -> bit 2 */
-       return (v & ~0x300) | ((v & 0x200) >> 1) | ((v & 0x100) >> 6);
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-        /* bit 8 -> bit 9, bit 2 -> bits 2 and 8 */
-       return (__force __poll_t)((v & ~0x100) | ((v & 0x100) << 1) |
-                               ((v & 4) << 6));
-}
-#endif
+#define POLLWRBAND     256
 
 #include <asm-generic/poll.h>
 #undef POLLREMOVE

diff --git a/arch/m68k/include/uapi/asm/poll.h b/arch/m68k/include/uapi/asm/poll.h
index d8be239e81417b4826d1549401b10be1891381a3..c3e3fcc15e1dc0874540ab58085ce5430df442a8 100644 (file)
--- a/arch/m68k/include/uapi/asm/poll.h
+++ b/arch/m68k/include/uapi/asm/poll.h
@@ -2,25 +2,8 @@
 #ifndef __m68k_POLL_H
 #define __m68k_POLL_H
 
-#ifndef __KERNEL__
 #define POLLWRNORM     POLLOUT
-#define POLLWRBAND     (__force __poll_t)256
-#else
-#define __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       __u16 v = (__force __u16)val;
-       /* bit 9 -> bit 8, bit 8 -> bit 2 */
-       return (v & ~0x300) | ((v & 0x200) >> 1) | ((v & 0x100) >> 6);
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-        /* bit 8 -> bit 9, bit 2 -> bits 2 and 8 */
-       return (__force __poll_t)((v & ~0x100) | ((v & 0x100) << 1) |
-                               ((v & 4) << 6));
-}
-#endif
+#define POLLWRBAND     256
 
 #include <asm-generic/poll.h>
 

diff --git a/arch/mips/include/uapi/asm/poll.h b/arch/mips/include/uapi/asm/poll.h
index 3173f89171284829631ed179faa18240c884d9a5..ad289d7b74340f174d22fd6703138b6354e9cdda 100644 (file)
--- a/arch/mips/include/uapi/asm/poll.h
+++ b/arch/mips/include/uapi/asm/poll.h
@@ -2,25 +2,8 @@
 #ifndef __ASM_POLL_H
 #define __ASM_POLL_H
 
-#ifndef __KERNEL__
 #define POLLWRNORM     POLLOUT
-#define POLLWRBAND     (__force __poll_t)0x0100
-#else
-#define __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       __u16 v = (__force __u16)val;
-       /* bit 9 -> bit 8, bit 8 -> bit 2 */
-       return (v & ~0x300) | ((v & 0x200) >> 1) | ((v & 0x100) >> 6);
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-        /* bit 8 -> bit 9, bit 2 -> bits 2 and 8 */
-       return (__force __poll_t)((v & ~0x100) | ((v & 0x100) << 1) |
-                               ((v & 4) << 6));
-}
-#endif
+#define POLLWRBAND     0x0100
 
 #include <asm-generic/poll.h>
 

diff --git a/arch/sparc/include/uapi/asm/poll.h b/arch/sparc/include/uapi/asm/poll.h
index 2a81e79aa3ea6328fed587e5abd58fef7af1a201..72356c99912579666d30fcbb09f33479f327980d 100644 (file)
--- a/arch/sparc/include/uapi/asm/poll.h
+++ b/arch/sparc/include/uapi/asm/poll.h
@@ -2,31 +2,11 @@
 #ifndef __SPARC_POLL_H
 #define __SPARC_POLL_H
 
-#ifndef __KERNEL__
 #define POLLWRNORM     POLLOUT
-#define POLLWRBAND     (__force __poll_t)256
-#define POLLMSG                (__force __poll_t)512
-#define POLLREMOVE     (__force __poll_t)1024
-#define POLLRDHUP       (__force __poll_t)2048
-#else
-#define __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       __u16 v = (__force __u16)val;
-        /* bit 9 -> bit 8, bit 8 -> bit 2, bit 13 -> bit 11 */
-       return (v & ~0x300) | ((v & 0x200) >> 1) | ((v & 0x100) >> 6) |
-                               ((v & 0x2000) >> 2);
-
-
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-        /* bit 8 -> bit 9, bit 2 -> bits 2 and 8 */
-       return (__force __poll_t)((v & ~0x100) | ((v & 0x100) << 1) |
-                               ((v & 4) << 6) | ((v & 0x800) << 2));
-}
-#endif
+#define POLLWRBAND     256
+#define POLLMSG                512
+#define POLLREMOVE     1024
+#define POLLRDHUP       2048
 
 #include <asm-generic/poll.h>
 

diff --git a/arch/xtensa/include/uapi/asm/poll.h b/arch/xtensa/include/uapi/asm/poll.h
index e3246d41182c2b3c9bf6274f3450a22c62500bf7..4d249040b33d25a26ee4c211c051950f4ecf41f1 100644 (file)
--- a/arch/xtensa/include/uapi/asm/poll.h
+++ b/arch/xtensa/include/uapi/asm/poll.h
@@ -12,26 +12,9 @@
 #ifndef _XTENSA_POLL_H
 #define _XTENSA_POLL_H
 
-#ifndef __KERNEL__
 #define POLLWRNORM     POLLOUT
-#define POLLWRBAND     (__force __poll_t)0x0100
-#define POLLREMOVE     (__force __poll_t)0x0800
-#else
-#define __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       __u16 v = (__force __u16)val;
-       /* bit 9 -> bit 8, bit 8 -> bit 2 */
-       return (v & ~0x300) | ((v & 0x200) >> 1) | ((v & 0x100) >> 6);
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-        /* bit 8 -> bit 9, bit 2 -> bits 2 and 8 */
-       return (__force __poll_t)((v & ~0x100) | ((v & 0x100) << 1) |
-                               ((v & 4) << 6));
-}
-#endif
+#define POLLWRBAND     0x0100
+#define POLLREMOVE     0x0800
 
 #include <asm-generic/poll.h>
 

diff --git a/include/linux/poll.h b/include/linux/poll.h
index d23104b329317edcf3a9d1a906917392647b18af..f45ebd017eaa9732379e4225d8057a60897b7281 100644 (file)
--- a/include/linux/poll.h
+++ b/include/linux/poll.h
@@ -108,4 +108,28 @@ extern int core_sys_select(int n, fd_set __user *inp, fd_set __user *outp,
 extern int poll_select_set_timeout(struct timespec64 *to, time64_t sec,
                                   long nsec);
 
+#define __MAP(v, from, to) \
+       (from < to ? (v & from) * (to/from) : (v & from) / (from/to))
+
+static inline __u16 mangle_poll(__poll_t val)
+{
+       __u16 v = (__force __u16)val;
+#define M(X) __MAP(v, (__force __u16)EPOLL##X, POLL##X)
+       return M(IN) | M(OUT) | M(PRI) | M(ERR) | M(NVAL) |
+               M(RDNORM) | M(RDBAND) | M(WRNORM) | M(WRBAND) |
+               M(HUP) | M(RDHUP) | M(MSG);
+#undef M
+}
+
+static inline __poll_t demangle_poll(u16 val)
+{
+#define M(X) (__force __poll_t)__MAP(val, POLL##X, (__force __u16)EPOLL##X)
+       return M(IN) | M(OUT) | M(PRI) | M(ERR) | M(NVAL) |
+               M(RDNORM) | M(RDBAND) | M(WRNORM) | M(WRBAND) |
+               M(HUP) | M(RDHUP) | M(MSG);
+#undef M
+}
+#undef __MAP
+
+
 #endif /* _LINUX_POLL_H */

diff --git a/include/uapi/asm-generic/poll.h b/include/uapi/asm-generic/poll.h
index 639fade14b2343a776c01993c328487533830696..41b509f410bf9b7f50f2c2e31542f170ee28f38a 100644 (file)
--- a/include/uapi/asm-generic/poll.h
+++ b/include/uapi/asm-generic/poll.h
@@ -3,50 +3,36 @@
 #define __ASM_GENERIC_POLL_H
 
 /* These are specified by iBCS2 */
-#define POLLIN         (__force __poll_t)0x0001
-#define POLLPRI                (__force __poll_t)0x0002
-#define POLLOUT                (__force __poll_t)0x0004
-#define POLLERR                (__force __poll_t)0x0008
-#define POLLHUP                (__force __poll_t)0x0010
-#define POLLNVAL       (__force __poll_t)0x0020
+#define POLLIN         0x0001
+#define POLLPRI                0x0002
+#define POLLOUT                0x0004
+#define POLLERR                0x0008
+#define POLLHUP                0x0010
+#define POLLNVAL       0x0020
 
 /* The rest seem to be more-or-less nonstandard. Check them! */
-#define POLLRDNORM     (__force __poll_t)0x0040
-#define POLLRDBAND     (__force __poll_t)0x0080
+#define POLLRDNORM     0x0040
+#define POLLRDBAND     0x0080
 #ifndef POLLWRNORM
-#define POLLWRNORM     (__force __poll_t)0x0100
+#define POLLWRNORM     0x0100
 #endif
 #ifndef POLLWRBAND
-#define POLLWRBAND     (__force __poll_t)0x0200
+#define POLLWRBAND     0x0200
 #endif
 #ifndef POLLMSG
-#define POLLMSG                (__force __poll_t)0x0400
+#define POLLMSG                0x0400
 #endif
 #ifndef POLLREMOVE
-#define POLLREMOVE     (__force __poll_t)0x1000
+#define POLLREMOVE     0x1000
 #endif
 #ifndef POLLRDHUP
-#define POLLRDHUP       (__force __poll_t)0x2000
+#define POLLRDHUP       0x2000
 #endif
 
 #define POLLFREE       (__force __poll_t)0x4000        /* currently only for epoll */
 
 #define POLL_BUSY_LOOP (__force __poll_t)0x8000
 
-#ifdef __KERNEL__
-#ifndef __ARCH_HAS_MANGLED_POLL
-static inline __u16 mangle_poll(__poll_t val)
-{
-       return (__force __u16)val;
-}
-
-static inline __poll_t demangle_poll(__u16 v)
-{
-       return (__force __poll_t)v;
-}
-#endif
-#endif
-
 struct pollfd {
        int fd;
        short events;