scsi: cxlflash: Update cxl-specific arguments to generic cookie
authorUma Krishnan <ukrishn@linux.vnet.ibm.com>
Wed, 3 Jan 2018 22:54:25 +0000 (16:54 -0600)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 11 Jan 2018 04:24:56 +0000 (23:24 -0500)
Convert cxl-specific pointers to generic cookies to facilitate future
enhancements.

Signed-off-by: Uma Krishnan <ukrishn@linux.vnet.ibm.com>
Acked-by: Matthew R. Ochs <mrochs@linux.vnet.ibm.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/cxlflash/common.h
drivers/scsi/cxlflash/main.c
drivers/scsi/cxlflash/superpipe.c
drivers/scsi/cxlflash/superpipe.h

index 6d95e8e147e0823741be9b9cc68ed1f4188b4a5c..d2a180de7e6c6db8fe580337f31e8b2b1e302419 100644 (file)
@@ -129,7 +129,7 @@ struct cxlflash_cfg {
        int lr_port;
        atomic_t scan_host_needed;
 
-       struct cxl_afu *cxl_afu;
+       void *afu_cookie;
 
        atomic_t recovery_threads;
        struct mutex ctx_recovery_mutex;
@@ -203,7 +203,7 @@ struct hwq {
         * fields after this point
         */
        struct afu *afu;
-       struct cxl_context *ctx;
+       void *ctx_cookie;
        struct cxl_ioctl_start_work work;
        struct sisl_host_map __iomem *host_map;         /* MC host map */
        struct sisl_ctrl_map __iomem *ctrl_map;         /* MC control map */
index 48d366304582c244512c8d28c4a199ef1868f84a..3880d522368fc95ff87d65bb1298dcadb4896dd7 100644 (file)
@@ -739,7 +739,7 @@ static void term_intr(struct cxlflash_cfg *cfg, enum undo_level level,
 
        hwq = get_hwq(afu, index);
 
-       if (!hwq->ctx) {
+       if (!hwq->ctx_cookie) {
                dev_err(dev, "%s: returning with NULL MC\n", __func__);
                return;
        }
@@ -748,13 +748,13 @@ static void term_intr(struct cxlflash_cfg *cfg, enum undo_level level,
        case UNMAP_THREE:
                /* SISL_MSI_ASYNC_ERROR is setup only for the primary HWQ */
                if (index == PRIMARY_HWQ)
-                       cxl_unmap_afu_irq(hwq->ctx, 3, hwq);
+                       cxl_unmap_afu_irq(hwq->ctx_cookie, 3, hwq);
        case UNMAP_TWO:
-               cxl_unmap_afu_irq(hwq->ctx, 2, hwq);
+               cxl_unmap_afu_irq(hwq->ctx_cookie, 2, hwq);
        case UNMAP_ONE:
-               cxl_unmap_afu_irq(hwq->ctx, 1, hwq);
+               cxl_unmap_afu_irq(hwq->ctx_cookie, 1, hwq);
        case FREE_IRQ:
-               cxl_free_afu_irqs(hwq->ctx);
+               cxl_free_afu_irqs(hwq->ctx_cookie);
                /* fall through */
        case UNDO_NOOP:
                /* No action required */
@@ -783,15 +783,15 @@ static void term_mc(struct cxlflash_cfg *cfg, u32 index)
 
        hwq = get_hwq(afu, index);
 
-       if (!hwq->ctx) {
+       if (!hwq->ctx_cookie) {
                dev_err(dev, "%s: returning with NULL MC\n", __func__);
                return;
        }
 
-       WARN_ON(cxl_stop_context(hwq->ctx));
+       WARN_ON(cxl_stop_context(hwq->ctx_cookie));
        if (index != PRIMARY_HWQ)
-               WARN_ON(cxl_release_context(hwq->ctx));
-       hwq->ctx = NULL;
+               WARN_ON(cxl_release_context(hwq->ctx_cookie));
+       hwq->ctx_cookie = NULL;
 
        spin_lock_irqsave(&hwq->hsq_slock, lock_flags);
        flush_pending_cmds(hwq);
@@ -1611,7 +1611,7 @@ static int start_context(struct cxlflash_cfg *cfg, u32 index)
        struct hwq *hwq = get_hwq(cfg->afu, index);
        int rc = 0;
 
-       rc = cxl_start_context(hwq->ctx,
+       rc = cxl_start_context(hwq->ctx_cookie,
                               hwq->work.work_element_descriptor,
                               NULL);
 
@@ -1748,7 +1748,7 @@ static void init_pcr(struct cxlflash_cfg *cfg)
        for (i = 0; i < afu->num_hwqs; i++) {
                hwq = get_hwq(afu, i);
 
-               hwq->ctx_hndl = (u16) cxl_process_element(hwq->ctx);
+               hwq->ctx_hndl = (u16) cxl_process_element(hwq->ctx_cookie);
                hwq->host_map = &afu->afu_map->hosts[hwq->ctx_hndl].host;
                hwq->ctrl_map = &afu->afu_map->ctrls[hwq->ctx_hndl].ctrl;
 
@@ -1926,7 +1926,7 @@ static enum undo_level init_intr(struct cxlflash_cfg *cfg,
                                 struct hwq *hwq)
 {
        struct device *dev = &cfg->dev->dev;
-       struct cxl_context *ctx = hwq->ctx;
+       void *ctx = hwq->ctx_cookie;
        int rc = 0;
        enum undo_level level = UNDO_NOOP;
        bool is_primary_hwq = (hwq->index == PRIMARY_HWQ);
@@ -1980,7 +1980,7 @@ out:
  */
 static int init_mc(struct cxlflash_cfg *cfg, u32 index)
 {
-       struct cxl_context *ctx;
+       void *ctx;
        struct device *dev = &cfg->dev->dev;
        struct hwq *hwq = get_hwq(cfg->afu, index);
        int rc = 0;
@@ -1999,8 +1999,8 @@ static int init_mc(struct cxlflash_cfg *cfg, u32 index)
                goto err1;
        }
 
-       WARN_ON(hwq->ctx);
-       hwq->ctx = ctx;
+       WARN_ON(hwq->ctx_cookie);
+       hwq->ctx_cookie = ctx;
 
        /* Set it up as a master with the CXL */
        cxl_set_master(ctx);
@@ -2040,7 +2040,7 @@ err2:
        if (index != PRIMARY_HWQ)
                cxl_release_context(ctx);
 err1:
-       hwq->ctx = NULL;
+       hwq->ctx_cookie = NULL;
        goto out;
 }
 
@@ -2095,7 +2095,7 @@ static int init_afu(struct cxlflash_cfg *cfg)
        struct hwq *hwq;
        int i;
 
-       cxl_perst_reloads_same_image(cfg->cxl_afu, true);
+       cxl_perst_reloads_same_image(cfg->afu_cookie, true);
 
        afu->num_hwqs = afu->desired_hwqs;
        for (i = 0; i < afu->num_hwqs; i++) {
@@ -2109,7 +2109,7 @@ static int init_afu(struct cxlflash_cfg *cfg)
 
        /* Map the entire MMIO space of the AFU using the first context */
        hwq = get_hwq(afu, PRIMARY_HWQ);
-       afu->afu_map = cxl_psa_map(hwq->ctx);
+       afu->afu_map = cxl_psa_map(hwq->ctx_cookie);
        if (!afu->afu_map) {
                dev_err(dev, "%s: cxl_psa_map failed\n", __func__);
                rc = -ENOMEM;
@@ -3702,7 +3702,7 @@ static int cxlflash_probe(struct pci_dev *pdev,
 
        pci_set_drvdata(pdev, cfg);
 
-       cfg->cxl_afu = cxl_pci_to_afu(pdev);
+       cfg->afu_cookie = cxl_pci_to_afu(pdev);
 
        rc = init_pci(cfg);
        if (rc) {
index 170fff5aeff6790f68d44103efc594e62cb4a949..18f62407691405cbe0dbb258b2f720f282c5a8f2 100644 (file)
@@ -810,14 +810,13 @@ err:
  * init_context() - initializes a previously allocated context
  * @ctxi:      Previously allocated context
  * @cfg:       Internal structure associated with the host.
- * @ctx:       Previously obtained CXL context reference.
+ * @ctx:       Previously obtained context cookie.
  * @ctxid:     Previously obtained process element associated with CXL context.
  * @file:      Previously obtained file associated with CXL context.
  * @perms:     User-specified permissions.
  */
 static void init_context(struct ctx_info *ctxi, struct cxlflash_cfg *cfg,
-                        struct cxl_context *ctx, int ctxid, struct file *file,
-                        u32 perms)
+                        void *ctx, int ctxid, struct file *file, u32 perms)
 {
        struct afu *afu = cfg->afu;
 
@@ -976,7 +975,7 @@ static int cxlflash_disk_detach(struct scsi_device *sdev,
  */
 static int cxlflash_cxl_release(struct inode *inode, struct file *file)
 {
-       struct cxl_context *ctx = cxl_fops_get_context(file);
+       void *ctx = cxl_fops_get_context(file);
        struct cxlflash_cfg *cfg = container_of(file->f_op, struct cxlflash_cfg,
                                                cxl_fops);
        struct device *dev = &cfg->dev->dev;
@@ -1089,7 +1088,7 @@ static int cxlflash_mmap_fault(struct vm_fault *vmf)
 {
        struct vm_area_struct *vma = vmf->vma;
        struct file *file = vma->vm_file;
-       struct cxl_context *ctx = cxl_fops_get_context(file);
+       void *ctx = cxl_fops_get_context(file);
        struct cxlflash_cfg *cfg = container_of(file->f_op, struct cxlflash_cfg,
                                                cxl_fops);
        struct device *dev = &cfg->dev->dev;
@@ -1162,7 +1161,7 @@ static const struct vm_operations_struct cxlflash_mmap_vmops = {
  */
 static int cxlflash_cxl_mmap(struct file *file, struct vm_area_struct *vma)
 {
-       struct cxl_context *ctx = cxl_fops_get_context(file);
+       void *ctx = cxl_fops_get_context(file);
        struct cxlflash_cfg *cfg = container_of(file->f_op, struct cxlflash_cfg,
                                                cxl_fops);
        struct device *dev = &cfg->dev->dev;
@@ -1317,7 +1316,7 @@ static int cxlflash_disk_attach(struct scsi_device *sdev,
        u64 rctxid = 0UL;
        struct file *file = NULL;
 
-       struct cxl_context *ctx = NULL;
+       void *ctx = NULL;
 
        int fd = -1;
 
@@ -1529,7 +1528,7 @@ static int recover_context(struct cxlflash_cfg *cfg,
        int fd = -1;
        int ctxid = -1;
        struct file *file;
-       struct cxl_context *ctx;
+       void *ctx;
        struct afu *afu = cfg->afu;
 
        ctx = cxl_dev_context_init(cfg->dev);
index 0b597682991375cb6d48ffefbd0e1cc4cd341619..62097df435e2d07d52850a2e484c93941d86b8fd 100644 (file)
@@ -104,7 +104,7 @@ struct ctx_info {
        bool err_recovery_active;
        struct mutex mutex; /* Context protection */
        struct kref kref;
-       struct cxl_context *ctx;
+       void *ctx;
        struct cxlflash_cfg *cfg;
        struct list_head luns;  /* LUNs attached to this context */
        const struct vm_operations_struct *cxl_mmap_vmops;