]> git.proxmox.com Git - mirror_ubuntu-zesty-kernel.git/blobdiff - drivers/scsi/cxlflash/main.c
scsi: cxlflash: Separate AFU internal command handling from AFU sync specifics
[mirror_ubuntu-zesty-kernel.git] / drivers / scsi / cxlflash / main.c
index 113797aafd7e966117d029287fecad9c957a35b7..7732dfc099c42d29745d0ab79d6f9aa1a24f21d4 100644 (file)
@@ -34,6 +34,10 @@ MODULE_AUTHOR("Manoj N. Kumar <manoj@linux.vnet.ibm.com>");
 MODULE_AUTHOR("Matthew R. Ochs <mrochs@linux.vnet.ibm.com>");
 MODULE_LICENSE("GPL");
 
+static struct class *cxlflash_class;
+static u32 cxlflash_major;
+static DECLARE_BITMAP(cxlflash_minor, CXLFLASH_MAX_ADAPTERS);
+
 /**
  * process_cmd_err() - command error handler
  * @cmd:       AFU command that experienced the error.
@@ -162,8 +166,13 @@ static void cmd_complete(struct afu_cmd *cmd)
        struct afu *afu = cmd->parent;
        struct cxlflash_cfg *cfg = afu->parent;
        struct device *dev = &cfg->dev->dev;
+       struct hwq *hwq = get_hwq(afu, cmd->hwq_index);
        bool cmd_is_tmf;
 
+       spin_lock_irqsave(&hwq->hsq_slock, lock_flags);
+       list_del(&cmd->list);
+       spin_unlock_irqrestore(&hwq->hsq_slock, lock_flags);
+
        if (cmd->scp) {
                scp = cmd->scp;
                if (unlikely(cmd->sa.ioasc))
@@ -189,55 +198,101 @@ static void cmd_complete(struct afu_cmd *cmd)
 }
 
 /**
- * context_reset() - reset command owner context via specified register
- * @cmd:       AFU command that timed out.
+ * flush_pending_cmds() - flush all pending commands on this hardware queue
+ * @hwq:       Hardware queue to flush.
+ *
+ * The hardware send queue lock associated with this hardware queue must be
+ * held when calling this routine.
+ */
+static void flush_pending_cmds(struct hwq *hwq)
+{
+       struct afu_cmd *cmd, *tmp;
+       struct scsi_cmnd *scp;
+
+       list_for_each_entry_safe(cmd, tmp, &hwq->pending_cmds, list) {
+               /* Bypass command when on a doneq, cmd_complete() will handle */
+               if (!list_empty(&cmd->queue))
+                       continue;
+
+               list_del(&cmd->list);
+
+               if (cmd->scp) {
+                       scp = cmd->scp;
+                       scp->result = (DID_IMM_RETRY << 16);
+                       scp->scsi_done(scp);
+               } else {
+                       cmd->cmd_aborted = true;
+                       complete(&cmd->cevent);
+               }
+       }
+}
+
+/**
+ * context_reset() - reset context via specified register
+ * @hwq:       Hardware queue owning the context to be reset.
  * @reset_reg: MMIO register to perform reset.
+ *
+ * When the reset is successful, the SISLite specification guarantees that
+ * the AFU has aborted all currently pending I/O. Accordingly, these commands
+ * must be flushed.
+ *
+ * Return: 0 on success, -errno on failure
  */
-static void context_reset(struct afu_cmd *cmd, __be64 __iomem *reset_reg)
+static int context_reset(struct hwq *hwq, __be64 __iomem *reset_reg)
 {
-       int nretry = 0;
-       u64 rrin = 0x1;
-       struct afu *afu = cmd->parent;
-       struct cxlflash_cfg *cfg = afu->parent;
+       struct cxlflash_cfg *cfg = hwq->afu->parent;
        struct device *dev = &cfg->dev->dev;
+       int rc = -ETIMEDOUT;
+       int nretry = 0;
+       u64 val = 0x1;
+       ulong lock_flags;
 
-       dev_dbg(dev, "%s: cmd=%p\n", __func__, cmd);
+       dev_dbg(dev, "%s: hwq=%p\n", __func__, hwq);
 
-       writeq_be(rrin, reset_reg);
+       spin_lock_irqsave(&hwq->hsq_slock, lock_flags);
+
+       writeq_be(val, reset_reg);
        do {
-               rrin = readq_be(reset_reg);
-               if (rrin != 0x1)
+               val = readq_be(reset_reg);
+               if ((val & 0x1) == 0x0) {
+                       rc = 0;
                        break;
+               }
+
                /* Double delay each time */
                udelay(1 << nretry);
        } while (nretry++ < MC_ROOM_RETRY_CNT);
 
-       dev_dbg(dev, "%s: returning rrin=%016llx nretry=%d\n",
-               __func__, rrin, nretry);
+       if (!rc)
+               flush_pending_cmds(hwq);
+
+       spin_unlock_irqrestore(&hwq->hsq_slock, lock_flags);
+
+       dev_dbg(dev, "%s: returning rc=%d, val=%016llx nretry=%d\n",
+               __func__, rc, val, nretry);
+       return rc;
 }
 
 /**
- * context_reset_ioarrin() - reset command owner context via IOARRIN register
- * @cmd:       AFU command that timed out.
+ * context_reset_ioarrin() - reset context via IOARRIN register
+ * @hwq:       Hardware queue owning the context to be reset.
+ *
+ * Return: 0 on success, -errno on failure
  */
-static void context_reset_ioarrin(struct afu_cmd *cmd)
+static int context_reset_ioarrin(struct hwq *hwq)
 {
-       struct afu *afu = cmd->parent;
-       struct hwq *hwq = get_hwq(afu, cmd->hwq_index);
-
-       context_reset(cmd, &hwq->host_map->ioarrin);
+       return context_reset(hwq, &hwq->host_map->ioarrin);
 }
 
 /**
- * context_reset_sq() - reset command owner context w/ SQ Context Reset register
- * @cmd:       AFU command that timed out.
+ * context_reset_sq() - reset context via SQ_CONTEXT_RESET register
+ * @hwq:       Hardware queue owning the context to be reset.
+ *
+ * Return: 0 on success, -errno on failure
  */
-static void context_reset_sq(struct afu_cmd *cmd)
+static int context_reset_sq(struct hwq *hwq)
 {
-       struct afu *afu = cmd->parent;
-       struct hwq *hwq = get_hwq(afu, cmd->hwq_index);
-
-       context_reset(cmd, &hwq->host_map->sq_ctx_reset);
+       return context_reset(hwq, &hwq->host_map->sq_ctx_reset);
 }
 
 /**
@@ -261,7 +316,7 @@ static int send_cmd_ioarrin(struct afu *afu, struct afu_cmd *cmd)
         * To avoid the performance penalty of MMIO, spread the update of
         * 'room' over multiple commands.
         */
-       spin_lock_irqsave(&hwq->rrin_slock, lock_flags);
+       spin_lock_irqsave(&hwq->hsq_slock, lock_flags);
        if (--hwq->room < 0) {
                room = readq_be(&hwq->host_map->cmd_room);
                if (room <= 0) {
@@ -275,9 +330,10 @@ static int send_cmd_ioarrin(struct afu *afu, struct afu_cmd *cmd)
                hwq->room = room - 1;
        }
 
+       list_add(&cmd->list, &hwq->pending_cmds);
        writeq_be((u64)&cmd->rcb, &hwq->host_map->ioarrin);
 out:
-       spin_unlock_irqrestore(&hwq->rrin_slock, lock_flags);
+       spin_unlock_irqrestore(&hwq->hsq_slock, lock_flags);
        dev_dbg(dev, "%s: cmd=%p len=%u ea=%016llx rc=%d\n", __func__,
                cmd, cmd->rcb.data_len, cmd->rcb.data_ea, rc);
        return rc;
@@ -315,6 +371,8 @@ static int send_cmd_sq(struct afu *afu, struct afu_cmd *cmd)
                hwq->hsq_curr++;
        else
                hwq->hsq_curr = hwq->hsq_start;
+
+       list_add(&cmd->list, &hwq->pending_cmds);
        writeq_be((u64)hwq->hsq_curr, &hwq->host_map->sq_tail);
 
        spin_unlock_irqrestore(&hwq->hsq_slock, lock_flags);
@@ -332,8 +390,7 @@ out:
  * @afu:       AFU associated with the host.
  * @cmd:       AFU command that was sent.
  *
- * Return:
- *     0 on success, -1 on timeout/error
+ * Return: 0 on success, -errno on failure
  */
 static int wait_resp(struct afu *afu, struct afu_cmd *cmd)
 {
@@ -343,20 +400,58 @@ static int wait_resp(struct afu *afu, struct afu_cmd *cmd)
        ulong timeout = msecs_to_jiffies(cmd->rcb.timeout * 2 * 1000);
 
        timeout = wait_for_completion_timeout(&cmd->cevent, timeout);
-       if (!timeout) {
-               afu->context_reset(cmd);
-               rc = -1;
-       }
+       if (!timeout)
+               rc = -ETIMEDOUT;
+
+       if (cmd->cmd_aborted)
+               rc = -EAGAIN;
 
        if (unlikely(cmd->sa.ioasc != 0)) {
                dev_err(dev, "%s: cmd %02x failed, ioasc=%08x\n",
                        __func__, cmd->rcb.cdb[0], cmd->sa.ioasc);
-               rc = -1;
+               rc = -EIO;
        }
 
        return rc;
 }
 
+/**
+ * cmd_to_target_hwq() - selects a target hardware queue for a SCSI command
+ * @host:      SCSI host associated with device.
+ * @scp:       SCSI command to send.
+ * @afu:       SCSI command to send.
+ *
+ * Hashes a command based upon the hardware queue mode.
+ *
+ * Return: Trusted index of target hardware queue
+ */
+static u32 cmd_to_target_hwq(struct Scsi_Host *host, struct scsi_cmnd *scp,
+                            struct afu *afu)
+{
+       u32 tag;
+       u32 hwq = 0;
+
+       if (afu->num_hwqs == 1)
+               return 0;
+
+       switch (afu->hwq_mode) {
+       case HWQ_MODE_RR:
+               hwq = afu->hwq_rr_count++ % afu->num_hwqs;
+               break;
+       case HWQ_MODE_TAG:
+               tag = blk_mq_unique_tag(scp->request);
+               hwq = blk_mq_unique_tag_to_hwq(tag);
+               break;
+       case HWQ_MODE_CPU:
+               hwq = smp_processor_id() % afu->num_hwqs;
+               break;
+       default:
+               WARN_ON_ONCE(1);
+       }
+
+       return hwq;
+}
+
 /**
  * send_tmf() - sends a Task Management Function (TMF)
  * @afu:       AFU to checkout from.
@@ -368,10 +463,12 @@ static int wait_resp(struct afu *afu, struct afu_cmd *cmd)
  */
 static int send_tmf(struct afu *afu, struct scsi_cmnd *scp, u64 tmfcmd)
 {
-       struct cxlflash_cfg *cfg = shost_priv(scp->device->host);
+       struct Scsi_Host *host = scp->device->host;
+       struct cxlflash_cfg *cfg = shost_priv(host);
        struct afu_cmd *cmd = sc_to_afucz(scp);
        struct device *dev = &cfg->dev->dev;
-       struct hwq *hwq = get_hwq(afu, PRIMARY_HWQ);
+       int hwq_index = cmd_to_target_hwq(host, scp, afu);
+       struct hwq *hwq = get_hwq(afu, hwq_index);
        ulong lock_flags;
        int rc = 0;
        ulong to;
@@ -388,7 +485,7 @@ static int send_tmf(struct afu *afu, struct scsi_cmnd *scp, u64 tmfcmd)
        cmd->scp = scp;
        cmd->parent = afu;
        cmd->cmd_tmf = true;
-       cmd->hwq_index = hwq->index;
+       cmd->hwq_index = hwq_index;
 
        cmd->rcb.ctx_id = hwq->ctx_hndl;
        cmd->rcb.msi = SISL_MSI_RRQ_UPDATED;
@@ -448,7 +545,8 @@ static int cxlflash_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *scp)
        struct device *dev = &cfg->dev->dev;
        struct afu_cmd *cmd = sc_to_afucz(scp);
        struct scatterlist *sg = scsi_sglist(scp);
-       struct hwq *hwq = get_hwq(afu, PRIMARY_HWQ);
+       int hwq_index = cmd_to_target_hwq(host, scp, afu);
+       struct hwq *hwq = get_hwq(afu, hwq_index);
        u16 req_flags = SISL_REQ_FLAGS_SUP_UNDERRUN;
        ulong lock_flags;
        int rc = 0;
@@ -498,7 +596,7 @@ static int cxlflash_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *scp)
 
        cmd->scp = scp;
        cmd->parent = afu;
