[11] RFR(S) : 8205687 : TimeoutHandler generates huge core files

[Igor Ignatyev]

Hi David,

thanks for your review. I've updated copyright years and added the next comment in GathererFactory.java:
> +            // os-specific action set must be last, b/c they can kill the process 


regarding checking w/ the people operating their own farms, I agree it'd be nice to do for changes like this one, but I don't know a better channel to do other than an email to dev alias, which wouldn't be much different from this RFR. 

In this particular case, I expect the hosts in all test farms to have core dumping enabled and core dumps from crashes are, from my observations, used more frequently than dumps from timeouts, so it shouldn't cause big issues.

Thanks,
-- Igor

> On Aug 7, 2018, at 9:00 PM, David Holmes <david.holmes at oracle.com> wrote:
> 
> Hi Igor,
> 
> This all seems okay. However we probably need to check that the people operating their own build/test farms are okay with this change.
> 
> Don't forget to update copyright years.
> 
> Minor comment below ...
> 
> On 8/08/2018 7:19 AM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8205687/webrev.00/index.html
>>> 18 lines changed: 3 ins; 5 del; 10 mod;
>> Hi all,
>> could you please review this small fix for TimeoutHandler? gcore dumps non committed memory, which is a problem when JVM is run w/ ZGC or in other configurations where memory is reserved but not committed. the fix replaces invocation of gcore w/ 'kill -ABRT'. as kill has to be the last command run by TimeoutHandler, I've changed jdk.test.failurehandler.ToolKit to run action after we get a list of the process's children and also changed GathererFactory to run "common" part before os-specific part.
> 
> Can you add a comment in GathererFactory to explain that as well please.
> 
> Thanks,
> David
> 
>> NB 'kill' will generate core file only if a host has appropriate core file limit and core file handling settings, therefore you might observe that core files aren't generated if your hosts aren't set up correspondingly.
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8205687
>> webrev: http://cr.openjdk.java.net/~iignatyev//8205687/webrev.00/index.html
>> Thanks,
>> -- Igor