Hello , here is a new webrev :
http://cr.openjdk.java.net/~mbaesken/webrevs/8209115.1/
<http://cr.openjdk.java.net/%7Embaesken/webrevs/8209115.1/>
I set PNG_POWERPC_VSX_OPT now only on the linux ppc platforms .
Best regards, Matthias
*From:*Baesken, Matthias
*Sent:* Donnerstag, 9. August 2018 08:48
*To:* 'Philip Race' <philip.r...@oracle.com>
*Cc:* 'build-dev@openjdk.java.net' <build-dev@openjdk.java.net>;
awt-...@openjdk.java.net
*Subject:* RE: [RFR]: 8209115: adjust libsplashscreen linux ppc64le
builds for easier libpng update - was : RE: RFR 8195615 :
libsplashscreen linux ppc64le build error after libpng update
ØSeems like the ARM one has been harmless for an x64 build
* so we could for now assume the PPC one is too
Hi Phil ,
I will guard the define for the platform where we want to
use it - I think this is better for the future than just setting it
on all platforms .
And btw. it has to be
PNG_POWERPC_VSX_OPT not PNG_POWERPC_VSX
☹ - my mistake .
For some reason the current setting still builds on our ppc64 (le)
test machine in the current codebase , strange !
Best regards, Matthias
*From:*Philip Race <philip.r...@oracle.com
<mailto:philip.r...@oracle.com>>
*Sent:* Donnerstag, 9. August 2018 04:09
*To:* Baesken, Matthias <matthias.baes...@sap.com
<mailto:matthias.baes...@sap.com>>
*Cc:* 'build-dev@openjdk.java.net' <build-dev@openjdk.java.net
<mailto:build-dev@openjdk.java.net>>; awt-...@openjdk.java.net
<mailto:awt-...@openjdk.java.net>
*Subject:* Re: [RFR]: 8209115: adjust libsplashscreen linux ppc64le
builds for easier libpng update - was : RE: RFR 8195615 :
libsplashscreen linux ppc64le build error after libpng update
The only comment I have is about this
- LIBSPLASHSCREEN_CFLAGS += -DSPLASHSCREEN -DPNG_NO_MMX_CODE
-DPNG_ARM_NEON_OPT=0
+ LIBSPLASHSCREEN_CFLAGS += -DSPLASHSCREEN -DPNG_NO_MMX_CODE
-DPNG_ARM_NEON_OPT=0 -DPNG_POWERPC_VSX=0
I suppose these platform defines are harmless and un-referenced unless
building for
the specified platform. Seems like the ARM one has been harmless for
an x64 build
so we could for now assume the PPC one is too.
Or you could use jdk-submit to prove this :-)
-phil.
On 8/8/18, 12:59 AM, Baesken, Matthias wrote:
Hello ,
please review this small adjustment for the build of
libsplashscreen , especially the libpng parts .
Back then, we had to fix the build of the libpng parts in
libsplashscreen on linux ppc64le with “8195615 :
libsplashscreen linux ppc64le build error after libpng update” .
However this introduced a small adjustment to pngpriv.h that
needs to be kept every time libpng is updated (happened
recently when Phil updated the lib).
So I better want to remove the adjustment to pngpriv from
8195615 , and instead change the build settings to fix the
compilation on linuxppc64 le .
Webrev/bug :
http://cr.openjdk.java.net/~mbaesken/webrevs/8209115.0/
<http://cr.openjdk.java.net/%7Embaesken/webrevs/8209115.0/>
https://bugs.openjdk.java.net/browse/JDK-8209115
Thanks ,
Matthias
*From:*Philip Race <philip.r...@oracle.com>
<mailto:philip.r...@oracle.com>
*Sent:* Dienstag, 7. August 2018 17:15
*To:* Baesken, Matthias <matthias.baes...@sap.com>
<mailto:matthias.baes...@sap.com>
*Cc:* Doerr, Martin <martin.do...@sap.com>
<mailto:martin.do...@sap.com>; 2d-...@openjdk.java.net
<mailto:2d-...@openjdk.java.net>; Simonis, Volker
<volker.simo...@sap.com> <mailto:volker.simo...@sap.com>;
Lindenmaier, Goetz <goetz.lindenma...@sap.com>
<mailto:goetz.lindenma...@sap.com>
*Subject:* Re: libsplashscreen compilation on ppc64 ( le) - was :
RE: RFR 8195615 : libsplashscreen linux ppc64le build error after
libpng update
Works for me. Include build-dev on the review.
And splashscreen is considered an AWT feature so it should be
awt-dev not 2d-dev
although you may want to reference back to this earlier exchange.
-phil.
On 8/7/18, 8:04 AM, Baesken, Matthias wrote:
Hello, should I prepare a change setting the
-DPNG_POWERPC_VSX=0 flag in the makefile (see below) ?
Might make future libpng updates more simple .
Best regards, Matthias
*From:*Baesken, Matthias
*Sent:* Donnerstag, 2. August 2018 17:28
*To:* 'Phil Race' <philip.r...@oracle.com>
<mailto:philip.r...@oracle.com>; Doerr, Martin
<martin.do...@sap.com> <mailto:martin.do...@sap.com>
*Cc:* Simonis, Volker <volker.simo...@sap.com>
<mailto:volker.simo...@sap.com>; Lindenmaier, Goetz
<goetz.lindenma...@sap.com> <mailto:goetz.lindenma...@sap.com>
*Subject:* RE: RFR 8195615 : libsplashscreen linux ppc64le
build error after libpng update - was : RE: jdk-hs ppc64le
build error, probably related to libpng update
Hi Phil, I added -DPNG_POWERPC_VSX=0 to
Awt2dLibraries.gmk for the sphlashscreen library build, and
removed (uncommented) the workaround in pngpriv.h .
Build on the head of jdk11 was fine on my linux
ppc64le test machine .
Best regards, Matthias
Diff:
/open_jdk/jdk11> hg diff
diff -r 26cca23c165a make/lib/Awt2dLibraries.gmk
--- a/make/lib/Awt2dLibraries.gmk Thu Aug 02 09:49:04
2018 +0200
+++ b/make/lib/Awt2dLibraries.gmk Thu Aug 02 16:50:09
2018 +0200
@@ -794,7 +794,8 @@
LIBSPLASHSCREEN_EXCLUDE_SRC_PATTERNS := unix
endif
- LIBSPLASHSCREEN_CFLAGS += -DSPLASHSCREEN -DPNG_NO_MMX_CODE
-DPNG_ARM_NEON_OPT=0
+ # disable ppc64 opts
+ LIBSPLASHSCREEN_CFLAGS += -DSPLASHSCREEN -DPNG_NO_MMX_CODE
-DPNG_ARM_NEON_OPT=0 -DPNG_POWERPC_VSX=0
ifeq ($(OPENJDK_TARGET_OS), macosx)
LIBSPLASHSCREEN_CFLAGS += -DWITH_MACOSX
diff -r 26cca23c165a
src/java.desktop/share/native/libsplashscreen/libpng/pngpriv.h
---
a/src/java.desktop/share/native/libsplashscreen/libpng/pngpriv.h
Thu Aug 02 09:49:04 2018 +0200
+++
b/src/java.desktop/share/native/libsplashscreen/libpng/pngpriv.h
Thu Aug 02 16:50:09 2018 +0200
@@ -290,12 +290,12 @@
# endif
#endif /* PNG_MIPS_MSA_OPT > 0 */
-#ifdef PNG_POWERPC_VSX_API_SUPPORTED
+/* #ifdef PNG_POWERPC_VSX_API_SUPPORTED */
#if PNG_POWERPC_VSX_OPT > 0
# define PNG_FILTER_OPTIMIZATIONS png_init_filter_functions_vsx
# define PNG_POWERPC_VSX_IMPLEMENTATION 1
#endif
-#endif
+/* #endif */
