All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: Lee Jones <lee.jones@linaro.org>
Cc: Nicolas Pitre <nicolas.pitre@linaro.org>,
	ext Nishanth Menon <nm@ti.com>,
	ext Tony Lindgren <tony@atomide.com>, Greg KH <greg@kroah.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	Ambresh <a0393775@ti.com>,
	Saravana Kannan <skannan@codeaurora.org>,
	Andrei Warkentin <andreiw@motorola.com>,
	Rabin VINCENT <rabin.vincent@stericsson.com>,
	Russell King <linux@arm.linux.org.uk>,
	Jonas ABERG <jonas.aberg@stericsson.com>,
	ext Kevin Hilman <khilman@deeprootsystems.com>,
	Peter De-Schrijver <Peter.De-Schrijver@nokia.com>,
	David Brown <davidb@codeaurora.org>,
	Maxime Coquelin <maxime.coquelin-nonst@stericsson.com>,
	"linux-arm-msm@vger.kernel.org" <linux-arm-msm@vger.kernel.org>,
	Loic PALLARDY <loic.pallardy@stericsson.com>,
	Eduardo Valentin <eduardo.valentin@nokia.com>,
	maxime_coquelin@yahoo.fr, Ryan Mallon <ryan@bluewatersys.com>,
	Linux-OMAP <linux-omap@vger.kernel.org>linux-ar
Subject: Re: [RFC PATCHv2 1/2] Export SoC info through sysfs
Date: Fri, 8 Apr 2011 17:02:17 +0200	[thread overview]
Message-ID: <201104081702.17409.arnd@arndb.de> (raw)
In-Reply-To: <4D9EBC2C.2050809@linaro.org>

On Friday 08 April 2011, Lee Jones wrote:
> On 08/04/11 04:35, Arnd Bergmann wrote:
> > On Friday 08 April 2011, Greg KH wrote:
> >> Symlinks are a requirement as multiple cpus can be attached to a single
> >> SoC.
> >>
> >> What about multiple cpus that are attached to multiple SoCs?  Why even
> >> try to describe this relationship, what would userspace get out of this
> >> information?
> > 
> > The only one I can think of is node affinity. I've worked with a system
> > (IBM QS2x Cell blade) that had two SoCs with multiple CPUs each. There was
> > a significant performance penalty when talking to devices on the remote
> > SoC. In that case, we used the NUMA node information in sys to deal
> > with it, but we might not want to use the NUMA infrastructure on systems
> > that only have RAM on one node.
> 
> I'm struggling to see a scenario where we'd have multiple SoCs on a
> single device and only one filesystem. ux500 sub-arch code is yet to
> facilitate SoC counting functionality. If were are to implement per-SoC
> directories would this need to be added?

How can you say we won't need it when I've just given you a specific
example of a widely shipping machine?

It's not important that the kernel internal code supports multiple SoCs as
long as there is no in-tree user for this. It can always be added at a later
point.

What is important is that the user interface accomodates for it. If you
name the device simply /sys/devices/soc/, any user program looking at
that will have to be modified when we have more than one. We cannot
allow that to happen.

	Arnd

WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [RFC PATCHv2 1/2] Export SoC info through sysfs
Date: Fri, 8 Apr 2011 17:02:17 +0200	[thread overview]
Message-ID: <201104081702.17409.arnd@arndb.de> (raw)
In-Reply-To: <4D9EBC2C.2050809@linaro.org>

On Friday 08 April 2011, Lee Jones wrote:
> On 08/04/11 04:35, Arnd Bergmann wrote:
> > On Friday 08 April 2011, Greg KH wrote:
> >> Symlinks are a requirement as multiple cpus can be attached to a single
> >> SoC.
> >>
> >> What about multiple cpus that are attached to multiple SoCs?  Why even
> >> try to describe this relationship, what would userspace get out of this
> >> information?
> > 
> > The only one I can think of is node affinity. I've worked with a system
> > (IBM QS2x Cell blade) that had two SoCs with multiple CPUs each. There was
> > a significant performance penalty when talking to devices on the remote
> > SoC. In that case, we used the NUMA node information in sys to deal
> > with it, but we might not want to use the NUMA infrastructure on systems
> > that only have RAM on one node.
> 
> I'm struggling to see a scenario where we'd have multiple SoCs on a
> single device and only one filesystem. ux500 sub-arch code is yet to
> facilitate SoC counting functionality. If were are to implement per-SoC
> directories would this need to be added?

How can you say we won't need it when I've just given you a specific
example of a widely shipping machine?

It's not important that the kernel internal code supports multiple SoCs as
long as there is no in-tree user for this. It can always be added at a later
point.

