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