linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Matt Sealey <matt@genesi-usa.com>
Cc: sfr@canb.auug.org.au, paulus@samba.org, linuxppc-dev@ozlabs.org
Subject: Re: [POWERPC] convert string i/o operations to C
Date: Wed, 20 Sep 2006 09:17:15 +1000	[thread overview]
Message-ID: <1158707835.6002.177.camel@localhost.localdomain> (raw)
In-Reply-To: <45104304.3000205@genesi-usa.com>


> But it couldn't hurt, right? There has to be an application note per-CPU 
> on the correct sequence of operations for such an access (I seem to have 
> collected a directory full for firmware development), it seems a little 
> odd to pick and choose one instruction over another for one thing, and 
> then say you need to do it to support the 601 of all things, and run 
> this code against the G3/G4/G5 which perhaps doesn't care or is more 
> intelligent about it (or is guaranteed to have a more intelligent host 
> bridge at least).
> 
> Maybe I'm talking crap, please say so :D

No, your basic idea is valid. One of the thing we are tackling doing is
to add to our cpu feature "nop'ing out" mecanism something closer to
what x86 does nowadays which is to have alternate sequence of
instructions depending on the CPU.

I just think however that the MMIO loads aren't a very high priority
spot for such an optimisation :) Spinlocks are more interesting and we
are looking into doing something like x86 where you don't need to build
a UP kernel, the SMP kernel will automatically replace the spinlocks
with simple preempt_disable/enable at boot if running on only one CPU.
We could do similar things with some barriers etc...

Ben.

  parent reply	other threads:[~2006-09-19 23:17 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-09-19 12:23 [POWERPC] convert string i/o operations to C Stephen Rothwell
2006-09-19 12:42 ` David Howells
2006-09-19 23:07   ` Stephen Rothwell
2006-09-19 18:29 ` Linas Vepstas
2006-09-19 18:52   ` Kim Phillips
2006-09-19 18:58     ` Matt Sealey
2006-09-19 19:10       ` Segher Boessenkool
2006-09-19 19:20         ` Matt Sealey
2006-09-19 19:47           ` Segher Boessenkool
2006-09-19 23:17           ` Benjamin Herrenschmidt [this message]
2006-09-19 23:14       ` Benjamin Herrenschmidt
2006-09-19 19:04     ` Linas Vepstas
2006-09-19 19:06   ` Segher Boessenkool
2006-09-19 23:12   ` Benjamin Herrenschmidt
2006-09-20  0:03     ` Segher Boessenkool
2006-09-20  0:08       ` Benjamin Herrenschmidt
2006-09-20  0:29         ` Segher Boessenkool
2006-09-20  3:35 ` Stephen Rothwell
2006-09-20  4:38   ` Michael Ellerman

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=1158707835.6002.177.camel@localhost.localdomain \
    --to=benh@kernel.crashing.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=matt@genesi-usa.com \
    --cc=paulus@samba.org \
    --cc=sfr@canb.auug.org.au \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).