-       cmd->hwq_index = hwq->index;
+       cmd->hwq_index = hwq_index;
 
        cmd->rcb.ctx_id = hwq->ctx_hndl;
        cmd->rcb.msi = SISL_MSI_RRQ_UPDATED;
@@ -544,6 +642,20 @@ static void free_mem(struct cxlflash_cfg *cfg)
        }
 }
 
+/**
+ * cxlflash_reset_sync() - synchronizing point for asynchronous resets
+ * @cfg:       Internal structure associated with the host.
+ */
+static void cxlflash_reset_sync(struct cxlflash_cfg *cfg)
+{
+       if (cfg->async_reset_cookie == 0)
+               return;
+
+       /* Wait until all async calls prior to this cookie have completed */
+       async_synchronize_cookie(cfg->async_reset_cookie + 1);
+       cfg->async_reset_cookie = 0;
+}
+
 /**
  * stop_afu() - stops the AFU command timers and unmaps the MMIO space
  * @cfg:       Internal structure associated with the host.
@@ -560,6 +672,8 @@ static void stop_afu(struct cxlflash_cfg *cfg)
        int i;
 
        cancel_work_sync(&cfg->work_q);
+       if (!current_is_async())
+               cxlflash_reset_sync(cfg);
 
        if (likely(afu)) {
                while (atomic_read(&afu->cmds_active))
@@ -637,6 +751,7 @@ static void term_mc(struct cxlflash_cfg *cfg, u32 index)
        struct afu *afu = cfg->afu;
        struct device *dev = &cfg->dev->dev;
        struct hwq *hwq;
+       ulong lock_flags;
 
        if (!afu) {
                dev_err(dev, "%s: returning with NULL afu\n", __func__);
@@ -654,6 +769,10 @@ static void term_mc(struct cxlflash_cfg *cfg, u32 index)
        if (index != PRIMARY_HWQ)
                WARN_ON(cxl_release_context(hwq->ctx));
        hwq->ctx = NULL;
+
+       spin_lock_irqsave(&hwq->hsq_slock, lock_flags);
+       flush_pending_cmds(hwq);
+       spin_unlock_irqrestore(&hwq->hsq_slock, lock_flags);
 }
 
 /**
@@ -747,6 +866,47 @@ static void notify_shutdown(struct cxlflash_cfg *cfg, bool wait)
        }
 }
 
+/**
+ * cxlflash_get_minor() - gets the first available minor number
+ *
+ * Return: Unique minor number that can be used to create the character device.
+ */
+static int cxlflash_get_minor(void)
+{
+       int minor;
+       long bit;
+
+       bit = find_first_zero_bit(cxlflash_minor, CXLFLASH_MAX_ADAPTERS);
+       if (bit >= CXLFLASH_MAX_ADAPTERS)
+               return -1;
+
+       minor = bit & MINORMASK;
+       set_bit(minor, cxlflash_minor);
+       return minor;
+}
+
+/**
+ * cxlflash_put_minor() - releases the minor number
+ * @minor:     Minor number that is no longer needed.
+ */
+static void cxlflash_put_minor(int minor)
+{
+       clear_bit(minor, cxlflash_minor);
+}
+
+/**
+ * cxlflash_release_chrdev() - release the character device for the host
+ * @cfg:       Internal structure associated with the host.
+ */
+static void cxlflash_release_chrdev(struct cxlflash_cfg *cfg)
+{
+       put_device(cfg->chardev);
+       device_unregister(cfg->chardev);
+       cfg->chardev = NULL;
+       cdev_del(&cfg->cdev);
+       cxlflash_put_minor(MINOR(cfg->cdev.dev));
+}
+
 /**
  * cxlflash_remove() - PCI entry point to tear down host
  * @pdev:      PCI device associated with the host.
@@ -782,6 +942,8 @@ static void cxlflash_remove(struct pci_dev *pdev)
        cxlflash_stop_term_user_contexts(cfg);
 
        switch (cfg->init_state) {
+       case INIT_STATE_CDEV:
+               cxlflash_release_chrdev(cfg);
        case INIT_STATE_SCSI:
                cxlflash_term_local_luns(cfg);
                scsi_remove_host(cfg->host);
@@ -1682,7 +1844,10 @@ static int start_afu(struct cxlflash_cfg *cfg)
                hwq->hrrq_end = &hwq->rrq_entry[NUM_RRQ_ENTRY - 1];
                hwq->hrrq_curr = hwq->hrrq_start;
                hwq->toggle = 1;
+
+               /* Initialize spin locks */
                spin_lock_init(&hwq->hrrq_slock);
