RFR: 8290025: Remove the Sweeper [v10]
Erik Österlund
eosterlund at openjdk.org
Tue Aug 16 11:55:24 UTC 2022
On Mon, 15 Aug 2022 20:42:52 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:
>>> Hi Erik,
>>>
>>> I'll take a closer look next week, and by then we will have more test runs to get a better picture.
>>>
>>> Meanwhile, could you please merge the change to the current head? This would help us with our ideosyncratic CI. Thank you!
>>>
>>> ..Thomas
>>
>> Thank you for having a look at this. I merged to the latest head, hope that helps!
>
>> Hi Erik (@fisk),
>>
>> We have a real-life workload with 20k - 50k compiled methods. I'd like to test the patch with it. The workload uses jdk17.
>>
>> How difficult is to port the patch to 17?
>
> It's doable but a little bit tricky as the nmethod entry barriers for G1 were inteoduced with loom, which didn't exist in 17.
> > > Hi Erik (@fisk),
> > > We have a real-life workload with 20k - 50k compiled methods. I'd like to test the patch with it. The workload uses jdk17.
> > > How difficult is to port the patch to 17?
> >
> >
> > It's doable but a little bit tricky as the nmethod entry barriers for G1 were inteoduced with loom, which didn't exist in 17.
>
> I see. Would 18 or 19 be more suitable?
Yes, 19 would be more suitable. The easiest thing might be to take the current commit and build it.
-------------
PR: https://git.openjdk.org/jdk/pull/9741
More information about the hotspot-dev
mailing list