RFR(xs): 8184344: [aix] libjvm.so TOC overflow for slowdebug

[Volker Simonis]

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.stuefe at 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.simonis at gmail.com>
> wrote:
>>
>> Looks good!
>>
>> Thanks,
>> Volker
>>
>> On Tue, Jul 18, 2017 at 8:30 AM, Thomas Stüfe <thomas.stuefe at 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.stuefe at 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.simonis at 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.langer at 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-bounces at openjdk.java.net] On Behalf Of
>> >>> > Thomas Stüfe
>> >>> > Sent: Montag, 17. Juli 2017 12:55
>> >>> > To: ppc-aix-port-dev at 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
>> >>> >
>> >>
>> >>
>> >
>
>