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_ioctl.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_ioctl.c')
| -rw-r--r-- | fs/xfs/xfs_ioctl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/xfs/xfs_ioctl.c b/fs/xfs/xfs_ioctl.c index d99a90518909..b436e17c753e 100644 --- a/fs/xfs/xfs_ioctl.c +++ b/fs/xfs/xfs_ioctl.c | |||
| @@ -559,7 +559,7 @@ xfs_attrmulti_by_handle( | |||
| 559 | ops[i].am_flags); | 559 | ops[i].am_flags); |
| 560 | break; | 560 | break; |
| 561 | case ATTR_OP_SET: | 561 | case ATTR_OP_SET: |
| 562 | ops[i].am_error = mnt_want_write(parfilp->f_path.mnt); | 562 | ops[i].am_error = mnt_want_write_file(parfilp); |
| 563 | if (ops[i].am_error) | 563 | if (ops[i].am_error) |
| 564 | break; | 564 | break; |
| 565 | ops[i].am_error = xfs_attrmulti_attr_set( | 565 | ops[i].am_error = xfs_attrmulti_attr_set( |
| @@ -569,7 +569,7 @@ xfs_attrmulti_by_handle( | |||
| 569 | mnt_drop_write(parfilp->f_path.mnt); | 569 | mnt_drop_write(parfilp->f_path.mnt); |
| 570 | break; | 570 | break; |
| 571 | case ATTR_OP_REMOVE: | 571 | case ATTR_OP_REMOVE: |
| 572 | ops[i].am_error = mnt_want_write(parfilp->f_path.mnt); | 572 | ops[i].am_error = mnt_want_write_file(parfilp); |
| 573 | if (ops[i].am_error) | 573 | if (ops[i].am_error) |
| 574 | break; | 574 | break; |
| 575 | ops[i].am_error = xfs_attrmulti_attr_remove( | 575 | ops[i].am_error = xfs_attrmulti_attr_remove( |