[v3,0/3] mdsmap: fix mds choosing
mbox series

Message ID 20191126122422.12396-1-xiubli@redhat.com
Headers show
Series
  • mdsmap: fix mds choosing
Related show

Message

Xiubo Li Nov. 26, 2019, 12:24 p.m. UTC
From: Xiubo Li <xiubli@redhat.com>

V2:
- ignore laggy for the auth mds case
- for the random mds choosing, get one none laggy first and only
  when there has no we will fall back to choose a laggy one if there
  has.

V3:
- add the mds sanity check

Xiubo Li (3):
  mdsmap: add more debug info when decoding
  mdsmap: fix mdsmap cluster available check based on laggy number
  mdsmap: only choose one MDS who is in up:active state without laggy

 fs/ceph/mds_client.c        | 13 +++++--
 fs/ceph/mdsmap.c            | 74 ++++++++++++++++++++-----------------
 include/linux/ceph/mdsmap.h |  1 +
 3 files changed, 51 insertions(+), 37 deletions(-)

Comments

Jeff Layton Nov. 27, 2019, 11:07 a.m. UTC | #1
On Tue, 2019-11-26 at 07:24 -0500, xiubli@redhat.com wrote:
> From: Xiubo Li <xiubli@redhat.com>
> 
> V2:
> - ignore laggy for the auth mds case
> - for the random mds choosing, get one none laggy first and only
>   when there has no we will fall back to choose a laggy one if there
>   has.
> 
> V3:
> - add the mds sanity check
> 
> Xiubo Li (3):
>   mdsmap: add more debug info when decoding
>   mdsmap: fix mdsmap cluster available check based on laggy number
>   mdsmap: only choose one MDS who is in up:active state without laggy
> 
>  fs/ceph/mds_client.c        | 13 +++++--
>  fs/ceph/mdsmap.c            | 74 ++++++++++++++++++++-----------------
>  include/linux/ceph/mdsmap.h |  1 +
>  3 files changed, 51 insertions(+), 37 deletions(-)
> 

Ok, I think this looks sane enough. I'll merge it into the testing
branch and we'll see how it does.

Thanks,