Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[2018-08] [debugger] Deadlock while debugging a process #14392

Merged
merged 1 commit into from
May 10, 2019

Conversation

monojenkins
Copy link
Contributor

A deadlock happens when during a process_suspend there is a pending invoke and then when it's calling the invoke there is another call to process_suspend in the same thread.

Backport of #14368.

/cc @thaystg

…nvoke and then when it's calling the invoke there is another call to process_suspend in the same thread.
@monojenkins monojenkins added this to the 2018-08 (5.18.xx) milestone May 7, 2019
@lewurm
Copy link
Contributor

lewurm commented May 7, 2019

@marek-safar can we land this in 2018-08?

@alanmcgovern
Copy link
Member

Is there a pkg i can download to test with on mac?

@akoeplinger
Copy link
Member

@monojenkins build pkg

@thaystg
Copy link
Contributor

thaystg commented May 8, 2019

@monojenkins build failed

@thaystg
Copy link
Contributor

thaystg commented May 8, 2019

@alanmcgovern , the package is ready!! :)

@alanmcgovern
Copy link
Member

For the first time in... i have no idea how long... i was able to have multiple debug sessions and not see the infamous "The runtime has deadlocked" dialog box in VSMac. This is amazing.

/cc @Therzok : I belive you see these deadlocks on a daily basis too, so please check out this build. It's literally made it possible to debug again!

@Therzok
Copy link
Contributor

Therzok commented May 9, 2019

Yes, it is a world of difference!

@marek-safar marek-safar merged commit 6843da5 into mono:2018-08 May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants