RE: RFR : 8202322: AIX: symbol visibility flags not support on xlc 12.1

2018-06-04 Thread Ichiroh Takiguchi
Hello Matthias and Christoph. Thank you for your explanations. I did not have enough knowledge about "visibility". I created following patches. diff -r 02934b0d661b src/java.base/share/native/libjimage/NativeImageBuffer.cpp --- a/src/java.base/share/native/libj

Re: RFR: JDK-8202384: Introduce altserver jvm variant with speculative execution disabled

2018-06-04 Thread David Holmes
Sorry to be late to this party ... On 5/06/2018 6:10 AM, Erik Joelsson wrote: New webrev: http://cr.openjdk.java.net/~erikj/8202384/webrev.02/ Renamed the new jvm variant to "hardened". As it is a hardened server build I'd prefer if that were somehow reflected in the name. Though really I do

Re: RFR: JDK-8202384: Introduce altserver jvm variant with speculative execution disabled

2018-06-04 Thread jesper . wilhelmsson
Looks good to me. /Jesper > On 4 Jun 2018, at 22:10, Erik Joelsson wrote: > > New webrev: http://cr.openjdk.java.net/~erikj/8202384/webrev.02/ > > Renamed the new jvm variant to "hardened". > > /Erik > > > On 2018-06-04 09:54, [email protected] wrote: >>> On 4 Jun 2018, at 17:52,

Re: RFR: JDK-8200132: Remove jre images and bundles

2018-06-04 Thread Erik Joelsson
Given the feedback, I have revised the patch to restore the ability to produce a legacy jre image, but it is still removed from the default "product-images" and "images" targets. To build it you need to specify it explicitly as "legacy-jre-image" (or mac-legacy-jre-bundle for the macosx specifi

Re: RFR: JDK-8200132: Remove jre images and bundles

2018-06-04 Thread Bob Vandette
> On Jun 4, 2018, at 4:32 PM, Jonathan Gibbons > wrote: > > > > On 06/02/2018 12:20 AM, Alan Bateman wrote: >> On 02/06/2018 08:05, Aleksey Shipilev wrote: >>> : >>> Unfortunately, in the age of containers, distribution size matters. It >>> makes the whole sense to ship >>> JRE in Docker co

Re: RFR: JDK-8200132: Remove jre images and bundles

2018-06-04 Thread Jonathan Gibbons
On 06/02/2018 12:20 AM, Alan Bateman wrote: On 02/06/2018 08:05, Aleksey Shipilev wrote: : Unfortunately, in the age of containers, distribution size matters. It makes the whole sense to ship JRE in Docker containers to provide the execution environment for the upper layers. Remember, hardl

Re: RFR: JDK-8202384: Introduce altserver jvm variant with speculative execution disabled

2018-06-04 Thread Erik Joelsson
New webrev: http://cr.openjdk.java.net/~erikj/8202384/webrev.02/ Renamed the new jvm variant to "hardened". /Erik On 2018-06-04 09:54, [email protected] wrote: On 4 Jun 2018, at 17:52, Erik Joelsson wrote: Hello, On 2018-06-01 14:00, Aleksey Shipilev wrote: On 06/01/2018 10:53

Re: RFR: JDK-8200132: Remove jre images and bundles

2018-06-04 Thread Alan Bateman
On 04/06/2018 20:45, Bob Vandette wrote: If we do this, shouldn’t we provide an aggregator module to allow developers to easily create a Java runtime with the same list of modules?  AFAIK, this doesn’t exists. The java.se  module is not the same.  It’s missing many modules. Th

Re: RFR: JDK-8200132: Remove jre images and bundles

2018-06-04 Thread Bob Vandette
If we do this, shouldn’t we provide an aggregator module to allow developers to easily create a Java runtime with the same list of modules? AFAIK, this doesn’t exists. The java.se module is not the same. It’s missing many modules. Bob. > On Jun 2, 2018, at 1:01 PM, Alan Bat

Re: [8] RFR (XS) 8204053 : libsaproc.so not linked with -z,noexecstack

2018-06-04 Thread Erik Joelsson
Looks ok. /Erik On 2018-06-02 16:17, David Buck wrote: Hi! Please review this very small fix. It only applies to JDK 8. bug report: https://bugs.openjdk.java.net/browse/JDK-8204053 webrev: http://cr.openjdk.java.net/~dbuck/8204053.0/ JPRT hotspot testset run and passed. Efficacy of fix man

Re: RFR: JDK-8202384: Introduce altserver jvm variant with speculative execution disabled

2018-06-04 Thread jesper . wilhelmsson
> On 4 Jun 2018, at 17:52, Erik Joelsson wrote: > > Hello, > > On 2018-06-01 14:00, Aleksey Shipilev wrote: >> On 06/01/2018 10:53 PM, Erik Joelsson wrote: >>> This patch defines flags for disabling speculative execution for GCC and >>> Visual Studio and applies >>> them to all binaries except

Re: RFR: JDK-8202384: Introduce altserver jvm variant with speculative execution disabled

2018-06-04 Thread Erik Joelsson
Hello, On 2018-06-01 14:00, Aleksey Shipilev wrote: On 06/01/2018 10:53 PM, Erik Joelsson wrote: This patch defines flags for disabling speculative execution for GCC and Visual Studio and applies them to all binaries except libjvm when available in the compiler. It defines a new jvm feature n

RE: RFR: JDK-8204211: windows : handle potential C++ exception in GDIRenderer -was : RE: [OpenJDK 2D-Dev] java2d coding using SAFE_SIZE_ARRAY_ALLOC / safe_Malloc

2018-06-04 Thread Langer, Christoph
Hi Matthias, looks good to me. Don't forget the Copyright years. Best regards Christoph > -Original Message- > From: Baesken, Matthias > Sent: Montag, 4. Juni 2018 16:20 > To: Thomas Stüfe ; 'build- > [email protected]' ; awt- > [email protected] > Cc: 2d-dev <[email protected]

RE: RFR: JDK-8204211: windows : handle potential C++ exception in GDIRenderer -was : RE: [OpenJDK 2D-Dev] java2d coding using SAFE_SIZE_ARRAY_ALLOC / safe_Malloc

2018-06-04 Thread Baesken, Matthias
Hello, I prepared a second webrev. - use now const-reference in the catch-statements as suggested by Thomas - reenabled the cl warning showing the exception issues in make/lib/Awt2dLibraries.gmk - found a second place in WPrinterJob.cpp with similar issues after reenabling the warni

Re: RFR: 8204210: Implementation: JEP 333: ZGC: A Scalable Low-Latency Garbage Collector (Experimental)

2018-06-04 Thread Per Liden
Hi Magnus, Thanks for reviewing. On 06/04/2018 01:20 PM, Magnus Ihse Bursie wrote: Hi Per, Please always include build-dev when proposing build changes. The changes in make/autoconf/hotspot.m4 looks a bit suspect. Since you are adding zgc to disabled jvm features, which always override enabl

Re: RFR: 8204210: Implementation: JEP 333: ZGC: A Scalable Low-Latency Garbage Collector (Experimental)

2018-06-04 Thread Magnus Ihse Bursie
Hi Per, Please always include build-dev when proposing build changes. The changes in make/autoconf/hotspot.m4 looks a bit suspect. Since you are adding zgc to disabled jvm features, which always override enabled features, it will never be possible to enable zgc on any other platform. Is this