]> git.djapps.eu Git - pkg/ggml/sources/whisper.cpp/commit
ggml: Fix data race in ggml threadpool (llama/11736)
authorKarol Kontny <redacted>
Sat, 8 Feb 2025 14:30:53 +0000 (15:30 +0100)
committerGeorgi Gerganov <redacted>
Thu, 27 Feb 2025 06:55:36 +0000 (08:55 +0200)
commit5981352bb5189b062c90b9953b8ceb8ca1da4b5e
treea4cd862b271397911ee32b397380c95defd2202a
parent7561da244e9b270c600365991f2a7fb0c30e2b17
ggml: Fix data race in ggml threadpool (llama/11736)

After the barrier in last iteration is executed, still the loop termination
condition will be executed. However main thread can destroy the cgraph object
and its nodes already, then another thread will access it, but the thing is already gone.
Also trouble can happen when n_nodes == 0 or abort is called, but I'm not sure if the
prior situation is possible.

Last syncronization should be done after the loop to ensure the cgraph/cplan won't be
accessed after the main thread exits from the function.
ggml/src/ggml-cpu/ggml-cpu.c