From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?q?Matias=20Bj=C3=B8rling?= Subject: [PATCH v4 1/8] nvme: add special param for nvme_submit_sync_cmd Date: Fri, 5 Jun 2015 14:54:23 +0200 Message-ID: <1433508870-28251-2-git-send-email-m@bjorling.me> References: <1433508870-28251-1-git-send-email-m@bjorling.me> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: javier@lightnvm.io, Stephen.Bates@pmcs.com, keith.busch@intel.com, =?UTF-8?q?Matias=20Bj=C3=B8rling?= To: hch@infradead.org, axboe@fb.com, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-nvme@lists.infradead.org Return-path: In-Reply-To: <1433508870-28251-1-git-send-email-m@bjorling.me> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org In preparation for LightNVM, it requires a hook for internal commands to resolve its state on command completion. Use req->special for this and move the command result to req->sense. Signed-off-by: Matias Bj=C3=B8rling --- drivers/block/nvme-core.c | 19 ++++++++++--------- drivers/block/nvme-scsi.c | 4 ++-- include/linux/nvme.h | 2 +- 3 files changed, 13 insertions(+), 12 deletions(-) diff --git a/drivers/block/nvme-core.c b/drivers/block/nvme-core.c index 6ed1356..d2955fe 100644 --- a/drivers/block/nvme-core.c +++ b/drivers/block/nvme-core.c @@ -614,7 +614,7 @@ static void req_completion(struct nvme_queue *nvmeq= , void *ctx, req->errors =3D 0; if (req->cmd_type =3D=3D REQ_TYPE_DRV_PRIV) { u32 result =3D le32_to_cpup(&cqe->result); - req->special =3D (void *)(uintptr_t)result; + req->sense =3D (void *)(uintptr_t)result; } =20 if (cmd_rq->aborted) @@ -998,7 +998,7 @@ static irqreturn_t nvme_irq_check(int irq, void *da= ta) */ int __nvme_submit_sync_cmd(struct request_queue *q, struct nvme_comman= d *cmd, void *buffer, void __user *ubuffer, unsigned bufflen, - u32 *result, unsigned timeout) + u32 *result, unsigned timeout, void *special) { bool write =3D cmd->common.opcode & 1; struct bio *bio =3D NULL; @@ -1019,7 +1019,7 @@ int __nvme_submit_sync_cmd(struct request_queue *= q, struct nvme_command *cmd, =20 req->cmd =3D (unsigned char *)cmd; req->cmd_len =3D sizeof(struct nvme_command); - req->special =3D (void *)0; + req->special =3D special; =20 if (buffer && bufflen) { ret =3D blk_rq_map_kern(q, req, buffer, bufflen, __GFP_WAIT); @@ -1036,7 +1036,7 @@ int __nvme_submit_sync_cmd(struct request_queue *= q, struct nvme_command *cmd, if (bio) blk_rq_unmap_user(bio); if (result) - *result =3D (u32)(uintptr_t)req->special; + *result =3D (u32)(uintptr_t)req->sense; ret =3D req->errors; out: blk_mq_free_request(req); @@ -1046,7 +1046,8 @@ int __nvme_submit_sync_cmd(struct request_queue *= q, struct nvme_command *cmd, int nvme_submit_sync_cmd(struct request_queue *q, struct nvme_command = *cmd, void *buffer, unsigned bufflen) { - return __nvme_submit_sync_cmd(q, cmd, buffer, NULL, bufflen, NULL, 0)= ; + return __nvme_submit_sync_cmd(q, cmd, buffer, NULL, bufflen, NULL, 0, + NULL); } =20 static int nvme_submit_async_admin_req(struct nvme_dev *dev) @@ -1209,7 +1210,7 @@ int nvme_get_features(struct nvme_dev *dev, unsig= ned fid, unsigned nsid, c.features.fid =3D cpu_to_le32(fid); =20 return __nvme_submit_sync_cmd(dev->admin_q, &c, NULL, NULL, 0, - result, 0); + result, 0, NULL); } =20 int nvme_set_features(struct nvme_dev *dev, unsigned fid, unsigned dwo= rd11, @@ -1224,7 +1225,7 @@ int nvme_set_features(struct nvme_dev *dev, unsig= ned fid, unsigned dword11, c.features.dword11 =3D cpu_to_le32(dword11); =20 return __nvme_submit_sync_cmd(dev->admin_q, &c, NULL, NULL, 0, - result, 0); + result, 0, NULL); } =20 int nvme_get_log_page(struct nvme_dev *dev, struct nvme_smart_log **lo= g) @@ -1787,7 +1788,7 @@ static int nvme_submit_io(struct nvme_ns *ns, str= uct nvme_user_io __user *uio) c.rw.metadata =3D cpu_to_le64(meta_dma); =20 status =3D __nvme_submit_sync_cmd(ns->queue, &c, NULL, - (void __user *)io.addr, length, NULL, 0); + (void __user *)io.addr, length, NULL, 0, NULL); unmap: if (meta) { if (status =3D=3D NVME_SC_SUCCESS && !write) { @@ -1831,7 +1832,7 @@ static int nvme_user_cmd(struct nvme_dev *dev, st= ruct nvme_ns *ns, =20 status =3D __nvme_submit_sync_cmd(ns ? ns->queue : dev->admin_q, &c, NULL, (void __user *)cmd.addr, cmd.data_len, - &cmd.result, timeout); + &cmd.result, timeout, NULL); if (status >=3D 0) { if (put_user(cmd.result, &ucmd->result)) return -EFAULT; diff --git a/drivers/block/nvme-scsi.c b/drivers/block/nvme-scsi.c index 8e6223e..ad35bb7 100644 --- a/drivers/block/nvme-scsi.c +++ b/drivers/block/nvme-scsi.c @@ -1297,7 +1297,7 @@ static int nvme_trans_send_download_fw_cmd(struct= nvme_ns *ns, struct sg_io_hdr c.dlfw.offset =3D cpu_to_le32(offset/BYTES_TO_DWORDS); =20 nvme_sc =3D __nvme_submit_sync_cmd(dev->admin_q, &c, NULL, - hdr->dxferp, tot_len, NULL, 0); + hdr->dxferp, tot_len, NULL, 0, NULL); return nvme_trans_status_code(hdr, nvme_sc); } =20 @@ -1704,7 +1704,7 @@ static int nvme_trans_do_nvme_io(struct nvme_ns *= ns, struct sg_io_hdr *hdr, break; } nvme_sc =3D __nvme_submit_sync_cmd(ns->queue, &c, NULL, - next_mapping_addr, unit_len, NULL, 0); + next_mapping_addr, unit_len, NULL, 0, NULL); if (nvme_sc) break; =20 diff --git a/include/linux/nvme.h b/include/linux/nvme.h index 986bf8a..fce2090 100644 --- a/include/linux/nvme.h +++ b/include/linux/nvme.h @@ -150,7 +150,7 @@ int nvme_submit_sync_cmd(struct request_queue *q, s= truct nvme_command *cmd, void *buf, unsigned bufflen); int __nvme_submit_sync_cmd(struct request_queue *q, struct nvme_comman= d *cmd, void *buffer, void __user *ubuffer, unsigned bufflen, - u32 *result, unsigned timeout); + u32 *result, unsigned timeout, void *special); int nvme_identify_ctrl(struct nvme_dev *dev, struct nvme_id_ctrl **id)= ; int nvme_identify_ns(struct nvme_dev *dev, unsigned nsid, struct nvme_id_ns **id); --=20 2.1.4