What is important is that the user interface accomodates for it. If you
name the device simply /sys/devices/soc/, any user program looking at
that will have to be modified when we have more than one. We cannot
allow that to happen.

	Arnd

  reply	other threads:[~2011-04-08 15:02 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-11 12:35 [RFC PATCHv2 0/2] Export SoC info through sysfs Maxime Coquelin
2011-03-11 12:35 ` Maxime Coquelin
2011-03-11 12:35 ` Maxime Coquelin
2011-03-11 12:35 ` [RFC PATCHv2 1/2] " Maxime Coquelin
2011-03-11 12:35   ` Maxime Coquelin
2011-03-11 12:35   ` Maxime Coquelin
2011-03-11 13:50   ` Eduardo Valentin
2011-03-11 13:50     ` Eduardo Valentin
2011-03-11 15:40     ` Maxime Coquelin
2011-03-11 15:40       ` Maxime Coquelin
2011-03-11 17:35       ` Eduardo Valentin
2011-03-11 17:35         ` Eduardo Valentin
2011-03-11 14:32   ` Arnd Bergmann
2011-03-11 14:32     ` Arnd Bergmann
2011-03-11 14:31     ` Eduardo Valentin
2011-03-11 14:31       ` Eduardo Valentin
2011-03-11 15:40     ` Ben Dooks
2011-03-11 15:40       ` Ben Dooks
2011-03-11 17:38       ` Eduardo Valentin
2011-03-11 17:38         ` Eduardo Valentin
2011-03-11 15:52     ` Greg KH
2011-03-11 15:52       ` Greg KH
2011-03-11 17:58       ` Eduardo Valentin
2011-03-11 17:58         ` Eduardo Valentin
2011-03-11 19:33         ` Greg KH
2011-03-11 19:33           ` Greg KH
2011-03-11 21:42           ` Arnd Bergmann
2011-03-11 21:42             ` Arnd Bergmann
2011-03-11 22:03             ` Greg KH
2011-03-11 22:03               ` Greg KH
2011-03-11 22:13               ` Arnd Bergmann
2011-03-11 22:13                 ` Arnd Bergmann
2011-04-07 16:24                 ` Lee Jones
2011-04-07 16:24                   ` Lee Jones
2011-04-07 21:29                   ` Arnd Bergmann
2011-04-07 21:29                     ` Arnd Bergmann
2011-04-07 21:46                     ` Ryan Mallon
2011-04-07 21:46                       ` Ryan Mallon
2011-04-07 22:01                       ` Nicolas Pitre
2011-04-07 22:01                         ` Nicolas Pitre
2011-04-07 22:07                         ` Ryan Mallon
2011-04-07 22:07                           ` Ryan Mallon
2011-04-07 22:45                           ` Nicolas Pitre
2011-04-07 22:45                             ` Nicolas Pitre
2011-04-07 22:56                             ` Ryan Mallon
2011-04-07 22:56                               ` Ryan Mallon
2011-04-07 23:19                               ` Arnd Bergmann
2011-04-07 23:19                                 ` Arnd Bergmann
2011-04-07 23:29                                 ` Greg KH
2011-04-07 23:29                                   ` Greg KH
2011-04-08  3:35                                   ` Arnd Bergmann
2011-04-08  3:35                                     ` Arnd Bergmann
2011-04-08  7:41                                     ` Lee Jones
2011-04-08  7:41                                       ` Lee Jones
2011-04-08 15:02                                       ` Arnd Bergmann [this message]
2011-04-08 15:02                                         ` Arnd Bergmann
2011-04-08 15:43                                         ` Lee Jones
2011-04-08 15:43                                           ` Lee Jones
2011-04-08 20:22                                           ` Arnd Bergmann
2011-04-08 20:22                                             ` Arnd Bergmann
2011-03-11 12:35 ` [RFC PATCHv2 2/2] ux500: Export U8500 " Maxime Coquelin
2011-03-11 12:35   ` Maxime Coquelin
2011-03-11 12:35   ` Maxime Coquelin
2011-03-11 14:11   ` Jean-Christophe PLAGNIOL-VILLARD
2011-03-11 14:11     ` Jean-Christophe PLAGNIOL-VILLARD
2011-03-11 15:20     ` Linus Walleij
2011-03-11 15:20       ` Linus Walleij
2011-03-11 17:24       ` Jean-Christophe PLAGNIOL-VILLARD
2011-03-11 17:24         ` Jean-Christophe PLAGNIOL-VILLARD

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=201104081702.17409.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=Peter.De-Schrijver@nokia.com \
    --cc=a0393775@ti.com \
    --cc=andreiw@motorola.com \
    --cc=davidb@codeaurora.org \
    --cc=eduardo.valentin@nokia.com \
    --cc=greg@kroah.com \
    --cc=jonas.aberg@stericsson.com \
    --cc=khilman@deeprootsystems.com \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=loic.pallardy@stericsson.com \
    --cc=maxime.coquelin-nonst@stericsson.com \
    --cc=maxime_coquelin@yahoo.fr \
    --cc=nicolas.pitre@linaro.org \
    --cc=nm@ti.com \
    --cc=rabin.vincent@stericsson.com \
    --cc=ryan@bluewatersys.com \
    --cc=skannan@codeaurora.org \
    --cc=tony@atomide.com \
    /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.