| Message ID | 20230127135755.79929-1-mathieu.desnoyers@efficios.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | selftests: Fix incorrect kernel headers search path | expand |
On 1/27/23 06:57, Mathieu Desnoyers wrote: > Hi, > > This series fixes incorrect kernel header search path in kernel > selftests. > > Near the end of the series, a few changes are not tagged as "Fixes" > because the current behavior is to rely on the kernel sources uapi files > rather than on the installed kernel header files. Nevertheless, those > are updated for consistency. > > There are situations where "../../../../include/" was added to -I search > path, which is bogus for userspace tests and caused issues with types.h. > Those are removed. > > Thanks, > > Mathieu > > Mathieu Desnoyers (34): The below patches are now applied to linux-kselftest next for Linux 6.3-rc1 > selftests: arm64: Fix incorrect kernel headers search path > selftests: clone3: Fix incorrect kernel headers search path > selftests: core: Fix incorrect kernel headers search path > selftests: dma: Fix incorrect kernel headers search path > selftests: dmabuf-heaps: Fix incorrect kernel headers search path > selftests: drivers: Fix incorrect kernel headers search path > selftests: filesystems: Fix incorrect kernel headers search path > selftests: futex: Fix incorrect kernel headers search path > selftests: gpio: Fix incorrect kernel headers search path > selftests: ipc: Fix incorrect kernel headers search path > selftests: kcmp: Fix incorrect kernel headers search path > selftests: media_tests: Fix incorrect kernel headers search path > selftests: membarrier: Fix incorrect kernel headers search path > selftests: mount_setattr: Fix incorrect kernel headers search path > selftests: move_mount_set_group: Fix incorrect kernel headers search > path > selftests: perf_events: Fix incorrect kernel headers search path > selftests: pid_namespace: Fix incorrect kernel headers search path > selftests: pidfd: Fix incorrect kernel headers search path > selftests: ptp: Fix incorrect kernel headers search path > selftests: rseq: Fix incorrect kernel headers search path > selftests: sched: Fix incorrect kernel headers search path > selftests: seccomp: Fix incorrect kernel headers search path > selftests: sync: Fix incorrect kernel headers search path > selftests: user_events: Fix incorrect kernel headers search path > selftests: vm: Fix incorrect kernel headers search path > selftests: x86: Fix incorrect kernel headers search path > selftests: iommu: Use installed kernel headers search path > selftests: memfd: Use installed kernel headers search path > selftests: ptrace: Use installed kernel headers search path > selftests: tdx: Use installed kernel headers search path > These will be applied by maintainers to their trees. > selftests: bpf: Fix incorrect kernel headers search path # 02/34 > selftests: net: Fix incorrect kernel headers search path # 17/34 > selftests: powerpc: Fix incorrect kernel headers search path # 21/34 > selftests: bpf docs: Use installed kernel headers search path # 30/34 thanks, -- Shuah
On Mon, Jan 30, 2023 at 2:46 PM Shuah Khan <skhan@linuxfoundation.org> wrote: > > On 1/27/23 06:57, Mathieu Desnoyers wrote: > > Hi, > > > > This series fixes incorrect kernel header search path in kernel > > selftests. > > > > Near the end of the series, a few changes are not tagged as "Fixes" > > because the current behavior is to rely on the kernel sources uapi files > > rather than on the installed kernel header files. Nevertheless, those > > are updated for consistency. > > > > There are situations where "../../../../include/" was added to -I search > > path, which is bogus for userspace tests and caused issues with types.h. > > Those are removed. > > > > Thanks, > > > > Mathieu > > > > Mathieu Desnoyers (34): > > The below patches are now applied to linux-kselftest next for Linux 6.3-rc1 > > > selftests: arm64: Fix incorrect kernel headers search path > > selftests: clone3: Fix incorrect kernel headers search path > > selftests: core: Fix incorrect kernel headers search path > > selftests: dma: Fix incorrect kernel headers search path > > selftests: dmabuf-heaps: Fix incorrect kernel headers search path > > selftests: drivers: Fix incorrect kernel headers search path > > selftests: filesystems: Fix incorrect kernel headers search path > > selftests: futex: Fix incorrect kernel headers search path > > selftests: gpio: Fix incorrect kernel headers search path > > selftests: ipc: Fix incorrect kernel headers search path > > selftests: kcmp: Fix incorrect kernel headers search path > > selftests: media_tests: Fix incorrect kernel headers search path > > selftests: membarrier: Fix incorrect kernel headers search path > > selftests: mount_setattr: Fix incorrect kernel headers search path > > selftests: move_mount_set_group: Fix incorrect kernel headers search > > path > > selftests: perf_events: Fix incorrect kernel headers search path > > selftests: pid_namespace: Fix incorrect kernel headers search path > > selftests: pidfd: Fix incorrect kernel headers search path > > selftests: ptp: Fix incorrect kernel headers search path > > selftests: rseq: Fix incorrect kernel headers search path > > selftests: sched: Fix incorrect kernel headers search path > > selftests: seccomp: Fix incorrect kernel headers search path > > selftests: sync: Fix incorrect kernel headers search path > > selftests: user_events: Fix incorrect kernel headers search path > > selftests: vm: Fix incorrect kernel headers search path > > selftests: x86: Fix incorrect kernel headers search path > > selftests: iommu: Use installed kernel headers search path > > selftests: memfd: Use installed kernel headers search path > > selftests: ptrace: Use installed kernel headers search path > > selftests: tdx: Use installed kernel headers search path > > > > These will be applied by maintainers to their trees. Not in this form. They break the build. > > selftests: bpf: Fix incorrect kernel headers search path # 02/34 > > selftests: net: Fix incorrect kernel headers search path # 17/34 > > selftests: powerpc: Fix incorrect kernel headers search path # 21/34 > > selftests: bpf docs: Use installed kernel headers search path # 30/34 > > thanks, > -- Shuah
On 1/30/23 16:45, Alexei Starovoitov wrote: > On Mon, Jan 30, 2023 at 2:46 PM Shuah Khan <skhan@linuxfoundation.org> wrote: >> >> On 1/27/23 06:57, Mathieu Desnoyers wrote: >>> Hi, >>> >>> This series fixes incorrect kernel header search path in kernel >>> selftests. >>> >>> Near the end of the series, a few changes are not tagged as "Fixes" >>> because the current behavior is to rely on the kernel sources uapi files >>> rather than on the installed kernel header files. Nevertheless, those >>> are updated for consistency. >>> >>> There are situations where "../../../../include/" was added to -I search >>> path, which is bogus for userspace tests and caused issues with types.h. >>> Those are removed. >>> >>> Thanks, >>> >>> Mathieu >>> >>> Mathieu Desnoyers (34): >> >> The below patches are now applied to linux-kselftest next for Linux 6.3-rc1 >> >>> selftests: arm64: Fix incorrect kernel headers search path >>> selftests: clone3: Fix incorrect kernel headers search path >>> selftests: core: Fix incorrect kernel headers search path >>> selftests: dma: Fix incorrect kernel headers search path >>> selftests: dmabuf-heaps: Fix incorrect kernel headers search path >>> selftests: drivers: Fix incorrect kernel headers search path >>> selftests: filesystems: Fix incorrect kernel headers search path >>> selftests: futex: Fix incorrect kernel headers search path >>> selftests: gpio: Fix incorrect kernel headers search path >>> selftests: ipc: Fix incorrect kernel headers search path >>> selftests: kcmp: Fix incorrect kernel headers search path >>> selftests: media_tests: Fix incorrect kernel headers search path >>> selftests: membarrier: Fix incorrect kernel headers search path >>> selftests: mount_setattr: Fix incorrect kernel headers search path >>> selftests: move_mount_set_group: Fix incorrect kernel headers search >>> path >>> selftests: perf_events: Fix incorrect kernel headers search path >>> selftests: pid_namespace: Fix incorrect kernel headers search path >>> selftests: pidfd: Fix incorrect kernel headers search path >>> selftests: ptp: Fix incorrect kernel headers search path >>> selftests: rseq: Fix incorrect kernel headers search path >>> selftests: sched: Fix incorrect kernel headers search path >>> selftests: seccomp: Fix incorrect kernel headers search path >>> selftests: sync: Fix incorrect kernel headers search path >>> selftests: user_events: Fix incorrect kernel headers search path >>> selftests: vm: Fix incorrect kernel headers search path >>> selftests: x86: Fix incorrect kernel headers search path >>> selftests: iommu: Use installed kernel headers search path >>> selftests: memfd: Use installed kernel headers search path >>> selftests: ptrace: Use installed kernel headers search path >>> selftests: tdx: Use installed kernel headers search path >>> >> >> These will be applied by maintainers to their trees. > > Not in this form. They break the build. Mathieu is sending you the patches in the format you requested in the thread on this patch. thanks, -- Shuah
On Mon, Jan 30, 2023 at 3:48 PM Shuah Khan <skhan@linuxfoundation.org> wrote: > > >> > >> These will be applied by maintainers to their trees. > > > > Not in this form. They break the build. > > Mathieu is sending you the patches in the format you requested in > the thread on this patch. It's not the format, but the patch itself is incorrect.
Hi Mathieu, On 1/30/23 15:29, Shuah Khan wrote: > On 1/27/23 06:57, Mathieu Desnoyers wrote: >> Hi, >> >> This series fixes incorrect kernel header search path in kernel >> selftests. >> >> Near the end of the series, a few changes are not tagged as "Fixes" >> because the current behavior is to rely on the kernel sources uapi files >> rather than on the installed kernel header files. Nevertheless, those >> are updated for consistency. >> >> There are situations where "../../../../include/" was added to -I search >> path, which is bogus for userspace tests and caused issues with types.h. >> Those are removed. >> Thanks again for taking care of this. I did out of tree build testing on x86 on linux-kselftest next with these patches below. I haven't seen any problems introduced by the patch set. >> selftests: dma: Fix incorrect kernel headers search path This one needs a change and I will send a patch on top of yours. Even with that this test depends on unexported header from the repo and won't build out of tree. This is not related to your change. >> selftests: mount_setattr: Fix incorrect kernel headers search path This one fails to build with our without patch - an existing error. I have to do cross-build tests on arm64 and other arch patches still. This will happen later this week. >> selftests: arm64: Fix incorrect kernel headers search path drivers patch below had arch specific tests - testing todo The rest looks good. I will try to run bpf patches on my system. I do have clang, llvm installed on mine. TODO thanks, -- Shuah
On 2/1/23 19:07, Shuah Khan wrote: > Hi Mathieu, > > On 1/30/23 15:29, Shuah Khan wrote: >> On 1/27/23 06:57, Mathieu Desnoyers wrote: >>> Hi, >>> >>> This series fixes incorrect kernel header search path in kernel >>> selftests. >>> >>> Near the end of the series, a few changes are not tagged as "Fixes" >>> because the current behavior is to rely on the kernel sources uapi files >>> rather than on the installed kernel header files. Nevertheless, those >>> are updated for consistency. >>> >>> There are situations where "../../../../include/" was added to -I search >>> path, which is bogus for userspace tests and caused issues with types.h. >>> Those are removed. >>> > > Thanks again for taking care of this. I did out of tree build testing on > x86 on linux-kselftest next with these patches below. I haven't seen > any problems introduced by the patch set. > >>> selftests: dma: Fix incorrect kernel headers search path > This one needs a change and I will send a patch on top of yours. > Even with that this test depends on unexported header from the > repo and won't build out of tree. This is not related to your > change. > >>> selftests: mount_setattr: Fix incorrect kernel headers search path > This one fails to build with our without patch - an existing error. > > I have to do cross-build tests on arm64 and other arch patches still. > This will happen later this week. arm64, s390 patches look good. thanks, -- Shuah
On 2/3/23 18:06, Shuah Khan wrote: > On 2/1/23 19:07, Shuah Khan wrote: >> Hi Mathieu, >> >> On 1/30/23 15:29, Shuah Khan wrote: >>> On 1/27/23 06:57, Mathieu Desnoyers wrote: >>>> Hi, >>>> >>>> This series fixes incorrect kernel header search path in kernel >>>> selftests. >>>> >>>> Near the end of the series, a few changes are not tagged as "Fixes" >>>> because the current behavior is to rely on the kernel sources uapi files >>>> rather than on the installed kernel header files. Nevertheless, those >>>> are updated for consistency. >>>> >>>> There are situations where "../../../../include/" was added to -I search >>>> path, which is bogus for userspace tests and caused issues with types.h. >>>> Those are removed. >>>> >> >> Thanks again for taking care of this. I did out of tree build testing on >> x86 on linux-kselftest next with these patches below. I haven't seen >> any problems introduced by the patch set. >> >>>> selftests: dma: Fix incorrect kernel headers search path >> This one needs a change and I will send a patch on top of yours. >> Even with that this test depends on unexported header from the >> repo and won't build out of tree. This is not related to your >> change. >> >>>> selftests: mount_setattr: Fix incorrect kernel headers search path >> This one fails to build with our without patch - an existing error. >> >> I have to do cross-build tests on arm64 and other arch patches still. >> This will happen later this week. > > arm64, s390 patches look good. > I am seeing problem with selftests/dma and selfttests/user_events. 1. selftests: dma: Fix incorrect kernel headers search path dma test no longer builds. This test depends on linux/map_benchmark.h which is not included in uapi The order of include directorries -isystem followed by installed kernel headers, breaks the test build with the change to use KHDR_INCLUDES I am going to revert this patch for now and figure a longer term fix. The problem is the dependency on a non-uapi file: linux/map_benchmark.h Fixes: 8ddde07a3d28 ("dma-mapping: benchmark: extract a common header file for map_benchmark definition") change added this dependency on including linux/map_benchmark.h Christoph, Do you see this map_benchmark.h as part of uapi? 2. selftests: user_events: Fix incorrect kernel headers search path This one depends on linux/user_events.h which has bee removed from uapi in this commit: commit 5cfff569cab8bf544bab62c911c5d6efd5af5e05 Author: Steven Rostedt (Google) <rostedt@goodmis.org> Date: Fri Apr 1 14:39:03 2022 -0400 tracing: Move user_events.h temporarily out of include/uapi This isn't a regression from 6.2 - this test stopped building once user_events.h has been removed from uapi. I will add a note that this test depends on a non-uapi header and can't be built at the moment. thanks, -- Shuah
On Fri, 27 Jan 2023 08:57:21 -0500, Mathieu Desnoyers wrote: > This series fixes incorrect kernel header search path in kernel > selftests. > > Near the end of the series, a few changes are not tagged as "Fixes" > because the current behavior is to rely on the kernel sources uapi files > rather than on the installed kernel header files. Nevertheless, those > are updated for consistency. > > [...] Applied to powerpc/next. [21/34] selftests: powerpc: Fix incorrect kernel headers search path https://git.kernel.org/powerpc/c/4f11410bf6da87defe8fd59b0413f0d9f71744da cheers
Hi, This series fixes incorrect kernel header search path in kernel selftests. Near the end of the series, a few changes are not tagged as "Fixes" because the current behavior is to rely on the kernel sources uapi files rather than on the installed kernel header files. Nevertheless, those are updated for consistency. There are situations where "../../../../include/" was added to -I search path, which is bogus for userspace tests and caused issues with types.h. Those are removed. Thanks, Mathieu Mathieu Desnoyers (34): selftests: arm64: Fix incorrect kernel headers search path selftests: bpf: Fix incorrect kernel headers search path selftests: clone3: Fix incorrect kernel headers search path selftests: core: Fix incorrect kernel headers search path selftests: dma: Fix incorrect kernel headers search path selftests: dmabuf-heaps: Fix incorrect kernel headers search path selftests: drivers: Fix incorrect kernel headers search path selftests: filesystems: Fix incorrect kernel headers search path selftests: futex: Fix incorrect kernel headers search path selftests: gpio: Fix incorrect kernel headers search path selftests: ipc: Fix incorrect kernel headers search path selftests: kcmp: Fix incorrect kernel headers search path selftests: media_tests: Fix incorrect kernel headers search path selftests: membarrier: Fix incorrect kernel headers search path selftests: mount_setattr: Fix incorrect kernel headers search path selftests: move_mount_set_group: Fix incorrect kernel headers search path selftests: net: Fix incorrect kernel headers search path selftests: perf_events: Fix incorrect kernel headers search path selftests: pid_namespace: Fix incorrect kernel headers search path selftests: pidfd: Fix incorrect kernel headers search path selftests: powerpc: Fix incorrect kernel headers search path selftests: ptp: Fix incorrect kernel headers search path selftests: rseq: Fix incorrect kernel headers search path selftests: sched: Fix incorrect kernel headers search path selftests: seccomp: Fix incorrect kernel headers search path selftests: sync: Fix incorrect kernel headers search path selftests: user_events: Fix incorrect kernel headers search path selftests: vm: Fix incorrect kernel headers search path selftests: x86: Fix incorrect kernel headers search path selftests: bpf docs: Use installed kernel headers search path selftests: iommu: Use installed kernel headers search path selftests: memfd: Use installed kernel headers search path selftests: ptrace: Use installed kernel headers search path selftests: tdx: Use installed kernel headers search path tools/testing/selftests/arm64/fp/Makefile | 2 +- tools/testing/selftests/arm64/tags/Makefile | 2 +- tools/testing/selftests/bpf/Makefile | 2 +- tools/testing/selftests/bpf/Makefile.docs | 2 +- tools/testing/selftests/clone3/Makefile | 2 +- tools/testing/selftests/core/Makefile | 2 +- tools/testing/selftests/dma/Makefile | 2 +- tools/testing/selftests/dmabuf-heaps/Makefile | 2 +- tools/testing/selftests/dmabuf-heaps/dmabuf-heap.c | 3 +-- tools/testing/selftests/drivers/dma-buf/Makefile | 2 +- tools/testing/selftests/drivers/s390x/uvdevice/Makefile | 3 +-- tools/testing/selftests/filesystems/Makefile | 2 +- tools/testing/selftests/filesystems/binderfs/Makefile | 2 +- tools/testing/selftests/filesystems/epoll/Makefile | 2 +- tools/testing/selftests/futex/functional/Makefile | 2 +- tools/testing/selftests/gpio/Makefile | 2 +- tools/testing/selftests/iommu/Makefile | 3 +-- tools/testing/selftests/ipc/Makefile | 2 +- tools/testing/selftests/kcmp/Makefile | 2 +- tools/testing/selftests/media_tests/Makefile | 2 +- tools/testing/selftests/membarrier/Makefile | 2 +- tools/testing/selftests/memfd/Makefile | 4 +--- tools/testing/selftests/mount_setattr/Makefile | 2 +- tools/testing/selftests/move_mount_set_group/Makefile | 2 +- tools/testing/selftests/net/Makefile | 2 +- tools/testing/selftests/net/bpf/Makefile | 2 +- tools/testing/selftests/net/mptcp/Makefile | 2 +- tools/testing/selftests/net/openvswitch/Makefile | 2 +- tools/testing/selftests/perf_events/Makefile | 2 +- tools/testing/selftests/pid_namespace/Makefile | 2 +- tools/testing/selftests/pidfd/Makefile | 2 +- tools/testing/selftests/powerpc/ptrace/Makefile | 2 +- tools/testing/selftests/powerpc/security/Makefile | 2 +- tools/testing/selftests/powerpc/syscalls/Makefile | 2 +- tools/testing/selftests/powerpc/tm/Makefile | 2 +- tools/testing/selftests/ptp/Makefile | 2 +- tools/testing/selftests/ptrace/Makefile | 2 +- tools/testing/selftests/rseq/Makefile | 2 +- tools/testing/selftests/sched/Makefile | 2 +- tools/testing/selftests/seccomp/Makefile | 2 +- tools/testing/selftests/sync/Makefile | 2 +- tools/testing/selftests/tdx/Makefile | 2 +- tools/testing/selftests/tdx/tdx_guest_test.c | 2 +- tools/testing/selftests/user_events/Makefile | 2 +- tools/testing/selftests/vm/Makefile | 2 +- tools/testing/selftests/x86/Makefile | 2 +- 46 files changed, 46 insertions(+), 51 deletions(-)