RFR: Adapt upstream object pinning API
Zhengyu Gu
zgu at redhat.com
Wed Apr 4 17:34:02 UTC 2018
On 04/04/2018 01:17 PM, Aleksey Shipilev wrote:
> On 04/04/2018 07:02 PM, Zhengyu Gu wrote:
>> Adapted upstream object pinning API, also made adjustment to critical native support to use new API.
>>
>>
>> Webrev: http://cr.openjdk.java.net/~zgu/shenandoah/pin_interface/webrev.00/
>
> *) The changeset contains JDK-8200429 only, make sure you are pushing the correct changeset:
> http://cr.openjdk.java.net/~zgu/shenandoah/pin_interface/webrev.00/shenandoah-jdk10.changeset
Yes, we need this one for new API. The patch is from upstream jdk repo,
but did not apply cleanly.
>
> *) Since you are renaming the RT stubs, should you also rename the MacroAssembler entries? Does it
> do something special for arrays, or is it just the stub for the call to
> SharedRuntime::{pin|unpin}_object?
>
> MacroAssembler::pin_critical_native_array
> MacroAssembler::unpin_critical_native_array
No. SharedRuntime::{pin|unpin}_object are generic.
MacroAssembler::pin/unpin_critical_native_array generate code to make RT
calls for array arguments.
Thanks,
-Zhengyu
>
> Thanks,
> -Aleksey
>
More information about the shenandoah-dev
mailing list