Re: [PATCH] drm: add func to better detect wether swiotlb is needed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



.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. 
> 
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux