8253647: Remove dead code in os::create_thread() on Linux/BSD

zhengyu123 · zhengyu123 · commit 4167540074fa · 2020-09-25T23:33:42.000Z
Reviewed-by: stuefe, dholmes
diff --git a/src/hotspot/os/bsd/os_bsd.cpp b/src/hotspot/os/bsd/os_bsd.cpp
@@ -782,13 +782,6 @@ bool os::create_thread(Thread* thread, ThreadType thr_type,
 
   }
 
-  // Aborted due to thread limit being reached
-  if (state == ZOMBIE) {
-    thread->set_osthread(NULL);
-    delete osthread;
-    return false;
-  }
-
   // The thread is returned suspended (in state INITIALIZED),
   // and is started higher up in the call chain
   assert(state == INITIALIZED, "race condition");
diff --git a/src/hotspot/os/linux/os_linux.cpp b/src/hotspot/os/linux/os_linux.cpp
@@ -978,13 +978,6 @@ bool os::create_thread(Thread* thread, ThreadType thr_type,
     }
   }
 
-  // Aborted due to thread limit being reached
-  if (state == ZOMBIE) {
-    thread->set_osthread(NULL);
-    delete osthread;
-    return false;
-  }
-
   // The thread is returned suspended (in state INITIALIZED),
   // and is started higher up in the call chain
   assert(state == INITIALIZED, "race condition");

Original file line number	Diff line number	Diff line change
`@@ -978,13 +978,6 @@ bool os::create_thread(Thread* thread, ThreadType thr_type,`
`978`	`978`	`}`
`979`	`979`	`}`
`980`	`980`
`981`		`- // Aborted due to thread limit being reached`
`982`		`- if (state == ZOMBIE) {`
`983`		`- thread->set_osthread(NULL);`
`984`		`- delete osthread;`
`985`		`- return false;`
`986`		`- }`
`987`		`-`
`988`	`981`	`// The thread is returned suspended (in state INITIALIZED),`
`989`	`982`	`// and is started higher up in the call chain`
`990`	`983`	`assert(state == INITIALIZED, "race condition");`