diff options
| author | Lei Rao <lei.rao@intel.com> | 2022-11-29 12:48:11 +0300 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2022-11-30 16:34:17 +0300 |
| commit | a56ea6147facce4ac1fc38675455f9733d96232b (patch) | |
| tree | 385330cd65bbab17d9f89cbe0929ba97e6f1d60e /drivers/nvme/host/pci.c | |
| parent | 7d4a93176e0142ce16d23c849a47d5b00b856296 (diff) | |
| download | linux-a56ea6147facce4ac1fc38675455f9733d96232b.tar.xz | |
nvme-pci: clear the prp2 field when not used
If the prp2 field is not filled in nvme_setup_prp_simple(), the prp2
field is garbage data. According to nvme spec, the prp2 is reserved if
the data transfer does not cross a memory page boundary, so clear it to
zero if it is not used.
Signed-off-by: Lei Rao <lei.rao@intel.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/nvme/host/pci.c')
| -rw-r--r-- | drivers/nvme/host/pci.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c index f4335519399d..488ad7dabeb8 100644 --- a/drivers/nvme/host/pci.c +++ b/drivers/nvme/host/pci.c @@ -797,6 +797,8 @@ static blk_status_t nvme_setup_prp_simple(struct nvme_dev *dev, cmnd->dptr.prp1 = cpu_to_le64(iod->first_dma); if (bv->bv_len > first_prp_len) cmnd->dptr.prp2 = cpu_to_le64(iod->first_dma + first_prp_len); + else + cmnd->dptr.prp2 = 0; return BLK_STS_OK; } |