| Message ID | 20230713135440.3651409-5-ryan.roberts@arm.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | e515bce98deb06b392ca79b5c342a849372be7ae |
| Headers | show |
| Series | selftests/mm fixes for arm64 | expand |
On 13.07.23 15:54, Ryan Roberts wrote: > mrelease_test defaults to defining __NR_pidfd_open and > __NR_process_mrelease syscall numbers to -1, if they are not defined > anywhere else, and the suite would then be marked as skipped as a > result. > > arm64 (at least the stock debian toolchain that I'm using) requires > including <sys/syscall.h> to pull in the defines for these syscalls. So > let's add this header. With this in place, the test is passing on arm64. > > Signed-off-by: Ryan Roberts <ryan.roberts@arm.com> > --- > tools/testing/selftests/mm/mrelease_test.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/testing/selftests/mm/mrelease_test.c b/tools/testing/selftests/mm/mrelease_test.c > index dca21042b679..d822004a374e 100644 > --- a/tools/testing/selftests/mm/mrelease_test.c > +++ b/tools/testing/selftests/mm/mrelease_test.c > @@ -7,6 +7,7 @@ > #include <stdbool.h> > #include <stdio.h> > #include <stdlib.h> > +#include <sys/syscall.h> > #include <sys/wait.h> > #include <unistd.h> > #include <asm-generic/unistd.h> Reviewed-by: David Hildenbrand <david@redhat.com>
diff --git a/tools/testing/selftests/mm/mrelease_test.c b/tools/testing/selftests/mm/mrelease_test.c index dca21042b679..d822004a374e 100644 --- a/tools/testing/selftests/mm/mrelease_test.c +++ b/tools/testing/selftests/mm/mrelease_test.c @@ -7,6 +7,7 @@ #include <stdbool.h> #include <stdio.h> #include <stdlib.h> +#include <sys/syscall.h> #include <sys/wait.h> #include <unistd.h> #include <asm-generic/unistd.h>
mrelease_test defaults to defining __NR_pidfd_open and __NR_process_mrelease syscall numbers to -1, if they are not defined anywhere else, and the suite would then be marked as skipped as a result. arm64 (at least the stock debian toolchain that I'm using) requires including <sys/syscall.h> to pull in the defines for these syscalls. So let's add this header. With this in place, the test is passing on arm64. Signed-off-by: Ryan Roberts <ryan.roberts@arm.com> --- tools/testing/selftests/mm/mrelease_test.c | 1 + 1 file changed, 1 insertion(+)