On 12/21/22 09:39, Shuah Khan wrote:
On 12/16/22 07:35, Gerd Bayer wrote:
On Tue, 2022-12-06 at 11:13 +0800, chenxiang (M) wrote:
在 2022/12/5 21:50, Gerd Bayer 写道:
Since [1] the user-space program dma_map_benchmark shares the
header file
linux/map_benchmark.h with the kernel driver in
kernel/dma/map_benchmark.c.
With latest kernel version this does not compile any more.
While https://kernelnewbies.org/KernelHeaders suggests otherwise,
allow it
to use of kernel headers through the uapi/ include direcotry. I
assume we can
do so safely, since the controlling user-space program is
distributed with
the kernel.
With this change dma_map_benchmark compiles with just the obvious
warning
about uapi usage on ARCH=x86 and s390 and runs on ARCH=s390.
[1] commit 8ddde07a3d28 ("dma-mapping: benchmark: extract a common
header file for map_benchmark definition")
Signed-off-by: Gerd Bayer <gbayer@xxxxxxxxxxxxx>
It also solves the compile error on arm64 platform.
Acked-by: Xiang Chen <chenxiang66@xxxxxxxxxxxxx>
Hi Xiang Chen, hi Shuah,
so is there any chance to have this compile fix integrated into linux-
next still, or is this too late now?
Plan to apply this after the merge window closes
The reason for the delay is this patch doesn't have the right commit
header. This should have been "selftests:dma -----"
It makes it easier for maintainers to parse and pay attention to the
patches if they indicate the area they fall under. Something to
keep in mind for your future patches.
Please resend the patch with the Acks you received and a correct
commit header.
thanks,
-- Shuah
