The patch titled Subject: arch/frv: add option to skip sync on DMA map has been added to the -mm tree. Its filename is arch-frv-add-option-to-skip-sync-on-dma-map.patch This patch should soon appear at http://ozlabs.org/~akpm/mmots/broken-out/arch-frv-add-option-to-skip-sync-on-dma-map.patch and later at http://ozlabs.org/~akpm/mmotm/broken-out/arch-frv-add-option-to-skip-sync-on-dma-map.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** The -mm tree is included into linux-next and is updated there every 3-4 working days ------------------------------------------------------ From: Alexander Duyck <alexander.h.duyck@xxxxxxxxx> Subject: arch/frv: add option to skip sync on DMA map The use of DMA_ATTR_SKIP_CPU_SYNC was not consistent across all of the DMA APIs in the arch/arm folder. This change is meant to correct that so that we get consistent behavior. Link: http://lkml.kernel.org/r/20161110113447.76501.93160.stgit@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx Signed-off-by: Alexander Duyck <alexander.h.duyck@xxxxxxxxx> Cc: David Howells <dhowells@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- arch/frv/mb93090-mb00/pci-dma-nommu.c | 14 ++++++++++---- arch/frv/mb93090-mb00/pci-dma.c | 9 +++++++-- 2 files changed, 17 insertions(+), 6 deletions(-) diff -puN arch/frv/mb93090-mb00/pci-dma-nommu.c~arch-frv-add-option-to-skip-sync-on-dma-map arch/frv/mb93090-mb00/pci-dma-nommu.c --- a/arch/frv/mb93090-mb00/pci-dma-nommu.c~arch-frv-add-option-to-skip-sync-on-dma-map +++ a/arch/frv/mb93090-mb00/pci-dma-nommu.c @@ -109,16 +109,19 @@ static int frv_dma_map_sg(struct device int nents, enum dma_data_direction direction, unsigned long attrs) { - int i; struct scatterlist *sg; + int i; + + BUG_ON(direction == DMA_NONE); + + if (attrs & DMA_ATTR_SKIP_CPU_SYNC) + return nents; for_each_sg(sglist, sg, nents, i) { frv_cache_wback_inv(sg_dma_address(sg), sg_dma_address(sg) + sg_dma_len(sg)); } - BUG_ON(direction == DMA_NONE); - return nents; } @@ -127,7 +130,10 @@ static dma_addr_t frv_dma_map_page(struc enum dma_data_direction direction, unsigned long attrs) { BUG_ON(direction == DMA_NONE); - flush_dcache_page(page); + + if (!(attrs & DMA_ATTR_SKIP_CPU_SYNC)) + flush_dcache_page(page); + return (dma_addr_t) page_to_phys(page) + offset; } diff -puN arch/frv/mb93090-mb00/pci-dma.c~arch-frv-add-option-to-skip-sync-on-dma-map arch/frv/mb93090-mb00/pci-dma.c --- a/arch/frv/mb93090-mb00/pci-dma.c~arch-frv-add-option-to-skip-sync-on-dma-map +++ a/arch/frv/mb93090-mb00/pci-dma.c @@ -40,13 +40,16 @@ static int frv_dma_map_sg(struct device int nents, enum dma_data_direction direction, unsigned long attrs) { + struct scatterlist *sg; unsigned long dampr2; void *vaddr; int i; - struct scatterlist *sg; BUG_ON(direction == DMA_NONE); + if (attrs & DMA_ATTR_SKIP_CPU_SYNC) + return nents; + dampr2 = __get_DAMPR(2); for_each_sg(sglist, sg, nents, i) { @@ -70,7 +73,9 @@ static dma_addr_t frv_dma_map_page(struc unsigned long offset, size_t size, enum dma_data_direction direction, unsigned long attrs) { - flush_dcache_page(page); + if (!(attrs & DMA_ATTR_SKIP_CPU_SYNC)) + flush_dcache_page(page); + return (dma_addr_t) page_to_phys(page) + offset; } _ Patches currently in -mm which might be from alexander.h.duyck@xxxxxxxxx are arch-arc-add-option-to-skip-sync-on-dma-mapping.patch arch-arm-add-option-to-skip-sync-on-dma-map-and-unmap.patch arch-avr32-add-option-to-skip-sync-on-dma-map.patch arch-blackfin-add-option-to-skip-sync-on-dma-map.patch arch-c6x-add-option-to-skip-sync-on-dma-map-and-unmap.patch arch-frv-add-option-to-skip-sync-on-dma-map.patch arch-hexagon-add-option-to-skip-dma-sync-as-a-part-of-mapping.patch arch-m68k-add-option-to-skip-dma-sync-as-a-part-of-mapping.patch arch-metag-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-microblaze-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-mips-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-nios2-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-openrisc-add-option-to-skip-dma-sync-as-a-part-of-mapping.patch arch-parisc-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-powerpc-add-option-to-skip-dma-sync-as-a-part-of-mapping.patch arch-sh-add-option-to-skip-dma-sync-as-a-part-of-mapping.patch arch-sparc-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-tile-add-option-to-skip-dma-sync-as-a-part-of-map-and-unmap.patch arch-xtensa-add-option-to-skip-dma-sync-as-a-part-of-mapping.patch dma-add-calls-for-dma_map_page_attrs-and-dma_unmap_page_attrs.patch mm-add-support-for-releasing-multiple-instances-of-a-page.patch igb-update-driver-to-make-use-of-dma_attr_skip_cpu_sync.patch igb-update-code-to-better-handle-incrementing-page-count.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html