diff options
author | Michal Hocko <mhocko@suse.com> | 2016-10-11 16:56:01 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2016-10-11 18:06:33 -0400 |
commit | 5114a97a8bce7f4ead29a32b67dee85438699b9e (patch) | |
tree | 1033f6f06e9d19d79271a82f09b9d43114953a42 /fs/afs/write.c | |
parent | 97139d4a6f26445de47b378cddd5192c0278f863 (diff) |
fs: use mapping_set_error instead of opencoded set_bit
The mapping_set_error() helper sets the correct AS_ flag for the mapping
so there is no reason to open code it. Use the helper directly.
[akpm@linux-foundation.org: be honest about conversion from -ENXIO to -EIO]
Link: http://lkml.kernel.org/r/20160912111608.2588-2-mhocko@kernel.org
Signed-off-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/afs/write.c')
-rw-r--r-- | fs/afs/write.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/afs/write.c b/fs/afs/write.c index 14d506efd1aa..f865c3f05bea 100644 --- a/fs/afs/write.c +++ b/fs/afs/write.c | |||
@@ -398,8 +398,7 @@ no_more: | |||
398 | switch (ret) { | 398 | switch (ret) { |
399 | case -EDQUOT: | 399 | case -EDQUOT: |
400 | case -ENOSPC: | 400 | case -ENOSPC: |
401 | set_bit(AS_ENOSPC, | 401 | mapping_set_error(wb->vnode->vfs_inode.i_mapping, -ENOSPC); |
402 | &wb->vnode->vfs_inode.i_mapping->flags); | ||
403 | break; | 402 | break; |
404 | case -EROFS: | 403 | case -EROFS: |
405 | case -EIO: | 404 | case -EIO: |
@@ -409,7 +408,7 @@ no_more: | |||
409 | case -ENOMEDIUM: | 408 | case -ENOMEDIUM: |
410 | case -ENXIO: | 409 | case -ENXIO: |
411 | afs_kill_pages(wb->vnode, true, first, last); | 410 | afs_kill_pages(wb->vnode, true, first, last); |
412 | set_bit(AS_EIO, &wb->vnode->vfs_inode.i_mapping->flags); | 411 | mapping_set_error(wb->vnode->vfs_inode.i_mapping, -EIO); |
413 | break; | 412 | break; |
414 | case -EACCES: | 413 | case -EACCES: |
415 | case -EPERM: | 414 | case -EPERM: |