remove the mReconstructionMutex in load

During loading of snapshot, there will be a single-threaded
decoder that aquires the same mReconstructionMutex, repeatedly.
Since the mReconstructionMutex is intentionally changed to
be non-recursive, we should not aquire it at the beginning of
load call; otherwise, we will be deadlock the decoder thread.

Reviewed-by: Marcin Radomski <dextero@google.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/33018>
This commit is contained in:
Bo Hu 2025-01-07 10:06:16 -08:00 committed by Marge Bot
parent 2f3c3459a8
commit 26ce3b0ba1

View file

@ -68,7 +68,6 @@ public:
}
void load(android::base::Stream* stream, GfxApiLogger& gfx_logger, HealthMonitor<>* healthMonitor) {
std::lock_guard<std::mutex> lock(mReconstructionMutex);
mReconstruction.load(stream, gfx_logger, healthMonitor);
}