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