forked from ~ljy/RK356X_SDK_RELEASE

hc
2023-12-09 95099d4622f8cb224d94e314c7a8e0df60b13f87
kernel/fs/overlayfs/super.c
....@@ -1,10 +1,7 @@
1
+// SPDX-License-Identifier: GPL-2.0-only
12 /*
23 *
34 * Copyright (C) 2011 Novell Inc.
4
- *
5
- * This program is free software; you can redistribute it and/or modify it
6
- * under the terms of the GNU General Public License version 2 as published by
7
- * the Free Software Foundation.
85 */
96
107 #include <uapi/linux/magic.h>
....@@ -23,6 +20,7 @@
2320 MODULE_AUTHOR("Miklos Szeredi <miklos@szeredi.hu>");
2421 MODULE_DESCRIPTION("Overlay filesystem");
2522 MODULE_LICENSE("GPL");
23
+MODULE_IMPORT_NS(ANDROID_GKI_VFS_EXPORT_ONLY);
2624
2725
2826 struct ovl_dir_cache;
....@@ -31,29 +29,29 @@
3129
3230 static bool ovl_redirect_dir_def = IS_ENABLED(CONFIG_OVERLAY_FS_REDIRECT_DIR);
3331 module_param_named(redirect_dir, ovl_redirect_dir_def, bool, 0644);
34
-MODULE_PARM_DESC(ovl_redirect_dir_def,
32
+MODULE_PARM_DESC(redirect_dir,
3533 "Default to on or off for the redirect_dir feature");
3634
3735 static bool ovl_redirect_always_follow =
3836 IS_ENABLED(CONFIG_OVERLAY_FS_REDIRECT_ALWAYS_FOLLOW);
3937 module_param_named(redirect_always_follow, ovl_redirect_always_follow,
4038 bool, 0644);
41
-MODULE_PARM_DESC(ovl_redirect_always_follow,
39
+MODULE_PARM_DESC(redirect_always_follow,
4240 "Follow redirects even if redirect_dir feature is turned off");
4341
4442 static bool ovl_index_def = IS_ENABLED(CONFIG_OVERLAY_FS_INDEX);
4543 module_param_named(index, ovl_index_def, bool, 0644);
46
-MODULE_PARM_DESC(ovl_index_def,
44
+MODULE_PARM_DESC(index,
4745 "Default to on or off for the inodes index feature");
4846
4947 static bool ovl_nfs_export_def = IS_ENABLED(CONFIG_OVERLAY_FS_NFS_EXPORT);
5048 module_param_named(nfs_export, ovl_nfs_export_def, bool, 0644);
51
-MODULE_PARM_DESC(ovl_nfs_export_def,
49
+MODULE_PARM_DESC(nfs_export,
5250 "Default to on or off for the NFS export feature");
5351
5452 static bool ovl_xino_auto_def = IS_ENABLED(CONFIG_OVERLAY_FS_XINO_AUTO);
5553 module_param_named(xino_auto, ovl_xino_auto_def, bool, 0644);
56
-MODULE_PARM_DESC(ovl_xino_auto_def,
54
+MODULE_PARM_DESC(xino_auto,
5755 "Auto enable xino feature");
5856
5957 static bool __read_mostly ovl_override_creds_def = true;
....@@ -71,7 +69,7 @@
7169
7270 static bool ovl_metacopy_def = IS_ENABLED(CONFIG_OVERLAY_FS_METACOPY);
7371 module_param_named(metacopy, ovl_metacopy_def, bool, 0644);
74
-MODULE_PARM_DESC(ovl_metacopy_def,
72
+MODULE_PARM_DESC(metacopy,
7573 "Default to on or off for the metadata only copy up feature");
7674
7775 static void ovl_dentry_release(struct dentry *dentry)
....@@ -124,53 +122,54 @@
124122 return dentry;
125123 }
126124
127
-static int ovl_dentry_revalidate(struct dentry *dentry, unsigned int flags)
125
+static int ovl_revalidate_real(struct dentry *d, unsigned int flags, bool weak)
128126 {
129
- struct ovl_entry *oe = dentry->d_fsdata;
130
- unsigned int i;
131127 int ret = 1;
132128
133
- for (i = 0; i < oe->numlower; i++) {
134
- struct dentry *d = oe->lowerstack[i].dentry;
135
-
136
- if (d->d_flags & DCACHE_OP_REVALIDATE) {
137
- ret = d->d_op->d_revalidate(d, flags);
138
- if (ret < 0)
139
- return ret;
140
- if (!ret) {
141
- if (!(flags & LOOKUP_RCU))
142
- d_invalidate(d);
143
- return -ESTALE;
144
- }
145
- }
146
- }
147
- return 1;
148
-}
149
-
150
-static int ovl_dentry_weak_revalidate(struct dentry *dentry, unsigned int flags)
151
-{
152
- struct ovl_entry *oe = dentry->d_fsdata;
153
- unsigned int i;
154
- int ret = 1;
155
-
156
- for (i = 0; i < oe->numlower; i++) {
157
- struct dentry *d = oe->lowerstack[i].dentry;
158
-
159
- if (d->d_flags & DCACHE_OP_WEAK_REVALIDATE) {
160
- ret = d->d_op->d_weak_revalidate(d, flags);
161
- if (ret <= 0)
162
- break;
129
+ if (weak) {
130
+ if (d->d_flags & DCACHE_OP_WEAK_REVALIDATE)
131
+ ret = d->d_op->d_weak_revalidate(d, flags);
132
+ } else if (d->d_flags & DCACHE_OP_REVALIDATE) {
133
+ ret = d->d_op->d_revalidate(d, flags);
134
+ if (!ret) {
135
+ if (!(flags & LOOKUP_RCU))
136
+ d_invalidate(d);
137
+ ret = -ESTALE;
163138 }
164139 }
165140 return ret;
166141 }
167142
168
-static const struct dentry_operations ovl_dentry_operations = {
169
- .d_release = ovl_dentry_release,
170
- .d_real = ovl_d_real,
171
-};
143
+static int ovl_dentry_revalidate_common(struct dentry *dentry,
144
+ unsigned int flags, bool weak)
145
+{
146
+ struct ovl_entry *oe = dentry->d_fsdata;
147
+ struct dentry *upper;
148
+ unsigned int i;
149
+ int ret = 1;
172150
173
-static const struct dentry_operations ovl_reval_dentry_operations = {
151
+ upper = ovl_dentry_upper(dentry);
152
+ if (upper)
153
+ ret = ovl_revalidate_real(upper, flags, weak);
154
+
155
+ for (i = 0; ret > 0 && i < oe->numlower; i++) {
156
+ ret = ovl_revalidate_real(oe->lowerstack[i].dentry, flags,
157
+ weak);
158
+ }
159
+ return ret;
160
+}
161
+
162
+static int ovl_dentry_revalidate(struct dentry *dentry, unsigned int flags)
163
+{
164
+ return ovl_dentry_revalidate_common(dentry, flags, false);
165
+}
166
+
167
+static int ovl_dentry_weak_revalidate(struct dentry *dentry, unsigned int flags)
168
+{
169
+ return ovl_dentry_revalidate_common(dentry, flags, true);
170
+}
171
+
172
+static const struct dentry_operations ovl_dentry_operations = {
174173 .d_release = ovl_dentry_release,
175174 .d_real = ovl_d_real,
176175 .d_revalidate = ovl_dentry_revalidate,
....@@ -198,11 +197,13 @@
198197 return &oi->vfs_inode;
199198 }
200199
201
-static void ovl_i_callback(struct rcu_head *head)
200
+static void ovl_free_inode(struct inode *inode)
202201 {
203
- struct inode *inode = container_of(head, struct inode, i_rcu);
202
+ struct ovl_inode *oi = OVL_I(inode);
204203
205
- kmem_cache_free(ovl_inode_cachep, OVL_I(inode));
204
+ kfree(oi->redirect);
205
+ mutex_destroy(&oi->lock);
206
+ kmem_cache_free(ovl_inode_cachep, oi);
206207 }
207208
208209 static void ovl_destroy_inode(struct inode *inode)
....@@ -215,36 +216,36 @@
215216 ovl_dir_cache_free(inode);
216217 else
217218 iput(oi->lowerdata);
218
- kfree(oi->redirect);
219
- mutex_destroy(&oi->lock);
220
-
221
- call_rcu(&inode->i_rcu, ovl_i_callback);
222219 }
223220
224221 static void ovl_free_fs(struct ovl_fs *ofs)
225222 {
223
+ struct vfsmount **mounts;
226224 unsigned i;
227225
228226 iput(ofs->workbasedir_trap);
229227 iput(ofs->indexdir_trap);
230228 iput(ofs->workdir_trap);
231
- iput(ofs->upperdir_trap);
229
+ dput(ofs->whiteout);
232230 dput(ofs->indexdir);
233231 dput(ofs->workdir);
234232 if (ofs->workdir_locked)
235233 ovl_inuse_unlock(ofs->workbasedir);
236234 dput(ofs->workbasedir);
237235 if (ofs->upperdir_locked)
238
- ovl_inuse_unlock(ofs->upper_mnt->mnt_root);
239
- mntput(ofs->upper_mnt);
240
- for (i = 0; i < ofs->numlower; i++) {
241
- iput(ofs->lower_layers[i].trap);
242
- mntput(ofs->lower_layers[i].mnt);
236
+ ovl_inuse_unlock(ovl_upper_mnt(ofs)->mnt_root);
237
+
238
+ /* Hack! Reuse ofs->layers as a vfsmount array before freeing it */
239
+ mounts = (struct vfsmount **) ofs->layers;
240
+ for (i = 0; i < ofs->numlayer; i++) {
241
+ iput(ofs->layers[i].trap);
242
+ mounts[i] = ofs->layers[i].mnt;
243243 }
244
- for (i = 0; i < ofs->numlowerfs; i++)
245
- free_anon_bdev(ofs->lower_fs[i].pseudo_dev);
246
- kfree(ofs->lower_layers);
247
- kfree(ofs->lower_fs);
244
+ kern_unmount_array(mounts, ofs->numlayer);
245
+ kfree(ofs->layers);
246
+ for (i = 0; i < ofs->numfs; i++)
247
+ free_anon_bdev(ofs->fs[i].pseudo_dev);
248
+ kfree(ofs->fs);
248249
249250 kfree(ofs->config.lowerdir);
250251 kfree(ofs->config.upperdir);
....@@ -269,12 +270,23 @@
269270 struct super_block *upper_sb;
270271 int ret;
271272
272
- if (!ofs->upper_mnt)
273
- return 0;
273
+ ret = ovl_sync_status(ofs);
274
+ /*
275
+ * We have to always set the err, because the return value isn't
276
+ * checked in syncfs, and instead indirectly return an error via
277
+ * the sb's writeback errseq, which VFS inspects after this call.
278
+ */
279
+ if (ret < 0) {
280
+ errseq_set(&sb->s_wb_err, -EIO);
281
+ return -EIO;
282
+ }
283
+
284
+ if (!ret)
285
+ return ret;
274286
275287 /*
276
- * If this is a sync(2) call or an emergency sync, all the super blocks
277
- * will be iterated, including upper_sb, so no need to do anything.
288
+ * Not called for sync(2) call or an emergency sync (SB_I_SKIP_SYNC).
289
+ * All the super blocks will be iterated, including upper_sb.
278290 *
279291 * If this is a syncfs(2) call, then we do need to call
280292 * sync_filesystem() on upper_sb, but enough if we do it when being
....@@ -283,7 +295,7 @@
283295 if (!wait)
284296 return 0;
285297
286
- upper_sb = ofs->upper_mnt->mnt_sb;
298
+ upper_sb = ovl_upper_mnt(ofs)->mnt_sb;
287299
288300 down_read(&upper_sb->s_umount);
289301 ret = sync_filesystem(upper_sb);
....@@ -321,19 +333,13 @@
321333 /* Will this overlay be forced to mount/remount ro? */
322334 static bool ovl_force_readonly(struct ovl_fs *ofs)
323335 {
324
- return (!ofs->upper_mnt || !ofs->workdir);
336
+ return (!ovl_upper_mnt(ofs) || !ofs->workdir);
325337 }
326338
327339 static const char *ovl_redirect_mode_def(void)
328340 {
329341 return ovl_redirect_dir_def ? "on" : "off";
330342 }
331
-
332
-enum {
333
- OVL_XINO_OFF,
334
- OVL_XINO_AUTO,
335
- OVL_XINO_ON,
336
-};
337343
338344 static const char * const ovl_xino_str[] = {
339345 "off",
....@@ -371,11 +377,13 @@
371377 if (ofs->config.nfs_export != ovl_nfs_export_def)
372378 seq_printf(m, ",nfs_export=%s", ofs->config.nfs_export ?
373379 "on" : "off");
374
- if (ofs->config.xino != ovl_xino_def())
380
+ if (ofs->config.xino != ovl_xino_def() && !ovl_same_fs(sb))
375381 seq_printf(m, ",xino=%s", ovl_xino_str[ofs->config.xino]);
376382 if (ofs->config.metacopy != ovl_metacopy_def)
377383 seq_printf(m, ",metacopy=%s",
378384 ofs->config.metacopy ? "on" : "off");
385
+ if (ofs->config.ovl_volatile)
386
+ seq_puts(m, ",volatile");
379387 if (ofs->config.override_creds != ovl_override_creds_def)
380388 seq_show_option(m, "override_creds",
381389 ofs->config.override_creds ? "on" : "off");
....@@ -385,15 +393,27 @@
385393 static int ovl_remount(struct super_block *sb, int *flags, char *data)
386394 {
387395 struct ovl_fs *ofs = sb->s_fs_info;
396
+ struct super_block *upper_sb;
397
+ int ret = 0;
388398
389399 if (!(*flags & SB_RDONLY) && ovl_force_readonly(ofs))
390400 return -EROFS;
391401
392
- return 0;
402
+ if (*flags & SB_RDONLY && !sb_rdonly(sb)) {
403
+ upper_sb = ovl_upper_mnt(ofs)->mnt_sb;
404
+ if (ovl_should_sync(ofs)) {
405
+ down_read(&upper_sb->s_umount);
406
+ ret = sync_filesystem(upper_sb);
407
+ up_read(&upper_sb->s_umount);
408
+ }
409
+ }
410
+
411
+ return ret;
393412 }
394413
395414 static const struct super_operations ovl_super_operations = {
396415 .alloc_inode = ovl_alloc_inode,
416
+ .free_inode = ovl_free_inode,
397417 .destroy_inode = ovl_destroy_inode,
398418 .drop_inode = generic_delete_inode,
399419 .put_super = ovl_put_super,
....@@ -418,6 +438,7 @@
418438 OPT_XINO_AUTO,
419439 OPT_METACOPY_ON,
420440 OPT_METACOPY_OFF,
441
+ OPT_VOLATILE,
421442 OPT_OVERRIDE_CREDS_ON,
422443 OPT_OVERRIDE_CREDS_OFF,
423444 OPT_ERR,
....@@ -438,6 +459,7 @@
438459 {OPT_XINO_AUTO, "xino=auto"},
439460 {OPT_METACOPY_ON, "metacopy=on"},
440461 {OPT_METACOPY_OFF, "metacopy=off"},
462
+ {OPT_VOLATILE, "volatile"},
441463 {OPT_OVERRIDE_CREDS_ON, "override_creds=on"},
442464 {OPT_OVERRIDE_CREDS_OFF, "override_creds=off"},
443465 {OPT_ERR, NULL}
....@@ -481,7 +503,7 @@
481503 if (ovl_redirect_always_follow)
482504 config->redirect_follow = true;
483505 } else if (strcmp(mode, "nofollow") != 0) {
484
- pr_err("overlayfs: bad mount option \"redirect_dir=%s\"\n",
506
+ pr_err("bad mount option \"redirect_dir=%s\"\n",
485507 mode);
486508 return -EINVAL;
487509 }
....@@ -494,6 +516,7 @@
494516 char *p;
495517 int err;
496518 bool metacopy_opt = false, redirect_opt = false;
519
+ bool nfs_export_opt = false, index_opt = false;
497520
498521 config->redirect_mode = kstrdup(ovl_redirect_mode_def(), GFP_KERNEL);
499522 if (!config->redirect_mode)
....@@ -544,18 +567,22 @@
544567
545568 case OPT_INDEX_ON:
546569 config->index = true;
570
+ index_opt = true;
547571 break;
548572
549573 case OPT_INDEX_OFF:
550574 config->index = false;
575
+ index_opt = true;
551576 break;
552577
553578 case OPT_NFS_EXPORT_ON:
554579 config->nfs_export = true;
580
+ nfs_export_opt = true;
555581 break;
556582
557583 case OPT_NFS_EXPORT_OFF:
558584 config->nfs_export = false;
585
+ nfs_export_opt = true;
559586 break;
560587
561588 case OPT_XINO_ON:
....@@ -577,6 +604,11 @@
577604
578605 case OPT_METACOPY_OFF:
579606 config->metacopy = false;
607
+ metacopy_opt = true;
608
+ break;
609
+
610
+ case OPT_VOLATILE:
611
+ config->ovl_volatile = true;
580612 break;
581613
582614 case OPT_OVERRIDE_CREDS_ON:
....@@ -588,17 +620,30 @@
588620 break;
589621
590622 default:
591
- pr_err("overlayfs: unrecognized mount option \"%s\" or missing value\n", p);
623
+ pr_err("unrecognized mount option \"%s\" or missing value\n",
624
+ p);
592625 return -EINVAL;
593626 }
594627 }
595628
596
- /* Workdir is useless in non-upper mount */
597
- if (!config->upperdir && config->workdir) {
598
- pr_info("overlayfs: option \"workdir=%s\" is useless in a non-upper mount, ignore\n",
599
- config->workdir);
600
- kfree(config->workdir);
601
- config->workdir = NULL;
629
+ /* Workdir/index are useless in non-upper mount */
630
+ if (!config->upperdir) {
631
+ if (config->workdir) {
632
+ pr_info("option \"workdir=%s\" is useless in a non-upper mount, ignore\n",
633
+ config->workdir);
634
+ kfree(config->workdir);
635
+ config->workdir = NULL;
636
+ }
637
+ if (config->index && index_opt) {
638
+ pr_info("option \"index=on\" is useless in a non-upper mount, ignore\n");
639
+ index_opt = false;
640
+ }
641
+ config->index = false;
642
+ }
643
+
644
+ if (!config->upperdir && config->ovl_volatile) {
645
+ pr_info("option \"volatile\" is meaningless in a non-upper mount, ignoring it.\n");
646
+ config->ovl_volatile = false;
602647 }
603648
604649 err = ovl_parse_redirect_mode(config, config->redirect_mode);
....@@ -615,7 +660,7 @@
615660 /* Resolve metacopy -> redirect_dir dependency */
616661 if (config->metacopy && !config->redirect_dir) {
617662 if (metacopy_opt && redirect_opt) {
618
- pr_err("overlayfs: conflicting options: metacopy=on,redirect_dir=%s\n",
663
+ pr_err("conflicting options: metacopy=on,redirect_dir=%s\n",
619664 config->redirect_mode);
620665 return -EINVAL;
621666 }
....@@ -624,12 +669,56 @@
624669 * There was an explicit redirect_dir=... that resulted
625670 * in this conflict.
626671 */
627
- pr_info("overlayfs: disabling metacopy due to redirect_dir=%s\n",
672
+ pr_info("disabling metacopy due to redirect_dir=%s\n",
628673 config->redirect_mode);
629674 config->metacopy = false;
630675 } else {
631676 /* Automatically enable redirect otherwise. */
632677 config->redirect_follow = config->redirect_dir = true;
678
+ }
679
+ }
680
+
681
+ /* Resolve nfs_export -> index dependency */
682
+ if (config->nfs_export && !config->index) {
683
+ if (!config->upperdir && config->redirect_follow) {
684
+ pr_info("NFS export requires \"redirect_dir=nofollow\" on non-upper mount, falling back to nfs_export=off.\n");
685
+ config->nfs_export = false;
686
+ } else if (nfs_export_opt && index_opt) {
687
+ pr_err("conflicting options: nfs_export=on,index=off\n");
688
+ return -EINVAL;
689
+ } else if (index_opt) {
690
+ /*
691
+ * There was an explicit index=off that resulted
692
+ * in this conflict.
693
+ */
694
+ pr_info("disabling nfs_export due to index=off\n");
695
+ config->nfs_export = false;
696
+ } else {
697
+ /* Automatically enable index otherwise. */
698
+ config->index = true;
699
+ }
700
+ }
701
+
702
+ /* Resolve nfs_export -> !metacopy dependency */
703
+ if (config->nfs_export && config->metacopy) {
704
+ if (nfs_export_opt && metacopy_opt) {
705
+ pr_err("conflicting options: nfs_export=on,metacopy=on\n");
706
+ return -EINVAL;
707
+ }
708
+ if (metacopy_opt) {
709
+ /*
710
+ * There was an explicit metacopy=on that resulted
711
+ * in this conflict.
712
+ */
713
+ pr_info("disabling nfs_export due to metacopy=on\n");
714
+ config->nfs_export = false;
715
+ } else {
716
+ /*
717
+ * There was an explicit nfs_export=on that resulted
718
+ * in this conflict.
719
+ */
720
+ pr_info("disabling metacopy due to nfs_export=on\n");
721
+ config->metacopy = false;
633722 }
634723 }
635724
....@@ -643,15 +732,12 @@
643732 const char *name, bool persist)
644733 {
645734 struct inode *dir = ofs->workbasedir->d_inode;
646
- struct vfsmount *mnt = ofs->upper_mnt;
735
+ struct vfsmount *mnt = ovl_upper_mnt(ofs);
647736 struct dentry *work;
648737 int err;
649738 bool retried = false;
650
- bool locked = false;
651739
652740 inode_lock_nested(dir, I_MUTEX_PARENT);
653
- locked = true;
654
-
655741 retry:
656742 work = lookup_one_len(name, ofs->workbasedir, strlen(name));
657743
....@@ -670,8 +756,12 @@
670756 goto out_unlock;
671757
672758 retried = true;
673
- ovl_workdir_cleanup(dir, mnt, work, 0);
759
+ err = ovl_workdir_cleanup(dir, mnt, work, 0);
674760 dput(work);
761
+ if (err == -EINVAL) {
762
+ work = ERR_PTR(err);
763
+ goto out_unlock;
764
+ }
675765 goto retry;
676766 }
677767
....@@ -716,15 +806,13 @@
716806 goto out_err;
717807 }
718808 out_unlock:
719
- if (locked)
720
- inode_unlock(dir);
721
-
809
+ inode_unlock(dir);
722810 return work;
723811
724812 out_dput:
725813 dput(work);
726814 out_err:
727
- pr_warn("overlayfs: failed to create directory %s/%s (errno: %i); mounting read-only\n",
815
+ pr_warn("failed to create directory %s/%s (errno: %i); mounting read-only\n",
728816 ofs->config.workdir, name, -err);
729817 work = NULL;
730818 goto out_unlock;
....@@ -748,21 +836,21 @@
748836 int err = -EINVAL;
749837
750838 if (!*name) {
751
- pr_err("overlayfs: empty lowerdir\n");
839
+ pr_err("empty lowerdir\n");
752840 goto out;
753841 }
754842 err = kern_path(name, LOOKUP_FOLLOW, path);
755843 if (err) {
756
- pr_err("overlayfs: failed to resolve '%s': %i\n", name, err);
844
+ pr_err("failed to resolve '%s': %i\n", name, err);
757845 goto out;
758846 }
759847 err = -EINVAL;
760848 if (ovl_dentry_weird(path->dentry)) {
761
- pr_err("overlayfs: filesystem on '%s' not supported\n", name);
849
+ pr_err("filesystem on '%s' not supported\n", name);
762850 goto out_put;
763851 }
764852 if (!d_is_dir(path->dentry)) {
765
- pr_err("overlayfs: '%s' not a directory\n", name);
853
+ pr_err("'%s' not a directory\n", name);
766854 goto out_put;
767855 }
768856 return 0;
....@@ -782,13 +870,12 @@
782870 ovl_unescape(tmp);
783871 err = ovl_mount_dir_noesc(tmp, path);
784872
785
- if (!err)
786
- if (ovl_dentry_remote(path->dentry)) {
787
- pr_err("overlayfs: filesystem on '%s' not supported as upperdir\n",
788
- tmp);
789
- path_put_init(path);
790
- err = -EINVAL;
791
- }
873
+ if (!err && path->dentry->d_flags & DCACHE_OP_REAL) {
874
+ pr_err("filesystem on '%s' not supported as upperdir\n",
875
+ tmp);
876
+ path_put_init(path);
877
+ err = -EINVAL;
878
+ }
792879 kfree(tmp);
793880 }
794881 return err;
....@@ -801,7 +888,7 @@
801888 int err = vfs_statfs(path, &statfs);
802889
803890 if (err)
804
- pr_err("overlayfs: statfs failed on '%s'\n", name);
891
+ pr_err("statfs failed on '%s'\n", name);
805892 else
806893 ofs->namelen = max(ofs->namelen, statfs.f_namelen);
807894
....@@ -809,23 +896,20 @@
809896 }
810897
811898 static int ovl_lower_dir(const char *name, struct path *path,
812
- struct ovl_fs *ofs, int *stack_depth, bool *remote)
899
+ struct ovl_fs *ofs, int *stack_depth)
813900 {
814901 int fh_type;
815902 int err;
816903
817904 err = ovl_mount_dir_noesc(name, path);
818905 if (err)
819
- goto out;
906
+ return err;
820907
821908 err = ovl_check_namelen(path, ofs, name);
822909 if (err)
823
- goto out_put;
910
+ return err;
824911
825912 *stack_depth = max(*stack_depth, path->mnt->mnt_sb->s_stack_depth);
826
-
827
- if (ovl_dentry_remote(path->dentry))
828
- *remote = true;
829913
830914 /*
831915 * The inodes index feature and NFS export need to encode and decode
....@@ -836,20 +920,15 @@
836920 (ofs->config.index && ofs->config.upperdir)) && !fh_type) {
837921 ofs->config.index = false;
838922 ofs->config.nfs_export = false;
839
- pr_warn("overlayfs: fs on '%s' does not support file handles, falling back to index=off,nfs_export=off.\n",
923
+ pr_warn("fs on '%s' does not support file handles, falling back to index=off,nfs_export=off.\n",
840924 name);
841925 }
842926
843927 /* Check if lower fs has 32bit inode numbers */
844928 if (fh_type != FILEID_INO32_GEN)
845
- ofs->xino_bits = 0;
929
+ ofs->xino_mode = -1;
846930
847931 return 0;
848
-
849
-out_put:
850
- path_put_init(path);
851
-out:
852
- return err;
853932 }
854933
855934 /* Workdir should not be subdir of upperdir and vice versa */
....@@ -887,17 +966,9 @@
887966 static int __maybe_unused
888967 ovl_posix_acl_xattr_get(const struct xattr_handler *handler,
889968 struct dentry *dentry, struct inode *inode,
890
- const char *name, void *buffer, size_t size)
969
+ const char *name, void *buffer, size_t size, int flags)
891970 {
892
- return ovl_xattr_get(dentry, inode, handler->name, buffer, size);
893
-}
894
-
895
-static int __maybe_unused
896
-__ovl_posix_acl_xattr_get(const struct xattr_handler *handler,
897
- struct dentry *dentry, struct inode *inode,
898
- const char *name, void *buffer, size_t size)
899
-{
900
- return __ovl_xattr_get(dentry, inode, handler->name, buffer, size);
971
+ return ovl_xattr_get(dentry, inode, handler->name, buffer, size, flags);
901972 }
902973
903974 static int __maybe_unused
....@@ -960,7 +1031,8 @@
9601031
9611032 static int ovl_own_xattr_get(const struct xattr_handler *handler,
9621033 struct dentry *dentry, struct inode *inode,
963
- const char *name, void *buffer, size_t size)
1034
+ const char *name, void *buffer, size_t size,
1035
+ int flags)
9641036 {
9651037 return -EOPNOTSUPP;
9661038 }
....@@ -975,16 +1047,10 @@
9751047
9761048 static int ovl_other_xattr_get(const struct xattr_handler *handler,
9771049 struct dentry *dentry, struct inode *inode,
978
- const char *name, void *buffer, size_t size)
1050
+ const char *name, void *buffer, size_t size,
1051
+ int flags)
9791052 {
980
- return ovl_xattr_get(dentry, inode, name, buffer, size);
981
-}
982
-
983
-static int __ovl_other_xattr_get(const struct xattr_handler *handler,
984
- struct dentry *dentry, struct inode *inode,
985
- const char *name, void *buffer, size_t size)
986
-{
987
- return __ovl_xattr_get(dentry, inode, name, buffer, size);
1053
+ return ovl_xattr_get(dentry, inode, name, buffer, size, flags);
9881054 }
9891055
9901056 static int ovl_other_xattr_set(const struct xattr_handler *handler,
....@@ -1000,7 +1066,6 @@
10001066 .name = XATTR_NAME_POSIX_ACL_ACCESS,
10011067 .flags = ACL_TYPE_ACCESS,
10021068 .get = ovl_posix_acl_xattr_get,
1003
- .__get = __ovl_posix_acl_xattr_get,
10041069 .set = ovl_posix_acl_xattr_set,
10051070 };
10061071
....@@ -1009,7 +1074,6 @@
10091074 .name = XATTR_NAME_POSIX_ACL_DEFAULT,
10101075 .flags = ACL_TYPE_DEFAULT,
10111076 .get = ovl_posix_acl_xattr_get,
1012
- .__get = __ovl_posix_acl_xattr_get,
10131077 .set = ovl_posix_acl_xattr_set,
10141078 };
10151079
....@@ -1022,7 +1086,6 @@
10221086 static const struct xattr_handler ovl_other_xattr_handler = {
10231087 .prefix = "", /* catch all */
10241088 .get = ovl_other_xattr_get,
1025
- .__get = __ovl_other_xattr_get,
10261089 .set = ovl_other_xattr_set,
10271090 };
10281091
....@@ -1046,7 +1109,7 @@
10461109 err = PTR_ERR_OR_ZERO(trap);
10471110 if (err) {
10481111 if (err == -ELOOP)
1049
- pr_err("overlayfs: conflicting %s path\n", name);
1112
+ pr_err("conflicting %s path\n", name);
10501113 return err;
10511114 }
10521115
....@@ -1063,18 +1126,18 @@
10631126 static int ovl_report_in_use(struct ovl_fs *ofs, const char *name)
10641127 {
10651128 if (ofs->config.index) {
1066
- pr_err("overlayfs: %s is in-use as upperdir/workdir of another mount, mount with '-o index=off' to override exclusive upperdir protection.\n",
1129
+ pr_err("%s is in-use as upperdir/workdir of another mount, mount with '-o index=off' to override exclusive upperdir protection.\n",
10671130 name);
10681131 return -EBUSY;
10691132 } else {
1070
- pr_warn("overlayfs: %s is in-use as upperdir/workdir of another mount, accessing files from both mounts will result in undefined behavior.\n",
1133
+ pr_warn("%s is in-use as upperdir/workdir of another mount, accessing files from both mounts will result in undefined behavior.\n",
10711134 name);
10721135 return 0;
10731136 }
10741137 }
10751138
10761139 static int ovl_get_upper(struct super_block *sb, struct ovl_fs *ofs,
1077
- struct path *upperpath)
1140
+ struct ovl_layer *upper_layer, struct path *upperpath)
10781141 {
10791142 struct vfsmount *upper_mnt;
10801143 int err;
....@@ -1085,7 +1148,7 @@
10851148
10861149 /* Upper fs should not be r/o */
10871150 if (sb_rdonly(upperpath->mnt->mnt_sb)) {
1088
- pr_err("overlayfs: upper fs is r/o, try multi-lower layers mount\n");
1151
+ pr_err("upper fs is r/o, try multi-lower layers mount\n");
10891152 err = -EINVAL;
10901153 goto out;
10911154 }
....@@ -1094,7 +1157,7 @@
10941157 if (err)
10951158 goto out;
10961159
1097
- err = ovl_setup_trap(sb, upperpath->dentry, &ofs->upperdir_trap,
1160
+ err = ovl_setup_trap(sb, upperpath->dentry, &upper_layer->trap,
10981161 "upperdir");
10991162 if (err)
11001163 goto out;
....@@ -1102,15 +1165,29 @@
11021165 upper_mnt = clone_private_mount(upperpath);
11031166 err = PTR_ERR(upper_mnt);
11041167 if (IS_ERR(upper_mnt)) {
1105
- pr_err("overlayfs: failed to clone upperpath\n");
1168
+ pr_err("failed to clone upperpath\n");
11061169 goto out;
11071170 }
11081171
11091172 /* Don't inherit atime flags */
11101173 upper_mnt->mnt_flags &= ~(MNT_NOATIME | MNT_NODIRATIME | MNT_RELATIME);
1111
- ofs->upper_mnt = upper_mnt;
1174
+ upper_layer->mnt = upper_mnt;
1175
+ upper_layer->idx = 0;
1176
+ upper_layer->fsid = 0;
11121177
1113
- if (ovl_inuse_trylock(ofs->upper_mnt->mnt_root)) {
1178
+ /*
1179
+ * Inherit SB_NOSEC flag from upperdir.
1180
+ *
1181
+ * This optimization changes behavior when a security related attribute
1182
+ * (suid/sgid/security.*) is changed on an underlying layer. This is
1183
+ * okay because we don't yet have guarantees in that case, but it will
1184
+ * need careful treatment once we want to honour changes to underlying
1185
+ * filesystems.
1186
+ */
1187
+ if (upper_mnt->mnt_sb->s_flags & SB_NOSEC)
1188
+ sb->s_flags |= SB_NOSEC;
1189
+
1190
+ if (ovl_inuse_trylock(ovl_upper_mnt(ofs)->mnt_root)) {
11141191 ofs->upperdir_locked = true;
11151192 } else {
11161193 err = ovl_report_in_use(ofs, "upperdir");
....@@ -1123,11 +1200,112 @@
11231200 return err;
11241201 }
11251202
1203
+/*
1204
+ * Returns 1 if RENAME_WHITEOUT is supported, 0 if not supported and
1205
+ * negative values if error is encountered.
1206
+ */
1207
+static int ovl_check_rename_whiteout(struct dentry *workdir)
1208
+{
1209
+ struct inode *dir = d_inode(workdir);
1210
+ struct dentry *temp;
1211
+ struct dentry *dest;
1212
+ struct dentry *whiteout;
1213
+ struct name_snapshot name;
1214
+ int err;
1215
+
1216
+ inode_lock_nested(dir, I_MUTEX_PARENT);
1217
+
1218
+ temp = ovl_create_temp(workdir, OVL_CATTR(S_IFREG | 0));
1219
+ err = PTR_ERR(temp);
1220
+ if (IS_ERR(temp))
1221
+ goto out_unlock;
1222
+
1223
+ dest = ovl_lookup_temp(workdir);
1224
+ err = PTR_ERR(dest);
1225
+ if (IS_ERR(dest)) {
1226
+ dput(temp);
1227
+ goto out_unlock;
1228
+ }
1229
+
1230
+ /* Name is inline and stable - using snapshot as a copy helper */
1231
+ take_dentry_name_snapshot(&name, temp);
1232
+ err = ovl_do_rename(dir, temp, dir, dest, RENAME_WHITEOUT);
1233
+ if (err) {
1234
+ if (err == -EINVAL)
1235
+ err = 0;
1236
+ goto cleanup_temp;
1237
+ }
1238
+
1239
+ whiteout = lookup_one_len(name.name.name, workdir, name.name.len);
1240
+ err = PTR_ERR(whiteout);
1241
+ if (IS_ERR(whiteout))
1242
+ goto cleanup_temp;
1243
+
1244
+ err = ovl_is_whiteout(whiteout);
1245
+
1246
+ /* Best effort cleanup of whiteout and temp file */
1247
+ if (err)
1248
+ ovl_cleanup(dir, whiteout);
1249
+ dput(whiteout);
1250
+
1251
+cleanup_temp:
1252
+ ovl_cleanup(dir, temp);
1253
+ release_dentry_name_snapshot(&name);
1254
+ dput(temp);
1255
+ dput(dest);
1256
+
1257
+out_unlock:
1258
+ inode_unlock(dir);
1259
+
1260
+ return err;
1261
+}
1262
+
1263
+static struct dentry *ovl_lookup_or_create(struct dentry *parent,
1264
+ const char *name, umode_t mode)
1265
+{
1266
+ size_t len = strlen(name);
1267
+ struct dentry *child;
1268
+
1269
+ inode_lock_nested(parent->d_inode, I_MUTEX_PARENT);
1270
+ child = lookup_one_len(name, parent, len);
1271
+ if (!IS_ERR(child) && !child->d_inode)
1272
+ child = ovl_create_real(parent->d_inode, child,
1273
+ OVL_CATTR(mode));
1274
+ inode_unlock(parent->d_inode);
1275
+ dput(parent);
1276
+
1277
+ return child;
1278
+}
1279
+
1280
+/*
1281
+ * Creates $workdir/work/incompat/volatile/dirty file if it is not already
1282
+ * present.
1283
+ */
1284
+static int ovl_create_volatile_dirty(struct ovl_fs *ofs)
1285
+{
1286
+ unsigned int ctr;
1287
+ struct dentry *d = dget(ofs->workbasedir);
1288
+ static const char *const volatile_path[] = {
1289
+ OVL_WORKDIR_NAME, "incompat", "volatile", "dirty"
1290
+ };
1291
+ const char *const *name = volatile_path;
1292
+
1293
+ for (ctr = ARRAY_SIZE(volatile_path); ctr; ctr--, name++) {
1294
+ d = ovl_lookup_or_create(d, *name, ctr > 1 ? S_IFDIR : S_IFREG);
1295
+ if (IS_ERR(d))
1296
+ return PTR_ERR(d);
1297
+ }
1298
+ dput(d);
1299
+ return 0;
1300
+}
1301
+
11261302 static int ovl_make_workdir(struct super_block *sb, struct ovl_fs *ofs,
11271303 struct path *workpath)
11281304 {
1129
- struct vfsmount *mnt = ofs->upper_mnt;
1130
- struct dentry *temp;
1305
+ struct vfsmount *mnt = ovl_upper_mnt(ofs);
1306
+ struct dentry *temp, *workdir;
1307
+ bool rename_whiteout;
1308
+ bool d_type;
11311309 int fh_type;
11321310 int err;
11331311
....@@ -1135,9 +1313,12 @@
11351313 if (err)
11361314 return err;
11371315
1138
- ofs->workdir = ovl_workdir_create(ofs, OVL_WORKDIR_NAME, false);
1139
- if (!ofs->workdir)
1316
+ workdir = ovl_workdir_create(ofs, OVL_WORKDIR_NAME, false);
1317
+ err = PTR_ERR(workdir);
1318
+ if (IS_ERR_OR_NULL(workdir))
11401319 goto out;
1320
+
1321
+ ofs->workdir = workdir;
11411322
11421323 err = ovl_setup_trap(sb, ofs->workdir, &ofs->workdir_trap, "workdir");
11431324 if (err)
....@@ -1153,12 +1334,9 @@
11531334 if (err < 0)
11541335 goto out;
11551336
1156
- /*
1157
- * We allowed this configuration and don't want to break users over
1158
- * kernel upgrade. So warn instead of erroring out.
1159
- */
1160
- if (!err)
1161
- pr_warn("overlayfs: upper fs needs to support d_type.\n");
1337
+ d_type = err;
1338
+ if (!d_type)
1339
+ pr_warn("upper fs needs to support d_type.\n");
11621340
11631341 /* Check if upper/work fs supports O_TMPFILE */
11641342 temp = ovl_do_tmpfile(ofs->workdir, S_IFREG | 0);
....@@ -1166,36 +1344,70 @@
11661344 if (ofs->tmpfile)
11671345 dput(temp);
11681346 else
1169
- pr_warn("overlayfs: upper fs does not support tmpfile.\n");
1347
+ pr_warn("upper fs does not support tmpfile.\n");
1348
+
1349
+
1350
+ /* Check if upper/work fs supports RENAME_WHITEOUT */
1351
+ err = ovl_check_rename_whiteout(ofs->workdir);
1352
+ if (err < 0)
1353
+ goto out;
1354
+
1355
+ rename_whiteout = err;
1356
+ if (!rename_whiteout)
1357
+ pr_warn("upper fs does not support RENAME_WHITEOUT.\n");
11701358
11711359 /*
11721360 * Check if upper/work fs supports trusted.overlay.* xattr
11731361 */
1174
- err = ovl_do_setxattr(ofs->workdir, OVL_XATTR_OPAQUE, "0", 1, 0);
1362
+ err = ovl_do_setxattr(ofs, ofs->workdir, OVL_XATTR_OPAQUE, "0", 1);
11751363 if (err) {
11761364 ofs->noxattr = true;
11771365 ofs->config.index = false;
11781366 ofs->config.metacopy = false;
1179
- pr_warn("overlayfs: upper fs does not support xattr, falling back to index=off and metacopy=off.\n");
1367
+ pr_warn("upper fs does not support xattr, falling back to index=off and metacopy=off.\n");
11801368 err = 0;
11811369 } else {
1182
- vfs_removexattr(ofs->workdir, OVL_XATTR_OPAQUE);
1370
+ ovl_do_removexattr(ofs, ofs->workdir, OVL_XATTR_OPAQUE);
1371
+ }
1372
+
1373
+ /*
1374
+ * We allowed sub-optimal upper fs configuration and don't want to break
1375
+ * users over kernel upgrade, but we never allowed remote upper fs, so
1376
+ * we can enforce strict requirements for remote upper fs.
1377
+ */
1378
+ if (ovl_dentry_remote(ofs->workdir) &&
1379
+ (!d_type || !rename_whiteout || ofs->noxattr)) {
1380
+ pr_err("upper fs missing required features.\n");
1381
+ err = -EINVAL;
1382
+ goto out;
1383
+ }
1384
+
1385
+ /*
1386
+ * For volatile mount, create a incompat/volatile/dirty file to keep
1387
+ * track of it.
1388
+ */
1389
+ if (ofs->config.ovl_volatile) {
1390
+ err = ovl_create_volatile_dirty(ofs);
1391
+ if (err < 0) {
1392
+ pr_err("Failed to create volatile/dirty file.\n");
1393
+ goto out;
1394
+ }
11831395 }
11841396
11851397 /* Check if upper/work fs supports file handles */
11861398 fh_type = ovl_can_decode_fh(ofs->workdir->d_sb);
11871399 if (ofs->config.index && !fh_type) {
11881400 ofs->config.index = false;
1189
- pr_warn("overlayfs: upper fs does not support file handles, falling back to index=off.\n");
1401
+ pr_warn("upper fs does not support file handles, falling back to index=off.\n");
11901402 }
11911403
11921404 /* Check if upper fs has 32bit inode numbers */
11931405 if (fh_type != FILEID_INO32_GEN)
1194
- ofs->xino_bits = 0;
1406
+ ofs->xino_mode = -1;
11951407
11961408 /* NFS export of r/w mount depends on index */
11971409 if (ofs->config.nfs_export && !ofs->config.index) {
1198
- pr_warn("overlayfs: NFS export requires \"index=on\", falling back to nfs_export=off.\n");
1410
+ pr_warn("NFS export requires \"index=on\", falling back to nfs_export=off.\n");
11991411 ofs->config.nfs_export = false;
12001412 }
12011413 out:
....@@ -1215,11 +1427,11 @@
12151427
12161428 err = -EINVAL;
12171429 if (upperpath->mnt != workpath.mnt) {
1218
- pr_err("overlayfs: workdir and upperdir must reside under the same mount\n");
1430
+ pr_err("workdir and upperdir must reside under the same mount\n");
12191431 goto out;
12201432 }
12211433 if (!ovl_workdir_ok(workpath.dentry, upperpath->dentry)) {
1222
- pr_err("overlayfs: workdir and upperdir must be separate subtrees\n");
1434
+ pr_err("workdir and upperdir must be separate subtrees\n");
12231435 goto out;
12241436 }
12251437
....@@ -1249,7 +1461,8 @@
12491461 static int ovl_get_indexdir(struct super_block *sb, struct ovl_fs *ofs,
12501462 struct ovl_entry *oe, struct path *upperpath)
12511463 {
1252
- struct vfsmount *mnt = ofs->upper_mnt;
1464
+ struct vfsmount *mnt = ovl_upper_mnt(ofs);
1465
+ struct dentry *indexdir;
12531466 int err;
12541467
12551468 err = mnt_want_write(mnt);
....@@ -1257,15 +1470,25 @@
12571470 return err;
12581471
12591472 /* Verify lower root is upper root origin */
1260
- err = ovl_verify_origin(upperpath->dentry, oe->lowerstack[0].dentry,
1261
- true);
1473
+ err = ovl_verify_origin(ofs, upperpath->dentry,
1474
+ oe->lowerstack[0].dentry, true);
12621475 if (err) {
1263
- pr_err("overlayfs: failed to verify upper root origin\n");
1476
+ pr_err("failed to verify upper root origin\n");
12641477 goto out;
12651478 }
12661479
1267
- ofs->indexdir = ovl_workdir_create(ofs, OVL_INDEXDIR_NAME, true);
1268
- if (ofs->indexdir) {
1480
+ /* index dir will act also as workdir */
1481
+ iput(ofs->workdir_trap);
1482
+ ofs->workdir_trap = NULL;
1483
+ dput(ofs->workdir);
1484
+ ofs->workdir = NULL;
1485
+ indexdir = ovl_workdir_create(ofs, OVL_INDEXDIR_NAME, true);
1486
+ if (IS_ERR(indexdir)) {
1487
+ err = PTR_ERR(indexdir);
1488
+ } else if (indexdir) {
1489
+ ofs->indexdir = indexdir;
1490
+ ofs->workdir = dget(indexdir);
1491
+
12691492 err = ovl_setup_trap(sb, ofs->indexdir, &ofs->indexdir_trap,
12701493 "indexdir");
12711494 if (err)
....@@ -1279,80 +1502,143 @@
12791502 * "trusted.overlay.upper" to indicate that index may have
12801503 * directory entries.
12811504 */
1282
- if (ovl_check_origin_xattr(ofs->indexdir)) {
1283
- err = ovl_verify_set_fh(ofs->indexdir, OVL_XATTR_ORIGIN,
1505
+ if (ovl_check_origin_xattr(ofs, ofs->indexdir)) {
1506
+ err = ovl_verify_set_fh(ofs, ofs->indexdir,
1507
+ OVL_XATTR_ORIGIN,
12841508 upperpath->dentry, true, false);
12851509 if (err)
1286
- pr_err("overlayfs: failed to verify index dir 'origin' xattr\n");
1510
+ pr_err("failed to verify index dir 'origin' xattr\n");
12871511 }
1288
- err = ovl_verify_upper(ofs->indexdir, upperpath->dentry, true);
1512
+ err = ovl_verify_upper(ofs, ofs->indexdir, upperpath->dentry,
1513
+ true);
12891514 if (err)
1290
- pr_err("overlayfs: failed to verify index dir 'upper' xattr\n");
1515
+ pr_err("failed to verify index dir 'upper' xattr\n");
12911516
12921517 /* Cleanup bad/stale/orphan index entries */
12931518 if (!err)
12941519 err = ovl_indexdir_cleanup(ofs);
12951520 }
12961521 if (err || !ofs->indexdir)
1297
- pr_warn("overlayfs: try deleting index dir or mounting with '-o index=off' to disable inodes index.\n");
1522
+ pr_warn("try deleting index dir or mounting with '-o index=off' to disable inodes index.\n");
12981523
12991524 out:
13001525 mnt_drop_write(mnt);
13011526 return err;
13021527 }
13031528
1304
-/* Get a unique fsid for the layer */
1305
-static int ovl_get_fsid(struct ovl_fs *ofs, struct super_block *sb)
1529
+static bool ovl_lower_uuid_ok(struct ovl_fs *ofs, const uuid_t *uuid)
13061530 {
1531
+ unsigned int i;
1532
+
1533
+ if (!ofs->config.nfs_export && !ovl_upper_mnt(ofs))
1534
+ return true;
1535
+
1536
+ /*
1537
+ * We allow using single lower with null uuid for index and nfs_export
1538
+ * for example to support those features with single lower squashfs.
1539
+ * To avoid regressions in setups of overlay with re-formatted lower
1540
+ * squashfs, do not allow decoding origin with lower null uuid unless
1541
+ * user opted-in to one of the new features that require following the
1542
+ * lower inode of non-dir upper.
1543
+ */
1544
+ if (!ofs->config.index && !ofs->config.metacopy && !ofs->config.xino &&
1545
+ uuid_is_null(uuid))
1546
+ return false;
1547
+
1548
+ for (i = 0; i < ofs->numfs; i++) {
1549
+ /*
1550
+ * We use uuid to associate an overlay lower file handle with a
1551
+ * lower layer, so we can accept lower fs with null uuid as long
1552
+ * as all lower layers with null uuid are on the same fs.
1553
+ * if we detect multiple lower fs with the same uuid, we
1554
+ * disable lower file handle decoding on all of them.
1555
+ */
1556
+ if (ofs->fs[i].is_lower &&
1557
+ uuid_equal(&ofs->fs[i].sb->s_uuid, uuid)) {
1558
+ ofs->fs[i].bad_uuid = true;
1559
+ return false;
1560
+ }
1561
+ }
1562
+ return true;
1563
+}
1564
+
1565
+/* Get a unique fsid for the layer */
1566
+static int ovl_get_fsid(struct ovl_fs *ofs, const struct path *path)
1567
+{
1568
+ struct super_block *sb = path->mnt->mnt_sb;
13071569 unsigned int i;
13081570 dev_t dev;
13091571 int err;
1572
+ bool bad_uuid = false;
13101573
1311
- /* fsid 0 is reserved for upper fs even with non upper overlay */
1312
- if (ofs->upper_mnt && ofs->upper_mnt->mnt_sb == sb)
1313
- return 0;
1574
+ for (i = 0; i < ofs->numfs; i++) {
1575
+ if (ofs->fs[i].sb == sb)
1576
+ return i;
1577
+ }
13141578
1315
- for (i = 0; i < ofs->numlowerfs; i++) {
1316
- if (ofs->lower_fs[i].sb == sb)
1317
- return i + 1;
1579
+ if (!ovl_lower_uuid_ok(ofs, &sb->s_uuid)) {
1580
+ bad_uuid = true;
1581
+ if (ofs->config.index || ofs->config.nfs_export) {
1582
+ ofs->config.index = false;
1583
+ ofs->config.nfs_export = false;
1584
+ pr_warn("%s uuid detected in lower fs '%pd2', falling back to index=off,nfs_export=off.\n",
1585
+ uuid_is_null(&sb->s_uuid) ? "null" :
1586
+ "conflicting",
1587
+ path->dentry);
1588
+ }
13181589 }
13191590
13201591 err = get_anon_bdev(&dev);
13211592 if (err) {
1322
- pr_err("overlayfs: failed to get anonymous bdev for lowerpath\n");
1593
+ pr_err("failed to get anonymous bdev for lowerpath\n");
13231594 return err;
13241595 }
13251596
1326
- ofs->lower_fs[ofs->numlowerfs].sb = sb;
1327
- ofs->lower_fs[ofs->numlowerfs].pseudo_dev = dev;
1328
- ofs->numlowerfs++;
1597
+ ofs->fs[ofs->numfs].sb = sb;
1598
+ ofs->fs[ofs->numfs].pseudo_dev = dev;
1599
+ ofs->fs[ofs->numfs].bad_uuid = bad_uuid;
13291600
1330
- return ofs->numlowerfs;
1601
+ return ofs->numfs++;
13311602 }
13321603
1333
-static int ovl_get_lower_layers(struct super_block *sb, struct ovl_fs *ofs,
1334
- struct path *stack, unsigned int numlower)
1604
+static int ovl_get_layers(struct super_block *sb, struct ovl_fs *ofs,
1605
+ struct path *stack, unsigned int numlower,
1606
+ struct ovl_layer *layers)
13351607 {
13361608 int err;
13371609 unsigned int i;
13381610
13391611 err = -ENOMEM;
1340
- ofs->lower_layers = kcalloc(numlower, sizeof(struct ovl_layer),
1341
- GFP_KERNEL);
1342
- if (ofs->lower_layers == NULL)
1612
+ ofs->fs = kcalloc(numlower + 1, sizeof(struct ovl_sb), GFP_KERNEL);
1613
+ if (ofs->fs == NULL)
13431614 goto out;
13441615
1345
- ofs->lower_fs = kcalloc(numlower, sizeof(struct ovl_sb),
1346
- GFP_KERNEL);
1347
- if (ofs->lower_fs == NULL)
1616
+ /* idx/fsid 0 are reserved for upper fs even with lower only overlay */
1617
+ ofs->numfs++;
1618
+
1619
+ /*
1620
+ * All lower layers that share the same fs as upper layer, use the same
1621
+ * pseudo_dev as upper layer. Allocate fs[0].pseudo_dev even for lower
1622
+ * only overlay to simplify ovl_fs_free().
1623
+ * is_lower will be set if upper fs is shared with a lower layer.
1624
+ */
1625
+ err = get_anon_bdev(&ofs->fs[0].pseudo_dev);
1626
+ if (err) {
1627
+ pr_err("failed to get anonymous bdev for upper fs\n");
13481628 goto out;
1629
+ }
1630
+
1631
+ if (ovl_upper_mnt(ofs)) {
1632
+ ofs->fs[0].sb = ovl_upper_mnt(ofs)->mnt_sb;
1633
+ ofs->fs[0].is_lower = false;
1634
+ }
13491635
13501636 for (i = 0; i < numlower; i++) {
13511637 struct vfsmount *mnt;
13521638 struct inode *trap;
13531639 int fsid;
13541640
1355
- err = fsid = ovl_get_fsid(ofs, stack[i].mnt->mnt_sb);
1641
+ err = fsid = ovl_get_fsid(ofs, &stack[i]);
13561642 if (err < 0)
13571643 goto out;
13581644
....@@ -1378,7 +1664,7 @@
13781664 mnt = clone_private_mount(&stack[i]);
13791665 err = PTR_ERR(mnt);
13801666 if (IS_ERR(mnt)) {
1381
- pr_err("overlayfs: failed to clone lowerpath\n");
1667
+ pr_err("failed to clone lowerpath\n");
13821668 iput(trap);
13831669 goto out;
13841670 }
....@@ -1389,41 +1675,45 @@
13891675 */
13901676 mnt->mnt_flags |= MNT_READONLY | MNT_NOATIME;
13911677
1392
- ofs->lower_layers[ofs->numlower].trap = trap;
1393
- ofs->lower_layers[ofs->numlower].mnt = mnt;
1394
- ofs->lower_layers[ofs->numlower].idx = i + 1;
1395
- ofs->lower_layers[ofs->numlower].fsid = fsid;
1396
- if (fsid) {
1397
- ofs->lower_layers[ofs->numlower].fs =
1398
- &ofs->lower_fs[fsid - 1];
1399
- }
1400
- ofs->numlower++;
1678
+ layers[ofs->numlayer].trap = trap;
1679
+ layers[ofs->numlayer].mnt = mnt;
1680
+ layers[ofs->numlayer].idx = ofs->numlayer;
1681
+ layers[ofs->numlayer].fsid = fsid;
1682
+ layers[ofs->numlayer].fs = &ofs->fs[fsid];
1683
+ ofs->numlayer++;
1684
+ ofs->fs[fsid].is_lower = true;
14011685 }
14021686
14031687 /*
14041688 * When all layers on same fs, overlay can use real inode numbers.
1405
- * With mount option "xino=on", mounter declares that there are enough
1406
- * free high bits in underlying fs to hold the unique fsid.
1689
+ * With mount option "xino=<on|auto>", mounter declares that there are
1690
+ * enough free high bits in underlying fs to hold the unique fsid.
14071691 * If overlayfs does encounter underlying inodes using the high xino
14081692 * bits reserved for fsid, it emits a warning and uses the original
1409
- * inode number.
1693
+ * inode number or a non persistent inode number allocated from a
1694
+ * dedicated range.
14101695 */
1411
- if (!ofs->numlowerfs || (ofs->numlowerfs == 1 && !ofs->upper_mnt)) {
1412
- ofs->xino_bits = 0;
1413
- ofs->config.xino = OVL_XINO_OFF;
1414
- } else if (ofs->config.xino == OVL_XINO_ON && !ofs->xino_bits) {
1696
+ if (ofs->numfs - !ovl_upper_mnt(ofs) == 1) {
1697
+ if (ofs->config.xino == OVL_XINO_ON)
1698
+ pr_info("\"xino=on\" is useless with all layers on same fs, ignore.\n");
1699
+ ofs->xino_mode = 0;
1700
+ } else if (ofs->config.xino == OVL_XINO_OFF) {
1701
+ ofs->xino_mode = -1;
1702
+ } else if (ofs->xino_mode < 0) {
14151703 /*
1416
- * This is a roundup of number of bits needed for numlowerfs+1
1417
- * (i.e. ilog2(numlowerfs+1 - 1) + 1). fsid 0 is reserved for
1418
- * upper fs even with non upper overlay.
1704
+ * This is a roundup of number of bits needed for encoding
1705
+ * fsid, where fsid 0 is reserved for upper fs (even with
1706
+ * lower only overlay) +1 extra bit is reserved for the non
1707
+ * persistent inode number range that is used for resolving
1708
+ * xino lower bits overflow.
14191709 */
1420
- BUILD_BUG_ON(ilog2(OVL_MAX_STACK) > 31);
1421
- ofs->xino_bits = ilog2(ofs->numlowerfs) + 1;
1710
+ BUILD_BUG_ON(ilog2(OVL_MAX_STACK) > 30);
1711
+ ofs->xino_mode = ilog2(ofs->numfs - 1) + 2;
14221712 }
14231713
1424
- if (ofs->xino_bits) {
1425
- pr_info("overlayfs: \"xino\" feature enabled using %d upper inode bits.\n",
1426
- ofs->xino_bits);
1714
+ if (ofs->xino_mode > 0) {
1715
+ pr_info("\"xino\" feature enabled using %d upper inode bits.\n",
1716
+ ofs->xino_mode);
14271717 }
14281718
14291719 err = 0;
....@@ -1432,45 +1722,26 @@
14321722 }
14331723
14341724 static struct ovl_entry *ovl_get_lowerstack(struct super_block *sb,
1435
- struct ovl_fs *ofs)
1725
+ const char *lower, unsigned int numlower,
1726
+ struct ovl_fs *ofs, struct ovl_layer *layers)
14361727 {
14371728 int err;
1438
- char *lowertmp, *lower;
14391729 struct path *stack = NULL;
1440
- unsigned int stacklen, numlower = 0, i;
1441
- bool remote = false;
1730
+ unsigned int i;
14421731 struct ovl_entry *oe;
14431732
1444
- err = -ENOMEM;
1445
- lowertmp = kstrdup(ofs->config.lowerdir, GFP_KERNEL);
1446
- if (!lowertmp)
1447
- goto out_err;
1448
-
1449
- err = -EINVAL;
1450
- stacklen = ovl_split_lowerdirs(lowertmp);
1451
- if (stacklen > OVL_MAX_STACK) {
1452
- pr_err("overlayfs: too many lower directories, limit is %d\n",
1453
- OVL_MAX_STACK);
1454
- goto out_err;
1455
- } else if (!ofs->config.upperdir && stacklen == 1) {
1456
- pr_err("overlayfs: at least 2 lowerdir are needed while upperdir nonexistent\n");
1457
- goto out_err;
1458
- } else if (!ofs->config.upperdir && ofs->config.nfs_export &&
1459
- ofs->config.redirect_follow) {
1460
- pr_warn("overlayfs: NFS export requires \"redirect_dir=nofollow\" on non-upper mount, falling back to nfs_export=off.\n");
1461
- ofs->config.nfs_export = false;
1733
+ if (!ofs->config.upperdir && numlower == 1) {
1734
+ pr_err("at least 2 lowerdir are needed while upperdir nonexistent\n");
1735
+ return ERR_PTR(-EINVAL);
14621736 }
14631737
1464
- err = -ENOMEM;
1465
- stack = kcalloc(stacklen, sizeof(struct path), GFP_KERNEL);
1738
+ stack = kcalloc(numlower, sizeof(struct path), GFP_KERNEL);
14661739 if (!stack)
1467
- goto out_err;
1740
+ return ERR_PTR(-ENOMEM);
14681741
14691742 err = -EINVAL;
1470
- lower = lowertmp;
1471
- for (numlower = 0; numlower < stacklen; numlower++) {
1472
- err = ovl_lower_dir(lower, &stack[numlower], ofs,
1473
- &sb->s_stack_depth, &remote);
1743
+ for (i = 0; i < numlower; i++) {
1744
+ err = ovl_lower_dir(lower, &stack[i], ofs, &sb->s_stack_depth);
14741745 if (err)
14751746 goto out_err;
14761747
....@@ -1480,11 +1751,11 @@
14801751 err = -EINVAL;
14811752 sb->s_stack_depth++;
14821753 if (sb->s_stack_depth > FILESYSTEM_MAX_STACK_DEPTH) {
1483
- pr_err("overlayfs: maximum fs stacking depth exceeded\n");
1754
+ pr_err("maximum fs stacking depth exceeded\n");
14841755 goto out_err;
14851756 }
14861757
1487
- err = ovl_get_lower_layers(sb, ofs, stack, numlower);
1758
+ err = ovl_get_layers(sb, ofs, stack, numlower, layers);
14881759 if (err)
14891760 goto out_err;
14901761
....@@ -1495,19 +1766,13 @@
14951766
14961767 for (i = 0; i < numlower; i++) {
14971768 oe->lowerstack[i].dentry = dget(stack[i].dentry);
1498
- oe->lowerstack[i].layer = &ofs->lower_layers[i];
1769
+ oe->lowerstack[i].layer = &ofs->layers[i+1];
14991770 }
1500
-
1501
- if (remote)
1502
- sb->s_d_op = &ovl_reval_dentry_operations;
1503
- else
1504
- sb->s_d_op = &ovl_dentry_operations;
15051771
15061772 out:
15071773 for (i = 0; i < numlower; i++)
15081774 path_put(&stack[i]);
15091775 kfree(stack);
1510
- kfree(lowertmp);
15111776
15121777 return oe;
15131778
....@@ -1537,7 +1802,7 @@
15371802 while (!err && parent != next) {
15381803 if (is_lower && ovl_lookup_trap_inode(sb, parent)) {
15391804 err = -ELOOP;
1540
- pr_err("overlayfs: overlapping %s path\n", name);
1805
+ pr_err("overlapping %s path\n", name);
15411806 } else if (ovl_is_inuse(parent)) {
15421807 err = ovl_report_in_use(ofs, name);
15431808 }
....@@ -1559,8 +1824,8 @@
15591824 {
15601825 int i, err;
15611826
1562
- if (ofs->upper_mnt) {
1563
- err = ovl_check_layer(sb, ofs, ofs->upper_mnt->mnt_root,
1827
+ if (ovl_upper_mnt(ofs)) {
1828
+ err = ovl_check_layer(sb, ofs, ovl_upper_mnt(ofs)->mnt_root,
15641829 "upperdir", false);
15651830 if (err)
15661831 return err;
....@@ -1578,9 +1843,9 @@
15781843 return err;
15791844 }
15801845
1581
- for (i = 0; i < ofs->numlower; i++) {
1846
+ for (i = 1; i < ofs->numlayer; i++) {
15821847 err = ovl_check_layer(sb, ofs,
1583
- ofs->lower_layers[i].mnt->mnt_root,
1848
+ ofs->layers[i].mnt->mnt_root,
15841849 "lowerdir", true);
15851850 if (err)
15861851 return err;
....@@ -1589,14 +1854,57 @@
15891854 return 0;
15901855 }
15911856
1857
+static struct dentry *ovl_get_root(struct super_block *sb,
1858
+ struct dentry *upperdentry,
1859
+ struct ovl_entry *oe)
1860
+{
1861
+ struct dentry *root;
1862
+ struct ovl_path *lowerpath = &oe->lowerstack[0];
1863
+ unsigned long ino = d_inode(lowerpath->dentry)->i_ino;
1864
+ int fsid = lowerpath->layer->fsid;
1865
+ struct ovl_inode_params oip = {
1866
+ .upperdentry = upperdentry,
1867
+ .lowerpath = lowerpath,
1868
+ };
1869
+
1870
+ root = d_make_root(ovl_new_inode(sb, S_IFDIR, 0));
1871
+ if (!root)
1872
+ return NULL;
1873
+
1874
+ root->d_fsdata = oe;
1875
+
1876
+ if (upperdentry) {
1877
+ /* Root inode uses upper st_ino/i_ino */
1878
+ ino = d_inode(upperdentry)->i_ino;
1879
+ fsid = 0;
1880
+ ovl_dentry_set_upper_alias(root);
1881
+ if (ovl_is_impuredir(sb, upperdentry))
1882
+ ovl_set_flag(OVL_IMPURE, d_inode(root));
1883
+ }
1884
+
1885
+ /* Root is always merge -> can have whiteouts */
1886
+ ovl_set_flag(OVL_WHITEOUTS, d_inode(root));
1887
+ ovl_dentry_set_flag(OVL_E_CONNECTED, root);
1888
+ ovl_set_upperdata(d_inode(root));
1889
+ ovl_inode_init(d_inode(root), &oip, ino, fsid);
1890
+ ovl_dentry_update_reval(root, upperdentry, DCACHE_OP_WEAK_REVALIDATE);
1891
+
1892
+ return root;
1893
+}
1894
+
15921895 static int ovl_fill_super(struct super_block *sb, void *data, int silent)
15931896 {
15941897 struct path upperpath = { };
15951898 struct dentry *root_dentry;
15961899 struct ovl_entry *oe;
15971900 struct ovl_fs *ofs;
1901
+ struct ovl_layer *layers;
15981902 struct cred *cred;
1903
+ char *splitlower = NULL;
1904
+ unsigned int numlower;
15991905 int err;
1906
+
1907
+ sb->s_d_op = &ovl_dentry_operations;
16001908
16011909 err = -ENOMEM;
16021910 ofs = kzalloc(sizeof(struct ovl_fs), GFP_KERNEL);
....@@ -1606,6 +1914,9 @@
16061914 ofs->creator_cred = cred = prepare_creds();
16071915 if (!cred)
16081916 goto out_err;
1917
+
1918
+ /* Is there a reason anyone would want not to share whiteouts? */
1919
+ ofs->share_whiteout = true;
16091920
16101921 ofs->config.index = ovl_index_def;
16111922 ofs->config.nfs_export = ovl_nfs_export_def;
....@@ -1618,28 +1929,66 @@
16181929 err = -EINVAL;
16191930 if (!ofs->config.lowerdir) {
16201931 if (!silent)
1621
- pr_err("overlayfs: missing 'lowerdir'\n");
1932
+ pr_err("missing 'lowerdir'\n");
16221933 goto out_err;
16231934 }
16241935
1936
+ err = -ENOMEM;
1937
+ splitlower = kstrdup(ofs->config.lowerdir, GFP_KERNEL);
1938
+ if (!splitlower)
1939
+ goto out_err;
1940
+
1941
+ numlower = ovl_split_lowerdirs(splitlower);
1942
+ if (numlower > OVL_MAX_STACK) {
1943
+ pr_err("too many lower directories, limit is %d\n",
1944
+ OVL_MAX_STACK);
1945
+ goto out_err;
1946
+ }
1947
+
1948
+ layers = kcalloc(numlower + 1, sizeof(struct ovl_layer), GFP_KERNEL);
1949
+ if (!layers)
1950
+ goto out_err;
1951
+
1952
+ ofs->layers = layers;
1953
+ /* Layer 0 is reserved for upper even if there's no upper */
1954
+ ofs->numlayer = 1;
1955
+
16251956 sb->s_stack_depth = 0;
16261957 sb->s_maxbytes = MAX_LFS_FILESIZE;
1958
+ atomic_long_set(&ofs->last_ino, 1);
16271959 /* Assume underlaying fs uses 32bit inodes unless proven otherwise */
1628
- if (ofs->config.xino != OVL_XINO_OFF)
1629
- ofs->xino_bits = BITS_PER_LONG - 32;
1960
+ if (ofs->config.xino != OVL_XINO_OFF) {
1961
+ ofs->xino_mode = BITS_PER_LONG - 32;
1962
+ if (!ofs->xino_mode) {
1963
+ pr_warn("xino not supported on 32bit kernel, falling back to xino=off.\n");
1964
+ ofs->config.xino = OVL_XINO_OFF;
1965
+ }
1966
+ }
16301967
16311968 /* alloc/destroy_inode needed for setting up traps in inode cache */
16321969 sb->s_op = &ovl_super_operations;
16331970
16341971 if (ofs->config.upperdir) {
1972
+ struct super_block *upper_sb;
1973
+
16351974 if (!ofs->config.workdir) {
1636
- pr_err("overlayfs: missing 'workdir'\n");
1975
+ pr_err("missing 'workdir'\n");
16371976 goto out_err;
16381977 }
16391978
1640
- err = ovl_get_upper(sb, ofs, &upperpath);
1979
+ err = ovl_get_upper(sb, ofs, &layers[0], &upperpath);
16411980 if (err)
16421981 goto out_err;
1982
+
1983
+ upper_sb = ovl_upper_mnt(ofs)->mnt_sb;
1984
+ if (!ovl_should_sync(ofs)) {
1985
+ ofs->errseq = errseq_sample(&upper_sb->s_wb_err);
1986
+ if (errseq_check(&upper_sb->s_wb_err, ofs->errseq)) {
1987
+ err = -EIO;
1988
+ pr_err("Cannot mount volatile when upperdir has an unseen error. Sync upperdir fs to clear state.\n");
1989
+ goto out_err;
1990
+ }
1991
+ }
16431992
16441993 err = ovl_get_workdir(sb, ofs, &upperpath);
16451994 if (err)
....@@ -1648,31 +1997,26 @@
16481997 if (!ofs->workdir)
16491998 sb->s_flags |= SB_RDONLY;
16501999
1651
- sb->s_stack_depth = ofs->upper_mnt->mnt_sb->s_stack_depth;
1652
- sb->s_time_gran = ofs->upper_mnt->mnt_sb->s_time_gran;
1653
-
2000
+ sb->s_stack_depth = upper_sb->s_stack_depth;
2001
+ sb->s_time_gran = upper_sb->s_time_gran;
16542002 }
1655
- oe = ovl_get_lowerstack(sb, ofs);
2003
+ oe = ovl_get_lowerstack(sb, splitlower, numlower, ofs, layers);
16562004 err = PTR_ERR(oe);
16572005 if (IS_ERR(oe))
16582006 goto out_err;
16592007
16602008 /* If the upper fs is nonexistent, we mark overlayfs r/o too */
1661
- if (!ofs->upper_mnt)
2009
+ if (!ovl_upper_mnt(ofs))
16622010 sb->s_flags |= SB_RDONLY;
16632011
1664
- if (!(ovl_force_readonly(ofs)) && ofs->config.index) {
2012
+ if (!ovl_force_readonly(ofs) && ofs->config.index) {
16652013 err = ovl_get_indexdir(sb, ofs, oe, &upperpath);
16662014 if (err)
16672015 goto out_free_oe;
16682016
16692017 /* Force r/o mount with no index dir */
1670
- if (!ofs->indexdir) {
1671
- dput(ofs->workdir);
1672
- ofs->workdir = NULL;
2018
+ if (!ofs->indexdir)
16732019 sb->s_flags |= SB_RDONLY;
1674
- }
1675
-
16762020 }
16772021
16782022 err = ovl_check_overlapping_layers(sb, ofs);
....@@ -1682,14 +2026,14 @@
16822026 /* Show index=off in /proc/mounts for forced r/o mount */
16832027 if (!ofs->indexdir) {
16842028 ofs->config.index = false;
1685
- if (ofs->upper_mnt && ofs->config.nfs_export) {
1686
- pr_warn("overlayfs: NFS export requires an index dir, falling back to nfs_export=off.\n");
2029
+ if (ovl_upper_mnt(ofs) && ofs->config.nfs_export) {
2030
+ pr_warn("NFS export requires an index dir, falling back to nfs_export=off.\n");
16872031 ofs->config.nfs_export = false;
16882032 }
16892033 }
16902034
16912035 if (ofs->config.metacopy && ofs->config.nfs_export) {
1692
- pr_warn("overlayfs: NFS export is not supported with metadata only copy up, falling back to nfs_export=off.\n");
2036
+ pr_warn("NFS export is not supported with metadata only copy up, falling back to nfs_export=off.\n");
16932037 ofs->config.nfs_export = false;
16942038 }
16952039
....@@ -1703,27 +2047,15 @@
17032047 sb->s_xattr = ovl_xattr_handlers;
17042048 sb->s_fs_info = ofs;
17052049 sb->s_flags |= SB_POSIXACL;
2050
+ sb->s_iflags |= SB_I_SKIP_SYNC;
17062051
17072052 err = -ENOMEM;
1708
- root_dentry = d_make_root(ovl_new_inode(sb, S_IFDIR, 0));
2053
+ root_dentry = ovl_get_root(sb, upperpath.dentry, oe);
17092054 if (!root_dentry)
17102055 goto out_free_oe;
17112056
1712
- root_dentry->d_fsdata = oe;
1713
-
17142057 mntput(upperpath.mnt);
1715
- if (upperpath.dentry) {
1716
- ovl_dentry_set_upper_alias(root_dentry);
1717
- if (ovl_is_impuredir(upperpath.dentry))
1718
- ovl_set_flag(OVL_IMPURE, d_inode(root_dentry));
1719
- }
1720
-
1721
- /* Root is always merge -> can have whiteouts */
1722
- ovl_set_flag(OVL_WHITEOUTS, d_inode(root_dentry));
1723
- ovl_dentry_set_flag(OVL_E_CONNECTED, root_dentry);
1724
- ovl_set_upperdata(d_inode(root_dentry));
1725
- ovl_inode_init(d_inode(root_dentry), upperpath.dentry,
1726
- ovl_dentry_lower(root_dentry), NULL);
2058
+ kfree(splitlower);
17272059
17282060 sb->s_root = root_dentry;
17292061 return 0;
....@@ -1732,6 +2064,7 @@
17322064 ovl_entry_stack_free(oe);
17332065 kfree(oe);
17342066 out_err:
2067
+ kfree(splitlower);
17352068 path_put(&upperpath);
17362069 ovl_free_fs(ofs);
17372070 out:
....@@ -1771,9 +2104,15 @@
17712104 if (ovl_inode_cachep == NULL)
17722105 return -ENOMEM;
17732106
1774
- err = register_filesystem(&ovl_fs_type);
1775
- if (err)
1776
- kmem_cache_destroy(ovl_inode_cachep);
2107
+ err = ovl_aio_request_cache_init();
2108
+ if (!err) {
2109
+ err = register_filesystem(&ovl_fs_type);
2110
+ if (!err)
2111
+ return 0;
2112
+
2113
+ ovl_aio_request_cache_destroy();
2114
+ }
2115
+ kmem_cache_destroy(ovl_inode_cachep);
17772116
17782117 return err;
17792118 }
....@@ -1788,7 +2127,7 @@
17882127 */
17892128 rcu_barrier();
17902129 kmem_cache_destroy(ovl_inode_cachep);
1791
-
2130
+ ovl_aio_request_cache_destroy();
17922131 }
17932132
17942133 module_init(ovl_init);