> The osnoise_hotplug_workfn() grabs two mutexes and cpu_read_lock(). It has > various gotos to handle unlocking them. Switch them over to guard() and > let the compiler worry about it. > > The osnoise_cpus_read() has a temporary mask_str allocated and there's > some gotos to make sure it gets freed on error paths. Switch that over to > __free() to let the compiler worry about it. I would find it safer and cleaner to separate adjustments for these programming interfaces. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.13-rc3#n81 Regards, Markus