linux-toradex.git/drivers/target/loopback/tcm_loop.c, branch v3.6-rc1

target: remove transport_generic_process_write

2012-07-17T00:35:16+00:00

Just call target_execute_cmd directly.  Also, convert loopback, sbp,
usb-gadget to use the newly exported target_execute_cmd().

Signed-off-by: Christoph Hellwig 
Signed-off-by: Nicholas Bellinger

target: remove control CDB flags

2012-07-17T00:25:55+00:00

We don't need three flags to classifiy the CDB as we can check for a NULL S/G
list for a dataless command, and can infer from the absence of the data flag
that we deal with a control CDB.  Also remove the _SG_IO from the data CDB
flag as all I/O is dont on S/G lists now.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Nicholas Bellinger

target: Rename target_allocate_tasks to target_setup_cmd_from_cdb

2012-04-15T00:40:36+00:00

This patch renames a horribly misnamed function that no longer allocate
tasks to something more descriptive for it's modern use in target core.

(nab: Fix up ib_srpt to use this as well ahead of a target_submit_cmd
conversion)

Signed-off-by: Andy Grover 
Signed-off-by: Nicholas Bellinger

loopback: Fix transport_generic_allocate_tasks error handling

2012-03-18T01:07:27+00:00

This patch addresses a tcm_loop bug with transport_generic_allocate_tasks()
return checking in tcm_loop_submission_work() where other non zero return
codes (including -EBUSY for reservation conflicts) are incorrectly falling
through to transport_generic_map_mem_to_cmd() -> transport_handle_cdb_direct().

This bug was introduced into target-pending/for-next-merge with the following
for-3.4 commit:

commit 16786454acec0e0e55e32d508b3058b32c1f23f3
Author: Christoph Hellwig 
Date:   Thu Feb 2 17:04:42 2012 -0500

    tcm_loop: switch to using transport_handle_cdb_direct

Cc: Christoph Hellwig 
Signed-off-by: Nicholas Bellinger

target: remove obvious warnings

2012-03-16T02:16:09+00:00

Get rid of a bunch of write-only variables.  In a number of cases I
suspect actual bugs to be present, so I left all of those for a second
look.

(nab: fix lio-core patch fuzz)

Signed-off-by: Joern Engel 
Signed-off-by: Nicholas Bellinger

target: Drop unused legacy target_core_fabric_ops API callers

2012-03-10T22:42:55+00:00

This patch drops the following unused legacy API callers from target_core_fabric.h:

*) TFO->fall_back_to_erl0()
*) TFO->stop_session()
*) TFO->sess_logged_in()
*) TFO->is_state_remove()

This patch also removes the stub usage in loopback, tcm_fc, iscsi_target,
and ib_srpt fabric modules.

Signed-off-by: Nicholas Bellinger

tcm_loop: Set residual field for SCSI commands

2012-02-25T22:37:50+00:00

If the target core signals an over- or under-run, tcm_loop should call
scsi_set_resid() to tell the SCSI midlayer about the residual data length.

The difference can be seen by doing something like

    strace -eioctl sg_raw -r 1024 /dev/sda 8 0 0 0 1 0 > /dev/null

and looking at the "resid=" part of the SG_IO ioctl -- after this patch,
the field is correctly reported as 512.

Signed-off-by: Roland Dreier 
Cc: stable@vger.kernel.org
Signed-off-by: Nicholas Bellinger

tcm_loop: switch to using transport_handle_cdb_direct

2012-02-25T22:37:48+00:00

Now that we use a workqueue for I/O submission there is no need to use
transport_generic_handle_cdb_map any more.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Nicholas Bellinger

tcm_loop: defer all command submissions to workqueue

2012-02-25T22:37:48+00:00

Apply the qla2xxx model of submitting all commands from a workqueue.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Nicholas Bellinger

tcm_loop: kill tcm_loop_allocate_core_cmd

2012-02-25T22:37:48+00:00

This function makes little sense as a separate abstraction as it's deeply
interwinded with the control flow of its only caller.  Merged it into
tcm_loop_queuecommand after factoring out a helper to convert the task
attribute representation.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Nicholas Bellinger