RFR(M) 8242165: SA sysprops support fails to dump all system properties

[Chris Plummer]

Since JDK-8193639 has now been pushed, this test no longer needs to be 
problem listed on solaris, so I undid that change to ProblemList.txt. 
However, it does fail on with ZGC due to JDK-8220624, so it is now 
problem listed in ProblemList-zgc.txt. I also fixed a minor sort order 
in the problem list:

diff --git a/test/hotspot/jtreg/ProblemList-zgc.txt 
b/test/hotspot/jtreg/ProblemList-zgc.txt
--- a/test/hotspot/jtreg/ProblemList-zgc.txt
+++ b/test/hotspot/jtreg/ProblemList-zgc.txt
@@ -28,8 +28,9 @@
  #############################################################################

  resourcehogs/serviceability/sa/TestHeapDumpForLargeArray.java 
8220624   generic-all
+serviceability/sa/CDSJMapClstats.java 8220624   generic-all
+serviceability/sa/ClhsdClasses.java 8220624   generic-all
  serviceability/sa/ClhsdbDumpheap.java 8220624   generic-all
-serviceability/sa/CDSJMapClstats.java 8220624   generic-all
  serviceability/sa/ClhsdbCDSJstackPrintAll.java 8220624   generic-all
  serviceability/sa/ClhsdbFindPC.java#id0 8220624   generic-all
  serviceability/sa/ClhsdbFindPC.java#id1 8220624   generic-all

thanks,

Chris

On 4/5/20 10:49 PM, Chris Plummer wrote:
> Hello,
>
> Please help review the following:
>
> https://bugs.openjdk.java.net/browse/JDK-8242165
> http://cr.openjdk.java.net/~cjplummer/8242165/webrev.03
>
> Please see the CR for an explanation of the bug and the fix. If you 
> need some help with the SA code, let me know and I'll provide some 
> details. It was pretty much all new to me, so I tried to put the 
> needed details in the CR.
>
> For the test, I compare the list of properties dumped using 3 
> different methods to make sure the same set is dumped for each:
>
> 1. jhsdb jinfo --sysprops
> 2. jinfo -sysprops
> 3. Simple app (LingeredAppSysProps) that calls 
> System.getProperties().list(System.out)
>
> The app output is considered the master list that is compared against 
> the others, and I also make sure each list has the same number of 
> properties.
>
> thanks,
>
> Chris