summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorDamien Le Moal2017-07-06 13:21:15 +0200
committerJens Axboe2017-07-06 17:43:20 +0200
commit615d22a51c04856efe62af6e1d5b450aaf5cc2c0 (patch)
tree2d80a79b5a229c70969784fe3b1408e32b32679c /drivers
parentmtip32xx: avoid to read HOST_CAP from HW in .queue_rq() (diff)
downloadkernel-qcow2-linux-615d22a51c04856efe62af6e1d5b450aaf5cc2c0.tar.gz
kernel-qcow2-linux-615d22a51c04856efe62af6e1d5b450aaf5cc2c0.tar.xz
kernel-qcow2-linux-615d22a51c04856efe62af6e1d5b450aaf5cc2c0.zip
block: Fix __blkdev_issue_zeroout loop
The BIO issuing loop in __blkdev_issue_zeroout() is allocating BIOs with a maximum number of bvec (pages) equal to min(nr_sects, (sector_t)BIO_MAX_PAGES) This works since the requested number of bvecs will always be limited to the absolute maximum number supported (BIO_MAX_PAGES), but this is ineficient as too many bvec entries may be requested due to the different units being used in the min() operation (number of sectors vs number of pages). To fix this, introduce the helper __blkdev_sectors_to_bio_pages() to correctly calculate the number of bvecs for zeroout BIOs as the issuing loop progresses. The calculation is done using consistent units and makes sure that the number of pages return is at least 1 (for cases where the number of sectors is less that the number of sectors in a page). Also remove a trailing space after the bit shift in the internal loop min() call. Signed-off-by: Damien Le Moal <damien.lemoal@wdc.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'drivers')
0 files changed, 0 insertions, 0 deletions