+               spin_lock_init(&hwq->hsq_slock);
 
                /* Initialize SQ */
                if (afu_is_sq_cmd_mode(afu)) {
@@ -1691,7 +1856,6 @@ static int start_afu(struct cxlflash_cfg *cfg)
                        hwq->hsq_end = &hwq->sq[NUM_SQ_ENTRY - 1];
                        hwq->hsq_curr = hwq->hsq_start;
 
-                       spin_lock_init(&hwq->hsq_slock);
                        atomic_set(&hwq->hsq_credits, NUM_SQ_ENTRY - 1);
                }
 
@@ -1781,6 +1945,7 @@ static int init_mc(struct cxlflash_cfg *cfg, u32 index)
 
        hwq->afu = cfg->afu;
        hwq->index = index;
+       INIT_LIST_HEAD(&hwq->pending_cmds);
 
        if (index == PRIMARY_HWQ)
                ctx = cxl_get_context(cfg->dev);
@@ -1944,7 +2109,6 @@ static int init_afu(struct cxlflash_cfg *cfg)
        for (i = 0; i < afu->num_hwqs; i++) {
                hwq = get_hwq(afu, i);
 
-               spin_lock_init(&hwq->rrin_slock);
                hwq->room = readq_be(&hwq->host_map->cmd_room);
        }
 
@@ -1963,29 +2127,107 @@ err1:
 }
 
 /**
- * cxlflash_afu_sync() - builds and sends an AFU sync command
+ * afu_reset() - resets the AFU
+ * @cfg:       Internal structure associated with the host.
+ *
+ * Return: 0 on success, -errno on failure
+ */
+static int afu_reset(struct cxlflash_cfg *cfg)
+{
+       struct device *dev = &cfg->dev->dev;
+       int rc = 0;
+
+       /* Stop the context before the reset. Since the context is
+        * no longer available restart it after the reset is complete
+        */
+       term_afu(cfg);
+
+       rc = init_afu(cfg);
+
+       dev_dbg(dev, "%s: returning rc=%d\n", __func__, rc);
+       return rc;
+}
+
+/**
+ * drain_ioctls() - wait until all currently executing ioctls have completed
+ * @cfg:       Internal structure associated with the host.
+ *
+ * Obtain write access to read/write semaphore that wraps ioctl
+ * handling to 'drain' ioctls currently executing.
+ */
+static void drain_ioctls(struct cxlflash_cfg *cfg)
+{
+       down_write(&cfg->ioctl_rwsem);
+       up_write(&cfg->ioctl_rwsem);
+}
+
+/**
+ * cxlflash_async_reset_host() - asynchronous host reset handler
+ * @data:      Private data provided while scheduling reset.
+ * @cookie:    Cookie that can be used for checkpointing.
+ */
+static void cxlflash_async_reset_host(void *data, async_cookie_t cookie)
+{
+       struct cxlflash_cfg *cfg = data;
+       struct device *dev = &cfg->dev->dev;
+       int rc = 0;
+
+       if (cfg->state != STATE_RESET) {
+               dev_dbg(dev, "%s: Not performing a reset, state=%d\n",
+                       __func__, cfg->state);
+               goto out;
+       }
+
+       drain_ioctls(cfg);
+       cxlflash_mark_contexts_error(cfg);
+       rc = afu_reset(cfg);
+       if (rc)
+               cfg->state = STATE_FAILTERM;
+       else
+               cfg->state = STATE_NORMAL;
+       wake_up_all(&cfg->reset_waitq);
+
+out:
+       scsi_unblock_requests(cfg->host);
+}
+
+/**
+ * cxlflash_schedule_async_reset() - schedule an asynchronous host reset
+ * @cfg:       Internal structure associated with the host.
+ */
+static void cxlflash_schedule_async_reset(struct cxlflash_cfg *cfg)
+{
+       struct device *dev = &cfg->dev->dev;
+
+       if (cfg->state != STATE_NORMAL) {
+               dev_dbg(dev, "%s: Not performing reset state=%d\n",
+                       __func__, cfg->state);
+               return;
+       }
+
+       cfg->state = STATE_RESET;
+       scsi_block_requests(cfg->host);
+       cfg->async_reset_cookie = async_schedule(cxlflash_async_reset_host,
+                                                cfg);
+}
+
+/**
+ * send_afu_cmd() - builds and sends an internal AFU command
  * @afu:       AFU associated with the host.
- * @ctx_hndl_u:        Identifies context requesting sync.
- * @res_hndl_u:        Identifies resource requesting sync.
- * @mode:      Type of sync to issue (lightweight, heavyweight, global).
+ * @rcb:       Pre-populated IOARCB describing command to send.
  *
- * The AFU can only take 1 sync command at a time. This routine enforces this
- * limitation by using a mutex to provide exclusive access to the AFU during
- * the sync. This design point requires calling threads to not be on interrupt
- * context due to the possibility of sleeping during concurrent sync operations.
+ * The AFU can only take one internal AFU command at a time. This limitation is
+ * enforced by using a mutex to provide exclusive access to the AFU during the
+ * operation. This design point requires calling threads to not be on interrupt
+ * context due to the possibility of sleeping during concurrent AFU operations.
  *
- * AFU sync operations are only necessary and allowed when the device is
- * operating normally. When not operating normally, sync requests can occur as
- * part of cleaning up resources associated with an adapter prior to removal.
- * In this scenario, these requests are simply ignored (safe due to the AFU
- * going away).
+ * The command status is optionally passed back to the caller when the caller
+ * populates the IOASA field of the IOARCB with a pointer to an IOASA structure.
  *
  * Return:
- *     0 on success
- *     -1 on failure
+ *     0 on success, -errno on failure
  */
