linux-kernel - Re: add an API to check if a streamming mapping needs sync calls

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <b97104e1-433c-8e35-59c6-b4dad047464c@intel.com>
Date:   Mon, 29 Jun 2020 15:39:01 +0200
From:   Björn Töpel <bjorn.topel@...el.com>
To:     Christoph Hellwig <hch@....de>
Cc:     Magnus Karlsson <magnus.karlsson@...el.com>,
        Jonathan Lemon <jonathan.lemon@...il.com>,
        iommu@...ts.linux-foundation.org, netdev@...r.kernel.org,
        bpf@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: add an API to check if a streamming mapping needs sync calls

On 2020-06-29 15:03, Christoph Hellwig wrote:
> Hi all,
> 
> this series lifts the somewhat hacky checks in the XSK code if a DMA
> streaming mapping needs dma_sync_single_for_{device,cpu} calls to the
> DMA API.
>

Thanks a lot for working on, and fixing this, Christoph!

I took the series for a spin, and there are (obviously) no performance
regressions.

Would the patches go through the net/bpf trees or somewhere else?

For the series:
Tested-by: Björn Töpel <bjorn.topel@...el.com>
Acked-by: Björn Töpel <bjorn.topel@...el.com>


Björn

> 
> Diffstat:
>   Documentation/core-api/dma-api.rst |    8 +++++
>   include/linux/dma-direct.h         |    1
>   include/linux/dma-mapping.h        |    5 +++
>   include/net/xsk_buff_pool.h        |    6 ++--
>   kernel/dma/direct.c                |    6 ++++
>   kernel/dma/mapping.c               |   10 ++++++
>   net/xdp/xsk_buff_pool.c            |   54 ++-----------------------------------
>   7 files changed, 37 insertions(+), 53 deletions(-)
>