On Fri, May 03, 2019 at 09:50:08AM +0300, Nikolay Borisov wrote: > > @@ -1474,7 +1456,9 @@ static void __bio_unmap_user(struct bio *bio) > > */ > > void bio_unmap_user(struct bio *bio) > > { > > - __bio_unmap_user(bio); > > + bio_set_pages_dirty(bio); > > Doesn't this need to be : > > if (bio_data_dir(bio) == READ) > bio_set_pages_dirty() Yes, indeed.