public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: Kevin Hilman <khilman@deeprootsystems.com>
To: "Woodruff, Richard" <r-woodruff2@ti.com>
Cc: Tony Lindgren <tony@atomide.com>,
	"Shilimkar, Santosh" <santosh.shilimkar@ti.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"linux-arm-kernel@lists.arm.linux.org.uk"
	<linux-arm-kernel@lists.arm.linux.org.uk>
Subject: Re: Reclaim address space on omaps, Tony on vacation in July
Date: Fri, 26 Jun 2009 14:21:19 -0700	[thread overview]
Message-ID: <87prcqbsj4.fsf@deeprootsystems.com> (raw)
In-Reply-To: <13B9B4C6EF24D648824FF11BE8967162038E4C7D46@dlee02.ent.ti.com> (Richard Woodruff's message of "Fri\, 26 Jun 2009 13\:14\:03 -0500")

"Woodruff, Richard" <r-woodruff2@ti.com> writes:

>> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
>> owner@vger.kernel.org] On Behalf Of Tony Lindgren
>
>> > > Last week I posted some more omap io.h clean-up patches to
>> > > the linux-omap
>> > > list [1], and started looking at what it would take to
>> > > reclaim lost address
>> > > space on top of the io.h clean-up patches.
>> > >
>> > > Looks like we should be able to reclaim about 454MB of the 640MB
>> > > of the lost IO address space by doing the following:
>
> Using section entries to cover ranges provides superior performance.  1 TLB will cover a whole series of accesses when doing things like servicing an interrupt.  Using fewer TLBs to cover system accesses leaves more in tact for application usage.  I've not see recent data but long back TLB optimization provided sizeable performance boost for many applications.
>
> Additionally debug is simplified by having the simple static map around.
>
> Russell recently pointed out and David showed an example of how ioremap() can take an arch specific function which can return virtual address from already defined static ranges.  I do think this is a nice clean up and optimization (to stop some double mapping which now happens with ioremap).
>
> http://www.mail-archive.com/linux-omap@vger.kernel.org/msg03813.html
>
> So, I think yes moving to ioremap in all drivers is good, but it would use underlying static maps where they exist.

Current linux-omap has worked like this for a while now.  IOW,
ioremap() will first check any static mappings before doing a real
remap.

So, without any changes to the memory map, everything could be converted
to ioremap + read/write already.

Kevin

  reply	other threads:[~2009-06-26 21:21 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-26  6:25 RFC: Reclaim address space on omaps, Tony on vacation in July Tony Lindgren
2009-06-26  7:45 ` Shilimkar, Santosh
2009-06-26  8:46   ` Tony Lindgren
2009-06-26 18:14     ` Woodruff, Richard
2009-06-26 21:21       ` Kevin Hilman [this message]
2009-06-26  8:37 ` RFC: " Ben Dooks
2009-06-26  8:57   ` Tony Lindgren
2009-06-26  9:02     ` Tony Lindgren

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=87prcqbsj4.fsf@deeprootsystems.com \
    --to=khilman@deeprootsystems.com \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    --cc=linux-omap@vger.kernel.org \
    --cc=r-woodruff2@ti.com \
    --cc=santosh.shilimkar@ti.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox