<div dir="ltr"><div dir="ltr">Ah, sorry about that. I only tested locally with "<span style="color:rgb(0,0,0)">test/hotspot/jtreg:tier1".</span> And my configure script has --disable-hotspot-gtest, so it didn't execute the gtest (I'll remove that configure flag). Usually JC would help me run tests through the <a href="https://wiki.openjdk.java.net/display/Build/Submit+Repo">submit repo</a>, but he's on vacation. Will do better testing next time.<div><div><br></div><div>I saw the patch has been pushed. Thanks for the feedback and help!</div><div></div><div><br></div><div><div><div><div dir="ltr" class="gmail-m_-2744525414612066458m_-4401808482136737042gmail_signature"><div dir="ltr">-Man</div></div></div><br></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 1, 2018 at 5:26 PM Kim Barrett <<a href="mailto:kim.barrett@oracle.com" target="_blank">kim.barrett@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> On Sep 29, 2018, at 3:19 PM, Kim Barrett <<a href="mailto:kim.barrett@oracle.com" target="_blank">kim.barrett@oracle.com</a>> wrote:<br>
> <br>
>> On Sep 28, 2018, at 9:46 PM, Man Cao <<a href="mailto:manc@google.com" target="_blank">manc@google.com</a>> wrote:<br>
>> <br>
>> Hi,<br>
>> <br>
>> Could someone review this minor logging tag change for ParallelGC? It avoids missing log messages with -Xlog:gc*=debug and -Xlog:gc*=trace.<br>
>> <br>
>> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8210716" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8210716</a><br>
>> Webrev: <a href="http://cr.openjdk.java.net/~manc/8210716/webrev.00/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~manc/8210716/webrev.00/</a><br>
>> <br>
>> PS: I'd need someone help me push it as I just became an Author.<br>
>> <br>
>> Thanks,<br>
>> Man<br>
> <br>
> All other uses of the "ergo" tag in gc logging place it as the second<br>
> tag, e.g. "(gc, ergo, ...)" or "(gc, ergo)".  Not that the order<br>
> matters for logging control, but consistency can help with searching<br>
> and pattern matching while reading.<br>
<br>
FYI, it turns out the tag ordering does need to be consistent.  There is a gtest that complains if they aren’t. <br>
<br>
[ RUN      ] LogTagSet.duplicates_test<br>
test/hotspot/gtest/logging/test_logTagSet.cpp:169: Failure<br>
Failed<br>
duplicate LogTagSets found: 'gc,ergo,heap' vs 'gc,heap,ergo' (tags must always be specified in the same order for each tagset)<br>
[  FAILED  ] LogTagSet.duplicates_test (1 ms)<br>
<br>
Was the original patch run through dev-submit? I think that should have failed because of the failing gtest.<br>
<br>
</blockquote></div>