RFR(S): 8158260: PPC64: unaligned Unsafe.getInt can lead to the generation of illegal instructions
Volker Simonis
volker.simonis at gmail.com
Tue Jun 14 18:12:19 UTC 2016
Hi,
can I please get a review for this small change which fixes a code
generation problem on ppc64:
http://cr.openjdk.java.net/~simonis/webrevs/2016/8158260/
https://bugs.openjdk.java.net/browse/JDK-8158260
In the case where we access an unaligned address trough Unsafe, we
could generate an illegal instruction because we didn't correctly
specify the memory operand of some match rules. The fix is trivial.
Although this change is ppc64-specific I think I need a sponsor
because the change contains a new regression test which is supposed to
run on all platforms.
On platforms where Unsafe.unalignedAccess() will return false (e.g.
Solaris/SPARC) this test will return immediately (and I've verified
that it works).
I've also added aarch64 to the list of recipients because on aarch64
Unsafe.unalignedAccess() is apparently supported and it would be nice
if somebody could check that the new regression test is indeed working
there.
@Hiroshi, Gustavo: could you please verify that this change is enough
to fix the initial problems you've observed when running Cassandra on
Linux/ppc64? The changes from ppc64.ad should easily apply to the
8u-dev repository. Once we've got this into jdk9 I'll trigger a
downport request to get the fix into jdk8 as well.
Thank you and best regards,
Volker
More information about the ppc-aix-port-dev
mailing list