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

[Garrett D'Amore]

I just experienced the nvidia hang ... oh well, this code doesn't fix that... but it was just a hopeful prayer anyway.  I'm still happy with Michael's changes, and I think we should go ahead and integrate them.  The fact that it doesn't fix my occasional Nvidia hang is not that suprising since we have not diagnosed it (and can't, since the driver is closed source!)

Thanks.

  -- Garrett D'Amore

On May 23, 2011, at 3:40 PM, "Garrett D'Amore" <garrett at nexenta.com> wrote:

> I've been testing this myself and so far it seems good on my laptop.  I need to run it through a few more reboot cycles to know if it fixes the Nvidia startup problem.  (I'm hopeful, although my Nvidia problem has never been adequately diagnosed.)
> 
>  -- Garrett D'Amore
> 
> On May 12, 2011, at 10:34 PM, "Michael Tsymbalyuk" <mtzaurus at gmail.com> wrote:
> 
>> This 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(). New version uses #ifndef __sparc and
>> explicitly checks for caching flags passed to the function.
>> 
>> http://cr.illumos.org/view/1qszmsk0/illumos-534-webrev/
>> 
>> Sincerely,
>> Michael Tsymbalyuk
>> 
>> 
>> 
>> _______________________________________________
>> Developer mailing list
>> Developer at lists.illumos.org
>> http://lists.illumos.org/m/listinfo/developer
> 
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer