Message ID | 20241002152849.111841-1-surajsonawane0215@gmail.com |
---|---|
State | New |
Headers | show |
Series | selftests/vdso: Add linux/sched.h to fix CLONE_NEWTIME build error | expand |
On 10/2/24 09:28, SurajSonawane2415 wrote: > Fix build error in vdso_test_getrandom.c due to missing CLONE_NEWTIME. > Include linux/sched.h to define CLONE_NEWTIME. > Ensure successful compilation by resolving the missing header issue. > Did you run "make headers" before building this test? It builds just fine for me on my system. + Adding Jason Donenfeld > Signed-off-by: SurajSonawane2415 <surajsonawane0215@gmail.com> > --- > tools/testing/selftests/vDSO/vdso_test_getrandom.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/testing/selftests/vDSO/vdso_test_getrandom.c b/tools/testing/selftests/vDSO/vdso_test_getrandom.c > index 72a1d9b43..3f2a4dbf8 100644 > --- a/tools/testing/selftests/vDSO/vdso_test_getrandom.c > +++ b/tools/testing/selftests/vDSO/vdso_test_getrandom.c > @@ -22,6 +22,7 @@ > #include <linux/random.h> > #include <linux/compiler.h> > #include <linux/ptrace.h> > +#include <linux/sched.h> > > #include "../kselftest.h" > #include "parse_vdso.h" thanks, -- Shuah
Wasn't this already submitted and commented on? https://lore.kernel.org/all/20240919111841.20226-1-liaoyu15@huawei.com/
On 10/2/24 16:51, Jason A. Donenfeld wrote: > Wasn't this already submitted and commented on? > > https://lore.kernel.org/all/20240919111841.20226-1-liaoyu15@huawei.com/ Thank you Jason. Yes we reviewed this - I asked Yu Liao to send me v2 since the define is coming in from pthread.h indirectly. Suraj, Thank for finding and fixing the problem. However Yu Liao sent in patch for this before you did. We will take that patch. thanks, -- Shuah
diff --git a/tools/testing/selftests/vDSO/vdso_test_getrandom.c b/tools/testing/selftests/vDSO/vdso_test_getrandom.c index 72a1d9b43..3f2a4dbf8 100644 --- a/tools/testing/selftests/vDSO/vdso_test_getrandom.c +++ b/tools/testing/selftests/vDSO/vdso_test_getrandom.c @@ -22,6 +22,7 @@ #include <linux/random.h> #include <linux/compiler.h> #include <linux/ptrace.h> +#include <linux/sched.h> #include "../kselftest.h" #include "parse_vdso.h"
Fix build error in vdso_test_getrandom.c due to missing CLONE_NEWTIME. Include linux/sched.h to define CLONE_NEWTIME. Ensure successful compilation by resolving the missing header issue. Signed-off-by: SurajSonawane2415 <surajsonawane0215@gmail.com> --- tools/testing/selftests/vDSO/vdso_test_getrandom.c | 1 + 1 file changed, 1 insertion(+)