On 8/22/23 1:57 PM, Muhammad Usama Anjum wrote: > Hi Shuah, > > Christian Brauner had picked [PATCH 4/6]. Others are still not picked. > Please have a look. Soft reminder > > Thanks, > Usama > > On 8/5/23 12:37 PM, Muhammad Usama Anjum wrote: >> These duplicate defines should automatically be picked up from kernel >> headers. Use KHDR_INCLUDES to add kernel header files. >> >> Signed-off-by: Muhammad Usama Anjum <usama.anjum@xxxxxxxxxxxxx> >> --- >> tools/testing/selftests/capabilities/Makefile | 2 +- >> tools/testing/selftests/capabilities/test_execve.c | 8 -------- >> tools/testing/selftests/capabilities/validate_cap.c | 8 -------- >> 3 files changed, 1 insertion(+), 17 deletions(-) >> >> diff --git a/tools/testing/selftests/capabilities/Makefile b/tools/testing/selftests/capabilities/Makefile >> index 6e9d98d457d5b..411ac098308f1 100644 >> --- a/tools/testing/selftests/capabilities/Makefile >> +++ b/tools/testing/selftests/capabilities/Makefile >> @@ -2,7 +2,7 @@ >> TEST_GEN_FILES := validate_cap >> TEST_GEN_PROGS := test_execve >> >> -CFLAGS += -O2 -g -std=gnu99 -Wall >> +CFLAGS += -O2 -g -std=gnu99 -Wall $(KHDR_INCLUDES) >> LDLIBS += -lcap-ng -lrt -ldl >> >> include ../lib.mk >> diff --git a/tools/testing/selftests/capabilities/test_execve.c b/tools/testing/selftests/capabilities/test_execve.c >> index df0ef02b40367..e3a352b020a79 100644 >> --- a/tools/testing/selftests/capabilities/test_execve.c >> +++ b/tools/testing/selftests/capabilities/test_execve.c >> @@ -20,14 +20,6 @@ >> >> #include "../kselftest.h" >> >> -#ifndef PR_CAP_AMBIENT >> -#define PR_CAP_AMBIENT 47 >> -# define PR_CAP_AMBIENT_IS_SET 1 >> -# define PR_CAP_AMBIENT_RAISE 2 >> -# define PR_CAP_AMBIENT_LOWER 3 >> -# define PR_CAP_AMBIENT_CLEAR_ALL 4 >> -#endif >> - >> static int nerrs; >> static pid_t mpid; /* main() pid is used to avoid duplicate test counts */ >> >> diff --git a/tools/testing/selftests/capabilities/validate_cap.c b/tools/testing/selftests/capabilities/validate_cap.c >> index cdfc94268fe6e..60b4e7b716a75 100644 >> --- a/tools/testing/selftests/capabilities/validate_cap.c >> +++ b/tools/testing/selftests/capabilities/validate_cap.c >> @@ -9,14 +9,6 @@ >> >> #include "../kselftest.h" >> >> -#ifndef PR_CAP_AMBIENT >> -#define PR_CAP_AMBIENT 47 >> -# define PR_CAP_AMBIENT_IS_SET 1 >> -# define PR_CAP_AMBIENT_RAISE 2 >> -# define PR_CAP_AMBIENT_LOWER 3 >> -# define PR_CAP_AMBIENT_CLEAR_ALL 4 >> -#endif >> - >> #if __GLIBC__ > 2 || (__GLIBC__ == 2 && __GLIBC_MINOR__ >= 19) >> # define HAVE_GETAUXVAL >> #endif > -- BR, Muhammad Usama Anjum