Skip to content

Commit 6e8d969

Browse files
committed
asm-generic: partially revert "Unify uapi bitsperlong.h for arm64, riscv and loongarch"
Unifying the asm-generic headers across 32-bit and 64-bit architectures based on the compiler provided macros was a good idea and appears to work with all user space, but it caused a regression when building old kernels on systems that have the new headers installed in /usr/include, as this combination trips an inconsistency in the kernel's own tools/include headers that are a mix of userspace and kernel-internal headers. This affects kernel builds on arm64, riscv64 and loongarch64 systems that might end up using the "#define __BITS_PER_LONG 32" default from the old tools headers. Backporting the commit into stable kernels would address this, but it would still break building kernels without that backport, and waste time for developers trying to understand the problem. arm64 build machines are rather common, and on riscv64 this can also happen in practice, but loongarch64 is probably new enough to not be used much for building old kernels, so only revert the bits for arm64 and riscv. Link: https://lore.kernel.org/all/20230731160402.GB1823389@dev-arch.thelio-3990X/ Reported-by: Nathan Chancellor <nathan@kernel.org> Fixes: 8386f58 ("asm-generic: Unify uapi bitsperlong.h for arm64, riscv and loongarch") Acked-by: Catalin Marinas <catalin.marinas@arm.com> Acked-by: Palmer Dabbelt <palmer@rivosinc.com> Tested-by: Nathan Chancellor <nathan@kernel.org> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
1 parent 4dd595c commit 6e8d969

File tree

4 files changed

+76
-0
lines changed

4 files changed

+76
-0
lines changed
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
2+
/*
3+
* Copyright (C) 2012 ARM Ltd.
4+
*
5+
* This program is free software; you can redistribute it and/or modify
6+
* it under the terms of the GNU General Public License version 2 as
7+
* published by the Free Software Foundation.
8+
*
9+
* This program is distributed in the hope that it will be useful,
10+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
11+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12+
* GNU General Public License for more details.
13+
*
14+
* You should have received a copy of the GNU General Public License
15+
* along with this program. If not, see <http://www.gnu.org/licenses/>.
16+
*/
17+
#ifndef __ASM_BITSPERLONG_H
18+
#define __ASM_BITSPERLONG_H
19+
20+
#define __BITS_PER_LONG 64
21+
22+
#include <asm-generic/bitsperlong.h>
23+
24+
#endif /* __ASM_BITSPERLONG_H */
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
/* SPDX-License-Identifier: GPL-2.0-only WITH Linux-syscall-note */
2+
/*
3+
* Copyright (C) 2012 ARM Ltd.
4+
* Copyright (C) 2015 Regents of the University of California
5+
*/
6+
7+
#ifndef _UAPI_ASM_RISCV_BITSPERLONG_H
8+
#define _UAPI_ASM_RISCV_BITSPERLONG_H
9+
10+
#define __BITS_PER_LONG (__SIZEOF_POINTER__ * 8)
11+
12+
#include <asm-generic/bitsperlong.h>
13+
14+
#endif /* _UAPI_ASM_RISCV_BITSPERLONG_H */
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
2+
/*
3+
* Copyright (C) 2012 ARM Ltd.
4+
*
5+
* This program is free software; you can redistribute it and/or modify
6+
* it under the terms of the GNU General Public License version 2 as
7+
* published by the Free Software Foundation.
8+
*
9+
* This program is distributed in the hope that it will be useful,
10+
* but WITHOUT ANY WARRANTY; without even the implied warranty of
11+
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12+
* GNU General Public License for more details.
13+
*
14+
* You should have received a copy of the GNU General Public License
15+
* along with this program. If not, see <http://www.gnu.org/licenses/>.
16+
*/
17+
#ifndef __ASM_BITSPERLONG_H
18+
#define __ASM_BITSPERLONG_H
19+
20+
#define __BITS_PER_LONG 64
21+
22+
#include <asm-generic/bitsperlong.h>
23+
24+
#endif /* __ASM_BITSPERLONG_H */
Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
/* SPDX-License-Identifier: GPL-2.0-only */
2+
/*
3+
* Copyright (C) 2012 ARM Ltd.
4+
* Copyright (C) 2015 Regents of the University of California
5+
*/
6+
7+
#ifndef _UAPI_ASM_RISCV_BITSPERLONG_H
8+
#define _UAPI_ASM_RISCV_BITSPERLONG_H
9+
10+
#define __BITS_PER_LONG (__SIZEOF_POINTER__ * 8)
11+
12+
#include <asm-generic/bitsperlong.h>
13+
14+
#endif /* _UAPI_ASM_RISCV_BITSPERLONG_H */

0 commit comments

Comments
 (0)