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

Add sleep in hijacking test #41340

Merged
merged 1 commit into from
Aug 25, 2020
Merged

Conversation

jakobbotsch
Copy link
Member

Calling GC.Collect in a tight loop is unnecessary for the test and
causing timeouts in CI.

Fixes #40916

I have verified that this test still catches the regression if the following is commented:

#ifdef FEATURE_HIJACK
if ((m_State & TS_Hijacked) && (retAddrSlot == m_ppvHJRetAddrPtr))
{
return m_pvHJRetAddr;
}
#endif

@Dotnet-GitSync-Bot
Copy link
Collaborator

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@jakobbotsch jakobbotsch changed the title Add sleep in hijacking thread Add sleep in hijacking test Aug 25, 2020
Calling GC.Collect in a tight loop is unnecessary for the test and
causing timeouts in CI.

Fixes dotnet#40916
@jkotas jkotas added area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI test-bug Problem in test source code (most likely) labels Aug 25, 2020
Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Thank you!

@jkotas jkotas merged commit 8d987c6 into dotnet:master Aug 25, 2020
@jkotas
Copy link
Member

jkotas commented Aug 25, 2020

/backport to release/5.0

@github-actions
Copy link
Contributor

Started backporting to release/5.0: https://github.com/dotnet/runtime/actions/runs/224428212

@jakobbotsch jakobbotsch deleted the hijacking-sleep branch August 26, 2020 09:29
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI test-bug Problem in test source code (most likely)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test failure: JIT\\Methodical\\tailcall_v4\\hijacking\\hijacking.cmd
3 participants