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

[Roman Kennke]

Hi Chris, Per,

Shenandoah has the same problem, for the same reasons. And I'm not
convinced that G1 is safe either, at least with
-XX:+ClassUnloadingWithConcurrentMark.

In my opinion, this whole code section needs to be rewritten to use
something like CollectedHeap::object_iterate() (which is used by other
similar external code like JVMTI for heapdumping/walking). Simply
assuming a certain object layout is not very sustainable IMO.

Roman

> 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?
> 
> 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
> 
> 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20181129/5ab9f402/signature.asc>