lkml.org 
[lkml]   [2008]   [Apr]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] x86: create array based interface to change page attribute
Arjan van de Ven wrote:
> Thomas Hellström wrote:
>> Ouch, so we'll be needing an ioremap_wc(), I guess. We probably
>> shouldn't use the linear kernel map for this anyway, since that would
>> require a chipset flush for each ring commit. We can actually use
>> vmap() with a wc page protection for that, but an ioremap_wc() would
>> certainly save us a lot of trouble.
>
> please don't use vmap() as a hack for mapping IO memory when you
> should use ioremap_wc()....
I think you've misinterpreted. We must use vmap() if we need to map the
cpu side of the aperture, (real pages) and ioremap() if we map the
device side, which is io memory. Some devices accept both, but what I
was saying was we should try to avoid the vmap() alternative, since it,
like using the kernel mapping directly, requires a chipset flush after
update.
> Lets get this thing right rather than finding whichever hack of the day
> happens to work with the implementation from yesterday.
/Thomas



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2008-04-08 08:15    [W:0.569 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site