Revert the addition of subprocess termination handling via Thread

because it can cause double subprocess communicate calls

Signed-off-by: Min Lim <min.yeol.lim@intel.com>

Author: mlim19

gprofiler/utils/__init__.py

@@ -33,7 +33,7 @@
 from pathlib import Path
 from subprocess import CompletedProcess, Popen, TimeoutExpired
 from tempfile import TemporaryDirectory
-from threading import Event, Thread
+from threading import Event
 from types import FrameType
 from typing import Any, Callable, Dict, Iterator, List, Optional, Tuple, Union
 
@@ -93,7 +93,6 @@ def start_process(
     cmd: Union[str, List[str]],
     via_staticx: bool = False,
     tmpdir: Optional[Path] = None,
-    direct_call: bool = True,
     **kwargs: Any,
 ) -> Popen:
     if isinstance(cmd, str):
@@ -137,15 +136,6 @@ def start_process(
         **kwargs,
     )
 
-    def process_exit_handler() -> None:
-        process.communicate()
-        # Remove from _processes and do any cleanup
-        cleanup_process_reference(process)
-        logger.critical(f"Process {process.pid} exited, total processes tracked {len(_processes)}")
-
-    _processes.append(process)
-    if direct_call:
-        Thread(target=process_exit_handler, daemon=True).start()
     return process
 
 
@@ -230,7 +220,7 @@ def run_process(
     stderr: bytes
 
     reraise_exc: Optional[BaseException] = None
-    with start_process(cmd, via_staticx, direct_call=False, **kwargs) as process:
+    with start_process(cmd, via_staticx, **kwargs) as process:
         assert isinstance(process.args, str) or (
             isinstance(process.args, list) and all(isinstance(s, str) for s in process.args)
         ), process.args  # mypy