Thanks, Tim! Regards, Volker
Tim Bell <tim.b...@oracle.com> schrieb am Mi. 19. Juli 2017 um 03:30: > Hello - > > This fix has been pushed to OpenJDK jdk10/jdk10: > > http://hg.openjdk.java.net/jdk10/jdk10/rev/2fe66ca1e2b3 > > Tim > > On 07/18/17 01:58, Volker Simonis wrote: > > You need a sponsor from Oracle because of the generated configure file > > which also needs to be generated for the Oracle closed parts. > > > > Can somebody from the build folks please push this? > > > > Thanks, > > Volker > > > > > > On Tue, Jul 18, 2017 at 9:18 AM, Thomas Stüfe <thomas.stu...@gmail.com> > wrote: > >> Danke Volker! > >> > >> ... > >> > >> So, do I need a sponsor for this or can I push this on my own? > >> > >> Thanks, Thomas > >> > >> On Tue, Jul 18, 2017 at 8:58 AM, Volker Simonis < > volker.simo...@gmail.com> > >> wrote: > >>> > >>> Looks good! > >>> > >>> Thanks, > >>> Volker > >>> > >>> On Tue, Jul 18, 2017 at 8:30 AM, Thomas Stüfe <thomas.stu...@gmail.com > > > >>> wrote: > >>>> Hi Volker, > >>>> > >>>> new webrev: > >>>> > >>>> > http://cr.openjdk.java.net/~stuefe/webrevs/8184344-TOC-overflow/webrev.01/webrev/ > >>>> > >>>> Only change is the comment, as you suggested. > >>>> > >>>> Kind Regards, Thomas > >>>> > >>>> On Tue, Jul 18, 2017 at 7:46 AM, Thomas Stüfe < > thomas.stu...@gmail.com> > >>>> wrote: > >>>>> > >>>>> Thank you Volker! > >>>>> > >>>>> I'll restore the original comment and prepare a new webrev. > >>>>> > >>>>> ..Thomas > >>>>> > >>>>> On Mon, Jul 17, 2017 at 5:46 PM, Volker Simonis > >>>>> <volker.simo...@gmail.com> > >>>>> wrote: > >>>>>> > >>>>>> Hi Thomas, > >>>>>> > >>>>>> the change looks good, but I'd prefer if you leave the initial > comment > >>>>>> in place which also mentions "-qminimaltoc" as a way of resolving > TOC > >>>>>> overflow errors. > >>>>>> > >>>>>> I actually don't remember exactly, but I think "-qminimaltoc" works > by > >>>>>> creating distinct TOCs for each compilation unit. That comes with an > >>>>>> performance impact, but "-qpic=large" / "-bbigtoc" can have an > >>>>>> performance impact as well. > >>>>>> > >>>>>> As I said, for the slowdebug build your changes are fine. I'd just > >>>>>> like to keep the reference to "-qminimaltoc" for the case where we > >>>>>> have to re-evaluate the different solutions for the product build. > >>>>>> > >>>>>> Thank you and best regards, > >>>>>> Volker > >>>>>> > >>>>>> > >>>>>> On Mon, Jul 17, 2017 at 3:19 PM, Langer, Christoph > >>>>>> <christoph.lan...@sap.com> wrote: > >>>>>>> Hi Thomas, > >>>>>>> > >>>>>>> the fix looks ok to me. > >>>>>>> > >>>>>>> I’m copying build-dev because of the changes in > >>>>>>> generated-configure.sh. > >>>>>>> Don’t know if this can just be pushed from extern or if it needs > >>>>>>> some > >>>>>>> special handling from Oracle folks or other things to take care of? > >>>>>>> > >>>>>>> Best regards > >>>>>>> Christoph > >>>>>>> > >>>>>>> From: ppc-aix-port-dev > >>>>>>> [mailto:ppc-aix-port-dev-boun...@openjdk.java.net] On Behalf Of > >>>>>>> Thomas Stüfe > >>>>>>> Sent: Montag, 17. Juli 2017 12:55 > >>>>>>> To: ppc-aix-port-...@openjdk.java.net > >>>>>>> Subject: RFR(xs): 8184344: [aix] libjvm.so TOC overflow for > >>>>>>> slowdebug > >>>>>>> > >>>>>>> Hi all, > >>>>>>> > >>>>>>> may I please have a review for the following fix: > >>>>>>> > >>>>>>> webrev: > >>>>>>> > >>>>>>> > http://cr.openjdk.java.net/~stuefe/webrevs/8184344-TOC-overflow/webrev.00/webrev/ > >>>>>>> Issue: https://bugs.openjdk.java.net/browse/JDK-8184344 > >>>>>>> > >>>>>>> Basically, the TOC on AIX overflows on slowdebug builds. I am not > >>>>>>> sure > >>>>>>> yet which change caused that - I suspect one of the recent > >>>>>>> template-metaprogramming changes but have not investigated yet. > >>>>>>> > >>>>>>> Thanks, Thomas > >>>>>>> > >>>>> > >>>>> > >>>> > >> > >> > >