Re: [PATCH] tcm_fc: Add unlikely for timeout handling

["Nicholas A. Bellinger" <nab@xxxxxxxxxxxxxxx>]

On Tue, 2012-04-03 at 10:24 -0700, Mark Rustad wrote:
> Signed-off-by: Mark Rustad <mark.d.rustad@xxxxxxxxx>
> Acked-by: Kiran Patil <Kiran.patil@xxxxxxxxx>
> ---
> 
>  drivers/target/tcm_fc/tfc_cmd.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 

Applied.  Thanks MDR!

> diff --git a/drivers/target/tcm_fc/tfc_cmd.c b/drivers/target/tcm_fc/tfc_cmd.c
> index 97961a7..56f6b5b 100644
> --- a/drivers/target/tcm_fc/tfc_cmd.c
> +++ b/drivers/target/tcm_fc/tfc_cmd.c
> @@ -272,7 +272,8 @@ static void ft_recv_seq(struct fc_seq *sp, struct fc_frame *fp, void *arg)
>  	struct ft_cmd *cmd = arg;
>  	struct fc_frame_header *fh;
>  
> -	if (IS_ERR(fp)) {
> +	if (unlikely(IS_ERR(fp))) {
> +		struct fc_exch *ep = fc_seq_exch(sp);
>  		/* XXX need to find cmd if queued */
>  		cmd->seq = NULL;
>  		cmd->aborted = true;
> 
> --
> To unsubscribe from this list: send the line "unsubscribe target-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html