public inbox for linuxppc-dev@ozlabs.org 
 help / color / mirror / Atom feed
From: Sven Luther <sven.luther@wanadoo•fr>
To: Benjamin Herrenschmidt <benh@kernel•crashing.org>
Cc: akpm@osdl•org, Olaf Hering <olaf@aepfle•de>,
	Sven Luther <sl@bplan-gmbh•de>,
	tilmann@bitterberg•de, linuxppc-dev@ozlabs•org
Subject: Re: [PATCH] enable RTAS /proc for PowerPC/CHRP platform
Date: Fri, 20 Oct 2006 08:24:31 +0200	[thread overview]
Message-ID: <20061020062431.GB14390@powerlinux.fr> (raw)
In-Reply-To: <1161323797.10524.150.camel@localhost.localdomain>

On Fri, Oct 20, 2006 at 03:56:37PM +1000, Benjamin Herrenschmidt wrote:
> On Fri, 2006-10-20 at 07:44 +0200, Olaf Hering wrote:
> > On Fri, Oct 20, Benjamin Herrenschmidt wrote:
> > 
> > > On Thu, 2006-10-19 at 09:03 +0200, Olaf Hering wrote:
> > > > On Thu, Oct 19, Benjamin Herrenschmidt wrote:
> > > > 
> > > > > On Wed, 2006-10-18 at 09:38 +0200, Olaf Hering wrote:
> > > > > > On Wed, Oct 18, Michael Ellerman wrote:
> > > > > > 
> > > > > > > But it's called '/proc/ppc64' right now on lots of machines, so you
> > > > > > > can't go changing it.
> > > > > > 
> > > > > > if test -d /proc/ppc64 .. is a quick way to check wether the system is
> > > > > > 64bit or not. Similar to if test -d /proc/iSeries ..
> > > > > 
> > > > > That's really ugly ! You really do that ? Gack...
> > > > 
> > > > What way should I use?
> > > 
> > > uname ? /proc/cpuinfo ? Whatever but not that.
> > 
> > uname can be faked with powerpc32 or setarch. grep POWER /proc/cpuinfo
> > might be an option. I think its only used in rpm preinstall scripts to
> > reject 64bit kernel on 32bit systems or the other way around.
> 
> grep POWER won't do good on 970 among others.
> 
> uname can be faked but why do you care ?
> 
> In fact, is somebody is faking it, maybe it's because they -want- your
> scripts to think they are running on a 32 bits box....
> 
> I still think uname is the way to go.
>  

Ben, would it make sense to add two informations to /proc/cpuinfo ?
It would be great to have a field there which will allow to check for the
kernel flavour to run, namely if the kernel/processor/whatever is running in
32bit or 64bit mode. The second field would be a generalized machine field,
or plateform or whatever, which would easily allow to differentiate between an
apple machine, a ibm chrp, a pegasos or other genesi product, etc. Without
necessarily having to maintain a huge amount of userlanf mappings from all
those different machine: fields, which all have CHRP, true, but no finer
grained informations, while we do have it in the kernel.

Friendly,

Sven Luther

  reply	other threads:[~2006-10-20  6:28 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-17 11:29 [PATCH] enable RTAS /proc for PowerPC/CHRP platform Nicolas DET
2006-10-17 13:22 ` Christoph Hellwig
2006-10-17 22:22   ` Arnd Bergmann
2006-10-18 22:39     ` Benjamin Herrenschmidt
2006-10-20  8:13     ` Nicolas DET
2006-10-18  5:51   ` Nicolas DET
2006-10-18  6:05     ` Sven Luther
2006-10-18  6:15     ` Michael Ellerman
2006-10-18  6:34       ` Nicolas DET
2006-10-18  7:38       ` Olaf Hering
2006-10-18 22:38         ` Benjamin Herrenschmidt
2006-10-19  7:03           ` Olaf Hering
2006-10-19 23:35             ` Benjamin Herrenschmidt
2006-10-20  5:44               ` Olaf Hering
2006-10-20  5:56                 ` Benjamin Herrenschmidt
2006-10-20  6:24                   ` Sven Luther [this message]
2006-10-20  6:44                     ` Olaf Hering
2006-10-20  6:58                       ` Sven Luther
2006-10-20  7:12                         ` Benjamin Herrenschmidt
2006-10-20  7:36                           ` Sven Luther
2006-10-20  8:16                             ` Benjamin Herrenschmidt
2006-10-20  7:20                         ` Olaf Hering
2006-10-20  7:37                           ` Sven Luther
2006-10-20  7:49                             ` Olaf Hering
2006-10-20  8:12                             ` Segher Boessenkool
2006-10-20  8:52                               ` Sven Luther
2006-10-20 10:00                                 ` Olaf Hering
2006-10-20 13:59                                   ` Segher Boessenkool
2006-10-20  7:11                       ` Benjamin Herrenschmidt
2006-10-20  7:14                         ` Olaf Hering
2006-10-20  7:36                           ` Benjamin Herrenschmidt
2006-10-20 10:01                       ` Paul Mackerras

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=20061020062431.GB14390@powerlinux.fr \
    --to=sven.luther@wanadoo$(echo .)fr \
    --cc=akpm@osdl$(echo .)org \
    --cc=benh@kernel$(echo .)crashing.org \
    --cc=linuxppc-dev@ozlabs$(echo .)org \
    --cc=olaf@aepfle$(echo .)de \
    --cc=sl@bplan-gmbh$(echo .)de \
    --cc=tilmann@bitterberg$(echo .)de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox