fix #5077: cherry-pick revert for aacraid resets

reported both in our bug tracker and upstream to fix the affected hardware.

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

diff --git a/patches/kernel/0018-fix-5077-aacraid-resets.patch b/patches/kernel/0018-fix-5077-aacraid-resets.patch
new file mode 100644 (file)
index 0000000..1b8380d
--- /dev/null
+++ b/patches/kernel/0018-fix-5077-aacraid-resets.patch
@@ -0,0 +1,151 @@
+From c5becf57dd5659c687d41d623a69f42d63f59eb2 Mon Sep 17 00:00:00 2001
+From: "Martin K. Petersen" <martin.petersen@oracle.com>
+Date: Fri, 8 Dec 2023 12:09:38 -0500
+Subject: Revert "scsi: aacraid: Reply queue mapping to CPUs based on IRQ
+ affinity"
+
+This reverts commit 9dc704dcc09eae7d21b5da0615eb2ed79278f63e.
+
+Several reports have been made indicating that this commit caused
+hangs. Numerous attempts at root causing and fixing the issue have
+been unsuccessful so let's revert for now.
+
+Link: https://bugzilla.kernel.org/show_bug.cgi?id=217599
+Cc: <stable@vger.kernel.org>
+Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
+---
+ drivers/scsi/aacraid/aacraid.h |  1 -
+ drivers/scsi/aacraid/commsup.c |  6 +-----
+ drivers/scsi/aacraid/linit.c   | 14 --------------
+ drivers/scsi/aacraid/src.c     | 25 ++-----------------------
+ 4 files changed, 3 insertions(+), 43 deletions(-)
+
+diff --git a/drivers/scsi/aacraid/aacraid.h b/drivers/scsi/aacraid/aacraid.h
+index 73b6ac0c01f54..7d5a155073c62 100644
+--- a/drivers/scsi/aacraid/aacraid.h
++++ b/drivers/scsi/aacraid/aacraid.h
+@@ -1678,7 +1678,6 @@ struct aac_dev
+       u32                     handle_pci_error;
+       bool                    init_reset;
+       u8                      soft_reset_support;
+-      u8                      use_map_queue;
+ };
+ 
+ #define aac_adapter_interrupt(dev) \
+diff --git a/drivers/scsi/aacraid/commsup.c b/drivers/scsi/aacraid/commsup.c
+index 013a9a334972e..25cee03d7f973 100644
+--- a/drivers/scsi/aacraid/commsup.c
++++ b/drivers/scsi/aacraid/commsup.c
+@@ -223,12 +223,8 @@ int aac_fib_setup(struct aac_dev * dev)
+ struct fib *aac_fib_alloc_tag(struct aac_dev *dev, struct scsi_cmnd *scmd)
+ {
+       struct fib *fibptr;
+-      u32 blk_tag;
+-      int i;
+ 
+-      blk_tag = blk_mq_unique_tag(scsi_cmd_to_rq(scmd));
+-      i = blk_mq_unique_tag_to_tag(blk_tag);
+-      fibptr = &dev->fibs[i];
++      fibptr = &dev->fibs[scsi_cmd_to_rq(scmd)->tag];
+       /*
+        *      Null out fields that depend on being zero at the start of
+        *      each I/O
+diff --git a/drivers/scsi/aacraid/linit.c b/drivers/scsi/aacraid/linit.c
+index c4a36c0be527c..68f4dbcfff492 100644
+--- a/drivers/scsi/aacraid/linit.c
++++ b/drivers/scsi/aacraid/linit.c
+@@ -19,7 +19,6 @@
+ 
+ #include <linux/compat.h>
+ #include <linux/blkdev.h>
+-#include <linux/blk-mq-pci.h>
+ #include <linux/completion.h>
+ #include <linux/init.h>
+ #include <linux/interrupt.h>
+@@ -505,15 +504,6 @@ common_config:
+       return 0;
+ }
+ 
+-static void aac_map_queues(struct Scsi_Host *shost)
+-{
+-      struct aac_dev *aac = (struct aac_dev *)shost->hostdata;
+-
+-      blk_mq_pci_map_queues(&shost->tag_set.map[HCTX_TYPE_DEFAULT],
+-                            aac->pdev, 0);
+-      aac->use_map_queue = true;
+-}
+-
+ /**
+  *    aac_change_queue_depth          -       alter queue depths
+  *    @sdev:  SCSI device we are considering
+@@ -1498,7 +1488,6 @@ static const struct scsi_host_template aac_driver_template = {
+       .bios_param                     = aac_biosparm,
+       .shost_groups                   = aac_host_groups,
+       .slave_configure                = aac_slave_configure,
+-      .map_queues                     = aac_map_queues,
+       .change_queue_depth             = aac_change_queue_depth,
+       .sdev_groups                    = aac_dev_groups,
+       .eh_abort_handler               = aac_eh_abort,
+@@ -1786,8 +1775,6 @@ static int aac_probe_one(struct pci_dev *pdev, const struct pci_device_id *id)
+       shost->max_lun = AAC_MAX_LUN;
+ 
+       pci_set_drvdata(pdev, shost);
+-      shost->nr_hw_queues = aac->max_msix;
+-      shost->host_tagset = 1;
+ 
+       error = scsi_add_host(shost, &pdev->dev);
+       if (error)
+@@ -1919,7 +1906,6 @@ static void aac_remove_one(struct pci_dev *pdev)
+       struct aac_dev *aac = (struct aac_dev *)shost->hostdata;
+ 
+       aac_cancel_rescan_worker(aac);
+-      aac->use_map_queue = false;
+       scsi_remove_host(shost);
+ 
+       __aac_shutdown(aac);
+diff --git a/drivers/scsi/aacraid/src.c b/drivers/scsi/aacraid/src.c
+index 61949f3741886..11ef58204e96f 100644
+--- a/drivers/scsi/aacraid/src.c
++++ b/drivers/scsi/aacraid/src.c
+@@ -493,10 +493,6 @@ static int aac_src_deliver_message(struct fib *fib)
+ #endif
+ 
+       u16 vector_no;
+-      struct scsi_cmnd *scmd;
+-      u32 blk_tag;
+-      struct Scsi_Host *shost = dev->scsi_host_ptr;
+-      struct blk_mq_queue_map *qmap;
+ 
+       atomic_inc(&q->numpending);
+ 
+@@ -509,25 +505,8 @@ static int aac_src_deliver_message(struct fib *fib)
+               if ((dev->comm_interface == AAC_COMM_MESSAGE_TYPE3)
+                       && dev->sa_firmware)
+                       vector_no = aac_get_vector(dev);
+-              else {
+-                      if (!fib->vector_no || !fib->callback_data) {
+-                              if (shost && dev->use_map_queue) {
+-                                      qmap = &shost->tag_set.map[HCTX_TYPE_DEFAULT];
+-                                      vector_no = qmap->mq_map[raw_smp_processor_id()];
+-                              }
+-                              /*
+-                               *      We hardcode the vector_no for
+-                               *      reserved commands as a valid shost is
+-                               *      absent during the init
+-                               */
+-                              else
+-                                      vector_no = 0;
+-                      } else {
+-                              scmd = (struct scsi_cmnd *)fib->callback_data;
+-                              blk_tag = blk_mq_unique_tag(scsi_cmd_to_rq(scmd));
+-                              vector_no = blk_mq_unique_tag_to_hwq(blk_tag);
+-                      }
+-              }
++              else
++                      vector_no = fib->vector_no;
+ 
+               if (native_hba) {
+                       if (fib->flags & FIB_CONTEXT_FLAG_NATIVE_HBA_TMF) {
+-- 
+cgit 1.2.3-korg
+