On Mon, Jan 14, 2019 at 09:14:08PM +0100, Christoph Hellwig wrote:
> Otherwise is_swiotlb_buffer will return false positives when
> we first initialize a swiotlb buffer, but then free it because
> we have an IOMMU available.
> 
> Fixes: 55897af63091 ("dma-direct: merge swiotlb_dma_ops into the dma_direct 
> code")
> Reported-by: Sibren Vasse <[email protected]>
> Signed-off-by: Christoph Hellwig <[email protected]>
> Tested-by: Sibren Vasse <[email protected]>

Reviewed-by: Konrad Rzeszutek Wilk <[email protected]>

Thank you!
> ---
>  kernel/dma/swiotlb.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/kernel/dma/swiotlb.c b/kernel/dma/swiotlb.c
> index d6361776dc5c..1fb6fd68b9c7 100644
> --- a/kernel/dma/swiotlb.c
> +++ b/kernel/dma/swiotlb.c
> @@ -378,6 +378,8 @@ void __init swiotlb_exit(void)
>               memblock_free_late(io_tlb_start,
>                                  PAGE_ALIGN(io_tlb_nslabs << IO_TLB_SHIFT));
>       }
> +     io_tlb_start = 0;
> +     io_tlb_end = 0;
>       io_tlb_nslabs = 0;
>       max_segment = 0;
>  }
> -- 
> 2.20.1
> 
_______________________________________________
iommu mailing list
[email protected]
https://lists.linuxfoundation.org/mailman/listinfo/iommu

Reply via email to