From patchwork Wed Apr 25 15:01:52 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 134322 Delivered-To: patch@linaro.org Received: by 10.46.151.6 with SMTP id r6csp976306lji; Wed, 25 Apr 2018 08:02:30 -0700 (PDT) X-Google-Smtp-Source: AIpwx489jD/I+41KIfPo12c1xA/zw1bnlGFDiFWER/S/P1BuSWgF61mQj0aJE0WOllqxRwm8ATyR X-Received: by 10.99.157.12 with SMTP id i12mr23226747pgd.87.1524668550784; Wed, 25 Apr 2018 08:02:30 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1524668550; cv=none; d=google.com; s=arc-20160816; b=kTdfEzWa581aDCag9Le8R63+m60ZwnTDUtm/4/7fcbJHKrQtY0+Tn94mGqagrKuHya kkWpU6DKAoe5LLw45pSnaoZYghmjDDF6ReA82ss8dn9/WQCmwiweIwmZI/cZKsIqyAon BJCZKDuP4lKPhczz2qO4OdtX8XJCSlaLDQrwojUlz3o7Ef0ChvOH/5mn5hwaLnbI42Dn 70gzz+KujovtANM8NaSPJzpB5kIsL6rfcYTWxaOaioG2QpQyFY+X7kLB06ckJh7iDz2W 89oxW5GBWZrLgvm9xwUqwFjCZFG+BZcKPsPigJvv3a3bCMzdB+Hs+uFv65hKJY6ciHHo dgcg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=n21C6+7DXqa1/UHMo9GxpKf5rBOCrpEgn9YzE+CmHpc=; b=okU511gRIN5TXgRlnjaRYXew0GtyFYoBMtLmvZFBaaMBYmlTVJy2Ex03IrFdr8fxbV 47NZHzj1brqYjIzkmUn6jqY1Gz5n1u3ykXlaeP+hegHwD4r7VjGApjDng+m6Am6A/AJj kng9HRErTd6AX2DLDMIkhVNkcBcDoGUXuhKBhO5izuCouozqifgIpDo7BfWZ/e3lvDb4 jnQN7TfNAlTG73U2trSvd/944OA9rOPEgAQbT1lC6zKCRGD72HXrZAsmkgdNu9ZwjgxY b4BsvFdIGVsY2Tm3GRp6WsDgrTqwKvagSfxBStKZBnhCkh4kVR7mruAH8CjxdS8med6J lUHw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l15si11507103pgn.414.2018.04.25.08.02.23; Wed, 25 Apr 2018 08:02:30 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754747AbeDYPCU (ORCPT + 29 others); Wed, 25 Apr 2018 11:02:20 -0400 Received: from mout.kundenserver.de ([212.227.17.24]:45297 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753632AbeDYPCR (ORCPT ); Wed, 25 Apr 2018 11:02:17 -0400 Received: from wuerfel.lan ([95.208.111.237]) by mrelayeu.kundenserver.de (mreue101 [212.227.15.145]) with ESMTPA (Nemesis) id 0MAdid-1f5bhD0BOl-00BrqX; Wed, 25 Apr 2018 17:02:16 +0200 From: Arnd Bergmann To: y2038@lists.linaro.org, linux-kernel@vger.kernel.org Cc: Arnd Bergmann Subject: [PATCH v4 01/16] y2038: asm-generic: Extend sysvipc data structures Date: Wed, 25 Apr 2018 17:01:52 +0200 Message-Id: <20180425150207.822862-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 In-Reply-To: <20180425132242.1500539-1-arnd@arndb.de> References: <20180425132242.1500539-1-arnd@arndb.de> X-Provags-ID: V03:K1:JwebJoz2DT5XGQYwCKhblEr7blP17+NBwn+jq1isdz3p19V2cH2 yW8YyGscWzyR3xAtlaBXHi2kDwtYEV3yESQ+TFAlbEpuKnfzR+RePBDRHWXK6SL8ee9zytY nf95IGn85ZSKyPMj7wfevnSlXW6Vktu6iMCzzPx/sSU1eB6wCcZ3z2icX2OsCso3fxwxnhC lVGQg3BPM0V+byHlLvZFg== X-UI-Out-Filterresults: notjunk:1; V01:K0:HhBwvrEyZck=:NgqBgNRqCYjF6u98iFvehJ zMJMUi2guWWVsiozAKwLxmz4r8hZCWbIP8mZSZbC3PeI5q2JNZGELxbNRY32HEPpbKWgfDKQ0 gCn63/db8mVvPakmNLvftZo3gWHgnxs+VswXQAeDwxwDC+fV6dGQbaZIeZ2QIUTfEzAVqmdB2 373bP9mCEYCXfe7vSV+c088zQy6tjVebj8CV0qkgq112IWrtQNO8MjYVAD6nIFORr2gwImkNR eiym0VM3rKWcrjOv9iHvAuKxN7wlnIUamRqX2gzwdP3DXQ9+4A2C9POBk8IKoRetagkpOwxLq gHc39zkHaFZkWEREvkOPrN0bQ1U0lQOdwz+MKdA4Ddo75Ow9logK5JrjoGtWcMxTU5drXhURe xyHW38mYOiYQPTWpcDR9Hwe0MlEELWKZWwU3CvfP+lFljtzHt4+qQETzgh5D3mhTlB1bMmufm 0s/8O0NpfDdMR72CZdMLpiu+PsPOzy/ZIuGltQ3U2JP7MgkrVvb7YWcejciPZAHpDQgr+/Cap 7GU4iETEUcSw0+mKvkz7Je/tE3RpiOCVBKFh0leQrR+kSZNHxNVRPYeWgDGvbjJ35LZwXM0Ke rGAQtw3ZU7XTueRzKATf+syNEqJjCo9hY4ZtSh1yfbVXczztQYT5Mj8BC22DvQAPphhoonLRi gQCTRscXzPAEC10qsuLPPkj3uPB1oi2NWRzKpZzu8DVZr6idmz1c3CCDeoJ2JwjtutstyiBc1 QmrxUjo02gYywvnJy/DlOCcnp66mMPFm5mofig== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Most architectures now use the asm-generic copy of the sysvipc data structures (msqid64_ds, semid64_ds, shmid64_ds), which use 32-bit __kernel_time_t on 32-bit architectures but have padding behind them to allow extending the type to 64-bit. Unfortunately, that fails on all big-endian architectures, which have the padding on the wrong side. As so many of them get it wrong, we decided to not bother even trying to fix it up when we introduced the asm-generic copy. Instead we always use the padding word now to provide the upper 32 bits of the seconds value, regardless of the endianess. A libc implementation on a typical big-endian system can deal with this by providing its own copy of the structure definition to user space, and swapping the two 32-bit words before returning from the semctl/shmctl/msgctl system calls. Note that msqid64_ds and shmid64_ds were broken on x32 since commit f4b4aae18288 ("x86/headers/uapi: Fix __BITS_PER_LONG value for x32 builds"). I have sent a separate fix for that, but as we no longer have to worry about x32 here, I no longer worry about x32 here and use 'unsigned long' instead of __kernel_ulong_t. Signed-off-by: Arnd Bergmann --- include/uapi/asm-generic/msgbuf.h | 27 +++++++++++++------------- include/uapi/asm-generic/sembuf.h | 26 +++++++++++++++---------- include/uapi/asm-generic/shmbuf.h | 41 +++++++++++++++++++-------------------- 3 files changed, 49 insertions(+), 45 deletions(-) -- 2.9.0 diff --git a/include/uapi/asm-generic/msgbuf.h b/include/uapi/asm-generic/msgbuf.h index fb306ebdb36f..9fe4881557cb 100644 --- a/include/uapi/asm-generic/msgbuf.h +++ b/include/uapi/asm-generic/msgbuf.h @@ -18,31 +18,30 @@ * On big-endian systems, the padding is in the wrong place. * * Pad space is left for: - * - 64-bit time_t to solve y2038 problem * - 2 miscellaneous 32-bit values */ struct msqid64_ds { struct ipc64_perm msg_perm; +#if __BITS_PER_LONG == 64 __kernel_time_t msg_stime; /* last msgsnd time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused1; -#endif __kernel_time_t msg_rtime; /* last msgrcv time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused2; -#endif __kernel_time_t msg_ctime; /* last change time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused3; +#else + unsigned long msg_stime; /* last msgsnd time */ + unsigned long msg_stime_high; + unsigned long msg_rtime; /* last msgrcv time */ + unsigned long msg_rtime_high; + unsigned long msg_ctime; /* last change time */ + unsigned long msg_ctime_high; #endif - __kernel_ulong_t msg_cbytes; /* current number of bytes on queue */ - __kernel_ulong_t msg_qnum; /* number of messages in queue */ - __kernel_ulong_t msg_qbytes; /* max number of bytes on queue */ + unsigned long msg_cbytes; /* current number of bytes on queue */ + unsigned long msg_qnum; /* number of messages in queue */ + unsigned long msg_qbytes; /* max number of bytes on queue */ __kernel_pid_t msg_lspid; /* pid of last msgsnd */ __kernel_pid_t msg_lrpid; /* last receive pid */ - __kernel_ulong_t __unused4; - __kernel_ulong_t __unused5; + unsigned long __unused4; + unsigned long __unused5; }; #endif /* __ASM_GENERIC_MSGBUF_H */ diff --git a/include/uapi/asm-generic/sembuf.h b/include/uapi/asm-generic/sembuf.h index cbf9cfe977d6..0bae010f1b64 100644 --- a/include/uapi/asm-generic/sembuf.h +++ b/include/uapi/asm-generic/sembuf.h @@ -13,23 +13,29 @@ * everyone just ended up making identical copies without specific * optimizations, so we may just as well all use the same one. * - * 64 bit architectures typically define a 64 bit __kernel_time_t, + * 64 bit architectures use a 64-bit __kernel_time_t here, while + * 32 bit architectures have a pair of unsigned long values. * so they do not need the first two padding words. - * On big-endian systems, the padding is in the wrong place. * - * Pad space is left for: - * - 64-bit time_t to solve y2038 problem - * - 2 miscellaneous 32-bit values + * On big-endian systems, the padding is in the wrong place for + * historic reasons, so user space has to reconstruct a time_t + * value using + * + * user_semid_ds.sem_otime = kernel_semid64_ds.sem_otime + + * ((long long)kernel_semid64_ds.sem_otime_high << 32) + * + * Pad space is left for 2 miscellaneous 32-bit values */ struct semid64_ds { struct ipc64_perm sem_perm; /* permissions .. see ipc.h */ +#if __BITS_PER_LONG == 64 __kernel_time_t sem_otime; /* last semop time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused1; -#endif __kernel_time_t sem_ctime; /* last change time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused2; +#else + unsigned long sem_otime; /* last semop time */ + unsigned long sem_otime_high; + unsigned long sem_ctime; /* last change time */ + unsigned long sem_ctime_high; #endif unsigned long sem_nsems; /* no. of semaphores in array */ unsigned long __unused3; diff --git a/include/uapi/asm-generic/shmbuf.h b/include/uapi/asm-generic/shmbuf.h index 2b6c3bb97f97..e504422fc501 100644 --- a/include/uapi/asm-generic/shmbuf.h +++ b/include/uapi/asm-generic/shmbuf.h @@ -19,42 +19,41 @@ * * * Pad space is left for: - * - 64-bit time_t to solve y2038 problem * - 2 miscellaneous 32-bit values */ struct shmid64_ds { struct ipc64_perm shm_perm; /* operation perms */ size_t shm_segsz; /* size of segment (bytes) */ +#if __BITS_PER_LONG == 64 __kernel_time_t shm_atime; /* last attach time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused1; -#endif __kernel_time_t shm_dtime; /* last detach time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused2; -#endif __kernel_time_t shm_ctime; /* last change time */ -#if __BITS_PER_LONG != 64 - unsigned long __unused3; +#else + unsigned long shm_atime; /* last attach time */ + unsigned long shm_atime_high; + unsigned long shm_dtime; /* last detach time */ + unsigned long shm_dtime_high; + unsigned long shm_ctime; /* last change time */ + unsigned long shm_ctime_high; #endif __kernel_pid_t shm_cpid; /* pid of creator */ __kernel_pid_t shm_lpid; /* pid of last operator */ - __kernel_ulong_t shm_nattch; /* no. of current attaches */ - __kernel_ulong_t __unused4; - __kernel_ulong_t __unused5; + unsigned long shm_nattch; /* no. of current attaches */ + unsigned long __unused4; + unsigned long __unused5; }; struct shminfo64 { - __kernel_ulong_t shmmax; - __kernel_ulong_t shmmin; - __kernel_ulong_t shmmni; - __kernel_ulong_t shmseg; - __kernel_ulong_t shmall; - __kernel_ulong_t __unused1; - __kernel_ulong_t __unused2; - __kernel_ulong_t __unused3; - __kernel_ulong_t __unused4; + unsigned long shmmax; + unsigned long shmmin; + unsigned long shmmni; + unsigned long shmseg; + unsigned long shmall; + unsigned long __unused1; + unsigned long __unused2; + unsigned long __unused3; + unsigned long __unused4; }; #endif /* __ASM_GENERIC_SHMBUF_H */