diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-06-01 13:34:35 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-06-01 13:34:35 -0400 |
commit | 1193755ac6328ad240ba987e6ec41d5e8baf0680 (patch) | |
tree | 40bf847d7e3ebaa57b107151d14e6cd1d280cc6d /fs/fuse | |
parent | 4edebed86690eb8db9af3ab85baf4a34e73266cc (diff) | |
parent | 0ef97dcfce4179a2eba046b855ee2f91d6f1b414 (diff) |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs
Pull vfs changes from Al Viro.
"A lot of misc stuff. The obvious groups:
* Miklos' atomic_open series; kills the damn abuse of
->d_revalidate() by NFS, which was the major stumbling block for
all work in that area.
* ripping security_file_mmap() and dealing with deadlocks in the
area; sanitizing the neighborhood of vm_mmap()/vm_munmap() in
general.
* ->encode_fh() switched to saner API; insane fake dentry in
mm/cleancache.c gone.
* assorted annotations in fs (endianness, __user)
* parts of Artem's ->s_dirty work (jff2 and reiserfs parts)
* ->update_time() work from Josef.
* other bits and pieces all over the place.
Normally it would've been in two or three pull requests, but
signal.git stuff had eaten a lot of time during this cycle ;-/"
Fix up trivial conflicts in Documentation/filesystems/vfs.txt (the
'truncate_range' inode method was removed by the VM changes, the VFS
update adds an 'update_time()' method), and in fs/btrfs/ulist.[ch] (due
to sparse fix added twice, with other changes nearby).
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs: (95 commits)
nfs: don't open in ->d_revalidate
vfs: retry last component if opening stale dentry
vfs: nameidata_to_filp(): don't throw away file on error
vfs: nameidata_to_filp(): inline __dentry_open()
vfs: do_dentry_open(): don't put filp
vfs: split __dentry_open()
vfs: do_last() common post lookup
vfs: do_last(): add audit_inode before open
vfs: do_last(): only return EISDIR for O_CREAT
vfs: do_last(): check LOOKUP_DIRECTORY
vfs: do_last(): make ENOENT exit RCU safe
vfs: make follow_link check RCU safe
vfs: do_last(): use inode variable
vfs: do_last(): inline walk_component()
vfs: do_last(): make exit RCU safe
vfs: split do_lookup()
Btrfs: move over to use ->update_time
fs: introduce inode operation ->update_time
reiserfs: get rid of resierfs_sync_super
reiserfs: mark the superblock as dirty a bit later
...
Diffstat (limited to 'fs/fuse')
-rw-r--r-- | fs/fuse/file.c | 4 | ||||
-rw-r--r-- | fs/fuse/inode.c | 17 |
2 files changed, 8 insertions, 13 deletions
diff --git a/fs/fuse/file.c b/fs/fuse/file.c index 504e61b7fd75..9562109d3a87 100644 --- a/fs/fuse/file.c +++ b/fs/fuse/file.c | |||
@@ -962,7 +962,9 @@ static ssize_t fuse_file_aio_write(struct kiocb *iocb, const struct iovec *iov, | |||
962 | if (err) | 962 | if (err) |
963 | goto out; | 963 | goto out; |
964 | 964 | ||
965 | file_update_time(file); | 965 | err = file_update_time(file); |
966 | if (err) | ||
967 | goto out; | ||
966 | 968 | ||
967 | if (file->f_flags & O_DIRECT) { | 969 | if (file->f_flags & O_DIRECT) { |
968 | written = generic_file_direct_write(iocb, iov, &nr_segs, | 970 | written = generic_file_direct_write(iocb, iov, &nr_segs, |
diff --git a/fs/fuse/inode.c b/fs/fuse/inode.c index 56f6dcf30768..42678a33b7bb 100644 --- a/fs/fuse/inode.c +++ b/fs/fuse/inode.c | |||
@@ -627,12 +627,10 @@ static struct dentry *fuse_get_dentry(struct super_block *sb, | |||
627 | return ERR_PTR(err); | 627 | return ERR_PTR(err); |
628 | } | 628 | } |
629 | 629 | ||
630 | static int fuse_encode_fh(struct dentry *dentry, u32 *fh, int *max_len, | 630 | static int fuse_encode_fh(struct inode *inode, u32 *fh, int *max_len, |
631 | int connectable) | 631 | struct inode *parent) |
632 | { | 632 | { |
633 | struct inode *inode = dentry->d_inode; | 633 | int len = parent ? 6 : 3; |
634 | bool encode_parent = connectable && !S_ISDIR(inode->i_mode); | ||
635 | int len = encode_parent ? 6 : 3; | ||
636 | u64 nodeid; | 634 | u64 nodeid; |
637 | u32 generation; | 635 | u32 generation; |
638 | 636 | ||
@@ -648,14 +646,9 @@ static int fuse_encode_fh(struct dentry *dentry, u32 *fh, int *max_len, | |||
648 | fh[1] = (u32)(nodeid & 0xffffffff); | 646 | fh[1] = (u32)(nodeid & 0xffffffff); |
649 | fh[2] = generation; | 647 | fh[2] = generation; |
650 | 648 | ||
651 | if (encode_parent) { | 649 | if (parent) { |
652 | struct inode *parent; | ||
653 | |||
654 | spin_lock(&dentry->d_lock); | ||
655 | parent = dentry->d_parent->d_inode; | ||
656 | nodeid = get_fuse_inode(parent)->nodeid; | 650 | nodeid = get_fuse_inode(parent)->nodeid; |
657 | generation = parent->i_generation; | 651 | generation = parent->i_generation; |
658 | spin_unlock(&dentry->d_lock); | ||
659 | 652 | ||
660 | fh[3] = (u32)(nodeid >> 32); | 653 | fh[3] = (u32)(nodeid >> 32); |
661 | fh[4] = (u32)(nodeid & 0xffffffff); | 654 | fh[4] = (u32)(nodeid & 0xffffffff); |
@@ -663,7 +656,7 @@ static int fuse_encode_fh(struct dentry *dentry, u32 *fh, int *max_len, | |||
663 | } | 656 | } |
664 | 657 | ||
665 | *max_len = len; | 658 | *max_len = len; |
666 | return encode_parent ? 0x82 : 0x81; | 659 | return parent ? 0x82 : 0x81; |
667 | } | 660 | } |
668 | 661 | ||
669 | static struct dentry *fuse_fh_to_dentry(struct super_block *sb, | 662 | static struct dentry *fuse_fh_to_dentry(struct super_block *sb, |