nvme: fix write zeroes offset and count
The implementation used blocks units rather than the expected bytes.
Fixes: c03e7ef12a
("nvme: Implement Write Zeroes")
Reported-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
parent
23dece19da
commit
9d6459d21a
|
@ -324,8 +324,8 @@ static uint16_t nvme_write_zeros(NvmeCtrl *n, NvmeNamespace *ns, NvmeCmd *cmd,
|
||||||
const uint8_t data_shift = ns->id_ns.lbaf[lba_index].ds;
|
const uint8_t data_shift = ns->id_ns.lbaf[lba_index].ds;
|
||||||
uint64_t slba = le64_to_cpu(rw->slba);
|
uint64_t slba = le64_to_cpu(rw->slba);
|
||||||
uint32_t nlb = le16_to_cpu(rw->nlb) + 1;
|
uint32_t nlb = le16_to_cpu(rw->nlb) + 1;
|
||||||
uint64_t aio_slba = slba << (data_shift - BDRV_SECTOR_BITS);
|
uint64_t offset = slba << data_shift;
|
||||||
uint32_t aio_nlb = nlb << (data_shift - BDRV_SECTOR_BITS);
|
uint32_t count = nlb << data_shift;
|
||||||
|
|
||||||
if (unlikely(slba + nlb > ns->id_ns.nsze)) {
|
if (unlikely(slba + nlb > ns->id_ns.nsze)) {
|
||||||
trace_nvme_err_invalid_lba_range(slba, nlb, ns->id_ns.nsze);
|
trace_nvme_err_invalid_lba_range(slba, nlb, ns->id_ns.nsze);
|
||||||
|
@ -335,7 +335,7 @@ static uint16_t nvme_write_zeros(NvmeCtrl *n, NvmeNamespace *ns, NvmeCmd *cmd,
|
||||||
req->has_sg = false;
|
req->has_sg = false;
|
||||||
block_acct_start(blk_get_stats(n->conf.blk), &req->acct, 0,
|
block_acct_start(blk_get_stats(n->conf.blk), &req->acct, 0,
|
||||||
BLOCK_ACCT_WRITE);
|
BLOCK_ACCT_WRITE);
|
||||||
req->aiocb = blk_aio_pwrite_zeroes(n->conf.blk, aio_slba, aio_nlb,
|
req->aiocb = blk_aio_pwrite_zeroes(n->conf.blk, offset, count,
|
||||||
BDRV_REQ_MAY_UNMAP, nvme_rw_cb, req);
|
BDRV_REQ_MAY_UNMAP, nvme_rw_cb, req);
|
||||||
return NVME_NO_COMPLETE;
|
return NVME_NO_COMPLETE;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue