Author: yongari
Date: Wed Jul 10 01:32:58 2013
New Revision: 253130
URL: http://svnweb.freebsd.org/changeset/base/253130

Log:
  MFC r252404:
    Fix triggering false watchdog timeout as done in bce(4) when
    controller is in PAUSE state.

Modified:
  stable/9/sys/dev/bge/if_bge.c
  stable/9/sys/dev/bge/if_bgereg.h
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/bge/if_bge.c
==============================================================================
--- stable/9/sys/dev/bge/if_bge.c       Wed Jul 10 01:30:35 2013        
(r253129)
+++ stable/9/sys/dev/bge/if_bge.c       Wed Jul 10 01:32:58 2013        
(r253130)
@@ -5296,7 +5296,7 @@ bge_start_locked(struct ifnet *ifp)
                /*
                 * Set a timeout in case the chip goes out to lunch.
                 */
-               sc->bge_timer = 5;
+               sc->bge_timer = BGE_TX_TIMEOUT;
        }
 }
 
@@ -5801,12 +5801,40 @@ static void
 bge_watchdog(struct bge_softc *sc)
 {
        struct ifnet *ifp;
+       uint32_t status;
 
        BGE_LOCK_ASSERT(sc);
 
        if (sc->bge_timer == 0 || --sc->bge_timer)
                return;
 
+       /* If pause frames are active then don't reset the hardware. */
+       if ((CSR_READ_4(sc, BGE_RX_MODE) & BGE_RXMODE_FLOWCTL_ENABLE) != 0) {
+               status = CSR_READ_4(sc, BGE_RX_STS);
+               if ((status & BGE_RXSTAT_REMOTE_XOFFED) != 0) {
+                       /*
+                        * If link partner has us in XOFF state then wait for
+                        * the condition to clear.
+                        */
+                       CSR_WRITE_4(sc, BGE_RX_STS, status);
+                       sc->bge_timer = BGE_TX_TIMEOUT;
+                       return;
+               } else if ((status & BGE_RXSTAT_RCVD_XOFF) != 0 &&
+                   (status & BGE_RXSTAT_RCVD_XON) != 0) {
+                       /*
+                        * If link partner has us in XOFF state then wait for
+                        * the condition to clear.
+                        */
+                       CSR_WRITE_4(sc, BGE_RX_STS, status);
+                       sc->bge_timer = BGE_TX_TIMEOUT;
+                       return;
+               }
+               /*
+                * Any other condition is unexpected and the controller
+                * should be reset.
+                */
+       }
+
        ifp = sc->bge_ifp;
 
        if_printf(ifp, "watchdog timeout -- resetting\n");

Modified: stable/9/sys/dev/bge/if_bgereg.h
==============================================================================
--- stable/9/sys/dev/bge/if_bgereg.h    Wed Jul 10 01:30:35 2013        
(r253129)
+++ stable/9/sys/dev/bge/if_bgereg.h    Wed Jul 10 01:32:58 2013        
(r253130)
@@ -2928,6 +2928,7 @@ struct bge_dmamap_arg {
 #define        BGE_HWREV_TIGON_II      0x02
 #define        BGE_TIMEOUT             100000
 #define        BGE_TXCONS_UNSET                0xFFFF  /* impossible value */
+#define        BGE_TX_TIMEOUT          5
 
 struct bge_bcom_hack {
        int                     reg;
_______________________________________________
svn-src-stable-9@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-stable-9
To unsubscribe, send any mail to "svn-src-stable-9-unsubscr...@freebsd.org"

Reply via email to