Simple Request for review: 6535709 - lost interrupts on win32

David Holmes David.Holmes at oracle.com
Sun Mar 20 19:31:20 PDT 2011


Hi Dan,

Daniel D. Daugherty said the following on 03/21/11 12:12:
> On 3/20/2011 7:46 PM, David Holmes wrote:
>> http://cr.openjdk.java.net/~dholmes/6535709/webrev/
> Thumbs up.

Thanks for the review.

> Nice fix. And very nice analysis in the bug report(s).
> I'm going to crawl over the Serviceability bugs because I
> think there are a couple that are "interrupt related".

I did take a quick run through myself to see if there were others, but I 
didn't see anything obviously Windows+interrupt specific.

> src/share/vm/runtime/vmStructs.cpp
>    Did you mean to change the comment starter on line 1?

Nope. Fixed.

David

> Dan
> 
> 
>>
>> A trivial fix (in the end) for an issue already avoided on 
>> Solaris/Linux. It finally closes a set of bugs that have been open for 
>> years.
>>
>> David


More information about the hotspot-runtime-dev mailing list