Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Bugs
  • Category Music playback
  • Assigned To No-one
  • Operating System SW-codec
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by Horscht - 2008-02-12
Last edited by nicolas_p - 2008-04-02

FS#8601 - Disk spinup after every single song if dircache is disabled

just today I noticed, that my Ipod 5.5G (80GB) spins up the disk after every song from a playist. Songs are MP3 VBR and are aprox 5MB each. Last.FM log is disabled, as is runtime gathering. I tried the latest SVN (16290) as well (the build i noticed this one was 2 days old). I have only my Ipod to test this on, so I do not know wether or not other HD based players suffer the same issue.

Closed by  nicolas_p
2008-04-02 17:05
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

r16930

ryran commented on 2008-02-13 04:29

A couple days ago I experienced the same thing (a 2-second disk activity at the beginning of every song) with my 60GB 5g …. I actually even made a video of it with Treo–it kept doing it (through buffer refills and everything) until I stopped and restarted playback. Sadly, I haven’t been able to reproduce it. Are you saying that yours does this all the time now, with every song?

yes, mine does it all the time, with every song. I compiled a few revisions, and so far have nailed it down to a change between SVN 16010 and 16025. Will do some more bisecting later on.

Ok, did some more testing. Issue was brough up with SVN 16019 “Make the playlist index be incremented after the PCM track change. This fixes  FS#8206 . Special treatment is required to avoid breaking auto dir change.”

stoping and restarting playback does not fix this issue for me.

Good work Horscht.. that’s interesting.

This should be fixed by r16330.

Unfortunatelly, still persistent as of SVN16342.

it still spins up after every song. Also general Disk Trashing (bug connected with  FS#8568 ) still persistent as well.

here is my config.cfg

Does enabling dircache stop the extra spinups?

enabling dircache stops the disk from spinning up after every song. I tried 2 themes (rayboradio for OB, CabbieV2), with dircache enabled, once with WPS dsiplayed, once in the main menu.

with dircache disabled, i got disk spinups with both skins, on the wps and the menu,

with dircache enabled, no disk spinups on either skin in either the wps and menu.

nls commented on 2008-03-07 17:01

Closed FS8667 as a duplicate of this as it is the same bug, the important things from that task are as follows.

* Only happens when dircache is disabled

* Happens with playlists and NOT with dirplay

* Introduced in r16019

* The cause is the added call to playlist_peek() inside audio_check_new_track().

* playlist_check() wouldn’t be a suitable replacement check in its current for as it always returns true in dirplay mode.

This one-line mod works for me, but probably needs a bit of testing.

nls commented on 2008-04-02 11:42

While I don’t understand why the check for an automatic dir skip is needed at all, playlist_check() always
returns true when in dirplay mode with automatic dir advance enabled so with your change AFAIU auto_dir_skip would always be false when an actual automatic dir advance happens…

Indeed…. any better (i.e. simpler) ideas?

OK, this one works by adding a new playlist API call to get the required info.

However, the PROPER fix is to sort out auto dir change to make it work without playback.c needing to know; maybe look at the MASCODEC code in mpeg.c for inspiration.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing