]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit - fs/ceph/mds_client.c
ceph: use frag's MDS in either mode
authorYanhu Cao <gmayyyha@gmail.com>
Fri, 31 Jul 2020 08:25:13 +0000 (16:25 +0800)
committerIlya Dryomov <idryomov@gmail.com>
Tue, 4 Aug 2020 17:41:14 +0000 (19:41 +0200)
commit224c7b6778fe08e1880ef88867051bec0a154d6c
treec8050bd27e2ea9d6c7402af3204113b1dda656ff
parenta0102bda5bc0991c5c8c7c07770b236894a810fd
ceph: use frag's MDS in either mode

When doing some tests with multiple mds, we were seeing many mds
forwarding requests between them, causing clients to resend.

If the request is a modification operation and the mode is set to
USE_AUTH_MDS, then the auth mds should be selected to handle the
request. If auth mds for frag is already set, then it should be returned
directly without further processing.

The current logic is wrong because it only returns directly if
mode is USE_AUTH_MDS, but we want to do that for all modes. If we don't,
then when the frag's mds is not equal to cap session's mds, the request
will get sent to the wrong MDS needlessly.

Drop the mode check in this condition.

Signed-off-by: Yanhu Cao <gmayyyha@gmail.com>
Reviewed-by: Jeff Layton <jlayton@kernel.org>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
fs/ceph/mds_client.c