Hi Martin, > -----Original Message----- > From: Martin K. Petersen <martin.petersen@xxxxxxxxxx> > Sent: Tuesday, April 14, 2020 6:53 AM > To: Saurav Kashyap <skashyap@xxxxxxxxxxx> > Cc: martin.petersen@xxxxxxxxxx; GR-QLogic-Storage-Upstream <GR-QLogic- > Storage-Upstream@xxxxxxxxxxx>; linux-scsi@xxxxxxxxxxxxxxx; Javed Hasan > <jhasan@xxxxxxxxxxx>; netdev@xxxxxxxxxxxxxxx > Subject: [EXT] Re: [PATCH v3 2/7] qedf: Fix for the deviations from the SAM-4 > spec. > > External Email > > ---------------------------------------------------------------------- > > Saurav, > > This should be 3 patches since there are 3 different functional > changes. > > > - Upper limit for retry delay(QEDF_RETRY_DELAY_MAX) > > increased from 20 sec to 1 min. > > > - Log an event/message indicating throttling of I/O > > for the target and include scope and retry delay > > time returned by the target and the driver enforced delay. > > > - Synchronizing the update of the fcport->retry_delay_timestamp > > between qedf_queuecommand() and qedf_scsi_completion(). > > "Synchronize". > > Please describe why this needs to be synchronized. <SK> Sure, Shall I need to submit full patch set again? Thanks, ~Saurav > > -- > Martin K. Petersen Oracle Linux Engineering
