On Wed, Nov 16, 2011 at 10:46:57AM -0500, Konrad Rzeszutek Wilk wrote: > > +#ifdef CONFIG_SWIOTLB > > + if (swiotlb_nr_tbl()) { > > + return ttm_dma_populate(ttm, rdev->dev); > > + } > > +#endif > .. > > +#ifdef CONFIG_SWIOTLB > > + if (swiotlb_nr_tbl()) { > > + ttm_dma_unpopulate(ttm, rdev->dev); > > + return; > > + } > > +#endif > > > +#ifdef CONFIG_SWIOTLB > > + if (rdev->need_dma32 && swiotlb_nr_tbl()) { > > + sprintf(radeon_mem_types_names[i], "ttm_dma_page_pool"); > > + radeon_mem_types_list[i].name = radeon_mem_types_names[i]; > > + radeon_mem_types_list[i].show = &ttm_dma_page_alloc_debugfs; > > + radeon_mem_types_list[i].driver_features = 0; > > + radeon_mem_types_list[i++].data = NULL; > > + } > > +#endif > > So I think the last one does not need the rdev->need_dma32 as you had > removed it from the other sites. Yeah i forgot to remove it there too. I removed it on others so that swiotlb force works. Cheers, Jerome _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel