Data abort on Sansa Clip+ with latest daily build

[Bilgus]

I think I've figured out that its not the chunk allocator with this build
could you try it and see if it works for you as well?

https://gerrit.rockbox.org/r/c/rockbox/+/5155

since clearing the ram when we are done with it stops the crash (for me at least)
I think either someone else isn't initializing their allocation or buflib is returning
someone elses data

[Carlo]

Sure, I'm on it.

[Carlo]

No crashes after several hours of usage and attempts to reproduce the original issue. I believe we're in the clear.

[Bilgus]

I don't know if thats good or bad news

Now I don't know where to start but I'll update this thread when I figure it out

[Carlo]

Is the "playlist: Remove in-ram filename storage" commit a bugfix for this issue?

[Bilgus]

yep I couldn't reproduce after this so do test!

[bahus]

Something is not right with latest playlist changes. With Resume Playback in particular.
Bookmark on Stop option no longer works for me after clicking Resume Playback from main menu. Also after Resume Playback playlist is considered as dynamic as it asks "Erase dynamic playlist?" on attempt to play another file (I suspect that's the main reason why Bookmarking is not working)

[Carlo]

I've used the new builds extensively, and the data abort issue was completely resolved for me. I've noticed however that playing a file from a larger directory (500+ songs) for the first time now takes several seconds of "Loading..", instead of being instantanous like before the recent playlist changes, and navigating said directories feels quite sluggish. Is this behaviour related to switching the playlist from RAM to a plain file?

For comparison, the old "[BugFix] playlist.c DIRCACHE stop scanning when changing indices" build from 2023-01-12, which doesn't have the data abort bug, can play files from large directories and navigate their content without any speed issue.