Re: [PATCH v5a 5/5] treewide: Convert del_timer*() to timer_shutdown*()

From: Linus Torvalds <torvalds@linux-foundation.org>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: linux-kernel@vger.kernel.org,
	Thomas Gleixner <tglx@linutronix.de>,
	Stephen Boyd <sboyd@kernel.org>,
	Guenter Roeck <linux@roeck-us.net>,
	Anna-Maria Gleixner <anna-maria@linutronix.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	Julia Lawall <Julia.Lawall@inria.fr>
Subject: Re: [PATCH v5a 5/5] treewide: Convert del_timer*() to timer_shutdown*()
Date: Sun, 6 Nov 2022 14:40:14 -0800	[thread overview]
Message-ID: <CAHk-=whO5PXEzWwf=4=fvdqim6cGTczVoN4KJ5H+dabHo-OTHw@mail.gmail.com> (raw)
In-Reply-To: <20221106165220.4d7e5dac@rorschach.local.home>

On Sun, Nov 6, 2022 at 1:52 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
> I can update the change log to include:

Yup, full running instructions for coccinelle patches is probably a good idea.

I've done them semi-occasionally, but it's rare enough that I always
have to look it up anyway, and I suspect many others have never done
it, so having it in the commit message is probably a good idea.

>       when != ptr->timer.function = E;

I do think that you should just remove that E expression and the
"function = E" part.

Really, _any_ use of the timer after the timer delete makes it questionable,

It doesn't change the patch in my testing, but I think it's silly to
have that very specific pattern, when the more general case of "hey,
if you use the timer after deleting it, it's not obvious that it
should be a shutdown any more" just is more sensible anyway.

                 Linus