RFR: 8253779: Amalloc may be wasting space by overaligning [v2]
Thomas Stuefe
stuefe at openjdk.java.net
Fri Jul 9 15:02:54 UTC 2021
On Fri, 9 Jul 2021 14:43:37 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> I'll try to answer the main comments in one message. I hope I haven't lost any.
>
> 1. I don't want to add a switch to keep the original alignment. I think we should be confident that this change is correct or not make it! It's the beginning of the release. We have time to find any bugs that are unlikely to fall out.
Okay, sure.
> 2. Funny that metaspace was originally inherited from Arena, as you know, but I'd rather see a simpler implementation for Arena if rewritten than using all of Metaspace. Metaspace needs to deallocate from the middle and Arena doesn't. Arena uses malloc which is better imo except for on one platform. Rewriting Arena isn't something that's on the top of our list, please only file a bug if you think you're going to do it @tstuefe :)
Of course, sorry, I was not suggesting someone other should do it. I was just interested in your thoughts.
> 3. I'll file another RFE for renaming (ignore some comment above that I can't find now).
Change looks still good to me.
..Thomas
-------------
PR: https://git.openjdk.java.net/jdk/pull/4732
More information about the hotspot-dev
mailing list