Bug 8300 - Controller repeatedly (every 30 secs) queries for Cover art & mp3
: Controller repeatedly (every 30 secs) queries for Cover art & mp3
Status: RESOLVED FIXED
Product: SqueezePlay
Classification: Unclassified
Component: UI
: unspecified
: PC Windows Server 2003
: P2 normal (vote)
: 7.4.x
Assigned To: Ben Klaas
:
Depends on: 10064
Blocks: 9102
  Show dependency treegraph
 
Reported: 2008-06-01 19:36 UTC by Jonathan Carroll
Modified: 2009-12-23 12:05 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
Process monitor Log (46.66 KB, application/octet-stream)
2008-06-01 19:36 UTC, Jonathan Carroll
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Carroll 2008-06-01 19:36:21 UTC
Created attachment 3391 [details]
Process monitor Log

While trying to determine why my SqueezeCenter machine seemed to always have the hard disk going, I ran Process Monitor to have a look at what processes were accessing the disk.

I noticed that SqueezeCenter is querying for the same files over and over again every 30 seconds -- even when nothing is playing.

After turning off all of my receivers it turns out that the controller (I think) is causing this. Whatever is in the current playlist on the controller (and note, this is when the controller is either paused or stopped) determines the activity on the SC7 machine's disk.

Either turning off the controller or clearing the playlist on the controller stops the activity.

Is this behavior by design?

I am including a log in text format that demonstrates two passes (2nd pattern is 30 seconds after first and once again, the controller is stopped)

Regards,

Jonathan
Comment 1 James Richardson 2008-06-02 11:38:55 UTC
QA to investigate
Comment 2 Ben Klaas 2008-06-13 08:43:54 UTC
bump to QA for investigation. target for 7.1
Comment 3 Chris Owens 2008-06-19 10:50:34 UTC
This is a reminder that this bug is to QA for 7.1.  Please review these bugs!
Comment 4 Frode Lundgren 2008-06-25 13:25:02 UTC
I just wanted to confirm that I see the same behavior when using my SqueezeCenter 7.0.1-19705 with my Logitech Duet. It seems like as long as there are songs on the current playlist (Now Playing), this will occur. With my setup (playing FLAC files, not MP3s), I don't see the scan for album art, though.

In my case, the problem does not stop even if I turn off both the player _and_ the controller. The only way to stop the activity is to turn on the controller and the player and then clear the playlist. Please let me know if there's any additional information I can provide to help you debug.

Is this something you expect to address for 7.1?

Thanks!

- Frode
Comment 5 Ross Levine 2008-07-14 16:00:49 UTC
Jonathan,

Could you please let me know where your log came from? I can't seem to read it. Is it a Windows specific log? 
Comment 6 Jonathan Carroll 2008-07-15 09:28:23 UTC
It is a zip file with a plain text file inside. Is it the zip file you are having trouble opening or the text file you can't view?
Comment 7 KDF 2008-07-16 18:40:46 UTC
changing mime type helps.  it's a zip file so now it should be downloadable and the text file contained is readable (no idea what created the text however)

this is probably the now playing query, which checks for new data.  perhaps a carry-over of the web ui which used to refresh every 30s while playing to keep up with changing track info.  Controller could probably do better by listening to track change events if not doing that already.
Comment 8 Richard Titmuss 2008-07-24 04:35:14 UTC
Ben, could you take a look at this for 7.2.
Comment 9 Ben Klaas 2008-10-28 18:15:42 UTC
Flagging this as a high priority bug for 7.3
Comment 10 Ben Klaas 2008-11-14 12:13:10 UTC
I changed the subscription for playerstatus from subscribe:30 to subscribe:0. Now playerstatus messages will only be generated and sent from SC when triggered by changes (track changes, playlist modified, pause/stop hit, etc.).

I didn't setup the subscription to run automatically every 30 seconds, so I don't know what the reasoning for that was. I spent some time testing it today though, and behavior appears to be fine with the change.

Moving to FIXED
Comment 11 Jonathan Carroll 2008-11-15 19:14:29 UTC
Good info -- where do I change this setting in the UI?
Comment 12 Ben Klaas 2008-11-16 07:14:04 UTC
it's not a setting. controller firmware >= r3376 will have the change.

Comment 13 Ben Klaas 2008-11-21 12:08:51 UTC
because of bug#10064, this fix had to be reverted.
Comment 14 James Richardson 2008-12-19 08:01:38 UTC
Changing target to next release
Comment 15 Ben Klaas 2009-01-05 10:13:14 UTC
with only these exceptions, my bug list is being retargetted for 8.0

for 7.3.2 (very soon):
https://bugs-archive.lyrion.org/show_bug.cgi?id=7186 Browse albums/artists/genres/  years should remember previous state 

for 7.3.3 (later, but before 8.0):
https://bugs-archive.lyrion.org/show_bug.cgi?id=7139 Add editable "Network Settings" page

https://bugs-archive.lyrion.org/show_bug.cgi?id=7502 Add setup mode to set Static IPs (even if DHCP is present)

https://bugs-archive.lyrion.org/show_bug.cgi?id=7829 Jive menu broken if BLOCK_SIZE == list count + 1 and "Pla...

https://bugs-archive.lyrion.org/show_bug.cgi?id=8878 Party Mode

https://bugs-archive.lyrion.org/show_bug.cgi?id=10064 SqueezeCenter doesn't always auto-generate playerstatus w... (which will fix 8300)

https://bugs-archive.lyrion.org/show_bug.cgi?id=6334 add applet for network diagnostics (currently assigned to Tom)

Comment 16 Blackketter Dean 2009-07-22 08:39:12 UTC
Moving to the product SqueezePlay because this bug appears to apply to any player based on that application code.  Feel free to move it back if it's specific to the single original product.
Comment 17 Richard Titmuss 2009-07-27 01:13:15 UTC
Reset priority before triage.
Comment 18 Ben Klaas 2009-08-03 06:41:11 UTC
after 10064 is fixed, this will be a trivial fix.
Comment 19 Ben Klaas 2009-08-26 07:52:38 UTC
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Comment 20 SVN Bot 2009-12-23 12:05:15 UTC
 == Auto-comment from SVN commit #8262 to the jive repo by bklaas ==
 == https://svn.slimdevices.com/jive?view=revision&revision=8262 ==

Fixed Bug: 8300
Fixed Bug: 10064
Description: move playerstatus subscription to a "full push" model rather than push + 30 second poll

the danger here is if playerstatus isn't auto-generated when it should be (bug 10064) and instead of refreshing withing 30 seconds, it's stale until the next notification auto-generate, which could be much longer.

It's time to quit evading this bug though-- if 10064 still is out there, let's flush out those bugs and get them fixed rather than take this defensive stand that makes the network chatter much higher than it should be.