[0/2] Two SRP initiator bug fixes

Message

Bart Van Assche Jan. 17, 2019, 12:27 a.m. UTC

Hi Jason and Doug,

The two patches in this series fix bugs I ran into while testing the SRP
initiator driver against kernel v5.0-rc1. Please consider these patches
for kernel v5.0.

Thanks,

Bart.

Bart Van Assche (2):
  RDMA/srp: Avoid calling mutex_lock() from inside scsi_queue_rq()
  RDMA/srp: Rework SCSI device reset handling

 drivers/infiniband/ulp/srp/ib_srp.c | 34 ++++++++++++++++-------------
 1 file changed, 19 insertions(+), 15 deletions(-)

Comments

Jason Gunthorpe Jan. 18, 2019, 9:11 p.m. UTC | #1

On Wed, Jan 16, 2019 at 04:27:15PM -0800, Bart Van Assche wrote:
> Hi Jason and Doug,
> 
> The two patches in this series fix bugs I ran into while testing the SRP
> initiator driver against kernel v5.0-rc1. Please consider these patches
> for kernel v5.0.

applied to for-rc

Thanks,
Jason

Christoph Hellwig Jan. 19, 2019, 10:01 a.m. UTC | #2

On Wed, Jan 16, 2019 at 04:27:15PM -0800, Bart Van Assche wrote:
> Hi Jason and Doug,
> 
> The two patches in this series fix bugs I ran into while testing the SRP
> initiator driver against kernel v5.0-rc1. Please consider these patches
> for kernel v5.0.

Hi Bart,

can you please CC any srp patches to linux-scsi given that it is a SCSI
LLDD?

Jason Gunthorpe Jan. 21, 2019, 5:43 p.m. UTC | #3

On Sat, Jan 19, 2019 at 02:01:51AM -0800, Christoph Hellwig wrote:
> On Wed, Jan 16, 2019 at 04:27:15PM -0800, Bart Van Assche wrote:
> > Hi Jason and Doug,
> > 
> > The two patches in this series fix bugs I ran into while testing the SRP
> > initiator driver against kernel v5.0-rc1. Please consider these patches
> > for kernel v5.0.
> 
> Hi Bart,
> 
> can you please CC any srp patches to linux-scsi given that it is a SCSI
> LLDD?

Bart, please let me know if you want to revise these patches or not, I
can still drop them off the tree.

Thanks,
Jason

Bart Van Assche Jan. 21, 2019, 9:05 p.m. UTC | #4

On 1/21/19 9:43 AM, Jason Gunthorpe wrote:
> On Sat, Jan 19, 2019 at 02:01:51AM -0800, Christoph Hellwig wrote:
>> On Wed, Jan 16, 2019 at 04:27:15PM -0800, Bart Van Assche wrote:
>>> Hi Jason and Doug,
>>>
>>> The two patches in this series fix bugs I ran into while testing the SRP
>>> initiator driver against kernel v5.0-rc1. Please consider these patches
>>> for kernel v5.0.
>>
>> Hi Bart,
>>
>> can you please CC any srp patches to linux-scsi given that it is a SCSI
>> LLDD?
> 
> Bart, please let me know if you want to revise these patches or not, I
> can still drop them off the tree.

Hi Jason.

Please wait for the outcome of the discussion with Christoph.

Thanks,

Bart.

Bart Van Assche Jan. 21, 2019, 9:06 p.m. UTC | #5

On 1/19/19 2:01 AM, Christoph Hellwig wrote:
> On Wed, Jan 16, 2019 at 04:27:15PM -0800, Bart Van Assche wrote:
>> The two patches in this series fix bugs I ran into while testing the SRP
>> initiator driver against kernel v5.0-rc1. Please consider these patches
>> for kernel v5.0.
> 
> can you please CC any srp patches to linux-scsi given that it is a SCSI
> LLDD?

Hi Christoph,

That sounds like a good idea to me. I will do that.

Bart.

Jason Gunthorpe Jan. 21, 2019, 9:21 p.m. UTC | #6

On Mon, Jan 21, 2019 at 01:05:35PM -0800, Bart Van Assche wrote:
> On 1/21/19 9:43 AM, Jason Gunthorpe wrote:
> > On Sat, Jan 19, 2019 at 02:01:51AM -0800, Christoph Hellwig wrote:
> > > On Wed, Jan 16, 2019 at 04:27:15PM -0800, Bart Van Assche wrote:
> > > > Hi Jason and Doug,
> > > > 
> > > > The two patches in this series fix bugs I ran into while testing the SRP
> > > > initiator driver against kernel v5.0-rc1. Please consider these patches
> > > > for kernel v5.0.
> > > 
> > > Hi Bart,
> > > 
> > > can you please CC any srp patches to linux-scsi given that it is a SCSI
> > > LLDD?
> > 
> > Bart, please let me know if you want to revise these patches or not, I
> > can still drop them off the tree.
> 
> Hi Jason.
> 
> Please wait for the outcome of the discussion with Christoph.

Okay I dropped them off, please resend anything that needs to be
applied again.

Jason