Re: [for-linus][PATCH 01/10] tracing: Suppress sparse warnings triggered by is_signed_type()
From: Linus Torvalds
Date: Tue Aug 23 2022 - 21:51:28 EST
On Tue, Aug 23, 2022 at 5:09 PM Bart Van Assche <bvanassche@xxxxxxx> wrote:
>
> I'm probably doing something wrong but even with sparse commit 658ee8e0f631
> ("unrestricted values are unrestricted even after a cast") I see warnings
> being triggered by users of the is_signed_type() macro, warnings that
> disappear if I change the definition of the is_signed_type() macro into 0:
That's the
> It does require that kernel change to make
>
> #define is_signed_type(type) (((type)(-1)) <= (type)0)
part I was talking about.
So your kernel side patch looks fine, except I don't think you need
the '__force' - the sparse patches in my tree should make sparse happy
about casting '-1'.
But I didn't do very much testing.
Linus