src/hotspot/share/gc/shared/gcArguments.cpp

rev 48019 : 8191821: Finer granularity for GC verification
Reviewed-by:
rev 48020 : [mq]: 8191821-rev-sang-poonam

*** 83,94 ****
    UNSUPPORTED_OPTION(UseConcMarkSweepGC);
    FLAG_SET_ERGO_IF_DEFAULT(bool, UseSerialGC, true);
  #endif // INCLUDE_ALL_GCS
  }
  
! void GCArguments::parse_verification_type(const char* type) {
    log_warning(gc, verify)("VerifyGCType is not supported by this collector.");
  }
  
  void GCArguments::initialize_flags() {
  #if INCLUDE_ALL_GCS
    if (MinHeapFreeRatio == 100) {
--- 83,96 ----
    UNSUPPORTED_OPTION(UseConcMarkSweepGC);
    FLAG_SET_ERGO_IF_DEFAULT(bool, UseSerialGC, true);
  #endif // INCLUDE_ALL_GCS
  }
  
! bool GCArguments::parse_verification_type(const char* type) {
    log_warning(gc, verify)("VerifyGCType is not supported by this collector.");
+   // Return false to avoid multiple warnings.
+   return false;
  }
  
  void GCArguments::initialize_flags() {
  #if INCLUDE_ALL_GCS
    if (MinHeapFreeRatio == 100) {
*** 110,120 ****
      size_t length = strlen(VerifyGCType);
      char* type_list = NEW_C_HEAP_ARRAY(char, length + 1, mtInternal);
      strncpy(type_list, VerifyGCType, length + 1);
      char* token = strtok(type_list, delimiter);
      while (token != NULL) {
!       parse_verification_type(token);
        token = strtok(NULL, delimiter);
      }
      FREE_C_HEAP_ARRAY(char, type_list);
    }
  }
--- 112,125 ----
      size_t length = strlen(VerifyGCType);
      char* type_list = NEW_C_HEAP_ARRAY(char, length + 1, mtInternal);
      strncpy(type_list, VerifyGCType, length + 1);
      char* token = strtok(type_list, delimiter);
      while (token != NULL) {
!       bool success = parse_verification_type(token);
!       if (!success) {
!         break;
!       }
        token = strtok(NULL, delimiter);
      }
      FREE_C_HEAP_ARRAY(char, type_list);
    }
  }