TimerQueueTimer: potentially unnecessary volatile modifier on the private fields #43972
Description
runtime/src/libraries/System.Private.CoreLib/src/System/Threading/Timer.cs
Lines 425 to 426 in 3fca083
The field _canceled is only ever referenced inside of a lock, and _notifyWhenNoCallbacksRunning is only read outside of a lock after any concurrency should be possible. So unless there is some logic in another partial class TimerQueueTimer that makes it necessary, I don't see why volatile on these fields is needed.