Hi Matthias, looks good to me.
Best regards Christoph > -----Original Message----- > From: build-dev <[email protected]> On Behalf Of > Baesken, Matthias > Sent: Donnerstag, 17. Januar 2019 07:41 > To: Steve Groeger <[email protected]> > Cc: '[email protected]' <[email protected]>; ppc-aix- > [email protected]; ppc-aix-port-dev <ppc-aix-port-dev- > [email protected]> > Subject: [CAUTION] RE: RFR [XS] 8217233: Update build settings for AIX/xlc > > May I have a second review please ? > > Best regards, Matthias > > > From: Baesken, Matthias > Sent: Mittwoch, 16. Januar 2019 11:08 > To: 'Steve Groeger' <[email protected]> > Cc: '[email protected]' <[email protected]>; ppc-aix- > [email protected]; ppc-aix-port-dev <ppc-aix-port-dev- > [email protected]> > Subject: RE: RFR [XS] 8217233: Update build settings for AIX/xlc > > Hi Steve, > from my experience the situation is similar to Linux (small performance > change). > > Best regards, Matthias > > > From: Steve Groeger > <[email protected]<mailto:[email protected]>> > Sent: Mittwoch, 16. Januar 2019 10:49 > To: Baesken, Matthias > <[email protected]<mailto:[email protected]>> > Cc: '[email protected]' <build- > [email protected]<mailto:[email protected]>>; ppc-aix-port- > [email protected]<mailto:[email protected]>; ppc- > aix-port-dev <[email protected]<mailto:ppc-aix- > [email protected]>> > Subject: Re: RFR [XS] 8217233: Update build settings for AIX/xlc > > Hi Matthias, > > The change looks OK. > > The docs state: "This option is disabled by default because it can cause > performance degradation." > Do you know whether using this option actually causes any performance > degradation on AIX? > > Thanks > Steve Groeger > IBM Runtime Technologies > Hursley, Winchester > Tel: (44) 1962 816911 Mobex: 279990 Mobile: 07718 517 129 > Fax (44) 1962 816800 > Lotus Notes: Steve Groeger/UK/IBM > Internet: [email protected]<mailto:[email protected]> > > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 > 3AU > > > > From: "Baesken, Matthias" > <[email protected]<mailto:[email protected]>> > To: "'[email protected]'" <build- > [email protected]<mailto:[email protected]>>, "ppc-aix- > [email protected]<mailto:[email protected]>" > <[email protected]<mailto:ppc-aix-port- > [email protected]>> > Date: 16/01/2019 09:26 > Subject: RFR [XS] 8217233: Update build settings for AIX/xlc > Sent by: "ppc-aix-port-dev" <ppc-aix-port-dev- > [email protected]<mailto:ppc-aix-port-dev- > [email protected]>> > ________________________________ > > > > Hello, please review this small fix . > > It adds stack protection compile flags for xlc/AIX . > > See : > > https://www.ibm.com/support/knowledgecenter/SSGH2K_12.1.0/com.ibm. > xlc121.aix.doc/compiler_ref/opt_stackprotect.html > > > "Provides protection against malicious code or programming errors that > overwrite or corrupt the stack." > The patch will set the stack protection flags . > > > ( Similar changes have been done for Linux : 8199552: Update to build > scripts > https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/8b060cdf0251 > Introduced stack protection compile flags for Linux/gcc ( -fstack-protector > ) . > ) > > > Bug/webrev : > > https://bugs.openjdk.java.net/browse/JDK-8217233 > > http://cr.openjdk.java.net/~mbaesken/webrevs/8217233.0/ > > > Thanks, Matthias > > > Unless stated otherwise above: > IBM United Kingdom Limited - Registered in England and Wales with number > 741598. > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 > 3AU
