On Tue 2022-12-20 13:45:19, Steven Rostedt wrote:

[
  Linus,

    I ran the script against your latest master branch:
    commit b6bb9676f2165d518b35ba3bea5f1fcfc0d969bf

    As the timer_shutdown*() code is now in your tree, I figured
    we can start doing the conversions. At least add the trivial ones
    now as Thomas suggested that this gets applied at the end of the
    merge window, to avoid conflicts with linux-next during the
    development cycle. I can wait to Friday to run it again, and
    resubmit.

    What is the best way to handle this?
]

From: "Steven Rostedt (Google)" <rostedt@goodmis.org>

Due to several bugs caused by timers being re-armed after they are
shutdown and just before they are freed, a new state of timers was added
called "shutdown". After a timer is set to this state, then it can no
longer be re-armed.

The following script was run to find all the trivial locations where
del_timer() or del_timer_sync() is called in the same function that the
object holding the timer is freed. It also ignores any locations where the
timer->function is modified between the del_timer*() and the free(), as
that is not considered a "trivial" case.

This was created by using a coccinelle script and the following

commands:

LED parts looks good to me.

Getting it in just before -rc1 would be best solution for me.

Best regards,
								Pavel
-- 
People of Russia, stop Putin before his war on Ukraine escalates.