diff options
author | Guillaume Chazarain <guichaz@yahoo.fr> | 2007-05-08 03:23:25 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-05-08 14:14:57 -0400 |
commit | 3e9f45bd18191bbd05468b19b7064b8da8262aba (patch) | |
tree | 06c790a9cb1afc83d170447a277e51f5a1a5f303 /mm/page-writeback.c | |
parent | c83e44842074a87614c78eca70fa6467b0bc3c4a (diff) |
Factor outstanding I/O error handling
Cleanup: setting an outstanding error on a mapping was open coded too many
times. Factor it out in mapping_set_error().
Signed-off-by: Guillaume Chazarain <guichaz@yahoo.fr>
Cc: Steven Whitehouse <swhiteho@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/page-writeback.c')
-rw-r--r-- | mm/page-writeback.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/mm/page-writeback.c b/mm/page-writeback.c index 029dfad5a235..63cd88840eb2 100644 --- a/mm/page-writeback.c +++ b/mm/page-writeback.c | |||
@@ -683,12 +683,7 @@ retry: | |||
683 | } | 683 | } |
684 | 684 | ||
685 | ret = (*writepage)(page, wbc); | 685 | ret = (*writepage)(page, wbc); |
686 | if (ret) { | 686 | mapping_set_error(mapping, ret); |
687 | if (ret == -ENOSPC) | ||
688 | set_bit(AS_ENOSPC, &mapping->flags); | ||
689 | else | ||
690 | set_bit(AS_EIO, &mapping->flags); | ||
691 | } | ||
692 | 687 | ||
693 | if (unlikely(ret == AOP_WRITEPAGE_ACTIVATE)) | 688 | if (unlikely(ret == AOP_WRITEPAGE_ACTIVATE)) |
694 | unlock_page(page); | 689 | unlock_page(page); |