Bug 16642 - Radio stream produce high CPU load in Touch/Nokia770 skin
: Radio stream produce high CPU load in Touch/Nokia770 skin
Status: RESOLVED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Skins
: 7.5.x
: All Other
: -- normal (vote)
: 7.5.1
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-10-29 08:22 UTC by John Wiesenmueller
Modified: 2010-10-29 13:52 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Wiesenmueller 2010-10-29 08:22:29 UTC
Playing radio stream produce high CPU load in the "actual title" and "actual playlist" view of the Touch or Nokia770 skin.

Related German forum thread is

http://forums.slimdevices.com/showthread.php?t=82844

Michael Herger noted a a high reload frequency on the status screen and changed following code in the latest nighly 7.5.2.

Index: server/HTML/Nokia770/status.js
===================================================================
--- server/HTML/Nokia770/status.js	(revision 31444)
+++ server/HTML/Nokia770/status.js	(working copy)
@@ -52,7 +52,7 @@
 }
 
 function setProgressBarWidth() {
-	if ( _progressAt >= _progressEnd) {
+	if ( _progressEnd > 0 && _progressAt >= _progressEnd) {
 		_progressAt = _progressEnd;
 		clearIntervalCall();
 		doRefresh();

I've changed the part manually in my 7.5.2 and it solved the problem in the "actual playing" view but cpu load is still high in the "actual playlist" view.

Latest nightly not tested yet.
Comment 1 SVN Bot 2010-10-29 13:52:18 UTC
 == Auto-comment from SVN commit #31477 to the slim repo by mherger ==
 == http://svn.slimdevices.com/slim?view=revision&revision=31477 ==

Fixed Bug: 16642
Description: sanity check timeout. If it's < 0, then reset to default 10s.

This change should fix the playlist view issue. Please let me know if that's still not good enough.