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

BlockHound integration #1821

Merged
merged 15 commits into from
Mar 16, 2020
Merged

BlockHound integration #1821

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
edc388c
BlockHound integration POC
dkhalanskyjb Feb 3, 2020
ef33b34
Do not shadow ByteBuddy agent
dkhalanskyjb Feb 6, 2020
69f76dc
Fix tests not being run with the right dispatcher
dkhalanskyjb Feb 6, 2020
46bbf2a
Clean the stack traces from BlockHound artifacts
dkhalanskyjb Feb 6, 2020
d1e4798
Intermediate value
dkhalanskyjb Feb 18, 2020
5058081
Improve handling of stacktrace tests on JDK 8
dkhalanskyjb Feb 19, 2020
efb4720
Implement BlockHound integration properly
dkhalanskyjb Feb 19, 2020
6e5fe92
Up the BlockHound version
dkhalanskyjb Feb 19, 2020
7fc4a90
Fixes
dkhalanskyjb Mar 13, 2020
3fc6abc
Improve build configuration of integration tests
dkhalanskyjb Mar 16, 2020
76ab90f
Add an integration test for coroutine debugger java agent
dkhalanskyjb Mar 16, 2020
e16face
Document BlockHound integration better
dkhalanskyjb Mar 16, 2020
15b54c9
Do not exclude the ByteBuddy agent from shading
dkhalanskyjb Mar 16, 2020
e685384
Documentation improvement
qwwdfsad Mar 16, 2020
20cc2ff
Update kotlinx-coroutines-debug/README.md
qwwdfsad Mar 16, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Document BlockHound integration better
  • Loading branch information
@dkhalanskyjb
dkhalanskyjb committed Mar 16, 2020
commit e16face08c7c3c548b758ce1d3d23131c770bc35
12 changes: 12 additions & 0 deletions kotlinx-coroutines-core/jvm/src/scheduling/CoroutineScheduler.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -952,10 +952,22 @@ internal class CoroutineScheduler(
}
}

/**
* Checks if the thread is part of a thread pool that supports coroutines.
*
* This function is needed for integration with BlockHound and accessed by violating the
* visibility modifier.
*/
@Suppress("UNUSED")
@JvmName("isSchedulerWorker")
internal fun isSchedulerWorker(thread: Thread) = thread is CoroutineScheduler.Worker
qwwdfsad marked this conversation as resolved.
Show resolved Hide resolved

/**
* Checks if the thread is running a CPU-bound task.
*
* This function is needed for integration with BlockHound and accessed by violating the
* visibility modifier.
*/
@Suppress("UNUSED")
@JvmName("mayNotBlock")
internal fun mayNotBlock(thread: Thread) = thread is CoroutineScheduler.Worker &&
Expand Down
3 changes: 3 additions & 0 deletions kotlinx-coroutines-debug/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,9 @@ suspension stacktraces.
Additionally, it is possible to process the list of such coroutines via [DebugProbes.dumpCoroutinesInfo] or dump isolated parts
of coroutines hierarchy referenced by a [Job] or [CoroutineScope] instances using [DebugProbes.printJob] and [DebugProbes.printScope] respectively.

Additionally, this module provides a [BlockHound](https://github.com/reactor/BlockHound) service
that detects when a blocking operation was called in a coroutine context that prohibits it.

### Using in your project

Add `kotlinx-coroutines-debug` to your project test dependencies:
Expand Down