From ba3e8dca975eafea80b335663eabf8b65465f987 Mon Sep 17 00:00:00 2001 From: DarkLight1337 Date: Fri, 17 Oct 2025 08:30:25 +0000 Subject: [PATCH] Comment Signed-off-by: DarkLight1337 --- vllm/benchmarks/serve_multi.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/vllm/benchmarks/serve_multi.py b/vllm/benchmarks/serve_multi.py index 1758470d022b3..b2e339081140d 100644 --- a/vllm/benchmarks/serve_multi.py +++ b/vllm/benchmarks/serve_multi.py @@ -98,9 +98,9 @@ def benchmark_one_run( subprocess.run(benchmark_cmd, check=True) finally: if server_process.poll() is None: - # Process might already be terminated + # In case some processes have been terminated with contextlib.suppress(ProcessLookupError): - # Kill entire process group + # We need to kill both API Server and Engine processes os.killpg(os.getpgid(server_process.pid), signal.SIGKILL) with result_path.open("rb") as f: