123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081 |
- From: Felix Fietkau <[email protected]>
- Subject: jffs2: use .rename2 and add RENAME_WHITEOUT support
- It is required for renames on overlayfs
- Signed-off-by: Felix Fietkau <[email protected]>
- ---
- --- a/fs/jffs2/dir.c
- +++ b/fs/jffs2/dir.c
- @@ -617,8 +617,8 @@ static int jffs2_rmdir (struct inode *di
- return ret;
- }
-
- -static int jffs2_mknod (struct mnt_idmap *idmap, struct inode *dir_i,
- - struct dentry *dentry, umode_t mode, dev_t rdev)
- +static int __jffs2_mknod (struct mnt_idmap *idmap, struct inode *dir_i,
- + struct dentry *dentry, umode_t mode, dev_t rdev, bool whiteout)
- {
- struct jffs2_inode_info *f, *dir_f;
- struct jffs2_sb_info *c;
- @@ -758,7 +758,11 @@ static int jffs2_mknod (struct mnt_idmap
- mutex_unlock(&dir_f->sem);
- jffs2_complete_reservation(c);
-
- - d_instantiate_new(dentry, inode);
- + if (!whiteout)
- + d_instantiate_new(dentry, inode);
- + else
- + unlock_new_inode(inode);
- +
- return 0;
-
- fail:
- @@ -766,6 +770,19 @@ static int jffs2_mknod (struct mnt_idmap
- return ret;
- }
-
- +static int jffs2_mknod (struct mnt_idmap *idmap, struct inode *dir_i,
- + struct dentry *dentry, umode_t mode, dev_t rdev)
- +{
- + return __jffs2_mknod(idmap, dir_i, dentry, mode, rdev, false);
- +}
- +
- +static int jffs2_whiteout (struct mnt_idmap *idmap, struct inode *old_dir,
- + struct dentry *old_dentry)
- +{
- + return __jffs2_mknod(idmap, old_dir, old_dentry, S_IFCHR | WHITEOUT_MODE,
- + WHITEOUT_DEV, true);
- +}
- +
- static int jffs2_rename (struct mnt_idmap *idmap,
- struct inode *old_dir_i, struct dentry *old_dentry,
- struct inode *new_dir_i, struct dentry *new_dentry,
- @@ -777,7 +794,7 @@ static int jffs2_rename (struct mnt_idma
- uint8_t type;
- uint32_t now;
-
- - if (flags & ~RENAME_NOREPLACE)
- + if (flags & ~(RENAME_NOREPLACE|RENAME_WHITEOUT))
- return -EINVAL;
-
- /* The VFS will check for us and prevent trying to rename a
- @@ -843,9 +860,14 @@ static int jffs2_rename (struct mnt_idma
- if (d_is_dir(old_dentry) && !victim_f)
- inc_nlink(new_dir_i);
-
- - /* Unlink the original */
- - ret = jffs2_do_unlink(c, JFFS2_INODE_INFO(old_dir_i),
- - old_dentry->d_name.name, old_dentry->d_name.len, NULL, now);
- + if (flags & RENAME_WHITEOUT)
- + /* Replace with whiteout */
- + ret = jffs2_whiteout(idmap, old_dir_i, old_dentry);
- + else
- + /* Unlink the original */
- + ret = jffs2_do_unlink(c, JFFS2_INODE_INFO(old_dir_i),
- + old_dentry->d_name.name,
- + old_dentry->d_name.len, NULL, now);
-
- /* We don't touch inode->i_nlink */
-
|