Re: [PATCH v2 01/11] qla2xxx: Remove unneeded link offline message.

Johannes Thumshirn
2016-01-28T08:33:49Z

On Wed, Jan 27, 2016 at 12:03:28PM -0500, Himanshu Madhani wrote:
> From: Chad Dupuis <chad.dupuis@qlogic.com>
>
> Signed-off-by: Chad Dupuis <chad.dupuis@qlogic.com>
> Signed-off-by: Himanshu Madhani <himanshu.madhani@qlogic.com>
> ---
> drivers/scsi/qla2xxx/qla_dbg.c | 1 +
> drivers/scsi/qla2xxx/qla_isr.c | 4 ----
> 2 files changed, 1 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/qla2xxx/qla_dbg.c b/drivers/scsi/qla2xxx/qla_dbg.c
> index cd0d94e..9e714cc 100644
> --- a/drivers/scsi/qla2xxx/qla_dbg.c
> +++ b/drivers/scsi/qla2xxx/qla_dbg.c
> @@ -27,6 +27,7 @@
> * | | | 0x303a |
> * | DPC Thread | 0x4023 | 0x4002,0x4013 |
> * | Async Events | 0x5089 | 0x502b-0x502f |
> + * | | | 0x505e |
> * | | | 0x5084,0x5075 |
> * | | | 0x503d,0x5044 |
> * | | | 0x507b,0x505f |
> diff --git a/drivers/scsi/qla2xxx/qla_isr.c b/drivers/scsi/qla2xxx/qla_isr.c
> index d4d65eb..edd97de 100644
> --- a/drivers/scsi/qla2xxx/qla_isr.c
> +++ b/drivers/scsi/qla2xxx/qla_isr.c
> @@ -934,10 +934,6 @@ skip_rio:
> break;
>
> global_port_update:
> - /* Port unavailable. */
> - ql_log(ql_log_warn, vha, 0x505e,
> - "Link is offline.\n");
> -
> if (atomic_read(&vha->loop_state) != LOOP_DOWN) {
> atomic_set(&vha->loop_state, LOOP_DOWN);
> atomic_set(&vha->loop_down_timer,
> --
> 1.7.7
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html

Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
--
Johannes Thumshirn Storage
jthumshirn@suse.de +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html