[illumos-Developer] Webrev #534: DMA KVA cache attributes incorrect

Garrett D'Amore garrett at damore.org
Tue May 10 07:53:27 PDT 2011


On Tue, 2011-05-10 at 18:08 +0400, Michael Tsymbalyuk wrote:
> Thiw webrev fixes the problem of improper handling of page table
> attributes related to caching while allocating a DMA-compatible
> memory via ddi_dma_mem_alloc(). The patch adds extra attribute check
> to ddi_dma_mem_alloc().
> 
> http://cr.illumos.org/view/asjt99s5/illumos-534-webrev/
> 
> Sincerely,
> Michael Tsymbalyuk
> 

This looks good if it is that simple.  I want to do some deeper analysis
to review your code.  Out of curiosity, did you test this at all?  If
so, how?

	- Garrett
> 
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer





More information about the Developer mailing list