Thanks Marcus for all three reviews!

> On 20 Oct 2016, at 15:50, Marcus Larsson <marcus.lars...@oracle.com> wrote:
> 
> Hi,
> 
> 
>> On 10/20/2016 01:42 PM, Staffan Larsen wrote:
>> Please review this small update to the list of tools that the jtreg 
>> timeouthandler runs when a timeout is hit. This change removes all calls to 
>> the old SA-tools and instead uses Diagnostic Commands to gather the same 
>> information. The possible drawback is that Diagnostic Commands do not work 
>> on processes where the JVM is completely hung, but in that case we can 
>> fallback to running the SA-tools on the core file that is also generated.
>> 
>> webrev: http://cr.openjdk.java.net/~sla/8168409/webrev.00 
>> <http://cr.openjdk.java.net/~sla/8168409/webrev.00>
> 
> Looks good to me.
> 
> Thanks,
> Marcus
> 
>> bug: https://bugs.openjdk.java.net/browse/JDK-8168409 
>> <https://bugs.openjdk.java.net/browse/JDK-8168409>
>> 
>> Thanks,
>> /Staffan
> 

Reply via email to