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.

Sign up for GitHub

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

Jump to bottom

api.startstop and api.detach test failing: "ERROR: thread X should NOT be under DynamoRIO" #2246

Open
derekbruening opened this issue Feb 27, 2017 · 5 comments
Open

api.startstop and api.detach test failing: "ERROR: thread X should NOT be under DynamoRIO" #2246

derekbruening opened this issue Feb 27, 2017 · 5 comments
Labels
Bug-AppFail Component-Tests Hotlist-ContinuousIntegration

Comments

@derekbruening
Copy link
Contributor

Xref #2068

I've never seen this test fail like this:

https://ci.appveyor.com/project/DynamoRIO/dynamorio/build/1.0.214/job/hwk44b43swkv08sn

====> FAILURE in debug-internal-32 <====
debug-internal-32: 215 tests passed, **** 3 tests failed, but ignoring 2 for i2145: ****
	code_api|api.detach 

ERROR: thread 7 should NOT be under DynamoRIO
all done: 150000 iters
@derekbruening
Copy link
Contributor Author

Happened again:
https://ci.appveyor.com/project/DynamoRIO/dynamorio/build/1.0.245/job/ntkv2wdvtdric957

@derekbruening derekbruening mentioned this issue Apr 18, 2017
Merged
@fhahn
Copy link
Contributor

fhahn commented Apr 20, 2017

happened again:
https://ci.appveyor.com/project/DynamoRIO/dynamorio/build/1.0.504/job/ue0k7iqxlff3mw2u

@fhahn fhahn mentioned this issue Apr 20, 2017
Merged
@derekbruening derekbruening mentioned this issue Apr 23, 2017
Merged
@fhahn fhahn mentioned this issue Apr 25, 2017
Merged
@derekbruening derekbruening mentioned this issue May 2, 2017
Merged
@derekbruening derekbruening mentioned this issue May 16, 2017
Merged
@toshipiazza
Copy link
Contributor

happened again:
https://ci.appveyor.com/project/DynamoRIO/dynamorio/build/1.0.824

@egrimley egrimley mentioned this issue Jul 3, 2017
Merged
@derekbruening derekbruening mentioned this issue Jul 18, 2017
Merged
@egrimley egrimley mentioned this issue Jul 19, 2017
Merged
This was referenced Jul 20, 2017
Merged
Merged
This was referenced Aug 2, 2017
Merged
Merged
Closed
derekbruening added a commit that referenced this issue Aug 3, 2017
@derekbruening
i#2145 appveyor failures: add two tests to the ignore list
9c94c28 
Unfortunately two failures are happening often enough that I'm adding them
to the #2145 list here: 32-bit api.detach #2246 and 64-bit drmgr-test #1369.
@derekbruening derekbruening mentioned this issue Aug 3, 2017
Merged
derekbruening added a commit that referenced this issue Aug 3, 2017
@derekbruening
i#2145 appveyor failures: add two tests to the ignore list (#2589)
f286372 
Unfortunately two failures are happening often enough that I'm adding them
to the #2145 list here: 32-bit api.detach #2246 and 64-bit drmgr-test #1369.
@derekbruening derekbruening mentioned this issue Aug 5, 2017
Merged
This was referenced Oct 1, 2017
Merged
Merged
derekbruening added a commit that referenced this issue Oct 27, 2017
@derekbruening
i#2145 appveyor: mark api.detach and api.startstop as flaky
7875cb3 
Marks api.detach and api.startstop as flaky as they have been failing
occasionally on Appveyor.

Issue: #2093, #2246, #2145
@derekbruening derekbruening mentioned this issue Oct 27, 2017
Merged
derekbruening added a commit that referenced this issue Oct 27, 2017
@derekbruening
i#2145 appveyor: mark api.detach and api.startstop as flaky (#2671)
14b89d2 
Marks api.detach and api.startstop as flaky as they have been failing
occasionally on Appveyor.

Issue: #2093, #2246, #2145
fhahn pushed a commit that referenced this issue Dec 4, 2017
@derekbruening @fhahn
i#2145 appveyor: mark api.detach and api.startstop as flaky (#2671)
b66621c 
Marks api.detach and api.startstop as flaky as they have been failing
occasionally on Appveyor.

Issue: #2093, #2246, #2145
@hgreving2304
Copy link

happened again: https://ci.appveyor.com/project/DynamoRIO/dynamorio/builds/28604707

@derekbruening derekbruening mentioned this issue Dec 15, 2020
Closed
@derekbruening derekbruening changed the title api.detach test failing non-det api.detach test failing: "ERROR: thread X should NOT be under DynamoRIO" Dec 15, 2020
@derekbruening derekbruening changed the title api.detach test failing: "ERROR: thread X should NOT be under DynamoRIO" api.startstop and api.detach test failing: "ERROR: thread X should NOT be under DynamoRIO" Dec 15, 2020
@derekbruening
Copy link
Contributor Author

This is happening in every run on GA Server2016 now. For now these are on the ignore list. But given that this has happened on Linux (#2068, folded into here) we shouldn't consider this a Windows-only issue.

This was referenced Dec 15, 2020
Open
Closed
derekbruening added a commit that referenced this issue Dec 15, 2020
@derekbruening
i#4131 Windows CI: Update ignore list bug numbers
e45e726 
Updates the Windows CI ignore list bug numbers to reflect the
newly-filed individual issues.

Re-adds client.drwrap-test-detach to the ignore list for 32-bit.

Issue: #2246, #4616, #4617, #4618, #4619, #4621, #4622, #4131, #4058
@derekbruening derekbruening mentioned this issue Dec 15, 2020
Merged
derekbruening added a commit that referenced this issue Dec 15, 2020
@derekbruening
i#4131 Windows CI: Update ignore list bug numbers (#4623)
11e2567 
Updates the Windows CI ignore list bug numbers to reflect the
newly-filed individual issues.

Re-adds client.drwrap-test-detach to the ignore list for 32-bit,
and adds histogram-offline to 64-bit (both were already there for
the other bitwidth).

Issue: #2246, #4616, #4617, #4618, #4619, #4621, #4622, #4131, #4058
@derekbruening derekbruening mentioned this issue Mar 13, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug-AppFail Component-Tests Hotlist-ContinuousIntegration
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@fhahn @toshipiazza @derekbruening @hgreving2304