]> git.dujemihanovic.xyz Git - u-boot.git/commit
Revert "bootdev: avoid infinite probe loop"
authorSimon Glass <sjg@chromium.org>
Thu, 15 Aug 2024 20:30:21 +0000 (14:30 -0600)
committerTom Rini <trini@konsulko.com>
Tue, 27 Aug 2024 16:09:07 +0000 (10:09 -0600)
commitaad8b18a529d7413f35ba413a9c122dbb76d08bb
treeb8bc9240cbd5a95a89d09c3c85f859161fd7119c
parent90270e0f71c53cca9c0347ce3baedf26b8ec4cdb
Revert "bootdev: avoid infinite probe loop"

This turns out to be insufficient to fix the problem, since when
bootdev_next_prio() exits, the caller has no idea that this really
is the end. Nor is it, since there may be other devices which should
be checked.

The caller iterates which calls iter_incr() which calls
bootdev_next_prio() again, which finds the same device and the loop
continues.

We never did create a test for this[1], which makes it hard to be
sure which problem was fixed.

The original code had the virtue of staying in the loop looking for a
bootdev, so let's go back to that and try to fix this another way.

A future patch will make bootdev_next_prio() continue after failure
which should provide same effect.

This reverts commit 9d92c418acfb7576e12e2bd53fed294bb9543724.

Signed-off-by: Simon Glass <sjg@chromium.org>
boot/bootdev-uclass.c