Skip to contentSkip to conversation

cxgb3: Wait longer for control packets on initialization

PATCH Applied Andre Detsch 3 participants 3 messages Apr 26, 2010 – Apr 27, 2010

Subsystems (2)
Networking Network drivers
lore
Share & tools
lore

Andre Detsch

author
#0 lore raw

PATCH 1/1 cxgb3: Wait longer for control packets on initialization

In some Power7 platforms, when using VIOS (Virtual I/O Server), we need to wait longer for control packets to finish transfer during initialization.
Without this change, initialization may fail prematurely.

Signed-off-by Wen Xiong <[email protected]>
Signed-off-by Andre Detsch <[email protected]>
drivers/net/cxgb3/cxgb3_main.c +1 -1
Index: linux-2.6.34-rc5/drivers/net/cxgb3/cxgb3_main.c
===================================================================
@@ -439,7 +439,7 @@ static void free_irq_resources(struct ad
 static int await_mgmt_replies(struct adapter *adap, unsigned long init_cnt,
 			      unsigned long n)
 {
-	int attempts = 5;
+	int attempts = 10;
 
 	while (adap->sge.qs[0].rspq.offload_pkts < init_cnt + n) {
 		if (!--attempts)
Acked-by on v1 1/1
#1 lore raw
Quoted 8 lines · Andre Detsch

In some Power7 platforms, when using VIOS (Virtual I/O Server), we need to wait longer for control packets to finish transfer during initialization.
Without this change, initialization may fail prematurely.

Signed-off-by Wen Xiong <[email protected]>
Signed-off-by Andre Detsch <[email protected]>
Acked-by Divy Le Ray <[email protected]>
Quoted 14 lines

Index: linux-2.6.34-rc5/drivers/net/cxgb3/cxgb3_main.c
===================================================================

drivers/net/cxgb3/cxgb3_main.c +1 -1
@@ -439,7 +439,7 @@ static void free_irq_resources(struct ad
 static int await_mgmt_replies(struct adapter *adap, unsigned long init_cnt,
 			      unsigned long n)
 {
-	int attempts = 5;
+	int attempts = 10;
 
 	while (adap->sge.qs[0].rspq.offload_pkts < init_cnt + n) {
 		if (!--attempts)
maintainer Applied
#2 lore raw

From: Divy Le Ray <[email protected]>
Date: Mon, 26 Apr 2010 19:16:28 -0700

Quoted 11 lines
Quoted 8 lines · Andre Detsch

In some Power7 platforms, when using VIOS (Virtual I/O Server), we need to wait longer for control packets to finish transfer during initialization.
Without this change, initialization may fail prematurely.

Signed-off-by Wen Xiong <[email protected]>
Signed-off-by Andre Detsch <[email protected]>
Acked-by Divy Le Ray <[email protected]>

Applied.