lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 29 Sep 2008 02:07:53 -0700
From:	"H. Peter Anvin" <hpa@...or.com>
To:	Krzysztof Helt <krzysztof.h1@...pl>
CC:	linux-kernel@...r.kernel.org, Ingo Molnar <mingo@...e.hu>,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH] x86: merge winchip-2 and winchip-2a cpu choices

Looks good to me.

Acked-by: H. Peter Anvin <hpa@...or.com>

Krzysztof Helt wrote:
> From: Krzysztof Helt <krzysztof.h1@...pl>
> 
> The Winchip-2 and Winchip-2A cpu choices select the
> same options for kernel and compiler. 
> 
> Merge them to save few bytes and reduce confusion.
> 
> Signed-off-by: Krzysztof Helt <krzysztof.h1@...pl>
> ---
> 
> diff -urp linux-ref/arch/um/defconfig linux-2.6.26/arch/um/defconfig
> --- linux-ref/arch/um/defconfig	2008-09-28 18:58:21.000000000 +0200
> +++ linux-2.6.26/arch/um/defconfig	2008-09-28 18:46:20.000000000 +0200
> @@ -42,7 +42,6 @@ CONFIG_M686=y
>  # CONFIG_MCRUSOE is not set
>  # CONFIG_MEFFICEON is not set
>  # CONFIG_MWINCHIPC6 is not set
> -# CONFIG_MWINCHIP2 is not set
>  # CONFIG_MWINCHIP3D is not set
>  # CONFIG_MGEODEGX1 is not set
>  # CONFIG_MGEODE_LX is not set
> diff -urp linux-ref/arch/x86/Kconfig.cpu linux-2.6.26/arch/x86/Kconfig.cpu
> --- linux-ref/arch/x86/Kconfig.cpu	2008-09-27 23:28:13.000000000 +0200
> +++ linux-2.6.26/arch/x86/Kconfig.cpu	2008-09-28 23:20:00.000000000 +0200
> @@ -38,8 +38,7 @@ config M386
>  	  - "Crusoe" for the Transmeta Crusoe series.
>  	  - "Efficeon" for the Transmeta Efficeon series.
>  	  - "Winchip-C6" for original IDT Winchip.
> -	  - "Winchip-2" for IDT Winchip 2.
> -	  - "Winchip-2A" for IDT Winchips with 3dNow! capabilities.
> +	  - "Winchip-2" for IDT Winchips with 3dNow! capabilities.
>  	  - "GeodeGX1" for Geode GX1 (Cyrix MediaGX).
>  	  - "Geode GX/LX" For AMD Geode GX and LX processors.
>  	  - "CyrixIII/VIA C3" for VIA Cyrix III or VIA C3.
> @@ -194,19 +193,11 @@ config MWINCHIPC6
>  	  treat this chip as a 586TSC with some extended instructions
>  	  and alignment requirements.
>  
> -config MWINCHIP2
> -	bool "Winchip-2"
> -	depends on X86_32
> -	help
> -	  Select this for an IDT Winchip-2.  Linux and GCC
> -	  treat this chip as a 586TSC with some extended instructions
> -	  and alignment requirements.
> -
>  config MWINCHIP3D
> -	bool "Winchip-2A/Winchip-3"
> +	bool "Winchip-2/Winchip-2A/Winchip-3"
>  	depends on X86_32
>  	help
> -	  Select this for an IDT Winchip-2A or 3.  Linux and GCC
> +	  Select this for an IDT Winchip-2, 2A or 3.  Linux and GCC
>  	  treat this chip as a 586TSC with some extended instructions
>  	  and alignment requirements.  Also enable out of order memory
>  	  stores for this CPU, which can increase performance of some
> @@ -318,7 +309,7 @@ config X86_L1_CACHE_SHIFT
>  	int
>  	default "7" if MPENTIUM4 || X86_GENERIC || GENERIC_CPU || MPSC
>  	default "4" if X86_ELAN || M486 || M386 || MGEODEGX1
> -	default "5" if MWINCHIP3D || MWINCHIP2 || MWINCHIPC6 || MCRUSOE || MEFFICEON || MCYRIXIII || MK6 || MPENTIUMIII || MPENTIUMII || M686 || M586MMX || M586TSC || M586 || MVIAC3_2 ||
> MGEODE_LX
> +	default "5" if MWINCHIP3D || MWINCHIPC6 || MCRUSOE || MEFFICEON || MCYRIXIII || MK6 || MPENTIUMIII || MPENTIUMII || M686 || M586MMX || M586TSC || M586 || MVIAC3_2 || MGEODE_LX
>  	default "6" if MK7 || MK8 || MPENTIUMM || MCORE2 || MVIAC7
>  
>  config X86_XADD
> @@ -360,7 +351,7 @@ config X86_POPAD_OK
>  
>  config X86_ALIGNMENT_16
>  	def_bool y
> -	depends on MWINCHIP3D || MWINCHIP2 || MWINCHIPC6 || MCYRIXIII || X86_ELAN || MK6 || M586MMX || M586TSC || M586 || M486 || MVIAC3_2 || MGEODEGX1
> +	depends on MWINCHIP3D || MWINCHIPC6 || MCYRIXIII || X86_ELAN || MK6 || M586MMX || M586TSC || M586 || M486 || MVIAC3_2 || MGEODEGX1
>  
>  config X86_INTEL_USERCOPY
>  	def_bool y
> @@ -368,7 +359,7 @@ config X86_INTEL_USERCOPY
>  
>  config X86_USE_PPRO_CHECKSUM
>  	def_bool y
> -	depends on MWINCHIP3D || MWINCHIP2 || MWINCHIPC6 || MCYRIXIII || MK7 || MK6 || MPENTIUM4 || MPENTIUMM || MPENTIUMIII || MPENTIUMII || M686 || MK8 || MVIAC3_2 || MEFFICEON ||
> MGEODE_LX || MCORE2
> +	depends on MWINCHIP3D || MWINCHIPC6 || MCYRIXIII || MK7 || MK6 || MPENTIUM4 || MPENTIUMM || MPENTIUMIII || MPENTIUMII || M686 || MK8 || MVIAC3_2 || MEFFICEON || MGEODE_LX ||
> MCORE2 
>  config X86_USE_3DNOW
>  	def_bool y
> @@ -376,7 +367,7 @@ config X86_USE_3DNOW
>  
>  config X86_OOSTORE
>  	def_bool y
> -	depends on (MWINCHIP3D || MWINCHIP2 || MWINCHIPC6) && MTRR
> +	depends on (MWINCHIP3D || MWINCHIPC6) && MTRR
>  
>  #
>  # P6_NOPs are a relatively minor optimization that require a family >=
> @@ -396,7 +387,7 @@ config X86_P6_NOP
>  
>  config X86_TSC
>  	def_bool y
> -	depends on ((MWINCHIP3D || MWINCHIP2 || MCRUSOE || MEFFICEON || MCYRIXIII || MK7 || MK6 || MPENTIUM4 || MPENTIUMM || MPENTIUMIII || MPENTIUMII || M686 || M586MMX || M586TSC ||
> MK8 || MVIAC3_2 || MVIAC7 || MGEODEGX1 || MGEODE_LX || MCORE2) && !X86_NUMAQ) || X86_64
> +	depends on ((MWINCHIP3D || MCRUSOE || MEFFICEON || MCYRIXIII || MK7 || MK6 || MPENTIUM4 || MPENTIUMM || MPENTIUMIII || MPENTIUMII || M686 || M586MMX || M586TSC || MK8 || MVIAC3_2
> || MVIAC7 || MGEODEGX1 || MGEODE_LX || MCORE2) && !X86_NUMAQ) || X86_64 
>  config X86_CMPXCHG64
>  	def_bool y
> @@ -417,7 +408,7 @@ config X86_MINIMUM_CPU_FAMILY
>  
>  config X86_DEBUGCTLMSR
>  	def_bool y
> -	depends on !(MK6 || MWINCHIPC6 || MWINCHIP2 || MWINCHIP3D || MCYRIXIII || M586MMX || M586TSC || M586 || M486 || M386)
> +	depends on !(MK6 || MWINCHIPC6 || MWINCHIP3D || MCYRIXIII || M586MMX || M586TSC || M586 || M486 || M386)
>  
>  menuconfig PROCESSOR_SELECT
>  	default y
> diff -urp linux-ref/arch/x86/Makefile_32.cpu linux-2.6.26/arch/x86/Makefile_32.cpu
> --- linux-ref/arch/x86/Makefile_32.cpu	2008-09-27 23:28:13.000000000 +0200
> +++ linux-2.6.26/arch/x86/Makefile_32.cpu	2008-09-28 18:36:55.000000000 +0200
> @@ -28,7 +28,6 @@ cflags-$(CONFIG_MK8)		+= $(call cc-optio
>  cflags-$(CONFIG_MCRUSOE)	+= -march=i686 $(align)-functions=0 $(align)-jumps=0 $(align)-loops=0
>  cflags-$(CONFIG_MEFFICEON)	+= -march=i686 $(call tune,pentium3) $(align)-functions=0 $(align)-jumps=0 $(align)-loops=0
>  cflags-$(CONFIG_MWINCHIPC6)	+= $(call cc-option,-march=winchip-c6,-march=i586)
> -cflags-$(CONFIG_MWINCHIP2)	+= $(call cc-option,-march=winchip2,-march=i586)
>  cflags-$(CONFIG_MWINCHIP3D)	+= $(call cc-option,-march=winchip2,-march=i586)
>  cflags-$(CONFIG_MCYRIXIII)	+= $(call cc-option,-march=c3,-march=i486) $(align)-functions=0 $(align)-jumps=0 $(align)-loops=0
>  cflags-$(CONFIG_MVIAC3_2)	+= $(call cc-option,-march=c3-2,-march=i686)
> diff -urp linux-ref/arch/x86/configs/i386_defconfig linux-2.6.26/arch/x86/configs/i386_defconfig
> --- linux-ref/arch/x86/configs/i386_defconfig	2008-09-27 23:28:14.000000000 +0200
> +++ linux-2.6.26/arch/x86/configs/i386_defconfig	2008-09-28 18:38:16.000000000 +0200
> @@ -213,7 +213,6 @@ CONFIG_M686=y
>  # CONFIG_MCRUSOE is not set
>  # CONFIG_MEFFICEON is not set
>  # CONFIG_MWINCHIPC6 is not set
> -# CONFIG_MWINCHIP2 is not set
>  # CONFIG_MWINCHIP3D is not set
>  # CONFIG_MGEODEGX1 is not set
>  # CONFIG_MGEODE_LX is not set
> diff -urp linux-ref/arch/x86/configs/x86_64_defconfig linux-2.6.26/arch/x86/configs/x86_64_defconfig
> --- linux-ref/arch/x86/configs/x86_64_defconfig	2008-09-27 23:28:14.000000000 +0200
> +++ linux-2.6.26/arch/x86/configs/x86_64_defconfig	2008-09-28 18:38:23.000000000 +0200
> @@ -210,7 +210,6 @@ CONFIG_X86_PC=y
>  # CONFIG_MCRUSOE is not set
>  # CONFIG_MEFFICEON is not set
>  # CONFIG_MWINCHIPC6 is not set
> -# CONFIG_MWINCHIP2 is not set
>  # CONFIG_MWINCHIP3D is not set
>  # CONFIG_MGEODEGX1 is not set
>  # CONFIG_MGEODE_LX is not set
> diff -urp linux-ref/include/asm-x86/module.h linux-2.6.26/include/asm-x86/module.h
> --- linux-ref/include/asm-x86/module.h	2008-09-28 18:54:26.000000000 +0200
> +++ linux-2.6.26/include/asm-x86/module.h	2008-09-28 18:43:33.000000000 +0200
> @@ -52,8 +52,6 @@ struct mod_arch_specific {};
>  #define MODULE_PROC_FAMILY "EFFICEON "
>  #elif defined CONFIG_MWINCHIPC6
>  #define MODULE_PROC_FAMILY "WINCHIPC6 "
> -#elif defined CONFIG_MWINCHIP2
> -#define MODULE_PROC_FAMILY "WINCHIP2 "
>  #elif defined CONFIG_MWINCHIP3D
>  #define MODULE_PROC_FAMILY "WINCHIP3D "
>  #elif defined CONFIG_MCYRIXIII
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists