diff options
| author | Christoph Hellwig <hch@lst.de> | 2018-07-24 14:04:13 +0200 |
|---|---|---|
| committer | Jens Axboe <axboe@kernel.dk> | 2018-07-24 14:39:28 -0600 |
| commit | 3bb5098310317ca62304bd21af6ccea57d799b06 (patch) | |
| tree | 67273c4ef20f233c77023131c5e1eac6831d4f39 /block | |
| parent | 24d5493f207ce0ce38df80ce86c907417e04594a (diff) | |
| download | lwn-3bb5098310317ca62304bd21af6ccea57d799b06.tar.gz lwn-3bb5098310317ca62304bd21af6ccea57d799b06.zip | |
block: bio_set_pages_dirty can't see NULL bv_page in a valid bio_vec
So don't bother handling it.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block')
| -rw-r--r-- | block/bio.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/block/bio.c b/block/bio.c index 504b42278099..07d3ffd95989 100644 --- a/block/bio.c +++ b/block/bio.c @@ -1636,10 +1636,8 @@ void bio_set_pages_dirty(struct bio *bio) int i; bio_for_each_segment_all(bvec, bio, i) { - struct page *page = bvec->bv_page; - - if (page && !PageCompound(page)) - set_page_dirty_lock(page); + if (!PageCompound(bvec->bv_page)) + set_page_dirty_lock(bvec->bv_page); } } EXPORT_SYMBOL_GPL(bio_set_pages_dirty); |