On 10/17/2016 12:24 PM, Sumit Saxena wrote: > This patch fixes the issue of wrong PhysArm was sent to firmware for R1 > VD downgrade. > > Signed-off-by: Kiran Kumar Kasturi <[email protected]> > Signed-off-by: Sumit Saxena <[email protected]> > --- > drivers/scsi/megaraid/megaraid_sas_fp.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/megaraid/megaraid_sas_fp.c > b/drivers/scsi/megaraid/megaraid_sas_fp.c > index e413113..f237d00 100644 > --- a/drivers/scsi/megaraid/megaraid_sas_fp.c > +++ b/drivers/scsi/megaraid/megaraid_sas_fp.c > @@ -782,7 +782,8 @@ static u8 mr_spanset_get_phy_params(struct > megasas_instance *instance, u32 ld, > (raid->regTypeReqOnRead != REGION_TYPE_UNUSED)))) > pRAID_Context->regLockFlags = REGION_TYPE_EXCLUSIVE; > else if (raid->level == 1) { > - pd = MR_ArPdGet(arRef, physArm + 1, map); > + physArm = physArm + 1; > + pd = MR_ArPdGet(arRef, physArm, map); > if (pd != MR_PD_INVALID) > *pDevHandle = MR_PdDevHandleGet(pd, map); > } > @@ -879,7 +880,8 @@ u8 MR_GetPhyParams(struct megasas_instance *instance, u32 > ld, u64 stripRow, > pRAID_Context->regLockFlags = REGION_TYPE_EXCLUSIVE; > else if (raid->level == 1) { > /* Get alternate Pd. */ > - pd = MR_ArPdGet(arRef, physArm + 1, map); > + physArm = physArm + 1; > + pd = MR_ArPdGet(arRef, physArm, map); > if (pd != MR_PD_INVALID) > /* Get dev handle from Pd */ > *pDevHandle = MR_PdDevHandleGet(pd, map); > Reviewed-by: Hannes Reinecke <[email protected]>
Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking [email protected] +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
