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

[Roman Kennke]

Ping? Should I go ahead and push it?

Thanks,
Roman

> 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/0bd4f528/signature.asc>