On Thu, 4 Jul 2024 14:07:57 GMT, Thomas Stuefe <[email protected]> wrote:

> If this is for src/hotspot/share/nmt/arrayWithFreeList.hpp, would it not be a 
> lot simpler to just implement it there, and give it another backing store?
> 
> In particular because after doing all this work it still won't even support 
> the feature I was hoping for, mainly the ability to put an indexed free list 
> atop of existing memory.

I did that first and it sure is simpler, but I'm not sure whether it's a good 
idea to have to support such a backing storage. See `resizable_array` in here: 
https://github.com/openjdk/jdk/pull/20002

 Still, it does do what you asked for, kind of, see: `GrowableArrayFromArray`. 
I can adapt AWFL to be able to use either `GAFA` or `GA`.

It's also not the only reason to do this.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/20031#issuecomment-2209118236

Reply via email to