Commit 02f39b23 authored Oct 04, 2016 by Andy Lutomirski Committed by Ingo Molnar Oct 07, 2016

x86/crypto, x86/fpu: Remove X86_FEATURE_EAGER_FPU #ifdef from the crc32c code



The crypto code was checking both use_eager_fpu() and
defined(X86_FEATURE_EAGER_FPU).  The latter was nonsensical, so
remove it.  This will avoid breakage when we remove
X86_FEATURE_EAGER_FPU.

Signed-off-by: Andy Lutomirski <luto@kernel.org>
Signed-off-by: Rik van Riel <riel@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Brian Gerst <brgerst@gmail.com>
Cc: Dave Hansen <dave.hansen@linux.intel.com>
Cc: Denys Vlasenko <dvlasenk@redhat.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Oleg Nesterov <oleg@redhat.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Quentin Casasnovas <quentin.casasnovas@oracle.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: pbonzini@redhat.com
Link: http://lkml.kernel.org/r/1475627678-20788-2-git-send-email-riel@redhat.com


Signed-off-by: Ingo Molnar <mingo@kernel.org>

parent 597f03f9

arch/x86/crypto/crc32c-intel_glue.c

+0 −5

Original line number	Diff line number	Diff line
		@@ -58,16 +58,11 @@
		asmlinkage unsigned int crc_pcl(const u8 *buffer, int len,
		unsigned int crc_init);
		static int crc32c_pcl_breakeven = CRC32C_PCL_BREAKEVEN_EAGERFPU;
		#if defined(X86_FEATURE_EAGER_FPU)
		#define set_pcl_breakeven_point() \
		do { \
		if (!use_eager_fpu()) \
		crc32c_pcl_breakeven = CRC32C_PCL_BREAKEVEN_NOEAGERFPU; \
		} while (0)
		#else
		#define set_pcl_breakeven_point() \
		(crc32c_pcl_breakeven = CRC32C_PCL_BREAKEVEN_NOEAGERFPU)
		#endif
		#endif /* CONFIG_X86_64 */

		static u32 crc32c_intel_le_hw_byte(u32 crc, unsigned char const *data, size_t length)