mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-26 19:00:25 +00:00
of: some unittest overlays not untracked
kernel test robot reported "WARNING: held lock freed!" triggered by
unittest_gpio_remove(), which should not have been called because
the related gpio overlay was not tracked. Another overlay that
was tracked had previously used the same id as the gpio overlay
but had not been untracked when the overlay was removed. Thus the
clean up function of_unittest_destroy_tracked_overlays() incorrectly
attempted to remove the reused overlay id.
Patch contents:
- Create tracking related helper functions
- Change BUG() to WARN_ON() for overlay id related issues
- Add some additional error checking for valid overlay id values
- Add the missing overlay untrack
- update comment on expectation that overlay ids are assigned in
sequence
Fixes: 492a22aceb
("of: unittest: overlay: Keep track of created overlays")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Frank Rowand <frank.rowand@sony.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Rob Herring <robh@kernel.org>
This commit is contained in:
parent
fb227f597d
commit
1adc86798f
1 changed files with 23 additions and 7 deletions
|
@ -1601,6 +1601,11 @@ static void __init of_unittest_overlay_gpio(void)
|
|||
* Similar to installing a driver as a module, the
|
||||
* driver is registered after applying the overlays.
|
||||
*
|
||||
* The overlays are applied by overlay_data_apply()
|
||||
* instead of of_unittest_apply_overlay() so that they
|
||||
* will not be tracked. Thus they will not be removed
|
||||
* by of_unittest_destroy_tracked_overlays().
|
||||
*
|
||||
* - apply overlay_gpio_01
|
||||
* - apply overlay_gpio_02a
|
||||
* - apply overlay_gpio_02b
|
||||
|
@ -1847,19 +1852,27 @@ static const char *overlay_name_from_nr(int nr)
|
|||
|
||||
static const char *bus_path = "/testcase-data/overlay-node/test-bus";
|
||||
|
||||
/* it is guaranteed that overlay ids are assigned in sequence */
|
||||
/* FIXME: it is NOT guaranteed that overlay ids are assigned in sequence */
|
||||
|
||||
#define MAX_UNITTEST_OVERLAYS 256
|
||||
static unsigned long overlay_id_bits[BITS_TO_LONGS(MAX_UNITTEST_OVERLAYS)];
|
||||
static int overlay_first_id = -1;
|
||||
|
||||
static long of_unittest_overlay_tracked(int id)
|
||||
{
|
||||
if (WARN_ON(id >= MAX_UNITTEST_OVERLAYS))
|
||||
return 0;
|
||||
return overlay_id_bits[BIT_WORD(id)] & BIT_MASK(id);
|
||||
}
|
||||
|
||||
static void of_unittest_track_overlay(int id)
|
||||
{
|
||||
if (overlay_first_id < 0)
|
||||
overlay_first_id = id;
|
||||
id -= overlay_first_id;
|
||||
|
||||
/* we shouldn't need that many */
|
||||
BUG_ON(id >= MAX_UNITTEST_OVERLAYS);
|
||||
if (WARN_ON(id >= MAX_UNITTEST_OVERLAYS))
|
||||
return;
|
||||
overlay_id_bits[BIT_WORD(id)] |= BIT_MASK(id);
|
||||
}
|
||||
|
||||
|
@ -1868,7 +1881,8 @@ static void of_unittest_untrack_overlay(int id)
|
|||
if (overlay_first_id < 0)
|
||||
return;
|
||||
id -= overlay_first_id;
|
||||
BUG_ON(id >= MAX_UNITTEST_OVERLAYS);
|
||||
if (WARN_ON(id >= MAX_UNITTEST_OVERLAYS))
|
||||
return;
|
||||
overlay_id_bits[BIT_WORD(id)] &= ~BIT_MASK(id);
|
||||
}
|
||||
|
||||
|
@ -1884,7 +1898,7 @@ static void of_unittest_destroy_tracked_overlays(void)
|
|||
defers = 0;
|
||||
/* remove in reverse order */
|
||||
for (id = MAX_UNITTEST_OVERLAYS - 1; id >= 0; id--) {
|
||||
if (!(overlay_id_bits[BIT_WORD(id)] & BIT_MASK(id)))
|
||||
if (!of_unittest_overlay_tracked(id))
|
||||
continue;
|
||||
|
||||
ovcs_id = id + overlay_first_id;
|
||||
|
@ -1901,7 +1915,7 @@ static void of_unittest_destroy_tracked_overlays(void)
|
|||
continue;
|
||||
}
|
||||
|
||||
overlay_id_bits[BIT_WORD(id)] &= ~BIT_MASK(id);
|
||||
of_unittest_untrack_overlay(id);
|
||||
}
|
||||
} while (defers > 0);
|
||||
}
|
||||
|
@ -1962,7 +1976,7 @@ static int __init of_unittest_apply_revert_overlay_check(int overlay_nr,
|
|||
int unittest_nr, int before, int after,
|
||||
enum overlay_type ovtype)
|
||||
{
|
||||
int ret, ovcs_id;
|
||||
int ret, ovcs_id, save_id;
|
||||
|
||||
/* unittest device must be in before state */
|
||||
if (of_unittest_device_exists(unittest_nr, ovtype) != before) {
|
||||
|
@ -1990,6 +2004,7 @@ static int __init of_unittest_apply_revert_overlay_check(int overlay_nr,
|
|||
return -EINVAL;
|
||||
}
|
||||
|
||||
save_id = ovcs_id;
|
||||
ret = of_overlay_remove(&ovcs_id);
|
||||
if (ret != 0) {
|
||||
unittest(0, "%s failed to be destroyed @\"%s\"\n",
|
||||
|
@ -1997,6 +2012,7 @@ static int __init of_unittest_apply_revert_overlay_check(int overlay_nr,
|
|||
unittest_path(unittest_nr, ovtype));
|
||||
return ret;
|
||||
}
|
||||
of_unittest_untrack_overlay(save_id);
|
||||
|
||||
/* unittest device must be again in before state */
|
||||
if (of_unittest_device_exists(unittest_nr, PDEV_OVERLAY) != before) {
|
||||
|
|
Loading…
Reference in a new issue