<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=utf-8"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0cm;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:#954F72;
text-decoration:underline;}
.MsoChpDefault
{mso-style-type:export-only;}
@page WordSection1
{size:612.0pt 792.0pt;
margin:70.85pt 70.85pt 2.0cm 70.85pt;}
div.WordSection1
{page:WordSection1;}
--></style></head><body lang=DE link=blue vlink="#954F72"><div class=WordSection1><p class=MsoNormal><span lang=EN-US>Hello,</span></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><span lang=EN-US>Isn’t formatting changes and units part of the UL GC logging changes, anyway?</span><o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><br>Gruss<br>Bernd<br>-- <br>http://bernd.eckenfels.net<br>From Win 10 Mobile</p><p class=MsoNormal><span style='font-size:12.0pt;font-family:"Times New Roman",serif'><o:p> </o:p></span></p><div style='mso-element:para-border-div;border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm'><p class=MsoNormal style='border:none;padding:0cm'><b>Von: </b><a href="mailto:per.liden@oracle.com">Per Liden</a><br><b>Gesendet: </b>Mittwoch, 13. April 2016 11:05<br><b>An: </b><a href="mailto:mattis.castegren@oracle.com">Mattis Castegren</a>; <a href="mailto:cheleswer.sahu@oracle.com">Cheleswer Sahu</a>; <a href="mailto:mikael.gerdin@oracle.com">Mikael Gerdin</a>; <a href="mailto:hotspot-gc-dev@openjdk.java.net">hotspot-gc-dev@openjdk.java.net</a><br><b>Betreff: </b>Re: RFR[9u-dev]: 8054326: Confusing message in "Current rem setstatistics"</p></div><p class=MsoNormal><span style='font-size:12.0pt;font-family:"Times New Roman",serif'><o:p> </o:p></span></p><p class=MsoNormal>Hi,</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>On 2016-04-13 09:57, Mattis Castegren wrote:</p><p class=MsoNormal>> Hi</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> This request came in some time ago as a low priority request from PSR (the Oracle performance team):</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> https://bug.oraclecorp.com/pls/bug/webbug_edit.edit_info_top?rptno=19356019</p><p class=MsoNormal>> PSR:FUNC: Confusing msg in "Current rem set statistics"</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Their main complaint is that 0 usually means exactly 0. So, I think the balance is between 0 being confusing when the value is non-zero vs the ease of parsing the logs. I see three ways to do this:</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> 1) Close the bug as Won't Fix</p><p class=MsoNormal>> 2) Change this specific issue, print bytes if the value is lower than 1k</p><p class=MsoNormal>> 3) Change this and a few other places.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> We in Sustaining don't have too strong of an opinion here. Stanley Guan from PSR recently commented the BugDB bug making the case for changing the output, but if the GC team is of the opinion that we should close this as Won't Fix with the motivation that parsing becomes harder with this change, then we can close the bug.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Anyone else who has an opinion or a motivation for any of the options above?</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>I would prefer 1 or 3, where 3 is be using the existing </p><p class=MsoNormal>proper_unit_for_byte_size/byte_size_in_proper_unit functions, unless </p><p class=MsoNormal>there's a good reason not to. I can't see any problems with printing B </p><p class=MsoNormal>for sizes under 10K rather than 1K, it's actually a feature.</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>cheers,</p><p class=MsoNormal>Per</p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Kind Regards</p><p class=MsoNormal>> /Mattis</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> -----Original Message-----</p><p class=MsoNormal>> From: Cheleswer Sahu</p><p class=MsoNormal>> Sent: den 11 april 2016 15:33</p><p class=MsoNormal>> To: Liden; Mikael Gerdin; hotspot-gc-dev@openjdk.java.net</p><p class=MsoNormal>> Cc: Mattis Castegren</p><p class=MsoNormal>> Subject: RE: RFR[9u-dev]: 8054326: Confusing message in "Current rem set statistics"</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Hi,</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> -----Original Message-----</p><p class=MsoNormal>> From: Per Liden</p><p class=MsoNormal>> Sent: Friday, April 08, 2016 1:57 PM</p><p class=MsoNormal>> To: Mikael Gerdin; Cheleswer Sahu; hotspot-gc-dev@openjdk.java.net</p><p class=MsoNormal>> Subject: Re: RFR[9u-dev]: 8054326: Confusing message in "Current rem set statistics"</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Hi,</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> On 2016-04-06 13:09, Mikael Gerdin wrote:</p><p class=MsoNormal>> [...]</p><p class=MsoNormal>>>> Hi,</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Please review the code changes for</p><p class=MsoNormal>>>> https://bugs.openjdk.java.net/browse/JDK-8054326.</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Webrev Link: http://cr.openjdk.java.net/~csahu/8054326/</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> We try to avoid printing sizes with dynamically selected binary prefixes</p><p class=MsoNormal>> (K/M/G) for the simple reason that it makes it harder to grep through and compare GC logs. If that "rule" applies here can be discussed.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> We already have the functions</p><p class=MsoNormal>> proper_unit_for_byte_size/byte_size_in_proper_unit to do roughly what you've done, but they are very rarely used for the reason stated above.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Mikael had directed me towards this function when we had offline chat regarding this, but this function prints in KB only if size is greater than 10k. Therefore I had written my own code to print this.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> There are a other places in this code/function where we're also printing potentially small values as K, which you haven't addressed here. Just addressing one of the places seems like an incomplete fix.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Yes, I agree there are other places too which can be considered. I just took the "Max" part because I was not sure about others.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> I'm personally not sure if this is a problem big enough to worth addressing at all. Maybe others have a different opinion, Thomas/Bengt/Mikeal/Stefan?</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Does any have any other opinion regarding this ? We in sustaining team although feel that showing "Max" as 0k is ambiguous.</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>> Regards,</p><p class=MsoNormal>> Cheleswer</p><p class=MsoNormal>> cheers,</p><p class=MsoNormal>> Per</p><p class=MsoNormal>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Bug Brief: In output of remset statistics "Max" is printing as 0k,</p><p class=MsoNormal>>>> which is confusing for user.</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Problem Identified : "Max" value is rounded to KB, which result in</p><p class=MsoNormal>>>> 0k, if the value is less than 1024B.</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Solution Proposed: If the value for "Max" is less than 1 KB (1024</p><p class=MsoNormal>>>> B), print the value in bytes (i.e. 1023B.) else</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> print the value in KB.</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Regards,</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>> Cheleswer</p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal>>>><o:p> </o:p></p><p class=MsoNormal><o:p> </o:p></p></div></body></html>