On 7/16/20 5:02 AM, Hou Pu wrote: > Hi, > We encountered "iSCSI Login negotiation failed" several times. > After enable debug log in iscsi_target_nego.c of iSCSI target. > Error shows: > "Got LOGIN_FLAGS_READ_ACTIVE=1, conn: xxxxxxxxxx >>>>" > > Patch 1 is trying to fix this problem. Please see comment in patch 1 > for details. > > Sorry for delay of v4. Version 3 of this patchset could be found here[1]. > > Changes from v4: > * In iscsi_target_do_login_rx(), call cancel_delayed_work() if it is final > login pdu. Also call cancel_delayed_work() if current negotiation is failed. > This is advised by Mike Christie. See below[1] for more details. > > Changes from v3: > * Fix style problem found by checkpatch.pl. > Ah sorry, I thought I replied a little later about my concerns being overly paranoid and your patch being ok. I don't see it on the list so not sure what happened. If you are still not liking the cancel call, I'm ok with v3. Add my reviewed by to either version you prefer. Reviewed-by: Mike Christie <michael.christie@xxxxxxxxxx>
