Hi Goetz, Eric,

I was unable to commit this patch before the jdk10 repo consolidation and
would like to do so now.

Last Webrev:
http://cr.openjdk.java.net/~stuefe/webrevs/8187228-make-data-segment-page-size-64K-by-default/webrev.01/webrev/

Nothing changed, the webrev is just rebased to the new repo.

I am unclear whether I am allowed to push this on my own? Does the build
group this?

Kind Regards, Thomas


On Thu, Sep 7, 2017 at 2:33 PM, Thomas Stüfe <thomas.stu...@gmail.com>
wrote:

> Thank you Goetz!
>
> On Thu, Sep 7, 2017 at 2:18 PM, Lindenmaier, Goetz <
> goetz.lindenma...@sap.com> wrote:
>
>> Hi Thomas,
>>
>> Looks good.
>>
>> Best regards,
>>   Goetz.
>>
>> > -----Original Message-----
>> > From: ppc-aix-port-dev [mailto:ppc-aix-port-dev-
>> > boun...@openjdk.java.net] On Behalf Of Thomas Stüfe
>> > Sent: Donnerstag, 7. September 2017 12:02
>> > To: ppc-aix-port-...@openjdk.java.net; build-dev <build-
>> > d...@openjdk.java.net>
>> > Subject: RFR(xs): 8187228: [aix] make data segment page size 64K by
>> default
>> >
>> > Hi all,
>> >
>> > may I please have a review for the following AIX-only patch.
>> >
>> > Issue:
>> > https://bugs.openjdk.java.net/browse/JDK-8187228
>> > <https://bugs.openjdk.java.net/browse/JDK-8187228>
>> >
>> >
>> > webrev:
>> > http://cr.openjdk.java.net/~stuefe/webrevs/8187228-make-data-segment-
>> > page-size-64K-by-default/webrev.00/webrev/
>> > <http://cr.openjdk.java.net/~stuefe/webrevs/8187228-make-data-
>> > segment-page-size-64K-by-default/webrev.00/webrev/>
>> >
>> >
>> > It changes the default page size for the data segment (C-heap and Posix
>> > Thread Stacks) to 64K.
>> >
>> >
>> > For completeness and simplicity sake, it also changes default page size
>> for
>> > text segment and primordial thread stacl to 64K, but that does not
>> matter
>> > much.
>> >
>> > For more details, please see the issue description.
>> >
>> > Thank you, Thomas
>>
>
>

Reply via email to