Skip to content

Commit

Permalink
scsi: mpt3sas: Remove CPU arch check to determine perf_mode
Browse files Browse the repository at this point in the history
Currently default perf_mode is set to 'balanced' on Intel architecture
machines and on other machines default perf_mode is set to 'latency' mode.

This CPU architecture check is removed and the default perf_mode mode is
set to 'balanced' mode on all machines.

User can choose the required performance mode using perf_mode module
parameter.

Signed-off-by: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
  • Loading branch information
Sreekanth Reddy authored and Martin K. Petersen committed Jun 27, 2019
1 parent 4e1c94b commit 381abbd
Showing 1 changed file with 1 addition and 15 deletions.
16 changes: 1 addition & 15 deletions drivers/scsi/mpt3sas/mpt3sas_base.c
Original file line number Diff line number Diff line change
Expand Up @@ -113,8 +113,7 @@ MODULE_PARM_DESC(perf_mode,
"interrupt coalescing is enabled on all queues,\n\t\t"
"2 - latency: high iops mode is disabled &\n\t\t"
"interrupt coalescing is enabled on all queues with timeout value 0xA,\n"
"\t\tdefault - on Intel architecture, default perf_mode is\n\t\t"
" 'balanced' and in others architectures the default mode is 'latency'"
"\t\tdefault - default perf_mode is 'balanced'"
);

enum mpt3sas_perf_mode {
Expand Down Expand Up @@ -2990,19 +2989,6 @@ _base_check_and_enable_high_iops_queues(struct MPT3SAS_ADAPTER *ioc,

if (perf_mode == MPT_PERF_MODE_DEFAULT) {

#if defined(CONFIG_X86)
/*
* Use global variable boot_cpu_data.x86_vendor to
* determine whether the architecture is Intel or not.
*/
if (boot_cpu_data.x86_vendor != X86_VENDOR_INTEL) {
ioc->high_iops_queues = 0;
return;
}
#else
ioc->high_iops_queues = 0;
return;
#endif
speed = pcie_get_speed_cap(ioc->pdev);
dev_info(&ioc->pdev->dev, "PCIe device speed is %s\n",
speed == PCIE_SPEED_2_5GT ? "2.5GHz" :
Expand Down

0 comments on commit 381abbd

Please sign in to comment.