On Sun, Aug 13, 2017 at 05:50:39PM -0400, Rob Clark wrote:
On Sun, Aug 13, 2017 at 5:13 PM, Tom Rini trini@konsulko.com wrote:
On Sun, Aug 13, 2017 at 06:45:20AM -0400, Rob Clark wrote:
Introduce directory traversal iterators, and implement fs_readdir() which is needed by EFI_LOADER.
The part re-working fat.c to use the directory iterators itself is nearly a 2:1 negative diffstat, and a pretty big cleanup. I fixed one or two other small issues along the way. It hasn't really been tested with a wide variaty of different fat filesystems (if someone has a collection of disk images to test with somewhere, let me know), but it seems at least not worse than what it is replacing.
Did you run test/fs/fs-test.sh and confirm there's at least no regressions? Thanks!
I've been having trouble getting fs-test.sh to pass even without my changes, so no..
Pass, or provide expected results? From the script, we expect: # Total Summary: TOTAL PASS: 132 TOTAL FAIL: 6
Not entirely sure if I have newer kernel/etc compared to the last person who ran fs-test.sh, or if there is something on u-boot side which changed without corresponding change in fs-test.sh.
Note that this script it part of my everything-my-lab-isn't-giving-me-fits loop. I'm probably going to split that loop into pure-SW and HW, as the HW part is what's driving me crazy most of the time.