[PATCH] drm: add func to better detect wether swiotlb is needed
Konrad Rzeszutek Wilk
konrad.wilk at oracle.com
Wed Feb 27 19:25:50 UTC 2019
.snip..
> > -u64 drm_get_max_iomem(void)
> > +bool drm_need_swiotlb(int dma_bits)
> > {
> > struct resource *tmp;
> > resource_size_t max_iomem = 0;
> >
> > + /*
> > + * Xen paravirtual hosts require swiotlb regardless of requested dma
> > + * transfer size.
> > + *
> > + * NOTE: Really, what it requires is use of the dma_alloc_coherent
> > + * allocator used in ttm_dma_populate() instead of
> > + * ttm_populate_and_map_pages(), which bounce buffers so much
> > in
> > + * Xen it leads to swiotlb buffer exhaustion.
> > + */
> > + if (xen_pv_domain())
>
> I've not been following all of the ins and outs of the discussion on this so apologies if I'm missing some context, but...
>
> This looks like the wrong test to me. I think it should be:
>
> if ( xen_swiotlb )
Ah, that could be as well.
>
More information about the amd-gfx
mailing list