On Mon, 2016-10-24 at 08:05 -0400, Alexander Duyck wrote:
> This change allows us to pass DMA_ATTR_SKIP_CPU_SYNC which allows us to
> avoid invoking cache line invalidation if the driver will just handle it
> later via a sync_for_cpu or sync_for_device call.
> 
> Cc: Mark Salter <msal...@redhat.com>
> Cc: Aurelien Jacquiot <a-jacqu...@ti.com>
> Cc: linux-c6x-...@linux-c6x.org
> Signed-off-by: Alexander Duyck <alexander.h.du...@intel.com>
> ---

Acked-by: Mark Salter <msal...@redhat.com>

>  arch/c6x/kernel/dma.c |   16 +++++++++++-----
>  1 file changed, 11 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/c6x/kernel/dma.c b/arch/c6x/kernel/dma.c
> index db4a6a3..d28df74 100644
> --- a/arch/c6x/kernel/dma.c
> +++ b/arch/c6x/kernel/dma.c
> @@ -42,14 +42,17 @@ static dma_addr_t c6x_dma_map_page(struct device *dev, 
> struct page *page,
>  {
>       dma_addr_t handle = virt_to_phys(page_address(page) + offset);
>  
> -     c6x_dma_sync(handle, size, dir);
> +     if (!(attrs & DMA_ATTR_SKIP_CPU_SYNC))
> +             c6x_dma_sync(handle, size, dir);
> +
>       return handle;
>  }
>  
>  static void c6x_dma_unmap_page(struct device *dev, dma_addr_t handle,
>               size_t size, enum dma_data_direction dir, unsigned long attrs)
>  {
> -     c6x_dma_sync(handle, size, dir);
> +     if (!(attrs & DMA_ATTR_SKIP_CPU_SYNC))
> +             c6x_dma_sync(handle, size, dir);
>  }
>  
>  static int c6x_dma_map_sg(struct device *dev, struct scatterlist *sglist,
> @@ -60,7 +63,8 @@ static int c6x_dma_map_sg(struct device *dev, struct 
> scatterlist *sglist,
>  
>       for_each_sg(sglist, sg, nents, i) {
>               sg->dma_address = sg_phys(sg);
> -             c6x_dma_sync(sg->dma_address, sg->length, dir);
> +             if (!(attrs & DMA_ATTR_SKIP_CPU_SYNC))
> +                     c6x_dma_sync(sg->dma_address, sg->length, dir);
>       }
>  
>       return nents;
> @@ -72,8 +76,10 @@ static void c6x_dma_unmap_sg(struct device *dev, struct 
> scatterlist *sglist,
>       struct scatterlist *sg;
>       int i;
>  
> -     for_each_sg(sglist, sg, nents, i)
> -             c6x_dma_sync(sg_dma_address(sg), sg->length, dir);
> +     for_each_sg(sglist, sg, nents, i) {
> +             if (!(attrs & DMA_ATTR_SKIP_CPU_SYNC))
> +                     c6x_dma_sync(sg_dma_address(sg), sg->length, dir);
> +     }
>  
>  }
>  
> 

Reply via email to