On Mon 2016-02-08 23:50:22, Jessica Yu wrote: > In load_module(), the going notifiers are called during error handling when > an error occurs after the coming notifiers have already been called. > However, a module's state is still MODULE_STATE_COMING when the going > notifiers are called in the error path. To be consistent, also set > mod->state to MODULE_STATE_GOING before calling the going notifiers. > > Signed-off-by: Jessica Yu <jeyu@xxxxxxxxxx> Reviewed-by: Petr Mladek <pmladek@xxxxxxxx> It makes perfect sense to set the state this way. But note that it is only partial win. We still stay in the COMING state when using the other goto targets for the error handling, e.g. bug_cleanup, ddebug_cleanup. I was a bit nervous by such a change at this stage of 4.5 release. I spent quite some time on checking various scenarios and I did not find any problem with it. Best Regards, Petr -- To unsubscribe from this list: send the line "unsubscribe live-patching" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
