Re: issue with patch "x86: no CPA on iounmap"

From: Arjan van de Ven
Date: Mon Feb 04 2008 - 23:43:49 EST


Siddha, Suresh B wrote:
This is wrt to x86 git commit f56d005d30342a45d8af2b75ecccc82200f09600
"x86: no CPA on iounmap"

This can use performance issue. When a GART driver unmaps a RAM page,
which was mapped as UC, this commit will still retain UC attribute
on the kernel identity mapping. This can cause mysterious performance issue
if this freed page gets used by kernel later.

For now we should change the attribute during iounmap and in future PAT
infrastructure will have necessary hooks to avoid the aliasing issues.


this is a hard one; because the flipside is someone ioremapping the same page twice
(which is not impossible if there's 2 1k bars)... and then one of them gets iounmapped,
which would turn the page cachable again.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/