RFR: JDK-8058510 - race condition in UART write

Alexey Konstantinov alexey.konstantinov at oracle.com
Tue Sep 16 21:12:09 UTC 2014


Hi Jen,

The change looks good.

Alexey

On 17.09.2014 0:58, Jen Dority wrote:
> Updated the webrev in place.
>
> Changed to assign sig->parameter when signalling before waiting, to 
> check for memory allocation failure and also fixed the error return 
> code for waiitForSignal -- was BUSY, now OOM.
>
> Jen
>
> On 9/16/2014 3:41 PM, Sergey Nazarkin wrote:
>> Hi Jen
>>
>> shouldn’t we assign signal parameter somewhere after line 317?
>>
>> /Sergey
>>
>> On 16 Sep 2014, at 23:24, Jen Dority <jen.dority at oracle.com> wrote:
>>
>>> Still looking for a review for this.
>>>
>>> On 9/15/2014 5:34 PM, Jen Dority wrote:
>>>> issue: https://bugs.openjdk.java.net/browse/JDK-8058510
>>>> webrev: http://cr.openjdk.java.net/~jld/8058510/
>>>>
>>>> Hi All,
>>>>
>>>> Need a review for this changeset which fixes a hang when writing to 
>>>> UART devices.
>>>>
>>>> Thanks,
>>>> Jen
>>>>
>>>>
>



More information about the dio-dev mailing list