livepatch: Consolidate klp_free functions

The code for freeing livepatch structures is a bit scattered and tricky:

  + direct calls to klp_free_*_limited() and kobject_put() are
    used to release partially initialized objects

  + klp_free_patch() removes the patch from the public list
    and releases all objects except for patch->kobj

  + object_put(&patch->kobj) and the related wait_for_completion()
    are called directly outside klp_mutex; this code is duplicated;

Now, we are going to remove the registration stage to simplify the API
and the code. This would require handling more situations in
klp_enable_patch() error paths.

More importantly, we are going to add a feature called atomic replace.
It will need to dynamically create func and object structures. We will
want to reuse the existing init() and free() functions. This would
create even more error path scenarios.

This patch implements more straightforward free functions:

  + checks kobj_added flag instead of @limit[*]

  + initializes patch->list early so that the check for empty list
    always works

  + The action(s) that has to be done outside klp_mutex are done
    in separate klp_free_patch_finish() function. It waits only
    when patch->kobj was really released via the _start() part.

The patch does not change the existing behavior.

[*] We need our own flag to track that the kobject was successfully
    added to the hierarchy.  Note that kobj.state_initialized only
    indicates that kobject has been initialized, not whether is has
    been added (and needs to be removed on cleanup).

Signed-off-by: Petr Mladek <pmladek@suse.com>
Cc: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: Miroslav Benes <mbenes@suse.cz>
Cc: Jessica Yu <jeyu@kernel.org>
Cc: Jiri Kosina <jikos@kernel.org>
Cc: Jason Baron <jbaron@akamai.com>
Acked-by: Miroslav Benes <mbenes@suse.cz>
Acked-by: Josh Poimboeuf <jpoimboe@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>

1 files changed, 6 insertions, 0 deletions

diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h
index 634e13876380..6978785bc059 100644
--- a/include/linux/livepatch.h
+++ b/include/linux/livepatch.h
@@ -45,6 +45,7 @@
  * @stack_node: list node for klp_ops func_stack list
  * @old_size:   size of the old function
  * @new_size:   size of the new function
+ * @kobj_added: @kobj has been added and needs freeing
  * @patched:    the func has been added to the klp_ops list
  * @transition: the func is currently being applied or reverted
  *
@@ -81,6 +82,7 @@ struct klp_func {
         struct kobject kobj;
         struct list_head stack_node;
         unsigned long old_size, new_size;
+        bool kobj_added;
         bool patched;
         bool transition;
 };
@@ -117,6 +119,7 @@ struct klp_callbacks {
  * @kobj:       kobject for sysfs resources
  * @mod:        kernel module associated with the patched object
  *              (NULL for vmlinux)
+ * @kobj_added: @kobj has been added and needs freeing
  * @patched:    the object's funcs have been added to the klp_ops list
  */
 struct klp_object {
@@ -128,6 +131,7 @@ struct klp_object {
         /* internal */
         struct kobject kobj;
         struct module *mod;
+        bool kobj_added;
         bool patched;
 };
 
@@ -137,6 +141,7 @@ struct klp_object {
  * @objs:       object entries for kernel objects to be patched
  * @list:       list node for global list of registered patches
  * @kobj:       kobject for sysfs resources
+ * @kobj_added: @kobj has been added and needs freeing
  * @enabled:    the patch is enabled (but operation may be incomplete)
  * @finish:     for waiting till it is safe to remove the patch module
  */
@@ -148,6 +153,7 @@ struct klp_patch {
         /* internal */
         struct list_head list;
         struct kobject kobj;
+        bool kobj_added;
         bool enabled;
         struct completion finish;
 };