On Fri, Jul 01, 2022 at 04:48:14PM -0300, Marcos Paulo de Souza wrote: > From: Petr Mladek <pmladek@xxxxxxxx> > > Separate code that is used in klp_shadow_get_or_alloc() under klp_mutex. > It splits a long spaghetti function into two. Also it unifies the error > handling. The old used a mix of duplicated code, direct returns, > and goto. The new code has only one unlock, free, and return calls. > > Background: The change was needed by an earlier variant of the code adding > garbage collection of shadow variables. It is not needed in > the end. But the change still looks like an useful clean up. > > It is code refactoring without any functional changes. > > Signed-off-by: Petr Mladek <pmladek@xxxxxxxx> Hi Marcos, Invalid Signed-off-by chain, it will need your SOB added as well. Also, when posting patches please add lkml to Cc so they're archived and visible to the wider community. -- Josh
