8253902: G1: Starting a new marking cycle before the conc mark thread fully completed causes assertion failure

Reviewed-by: kbarrett, sjohanss

Author: Thomas Schatzl

src/hotspot/share/gc/g1/g1ConcurrentMarkThread.cpp

@@ -164,8 +164,6 @@ void G1ConcurrentMarkThread::stop_service() {
 }
 
 bool G1ConcurrentMarkThread::wait_for_next_cycle() {
-  assert(!in_progress(), "should have been cleared");
-
   MonitorLocker ml(CGC_lock, Mutex::_no_safepoint_check_flag);
   while (!in_progress() && !should_terminate()) {
     ml.wait();