Hi Rob, On Mon, Jul 31, 2023 at 6:35 PM Rob Herring <robh@xxxxxxxxxx> wrote: > On Mon, Jul 31, 2023 at 10:14:25AM -0600, Rob Herring wrote: > > On Fri, Jul 28, 2023 at 10:50:26AM +0200, Geert Uytterhoeven wrote: > > > This patch series contains miscellaneous fixes and improvements for > > > dynamic DT overlays and the related unit tests. > > > > > > The first two patches are fixes for a lock-up and a crash. > > > The remaining patches are smaller fixes, enhancements and cleanups for > > > the overlay tests, including one new test. > > > > > > I ran into the crash when accidentally loading the wrong overlay (using > > > the out-of-tree DT overlay configfs[1]), and removing it afterwards. > > > As this case was not yet covered by the unittests, I added a test. > > > I enhanced the tests to clean up partial state after a failed > > > overlay apply attempt, which triggered the lock-up. > > > > > > Changes compared to v1[2]: > > > - Correct fixes tag and update description. > > > - Merge differently, as requested by Rob. > > > > > > Thanks for your comments! > > > > > > [1] https://elinux.org/R-Car/DT-Overlays > > > [2] https://lore.kernel.org/r/cover.1689776064.git.geert+renesas@xxxxxxxxx > > > > > > Geert Uytterhoeven (13): > > > of: dynamic: Do not use "%pOF" while holding devtree_lock > > > of: overlay: Call of_changeset_init() early > > > of: unittest: Fix overlay type in apply/revert check > > > of: unittest: Restore indentation in overlay_bad_add_dup_prop test > > > of: unittest: Improve messages and comments in apply/revert checks > > > of: unittest: Merge of_unittest_apply{,_revert}_overlay_check() > > > of: unittest: Cleanup partially-applied overlays > > > of: unittest: Add separators to of_unittest_overlay_high_level() > > > of: overlay: unittest: Add test for unresolved symbol > > > of: unittest-data: Convert remaining overlay DTS files to sugar syntax > > > of: unittest-data: Fix whitespace - blank lines > > > of: unittest-data: Fix whitespace - indentation > > > of: unittest-data: Fix whitespace - angular brackets > > > > I've applied patches 2-13. For patch 1, I sent an alternative[1]. > > I guess there's a dependency on patch 1 because I hang here: > > [ 1.341292] OF: overlay: ERROR: multiple fragments add and/or delete node /testcase-data-2/substation@100/motor-1/controller > [ 1.343222] OF: overlay: ERROR: multiple fragments add, update, and/or delete property /testcase-data-2/substation@100/motor-1/controller/name Yes there is: during removal of a partially-applied overlay, some errors are printed using %pOF, which must not be done while holding devtree_lock. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