-int cxlflash_afu_sync(struct afu *afu, ctx_hndl_t ctx_hndl_u,
-                     res_hndl_t res_hndl_u, u8 mode)
+static int send_afu_cmd(struct afu *afu, struct sisl_ioarcb *rcb)
 {
        struct cxlflash_cfg *cfg = afu->parent;
        struct device *dev = &cfg->dev->dev;
@@ -1993,6 +2235,7 @@ int cxlflash_afu_sync(struct afu *afu, ctx_hndl_t ctx_hndl_u,
        struct hwq *hwq = get_hwq(afu, PRIMARY_HWQ);
        char *buf = NULL;
        int rc = 0;
+       int nretry = 0;
        static DEFINE_MUTEX(sync_active);
 
        if (cfg->state != STATE_NORMAL) {
@@ -2003,39 +2246,52 @@ int cxlflash_afu_sync(struct afu *afu, ctx_hndl_t ctx_hndl_u,
 
        mutex_lock(&sync_active);
        atomic_inc(&afu->cmds_active);
-       buf = kzalloc(sizeof(*cmd) + __alignof__(*cmd) - 1, GFP_KERNEL);
+       buf = kmalloc(sizeof(*cmd) + __alignof__(*cmd) - 1, GFP_KERNEL);
        if (unlikely(!buf)) {
                dev_err(dev, "%s: no memory for command\n", __func__);
-               rc = -1;
+               rc = -ENOMEM;
                goto out;
        }
 
        cmd = (struct afu_cmd *)PTR_ALIGN(buf, __alignof__(*cmd));
+
+retry:
+       memset(cmd, 0, sizeof(*cmd));
+       memcpy(&cmd->rcb, rcb, sizeof(*rcb));
+       INIT_LIST_HEAD(&cmd->queue);
        init_completion(&cmd->cevent);
        cmd->parent = afu;
        cmd->hwq_index = hwq->index;
-
-       dev_dbg(dev, "%s: afu=%p cmd=%p %d\n", __func__, afu, cmd, ctx_hndl_u);
-
-       cmd->rcb.req_flags = SISL_REQ_FLAGS_AFU_CMD;
        cmd->rcb.ctx_id = hwq->ctx_hndl;
-       cmd->rcb.msi = SISL_MSI_RRQ_UPDATED;
-       cmd->rcb.timeout = MC_AFU_SYNC_TIMEOUT;
-
-       cmd->rcb.cdb[0] = 0xC0; /* AFU Sync */
-       cmd->rcb.cdb[1] = mode;
 
-       /* The cdb is aligned, no unaligned accessors required */
-       *((__be16 *)&cmd->rcb.cdb[2]) = cpu_to_be16(ctx_hndl_u);
-       *((__be32 *)&cmd->rcb.cdb[4]) = cpu_to_be32(res_hndl_u);
+       dev_dbg(dev, "%s: afu=%p cmd=%p type=%02x nretry=%d\n",
+               __func__, afu, cmd, cmd->rcb.cdb[0], nretry);
 
        rc = afu->send_cmd(afu, cmd);
-       if (unlikely(rc))
+       if (unlikely(rc)) {
+               rc = -ENOBUFS;
                goto out;
+       }
 
        rc = wait_resp(afu, cmd);
-       if (unlikely(rc))
-               rc = -1;
+       switch (rc) {
+       case -ETIMEDOUT:
+               rc = afu->context_reset(hwq);
+               if (rc) {
+                       cxlflash_schedule_async_reset(cfg);
+                       break;
+               }
+               /* fall through to retry */
+       case -EAGAIN:
+               if (++nretry < 2)
+                       goto retry;
+               /* fall through to exit */
+       default:
+               break;
+       }
+
+       if (rcb->ioasa)
+               *rcb->ioasa = cmd->sa;
 out:
        atomic_dec(&afu->cmds_active);
        mutex_unlock(&sync_active);
@@ -2045,38 +2301,88 @@ out:
 }
 
 /**
- * afu_reset() - resets the AFU
- * @cfg:       Internal structure associated with the host.
+ * cxlflash_afu_sync() - builds and sends an AFU sync command
+ * @afu:       AFU associated with the host.
+ * @ctx:       Identifies context requesting sync.
+ * @res:       Identifies resource requesting sync.
+ * @mode:      Type of sync to issue (lightweight, heavyweight, global).
  *
- * Return: 0 on success, -errno on failure
+ * AFU sync operations are only necessary and allowed when the device is
+ * operating normally. When not operating normally, sync requests can occur as
+ * part of cleaning up resources associated with an adapter prior to removal.
+ * In this scenario, these requests are simply ignored (safe due to the AFU
+ * going away).
+ *
+ * Return:
+ *     0 on success, -errno on failure
  */
-static int afu_reset(struct cxlflash_cfg *cfg)
+int cxlflash_afu_sync(struct afu *afu, ctx_hndl_t ctx, res_hndl_t res, u8 mode)
 {
+       struct cxlflash_cfg *cfg = afu->parent;
        struct device *dev = &cfg->dev->dev;
-       int rc = 0;
+       struct sisl_ioarcb rcb = { 0 };
 
-       /* Stop the context before the reset. Since the context is
-        * no longer available restart it after the reset is complete
-        */
-       term_afu(cfg);
+       dev_dbg(dev, "%s: afu=%p ctx=%u res=%u mode=%u\n",
+               __func__, afu, ctx, res, mode);
 
-       rc = init_afu(cfg);
+       rcb.req_flags = SISL_REQ_FLAGS_AFU_CMD;
+       rcb.msi = SISL_MSI_RRQ_UPDATED;
+       rcb.timeout = MC_AFU_SYNC_TIMEOUT;
 
-       dev_dbg(dev, "%s: returning rc=%d\n", __func__, rc);
-       return rc;
+       rcb.cdb[0] = SISL_AFU_CMD_SYNC;
+       rcb.cdb[1] = mode;
+       put_unaligned_be16(ctx, &rcb.cdb[2]);
+       put_unaligned_be32(res, &rcb.cdb[4]);
+
+       return send_afu_cmd(afu, &rcb);
 }
 
 /**
- * drain_ioctls() - wait until all currently executing ioctls have completed
- * @cfg:       Internal structure associated with the host.
+ * cxlflash_eh_abort_handler() - abort a SCSI command
+ * @scp:       SCSI command to abort.
  *
- * Obtain write access to read/write semaphore that wraps ioctl
- * handling to 'drain' ioctls currently executing.
+ * CXL Flash devices do not support a single command abort. Reset the context
+ * as per SISLite specification. Flush any pending commands in the hardware
+ * queue before the reset.
+ *
+ * Return: SUCCESS/FAILED as defined in scsi/scsi.h
  */
-static void drain_ioctls(struct cxlflash_cfg *cfg)
+static int cxlflash_eh_abort_handler(struct scsi_cmnd *scp)
 {
-       down_write(&cfg->ioctl_rwsem);
-       up_write(&cfg->ioctl_rwsem);
+       int rc = FAILED;
+       struct Scsi_Host *host = scp->device->host;
+       struct cxlflash_cfg *cfg = shost_priv(host);
+       struct afu_cmd *cmd = sc_to_afuc(scp);
+       struct device *dev = &cfg->dev->dev;
+       struct afu *afu = cfg->afu;
+       struct hwq *hwq = get_hwq(afu, cmd->hwq_index);
+
+       dev_dbg(dev, "%s: (scp=%p) %d/%d/%d/%llu "
+               "cdb=(%08x-%08x-%08x-%08x)\n", __func__, scp, host->host_no,
+               scp->device->channel, scp->device->id, scp->device->lun,
+               get_unaligned_be32(&((u32 *)scp->cmnd)[0]),
+               get_unaligned_be32(&((u32 *)scp->cmnd)[1]),
+               get_unaligned_be32(&((u32 *)scp->cmnd)[2]),
+               get_unaligned_be32(&((u32 *)scp->cmnd)[3]));
+
+       /* When the state is not normal, another reset/reload is in progress.
+        * Return failed and the mid-layer will invoke host reset handler.
+        */
+       if (cfg->state != STATE_NORMAL) {
+               dev_dbg(dev, "%s: Invalid state for abort, state=%d\n",
+                       __func__, cfg->state);
+               goto out;
+       }
+
+       rc = afu->context_reset(hwq);
+       if (unlikely(rc))
+               goto out;
+
+       rc = SUCCESS;
+
+out:
+       dev_dbg(dev, "%s: returning rc=%d\n", __func__, rc);
+       return rc;
 }
 
 /**
@@ -2654,6 +2960,74 @@ retry:
        return count;
 }
 
+static const char *hwq_mode_name[MAX_HWQ_MODE] = { "rr", "tag", "cpu" };
+
+/**
+ * hwq_mode_show() - presents the HWQ steering mode for the host
+ * @dev:       Generic device associated with the host.
+ * @attr:      Device attribute representing the HWQ steering mode.
+ * @buf:       Buffer of length PAGE_SIZE to report back the HWQ steering mode
+ *             as a character string.
+ *
+ * Return: The size of the ASCII string returned in @buf.
+ */
+static ssize_t hwq_mode_show(struct device *dev,
+                            struct device_attribute *attr, char *buf)
+{
+       struct cxlflash_cfg *cfg = shost_priv(class_to_shost(dev));
+       struct afu *afu = cfg->afu;
+
+       return scnprintf(buf, PAGE_SIZE, "%s\n", hwq_mode_name[afu->hwq_mode]);
+}
+
+/**
+ * hwq_mode_store() - sets the HWQ steering mode for the host
+ * @dev:       Generic device associated with the host.
+ * @attr:      Device attribute representing the HWQ steering mode.
+ * @buf:       Buffer of length PAGE_SIZE containing the HWQ steering mode
+ *             as a character string.
+ * @count:     Length of data resizing in @buf.
+ *
+ * rr = Round-Robin
+ * tag = Block MQ Tagging
+ * cpu = CPU Affinity
+ *
+ * Return: The size of the ASCII string returned in @buf.
+ */
+static ssize_t hwq_mode_store(struct device *dev,
+                             struct device_attribute *attr,
+                             const char *buf, size_t count)
+{
+       struct Scsi_Host *shost = class_to_shost(dev);
+       struct cxlflash_cfg *cfg = shost_priv(shost);
+       struct device *cfgdev = &cfg->dev->dev;
+       struct afu *afu = cfg->afu;
+       int i;
+       u32 mode = MAX_HWQ_MODE;
+
+       for (i = 0; i < MAX_HWQ_MODE; i++) {
+               if (!strncmp(hwq_mode_name[i], buf, strlen(hwq_mode_name[i]))) {
+                       mode = i;
+                       break;
+               }
+       }
+
+       if (mode >= MAX_HWQ_MODE) {
+               dev_info(cfgdev, "Invalid HWQ steering mode.\n");
+               return -EINVAL;
+       }
+
+       if ((mode == HWQ_MODE_TAG) && !shost_use_blk_mq(shost)) {
+               dev_info(cfgdev, "SCSI-MQ is not enabled, use a different "
+                        "HWQ steering mode.\n");
+               return -EINVAL;
+       }
+
+       afu->hwq_mode = mode;
+
+       return count;
+}
+
 /**
  * mode_show() - presents the current mode of the device
  * @dev:       Generic device associated with the device.
@@ -2686,6 +3060,7 @@ static DEVICE_ATTR_RO(port2_lun_table);
 static DEVICE_ATTR_RO(port3_lun_table);
 static DEVICE_ATTR_RW(irqpoll_weight);
 static DEVICE_ATTR_RW(num_hwqs);
+static DEVICE_ATTR_RW(hwq_mode);
 
 static struct device_attribute *cxlflash_host_attrs[] = {
        &dev_attr_port0,
@@ -2700,6 +3075,7 @@ static struct device_attribute *cxlflash_host_attrs[] = {
        &dev_attr_port3_lun_table,
        &dev_attr_irqpoll_weight,
        &dev_attr_num_hwqs,
+       &dev_attr_hwq_mode,
        NULL
 };
 
@@ -2723,6 +3099,7 @@ static struct scsi_host_template driver_template = {
        .ioctl = cxlflash_ioctl,
        .proc_name = CXLFLASH_NAME,
        .queuecommand = cxlflash_queuecommand,
+       .eh_abort_handler = cxlflash_eh_abort_handler,
        .eh_device_reset_handler = cxlflash_eh_device_reset_handler,
        .eh_host_reset_handler = cxlflash_eh_host_reset_handler,
        .change_queue_depth = cxlflash_change_queue_depth,
@@ -2812,6 +3189,86 @@ static void cxlflash_worker_thread(struct work_struct *work)
                scsi_scan_host(cfg->host);
 }
 
+/**
+ * cxlflash_chr_open() - character device open handler
+ * @inode:     Device inode associated with this character device.
+ * @file:      File pointer for this device.
+ *
+ * Only users with admin privileges are allowed to open the character device.
+ *
+ * Return: 0 on success, -errno on failure
+ */
+static int cxlflash_chr_open(struct inode *inode, struct file *file)
+{
+       struct cxlflash_cfg *cfg;
+
+       if (!capable(CAP_SYS_ADMIN))
+               return -EACCES;
+
+       cfg = container_of(inode->i_cdev, struct cxlflash_cfg, cdev);
+       file->private_data = cfg;
+
+       return 0;
+}
+
+/*
+ * Character device file operations
+ */
+static const struct file_operations cxlflash_chr_fops = {
+       .owner          = THIS_MODULE,
+       .open           = cxlflash_chr_open,
+};
+
+/**
+ * init_chrdev() - initialize the character device for the host
+ * @cfg:       Internal structure associated with the host.
+ *
+ * Return: 0 on success, -errno on failure
+ */
+static int init_chrdev(struct cxlflash_cfg *cfg)
+{
+       struct device *dev = &cfg->dev->dev;
+       struct device *char_dev;
+       dev_t devno;
+       int minor;
+       int rc = 0;
+
+       minor = cxlflash_get_minor();
+       if (unlikely(minor < 0)) {
+               dev_err(dev, "%s: Exhausted allowed adapters\n", __func__);
+               rc = -ENOSPC;
+               goto out;
+       }
+
+       devno = MKDEV(cxlflash_major, minor);
+       cdev_init(&cfg->cdev, &cxlflash_chr_fops);
+
+       rc = cdev_add(&cfg->cdev, devno, 1);
+       if (rc) {
+               dev_err(dev, "%s: cdev_add failed rc=%d\n", __func__, rc);
+               goto err1;
+       }
+
+       char_dev = device_create(cxlflash_class, NULL, devno,
+                                NULL, "cxlflash%d", minor);
+       if (IS_ERR(char_dev)) {
+               rc = PTR_ERR(char_dev);
+               dev_err(dev, "%s: device_create failed rc=%d\n",
+                       __func__, rc);
+               goto err2;
+       }
+
+       cfg->chardev = char_dev;
+out:
+       dev_dbg(dev, "%s: returning rc=%d\n", __func__, rc);
+       return rc;
+err2:
+       cdev_del(&cfg->cdev);
+err1:
+       cxlflash_put_minor(minor);
+       goto out;
+}
+
 /**
  * cxlflash_probe() - PCI entry point to add host
  * @pdev:      PCI device associated with the host.
@@ -2922,6 +3379,13 @@ static int cxlflash_probe(struct pci_dev *pdev,
        }
        cfg->init_state = INIT_STATE_SCSI;
 
+       rc = init_chrdev(cfg);
+       if (rc) {
+               dev_err(dev, "%s: init_chrdev failed rc=%d\n", __func__, rc);
+               goto out_remove;
+       }
+       cfg->init_state = INIT_STATE_CDEV;
+
        if (wq_has_sleeper(&cfg->reset_waitq)) {
                cfg->state = STATE_PROBED;
                wake_up_all(&cfg->reset_waitq);
@@ -3024,6 +3488,63 @@ static void cxlflash_pci_resume(struct pci_dev *pdev)
        scsi_unblock_requests(cfg->host);
 }
 
+/**
+ * cxlflash_devnode() - provides devtmpfs for devices in the cxlflash class
+ * @dev:       Character device.
+ * @mode:      Mode that can be used to verify access.
+ *
+ * Return: Allocated string describing the devtmpfs structure.
+ */
+static char *cxlflash_devnode(struct device *dev, umode_t *mode)
+{
+       return kasprintf(GFP_KERNEL, "cxlflash/%s", dev_name(dev));
+}
+
+/**
+ * cxlflash_class_init() - create character device class
+ *
+ * Return: 0 on success, -errno on failure
+ */
+static int cxlflash_class_init(void)
+{
+       dev_t devno;
+       int rc = 0;
+
+       rc = alloc_chrdev_region(&devno, 0, CXLFLASH_MAX_ADAPTERS, "cxlflash");
+       if (unlikely(rc)) {
+               pr_err("%s: alloc_chrdev_region failed rc=%d\n", __func__, rc);
+               goto out;
+       }
+
+       cxlflash_major = MAJOR(devno);
+
+       cxlflash_class = class_create(THIS_MODULE, "cxlflash");
+       if (IS_ERR(cxlflash_class)) {
+               rc = PTR_ERR(cxlflash_class);
+               pr_err("%s: class_create failed rc=%d\n", __func__, rc);
+               goto err;
+       }
+
+       cxlflash_class->devnode = cxlflash_devnode;
+out:
+       pr_debug("%s: returning rc=%d\n", __func__, rc);
+       return rc;
+err:
+       unregister_chrdev_region(devno, CXLFLASH_MAX_ADAPTERS);
+       goto out;
+}
+
+/**
+ * cxlflash_class_exit() - destroy character device class
+ */
+static void cxlflash_class_exit(void)
+{
+       dev_t devno = MKDEV(cxlflash_major, 0);
+
+       class_destroy(cxlflash_class);
+       unregister_chrdev_region(devno, CXLFLASH_MAX_ADAPTERS);
+}
+
 static const struct pci_error_handlers cxlflash_err_handler = {
        .error_detected = cxlflash_pci_error_detected,
        .slot_reset = cxlflash_pci_slot_reset,
@@ -3049,10 +3570,23 @@ static struct pci_driver cxlflash_driver = {
  */
 static int __init init_cxlflash(void)
 {
+       int rc;
+
        check_sizes();
        cxlflash_list_init();
+       rc = cxlflash_class_init();
+       if (unlikely(rc))
+               goto out;
 
-       return pci_register_driver(&cxlflash_driver);
+       rc = pci_register_driver(&cxlflash_driver);
+       if (unlikely(rc))
+               goto err;
+out:
+       pr_debug("%s: returning rc=%d\n", __func__, rc);
+       return rc;
+err:
+       cxlflash_class_exit();
+       goto out;
 }
 
 /**
@@ -3064,6 +3598,7 @@ static void __exit exit_cxlflash(void)
        cxlflash_free_errpage();
 
        pci_unregister_driver(&cxlflash_driver);
+       cxlflash_class_exit();
 }
 
 module_init(init_cxlflash);