Skip to content

Revert "Disable PerfFileStream while we wait for dotnet/runtime#67545 to be fixed. (#2371) - #2381

Merged
adamsitnik merged 1 commit into
dotnet:mainfrom
kouvel:DeadlockFixPerf
Apr 20, 2022
Merged

Revert "Disable PerfFileStream while we wait for dotnet/runtime#67545 to be fixed. (#2371)#2381
adamsitnik merged 1 commit into
dotnet:mainfrom
kouvel:DeadlockFixPerf

Conversation

@kouvel

@kouvel kouvel commented Apr 18, 2022

Copy link
Copy Markdown
Contributor

@kouvel kouvel self-assigned this Apr 18, 2022
@adamsitnik

Copy link
Copy Markdown
Member

@kouvel big thanks for the PR (and also the runtime bug fix!)

I am going to apply "NO MERGE" label for now and remove it once the benchmarks can be re-enabled:

@adamsitnik adamsitnik self-assigned this Apr 19, 2022
@kouvel

kouvel commented Apr 19, 2022

Copy link
Copy Markdown
Contributor Author

The dependency dotnet/runtime#68171 has been merged now

@kouvel

kouvel commented Apr 19, 2022

Copy link
Copy Markdown
Contributor Author

Just before the cutoff time for preview 4 though, I'll have to check the snap commit after the snap to see if it got included

@kouvel

kouvel commented Apr 19, 2022

Copy link
Copy Markdown
Contributor Author

dotnet/runtime#68171 made it into the preview 4 snap, so this PR should be unblocked

@adamsitnik adamsitnik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, :shipit: !

@adamsitnik
adamsitnik merged commit 8f9ed63 into dotnet:main Apr 20, 2022
@kouvel
kouvel deleted the DeadlockFixPerf branch April 20, 2022 17:13
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.

2 participants