RFR(XS) 8191924: Adjust DelegatingClassLoader's metadata space sizing algorithm

[Thomas Stüfe]

On Tue, Mar 13, 2018 at 7:09 PM, Zhengyu Gu <zgu at redhat.com> wrote:

> Hi Thomas,
>
>
>>
>> Looks fine. Good results :)
>>
>>
> Thanks for the review.
>
> Should we assert the Metaspace lock somewhere?
>>
>> I think we should add aggregated counters in SpaceManager by chunk type
>> to not having to walk-and-count each time, but that is for another day. I
>> have a patch like that somewhere in my backlog.
>>
>
> Let's do a round of cleanup separately. I do believe we can remove
> xxx_slow() version.
>
>
Yes, I think so too.

I have a working patch for the ChunkManager, and a half-done patch for
SpaceManager. May post this or next week depending on how much time I have
and on how fast my other cleanup patches go upstream.

Best, Thomas



> -Zhengyu
>
>
>
>> ..Thomas
>>
>>