RFR: 8305898: Alternative self-forwarding mechanism [v10]
Roman Kennke
rkennke at openjdk.org
Thu May 11 08:48:45 UTC 2023
On Thu, 11 May 2023 08:00:53 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:
>> Roman Kennke has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Merge branch 'JDK-8305896' into JDK-8305898
>> - Align fake-heap without GCC warnings (duh)
>
> src/hotspot/share/oops/oop.inline.hpp line 276:
>
>> 274: }
>> 275:
>> 276: void oopDesc::forward_failed() {
>
> It is a bit confusing that oopDesc::forward_failed is a setter, while markWord::forward_failed is a getter.
Yeah.
It's even more confusing that we now have the notion of forward-failed, which aims to hide the implementation detail of self-forwarding, but forwardee() still exposes it. And probably has to, because that is how the forwarding logic of GCs currently work, and I'm not sure it is useful to change that. I need to mull over this a bit.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13779#discussion_r1190843704
More information about the hotspot-gc-dev
mailing list