RFR 8227866: Shenandoah: Split weak root processing and class unloading in parallel cleaning task

[Zhengyu Gu]

Hi Thomas,

Although, Shenandoah no long uses ParallelCleaningTask, it still uses 
tasks defined in it, such as StringDedupCleaningTask, 
CodeCacheUnloadingTask and etc. So, it should still remain in "shared".

What you think?

Thanks,

-Zhengyu



On 7/18/19 6:02 AM, Thomas Schatzl wrote:
> Hi,
> 
> On Wed, 2019-07-17 at 14:13 -0400, Zhengyu Gu wrote:
>>
>> On 7/17/19 2:03 PM, Thomas Schatzl wrote:
>>> Hi,
>>>
>>> On Wed, 2019-07-17 at 13:41 -0400, Zhengyu Gu wrote:
>>>> Currently, Shenandoah uses shared ParallelCleaningTask to perform
>>>> STW cleaning task, the cleaning task performs class unloading
>>>> along
>>>
>>>     since the ParallelCleaningTask has originally moved from g1 code
>>> to shared to allow use with Shenandoah, now that the code is not
>>> used by Shenandoah any more, please move ParallelCleaningTask back
>>> to G1 specific code.
>>
>> Sure. Filed JDK-8227927.
> 
> I would think that it is appropriate to include the move to G1 code
> part of this change, but I am okay if you want to do it separately.
> 
> Thanks,
>    Thomas
> 
>