diff options
author | Mike Christie <mchristi@redhat.com> | 2016-08-04 16:23:34 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2016-08-04 16:25:33 -0400 |
commit | abf545484d31b68777a85c5c8f5b4bcde08283eb (patch) | |
tree | ee91e0a9e679e13cff54ede6b6b489627d72c1b1 /mm | |
parent | c1c87c2ba9ec06d8ba9e8a26c18c67a2ba9cd9c1 (diff) |
mm/block: convert rw_page users to bio op use
The rw_page users were not converted to use bio/req ops. As a result
bdev_write_page is not passing down REQ_OP_WRITE and the IOs will
be sent down as reads.
Signed-off-by: Mike Christie <mchristi@redhat.com>
Fixes: 4e1b2d52a80d ("block, fs, drivers: remove REQ_OP compat defs and related code")
Modified by me to:
1) Drop op_flags passing into ->rw_page(), as we don't use it.
2) Make op_is_write() and friends safe to use for !CONFIG_BLOCK
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'mm')
-rw-r--r-- | mm/filemap.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/mm/filemap.c b/mm/filemap.c index 3083ded98b15..daef091d4c50 100644 --- a/mm/filemap.c +++ b/mm/filemap.c | |||
@@ -887,9 +887,9 @@ EXPORT_SYMBOL(end_page_writeback); | |||
887 | * After completing I/O on a page, call this routine to update the page | 887 | * After completing I/O on a page, call this routine to update the page |
888 | * flags appropriately | 888 | * flags appropriately |
889 | */ | 889 | */ |
890 | void page_endio(struct page *page, int rw, int err) | 890 | void page_endio(struct page *page, int op, int err) |
891 | { | 891 | { |
892 | if (rw == READ) { | 892 | if (!op_is_write(op)) { |
893 | if (!err) { | 893 | if (!err) { |
894 | SetPageUptodate(page); | 894 | SetPageUptodate(page); |
895 | } else { | 895 | } else { |
@@ -897,7 +897,7 @@ void page_endio(struct page *page, int rw, int err) | |||
897 | SetPageError(page); | 897 | SetPageError(page); |
898 | } | 898 | } |
899 | unlock_page(page); | 899 | unlock_page(page); |
900 | } else { /* rw == WRITE */ | 900 | } else { |
901 | if (err) { | 901 | if (err) { |
902 | SetPageError(page); | 902 | SetPageError(page); |
903 | if (page->mapping) | 903 | if (page->mapping) |