lkml.org 
[lkml]   [2015]   [Jun]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.4 121/172] tcm_fc: missing curly braces in ft_invl_hw_context()
    Date
    From: Dan Carpenter <dan.carpenter@oracle.com>

    3.4.108-rc1 review patch. If anyone has any objections, please let me know.

    ------------------


    commit d556546e7ecd9fca199df4698943024d40044f8e upstream.

    This patch adds a missing set of conditional check braces in
    ft_invl_hw_context() originally introduced by commit dcd998ccd
    when handling DDP failures in ft_recv_write_data() code.

    commit dcd998ccdbf74a7d8fe0f0a44e85da1ed5975946
    Author: Kiran Patil <kiran.patil@intel.com>
    Date: Wed Aug 3 09:20:01 2011 +0000

    tcm_fc: Handle DDP/SW fc_frame_payload_get failures in ft_recv_write_data

    Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
    Cc: Kiran Patil <kiran.patil@intel.com>
    Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
    Signed-off-by: Zefan Li <lizefan@huawei.com>
    ---
    drivers/target/tcm_fc/tfc_io.c | 3 ++-
    1 file changed, 2 insertions(+), 1 deletion(-)

    diff --git a/drivers/target/tcm_fc/tfc_io.c b/drivers/target/tcm_fc/tfc_io.c
    index dc7c0db..8d5cf53 100644
    --- a/drivers/target/tcm_fc/tfc_io.c
    +++ b/drivers/target/tcm_fc/tfc_io.c
    @@ -330,7 +330,7 @@ void ft_invl_hw_context(struct ft_cmd *cmd)
    ep = fc_seq_exch(seq);
    if (ep) {
    lport = ep->lp;
    - if (lport && (ep->xid <= lport->lro_xid))
    + if (lport && (ep->xid <= lport->lro_xid)) {
    /*
    * "ddp_done" trigger invalidation of HW
    * specific DDP context
    @@ -345,6 +345,7 @@ void ft_invl_hw_context(struct ft_cmd *cmd)
    * identified using ep->xid)
    */
    cmd->was_ddp_setup = 0;
    + }
    }
    }
    }
    --
    1.9.1


    \
     
     \ /
      Last update: 2015-06-16 11:41    [W:4.129 / U:0.036 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site