Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Rockbox frontpage

Tasklist

FS#7257 - Playback resumes while paused when pressing back

Attached to Project: Rockbox
Opened by Beau Steward (Nimdae) - Monday, 04 June 2007, 05:47 GMT+2
Last edited by Linus Nielsen Feltzing (linusnielsen) - Thursday, 05 July 2007, 13:18 GMT+2
Task Type Bugs
Category Music playback
Status Closed
Assigned To No-one
Player Type SW-codec
Severity Medium
Priority Normal
Reported Version Daily build (which?)
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

This is a fresh build on an iPod 5G. I downloaded the latest source and manually built it on kermit.pimpinwithmuppets.com build server.

What happens is if I pause the playback, then hit the back button to go back to the beginning of the song, it does go back to the beginning of the song, but resumes playback. On my custom WPS, it still shows "paused", but the counters do count and it does play the music. If I hit pause/play, it will change the status of playback only, but does not affect actual playback. Hitting pause/play once again will pause playback.

This is not a new bug to me. When I last encountered it, I downloaded the latest source and built it on my build server, and it appeared to be fix. Unfortunately, I don't have build releases and dates to back this up.
This task depends upon

Closed by  Linus Nielsen Feltzing (linusnielsen)
Thursday, 05 July 2007, 13:18 GMT+2
Reason for closing:  Fixed
Comment by Nils Wallménius (nls) - Monday, 04 June 2007, 18:24 GMT+2
Confirmed on H320 with svn r13533
Comment by Beau Steward (Nimdae) - Monday, 04 June 2007, 23:39 GMT+2
This seems to be intermittant. Results with conditions:

Bug occured while only thing plugged into it was headphones.

Bug occured while nothing plugged in (plug in headphones or speakers later to hear playback).

Bug did not occur while plugged into car charger adapter (everything worked properly).

So far the pattern is the bug happens when nothing is plugged into the accessory/power port.
Comment by Douglas Valentine (Dwyloc) - Tuesday, 05 June 2007, 18:52 GMT+2
I am also seeing the same problem with both my iPod 5.5G and my iPod Nano with the current SVN build from 5 Jun 10:11.
Comment by Beau Steward (Nimdae) - Wednesday, 06 June 2007, 07:00 GMT+2
Unfortunately there is no longer a pattern to this problem as it occured while plugged into my car adapter today. It's like it maybe stores the state, maybe doesn't. Russian roulette if you will.
Comment by Steve Bavin (pondlife) - Tuesday, 12 June 2007, 10:41 GMT+2
It happens here reliably on my H340 (no car mode in use). It also happens in the sim, but I think that's due to a separate problem.
Comment by Douglas Valentine (Dwyloc) - Tuesday, 12 June 2007, 13:16 GMT+2
It also happens here every time on both my iPod nano and 5.5G iPod video.
I have not tested on my h140 as I have lent it to a friend,
but I can get it back to test on as well if it would help.

The problem also happens if I fast forward when pause as well as rewind on both DAPS.
Comment by Sarixe Avaliesz (sarixe) - Monday, 25 June 2007, 14:02 GMT+2
Confirmed on Cowon IAudio X5 with rockbox-iaudiox5-20070624
Comment by Michael Sevakis (MikeS) - Tuesday, 26 June 2007, 04:34 GMT+2
I think it's just a problem on SWCODEC in general and happens on all my players. I'm not sure why it happens but I did check and it doesn't seem to be a bug in the playback engine but likely in the pcm buffer perhaps due to some refilling triggering pcm playback. I plan on taking a closer look later.
Comment by Douglas Valentine (Dwyloc) - Wednesday, 04 July 2007, 14:54 GMT+2
After a lot of testing this morning looking for the commit that caused '"Pause on Headphone unplug" option no longer works on IPods.' (http://www.rockbox.org/tracker/task/7261) I think I have also found the commit that seems to causes this bug.

"19 May 19:30 Marcoen Hirschberg always reset the pcm_paused flag when stopping playback. fixes FS #7187"

As all builds tested before this commit do not show the rewind problem and all builds afterwards do.
Comment by Robert Keevil (obo) - Wednesday, 04 July 2007, 20:53 GMT+2
I don't know if this is the right approach, but the attached fixes the problem for me (plus  FS#7261 ) and as far as I can tell doesn't break the fix for  FS#7187 
   20070704-ffrw.patch (1.7 KiB) 
 firmware/pcm_playback.c                              |    3 ++-
 firmware/target/arm/s3c2440/gigabeat-fx/pcm-meg-fx.c |    3 ++-
 firmware/target/arm/pcm-pp.c                         |    3 ++-
 firmware/target/coldfire/pcm-coldfire.c              |    3 ++-
 4 files changed, 8 insertions(+), 4 deletions(-)

Comment by Steve Bavin (pondlife) - Wednesday, 04 July 2007, 21:02 GMT+2
Did you mean "pcm_paused = audio_status();" in playback.c? I've not tried the patch yet, just looked at it.

I'd think that it shouldn't be necessary to actually test andr modify the pause status like this; can it not be regarded as independent of the stop/play status? There would need to be a different fix for  FS#7107 , of course, so as not to assume that the two are tied in.
Comment by Michael Sevakis (MikeS) - Wednesday, 04 July 2007, 21:40 GMT+2
I'm right in the middle of redoing the pcm layer a bit for all targets to be atomic with locks against the audio interrupt and to give the pcm buffer its own play/paused state. This patch definitely isn't the right approach for a permanent fix since the problem really lies deeply in the playback.c behavior (yes, I take back the above comment in part) and the way it tracks its state in combination with some kludges in pcm. The pcm layer cannot concern itself with the audio status since it's something that can be used independently of core playback.
Comment by Douglas Valentine (Dwyloc) - Thursday, 05 July 2007, 11:23 GMT+2
I can confirm that the patch seems to fix both bugs on my iPod nano and video 5.5G.
Comment by Steve Bavin (pondlife) - Thursday, 05 July 2007, 11:28 GMT+2
That playback code really should read "pcm_paused = false;". audio_status() doesn't return a bool for starters.
Comment by Robert Keevil (obo) - Thursday, 05 July 2007, 11:41 GMT+2
Sorry, yes, it should. The change to pcm_playback.c was a remnant of me playing around, it should have been the same change as the other files. But as Michael said, it's not the right way of fixing it anyway.
Comment by Linus Nielsen Feltzing (linusnielsen) - Thursday, 05 July 2007, 12:00 GMT+2
However, we could fix it with this patch until Michael does it the right way.

Loading...