Messages in this thread | | | From | Thomas Gleixner <> | Subject | Re: [PATCH v6 4/6] timers: Add timer_shutdown_sync() to be called before freeing timers | Date | Mon, 14 Nov 2022 20:54:13 +0100 |
| |
On Mon, Nov 14 2022 at 14:28, Steven Rostedt wrote: > On Mon, 14 Nov 2022 20:13:28 +0100 > Thomas Gleixner <tglx@linutronix.de> wrote: >> The warning does not buy us anything, unless you want to go and amend >> all the usage sites which trigger it with 'if (mystruct->shutdown)' >> conditionals. > > The rationale for the warning was that it would let us know what drivers > need to be fixed for older kernels without the shutdown state. I'm > perfectly fine in removing the warning. We may just add it to the field > kernels so that we can know if there's any drivers that have issues that we > need to look at.
The warning is not guaranteed to catch the subtle cases. It might happen once in a blue-moon.
I rather argue that (once we agreed on the semantics) we should backport timer_shutdown() and the fixes which we add to Linus tree. Searching for potentially problematic places is a job for Coccinelle, though fixing them needs to have deep human inspection.
Backporting the core code and the corresponding fixes is way simpler than identifying the problematic cases via the unreliable warning and then coming up with a per driver solution by sprinkling 'if (in_shutdown)' conditionals all over the place.
Thanks,
tglx
| |