<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Hi Man,</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Thanks for the update — great to see your patch moving forward.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Your changes look well-aligned with the direction Thomas outlined and what I summarized earlier:
<code>SoftMaxHeapSize</code> guiding committed memory, integrated into resizing and IHOP logic, without overriding other tunables or introducing hard caps.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
I’ve updated JDK-8236073 to reflect both your and Thomas’s contributions and reassigned it to you. This change forms a solid foundation as we move toward a more responsive, multi-signal controller (AHS) for G1.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Appreciate you picking this up and looking forward to seeing the PR evolve.</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
Best,</div>
<div style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);" class="elementToProof">
Monica</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 11pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" id="Signature">
<table id="pbpsiglinktable">
<tbody>
<tr>
<td style="width: 22px; height: 26px;">
<div><a href="https://outlook.office.com/bookwithme/user/6dc2f1f46dfd446aa456d1c1245cecd6@microsoft.com?anonymous&ep=bwmEmailSignature"><img data-outlook-trace="F:1|T:1" src="cid:7f20e45b-54c7-47bb-9434-f9eefbbb0a22"></a></div>
</td>
<td></td>
<td style="width: 194.038px; height: 26px;">
<div style="color: rgb(0, 120, 212);"><a href="https://outlook.office.com/bookwithme/user/6dc2f1f46dfd446aa456d1c1245cecd6@microsoft.com?anonymous&ep=bwmEmailSignature" style="color: rgb(0, 120, 212); text-decoration: none;">Book time to meet with me</a></div>
</td>
<td></td>
</tr>
</tbody>
</table>
</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Man Cao <manc@google.com><br>
<b>Sent:</b> Friday, March 21, 2025 5:54 PM<br>
<b>To:</b> Monica Beckwith <Monica.Beckwith@microsoft.com><br>
<b>Cc:</b> Thomas Schatzl <thomas.schatzl@oracle.com>; hotspot-gc-dev@openjdk.org <hotspot-gc-dev@openjdk.org><br>
<b>Subject:</b> [EXTERNAL] Re: Moving Forward with AHS for G1</font>
<div> </div>
</div>
<div>
<div dir="ltr">
<div dir="ltr">
<div>Thank you for the summary and volunteering on this work! Apology for the lack of response from our side, due to other tasks and priorities.</div>
<div>I have been experimenting with implementing SoftMaxHeapSize for G1 (JDK-8236073), and using this knob instead of ProposedHeapSize for Google's AHS project. I could probably send out a Github PR next week.</div>
<div>From our side, we would really like to make sure the implementation for SoftMaxHeapSize (JDK-8236073) and CurrentMaxHeapSize (JDK-8204088) work with Google's AHS project. It would be more effective if we could test with our internal workload and benchmarks
during development.</div>
<div>Is it OK if I pick up the work for SoftMaxHeapSize (JDK-8236073) and CurrentMaxHeapSize (JDK-8204088)?</div>
<div><br>
</div>
<div>
<div dir="ltr" class="x_gmail_signature">
<div dir="ltr">-Man</div>
</div>
</div>
</div>
</div>
</div>
</body>
</html>