RE: RFR [XS] : 8226943: compile error in libfollowref003.cpp with XCode 10.2 on macosx

2019-06-30 Thread Baesken, Matthias
Hello, thanks for the review ! > I'd suggest to fix it in 13 as it is the test fix. I'll push it then to 13 , fine with me ! Best regards, Matthias > > Hi Matthias, > > The fix is good. > It worked before because both JVMTI_REFERENCE_ARRAY_ELEMENT > and JVMTI_HEAP_REFERENCE_ARRAY_ELEMENT ha

RFR: 8225715: jhsdb jmap fails to write binary heap dump of a jshell process

2019-06-30 Thread Fairoz Matte
Hi, Please review a fix that potentially handle NPE and allow heap to dump from a process were we get source file name as null. Background: For taking heap dump of JShell process, we get getSourceFileName() as null. Current implementation doesn't have null check. This fix will handle the null

Re: JFR thread sampling mechanism

2019-06-30 Thread Kirk Pepperdine
Hi Gil, I would support an improvement in sampling as there is an obvious bias which allows me to write benchmarks where JFR completely misses things it should find. That said, I’m not sure that waking a thread up every 10ms is a great idea as it is very disruptive to Linux thread scheduling. I

Re: JFR thread sampling mechanism

2019-06-30 Thread David Holmes
Hi Gil, Redirecting to the hotspot-jfr-dev alias. Cheers, David On 30/06/2019 5:20 pm, Gil Tene wrote: I would like to discuss a potential improvement to the JFR thread sampling mechanism, and would like to see if the change we'd propose has already been considered in the past. I believe that

JFR thread sampling mechanism

2019-06-30 Thread Gil Tene
I would like to discuss a potential improvement to the JFR thread sampling mechanism, and would like to see if the change we'd propose has already been considered in the past. I believe that the current thread sampling mechanism (mostly via hotspot/share/jfr/periodic/sampling/jfrThreadSampler.cpp)