diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2011-11-23 11:57:51 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-01-03 22:52:35 -0500 |
commit | a561be7100cd610bd2e082f3211c1dfb45835817 (patch) | |
tree | a1016a11df967be6f289a4e8ae29597ba39df17e /fs/xfs/xfs_ioctl32.c | |
parent | f47ec3f28354795f000c14bf18ed967ec81a3ec3 (diff) |
switch a bunch of places to mnt_want_write_file()
it's both faster (in case when file has been opened for write) and cleaner.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/xfs/xfs_ioctl32.c')
-rw-r--r-- | fs/xfs/xfs_ioctl32.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/xfs/xfs_ioctl32.c b/fs/xfs/xfs_ioctl32.c index 54e623bfbb85..dd4ba1d4c582 100644 --- a/fs/xfs/xfs_ioctl32.c +++ b/fs/xfs/xfs_ioctl32.c | |||
@@ -454,7 +454,7 @@ xfs_compat_attrmulti_by_handle( | |||
454 | &ops[i].am_length, ops[i].am_flags); | 454 | &ops[i].am_length, ops[i].am_flags); |
455 | break; | 455 | break; |
456 | case ATTR_OP_SET: | 456 | case ATTR_OP_SET: |
457 | ops[i].am_error = mnt_want_write(parfilp->f_path.mnt); | 457 | ops[i].am_error = mnt_want_write_file(parfilp); |
458 | if (ops[i].am_error) | 458 | if (ops[i].am_error) |
459 | break; | 459 | break; |
460 | ops[i].am_error = xfs_attrmulti_attr_set( | 460 | ops[i].am_error = xfs_attrmulti_attr_set( |
@@ -464,7 +464,7 @@ xfs_compat_attrmulti_by_handle( | |||
464 | mnt_drop_write(parfilp->f_path.mnt); | 464 | mnt_drop_write(parfilp->f_path.mnt); |
465 | break; | 465 | break; |
466 | case ATTR_OP_REMOVE: | 466 | case ATTR_OP_REMOVE: |
467 | ops[i].am_error = mnt_want_write(parfilp->f_path.mnt); | 467 | ops[i].am_error = mnt_want_write_file(parfilp); |
468 | if (ops[i].am_error) | 468 | if (ops[i].am_error) |
469 | break; | 469 | break; |
470 | ops[i].am_error = xfs_attrmulti_attr_remove( | 470 | ops[i].am_error = xfs_attrmulti_attr_remove( |