[OpenJDK 2D-Dev] [7u8] request for review: 7150594: VM chash in JCK api/java_awt/Image/ConvolveOp/ tests for 64 bit jdk8 on linux.
philip.race at oracle.com
Tue Aug 28 18:59:16 UTC 2012
On 8/20/2012 4:30 AM, Andrew Brygin wrote:
> could you please review a fix for 7150594?
> This problem is triggered by the fix for CR 7113017. In particular,
> this fix replaces the malloc.h with stddef.h in mlib_types.h. This
> change leads to compiling mlib_sys.c without forward declaration for
> memalign() routine, and cause following warnings:
> mlib_sys.c:96: warning: implicit declaration of function 'memalign'
> mlib_sys.c:96: warning: cast to pointer from integer of different size
> This cause the problem on systems where size of integer is less than
> size of pointers: the pointer value is clamped, and usage of the
> clamped pointer causes the observed crash.
> As per discussion for jdk8 fix, suggested solution for 7u8 is to roll
> back the original fix for 7113017.
> Fix discussion for jdk8:
> 7u8 changeset for 7113017:
> Please take a look.
> Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7150594
> Webrev: http://cr.openjdk.java.net/~bae/7150594/7u8/webrev.00/
More information about the 2d-dev