]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
raid6: remove __KERNEL__ ifdefs
authorChristoph Hellwig <hch@lst.de>
Mon, 18 May 2026 05:17:45 +0000 (07:17 +0200)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 29 May 2026 04:24:52 +0000 (21:24 -0700)
With the test code ported to kernel space, none of this is required.

Link: https://lore.kernel.org/20260518051804.462141-3-hch@lst.de
Signed-off-by: Christoph Hellwig <hch@lst.de>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Tested-by: Ard Biesheuvel <ardb@kernel.org> # kunit only on arm64
Cc: Albert Ou <aou@eecs.berkeley.edu>
Cc: Alexander Gordeev <agordeev@linux.ibm.com>
Cc: Alexandre Ghiti <alex@ghiti.fr>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: "Borislav Petkov (AMD)" <bp@alien8.de>
Cc: Catalin Marinas <catalin.marinas@arm.com>
Cc: Chris Mason <clm@fb.com>
Cc: Christian Borntraeger <borntraeger@linux.ibm.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Cc: David Sterba <dsterba@suse.com>
Cc: Heiko Carstens <hca@linux.ibm.com>
Cc: Herbert Xu <herbert@gondor.apana.org.au>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Huacai Chen <chenhuacai@kernel.org>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Li Nan <linan122@huawei.com>
Cc: Madhavan Srinivasan <maddy@linux.ibm.com>
Cc: Michael Ellerman <mpe@ellerman.id.au>
Cc: Nicholas Piggin <npiggin@gmail.com>
Cc: Palmer Dabbelt <palmer@dabbelt.com>
Cc: Song Liu <song@kernel.org>
Cc: Sven Schnelle <svens@linux.ibm.com>
Cc: Vasily Gorbik <gor@linux.ibm.com>
Cc: WANG Xuerui <kernel@xen0n.name>
Cc: Will Deacon <will@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
20 files changed:
include/linux/raid/pq.h
lib/raid6/algos.c
lib/raid6/altivec.uc
lib/raid6/avx2.c
lib/raid6/avx512.c
lib/raid6/loongarch.h [deleted file]
lib/raid6/loongarch_simd.c
lib/raid6/mktables.c
lib/raid6/mmx.c
lib/raid6/neon.c
lib/raid6/recov_avx2.c
lib/raid6/recov_avx512.c
lib/raid6/recov_loongarch_simd.c
lib/raid6/recov_neon.c
lib/raid6/recov_ssse3.c
lib/raid6/rvv.h
lib/raid6/sse1.c
lib/raid6/sse2.c
lib/raid6/vpermxor.uc
lib/raid6/x86.h [deleted file]

index 08c5995ea9806b2f4fb6745b6123b47a60fae57c..d26788fada585809341efc560b90d9e73fdb195d 100644 (file)
@@ -8,8 +8,6 @@
 #ifndef LINUX_RAID_RAID6_H
 #define LINUX_RAID_RAID6_H
 
-#ifdef __KERNEL__
-
 #include <linux/blkdev.h>
 #include <linux/mm.h>
 
@@ -19,59 +17,6 @@ static inline void *raid6_get_zero_page(void)
        return page_address(ZERO_PAGE(0));
 }
 
-#else /* ! __KERNEL__ */
-/* Used for testing in user space */
-
-#include <errno.h>
-#include <inttypes.h>
-#include <stddef.h>
-#include <string.h>
-#include <sys/mman.h>
-#include <sys/time.h>
-#include <sys/types.h>
-
-/* Not standard, but glibc defines it */
-#define BITS_PER_LONG __WORDSIZE
-
-typedef uint8_t  u8;
-typedef uint16_t u16;
-typedef uint32_t u32;
-typedef uint64_t u64;
-
-#ifndef PAGE_SIZE
-# define PAGE_SIZE 4096
-#endif
-#ifndef PAGE_SHIFT
-# define PAGE_SHIFT 12
-#endif
-extern const char raid6_empty_zero_page[PAGE_SIZE];
-
-#define __init
-#define __exit
-#ifndef __attribute_const__
-# define __attribute_const__ __attribute__((const))
-#endif
-#define noinline __attribute__((noinline))
-
-#define preempt_enable()
-#define preempt_disable()
-#define cpu_has_feature(x) 1
-#define enable_kernel_altivec()
-#define disable_kernel_altivec()
-
-#undef EXPORT_SYMBOL
-#define EXPORT_SYMBOL(sym)
-#undef EXPORT_SYMBOL_GPL
-#define EXPORT_SYMBOL_GPL(sym)
-#define MODULE_LICENSE(licence)
-#define MODULE_DESCRIPTION(desc)
-#define subsys_initcall(x)
-#define module_exit(x)
-
-#define IS_ENABLED(x) (x)
-#define CONFIG_RAID6_PQ_BENCHMARK 1
-#endif /* __KERNEL__ */
-
 /* Routine choices */
 struct raid6_calls {
        void (*gen_syndrome)(int, size_t, void **);
@@ -165,39 +110,4 @@ extern void (*raid6_2data_recov)(int disks, size_t bytes, int faila, int failb,
 extern void (*raid6_datap_recov)(int disks, size_t bytes, int faila,
                        void **ptrs);
 
-/* Some definitions to allow code to be compiled for testing in userspace */
-#ifndef __KERNEL__
-
-# define jiffies       raid6_jiffies()
-# define printk        printf
-# define pr_err(format, ...) fprintf(stderr, format, ## __VA_ARGS__)
-# define pr_info(format, ...) fprintf(stdout, format, ## __VA_ARGS__)
-# define GFP_KERNEL    0
-# define __get_free_pages(x, y)        ((unsigned long)mmap(NULL, PAGE_SIZE << (y), \
-                                                    PROT_READ|PROT_WRITE,   \
-                                                    MAP_PRIVATE|MAP_ANONYMOUS,\
-                                                    0, 0))
-# define free_pages(x, y)      munmap((void *)(x), PAGE_SIZE << (y))
-
-static inline void cpu_relax(void)
-{
-       /* Nothing */
-}
-
-#undef  HZ
-#define HZ 1000
-static inline uint32_t raid6_jiffies(void)
-{
-       struct timeval tv;
-       gettimeofday(&tv, NULL);
-       return tv.tv_sec*1000 + tv.tv_usec/1000;
-}
-
-static inline void *raid6_get_zero_page(void)
-{
-       return raid6_empty_zero_page;
-}
-
-#endif /* ! __KERNEL__ */
-
 #endif /* LINUX_RAID_RAID6_H */
index 5a9f4882e18dd9605d25867d77178601bca57ddb..985c60bb00a41a7df9ecae20666ca684248e1ad9 100644 (file)
  */
 
 #include <linux/raid/pq.h>
-#ifndef __KERNEL__
-#include <sys/mman.h>
-#include <stdio.h>
-#else
 #include <linux/module.h>
 #include <linux/gfp.h>
-#endif
 #include <kunit/visibility.h>
 
 struct raid6_calls raid6_call;
@@ -123,14 +118,7 @@ const struct raid6_recov_calls *const raid6_recov_algos[] = {
 };
 EXPORT_SYMBOL_IF_KUNIT(raid6_recov_algos);
 
-#ifdef __KERNEL__
 #define RAID6_TIME_JIFFIES_LG2 4
-#else
-/* Need more time to be stable in userspace */
-#define RAID6_TIME_JIFFIES_LG2 9
-#define time_before(x, y) ((x) < (y))
-#endif
-
 #define RAID6_TEST_DISKS       8
 #define RAID6_TEST_DISKS_ORDER 3
 
index d20ed0d114111cc55706d45739263d48be505f32..2c59963e58f9b3b75a60e56fa2f8d9d3a3901570 100644 (file)
 #ifdef CONFIG_ALTIVEC
 
 #include <altivec.h>
-#ifdef __KERNEL__
-# include <asm/cputable.h>
-# include <asm/switch_to.h>
-#endif /* __KERNEL__ */
+#include <asm/cputable.h>
+#include <asm/switch_to.h>
 
 /*
  * This is the C data type to use.  We use a vector of
@@ -113,11 +111,7 @@ int raid6_have_altivec(void);
 int raid6_have_altivec(void)
 {
        /* This assumes either all CPUs have Altivec or none does */
-# ifdef __KERNEL__
        return cpu_has_feature(CPU_FTR_ALTIVEC);
-# else
-       return 1;
-# endif
 }
 #endif
 
index 059024234dce15bd826afd33428494870b8e93ab..a1a5213918af85937ade9b0028e2524aedd60e64 100644 (file)
@@ -14,7 +14,7 @@
  */
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 static const struct raid6_avx2_constants {
        u64 x1d[4];
index 009bd0adeebf0388353a3cfbc531d3abecea1add..874998bcd7d78ce4336ab95e6fcf0a014a98cf85 100644 (file)
@@ -18,7 +18,7 @@
  */
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 static const struct raid6_avx512_constants {
        u64 x1d[8];
diff --git a/lib/raid6/loongarch.h b/lib/raid6/loongarch.h
deleted file mode 100644 (file)
index acfc33c..0000000
+++ /dev/null
@@ -1,38 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-/*
- * Copyright (C) 2023 WANG Xuerui <git@xen0n.name>
- *
- * raid6/loongarch.h
- *
- * Definitions common to LoongArch RAID-6 code only
- */
-
-#ifndef _LIB_RAID6_LOONGARCH_H
-#define _LIB_RAID6_LOONGARCH_H
-
-#ifdef __KERNEL__
-
-#include <asm/cpu-features.h>
-#include <asm/fpu.h>
-
-#else /* for user-space testing */
-
-#include <sys/auxv.h>
-
-/* have to supply these defines for glibc 2.37- and musl */
-#ifndef HWCAP_LOONGARCH_LSX
-#define HWCAP_LOONGARCH_LSX    (1 << 4)
-#endif
-#ifndef HWCAP_LOONGARCH_LASX
-#define HWCAP_LOONGARCH_LASX   (1 << 5)
-#endif
-
-#define kernel_fpu_begin()
-#define kernel_fpu_end()
-
-#define cpu_has_lsx    (getauxval(AT_HWCAP) & HWCAP_LOONGARCH_LSX)
-#define cpu_has_lasx   (getauxval(AT_HWCAP) & HWCAP_LOONGARCH_LASX)
-
-#endif /* __KERNEL__ */
-
-#endif /* _LIB_RAID6_LOONGARCH_H */
index aa5d9f924ca39694f7487cf3efc7295c78574c66..72f4d92d4876d6bbf420fb5ae30aa273a35192e7 100644 (file)
@@ -10,7 +10,8 @@
  */
 
 #include <linux/raid/pq.h>
-#include "loongarch.h"
+#include <asm/cpu-features.h>
+#include <asm/fpu.h>
 
 /*
  * The vector algorithms are currently priority 0, which means the generic
index 3be03793237c2e91b4d09d168f7400897c393e20..3de1dbf6846c4d374cbc8b5a8d61df238b4fd255 100644 (file)
@@ -56,9 +56,7 @@ int main(int argc, char *argv[])
        uint8_t v;
        uint8_t exptbl[256], invtbl[256];
 
-       printf("#ifdef __KERNEL__\n");
        printf("#include <linux/export.h>\n");
-       printf("#endif\n");
        printf("#include <linux/raid/pq.h>\n");
 
        /* Compute multiplication table */
@@ -76,9 +74,7 @@ int main(int argc, char *argv[])
                printf("\t},\n");
        }
        printf("};\n");
-       printf("#ifdef __KERNEL__\n");
        printf("EXPORT_SYMBOL(raid6_gfmul);\n");
-       printf("#endif\n");
 
        /* Compute vector multiplication table */
        printf("\nconst u8  __attribute__((aligned(256)))\n"
@@ -101,9 +97,7 @@ int main(int argc, char *argv[])
                printf("\t},\n");
        }
        printf("};\n");
-       printf("#ifdef __KERNEL__\n");
        printf("EXPORT_SYMBOL(raid6_vgfmul);\n");
-       printf("#endif\n");
 
        /* Compute power-of-2 table (exponent) */
        v = 1;
@@ -120,9 +114,7 @@ int main(int argc, char *argv[])
                }
        }
        printf("};\n");
-       printf("#ifdef __KERNEL__\n");
        printf("EXPORT_SYMBOL(raid6_gfexp);\n");
-       printf("#endif\n");
 
        /* Compute log-of-2 table */
        printf("\nconst u8 __attribute__((aligned(256)))\n"
@@ -140,9 +132,7 @@ int main(int argc, char *argv[])
                }
        }
        printf("};\n");
-       printf("#ifdef __KERNEL__\n");
        printf("EXPORT_SYMBOL(raid6_gflog);\n");
-       printf("#endif\n");
 
        /* Compute inverse table x^-1 == x^254 */
        printf("\nconst u8 __attribute__((aligned(256)))\n"
@@ -155,9 +145,7 @@ int main(int argc, char *argv[])
                }
        }
        printf("};\n");
