mbox series

[0/2] Ramrod timer and ep state fixes.

Message ID 20190419062952.25922-1-mrangankar@marvell.com (mailing list archive)
Headers show
Series Ramrod timer and ep state fixes. | expand

Message

Manish Rangankar April 19, 2019, 6:29 a.m. UTC
Hi Martin,

Please apply the following patches to the scsi tree at your
earliest convenience.

Thanks,
Manish

Manish Rangankar (2):
  qedi: Abort ep termination if offload not scheduled.
  qedi: Adjust termination and offload ramrod timers.

 drivers/scsi/qedi/qedi_iscsi.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

Comments

Lee Duncan April 19, 2019, 2:31 p.m. UTC | #1
Hi Manish:

On 4/18/19 11:29 PM, Manish Rangankar wrote:
> Hi Martin,
> 
> Please apply the following patches to the scsi tree at your
> earliest convenience.
> 
> Thanks,
> Manish
> 
> Manish Rangankar (2):
>   qedi: Abort ep termination if offload not scheduled.
>   qedi: Adjust termination and offload ramrod timers.
> 
>  drivers/scsi/qedi/qedi_iscsi.c | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
> 

Both of these patches look fine to me, but I'd like a bit more detail in
the description, like why you changed the code the way you did.
Manish Rangankar April 22, 2019, 4:28 a.m. UTC | #2
> -----Original Message-----
> From: Lee Duncan <lduncan@suse.com>
> Sent: Friday, April 19, 2019 8:01 PM
> To: Manish Rangankar <mrangankar@marvell.com>;
> martin.petersen@oracle.com; cleech@redhat.com
> Cc: linux-scsi@vger.kernel.org; QLogic-Storage-Upstream@cavium.com
> Subject: [EXT] Re: [PATCH 0/2] Ramrod timer and ep state fixes.
> 
> External Email
> 
> ----------------------------------------------------------------------
> Hi Manish:
> 
> On 4/18/19 11:29 PM, Manish Rangankar wrote:
> > Hi Martin,
> >
> > Please apply the following patches to the scsi tree at your earliest
> > convenience.
> >
> > Thanks,
> > Manish
> >
> > Manish Rangankar (2):
> >   qedi: Abort ep termination if offload not scheduled.
> >   qedi: Adjust termination and offload ramrod timers.
> >
> >  drivers/scsi/qedi/qedi_iscsi.c | 8 ++++++--
> >  1 file changed, 6 insertions(+), 2 deletions(-)
> >
> 
> Both of these patches look fine to me, but I'd like a bit more detail in the
> description, like why you changed the code the way you did.
> 

Sure, will send out the patches with detail description.