Commit 921aebfa authored by Matias Bjørling's avatar Matias Bjørling Committed by Jens Axboe

lightnvm: pblk: fix read_bitmap for 32bit archs

If using pblk on a 32bit architecture, and there is a need to
perform a partial read, the partial read bitmap will only have
allocated 32 entries, where as 64 are needed.

Make sure that the read_bitmap is initialized to 64bits on 32bit
architectures as well.
Signed-off-by: default avatarMatias Bjørling <mb@lightnvm.io>
Reviewed-by: default avatarIgor Konopko <igor.j.konopko@intel.com>
Reviewed-by: default avatarJavier González <javier@cnexlabs.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
parent 242e461f
...@@ -401,7 +401,7 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio) ...@@ -401,7 +401,7 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio)
struct pblk_g_ctx *r_ctx; struct pblk_g_ctx *r_ctx;
struct nvm_rq *rqd; struct nvm_rq *rqd;
unsigned int bio_init_idx; unsigned int bio_init_idx;
unsigned long read_bitmap; /* Max 64 ppas per request */ DECLARE_BITMAP(read_bitmap, NVM_MAX_VLBA);
int ret = NVM_IO_ERR; int ret = NVM_IO_ERR;
/* logic error: lba out-of-bounds. Ignore read request */ /* logic error: lba out-of-bounds. Ignore read request */
...@@ -413,7 +413,7 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio) ...@@ -413,7 +413,7 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio)
generic_start_io_acct(q, READ, bio_sectors(bio), &pblk->disk->part0); generic_start_io_acct(q, READ, bio_sectors(bio), &pblk->disk->part0);
bitmap_zero(&read_bitmap, nr_secs); bitmap_zero(read_bitmap, nr_secs);
rqd = pblk_alloc_rqd(pblk, PBLK_READ); rqd = pblk_alloc_rqd(pblk, PBLK_READ);
...@@ -444,19 +444,19 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio) ...@@ -444,19 +444,19 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio)
rqd->ppa_list = rqd->meta_list + pblk_dma_meta_size; rqd->ppa_list = rqd->meta_list + pblk_dma_meta_size;
rqd->dma_ppa_list = rqd->dma_meta_list + pblk_dma_meta_size; rqd->dma_ppa_list = rqd->dma_meta_list + pblk_dma_meta_size;
pblk_read_ppalist_rq(pblk, rqd, bio, blba, &read_bitmap); pblk_read_ppalist_rq(pblk, rqd, bio, blba, read_bitmap);
} else { } else {
pblk_read_rq(pblk, rqd, bio, blba, &read_bitmap); pblk_read_rq(pblk, rqd, bio, blba, read_bitmap);
} }
if (bitmap_full(&read_bitmap, nr_secs)) { if (bitmap_full(read_bitmap, nr_secs)) {
atomic_inc(&pblk->inflight_io); atomic_inc(&pblk->inflight_io);
__pblk_end_io_read(pblk, rqd, false); __pblk_end_io_read(pblk, rqd, false);
return NVM_IO_DONE; return NVM_IO_DONE;
} }
/* All sectors are to be read from the device */ /* All sectors are to be read from the device */
if (bitmap_empty(&read_bitmap, rqd->nr_ppas)) { if (bitmap_empty(read_bitmap, rqd->nr_ppas)) {
struct bio *int_bio = NULL; struct bio *int_bio = NULL;
/* Clone read bio to deal with read errors internally */ /* Clone read bio to deal with read errors internally */
...@@ -480,7 +480,7 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio) ...@@ -480,7 +480,7 @@ int pblk_submit_read(struct pblk *pblk, struct bio *bio)
/* The read bio request could be partially filled by the write buffer, /* The read bio request could be partially filled by the write buffer,
* but there are some holes that need to be read from the drive. * but there are some holes that need to be read from the drive.
*/ */
return pblk_partial_read(pblk, rqd, bio, bio_init_idx, &read_bitmap); return pblk_partial_read(pblk, rqd, bio, bio_init_idx, read_bitmap);
fail_rqd_free: fail_rqd_free:
pblk_free_rqd(pblk, rqd, PBLK_READ); pblk_free_rqd(pblk, rqd, PBLK_READ);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment