| Message ID | 20220126174421.1712795-1-broonie@kernel.org |
|---|---|
| State | Accepted |
| Commit | 396520759bd3bf4a557e4edba9a63afc13cc5773 |
| Headers | show |
| Series | kselftest/arm64: Remove local definitions of MTE prctls | expand |
On Wed, Jan 26, 2022 at 05:44:21PM +0000, Mark Brown wrote: > The GCR EL1 test unconditionally includes local definitions of the prctls > it tests. Since not only will the kselftest build infrastructure ensure > that the in tree uapi headers are available but the toolchain being used to > build kselftest may ensure that system uapi headers with MTE support are > available this causes the compiler to warn about duplicate definitions. > Remove these duplicate definitions. > > Signed-off-by: Mark Brown <broonie@kernel.org> Acked-by: Catalin Marinas <catalin.marinas@arm.com>
On 1/26/22 10:44 AM, Mark Brown wrote: > The GCR EL1 test unconditionally includes local definitions of the prctls > it tests. Since not only will the kselftest build infrastructure ensure > that the in tree uapi headers are available but the toolchain being used to > build kselftest may ensure that system uapi headers with MTE support are > available this causes the compiler to warn about duplicate definitions. > Remove these duplicate definitions. > > Signed-off-by: Mark Brown <broonie@kernel.org> > --- > .../selftests/arm64/mte/check_gcr_el1_cswitch.c | 11 ----------- > 1 file changed, 11 deletions(-) > > diff --git a/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c b/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c > index a876db1f096a..325bca0de0f6 100644 > --- a/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c > +++ b/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c > @@ -19,17 +19,6 @@ > #include "kselftest.h" > #include "mte_common_util.h" > > -#define PR_SET_TAGGED_ADDR_CTRL 55 > -#define PR_GET_TAGGED_ADDR_CTRL 56 > -# define PR_TAGGED_ADDR_ENABLE (1UL << 0) > -# define PR_MTE_TCF_SHIFT 1 > -# define PR_MTE_TCF_NONE (0UL << PR_MTE_TCF_SHIFT) > -# define PR_MTE_TCF_SYNC (1UL << PR_MTE_TCF_SHIFT) > -# define PR_MTE_TCF_ASYNC (2UL << PR_MTE_TCF_SHIFT) > -# define PR_MTE_TCF_MASK (3UL << PR_MTE_TCF_SHIFT) > -# define PR_MTE_TAG_SHIFT 3 > -# define PR_MTE_TAG_MASK (0xffffUL << PR_MTE_TAG_SHIFT) > - > #include "mte_def.h" > > #define NUM_ITERATIONS 1024 > Thank you for cleaning this up. Assuming it is going to go through arm tree: Acked-by: Shuah Khan <skhan@linuxfoundation.org> thanks, -- Shuah
On Wed, 26 Jan 2022 17:44:21 +0000, Mark Brown wrote: > The GCR EL1 test unconditionally includes local definitions of the prctls > it tests. Since not only will the kselftest build infrastructure ensure > that the in tree uapi headers are available but the toolchain being used to > build kselftest may ensure that system uapi headers with MTE support are > available this causes the compiler to warn about duplicate definitions. > Remove these duplicate definitions. > > [...] Applied to arm64 (for-next/kselftest), thanks! [1/1] kselftest/arm64: Remove local definitions of MTE prctls https://git.kernel.org/arm64/c/396520759bd3 Cheers,
diff --git a/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c b/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c index a876db1f096a..325bca0de0f6 100644 --- a/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c +++ b/tools/testing/selftests/arm64/mte/check_gcr_el1_cswitch.c @@ -19,17 +19,6 @@ #include "kselftest.h" #include "mte_common_util.h" -#define PR_SET_TAGGED_ADDR_CTRL 55 -#define PR_GET_TAGGED_ADDR_CTRL 56 -# define PR_TAGGED_ADDR_ENABLE (1UL << 0) -# define PR_MTE_TCF_SHIFT 1 -# define PR_MTE_TCF_NONE (0UL << PR_MTE_TCF_SHIFT) -# define PR_MTE_TCF_SYNC (1UL << PR_MTE_TCF_SHIFT) -# define PR_MTE_TCF_ASYNC (2UL << PR_MTE_TCF_SHIFT) -# define PR_MTE_TCF_MASK (3UL << PR_MTE_TCF_SHIFT) -# define PR_MTE_TAG_SHIFT 3 -# define PR_MTE_TAG_MASK (0xffffUL << PR_MTE_TAG_SHIFT) - #include "mte_def.h" #define NUM_ITERATIONS 1024
The GCR EL1 test unconditionally includes local definitions of the prctls it tests. Since not only will the kselftest build infrastructure ensure that the in tree uapi headers are available but the toolchain being used to build kselftest may ensure that system uapi headers with MTE support are available this causes the compiler to warn about duplicate definitions. Remove these duplicate definitions. Signed-off-by: Mark Brown <broonie@kernel.org> --- .../selftests/arm64/mte/check_gcr_el1_cswitch.c | 11 ----------- 1 file changed, 11 deletions(-)