Re: [PATCH v2 13/17] qla2xxx: Remove redundant code

2017-06-01 Thread Madhani, Himanshu
> On May 31, 2017, at 4:37 PM, Bart Van Assche > wrote: > > On Tue, 2017-05-30 at 10:54 -0700, Himanshu Madhani wrote: >> The reason for hard coding LUN ID to 0 is that, from the FC >> protocol perspective, ABTS does not have any knowledge of >> LUN ID. So, there is

Re: [PATCH v2 13/17] qla2xxx: Remove redundant code

2017-05-31 Thread Bart Van Assche
On Tue, 2017-05-30 at 10:54 -0700, Himanshu Madhani wrote: > The reason for hard coding LUN ID to 0 is that, from the FC > protocol perspective, ABTS does not have any knowledge of > LUN ID. So, there is no reason for qla2xxx driver to > manufacture the LUN ID. Sorry but I think this is

[PATCH v2 13/17] qla2xxx: Remove redundant code

2017-05-30 Thread Himanshu Madhani
From: Quinn Tran During ABTS or Abort task, qla2xxx does a pre-search for the se_cmd, based on command's tag. The same search is performed by TCM. Remove the extra search from qla2xxx. The reason for hard coding LUN ID to 0 is that, from the FC protocol perspective, ABTS