RFR: 8218733: SA: CollectedHeap provides broken implementation for used() and capacity()
Aleksey Shipilev
shade at redhat.com
Mon Feb 11 09:39:33 UTC 2019
On 2/11/19 9:13 AM, Stefan Karlsson wrote:
> Please review this patch to remove the broken implementation of CollectedHeap used() and capacity()
> and instead force all GCs to provide their own implementations.
>
> https://cr.openjdk.java.net/~stefank/8218733/webrev.01/
Looks good.
> Could you verify that the implementation for Epsilon is correct? I also haven't implemented capacity
> for Shenandoah, as the information isn't trivially available in the ShenandoahHeap SA class. Do you
> want to fix it as part of this patch, or should I create a separate RFE for Shenandoah?
Epsilon change looks trivially correct.
For Shenandoah, I think this would suffice:
@Override
public long capacity() {
- // FIXME
- return 0;
+ return numOfRegions() * ShenandoahHeapRegion.regionSizeBytes();
}
-Aleksey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190211/ad7b16bb/signature.asc>
More information about the serviceability-dev
mailing list