[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20101223135120.GL3636@n2100.arm.linux.org.uk>
Date: Thu, 23 Dec 2010 13:51:20 +0000
From: Russell King - ARM Linux <linux@....linux.org.uk>
To: Michal Nazarewicz <mina86@...a86.com>
Cc: Kyungmin Park <kmpark@...radead.org>,
linux-arm-kernel@...ts.infradead.org,
Daniel Walker <dwalker@...eaurora.org>,
Johan MOSSBERG <johan.xx.mossberg@...ricsson.com>,
Mel Gorman <mel@....ul.ie>, linux-kernel@...r.kernel.org,
linux-mm@...ck.org, Ankita Garg <ankita@...ibm.com>,
Andrew Morton <akpm@...ux-foundation.org>,
linux-media@...r.kernel.org,
KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>,
Marek Szyprowski <m.szyprowski@...sung.com>
Subject: Re: [PATCHv8 00/12] Contiguous Memory Allocator
On Thu, Dec 23, 2010 at 02:41:26PM +0100, Michal Nazarewicz wrote:
> Russell King - ARM Linux <linux@....linux.org.uk> writes:
> > Has anyone addressed my issue with it that this is wide-open for
> > abuse by allocating large chunks of memory, and then remapping
> > them in some way with different attributes, thereby violating the
> > ARM architecture specification?
> >
> > In other words, do we _actually_ have a use for this which doesn't
> > involve doing something like allocating 32MB of memory from it,
> > remapping it so that it's DMA coherent, and then performing DMA
> > on the resulting buffer?
>
> Huge pages.
>
> Also, don't treat it as coherent memory and just flush/clear/invalidate
> cache before and after each DMA transaction. I never understood what's
> wrong with that approach.
If you've ever used an ARM system with a VIVT cache, you'll know what's
wrong with this approach.
ARM systems with VIVT caches have extremely poor task switching
performance because they flush the entire data cache at every task switch
- to the extent that it makes system performance drop dramatically when
they become loaded.
Doing that for every DMA operation will kill the advantage we've gained
from having VIPT caches and ASIDs stone dead.
--
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