-       printf("#ifdef __KERNEL__\n");
        printf("EXPORT_SYMBOL(raid6_gfinv);\n");
-       printf("#endif\n");
 
        /* Compute inv(2^x + 1) (exponent-xor-inverse) table */
        printf("\nconst u8 __attribute__((aligned(256)))\n"
@@ -169,9 +157,7 @@ int main(int argc, char *argv[])
                               (j == 7) ? '\n' : ' ');
        }
        printf("};\n");
-       printf("#ifdef __KERNEL__\n");
        printf("EXPORT_SYMBOL(raid6_gfexi);\n");
-       printf("#endif\n");
 
        return 0;
 }
index 3a5bf53a297b4a87aa929e7e1210565ec1ea102d..e411f0cfbd9585c0161a363ef2d7b1710ab123b0 100644 (file)
@@ -14,7 +14,7 @@
 #ifdef CONFIG_X86_32
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 /* Shared with raid6/sse1.c */
 const struct raid6_mmx_constants {
index 6d9474ce6da9171e8d735b4c5a7e6e48c0c8ace7..47b8bb0afc65f9c24d76d03c1a56c060ee225621 100644 (file)
@@ -6,13 +6,7 @@
  */
 
 #include <linux/raid/pq.h>
-
-#ifdef __KERNEL__
 #include <asm/simd.h>
-#else
-#define scoped_ksimd()
-#define cpu_has_neon()         (1)
-#endif
 
 /*
  * There are 2 reasons these wrappers are kept in a separate compilation unit
index 97d598d2535ca68d34a92e95a8c524af2885e044..19fbd9c4dce63e239150c7e77a19cea0f29ef6a1 100644 (file)
@@ -5,7 +5,7 @@
  */
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 static int raid6_has_avx2(void)
 {
index 7986120ca44429e49b99b8181aa5526114cdd2f9..143f4976b2ade6c01cc6b695bef4be89b52face0 100644 (file)
@@ -7,7 +7,7 @@
  */
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 static int raid6_has_avx512(void)
 {
index 93dc515997a14e3356d5ef0b53c4b9961c3929af..eb3a1e79f01f7b3b5068ceefaef3d99f8464ebfd 100644 (file)
@@ -11,7 +11,8 @@
  */
 
 #include <linux/raid/pq.h>
-#include "loongarch.h"
+#include <asm/cpu-features.h>
+#include <asm/fpu.h>
 
 /*
  * Unlike with the syndrome calculation algorithms, there's no boot-time
index 9d99aeabd31a9098eca437c4e5c26c98cce87b5a..13d5df718c15fc7bac46bf101dd631c382a492ee 100644 (file)
@@ -5,14 +5,8 @@
  */
 
 #include <linux/raid/pq.h>
-
-#ifdef __KERNEL__
 #include <asm/simd.h>
 #include "neon.h"
-#else
-#define scoped_ksimd()
-#define cpu_has_neon()         (1)
-#endif
 
 static int raid6_has_neon(void)
 {
index 2e849185c32b3caec642e9010b76cb4f777342ad..146cdbf465bd7af6ea13a4ce7f23c7fc03abf3c5 100644 (file)
@@ -4,7 +4,7 @@
  */
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 static int raid6_has_ssse3(void)
 {
index 6d0708a2c8a4be9f0fe4808cbf47bd132586f267..b0a71b375962c563859d6ead6e194076ce037c6b 100644 (file)
@@ -7,17 +7,8 @@
  * Definitions for RISC-V RAID-6 code
  */
 
-#ifdef __KERNEL__
-#include <asm/vector.h>
-#else
-#define kernel_vector_begin()
-#define kernel_vector_end()
-#include <sys/auxv.h>
-#include <asm/hwcap.h>
-#define has_vector() (getauxval(AT_HWCAP) & COMPAT_HWCAP_ISA_V)
-#endif
-
 #include <linux/raid/pq.h>
+#include <asm/vector.h>
 
 static int rvv_has_vector(void)
 {
index 692fa3a93bf0bf8b956a443e51792f2948074f8c..794d5cfa0306f0380785c2495836a3eb8418dbc8 100644 (file)
@@ -19,7 +19,7 @@
 #ifdef CONFIG_X86_32
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 /* Defined in raid6/mmx.c */
 extern const struct raid6_mmx_constants {
index 2930220249c901e708eea76905fdcceaef29baf1..f9edf8a8d1c4281ae879df1e00e9045aec5f6639 100644 (file)
@@ -13,7 +13,7 @@
  */
 
 #include <linux/raid/pq.h>
-#include "x86.h"
+#include <asm/fpu/api.h>
 
 static const struct raid6_sse_constants {
        u64 x1d[2];
index 1bfb127fbfe81675ccdd656665cbff70a207f7f2..a8e76b1c956ed65f60a1ddc0d9707729f2691096 100644 (file)
 
 #include <altivec.h>
 #include <asm/ppc-opcode.h>
-#ifdef __KERNEL__
 #include <asm/cputable.h>
 #include <asm/switch_to.h>
-#endif
 
 typedef vector unsigned char unative_t;
 #define NSIZE sizeof(unative_t)
@@ -85,13 +83,8 @@ int raid6_have_altivec_vpermxor(void);
 int raid6_have_altivec_vpermxor(void)
 {
        /* Check if arch has both altivec and the vpermxor instructions */
-# ifdef __KERNEL__
        return (cpu_has_feature(CPU_FTR_ALTIVEC_COMP) &&
                cpu_has_feature(CPU_FTR_ARCH_207S));
-# else
-       return 1;
-#endif
-
 }
 #endif
 
diff --git a/lib/raid6/x86.h b/lib/raid6/x86.h
deleted file mode 100644 (file)
index 9a6ff37..0000000
+++ /dev/null
@@ -1,75 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-/* ----------------------------------------------------------------------- *
- *
- *   Copyright 2002-2004 H. Peter Anvin - All Rights Reserved
- *
- * ----------------------------------------------------------------------- */
-
-/*
- * raid6/x86.h
- *
- * Definitions common to x86 and x86-64 RAID-6 code only
- */
-
-#ifndef LINUX_RAID_RAID6X86_H
-#define LINUX_RAID_RAID6X86_H
-
-#if (defined(__i386__) || defined(__x86_64__)) && !defined(__arch_um__)
-
-#ifdef __KERNEL__ /* Real code */
-
-#include <asm/fpu/api.h>
-
-#else /* Dummy code for user space testing */
-
-static inline void kernel_fpu_begin(void)
-{
-}
-
-static inline void kernel_fpu_end(void)
-{
-}
-
-#define __aligned(x) __attribute__((aligned(x)))
-
-#define X86_FEATURE_MMX                (0*32+23) /* Multimedia Extensions */
-#define X86_FEATURE_FXSR       (0*32+24) /* FXSAVE and FXRSTOR instructions
-                                          * (fast save and restore) */
-#define X86_FEATURE_XMM                (0*32+25) /* Streaming SIMD Extensions */
-#define X86_FEATURE_XMM2       (0*32+26) /* Streaming SIMD Extensions-2 */
-#define X86_FEATURE_XMM3       (4*32+ 0) /* "pni" SSE-3 */
-#define X86_FEATURE_SSSE3      (4*32+ 9) /* Supplemental SSE-3 */
-#define X86_FEATURE_AVX        (4*32+28) /* Advanced Vector Extensions */
-#define X86_FEATURE_AVX2        (9*32+ 5) /* AVX2 instructions */
-#define X86_FEATURE_AVX512F     (9*32+16) /* AVX-512 Foundation */
-#define X86_FEATURE_AVX512DQ    (9*32+17) /* AVX-512 DQ (Double/Quad granular)
-                                          * Instructions
-                                          */
-#define X86_FEATURE_AVX512BW    (9*32+30) /* AVX-512 BW (Byte/Word granular)
-                                          * Instructions
-                                          */
-#define X86_FEATURE_AVX512VL    (9*32+31) /* AVX-512 VL (128/256 Vector Length)
-                                          * Extensions
-                                          */
-#define X86_FEATURE_MMXEXT     (1*32+22) /* AMD MMX extensions */
-
-/* Should work well enough on modern CPUs for testing */
-static inline int boot_cpu_has(int flag)
-{
-       u32 eax, ebx, ecx, edx;
-
-       eax = (flag & 0x100) ? 7 :
-               (flag & 0x20) ? 0x80000001 : 1;
-       ecx = 0;
-
-       asm volatile("cpuid"
-                    : "+a" (eax), "=b" (ebx), "=d" (edx), "+c" (ecx));
-
-       return ((flag & 0x100 ? ebx :
-               (flag & 0x80) ? ecx : edx) >> (flag & 31)) & 1;
-}
-
-#endif /* ndef __KERNEL__ */
-
-#endif
-#endif