]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
mds: switch submit_mutex to fair mutex for MDLog
authorXiubo Li <xiubli@redhat.com>
Mon, 6 Dec 2021 00:59:36 +0000 (08:59 +0800)
committerXiubo Li <xiubli@redhat.com>
Thu, 10 Nov 2022 01:48:20 +0000 (09:48 +0800)
commit0366b807c50345aebdc7b83103568468186ebe57
treea8c8c8e16654ec537dce79d8d2b4930fe911d58d
parent1cdcfccae7ef0a706eb3bea7742f36a6810d1245
mds: switch submit_mutex to fair mutex for MDLog

The implementations of the Mutex (e.g. std::mutex in C++) do not
guarantee fairness, they do not guarantee that the lock will be
acquired by threads in the order that they called the lock().

In most case this works well, but in overload case the client
requests handling thread and _submit_thread could always successfully
acquire the submit_mutex in a long time, which could make the
MDLog::trim() get stuck. That means the MDS daemons will fill journal
logs into the metadata pool, but couldn't trim the expired segments
in time.

This will switch the submit_mutex to fair mutex and it could make
sure that the all the submit_mutex waiters are in FIFO order and
could get a change to be excuted in time.

Fixes: https://tracker.ceph.com/issues/58000
Signed-off-by: Xiubo Li <xiubli@redhat.com>
src/mds/MDLog.h