ceph: keep consistent semantic in fscache related option combination

When specifying multiple fscache related options, the result isn't always
the same as option order, this fix will keep strict consistent meaning
by order.

Signed-off-by: Chengguang Xu <cgxu519@icloud.com>
Reviewed-by: "Yan, Zheng" <zyan@redhat.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

1 files changed, 4 insertions, 0 deletions

diff --git a/fs/ceph/super.c b/fs/ceph/super.c
index fb32379cd42c..ca8a830c07ad 100644
--- a/fs/ceph/super.c
+++ b/fs/ceph/super.c
@@ -314,9 +314,13 @@ static int parse_fsopt_token(char *c, void *private)
                 break;
         case Opt_fscache:
                 fsopt->flags |= CEPH_MOUNT_OPT_FSCACHE;
+                kfree(fsopt->fscache_uniq);
+                fsopt->fscache_uniq = NULL;
                 break;
         case Opt_nofscache:
                 fsopt->flags &= ~CEPH_MOUNT_OPT_FSCACHE;
+                kfree(fsopt->fscache_uniq);
+                fsopt->fscache_uniq = NULL;
                 break;
         case Opt_poolperm:
                 fsopt->flags &= ~CEPH_MOUNT_OPT_NOPOOLPERM;