diff options
| author |   Frank Rowand <frank.rowand@sony.com> | 2017-10-17 16:36:29 -0700 |
|---|---|---|
| committer |   Rob Herring <robh@kernel.org> | 2017-10-17 20:47:27 -0500 |
| commit | f948d6d8b792bb90041edc12eac35faf83030994 (patch) | |
| tree | f98bad2bb9e0701c3fc198e72575216e442006b5 /include/linux/of.h | |
| parent | of: overlay: loosen overly strict phandle clash check (diff) | |
| download | linux-dev-f948d6d8b792bb90041edc12eac35faf83030994.tar.xz linux-dev-f948d6d8b792bb90041edc12eac35faf83030994.zip | |
of: overlay: avoid race condition between applying multiple overlays
The process of applying an overlay consists of:
- unflatten an overlay FDT (flattened device tree) into an
EDT (expanded device tree)
- fixup the phandle values in the overlay EDT to fit in a
range above the phandle values in the live device tree
- create the overlay changeset to reflect the contents of
the overlay EDT
- apply the overlay changeset, to modify the live device tree,
potentially changing the maximum phandle value in the live
device tree
There is currently no protection against two overlay applies
concurrently determining what range of phandle values are in use
in the live device tree, and subsequently changing that range.
Add a mutex to prevent multiple overlay applies from occurring
simultaneously.
Move of_resolve_phandles() into of_overlay_apply() so that it does not
have to be duplicated by each caller of of_overlay_apply().
The test in of_resolve_phandles() that the overlay tree is detached is
temporarily disabled so that old style overlay unittests do not fail.
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
Signed-off-by: Rob Herring <robh@kernel.org>
Diffstat (limited to 'include/linux/of.h')
| -rw-r--r-- | include/linux/of.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/include/linux/of.h b/include/linux/of.h index 96edda95c6b0..ef4c9ff5955a 100644 --- a/include/linux/of.h +++ b/include/linux/of.h @@ -1279,9 +1279,6 @@ static inline int of_reconfig_get_state_change(unsigned long action, } #endif /* CONFIG_OF_DYNAMIC */ -/* CONFIG_OF_RESOLVE api */ -extern int of_resolve_phandles(struct device_node *tree); - /** * of_device_is_system_power_controller - Tells if system-power-controller is found for device_node * @np: Pointer to the given device_node |