Skip to content

Commit c5becf5

Browse files
Revert "scsi: aacraid: Reply queue mapping to CPUs based on IRQ affinity"
This reverts commit 9dc704d. 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>
1 parent 235f2b5 commit c5becf5

File tree

4 files changed

+3
-43
lines changed

4 files changed

+3
-43
lines changed

drivers/scsi/aacraid/aacraid.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1678,7 +1678,6 @@ struct aac_dev
16781678
u32 handle_pci_error;
16791679
bool init_reset;
16801680
u8 soft_reset_support;
1681-
u8 use_map_queue;
16821681
};
16831682

16841683
#define aac_adapter_interrupt(dev) \

drivers/scsi/aacraid/commsup.c

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -223,12 +223,8 @@ int aac_fib_setup(struct aac_dev * dev)
223223
struct fib *aac_fib_alloc_tag(struct aac_dev *dev, struct scsi_cmnd *scmd)
224224
{
225225
struct fib *fibptr;
226-
u32 blk_tag;
227-
int i;
228226

229-
blk_tag = blk_mq_unique_tag(scsi_cmd_to_rq(scmd));
230-
i = blk_mq_unique_tag_to_tag(blk_tag);
231-
fibptr = &dev->fibs[i];
227+
fibptr = &dev->fibs[scsi_cmd_to_rq(scmd)->tag];
232228
/*
233229
* Null out fields that depend on being zero at the start of
234230
* each I/O

drivers/scsi/aacraid/linit.c

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919

2020
#include <linux/compat.h>
2121
#include <linux/blkdev.h>
22-
#include <linux/blk-mq-pci.h>
2322
#include <linux/completion.h>
2423
#include <linux/init.h>
2524
#include <linux/interrupt.h>
@@ -505,15 +504,6 @@ static int aac_slave_configure(struct scsi_device *sdev)
505504
return 0;
506505
}
507506

508-
static void aac_map_queues(struct Scsi_Host *shost)
509-
{
510-
struct aac_dev *aac = (struct aac_dev *)shost->hostdata;
511-
512-
blk_mq_pci_map_queues(&shost->tag_set.map[HCTX_TYPE_DEFAULT],
513-
aac->pdev, 0);
514-
aac->use_map_queue = true;
515-
}
516-
517507
/**
518508
* aac_change_queue_depth - alter queue depths
519509
* @sdev: SCSI device we are considering
@@ -1498,7 +1488,6 @@ static const struct scsi_host_template aac_driver_template = {
14981488
.bios_param = aac_biosparm,
14991489
.shost_groups = aac_host_groups,
15001490
.slave_configure = aac_slave_configure,
1501-
.map_queues = aac_map_queues,
15021491
.change_queue_depth = aac_change_queue_depth,
15031492
.sdev_groups = aac_dev_groups,
15041493
.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)
17861775
shost->max_lun = AAC_MAX_LUN;
17871776

17881777
pci_set_drvdata(pdev, shost);
1789-
shost->nr_hw_queues = aac->max_msix;
1790-
shost->host_tagset = 1;
17911778

17921779
error = scsi_add_host(shost, &pdev->dev);
17931780
if (error)
@@ -1919,7 +1906,6 @@ static void aac_remove_one(struct pci_dev *pdev)
19191906
struct aac_dev *aac = (struct aac_dev *)shost->hostdata;
19201907

19211908
aac_cancel_rescan_worker(aac);
1922-
aac->use_map_queue = false;
19231909
scsi_remove_host(shost);
19241910

19251911
__aac_shutdown(aac);

drivers/scsi/aacraid/src.c

Lines changed: 2 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -493,10 +493,6 @@ static int aac_src_deliver_message(struct fib *fib)
493493
#endif
494494

495495
u16 vector_no;
496-
struct scsi_cmnd *scmd;
497-
u32 blk_tag;
498-
struct Scsi_Host *shost = dev->scsi_host_ptr;
499-
struct blk_mq_queue_map *qmap;
500496

501497
atomic_inc(&q->numpending);
502498

@@ -509,25 +505,8 @@ static int aac_src_deliver_message(struct fib *fib)
509505
if ((dev->comm_interface == AAC_COMM_MESSAGE_TYPE3)
510506
&& dev->sa_firmware)
511507
vector_no = aac_get_vector(dev);
512-
else {
513-
if (!fib->vector_no || !fib->callback_data) {
514-
if (shost && dev->use_map_queue) {
515-
qmap = &shost->tag_set.map[HCTX_TYPE_DEFAULT];
516-
vector_no = qmap->mq_map[raw_smp_processor_id()];
517-
}
518-
/*
519-
* We hardcode the vector_no for
520-
* reserved commands as a valid shost is
521-
* absent during the init
522-
*/
523-
else
524-
vector_no = 0;
525-
} else {
526-
scmd = (struct scsi_cmnd *)fib->callback_data;
527-
blk_tag = blk_mq_unique_tag(scsi_cmd_to_rq(scmd));
528-
vector_no = blk_mq_unique_tag_to_hwq(blk_tag);
529-
}
530-
}
508+
else
509+
vector_no = fib->vector_no;
531510

532511
if (native_hba) {
533512
if (fib->flags & FIB_CONTEXT_FLAG_NATIVE_HBA_TMF) {

0 commit comments

Comments
 (0)