From: "Pratyush Yadav (Google)" <[email protected]>

This allows other functions to also use the radix tree. While at it,
also use kho_get_mem_map_phys() instead of duplicating the code to get
the radix tree root from the FDT.

Signed-off-by: Pratyush Yadav (Google) <[email protected]>
---
 kernel/liveupdate/kexec_handover.c | 32 ++++++++++++------------------
 1 file changed, 13 insertions(+), 19 deletions(-)

diff --git a/kernel/liveupdate/kexec_handover.c 
b/kernel/liveupdate/kexec_handover.c
index d8dd0ede4f87..61e436f5077e 100644
--- a/kernel/liveupdate/kexec_handover.c
+++ b/kernel/liveupdate/kexec_handover.c
@@ -1334,6 +1334,7 @@ struct kho_in {
        char previous_release[__NEW_UTS_LEN + 1];
        u32 kexec_count;
        struct kho_debugfs dbg;
+       struct kho_radix_tree radix_tree;
 };
 
 static struct kho_in kho_in = {
@@ -1413,24 +1414,15 @@ EXPORT_SYMBOL_GPL(kho_retrieve_subtree);
 
 static int __init kho_mem_retrieve(const void *fdt)
 {
-       struct kho_radix_tree tree;
-       const phys_addr_t *mem;
-       int len;
-
-       /* Retrieve the KHO radix tree from passed-in FDT. */
-       mem = fdt_getprop(fdt, 0, KHO_FDT_MEMORY_MAP_PROP_NAME, &len);
-
-       if (!mem || len != sizeof(*mem)) {
-               pr_err("failed to get preserved KHO memory tree\n");
-               return -ENOENT;
-       }
-
-       if (!*mem)
-               return -EINVAL;
-
-       tree.root = phys_to_virt(*mem);
-       mutex_init(&tree.lock);
-       return kho_radix_walk_tree(&tree, kho_preserved_memory_reserve);
+       /*
+        * kho_get_mem_map() should always succeed. If it fails, kho_populate()
+        * catches that and never sets kho_in.scratch_phys, which stops memory
+        * retrieval.
+        */
+       kho_in.radix_tree.root = kho_get_mem_map(fdt);
+       mutex_init(&kho_in.radix_tree.lock);
+       return kho_radix_walk_tree(&kho_in.radix_tree,
+                                  kho_preserved_memory_reserve);
 }
 
 static __init int kho_out_fdt_setup(void)
@@ -1637,8 +1629,10 @@ void __init kho_memory_init(void)
        if (kho_in.scratch_phys) {
                kho_scratch = phys_to_virt(kho_in.scratch_phys);
 
-               if (kho_mem_retrieve(kho_get_fdt()))
+               if (kho_mem_retrieve(kho_get_fdt())) {
                        kho_in.fdt_phys = 0;
+                       kho_in.radix_tree.root = NULL;
+               }
        } else {
                kho_reserve_scratch();
        }
-- 
2.54.0.1032.g2f8565e1d1-goog


Reply via email to