RFR(xxxs, trivial): 8203865: Metaspace cleanup: Remove unused MemRegion in VirtualSpaceNode

David Holmes david.holmes at oracle.com
Mon May 28 06:47:07 UTC 2018


Hi Thomas,

On 28/05/2018 4:28 PM, Thomas Stüfe wrote:
> Hi all,
> 
> very tiny cleanup.
> 
> May I get a review (I consider this trivial):

I'd like someone to confirm that the deleted asserts are still logically 
being checked elsewhere.

Otherwise this cleanup seems ok.

Thanks,
David

> Bug: https://bugs.openjdk.java.net/browse/JDK-8203865
> Webrev: http://cr.openjdk.java.net/~stuefe/webrevs/8203865-Remove-unused-MemRegion-in-VirtualSpaceNode/webrev.00/webrev/
> 
> VirtualSpaceNode contains: MemRegion _reserved;
> 
> This member is not used and serves no purpose. It keeps track of the
> reserved memory location, but that is redundant since the underlying
> ReservedSpace does the same. It is also nowhere used. So it should be
> removed.
> 
> Best Regards, Thomas
> 


More information about the hotspot-runtime-dev mailing list