RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-18 Thread Long Li
Srinivasan ; Long Li ; > martin.peter...@oracle.com > Subject: Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O > > On Wed, 2017-05-17 at 23:54 -0700, Christoph Hellwig wrote: > > On Wed, May 17, 2017 at 11:05:18PM +, Bart Van Assche wrote: > > > Thank you for

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-18 Thread Bart Van Assche
On Wed, 2017-05-17 at 23:54 -0700, Christoph Hellwig wrote: > On Wed, May 17, 2017 at 11:05:18PM +, Bart Van Assche wrote: > > Thank you for the feedback. I'm working on a patch series that merges the > > scsi-sq > > and scsi-mq code paths for command initialization and that should fix the >

RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-18 Thread KY Srinivasan
om; Stephen Hemminger > ; KY Srinivasan > Subject: Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O > > On Wed, May 17, 2017 at 11:05:18PM +, Bart Van Assche wrote: > > Thank you for the feedback. I'm working on a patch series that merges the >

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-17 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-17 Thread Christoph Hellwig
On Wed, May 17, 2017 at 11:05:18PM +, Bart Van Assche wrote: > Thank you for the feedback. I'm working on a patch series that merges the > scsi-sq > and scsi-mq code paths for command initialization and that should fix the bug > you > encountered. While that sounds great (I tried it a while

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-17 Thread Bart Van Assche
er...@vger.kernel.org; Long Li ; > > martin.peter...@oracle.com > > Cc: Stephen Hemminger ; KY Srinivasan > > > > Subject: Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O > > > > On Mon, 2017-05-15 at 23:32 +, Long Li wrote: > > > Thanks for lookin

RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-16 Thread Long Li
inger ; KY Srinivasan > > Subject: Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O > > On Mon, 2017-05-15 at 23:32 +, Long Li wrote: > > Thanks for looking! Yes this is for chasing a bug. > > > > With the patch, we also zero the private data used by

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-15 Thread Bart Van Assche
On Mon, 2017-05-15 at 23:32 +, Long Li wrote: > Thanks for looking! Yes this is for chasing a bug. > > With the patch, we also zero the private data used by lower layer driver, in > addition to the private data in scsi_cmnd. Hello Long, What bug did you encounter, with which combination of

RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-15 Thread Long Li
> -Original Message- > From: Bart Van Assche [mailto:bart.vanass...@sandisk.com] > Sent: Monday, May 15, 2017 4:03 PM > To: j...@linux.vnet.ibm.com; linux-s...@vger.kernel.org; linux- > ker...@vger.kernel.org; Long Li ; > martin.peter...@oracle.com > Cc: Long Li > Subject: Re: [Possible Ph

RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-15 Thread KY Srinivasan
RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O > > > -Original Message- > > From: Long Li [mailto:lon...@exchange.microsoft.com] > > Sent: Wednesday, May 10, 2017 2:07 PM > > To: James E.J. Bottomley ; Martin K. Petersen > > ; linux-s...@vge

RE: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-15 Thread Long Li
> -Original Message- > From: Long Li [mailto:lon...@exchange.microsoft.com] > Sent: Wednesday, May 10, 2017 2:07 PM > To: James E.J. Bottomley ; Martin K. Petersen > ; linux-s...@vger.kernel.org; linux- > ker...@vger.kernel.org > Cc: Long Li > Subject: [PATCH] scsi: zero per-cmd driver dat