RFR: 8202381: (Solaris) SIGBUS in # V [libjvm.so+0xcee494] jni_GetIntField+0x224

[Kim Barrett]

> On Apr 30, 2018, at 11:53 AM, Erik Österlund <erik.osterlund at oracle.com> wrote:
> 
> Fixed. I changed it here too.
> 
> Full:
> http://cr.openjdk.java.net/~eosterlund/8202381/webrev.01/
> 
> Incremental:
> http://cr.openjdk.java.net/~eosterlund/8202381/webrev.00_01/

Changes look good.

However, I noticed a couple of issues that are probably out of scope
for getting JDK-8202381 fixed in a timely fashion.  Please file a
followup RFR or two to address these.

------------------------------------------------------------------------------
try_resolve_jobject_in_native slowpath is non-const reference.  Seems
like it should be const.  Sorry I missed this in the review of
JDK-8200235.

------------------------------------------------------------------------------ 
src/hotspot/cpu/sparc/jniFastGetField_sparc.cpp
JNI_FastGetField::generate_fast_get_long_field
JNIFastGetField::generate_fast_get_float_field

src/hotspot/cpu/x86/jniFastGetField_x86_64.cpp
JNI_FastGetField::generate_fast_get_float_field0

Seems like these functions should also be using
try_resolve_jobject_in_native.  Sorry I missed this in the review of
JDK-8200235.

I didn't look at other platforms to see if there are similar issues
elsewhere.

------------------------------------------------------------------------------