RFR: 7155300 Include pthread.h on all POSIX platforms except Solaris to improve portability

[Charles Lee]

On 03/21/2012 10:47 AM, Shi Jun Zhang wrote:
> On 3/12/2012 11:28 AM, Shi Jun Zhang wrote:
>> On 3/9/2012 6:05 PM, David Holmes wrote:
>>> On 9/03/2012 7:04 PM, Alan Bateman wrote:
>>>> On 09/03/2012 08:01, Shi Jun Zhang wrote:
>>>>> The situation in NativeThread.c is more complicated than other 2
>>>>> files. I'm not familiar with BSD or Mac. It seems that we don't need
>>>>> to signal threads on BSD or Mac. And INTERRUPT_SIGNAL on AIX will
>>>>> definitely be different from the one on Linux. I think we'd better
>>>>> separate the changes in NativeThread.c from this patch and try to
>>>>> solve it later.
>>>> Right, if signals are required there is likely to be differences 
>>>> across
>>>> platforms. It is also likely that this code will need to be changed 
>>>> for
>>>> Mac too as there are a couple of preemptive close issues to sort out
>>>> (for file operations, sockets are okay).
>>>>
>>>>>>
>>>>>> The change to socket_md.c looks okay to me but you will need to
>>>>>> re-base your patch due to the Mac port in jdk8/tl.
>>>>> I'm a new comer and i got known from Charles about the difference
>>>>> between jdk8 and jdk8/tl. The latest webrev is based on jdk8/tl.
>>>>>
>>>>> http://cr.openjdk.java.net/~zhangshj/pthread/webrev.01/
>>>> The changes in this webrev look okay to me.
>>>
>>> In java_md.c
>>>
>>> 1445       /* See above. Continue in current thread if thr_create() 
>>> failed */
>>>
>>> The "see above" is now a "see below".
>>>
>>> I think the launcher changes are okay because BSD/OSX won't use this 
>>> file.
>>>
>>> I think the socket changes are okay as long as BSD builds and OSX 
>>> builds define _ALLBSD_SOURCE. I still don't fully understand if a 
>>> BSD build and an OSX build are distinct.
>>>
>>> David
>>>
>>>> -Alan.
>>>
>> The comment "see above" has been changed to "see below".
>>
>> http://cr.openjdk.java.net/~zhangshj/pthread/webrev.02/
>>
> Hi Alan/David,
>
> There is no response on this thread for long time. I created a sun bug 
> 7155300, could you help to review it?
>
> The webrev link is 
> http://cr.openjdk.java.net/~zhangshj/pthread/webrev.02/
>
Hi Chance,

Here is the changeset. Please verify it.

Changeset: 1d418ec212ea
Author:    zhangshj
Date:      2012-03-22 12:30 +0800
URL:http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1d418ec212ea

7155300: Include pthread.h on all POSIX platforms except Solaris to improve portability
Reviewed-by: alanb, dholmes

! src/solaris/bin/java_md.c
! src/solaris/transport/socket/socket_md.c


Thank you all for reviewing.

-- 
Yours Charles