RFR: 8224815: Remove non-GC uses of CollectedHeap::is_in_reserved()
Erik Osterlund
erik.osterlund at oracle.com
Thu Aug 29 19:53:15 UTC 2019
Hi Ioi,
Thanks for the review!
/Erik
> On 29 Aug 2019, at 21:47, Ioi Lam <ioi.lam at oracle.com> wrote:
>
> The CDS changes in filemap.cpp/hpp look good to me.
>
> Thanks
> - Ioi
>
>> On 8/9/19 12:43 AM, Erik Österlund wrote:
>> Hi,
>>
>> The CollectedHeap::is_in_reserved() function assumes the GC has a contiguous heap. This might not be the case, and hence the shared code should not make assumptions about that. It should use CollectedHeap::is_in() instead.
>>
>> However, CompressedOops inherently assumes the heap is contiguous, so I let it know about the reserved region.
>>
>> This patch depends on 8229278 and 8229189, which are both out for review right now.
>>
>> Bug:
>> https://bugs.openjdk.java.net/browse/JDK-8224815
>>
>> Webrev:
>> http://cr.openjdk.java.net/~eosterlund/8224815/webrev.00/
>>
>> Thanks,
>> /Erik
>
More information about the hotspot-dev
mailing list