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: > > Hi all, > > > > 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 Rob
