<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 24, 2023 at 9:08 AM David Holmes <<a href="mailto:david.holmes@oracle.com">david.holmes@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
On 24/05/2023 4:45 pm, Jean-Philippe Bempel wrote:<br>
> Hi David,<br>
> <br>
> On Wed, May 24, 2023 at 7:02 AM David Holmes <<a href="mailto:david.holmes@oracle.com" target="_blank">david.holmes@oracle.com</a>> wrote:<br>
>><br>
>>> class MyClass {<br>
>>>       private static void writeFile() {<br>
>><br>
>> I don't see this ever getting called. Is some code missing?<br>
> <br>
> No code is missing, code is not executed, The whole class is<br>
> re-transformed and just by the<br>
> presence of this method with this code in the classfile trigger the issue<br>
<br>
Ah now I get it. So basically you're filling up metaspace faster than it <br>
can be reclaimed. Other than class unloading I'm not sure what the <br>
reclamation mechanism is, so I'll defer to Thomas at this point. We <br>
definitely need to see more information such as logging or the jcmd <br>
output Thomas suggested.<br>
<br>
Cheers,<br>
David<br>
-----<br></blockquote><div><br></div><div>The space for the old bytecode gets added to an arena-internal structure, and we attempt to reuse those pieces of memory for allocations within the same arena.<br></div><div><br></div><div>If all classes are loaded, and all we do is transform them, the only chance of re-using those pieces are the next redefinitions - space allocated for new bytecode. So, when retransforming class B we allocate space for bytecode class B version 2 and will attempt to reuse space from class A version 1 that we retransformed before.</div><div><br></div><div>But if the new byte codes are consistently larger than the old ones, this may not often work. This may just be an obnoxious usage pattern.</div><div><br></div><div>I don't know if, at redefinition time, we are smart enough to reuse space for bytecodes if the new space requirement is <= old space size. That would be a simple first improvement.<br></div><div> </div><br></div></div>