Hi again, Sharath,

I missed a point. The tests serviceability/sa/ClhsdbCDSCore.java and test/hotspot/jtreg/gc/metaspace/CompressedClassSpaceSizeInJmapHeap.java would also have to be excluded for solaris.

Thanks,
Jini.

On 8/15/2018 9:46 PM, Jini George wrote:
Hi Sharath,

Your changes look good. One question:

Was the following test excluded by mistake ?

serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java

Would you please also delete these 2 lines since these tests have been removed ?

serviceability/sa/ClhsdbSymbol.java 8193639 solaris-all
serviceability/sa/ClhsdbSymbolTable.java 8193639 solaris-all

Thanks,
Jini


On 8/14/2018 12:48 PM, Sharath Ballal wrote:
Hello,

Pls review this small fix to problem list SA testcases failing on Solaris.

Bug ID: https://bugs.openjdk.java.net/browse/JDK-8209342

Webrev: http://cr.openjdk.java.net/~sballal/8209342/webrev.00/ <http://cr.openjdk.java.net/%7Esballal/8209342/webrev.00/>

Thanks,

Sharath

Reply via email to