Commit-ID:  0fe2795516b9e1c59b58b02bdf8658698117ec4e
Gitweb:     https://git.kernel.org/tip/0fe2795516b9e1c59b58b02bdf8658698117ec4e
Author:     Arnd Bergmann <a...@arndb.de>
AuthorDate: Mon, 18 Jun 2018 16:07:59 +0200
Committer:  Thomas Gleixner <t...@linutronix.de>
CommitDate: Tue, 19 Jun 2018 09:23:19 +0200

posix-timers: Fix nanosleep_copyout() for CONFIG_COMPAT_32BIT_TIME

Commit b5793b0d92c9 added support for building the nanosleep compat system
call on 32-bit architectures, but missed one change in nanosleep_copyout(),
which would trigger a BUG() as soon as any architecture is switched over to
use it.

Use the proper config symbol to enable the code path.

Fixes: Commit b5793b0d92c9 ("posix-timers: Make compat syscalls depend on 
CONFIG_COMPAT_32BIT_TIME")
Signed-off-by: Arnd Bergmann <a...@arndb.de>
Signed-off-by: Thomas Gleixner <t...@linutronix.de>
Cc: y2...@lists.linaro.org
Cc: Anna-Maria Gleixner <anna-ma...@linutronix.de>
Cc: Deepa Dinamani <deepa.ker...@gmail.com>
Cc: "Rafael J. Wysocki" <rafael.j.wyso...@intel.com>
Link: https://lkml.kernel.org/r/20180618140811.2998503-1-a...@arndb.de

---
 kernel/time/hrtimer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/time/hrtimer.c b/kernel/time/hrtimer.c
index 055a4a728c00..3e93c54bd3a1 100644
--- a/kernel/time/hrtimer.c
+++ b/kernel/time/hrtimer.c
@@ -1659,7 +1659,7 @@ EXPORT_SYMBOL_GPL(hrtimer_init_sleeper);
 int nanosleep_copyout(struct restart_block *restart, struct timespec64 *ts)
 {
        switch(restart->nanosleep.type) {
-#ifdef CONFIG_COMPAT
+#ifdef CONFIG_COMPAT_32BIT_TIME
        case TT_COMPAT:
                if (compat_put_timespec64(ts, restart->nanosleep.compat_rmtp))
                        return -EFAULT;

Reply via email to