All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonas Bonn <jonas@southpole.se>
To: Stafford Horne <shorne@gmail.com>,
	Stefan Kristiansson <stefan.kristiansson@saunalahti.fi>,
	Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH 6/7] openrisc: add SMP and NR_CPUS Kconfig options
Date: Mon, 19 Sep 2016 16:31:14 +0200	[thread overview]
Message-ID: <57DFF6B2.4020605@southpole.se> (raw)
In-Reply-To: <1474036985-20981-7-git-send-email-shorne@gmail.com>

On 09/16/2016 04:43 PM, Stafford Horne wrote:
> From: Stefan Kristiansson <stefan.kristiansson@saunalahti.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@saunalahti.fi>
> Signed-off-by: Stafford Horne <shorne@gmail.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

  reply	other threads:[~2016-09-19 14:31 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-16 14:42 [PATCH 0/7] openrisc: Misc fixes from backlog Stafford Horne
2016-09-16 14:42 ` [PATCH 1/7] Apply transparent_union attribute to union semun Stafford Horne
2016-09-16 15:57   ` kbuild test robot
2016-09-16 23:37   ` kbuild test robot
2016-09-17  0:06     ` Stafford Horne
2016-09-17  0:06       ` [OpenRISC] " Stafford Horne
2016-09-19 14:26   ` Jonas Bonn
2016-09-19 14:47     ` Stafford Horne
2016-09-16 14:43 ` [PATCH 2/7] openrisc: fix PTRS_PER_PGD define Stafford Horne
2016-09-19 14:27   ` Jonas Bonn
2016-09-16 14:43 ` [PATCH 3/7] openrisc: restore call-saved regs on sigreturn Stafford Horne
2016-09-19 14:28   ` Jonas Bonn
2016-09-19 14:50     ` Stafford Horne
2016-09-27 13:54       ` Stafford Horne
2016-09-16 14:43 ` [PATCH 4/7] openrisc: Add thread-local storage (TLS) support Stafford Horne
2016-09-19 14:25   ` Jonas Bonn
2016-09-19 14:43     ` Stafford Horne
2016-09-16 14:43 ` [PATCH 5/7] openrisc: Support both old (or32) and new (or1k) toolchain Stafford Horne
2016-09-18 15:26   ` Guenter Roeck
2016-09-19  6:02     ` Stafford Horne
2016-09-19  7:18       ` Guenter Roeck
2016-09-19  9:11         ` Stafford Horne
2016-09-19  9:11           ` [OpenRISC] " Stafford Horne
2016-09-19 13:17           ` Guenter Roeck
2016-09-19 13:17             ` [OpenRISC] " Guenter Roeck
2016-09-19 14:04             ` Stafford Horne
2016-09-19 14:04               ` [OpenRISC] " Stafford Horne
2016-09-19 14:35               ` Jonas Bonn
2016-09-19 14:35                 ` [OpenRISC] " Jonas Bonn
2016-09-19 14:55                 ` Guenter Roeck
2016-09-19 14:55                   ` [OpenRISC] " Guenter Roeck
2016-09-19 14:58                 ` Stefan Wallentowitz
2016-09-19 15:36                   ` Jeremy Bennett
2016-09-19 16:09                     ` Stafford Horne
2016-09-19 16:27                       ` Jeremy Bennett
2016-09-19 17:15                         ` Stefan Wallentowitz
2016-09-19 17:49                           ` Jeremy Bennett
2016-09-19 14:59                 ` Stefan Wallentowitz
2016-09-19 15:16                 ` Stafford Horne
2016-09-19 15:16                   ` [OpenRISC] " Stafford Horne
2016-09-19 14:39               ` Guenter Roeck
2016-09-19 14:39                 ` [OpenRISC] " Guenter Roeck
2016-09-20 10:01                 ` Stafford Horne
2016-09-20 10:01                   ` [OpenRISC] " Stafford Horne
2016-09-16 14:43 ` [PATCH 6/7] openrisc: add SMP and NR_CPUS Kconfig options Stafford Horne
2016-09-19 14:31   ` Jonas Bonn [this message]
2016-09-19 14:54     ` Stafford Horne
2016-09-16 14:43 ` [PATCH 7/7] openrisc: remove the redundant of_platform_populate Stafford Horne
2016-09-19 14:32   ` Jonas Bonn
2016-09-19 14:58     ` Stafford Horne
2016-09-19 16:14     ` Rob Herring
2016-09-16 14:51 ` [PATCH 0/7] openrisc: Misc fixes from backlog Stafford Horne
2016-09-16 14:51   ` [OpenRISC] " Stafford Horne

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=57DFF6B2.4020605@southpole.se \
    --to=jonas@southpole.se \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=shorne@gmail.com \
    --cc=stefan.kristiansson@saunalahti.fi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.