RFR: JDK-8261034: improve jcmd GC.class_histogram to support parallel

[Hamlin Li]

On Thu, 4 Feb 2021 08:14:47 GMT, Hamlin Li <mli at openjdk.org> wrote:

>> From what I've seen it's normal for the 32-bit builds to fail. I'm not sure why they are even being built.
>
>> From what I've seen it's normal for the 32-bit builds to fail. I'm not sure why they are even being built.
> 
> Thanks for the info, then I think it's safe for me to ignore the failures in linux x86.

> _Mailing list message from [David Holmes](mailto:david.holmes at oracle.com) on [hotspot-runtime-dev](mailto:hotspot-runtime-dev at openjdk.java.net):_
> 
> On 4/02/2021 5:39 pm, Hamlin Li wrote:
> 
> > On Thu, 4 Feb 2021 03:40:26 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
> > > > Hi @Hamlin-Li
> > > > I am not reviewer, but maybe you could add test for "parallel" in BasicJMapTest.java.
> > > 
> > > 
> > > > I am not reviewer, but maybe you could add test for "parallel" in BasicJMapTest.java.
> > > 
> > > 
> > > That test is for the `jmap` command, not the `GC.class_histogram` jcmd. As I pointed out, there are tests in `test/hotpot/jtreg/serviceability/dcmd/gc` that already test `GC.class_histogram`. That would be a better starting point.
> > 
> > 
> > @plummercj Hi Chris, I have a question which might not be related to the patch, my presubmit tests always failed on linux platforms, but I don't think the failure is related to my patch. Would you mind to share some point? I totally have no idea what's going wrong.
> > The error messages are as following:
> > E: Could not configure 'libc6:i386'.
> > E: Could not perform immediate configuration on 'libgcc-s1:i386'. Please see man 5 apt.conf under APT::Immediate-Configure for details. (2)
> > Error: Process completed with exit code 100.
> 
> I believe this has been fixed, so your repo may be missing that
> changeset at the moment.
> 
> David

Hi David, Thank you for the information.

-------------

PR: https://git.openjdk.java.net/jdk/pull/2379