On Wed, Jul 14, 2021 at 9:47 PM Jeff Layton <jlayton@xxxxxxxxxx> wrote:
>
> On Wed, 2021-07-14 at 15:35 +0530, Venky Shankar wrote:
> > Note that the new monitors are just shown in /proc/mounts.
> > Ceph does not (re)connect to new monitors yet.
> >
> > Signed-off-by: Venky Shankar <vshankar@xxxxxxxxxx>
> > ---
> > fs/ceph/super.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/fs/ceph/super.c b/fs/ceph/super.c
> > index d8c6168b7fcd..d3a5a3729c5b 100644
> > --- a/fs/ceph/super.c
> > +++ b/fs/ceph/super.c
> > @@ -1268,6 +1268,13 @@ static int ceph_reconfigure_fc(struct fs_context *fc)
> > else
> > ceph_clear_mount_opt(fsc, ASYNC_DIROPS);
> >
> > + if (strcmp(fsc->mount_options->mon_addr, fsopt->mon_addr)) {
> > + kfree(fsc->mount_options->mon_addr);
> > + fsc->mount_options->mon_addr = fsopt->mon_addr;
> > + fsopt->mon_addr = NULL;
> > + printk(KERN_NOTICE "ceph: monitor addresses recorded, but not used for reconnection");
>
> It's currently more in-vogue to use pr_notice() for this. I'll plan to
> make that (minor) change before I merge. No need to resend.
Got it. ACK.
>
> > + }
> > +
> > sync_filesystem(fc->root->d_sb);
> > return 0;
> > }
>
> --
> Jeff Layton <jlayton@xxxxxxxxxx>
>
--
Cheers,
Venky
