On Mon, 2011-02-14 at 20:09 +0100, Fubo Chen wrote:
> Does this patch make sense ?
>
> ---
> drivers/target/target_core_transport.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
Hi Fubo,
> diff --git a/drivers/target/target_core_transport.c
> b/drivers/target/target_core_transport.c
> index 236e22d..d0e1ce8 100644
> --- a/drivers/target/target_core_transport.c
> +++ b/drivers/target/target_core_transport.c
> @@ -5828,14 +5828,12 @@ void transport_send_task_abort(struct se_cmd *cmd)
> */
> int transport_generic_do_tmr(struct se_cmd *cmd)
> {
> - struct se_cmd *ref_cmd;
> struct se_device *dev = SE_DEV(cmd);
> struct se_tmr_req *tmr = cmd->se_tmr_req;
> int ret;
>
> switch (tmr->function) {
> case ABORT_TASK:
> - ref_cmd = tmr->ref_cmd;
> tmr->response = TMR_FUNCTION_REJECTED;
> break;
> case ABORT_TASK_SET:
The usage of ref_cmd here will be used in for-39 code, so this should be
kept in place for now..
Thanks,
--nab
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
