> 
> This is very helpful to match up the corresponding exchange to the actual
> I/O
> described by the fsp, particularly when you do a side-by-side comparison
> of
> the syslog with your trace.
> 
> Signed-off-by: Yi Zou <[email protected]>

Won't say this one this a bug-fix as I was saying in the cover, but it's really
Nice to have this when debugging on fcp layer for the i/o.

Yi

> ---
> 
>  drivers/scsi/libfc/fc_libfc.h |   16 ++++++++++++++--
>  1 files changed, 14 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/scsi/libfc/fc_libfc.h b/drivers/scsi/libfc/fc_libfc.h
> index 16d2162..eea0c35 100644
> --- a/drivers/scsi/libfc/fc_libfc.h
> +++ b/drivers/scsi/libfc/fc_libfc.h
> @@ -66,9 +66,21 @@ extern unsigned int fc_debug_logging;
> 
>  #define FC_FCP_DBG(pkt, fmt, args...)                                        
> \
>       FC_CHECK_LOGGING(FC_FCP_LOGGING,                                \
> -                      printk(KERN_INFO "host%u: fcp: %6.6x: " fmt,   \
> +     {                                                               \
> +             if ((pkt)->seq_ptr) {                                   \
> +                     struct fc_exch *_ep = NULL;                     \
> +                     _ep = fc_seq_exch((pkt)->seq_ptr);              \
> +                     printk(KERN_INFO "host%u: fcp: %6.6x: "         \
> +                             "xid %04x-%04x: " fmt,                  \
>                               (pkt)->lp->host->host_no,               \
> -                             pkt->rport->port_id, ##args))
> +                             (pkt)->rport->port_id,                  \
> +                             (_ep)->oxid, (_ep)->rxid, ##args);      \
> +             } else {                                                \
> +                     printk(KERN_INFO "host%u: fcp: %6.6x: " fmt,    \
> +                             (pkt)->lp->host->host_no,               \
> +                             (pkt)->rport->port_id, ##args);         \
> +             }                                                       \
> +     })
> 
>  #define FC_EXCH_DBG(exch, fmt, args...)                                      
> \
>       FC_CHECK_LOGGING(FC_EXCH_LOGGING,                               \
> 
> _______________________________________________
> devel mailing list
> [email protected]
> http://www.open-fcoe.org/mailman/listinfo/devel
_______________________________________________
devel mailing list
[email protected]
http://www.open-fcoe.org/mailman/listinfo/devel

Reply via email to