linux-kernel - Re: [PATCH v2 0/4] dma mapping benchmark: add support for dma_map

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: <fd0dfb53-127f-4529-adf8-db03269d5798@oracle.com>
Date: Tue, 6 May 2025 17:50:37 +0530
From: ALOK TIWARI <alok.a.tiwari@...cle.com>
To: Qinxin Xia <xiaqinxin@...wei.com>, baohua@...nel.org
Cc: yangyicong@...wei.com, hch@....de, iommu@...ts.linux.dev,
        jonathan.cameron@...wei.com, prime.zeng@...wei.com,
        fanghao11@...wei.com, linux-kernel@...r.kernel.org,
        linuxarm@...wei.com
Subject: Re: [PATCH v2 0/4] dma mapping benchmark: add support for dma_map_sg



On 06-05-2025 08:30, Qinxin Xia wrote:
> Modify the framework to adapt to more map modes, add benchmark
> support for dma_map_sg, and add support sg map mode in ioctl.
> 
> The result:
> [root@...alhost]# ./dma_map_benchmark -m 1 -g 8 -t 8 -s 30 -d 2
> dma mapping mode: DMA_MAP_SG_MODE
> dma mapping benchmark: threads:8 seconds:30 node:-1 dir:FROM_DEVICE granule/sg_nents: 8
> average map latency(us):1.4 standard deviation:0.3
> average unmap latency(us):1.3 standard deviation:0.3
> [root@...alhost]# ./dma_map_benchmark -m 0 -g 8 -t 8 -s 30 -d 2
> dma mapping mode: DMA_MAP_SINGLE_MODE
> dma mapping benchmark: threads:8 seconds:30 node:-1 dir:FROM_DEVICE granule/sg_nents: 8
> average map latency(us):1.0 standard deviation:0.3
> average unmap latency(us):1.3 standard deviation:0.5
> 
> ---
> Changes since V1:
> - Address the comments from Masami, added some comments and changed the unmap type to void.
> - Link: https://urldefense.com/v3/__https://lore.kernel.org/lkml/20250212022718.1995504-1-xiaqinxin@huawei.com/__;!!ACWV5N9M2RV99hQ!PZwCK7yspP9AzXLsodEKNp6_pWU8ir6IKfsJQJIpvfLQdOrNgjLpwNySORdXdT1JJFAbo82SMmIlnnYI3tSpGQ$
> 
> Qinxin Xia (4):
>    dma-mapping: benchmark: Add padding to ensure uABI remained consistent
>    dma mapping benchmark: modify the framework to adapt to more map modes
>    dma-mapping: benchmark: add support for dma_map_sg
>    dma mapping benchmark:add support for dma_map_sg

I noticed the subject lines use both "dma-mapping" and "dma mapping" ?
Could we make them consistent?
> 
>   include/linux/map_benchmark.h                 |  46 +++-
>   kernel/dma/map_benchmark.c                    | 222 ++++++++++++++++--
>   .../testing/selftests/dma/dma_map_benchmark.c |  16 +-
>   3 files changed, 249 insertions(+), 35 deletions(-)
> 
> --
> 2.33.0
> 
> 

Thanks,
Alok