Hi, I finally decided to move my repo to github (https://github.com/richterger/FreeRDP/commits/master) and make a pull request (https://github.com/FreeRDP/FreeRDP/pull/748) for my patches.
I looking forward to your comments Gerald > -----Original Message----- > From: Gerald Richter - ECOS > Sent: Monday, October 01, 2012 3:46 PM > To: Marc-André Moreau > Cc: freerdp-devel@lists.sourceforge.net > Subject: Re: [Freerdp-devel] PATCH: Fix 100% CPU load on Windows > > 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- > de...@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 ------------------------------------------------------------------------------ Don't let slow site performance ruin your business. Deploy New Relic APM Deploy New Relic app performance management and know exactly what is happening inside your Ruby, Python, PHP, Java, and .NET app Try New Relic at no cost today and get our sweet Data Nerd shirt too! http://p.sf.net/sfu/newrelic-dev2dev _______________________________________________ Freerdp-devel mailing list Freerdp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/freerdp-devel
