Hi Lin,

Adding in hotspot-gc-dev as they need to see how this interacts with GC worker threads, and whether it needs to be extended beyond G1.

I happened to spot one nit when browsing:

src/hotspot/share/gc/shared/collectedHeap.hpp

+   virtual bool run_par_heap_inspect_task(KlassInfoTable* cit,
+                                          BoolObjectClosure* filter,
+                                          size_t* missed_count,
+                                          size_t thread_num) {
+     return NULL;

s/NULL/false/

Cheers,
David

On 18/02/2020 2:15 pm, linzang(臧琳) wrote:
Dear All,
      May I ask your help to review the follow changes:
     webrev: http://cr.openjdk.java.net/~lzang/jmap-8214535/8215264/webrev_00/
      bug: https://bugs.openjdk.java.net/browse/JDK-8215624
      related CSR: https://bugs.openjdk.java.net/browse/JDK-8239290
      This patch enable parallel heap inspection of G1 for jmap histo.
     my simple test shown it can speed up 2x of jmap -histo with parallelThreadNum set to 2 for heap at ~500M on 4-core platform.

------------------------------------------------------------------------
BRs,
Lin

Reply via email to