Need to add the selftest folks. + [email protected] + Kees Cook <[email protected]> + Shuah Khan <[email protected]>
Sagi Shahar wrote: > Redefinition of is_signed_type() causes compilation warning for tests > which use kselftest_harness. Replace the definition with linux/overflow.h > > Signed-off-by: Sagi Shahar <[email protected]> Thanks! I've seen this as well and it fixes the warning for me as well. It might be worth picking up separate from this series depending on what the selftest folks say. Tested-by: Ira Weiny <[email protected]> Reviewed-by: Ira Weiny <[email protected]> > --- > tools/testing/selftests/kselftest_harness.h | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/tools/testing/selftests/kselftest_harness.h > b/tools/testing/selftests/kselftest_harness.h > index 2925e47db995..a580a0d33c65 100644 > --- a/tools/testing/selftests/kselftest_harness.h > +++ b/tools/testing/selftests/kselftest_harness.h > @@ -56,6 +56,7 @@ > #include <asm/types.h> > #include <ctype.h> > #include <errno.h> > +#include <linux/overflow.h> > #include <linux/unistd.h> > #include <poll.h> > #include <stdbool.h> > @@ -751,8 +752,6 @@ > for (; _metadata->trigger; _metadata->trigger = \ > __bail(_assert, _metadata)) > > -#define is_signed_type(var) (!!(((__typeof__(var))(-1)) < > (__typeof__(var))1)) > - > #define __EXPECT(_expected, _expected_str, _seen, _seen_str, _t, _assert) do > { \ > /* Avoid multiple evaluation of the cases */ \ > __typeof__(_expected) __exp = (_expected); \ > -- > 2.51.0.rc1.193.gad69d77794-goog >
