[PATCH 002 of 6] md: RAID6: clean up CPUID and FPU enter/exit code
From: "H. Peter Anvin" <[EMAIL PROTECTED]> - Use kernel_fpu_begin() and kernel_fpu_end() - Use boot_cpu_has() for feature testing even in userspace Signed-off-by: H. Peter Anvin <[EMAIL PROTECTED]> Signed-off-by: Neil Brown <[EMAIL PROTECTED]> ### Diffstat output ./drivers/md/raid6mmx.c | 16 --- ./drivers/md/raid6sse1.c | 17 --- ./drivers/md/raid6sse2.c | 22 +--- ./drivers/md/raid6x86.h | 218 +++ 4 files changed, 32 insertions(+), 241 deletions(-) diff .prev/drivers/md/raid6mmx.c ./drivers/md/raid6mmx.c --- .prev/drivers/md/raid6mmx.c 2007-02-20 17:11:51.0 +1100 +++ ./drivers/md/raid6mmx.c 2007-02-20 17:11:51.0 +1100 @@ -30,14 +30,8 @@ const struct raid6_mmx_constants { static int raid6_have_mmx(void) { -#ifdef __KERNEL__ /* Not really "boot_cpu" but "all_cpus" */ return boot_cpu_has(X86_FEATURE_MMX); -#else - /* User space test code */ - u32 features = cpuid_features(); - return ( (features & (1<<23)) == (1<<23) ); -#endif } /* @@ -48,13 +42,12 @@ static void raid6_mmx1_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - raid6_before_mmx(); + kernel_fpu_begin(); asm volatile("movq %0,%%mm0" : : "m" (raid6_mmx_constants.x1d)); asm volatile("pxor %mm5,%mm5"); /* Zero temp */ @@ -78,7 +71,7 @@ static void raid6_mmx1_gen_syndrome(int asm volatile("pxor %mm4,%mm4"); } - raid6_after_mmx(); + kernel_fpu_end(); } const struct raid6_calls raid6_mmxx1 = { @@ -96,13 +89,12 @@ static void raid6_mmx2_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - raid6_before_mmx(); + kernel_fpu_begin(); asm volatile("movq %0,%%mm0" : : "m" (raid6_mmx_constants.x1d)); asm volatile("pxor %mm5,%mm5"); /* Zero temp */ @@ -137,7 +129,7 @@ static void raid6_mmx2_gen_syndrome(int asm volatile("movq %%mm6,%0" : "=m" (q[d+8])); } - raid6_after_mmx(); + kernel_fpu_end(); } const struct raid6_calls raid6_mmxx2 = { diff .prev/drivers/md/raid6sse1.c ./drivers/md/raid6sse1.c --- .prev/drivers/md/raid6sse1.c2007-02-20 17:11:51.0 +1100 +++ ./drivers/md/raid6sse1.c2007-02-20 17:11:51.0 +1100 @@ -33,16 +33,10 @@ extern const struct raid6_mmx_constants static int raid6_have_sse1_or_mmxext(void) { -#ifdef __KERNEL__ /* Not really boot_cpu but "all_cpus" */ return boot_cpu_has(X86_FEATURE_MMX) && (boot_cpu_has(X86_FEATURE_XMM) || boot_cpu_has(X86_FEATURE_MMXEXT)); -#else - /* User space test code - this incorrectly breaks on some Athlons */ - u32 features = cpuid_features(); - return ( (features & (5<<23)) == (5<<23) ); -#endif } /* @@ -53,14 +47,12 @@ static void raid6_sse11_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - /* This is really MMX code, not SSE */ - raid6_before_mmx(); + kernel_fpu_begin(); asm volatile("movq %0,%%mm0" : : "m" (raid6_mmx_constants.x1d)); asm volatile("pxor %mm5,%mm5"); /* Zero temp */ @@ -94,8 +86,8 @@ static void raid6_sse11_gen_syndrome(int asm volatile("movntq %%mm4,%0" : "=m" (q[d])); } - raid6_after_mmx(); asm volatile("sfence" : : : "memory"); + kernel_fpu_end(); } const struct raid6_calls raid6_sse1x1 = { @@ -113,13 +105,12 @@ static void raid6_sse12_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - raid6_before_mmx(); + kernel_fpu_begin(); asm volatile("movq %0,%%mm0" : : "m" (raid6_mmx_constants.x1d)); asm volatile("pxor %mm5,%mm5"); /* Zero temp */ @@ -157,8 +148,8 @@ static void raid6_sse12_gen_syndrome(int asm volatile("movntq %%mm6,%0" : "=m" (q[d+8])); } - raid6_after_mmx(); asm volatile("sfence" : :: "memory"); + kernel_fpu_end(); } const struct raid6_calls raid6_sse1x2 = { diff .prev/drivers/md/raid6sse2.c ./drivers/md/raid6sse2.c --- .prev/drivers/md/raid6sse2.c
[PATCH 002 of 6] md: RAID6: clean up CPUID and FPU enter/exit code
From: H. Peter Anvin [EMAIL PROTECTED] - Use kernel_fpu_begin() and kernel_fpu_end() - Use boot_cpu_has() for feature testing even in userspace Signed-off-by: H. Peter Anvin [EMAIL PROTECTED] Signed-off-by: Neil Brown [EMAIL PROTECTED] ### Diffstat output ./drivers/md/raid6mmx.c | 16 --- ./drivers/md/raid6sse1.c | 17 --- ./drivers/md/raid6sse2.c | 22 +--- ./drivers/md/raid6x86.h | 218 +++ 4 files changed, 32 insertions(+), 241 deletions(-) diff .prev/drivers/md/raid6mmx.c ./drivers/md/raid6mmx.c --- .prev/drivers/md/raid6mmx.c 2007-02-20 17:11:51.0 +1100 +++ ./drivers/md/raid6mmx.c 2007-02-20 17:11:51.0 +1100 @@ -30,14 +30,8 @@ const struct raid6_mmx_constants { static int raid6_have_mmx(void) { -#ifdef __KERNEL__ /* Not really boot_cpu but all_cpus */ return boot_cpu_has(X86_FEATURE_MMX); -#else - /* User space test code */ - u32 features = cpuid_features(); - return ( (features (123)) == (123) ); -#endif } /* @@ -48,13 +42,12 @@ static void raid6_mmx1_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - raid6_before_mmx(sa); + kernel_fpu_begin(); asm volatile(movq %0,%%mm0 : : m (raid6_mmx_constants.x1d)); asm volatile(pxor %mm5,%mm5); /* Zero temp */ @@ -78,7 +71,7 @@ static void raid6_mmx1_gen_syndrome(int asm volatile(pxor %mm4,%mm4); } - raid6_after_mmx(sa); + kernel_fpu_end(); } const struct raid6_calls raid6_mmxx1 = { @@ -96,13 +89,12 @@ static void raid6_mmx2_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - raid6_before_mmx(sa); + kernel_fpu_begin(); asm volatile(movq %0,%%mm0 : : m (raid6_mmx_constants.x1d)); asm volatile(pxor %mm5,%mm5); /* Zero temp */ @@ -137,7 +129,7 @@ static void raid6_mmx2_gen_syndrome(int asm volatile(movq %%mm6,%0 : =m (q[d+8])); } - raid6_after_mmx(sa); + kernel_fpu_end(); } const struct raid6_calls raid6_mmxx2 = { diff .prev/drivers/md/raid6sse1.c ./drivers/md/raid6sse1.c --- .prev/drivers/md/raid6sse1.c2007-02-20 17:11:51.0 +1100 +++ ./drivers/md/raid6sse1.c2007-02-20 17:11:51.0 +1100 @@ -33,16 +33,10 @@ extern const struct raid6_mmx_constants static int raid6_have_sse1_or_mmxext(void) { -#ifdef __KERNEL__ /* Not really boot_cpu but all_cpus */ return boot_cpu_has(X86_FEATURE_MMX) (boot_cpu_has(X86_FEATURE_XMM) || boot_cpu_has(X86_FEATURE_MMXEXT)); -#else - /* User space test code - this incorrectly breaks on some Athlons */ - u32 features = cpuid_features(); - return ( (features (523)) == (523) ); -#endif } /* @@ -53,14 +47,12 @@ static void raid6_sse11_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - /* This is really MMX code, not SSE */ - raid6_before_mmx(sa); + kernel_fpu_begin(); asm volatile(movq %0,%%mm0 : : m (raid6_mmx_constants.x1d)); asm volatile(pxor %mm5,%mm5); /* Zero temp */ @@ -94,8 +86,8 @@ static void raid6_sse11_gen_syndrome(int asm volatile(movntq %%mm4,%0 : =m (q[d])); } - raid6_after_mmx(sa); asm volatile(sfence : : : memory); + kernel_fpu_end(); } const struct raid6_calls raid6_sse1x1 = { @@ -113,13 +105,12 @@ static void raid6_sse12_gen_syndrome(int u8 **dptr = (u8 **)ptrs; u8 *p, *q; int d, z, z0; - raid6_mmx_save_t sa; z0 = disks - 3; /* Highest data disk */ p = dptr[z0+1]; /* XOR parity */ q = dptr[z0+2]; /* RS syndrome */ - raid6_before_mmx(sa); + kernel_fpu_begin(); asm volatile(movq %0,%%mm0 : : m (raid6_mmx_constants.x1d)); asm volatile(pxor %mm5,%mm5); /* Zero temp */ @@ -157,8 +148,8 @@ static void raid6_sse12_gen_syndrome(int asm volatile(movntq %%mm6,%0 : =m (q[d+8])); } - raid6_after_mmx(sa); asm volatile(sfence : :: memory); + kernel_fpu_end(); } const struct raid6_calls raid6_sse1x2 = { diff .prev/drivers/md/raid6sse2.c ./drivers/md/raid6sse2.c --- .prev/drivers/md/raid6sse2.c2007-02-20 17:11:51.0 +1100 +++