Update lincheck to 2.5 #1520
Merged
Update lincheck to 2.5 #1520
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ndkoval
force-pushed
the
new-lincheck
branch
4 times, most recently
from
f94049c
to
09f29b8
Compare
|
|
ndkoval
force-pushed
the
new-lincheck
branch
4 times, most recently
from
64801b5
to
66084f5
Compare
elizarov
suggested changes
Sep 12, 2019
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeListLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/internal/SegmentQueueLCStressTest.kt
Outdated
Show resolved
Hide resolved
ndkoval
force-pushed
the
new-lincheck
branch
from
c95162f
to
3ae9a93
Compare
elizarov
suggested changes
Sep 21, 2019
There was a problem hiding this comment.
Also note that it fails on tests. See https://teamcity.jetbrains.com/viewLog.html?buildId=2530509&buildTypeId=KotlinTools_KotlinxCoroutines_BuildAggregated
on ChannelsLCStressTest.test[ArrayChannel(1)]:
= Invalid execution results: =
Parallel part:
| receiveViaSelect(): suspended | offer(2): true | poll(): null |
| | poll(): 2 | sendViaSelect(1): void |
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeTaskQueueLCStressTest.kt
Outdated
Show resolved
Hide resolved
elizarov
reviewed
Sep 27, 2019
ndkoval
force-pushed
the
new-lincheck
branch
from
aca59e7
to
8b7984c
Compare
ndkoval
force-pushed
the
new-lincheck
branch
6 times, most recently
from
7bc99f6
to
d089b64
Compare
elizarov
suggested changes
Oct 3, 2019
kotlinx-coroutines-core/jvm/test/linearizability/ChannelsLCStressTest.kt
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeTaskQueueLCStressTest.kt
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeTaskQueueLCStressTest.kt
Outdated
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeListLCStressTest.kt
Show resolved
Hide resolved
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeListLCStressTest.kt
Show resolved
Hide resolved
elizarov
reviewed
Oct 3, 2019
kotlinx-coroutines-core/jvm/test/linearizability/LockFreeListLCStressTest.kt
Show resolved
Hide resolved
ndkoval
force-pushed
the
new-lincheck
branch
3 times, most recently
from
aec3e3b
to
caee8d5
Compare
|
Still fails on tests. |
|
I also added a sequential channel implementation |
ndkoval
force-pushed
the
new-lincheck
branch
2 times, most recently
from
9ceec6d
to
f9c9170
Compare
elizarov
suggested changes
Nov 14, 2019
There was a problem hiding this comment.
Stress tests are still too slow. On Mac they take ~1h40m each. That's too much. In develop branch each stress test takes ~4m. Please, tune options to keep stress tests under 5m each.
kotlinx.coroutines.linearizability.LockFreeListLCStressTest (1h:40m:15s)
kotlinx.coroutines.linearizability.SegmentQueueLCStressTest (1h:40m:55s)
kotlinx.coroutines.linearizability.MCLockFreeTaskQueueWithRemoveLCStressTest (1h:38m:13s)
kotlinx.coroutines.linearizability.MCLockFreeTaskQueueWithRemoveLCStressTest.testWithRemoveForQuiescentConsistency (1h:38m:13s)
The build kotlinx-coroutines::Nightly Stress (Mac) #1645 {buildId=2630796} has been running for more than 480 minutes. Terminating...
1d47511
to
27fc5ba
Compare
ndkoval
force-pushed
the
new-lincheck
branch
2 times, most recently
from
8bd23c4
to
769078b
Compare
ndkoval
force-pushed
the
new-lincheck
branch
4 times, most recently
from
564fca6
to
fba1900
Compare
ndkoval
force-pushed
the
new-lincheck
branch
from
fba1900
to
4212689
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update lincheck to 2.5 and re-write tests with
suspendoperations using it