RFR: 8214484: ZGC: Exclude SA tests ClhsdbJhisto and TestHeapDumpFor*

[Per Liden]

Hi Chris,

On 2018-11-29 20:04, Chris Plummer wrote:
> Hi Per,
> 
> I think there may have been some discussion about this before. Are there 
> plans to eventually get SA heap walking support for ZGC, and is the list 
> of disabled tests being maintained somewhere?

The latest directive I've received is that we should not spend any more 
time or resources on adding ZGC support to the SA.

I personally agree with that. I think the benefits provided by the SA is 
dwarfed by the cost to maintain it, so we should phase it out.

You should be able to find the excluded tests by grepping for 'vm.gc != 
"Z"' among the SA tests.

cheers,
Per

> 
> thanks,
> 
> Chris
> 
> On 11/29/18 7:07 AM, Per Liden wrote:
>> There's no support for SA heap walking when using ZGC. Hence the tests 
>> for this should not execute when ZGC is enabled.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8214484
>> Webrev: http://cr.openjdk.java.net/~pliden/8214484/webrev.0
>>
>> /Per
> 
>