Re: Target core task management function handling rework

[Bart Van Assche <Bart.VanAssche@sandisk.com>]

On Tue, 2017-01-24 at 00:20 +0200, Sagi Grimberg wrote:
> Bart, I recall you had a patch at some point to periodically print
> out the hanging session commands in target_wait_for_sess_cmds(), do we
> want to get it in? I think we can all benefit from it.

(+target-devel)

Hello Sagi,

I have pushed the current state of these patches to the
scsi-target-for-v4.11 branch of
https://git.kernel.org/cgit/linux/kernel/git/bvanassche/linux.git/

So far these patches have been tested with iSCSI/TCP, ib_srpt and
qla2xxx drivers. The patches that are present on this branch are:

Bart Van Assche (36):
      qla2xxx: Avoid using variable-length arrays
      target/iscsi: Fix indentation in iscsi_target_start_negotiation()
      target/iscsi: Fix spelling of "perform"
      target/iscsi: Fix spelling of "reallegiance"
      target/iscsi: Introduce a helper function for TMF translation
      target/iscsi: Fix iSCSI task reassignment handling
      target/iscsi: Fix solicited data sequence offset calculations
      cxgbit: Fix endianness annotations
      target/tcm_fc: Remove a set-but-not-used variable
      target: Remove se_tmr_req.tmr_lun
      target: Make core_tmr_abort_task() consider all commands
      target: Make it possible to specify I_T nexus for SCSI abort
      tcm_qla2xxx: Let the target core look up the LUN of the aborted cmd
      target: Fix transport_wait_for_tasks() documentation
      target: Remove an overly chatty debug message
      target: Add a missing target_put_nacl() call
      target: Stop execution if CMD_T_STOP has been set
      target: Fix a reference leak in transport_cmd_check_stop_to_fabric()
      target: Inline transport_cmd_check_stop()
      target: Make ABORT and LUN RESET handling synchronous
      target: Simplify session shutdown code
      target: Remove the SCF_SEND_DELAYED_TAS command flag
      target: Inline transport_check_aborted_status()
      target: Remove the write_pending_status() callback function
      target: Remove several state tests from TMF code
      target: Remove command flag CMD_T_BUSY
      target: Simplify LUN RESET implementation
      target: Remove command flag CMD_T_DEV_ACTIVE
      target: Reduce number of __transport_wait_for_tasks() arguments
      target: Remove command flag CMD_T_TAS
      target: Remove unused arguments from __target_check_io_state()
      target: Change return type of transport_wait_for_tasks() into void
      target: Inline transport_put_cmd()
      target: Reduce the number of transport_lun_remove_cmd() callers
      target: Move target_remove_from_state_list() into target_release_cmd_kref()
      target: Avoid that XCOPY commands trigger a deadlock

Himanshu Madhani (2):
      qla2xxx: Remove SRR code
      qla2xxx: Remove unused reverse_ini_mode

Joe Carnuccio (1):
      qla2xxx: Simplify usage of SRB structure in driver

Quinn Tran (9):
      qla2xxx: Remove direct access of scsi_status field in se_cmd
      qla2xxx: Cleanup TMF code translation from qla_target
      qla2xxx: Make trace flags more readable
      qla2xxx: Fix wrong argument in sp done callback
      qla2xxx: Use d_id instead of s_id for more clarity
      qla2xxx: Track I-T nexus as single fc_port struct
      qla2xxx: Add framework for Async fabric discovery
      qla2xxx: Add Dual mode support in the driver
      qla2xxx: Improve RSCN handling in driver

Varun Prakash (1):
      target/iscsi: Fix unsolicited data seq_end_offset calculation

Bart.