Bug 65 - Windows service should run as high priority
: Windows service should run as high priority
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Windows Service
: unspecified
: PC Windows XP
: P2 normal (vote)
: Future
Assigned To: Blackketter Dean
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2003-12-19 21:42 UTC by Blackketter Dean
Modified: 2008-12-15 13:05 UTC (History)
1 user (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Blackketter Dean 2003-12-19 21:42:31 UTC
The windows service's priority should be upped to improve overall performance.
Comment 1 Vidur Apparao 2004-06-06 20:12:29 UTC
Suggested code from Paul Walker:

 eval{ require Win32::Process };
 if( $@ )
   {
     print OUT $@;
   }
 print OUT "Got WIN32 in $$\n";
 if( ! Win32::Process::Open( $procobj, $$, 0 ) )
   {
     print OUT "OPEN " . $! . "\n";
   }
 
 $procobj->GetPriorityClass( $priclass );
 print OUT  $priclass . "\n";
 
 if( ! $procobj->SetPriorityClass( 128 ) )
   {
     print OUT $! . "\n";
   }
 $procobj->GetPriorityClass( $priclass );
 print OUT  $priclass .  "\n";    
Comment 2 Dan Sully 2005-02-05 10:52:08 UTC
Do we want to add this for 6.0?
Comment 3 Blackketter Dean 2005-03-11 13:21:37 UTC
Will look at this again after 6.0
Comment 4 Dan Sully 2006-08-30 03:39:03 UTC
Chris Key checked in code to set the priority of both the server and scanner via Settings.
Comment 5 Blackketter Dean 2006-08-30 10:44:12 UTC
Does this patch change the default priority on windows?
Comment 6 Dan Sully 2006-08-30 12:02:40 UTC
No. Only the option to.

At one point I changed the priority of the scanner, and people complained. Chris' patch let's the choose.
Comment 7 Blackketter Dean 2006-08-30 12:49:31 UTC
Subject: Re:  Windows service should run as high priority


On Aug 30, 2006, at 12:02 PM, Slim Devices Bugzilla wrote:
> At one point I changed the priority of the scanner, and people  
> complained.
> Chris' patch let's the choose.
Ok.  I was more thinking of having the main process bumped up, not  
the scanner.  Bumping up the scanner will make a lot of people sad.

Comment 8 James Richardson 2008-12-15 13:05:17 UTC
This bug appears to have been fixed in the latest release!

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.

Make sure to include the version number of the software you are seeing the error with.