public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH 6/9] lpfc: Fix cq_id masking problem.
@ 2015-05-22 14:42 James Smart
  2015-05-24 20:24 ` Sebastian Herbszt
  2015-05-25 15:19 ` Hannes Reinecke
  0 siblings, 2 replies; 3+ messages in thread
From: James Smart @ 2015-05-22 14:42 UTC (permalink / raw)
  To: linux-scsi


Fix cq_id masking problem.

The driver inadvertainly constrained the id space to 8 bits, when its
a full 16 bits from hw. This resulted in bad id's being written to the
hardware. Resulted in symptoms such as incomplete initialization, no
detection of link events, etc.

Signed-off-by: Dick Kennedy <dick.kennedy@avagotech.com>
Signed-off-by: James Smart <james.smart@avagotech.com>
---
 drivers/scsi/lpfc/lpfc_hw4.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/lpfc/lpfc_hw4.h b/drivers/scsi/lpfc/lpfc_hw4.h
index b94fa46..33ec4fa 100644
--- a/drivers/scsi/lpfc/lpfc_hw4.h
+++ b/drivers/scsi/lpfc/lpfc_hw4.h
@@ -291,7 +291,7 @@ struct sli4_bls_rsp {
 struct lpfc_eqe {
 	uint32_t word0;
 #define lpfc_eqe_resource_id_SHIFT	16
-#define lpfc_eqe_resource_id_MASK	0x000000FF
+#define lpfc_eqe_resource_id_MASK	0x0000FFFF
 #define lpfc_eqe_resource_id_WORD	word0
 #define lpfc_eqe_minor_code_SHIFT	4
 #define lpfc_eqe_minor_code_MASK	0x00000FFF
-- 
1.7.11.7


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 6/9] lpfc: Fix cq_id masking problem.
  2015-05-22 14:42 [PATCH 6/9] lpfc: Fix cq_id masking problem James Smart
@ 2015-05-24 20:24 ` Sebastian Herbszt
  2015-05-25 15:19 ` Hannes Reinecke
  1 sibling, 0 replies; 3+ messages in thread
From: Sebastian Herbszt @ 2015-05-24 20:24 UTC (permalink / raw)
  To: James Smart; +Cc: linux-scsi, Sebastian Herbszt

James Smart wrote:
> 
> Fix cq_id masking problem.
> 
> The driver inadvertainly constrained the id space to 8 bits, when its

inadvertently, it's

> a full 16 bits from hw. This resulted in bad id's being written to the
> hardware. Resulted in symptoms such as incomplete initialization, no
> detection of link events, etc.

Sebastian

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 6/9] lpfc: Fix cq_id masking problem.
  2015-05-22 14:42 [PATCH 6/9] lpfc: Fix cq_id masking problem James Smart
  2015-05-24 20:24 ` Sebastian Herbszt
@ 2015-05-25 15:19 ` Hannes Reinecke
  1 sibling, 0 replies; 3+ messages in thread
From: Hannes Reinecke @ 2015-05-25 15:19 UTC (permalink / raw)
  To: James Smart, linux-scsi

On 05/22/2015 04:42 PM, James Smart wrote:
> 
> Fix cq_id masking problem.
> 
> The driver inadvertainly constrained the id space to 8 bits, when its
> a full 16 bits from hw. This resulted in bad id's being written to the
> hardware. Resulted in symptoms such as incomplete initialization, no
> detection of link events, etc.
> 
> Signed-off-by: Dick Kennedy <dick.kennedy@avagotech.com>
> Signed-off-by: James Smart <james.smart@avagotech.com>
> ---
Reviewed-by: Hannes Reinecke <hare@suse.de>

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
--
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

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-05-25 15:19 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-05-22 14:42 [PATCH 6/9] lpfc: Fix cq_id masking problem James Smart
2015-05-24 20:24 ` Sebastian Herbszt
2015-05-25 15:19 ` Hannes Reinecke

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox