-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Introduce CoroutineDispatcher.limitedParallelism #2918
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
Changes from 5 commits
f0a265a
00122c5
ecd36dd
5fe0261
4b752fe
ee27e2b
eafefb4
274cdb4
ce0f3dd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,109 @@ | ||
/* | ||
* Copyright 2016-2021 JetBrains s.r.o. Use of this source code is governed by the Apache 2.0 license. | ||
*/ | ||
|
||
package kotlinx.coroutines.internal | ||
|
||
import kotlinx.coroutines.* | ||
import kotlin.coroutines.* | ||
import kotlin.jvm.* | ||
|
||
/** | ||
* The result of .limitedParallelism(x) call, dispatcher | ||
qwwdfsad marked this conversation as resolved.
Show resolved
Hide resolved
|
||
* that wraps the given dispatcher, but limits the parallelism level, while | ||
* trying to emulate fairness. | ||
*/ | ||
internal class LimitedDispatcher( | ||
private val dispatcher: CoroutineDispatcher, | ||
private val parallelism: Int | ||
) : CoroutineDispatcher(), Runnable, Delay by (dispatcher as? Delay ?: DefaultDelay) { | ||
qwwdfsad marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
@Volatile | ||
private var runningWorkers = 0 | ||
|
||
private val queue = LockFreeTaskQueue<Runnable>(singleConsumer = false) | ||
|
||
@ExperimentalCoroutinesApi | ||
override fun limitedParallelism(parallelism: Int): CoroutineDispatcher { | ||
parallelism.checkParallelism() | ||
if (parallelism >= this.parallelism) return this | ||
return super.limitedParallelism(parallelism) | ||
} | ||
|
||
override fun run() { | ||
var fairnessCounter = 0 | ||
while (true) { | ||
val task = queue.removeFirstOrNull() | ||
if (task != null) { | ||
try { | ||
task.run() | ||
} catch (e: Throwable) { | ||
handleCoroutineException(EmptyCoroutineContext, e) | ||
} | ||
// 16 is our out-of-thin-air constant to emulate fairness. Used in JS dispatchers as well | ||
if (++fairnessCounter >= 16 && dispatcher.isDispatchNeeded(EmptyCoroutineContext)) { | ||
// Do "yield" to let other views to execute their runnable as well | ||
// Note that we do not decrement 'runningWorkers' as we still committed to do our part of work | ||
dispatcher.dispatch(EmptyCoroutineContext, this) | ||
return | ||
} | ||
continue | ||
} | ||
|
||
@Suppress("CAST_NEVER_SUCCEEDS") | ||
synchronized(this as SynchronizedObject) { | ||
--runningWorkers | ||
if (queue.size == 0) return | ||
++runningWorkers | ||
fairnessCounter = 0 | ||
} | ||
} | ||
} | ||
|
||
override fun dispatch(context: CoroutineContext, block: Runnable) { | ||
dispatchInternal(block) { | ||
if (dispatcher.isDispatchNeeded(EmptyCoroutineContext)) { | ||
dispatcher.dispatch(EmptyCoroutineContext, this) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is it ok not to pass There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Do you mean There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In general, we do not have any reasonable contracts or use-cases to I've changed it to |
||
} else { | ||
run() | ||
} | ||
} | ||
} | ||
|
||
@InternalCoroutinesApi | ||
override fun dispatchYield(context: CoroutineContext, block: Runnable) { | ||
dispatchInternal(block) { | ||
dispatcher.dispatchYield(context, this) | ||
} | ||
} | ||
|
||
private inline fun dispatchInternal(block: Runnable, dispatch: () -> Unit) { | ||
// Add task to queue so running workers will be able to see that | ||
if (tryAdd(block)) return | ||
/* | ||
* Protect against the race when the number of workers is enough, | ||
* but one (because of synchronized serialization) attempts to complete, | ||
* and we just observed the number of running workers smaller than the actual | ||
* number (hit right between `--runningWorkers` and `++runningWorkers` in `run()`) | ||
*/ | ||
if (enoughWorkers()) return | ||
dispatch() | ||
} | ||
|
||
private fun enoughWorkers(): Boolean { | ||
qwwdfsad marked this conversation as resolved.
Show resolved
Hide resolved
|
||
@Suppress("CAST_NEVER_SUCCEEDS") | ||
synchronized(this as SynchronizedObject) { | ||
if (runningWorkers >= parallelism) return true | ||
++runningWorkers | ||
return false | ||
} | ||
} | ||
|
||
private fun tryAdd(block: Runnable): Boolean { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Maybe rename to There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sure. I've extracted the method to reduce the bytecode size due to |
||
queue.addLast(block) | ||
return runningWorkers >= parallelism | ||
} | ||
} | ||
|
||
// Save a few bytecode ops | ||
internal fun Int.checkParallelism() = require(this >= 1) { "Expected positive parallelism level, but got $this" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The IDE highlight
EXPERIMENTAL
as being never used. Can't this line be removed altogether?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IDE is wrong here, JMH's
@Param
applied to enum enumerates all enum values in the benchmark