[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20081020115810.GC10594@elte.hu>
Date: Mon, 20 Oct 2008 13:58:10 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Keith Packard <keithp@...thp.com>
Cc: Jesse Barnes <jbarnes@...tuousgeek.org>,
Nick Piggin <nickpiggin@...oo.com.au>,
Dave Airlie <airlied@...ux.ie>,
Yinghai Lu <yinghai@...nel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
dri-devel@...ts.sf.net, Andrew Morton <akpm@...ux-foundation.org>,
Linus Torvalds <torvalds@...ux-foundation.org>
Subject: Re: io resources and cached mappings (was: [git pull] drm patches
for 2.6.27-rc1)
* Keith Packard <keithp@...thp.com> wrote:
> On Sun, 2008-10-19 at 19:53 +0200, Ingo Molnar wrote:
>
> > Note how simple and consistent it all gets: IO resources already
> > know their physical location and their size limits. Being able to
> > cache an ioremap in a mapping [and being able to use atomic kmaps on
> > 32-bit] is a relatively simple and natural extension to the concept.
>
> I'm not sure I see any value in caching mappings here; we're mostly
> interested in copying lots of data onto the card and so we use a lot
> of different mappings; atomic mappings are easy to use, and efficient
> enough.
yes but note that by caching the whole mapping on 64-bit we get
everything we want: trivially lockless, works from any CPU, can be
preempted at will, and there are no ugly INVLPG flushes anywhere.
you'll even get 2MB mappings automatically, if the BAR is aligned and
sized correctly.
32-bit we should handle as well but not design for it.
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists