RFR(9): 8176533: REGRESSION: a java process is not recognized by jcmd/jinfo/jstack/jmap tool

serguei.spitsyn at oracle.com serguei.spitsyn at oracle.com
Thu Mar 16 04:05:59 UTC 2017

Hi Robbin,

You have two reviews now, so it is Ok to push assuming the testing is good.
Please, let us know if anything else is needed.

It seems, today is the last day before an approval will be required for 
a push.


On 3/15/17 11:14, Dmitry Samersoff wrote:
> Robbin,
> Looks good to me.
> -Dmitry
> On 2017-03-15 20:41, Robbin Ehn wrote:
>> Hi all, please review:
>> When starting the VM with agentlib and suspended, jcmd/jinfo/jstack/jmap
>> don't work until a debugger have attach.
>> There is an underlying issue with AttachProvider that don't list that VM
>> even if it's attachable.
>> That issue is present in JDK 8 (“jcmd -l” don’t list this process).
>> This fix only fixes the regression by not validating the pid against
>> available VirtualMachineDescriptors.
>> One problem with not validating is that we will send SIGQUIT to that pid
>> even if it’s not a java process, but it’s the same behavior as JDK 8.
>> The regression was introduced by
>> https://bugs.openjdk.java.net/browse/JDK-8156537
>> VirtualMachineDescriptor was not easily create-able from just a string
>> pid, you need an AttachProvider.
>> I instead change so we use a collection of pids where applicable.
>> Webrev: http://cr.openjdk.java.net/~rehn/8176533/webrev/
>> Issue: https://bugs.openjdk.java.net/browse/JDK-8176533
>> Passes jdk/test/sun/tools/ and internal test group.
>> Manual tested the regression case.
>> A new issue for the AttachProvider will be filed.
>> Thanks
>> /Robbin

More information about the hotspot-runtime-dev mailing list