From 0afeb373b8404636770904da99d753a7d8d88eed Mon Sep 17 00:00:00 2001 From: Ian Moffett Date: Mon, 8 Apr 2024 23:03:00 -0400 Subject: kernel: nvme: Add drive read/write support Signed-off-by: Ian Moffett --- sys/dev/ic/nvme.c | 77 +++++++++++++++++++++++++++++++++++++++++++- sys/include/dev/ic/nvmevar.h | 4 +++ 2 files changed, 80 insertions(+), 1 deletion(-) (limited to 'sys') diff --git a/sys/dev/ic/nvme.c b/sys/dev/ic/nvme.c index 4a25735..55fbd88 100644 --- a/sys/dev/ic/nvme.c +++ b/sys/dev/ic/nvme.c @@ -31,6 +31,7 @@ #include #include #include +#include #include #include #include @@ -279,6 +280,73 @@ nvme_identify(struct nvme_state *state, struct nvme_id *id) return nvme_poll_submit_cmd(&state->adminq, cmd); } +/* + * Issue a read/write command for a specific + * namespace. + * + * `buf' must be 4k aligned. + */ +static int +nvme_rw(struct nvme_ns *ns, char *buf, off_t slba, size_t count, bool write) +{ + struct nvme_cmd cmd = {0}; + struct nvme_rw_cmd *rw = &cmd.rw; + + if (!is_4k_aligned(buf)) { + return -1; + } + + rw->opcode = write ? NVME_OP_WRITE : NVME_OP_READ; + rw->nsid = ns->nsid; + rw->slba = slba; + rw->len = count - 1; + rw->prp1 = VIRT_TO_PHYS(buf); + return nvme_poll_submit_cmd(&ns->ioq, cmd); +} + +static struct nvme_ns * +nvme_get_ns(size_t nsid) +{ + struct nvme_ns *ns; + + TAILQ_FOREACH(ns, &namespaces, link) { + if (ns->nsid == nsid) { + return ns; + } + } + + return NULL; +} + +static int +nvme_dev_rw(struct device *dev, struct sio_txn *sio, bool write) +{ + struct nvme_ns *ns; + + if (sio == NULL) { + return -1; + } + + ns = nvme_get_ns(dev->minor); + if (ns == NULL || sio->buf == NULL) { + return -1; + } + + return nvme_rw(ns, sio->buf, sio->offset, sio->len, write); +} + +static int +nvme_dev_read(struct device *dev, struct sio_txn *sio) +{ + return nvme_dev_rw(dev, sio, false); +} + +static int +nvme_dev_write(struct device *dev, struct sio_txn *sio) +{ + return nvme_dev_rw(dev, sio, true); +} + /* * Get identify data for namespace * @@ -314,6 +382,7 @@ nvme_init_ns(struct nvme_state *state, uint16_t nsid) { struct nvme_ns *ns = NULL; struct nvme_id_ns *id_ns = NULL; + struct device *dev; uint8_t lba_format; int status = 0; @@ -335,8 +404,13 @@ nvme_init_ns(struct nvme_state *state, uint16_t nsid) ns->lba_bsize = 1 << ns->lba_fmt.ds; ns->size = id_ns->size; ns->cntl = state; - nvme_create_ioq(ns, ns->nsid); + + dev = DEVICE_ALLOC(); + dev->read = nvme_dev_read; + dev->write = nvme_dev_write; + ns->dev_id = create_dev(dev, state->major, nsid); + TAILQ_INSERT_TAIL(&namespaces, ns, link); done: if (id_ns != NULL) @@ -481,6 +555,7 @@ nvme_init_controller(struct nvme_bar *bar) bar->asq = VIRT_TO_PHYS(adminq->sq); bar->acq = VIRT_TO_PHYS(adminq->cq); + state.major = device_alloc_major(); return nvme_enable_controller(&state); } diff --git a/sys/include/dev/ic/nvmevar.h b/sys/include/dev/ic/nvmevar.h index 50f5ae0..52fa5ad 100644 --- a/sys/include/dev/ic/nvmevar.h +++ b/sys/include/dev/ic/nvmevar.h @@ -38,6 +38,7 @@ #define NVME_OP_IDENTIFY 0x06 /* I/O commands */ +#define NVME_OP_WRITE 0x01 #define NVME_OP_READ 0x02 struct nvme_common_cmd { @@ -119,6 +120,7 @@ struct nvme_cmd { struct nvme_common_cmd common; struct nvme_create_iocq_cmd create_iocq; struct nvme_create_iosq_cmd create_iosq; + struct nvme_rw_cmd rw; }; }; @@ -243,6 +245,7 @@ struct nvme_queue { struct nvme_state { struct nvme_queue adminq; struct nvme_bar *bar; + dev_t major; }; /* NVMe namespace */ @@ -253,6 +256,7 @@ struct nvme_ns { struct nvme_queue ioq; /* I/O queue */ struct nvme_lbaf lba_fmt; /* LBA format */ struct nvme_state *cntl; /* NVMe controller */ + dev_t dev_id; TAILQ_ENTRY(nvme_ns) link; }; -- cgit v1.2.3