On 02/06/2017 11:00 AM, Shivasharan S wrote:
> For RAID1 FastPath writes, driver needs to allocate extra commands
> internally to accommodate for the extra peer command being sent.
> Currently driver is allocating 2 extra commands for each but only
> one extra command is necessary.
> Set RAID_1_10_RMW_CMDS to 2 and also change macro name to
> RAID_1_PEER_CMDS.
>
> Signed-off-by: Kashyap Desai <kashyap.desai@xxxxxxxxxxxx>
> Signed-off-by: Shivasharan S <shivasharan.srikanteshwara@xxxxxxxxxxxx>
> ---
> drivers/scsi/megaraid/megaraid_sas_fusion.c | 2 +-
> drivers/scsi/megaraid/megaraid_sas_fusion.h | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/scsi/megaraid/megaraid_sas_fusion.c b/drivers/scsi/megaraid/megaraid_sas_fusion.c
> index b5bb33e..6dffb17 100644
> --- a/drivers/scsi/megaraid/megaraid_sas_fusion.c
> +++ b/drivers/scsi/megaraid/megaraid_sas_fusion.c
> @@ -293,7 +293,7 @@ megasas_fusion_update_can_queue(struct megasas_instance *instance, int fw_boot_c
>
> if (instance->is_ventura)
> instance->max_mpt_cmds =
> - instance->max_fw_cmds * RAID_1_10_RMW_CMDS;
> + instance->max_fw_cmds * RAID_1_PEER_CMDS;
> else
> instance->max_mpt_cmds = instance->max_fw_cmds;
> }
> diff --git a/drivers/scsi/megaraid/megaraid_sas_fusion.h b/drivers/scsi/megaraid/megaraid_sas_fusion.h
> index 60566e4..c36f9f7 100644
> --- a/drivers/scsi/megaraid/megaraid_sas_fusion.h
> +++ b/drivers/scsi/megaraid/megaraid_sas_fusion.h
> @@ -101,7 +101,7 @@ enum MR_RAID_FLAGS_IO_SUB_TYPE {
> #define MEGASAS_FP_CMD_LEN 16
> #define MEGASAS_FUSION_IN_RESET 0
> #define THRESHOLD_REPLY_COUNT 50
> -#define RAID_1_10_RMW_CMDS 3
> +#define RAID_1_PEER_CMDS 2
> #define JBOD_MAPS_COUNT 2
>
> enum MR_FUSION_ADAPTER_TYPE {
>
Reviewed-by: Hannes Reinecke <hare@xxxxxxxx>
Cheers,
Hannes
--
Dr. Hannes Reinecke zSeries & Storage
hare@xxxxxxxx +49 911 74053 688
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
