Re: [PATCH v2 0/4] dma mapping benchmark: add support for dma_map_sg

[ALOK TIWARI <alok.a.tiwari@oracle.com>]

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@localhost]# ./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@localhost]# ./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