RFR (S): 8155232: Augment Workgang to run task with a given number of threads

Mikael Gerdin mikael.gerdin at oracle.com
Thu Apr 28 14:13:09 UTC 2016


Hi Thomas,

On 2016-04-28 14:14, Thomas Schatzl wrote:
> Hi Stefan,
>
> On Thu, 2016-04-28 at 13:58 +0200, Stefan Karlsson wrote:
>> Hi Thomas,
>>
>> This change looks fine.
>>
>> I have a slight reservation about using the word "thread" in this
>> context, since the tasks might be executed on a lower number of
>> threads.
>> The surrounding code uses the word "workers" instead of "threads".
>> Maybe
>> your function code be named run_task_on_workers, or maybe even use
>> an
>> overloaded run_task function?
>
>    made an overloaded run_task(). Sorry.
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8155232/webrev.1/

Looks good.

/Mikael

>
> Also fixed up all dependent changesets in place due to the triviality
> of this name change.
>
> Thanks,
>    Thomas
>


More information about the hotspot-gc-dev mailing list