lightnvm: use end_io callback instead of instance
authorMatias Bjørling <matias@cnexlabs.com>
Tue, 31 Jan 2017 12:17:17 +0000 (13:17 +0100)
committerJens Axboe <axboe@fb.com>
Tue, 31 Jan 2017 15:32:13 +0000 (08:32 -0700)
When the lightnvm core had the "gennvm" layer between the device and the
target, there was a need for the core to be able to figure out which
target it should send an end_io callback to. Leading to a "double"
end_io, first for the media manager instance, and then for the target
instance. Now that core and gennvm is merged, there is no longer a need
for this, and a single end_io callback will do.

Signed-off-by: Matias Bjørling <matias@cnexlabs.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
drivers/block/null_blk.c
drivers/lightnvm/core.c
drivers/lightnvm/rrpc.c
drivers/lightnvm/rrpc.h
drivers/nvme/host/lightnvm.c
include/linux/lightnvm.h

index e666095278ab0cd72cff4b366d85ed119e160b2d..a67b7ea1e3bfb06fff6f79a67320fe360ad0d936 100644 (file)
@@ -420,7 +420,8 @@ static void null_lnvm_end_io(struct request *rq, int error)
 {
        struct nvm_rq *rqd = rq->end_io_data;
 
-       nvm_end_io(rqd, error);
+       rqd->error = error;
+       nvm_end_io(rqd);
 
        blk_put_request(rq);
 }
index 80cd7677762d9d245cbced8d04810293a332ab11..4f4db991c4a6dd3443284628d01f880150b1aed5 100644 (file)
@@ -773,17 +773,16 @@ void nvm_free_rqd_ppalist(struct nvm_dev *dev, struct nvm_rq *rqd)
 }
 EXPORT_SYMBOL(nvm_free_rqd_ppalist);
 
-void nvm_end_io(struct nvm_rq *rqd, int error)
+void nvm_end_io(struct nvm_rq *rqd)
 {
        struct nvm_tgt_dev *tgt_dev = rqd->dev;
-       struct nvm_tgt_instance *ins = rqd->ins;
 
        /* Convert address space */
        if (tgt_dev)
                nvm_rq_dev_to_tgt(tgt_dev, rqd);
 
-       rqd->error = error;
-       ins->tt->end_io(rqd);
+       if (rqd->end_io)
+               rqd->end_io(rqd);
 }
 EXPORT_SYMBOL(nvm_end_io);
 
index 9fb7de395915ca8e3893f16273a5e6c20d4a763d..e00b1d7b976f0e892c7ff27ce697e3eaeadd6481 100644 (file)
@@ -779,7 +779,7 @@ static void rrpc_end_io_write(struct rrpc *rrpc, struct rrpc_rq *rrqd,
 
 static void rrpc_end_io(struct nvm_rq *rqd)
 {
-       struct rrpc *rrpc = container_of(rqd->ins, struct rrpc, instance);
+       struct rrpc *rrpc = rqd->private;
        struct nvm_tgt_dev *dev = rrpc->dev;
        struct rrpc_rq *rrqd = nvm_rq_to_pdu(rqd);
        uint8_t npages = rqd->nr_ppas;
@@ -972,8 +972,9 @@ static int rrpc_submit_io(struct rrpc *rrpc, struct bio *bio,
 
        bio_get(bio);
        rqd->bio = bio;
-       rqd->ins = &rrpc->instance;
+       rqd->private = rrpc;
        rqd->nr_ppas = nr_pages;
+       rqd->end_io = rrpc_end_io;
        rrq->flags = flags;
 
        err = nvm_submit_io(dev, rqd);
@@ -1532,7 +1533,6 @@ static void *rrpc_init(struct nvm_tgt_dev *dev, struct gendisk *tdisk)
        if (!rrpc)
                return ERR_PTR(-ENOMEM);
 
-       rrpc->instance.tt = &tt_rrpc;
        rrpc->dev = dev;
        rrpc->disk = tdisk;
 
@@ -1611,7 +1611,6 @@ static struct nvm_tgt_type tt_rrpc = {
 
        .make_rq        = rrpc_make_rq,
        .capacity       = rrpc_capacity,
-       .end_io         = rrpc_end_io,
 
        .init           = rrpc_init,
        .exit           = rrpc_exit,
index 94e4d73116b2948888a7539cded74e198b412634..fdb6ff902903e3f640543d79004517d6111c5a3c 100644 (file)
@@ -102,9 +102,6 @@ struct rrpc_lun {
 };
 
 struct rrpc {
-       /* instance must be kept in top to resolve rrpc in unprep */
-       struct nvm_tgt_instance instance;
-
        struct nvm_tgt_dev *dev;
        struct gendisk *disk;
 
index 3b6cd9bdba7eb4420dbf1bf39ff326d57b85bb24..21cac8523bd8e335434ba4bdf73c61e8cdb34f9d 100644 (file)
@@ -484,7 +484,8 @@ static void nvme_nvm_end_io(struct request *rq, int error)
        struct nvm_rq *rqd = rq->end_io_data;
 
        rqd->ppa_status = nvme_req(rq)->result.u64;
-       nvm_end_io(rqd, error);
+       rqd->error = error;
+       nvm_end_io(rqd);
 
        kfree(nvme_req(rq)->cmd);
        blk_mq_free_request(rq);
index ce0b2dac84ac6029481c19b821bec381b4947d13..17cd454f0d87e6f0164de89411d1a5440cefa0c2 100644 (file)
@@ -213,10 +213,6 @@ struct nvm_target {
        struct gendisk *disk;
 };
 
-struct nvm_tgt_instance {
-       struct nvm_tgt_type *tt;
-};
-
 #define ADDR_EMPTY (~0ULL)
 
 #define NVM_VERSION_MAJOR 1
@@ -227,7 +223,6 @@ struct nvm_rq;
 typedef void (nvm_end_io_fn)(struct nvm_rq *);
 
 struct nvm_rq {
-       struct nvm_tgt_instance *ins;
        struct nvm_tgt_dev *dev;
 
        struct bio *bio;
@@ -251,6 +246,8 @@ struct nvm_rq {
 
        u64 ppa_status; /* ppa media status */
        int error;
+
+       void *private;
 };
 
 static inline struct nvm_rq *nvm_rq_from_pdu(void *pdu)
@@ -450,7 +447,6 @@ struct nvm_tgt_type {
        /* target entry points */
        nvm_tgt_make_rq_fn *make_rq;
        nvm_tgt_capacity_fn *capacity;
-       nvm_end_io_fn *end_io;
 
        /* module-specific init/teardown */
        nvm_tgt_init_fn *init;
@@ -484,7 +480,7 @@ extern int nvm_get_l2p_tbl(struct nvm_tgt_dev *, u64, u32, nvm_l2p_update_fn *,
                           void *);
 extern int nvm_get_area(struct nvm_tgt_dev *, sector_t *, sector_t);
 extern void nvm_put_area(struct nvm_tgt_dev *, sector_t);
-extern void nvm_end_io(struct nvm_rq *, int);
+extern void nvm_end_io(struct nvm_rq *);
 extern int nvm_bb_tbl_fold(struct nvm_dev *, u8 *, int);
 extern int nvm_get_tgt_bb_tbl(struct nvm_tgt_dev *, struct ppa_addr, u8 *);