> -----Original Message----- > From: Nicholas Krause [mailto:xerofoify@xxxxxxxxx] > Sent: Friday, January 01, 2016 12:06 PM > To: kashyap.desai@xxxxxxxxxxxxx > Cc: sumit.saxena@xxxxxxxxxxxxx; uday.lingala@xxxxxxxxxxxxx; > JBottomley@xxxxxxxx; martin.petersen@xxxxxxxxxx; > megaraidlinux.pdl@xxxxxxxxxxxxx; linux-scsi@xxxxxxxxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx > Subject: [PATCH RESEND] megaraid:Fix for loop termination statment in the > function process_fw_state_change_wq > > This fixes the for loop terimation for the waiting period between the first and > second init to make the variable wait terminate at the value of 20 rather then 30 > in other to follow the comments about this for loop of waiting for about 20 > seconds rather then > 30 in the function fw_state_change_wq between initializations. > > Signed-off-by: Nicholas Krause <xerofoify@xxxxxxxxx> > --- > drivers/scsi/megaraid/megaraid_sas_base.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c > b/drivers/scsi/megaraid/megaraid_sas_base.c > index 71b884d..22fd333 100644 > --- a/drivers/scsi/megaraid/megaraid_sas_base.c > +++ b/drivers/scsi/megaraid/megaraid_sas_base.c > @@ -3168,9 +3168,8 @@ process_fw_state_change_wq(struct work_struct > *work) > "state 2 starting...\n"); > > /*waitting for about 20 second before start the second init*/ > - for (wait = 0; wait < 30; wait++) { > + for (wait = 0; wait < 20; wait++) > msleep(1000); > - } > > if (megasas_transition_to_ready(instance, 1)) { > printk(KERN_NOTICE "megaraid_sas:adapter not > ready\n"); There should be 30 seconds delay here, comments needs to be rectified here. I will take care of this in next patch set. > -- > 2.1.4 -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
