Commit c4758795 authored by Al Viro's avatar Al Viro
Browse files

sanitize ->fsync() for affs 



unfortunately, for affs (especially for affs directories) we have
no real way to keep track of metadata ownership.  So we have to
do more or less what file_fsync() does, but we do *not* need to
call write_super() there.

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 4427f0c3

fs/affs/affs.h

+1 −0
Original line number Diff line number Diff line
@@ -182,6 +182,7 @@ extern int affs_add_entry(struct inode *dir, struct inode *inode, struct dent 


void		affs_free_prealloc(struct inode *inode);

extern void	affs_truncate(struct inode *);

int		affs_file_fsync(struct file *, struct dentry *, int);



/* dir.c */

fs/affs/dir.c

+1 −1
Original line number Diff line number Diff line
@@ -21,7 +21,7 @@ const struct file_operations affs_dir_operations = { 
	.read		= generic_read_dir,

	.llseek		= generic_file_llseek,

	.readdir	= affs_readdir,

	.fsync		= file_fsync,

	.fsync		= affs_file_fsync,

};



/*

fs/affs/file.c

+13 −1
Original line number Diff line number Diff line
@@ -34,7 +34,7 @@ const struct file_operations affs_file_operations = { 
	.mmap		= generic_file_mmap,

	.open		= affs_file_open,

	.release	= affs_file_release,

	.fsync		= file_fsync,

	.fsync		= affs_file_fsync,

	.splice_read	= generic_file_splice_read,

};
@@ -915,3 +915,15 @@ affs_truncate(struct inode *inode) 
	}

	affs_free_prealloc(inode);

}



int affs_file_fsync(struct file *filp, struct dentry *dentry, int datasync)

{

	struct inode * inode = dentry->d_inode;

	int ret, err;



	ret = write_inode_now(inode, 0);

	err = sync_blockdev(inode->i_sb->s_bdev);

	if (!ret)

		ret = err;

	return ret;

}

CRA Git | Maintained and supported by SUSTech CRA and CCSE