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

Add coroutine tests from deadlock test repo #531

Draft
wants to merge 31 commits into
base: run-with-lincheck-api-coroutines
Choose a base branch
from
Draft

Add coroutine tests from deadlock test repo #531

wants to merge 31 commits into from
+29,382 −318

Conversation

bbrockbernd
Copy link
Collaborator

Closes #492

eupp and others added 26 commits January 31, 2025 13:06
@eupp
add thread pool test for runConcurrentTest
7478c93 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
add separate expected trace files for each jdk version for the thread…
e5f1d17 
… pool test

Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix Java lambda class filtering in test logs
74641a2 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
support ThreadPoolExecutor on newer versions of JDK
2fe5f02 
* On these JDK versions, `JavaLangAccess.start` API is used to start threads inside a thread pool.

Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
also instrument ThreadFlock jdk class, since it also uses `JavaLang…
5f8be04 
…Access.start`

Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fixes after rebase
8e17cfc 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
gpmc trace reporter: print function result if its trace is expanded
df09211 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix thread pool test
6027cd7 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix thread pool tests traces
ee7861a 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
avoid Java's lambdas in thread pool test
aa67f96 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
remove Java's lambda filtering (it does not work reliably anyway)
fd817a9 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
add a simple coroutines test for gpmc API
e708c8c 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fixes after rebase
a71ad8d 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
Allow ReplayModeLoopDetectorHelper to continue working after it exc…
619b8c4 
…eeds the replayed interleaving

Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
prepare coroutines trace output files for different JDK versions
b70cc73 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
check if the spin-loop replay was requested earlier
37d6430 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
support instrumentation of newly loaded classes (not only re-transfor…
e5ffdc0 
…med or re-defined classes)

Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
add DispatchedContinuation to the list of eagerly loaded classes as…
4a56a5d 
… a temporary workaround

Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
update coroutines trace representation tests expected output
42a13cf 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix bug on Java 8 due to KT-16727
4d1f63f 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
minor fix in BaseRunConcurrentRepresentationTest
2cbd91b 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix bug on Java 8 due to KT-16727 (patch)
da743e9 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix atomic_refs.txt expected trace output
c878347 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix custom_threads.txt expected trace output
437ef4b 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp
fix coroutines_jdk8.txt expected trace output
0545b97 
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@bbrockbernd
Add coroutine tests from deadlock test repo
263360a
@bbrockbernd bbrockbernd requested a review from eupp February 18, 2025 13:08
@bbrockbernd bbrockbernd linked an issue Feb 18, 2025 that may be closed by this pull request
Add more tests with Kotlin Coroutines #492
Open
@eupp eupp force-pushed the run-with-lincheck-api-coroutines branch from 0545b97 to ee44ac6 Compare February 20, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eupp eupp Awaiting requested review from eupp

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add more tests with Kotlin Coroutines
3 participants
@bbrockbernd @eupp @dmitrii-artuhov