Hi Marc,

 
thanks for your feedback. Unfortunately the patches are not on github, but on 
my harddisk, so I think a pull request will not work.

 
Sorry, I don't have the time to migrate this to github at the moment (also this 
might be a good idea in the long term), but Bernhard just send me a message 
that he will create the pull request in behave of me.

 
Gerald

 
 
 
From: Marc-André Moreau [mailto:marcandre.mor...@gmail.com] 
Sent: Monday, October 01, 2012 3:27 PM
To: Gerald Richter - ECOS
Cc: freerdp-devel@lists.sourceforge.net
Subject: Re: [Freerdp-devel] PATCH: Fix 100% CPU load on Windows

 
Hi,

 
It is better to provide them on github with pull requests

 
Best regards,

- Marc-Andre

On Mon, Oct 1, 2012 at 7:48 AM, <rich...@ecos.de <mailto:rich...@ecos.de> > 
wrote:

My attached patches seems to be removed by the list software. So how is the 
best way to provide the patches?

Gerald


> -----Original Message-----
> From: Gerald Richter - ECOS
> Sent: Monday, October 01, 2012 1:44 PM
> To: freerdp-devel@lists.sourceforge.net 
> <mailto:freerdp-devel@lists.sourceforge.net> 
> Subject: [Freerdp-devel] PATCH: Fix 100% CPU load on Windows
>
> Hi,
>
> the current git master cause 100% CPU load on windows, because
> WaitForMultipleObjects never waits, because the WSAEvent is always
> signaled.
>
> Also according to MS docs it is not necessary to reset a WSAEvent, but
> without the ResetEvent the event is always signaled... Adding a ResetEvent
> was the only solution I found, to solve the high CPU load.
>
> In addition I changed the timeout for WaitForMultipleObjects from 1ms to
> 10ms, to lower the CPU load.
>
> Gerald
>
>
>
>
> ------------------------------------------------------------------------------
> Got visibility?
> Most devs has no idea what their production app looks like.
> Find out how fast your code is with AppDynamics Lite.
> http://ad.doubleclick.net/clk;262219671;13503038;y 
> <http://ad.doubleclick.net/clk;262219671;13503038;y> ?
> http://info.appdynamics.com/FreeJavaPerformanceDownload.html 
> <http://info.appdynamics.com/FreeJavaPerformanceDownload.html> 
> _______________________________________________
> Freerdp-devel mailing list
> Freerdp-devel@lists.sourceforge.net 
> <mailto:Freerdp-devel@lists.sourceforge.net> 
> https://lists.sourceforge.net/lists/listinfo/freerdp-devel


------------------------------------------------------------------------------
Got visibility?
Most devs has no idea what their production app looks like.
Find out how fast your code is with AppDynamics Lite.
http://ad.doubleclick.net/clk;262219671;13503038;y 
<http://ad.doubleclick.net/clk;262219671;13503038;y> ?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html 
<http://info.appdynamics.com/FreeJavaPerformanceDownload.html> 
_______________________________________________
Freerdp-devel mailing list
Freerdp-devel@lists.sourceforge.net 
<mailto:Freerdp-devel@lists.sourceforge.net> 
https://lists.sourceforge.net/lists/listinfo/freerdp-devel

 
------------------------------------------------------------------------------
Got visibility?
Most devs has no idea what their production app looks like.
Find out how fast your code is with AppDynamics Lite.
http://ad.doubleclick.net/clk;262219671;13503038;y?
http://info.appdynamics.com/FreeJavaPerformanceDownload.html
_______________________________________________
Freerdp-devel mailing list
Freerdp-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freerdp-devel

Reply via email to