RFR: 8369622: GlobalChunkPoolMutex is recursively locked during error handling

[Paul Hübner]

On Tue, 21 Oct 2025 12:37:06 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/memory/arena.hpp line 41:
>> 
>>> 39: 
>>> 40: class ChunkPoolLocker : public StackObj {
>>> 41: public:
>> 
>> Nitpick FYI: I think the new `public` has a different indentation than the old one. Just wanted to point it out, is this something that should be addressed? I do not have an opinion on the topic.
>
> For some reason, much of the code I've seen has one space for access modifiers, so I'm going to stick with that.

@coleenp I meant that this indentation here is zero spaces, not one, as far as I can tell. But a very very minor thing.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27869#discussion_r2448056000