diff options
author | Simon Glass <sjg@chromium.org> | 2024-08-15 14:30:21 -0600 |
---|---|---|
committer | Tom Rini <trini@konsulko.com> | 2024-08-27 10:09:07 -0600 |
commit | aad8b18a529d7413f35ba413a9c122dbb76d08bb (patch) | |
tree | b8bc9240cbd5a95a89d09c3c85f859161fd7119c /tools/patman/commit.py | |
parent | 90270e0f71c53cca9c0347ce3baedf26b8ec4cdb (diff) |
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>
Diffstat (limited to 'tools/patman/commit.py')
0 files changed, 0 insertions, 0 deletions