Messages in this thread Patch in this message | | | From | Mike Tipton <> | Subject | [PATCH] clk: Fix children not voting entire parent chain during init | Date | Mon, 8 Nov 2021 20:34:38 -0800 |
| |
If a child's parent is set by calling __clk_init_parent() while the parent is still being registered in __clk_register(), then it can result in the child voting its direct parent without those votes propagating up the entire parent chain.
__clk_register() sets hw->core before grabbing the prepare_lock and before initializing hw->core->parent. Since hw->core is used indirectly by __clk_init_parent(), then children can find their parents before they're fully initialized. If children vote for their parents during this window, then those votes won't propagate past the direct parent.
This can happen when: 1. CRITICAL clocks are enabled in __clk_core_init(). 2. Reparenting enabled orphans in clk_core_reparent_orphans_nolock().
Fix this by not setting hw->core until we've already grabbed the prepare_lock in __clk_core_init(). This prevents orphaned children from finding and voting their parents before the parents are fully initialized.
Fixes: fc0c209c147f ("clk: Allow parents to be specified without string names") Signed-off-by: Mike Tipton <quic_mdtipton@quicinc.com> ---
This is very difficult to reproduce. We can't reproduce it at all internally, in fact. But some customers are able to reproduce it fairly easily and this patch fixes it for them.
drivers/clk/clk.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c index f467d63bbf1e..af562af9d54d 100644 --- a/drivers/clk/clk.c +++ b/drivers/clk/clk.c @@ -3418,6 +3418,13 @@ static int __clk_core_init(struct clk_core *core) clk_prepare_lock(); + /* + * Set hw->core after grabbing the prepare_lock to prevent race + * conditions with orphans finding and voting their parents before the + * parents are fully initialized. + */ + core->hw->core = core; + ret = clk_pm_runtime_get(core); if (ret) goto unlock; @@ -3582,8 +3589,10 @@ static int __clk_core_init(struct clk_core *core) out: clk_pm_runtime_put(core); unlock: - if (ret) + if (ret) { hlist_del_init(&core->child_node); + core->hw->core = NULL; + } clk_prepare_unlock(); @@ -3847,7 +3856,6 @@ __clk_register(struct device *dev, struct device_node *np, struct clk_hw *hw) core->num_parents = init->num_parents; core->min_rate = 0; core->max_rate = ULONG_MAX; - hw->core = core; ret = clk_core_populate_parent_map(core, init); if (ret) @@ -3865,7 +3873,7 @@ __clk_register(struct device *dev, struct device_node *np, struct clk_hw *hw) goto fail_create_clk; } - clk_core_link_consumer(hw->core, hw->clk); + clk_core_link_consumer(core, hw->clk); ret = __clk_core_init(core); if (!ret) -- 2.31.1
| |