EventLoopScheduler SemaphoreFullException

[mattpodwysocki]

Copied from https://rx.codeplex.com/workitem/37

EventLoopScheduler.Schedule may throw SemaphoreFullException. It appears that it may be due to a couple of race conditions in the Run method:

1. Between the call to Wait and acquisition of the lock.
2. Dequeueing expired items in a loop, without calling Wait for each item.

Related discussion:
http://social.msdn.microsoft.com/Forums/en-US/rx/thread/223f2447-4cb9-47ce-be21-6fd0e5b45e30/?prof=required

[mattpodwysocki]

malayeri wrote Nov 5, 2013 at 5:00 PM

Dave, your proposed fix on the forum thread looks like it could work, but I'm a bit nervous about changing any kind of concurrency code. Would you be able to submit a pull request, or alternatively, review any code changes I make?

Thanks!

[mattpodwysocki]

davedev wrote Nov 5, 2013 at 10:05 PM

Sure, I'll fix it and include a unit test. I'll make some time next weekend if that's alright.

[mattpodwysocki]

malayeri wrote Nov 5, 2013 at 10:55 PM

That would be great! Thanks! We plan to do a release of Rx around early December, so this timing would be perfect.

[mattpodwysocki]

davedev wrote Nov 10, 2013 at 5:40 PM

Submitted pull request:
https://rx.codeplex.com/SourceControl/network/forks/davedev/EventLoopSchedulerFix/contribution/5647