[PATCH 04/11] nvme: remove the unused dma_addr_t arguments to nvme_{get, set}_features
Christoph Hellwig
hch at lst.de
Sat May 9 08:53:22 PDT 2015
On Fri, May 08, 2015 at 03:03:51PM -0400, Matthew Wilcox wrote:
> > int nvme_get_features(struct nvme_dev *dev, unsigned fid, unsigned nsid,
> > - dma_addr_t dma_addr, u32 *result)
> > + u32 *result)
> > {
> > struct nvme_command c;
> >
> > memset(&c, 0, sizeof(c));
> > c.features.opcode = nvme_admin_get_features;
> > c.features.nsid = cpu_to_le32(nsid);
> > - c.features.prp1 = cpu_to_le64(dma_addr);
> > c.features.fid = cpu_to_le32(fid);
> >
> > return __nvme_submit_sync_cmd(dev->admin_q, &c, result, 0);
>
> Wait, what? The PRP1 field is used for LBA Range Type and Autonomous
> Power State Transitions.
None of which is, or ever was issued by the Linux driver.
More information about the Linux-nvme
mailing list