AutoResetEvent.WaitOne() 导致死锁

AutoResetEvent.WaitOne() cause deadlock

我正在编写一个带有临界区的应用程序。

而我决定使用AutoResetEvent来实现互斥。 这是代码

    public class MyViewModel
    {
        private AutoResetEvent lock = new AutoResetEvent(true);
        private aync Task CriticalRegion()
        {
            Dosomething();
        }


        public async Task Button_Click()
        {
            Debug.WriteLine("Entering Button_Click");
            lock.WaitOne();
            try
            {
                await CriticalRegion();
            }
            finally
            {
                lock.Set();
                Debug.WriteLine("Leaving Button_Click");

            }
        }

    }

我有一个按钮,其点击事件调用 Button_Click() 方法

工作正常。但是,如果我足够快地在第一次调用 Button_Click() 完成之前再次单击该按钮,整个应用程序将停止响应。

在调试中 window 我发现了这样的东西

Entering Button_Click
Entering Button_Click

看起来该方法从未完成。

我挣扎了一下,发现如果我把 lock.WaitOne(); 改成

   if (!sync.WaitOne(TimeSpan.FromSeconds(1)))
   {
       return;
   }

在这种情况下,我的应用程序能够避免死锁,但我不知道为什么会这样。

我只从我的 OS 课程和 C# 中的 asyncawait 模式中了解 IPC,我对 .Net 世界中的线程不太熟悉.

我真的很想了解幕后到底发生了什么。 感谢您的回复 ;)

AutoResetEvent.WaitOne() will block infinitely until you call AutoResetEvent.Set(),除了 WaitOne() 调用之后,你似乎从未做过。

引用 AutoResetEvent.WaitOne() 文档:

Blocks the current thread until the current WaitHandle receives a signal.

你有一个死锁,因为 WaitOne 阻塞了主线程(按钮点击处理程序在主线程上执行),而你在调用 await 时没有调用 ConfigureAwait(false) ,这意味着它会尝试 运行 主线程上 await 之后的代码,即使它被阻塞,也会导致死锁。

我建议阅读 this post 以更全面地解释死锁情况。

对于你的代码,我建议把锁放得更深,可能在异步任务中,并尝试使用更合适的锁定模式,最好是 lock statement,因为使用 Event正如汉斯在评论中所说,对象对于互斥来说很尴尬。