Patches without the replace flag might depend on each other. It makes
sense to enforce the order in which they are enabled and disabled.
The situation is different when the patch replaces all existing ones.
These patches should make the life easier for both: patch producers
and users. Such a patch should be ready to replace basically any
older patch. It should work well even in situations when the previous
patches were not installed or when they were disabled from some reasons.
The code is almost ready for this. There are needed two changes:
+ Disable only enabled patches in klp_discard_replaced_patches().
+ Stop enforcing the stack order for the patches with the replace flag.
Instead, we need to make sure that they are still usable (not
replaced). This check is already there.
Signed-off-by: Petr Mladek <pmladek@xxxxxxxx>
---
kernel/livepatch/core.c | 19 +++++++++++++------
1 file changed, 13 insertions(+), 6 deletions(-)
diff --git a/kernel/livepatch/core.c b/kernel/livepatch/core.c
index 0b3be6e14b80..c64371ffc063 100644
--- a/kernel/livepatch/core.c
+++ b/kernel/livepatch/core.c
@@ -355,11 +355,13 @@ void klp_discard_replaced_patches(struct klp_patch *new_patch, bool keep_module)
if (old_patch == new_patch)
return;
- klp_unpatch_objects(old_patch);
- old_patch->enabled = false;
+ if (old_patch->enabled) {
+ klp_unpatch_objects(old_patch);
+ old_patch->enabled = false;
- if (!keep_module)
- module_put(old_patch->mod);
+ if (!keep_module)
+ module_put(old_patch->mod);
+ }
/*
* Replaced patches could not get re-enabled to keep
@@ -453,8 +455,13 @@ static int __klp_enable_patch(struct klp_patch *patch)
if (!klp_is_patch_on_stack(patch))
return -EINVAL;
- /* Only the first disabled patch can be enabled. */
- if (patch->list.prev != &klp_patches &&
+ /*
+ * Only the first disabled patch can be enabled. This is not required
+ * for patches with the replace flags. They override even disabled
+ * patches that were registered earlier.
+ */
+ if (!patch->replace &&
+ patch->list.prev != &klp_patches &&
!list_prev_entry(patch, list)->enabled)
return -EBUSY;
--
2.13.6
--
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
