From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: linuxppc-dev@ozlabs.org
Cc: Olof Johansson <olof@lixom.net>,
Paul Mackerras <paulus@samba.org>,
Anton Blanchard <anton@samba.org>, Arnd Bergmann <arnd@arndb.de>
Subject: Re: [RFC/PATCH] powerpc: Rewrite IO allocation & mapping on powerpc64
Date: Mon, 14 May 2007 18:03:58 +1000 [thread overview]
Message-ID: <1179129838.32247.84.camel@localhost.localdomain> (raw)
In-Reply-To: <20070514075415.7999BDDEC8@ozlabs.org>
On Mon, 2007-05-14 at 17:53 +1000, Benjamin Herrenschmidt wrote:
> - Handle allocation constraints of PIO for all bridges including
> hot plugged ones within the 2GB space reserved for IO ports,
> so that devices on hotplugged busses will now work with drivers
> that assume IO ports fit in an int.
Olof, Arnd, that should fix your problem but it's not 2.6.22 material.
I'll post a separate patch for 2.6.22 that does a temporary band aid,
basically calling pci_setup_phb_io() + remap_bus_range() rather than
pci_setup_phb_io_dynamic() from the of_device/PCI code and add a
spinlock to reserve_phb_iospace().
I found also a couple of other nits for which I'll do patches for
2.6.22. For example, we don't mask out the "tokens" at the top of IO
addresses in iounmap() which means it probably doesn't work on cell.
Ouch.
There's a number of other issues that my bigger patch fixes for which I
won't bother with a 2.6.22 backport though.
Cheers,
Ben.
next prev parent reply other threads:[~2007-05-14 8:04 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-14 7:53 [RFC/PATCH] powerpc: Rewrite IO allocation & mapping on powerpc64 Benjamin Herrenschmidt
2007-05-14 7:59 ` Benjamin Herrenschmidt
2007-05-14 8:03 ` Benjamin Herrenschmidt [this message]
2007-05-14 9:18 ` Benjamin Herrenschmidt
2007-05-14 13:50 ` Segher Boessenkool
2007-05-14 20:53 ` Benjamin Herrenschmidt
2007-05-15 0:17 ` Paul Mackerras
2007-05-15 4:33 ` Segher Boessenkool
2007-05-15 6:04 ` Benjamin Herrenschmidt
2007-05-14 14:37 ` Arnd Bergmann
2007-05-14 15:04 ` Segher Boessenkool
2007-05-14 23:56 ` Arnd Bergmann
2007-05-15 0:05 ` Olof Johansson
2007-05-14 20:59 ` Benjamin Herrenschmidt
2007-05-14 23:39 ` Arnd Bergmann
2007-05-15 0:06 ` Benjamin Herrenschmidt
2007-05-15 10:56 ` Christoph Hellwig
2007-05-15 20:42 ` Benjamin Herrenschmidt
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=1179129838.32247.84.camel@localhost.localdomain \
--to=benh@kernel.crashing.org \
--cc=anton@samba.org \
--cc=arnd@arndb.de \
--cc=linuxppc-dev@ozlabs.org \
--cc=olof@lixom.net \
--cc=paulus@samba.org \
/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).