Patchwork [Open-FCoE,2/4] libfc: Differentiate echange timer cancellation debug statements

login
register
mail settings
Submitter Robert Love
Date June 10, 2013, 11:28 p.m.
Message ID <20130610232809.26115.29116.stgit@fritz>
Download mbox | patch
Permalink /patch/59/
State Accepted
Headers show

Comments

Robert Love - June 10, 2013, 11:28 p.m.
There are two debug statements with the same output string regarding
echange timer cancellation. This patch simply changes the output of
one string so that they can be differentiated.

Signed-off-by: Robert Love <robert.w.love@intel.com>
Tested-by: Jack Morgan<jack.morgan@intel.com>
---
 drivers/scsi/libfc/fc_exch.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Neil Horman - June 12, 2013, 11:45 a.m.
On Mon, Jun 10, 2013 at 04:28:09PM -0700, Robert Love wrote:
> There are two debug statements with the same output string regarding
> echange timer cancellation. This patch simply changes the output of
> one string so that they can be differentiated.
> 
> Signed-off-by: Robert Love <robert.w.love@intel.com>
> Tested-by: Jack Morgan<jack.morgan@intel.com>
> ---
>  drivers/scsi/libfc/fc_exch.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/libfc/fc_exch.c b/drivers/scsi/libfc/fc_exch.c
> index c0bd758..0316cbf 100644
> --- a/drivers/scsi/libfc/fc_exch.c
> +++ b/drivers/scsi/libfc/fc_exch.c
> @@ -1556,7 +1556,7 @@ static void fc_exch_abts_resp(struct fc_exch *ep, struct fc_frame *fp)
>  		    fc_exch_rctl_name(fh->fh_r_ctl));
>  
>  	if (cancel_delayed_work_sync(&ep->timeout_work)) {
> -		FC_EXCH_DBG(ep, "Exchange timer canceled\n");
> +		FC_EXCH_DBG(ep, "Exchange timer canceled due to ABTS response\n");
>  		fc_exch_release(ep);	/* release from pending timer hold */
>  	}
>  
> 
Acked-by: Neil Horman <nhorman@tuxdriver.com>

Patch

diff --git a/drivers/scsi/libfc/fc_exch.c b/drivers/scsi/libfc/fc_exch.c
index c0bd758..0316cbf 100644
--- a/drivers/scsi/libfc/fc_exch.c
+++ b/drivers/scsi/libfc/fc_exch.c
@@ -1556,7 +1556,7 @@  static void fc_exch_abts_resp(struct fc_exch *ep, struct fc_frame *fp)
 		    fc_exch_rctl_name(fh->fh_r_ctl));
 
 	if (cancel_delayed_work_sync(&ep->timeout_work)) {
-		FC_EXCH_DBG(ep, "Exchange timer canceled\n");
+		FC_EXCH_DBG(ep, "Exchange timer canceled due to ABTS response\n");
 		fc_exch_release(ep);	/* release from pending timer hold */
 	}