RFR: JDK-8199735: Mark word updates need to use Access API

[Roman Kennke]

Hi Mikael,

ok. So how do we go about it?
- I can hold back the patch, and have somebody inside work it out
(adding a few #include "oops/oop.inline.hpp" shouldn't hurt even without
the big patch). This would have the advantage that I can submit a round
of testing myself
- I can push the patch and you Oracles guys sort it out after the fact
- Somebody in Oracle can take my patch and sponsor it 'the old way' and
amend it with the internal fixes as necessary.
- something else?

Please let me know!

Thanks, Roman


> 
> Yes, it’s part of the (still closed) JFR sources. Somebody at Oracle will have to help update. Sorry for the inconvenience.
> 
> Cheers,
> Mikael
> 
>> On Apr 3, 2018, at 5:05 PM, Roman Kennke <rkennke at redhat.com> wrote:
>>
>> Thanks, Mikael!
>>
>> What is edgeStore.o and how is it built? I suspect it needs an
>> additional #include "oops/oop.inline.hpp" but I can't find the
>> corresponding source?
>>
>> Roman
>>
>>
>>>
>>> src/hotspot/share/oops/oop.hpp", line 70: Error: The function oopDesc::set_mark(markOopDesc*volatile) has not had a body defined.
>>> src/hotspot/share/oops/oop.hpp", line 66: Error: The function oopDesc::mark() const has not had a body defined.
>>> 2 Error(s) detected.
>>> lib/CompileJvm.gmk:212: recipe for target 'build/solaris-sparcv9-debug/hotspot/variant-server/libjvm/objs/edgeStore.o' failed
>>> make[3]: *** [build/solaris-sparcv9-debug/hotspot/variant-server/libjvm/objs/edgeStore.o] Error 1
>>>
>>>
>>> Cheers,
>>> Mikael
>>>
>>>> On Apr 3, 2018, at 4:39 PM, Roman Kennke <rkennke at redhat.com
>>>> <mailto:rkennke at redhat.com>> wrote:
>>>>
>>>> Am 03.04.2018 um 14:21 schrieb Aleksey Shipilev:
>>>>> On 03/26/2018 06:40 PM, Roman Kennke wrote:
>>>>>>>> Differential:
>>>>>>>> http://cr.openjdk.java.net/~rkennke/JDK-8199735/webrev.03.diff/
>>>>>>>> Full:
>>>>>>>> http://cr.openjdk.java.net/~rkennke/JDK-8199735/webrev.03/
>>>>>
>>>>> Looks good. A bit unfortunate it has to have many changes around due
>>>>> to the rename, but oh well.
>>>>>
>>>>> Thanks,
>>>>> -Aleksey
>>>>>
>>>>
>>>> Something seems to make Solaris/Sparc builds unhappy. Can somebody with
>>>> access to the test/build system check it?
>>>>
>>>> Thanks, Roman
>>>>
>>>>
>>>> Build Details: 2018-04-03-2039378.roman.source
>>>> 0 Failed Tests
>>>> Mach5 Tasks Results Summary
>>>>
>>>>    EXECUTED_WITH_FAILURE: 2
>>>>    KILLED: 0
>>>>    NA: 0
>>>>    UNABLE_TO_RUN: 4
>>>>    PASSED: 77
>>>>    FAILED: 0
>>>>    Build
>>>>
>>>>        2 Not run
>>>>            build_jdk_solaris-solaris-sparcv9-solaris-sparcv9-build-8
>>>> error while building, return value: 2
>>>>
>>>> build_jdk_solaris-solaris-sparcv9-debug-solaris-sparcv9-build-9 error
>>>> while building, return value: 2
>>>>
>>>>    Test
>>>>
>>>>        4 Not run
>>>>
>>>> tier1-solaris-sparc-jdk_closed_test_hotspot_jtreg_tier1_common-solaris-sparcv9-70
>>>> Dependency task failed:
>>>> mach5...laris-solaris-sparcv9-solaris-sparcv9-build-8
>>>>
>>>> tier1-solaris-sparc-jdk_closed_test_hotspot_jtreg_tier1_common-solaris-sparcv9-debug-71
>>>> Dependency task failed:
>>>> mach5...solaris-sparcv9-debug-solaris-sparcv9-build-9
>>>>
>>>> tier1-solaris-sparc-jdk_open_test_hotspot_jtreg_tier1_common-solaris-sparcv9-68
>>>> Dependency task failed:
>>>> mach5...laris-solaris-sparcv9-solaris-sparcv9-build-8
>>>>
>>>> tier1-solaris-sparc-jdk_open_test_hotspot_jtreg_tier1_common-solaris-sparcv9-debug-69
>>>> Dependency task failed:
>>>> mach5...solaris-sparcv9-debug-solaris-sparcv9-build-9
>>>>
>>>>
>>>>
>>>
>>
>>
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20180404/9952f8f8/signature.asc>