[ppc-aix-port/stage]: Request for approval: 8034801: AIX: (ch) Fix synchronization issue in AixPollPort.java

[Iris Clark]

Hi, Volker, 

You're correct.  Get review to push for stage.  We'll ask for bulk approval for integration into jdk8u.  That team would typically want to see a review thread for a backport so we want to make sure that we have similar reviews.

Everything looks fine to me.

Thanks,
iris

-----Original Message-----
From: Volker Simonis [mailto:volker.simonis at gmail.com] 
Sent: Tuesday, February 18, 2014 2:34 AM
To: Alan Bateman
Cc: nio-dev; ppc-aix-port-dev at openjdk.java.net
Subject: Re: [ppc-aix-port/stage]: Request for approval: 8034801: AIX: (ch) Fix synchronization issue in AixPollPort.java

Hi Alan,

thanks for the approval.

I'm actually not sure what the actual procedure is, but in general all changes for stage require a review, so I thought I'd better ask. We will then later on request another approval for the bulk integration into jdk8u.

Regards,
Volker


On Tue, Feb 18, 2014 at 11:04 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
> On 18/02/2014 09:50, Volker Simonis wrote:
>>
>> Hi,
>>
>> can I please get approval for downporting this AIX-only change from 
>> jdk9/dev to ppc-aix-port/stage which will then evetnually be pushed 
>> to
>> 8u-dev:
>>
>> http://cr.openjdk.java.net/~simonis/webrevs/8034801/
>> https://bugs.openjdk.java.net/browse/JDK-8034801
>> http://hg.openjdk.java.net/jdk9/dev/jdk/rev/c20c6a1eb13c
>>
> Does importing changes into ppc-aix-port/stage normally require approvals?
> If so then it looks okay to me. There will be of course need to be 
> approval on the jdk8u-dev list to bring all the changes into jdk8u later.
>
> -Alan