diff options
author | Petr Mladek <pmladek@suse.com> | 2019-05-03 15:26:24 +0200 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2019-05-03 21:11:22 +0200 |
commit | 4d141ab3416d90f87775f5dee725efdf40110a8f (patch) | |
tree | 656f94323db25996373b1bd775147f709f2cb836 /include/linux/livepatch.h | |
parent | 31adf2308f33dcae59009019675224be0978bc70 (diff) | |
download | linux-4d141ab3416d90f87775f5dee725efdf40110a8f.tar.gz linux-4d141ab3416d90f87775f5dee725efdf40110a8f.tar.bz2 linux-4d141ab3416d90f87775f5dee725efdf40110a8f.zip |
livepatch: Remove custom kobject state handling
kobject_init() always succeeds and sets the reference count to 1.
It allows to always free the structures via kobject_put() and
the related release callback.
Note that the custom kobject state handling was used only
because we did not know that kobject_put() can and actually
should get called even when kobject_init_and_add() fails.
The patch should not change the existing behavior.
Suggested-by: "Tobin C. Harding" <tobin@kernel.org>
Signed-off-by: Petr Mladek <pmladek@suse.com>
Reviewed-by: Kamalesh Babulal <kamalesh@linux.vnet.ibm.com>
Acked-by: Joe Lawrence <joe.lawrence@redhat.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'include/linux/livepatch.h')
-rw-r--r-- | include/linux/livepatch.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/include/linux/livepatch.h b/include/linux/livepatch.h index 53551f470722..a14bab1a0a3e 100644 --- a/include/linux/livepatch.h +++ b/include/linux/livepatch.h @@ -86,7 +86,6 @@ struct klp_func { struct list_head node; struct list_head stack_node; unsigned long old_size, new_size; - bool kobj_added; bool nop; bool patched; bool transition; @@ -141,7 +140,6 @@ struct klp_object { struct list_head func_list; struct list_head node; struct module *mod; - bool kobj_added; bool dynamic; bool patched; }; @@ -170,7 +168,6 @@ struct klp_patch { struct list_head list; struct kobject kobj; struct list_head obj_list; - bool kobj_added; bool enabled; bool forced; struct work_struct free_work; |