Print this page
rev 3618 : 7193946: Move warnings associated with UseMemSetInBOT flag
Summary: The warnings associated with the UseMemSetInBOT flag are duplicated in CMS and G1. The separate warnings have been removed and single instance of the warning has been placed in a common location.
Reviewed-by:

Split Close
Expand all
Collapse all
          --- old/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
          +++ new/src/share/vm/gc_implementation/g1/g1CollectedHeap.cpp
↓ open down ↓ 1946 lines elided ↑ open up ↑
1947 1947      q->initialize();
1948 1948      _task_queues->register_queue(i, q);
1949 1949    }
1950 1950  
1951 1951    clear_cset_start_regions();
1952 1952  
1953 1953    // Initialize the G1EvacuationFailureALot counters and flags.
1954 1954    NOT_PRODUCT(reset_evacuation_should_fail();)
1955 1955  
1956 1956    guarantee(_task_queues != NULL, "task_queues allocation failure.");
1957      -#ifdef SPARC
1958      -  // Issue a stern warning, but allow use for experimentation and debugging.
1959      -  if (VM_Version::is_sun4v() && UseMemSetInBOT) {
1960      -    assert(!FLAG_IS_DEFAULT(UseMemSetInBOT), "Error");
1961      -    warning("Experimental flag -XX:+UseMemSetInBOT is known to cause instability"
1962      -            " on sun4v; please understand that you are using at your own risk!");
1963      -  }
1964      -#endif
1965 1957  }
1966 1958  
1967 1959  jint G1CollectedHeap::initialize() {
1968 1960    CollectedHeap::pre_initialize();
1969 1961    os::enable_vtime();
1970 1962  
1971 1963    G1Log::init();
1972 1964  
1973 1965    // Necessary to satisfy locking discipline assertions.
1974 1966  
↓ open down ↓ 4496 lines elided ↑ open up ↑
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX