linux-kernel - Re: [PATCH 6/7] openrisc: add SMP and NR

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]

Message-ID: <57DFF6B2.4020605@southpole.se>
Date:   Mon, 19 Sep 2016 16:31:14 +0200
From:   Jonas Bonn <jonas@...thpole.se>
To:     Stafford Horne <shorne@...il.com>,
        Stefan Kristiansson <stefan.kristiansson@...nalahti.fi>,
        Andrew Morton <akpm@...ux-foundation.org>
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [PATCH 6/7] openrisc: add SMP and NR_CPUS Kconfig options

On 09/16/2016 04:43 PM, Stafford Horne wrote:
> From: Stefan Kristiansson <stefan.kristiansson@...nalahti.fi>

OpenRISC was not an SMP architecture last I looked... did the relevant 
spec updates get made?  If not, NAK.

The issue is, the OpenRISC architecture shouldn't be a moving target 
defined by what the kernel supports... there's a spec that the kernel 
conforms to.  This is important for the (few) real users of the 
architecture.

/Jonas

> Signed-off-by: Stefan Kristiansson <stefan.kristiansson@...nalahti.fi>
> Signed-off-by: Stafford Horne <shorne@...il.com>
> ---
>   arch/openrisc/Kconfig | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
>
> diff --git a/arch/openrisc/Kconfig b/arch/openrisc/Kconfig
> index 489e7f9..2bcf8c3 100644
> --- a/arch/openrisc/Kconfig
> +++ b/arch/openrisc/Kconfig
> @@ -98,6 +98,20 @@ config OPENRISC_HAVE_INST_DIV
>   	  Select this if your implementation has a hardware divide instruction
>   endmenu
>   
> +config NR_CPUS
> +	int "Maximum number of CPUs (2-32)"
> +	range 2 32
> +	depends on SMP
> +	default "2"
> +
> +config SMP
> +	bool "Symmetric Multi-Processing support"
> +	help
> +	  This enables support for systems with more than one CPU. If you have
> +	  a system with only one CPU, say N. If you have a system with more
> +	  than one CPU, say Y.
> +
> +	  If you don't know what to do here, say N.
>   
>   source kernel/Kconfig.hz
>   source kernel/Kconfig.preempt