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)) {
One other problem with this patch...
The above needs to use strcmp_null(). I'll plan to make that change too.
> + 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");
> + }
> +
> sync_filesystem(fc->root->d_sb);
> return 0;
> }
--
Jeff Layton <jlayton@xxxxxxxxxx>
