Author: np
Date: Tue Mar 10 21:44:20 2020
New Revision: 358859
URL: https://svnweb.freebsd.org/changeset/base/358859

Log:
  cxgbe(4): Do not try to use 0 as an rx buffer address when the driver is
  already allocating from the safe zone and the allocation fails.
  
  This bug was introduced in r357481.
  
  MFC after:    3 days
  Sponsored by: Chelsio Communications

Modified:
  head/sys/dev/cxgbe/t4_sge.c

Modified: head/sys/dev/cxgbe/t4_sge.c
==============================================================================
--- head/sys/dev/cxgbe/t4_sge.c Tue Mar 10 20:30:21 2020        (r358858)
+++ head/sys/dev/cxgbe/t4_sge.c Tue Mar 10 21:44:20 2020        (r358859)
@@ -4382,10 +4382,12 @@ refill_fl(struct adapter *sc, struct sge_fl *fl, int n
                MPASS(sd->cl == NULL);
                rxb = &sc->sge.rx_buf_info[fl->zidx];
                cl = uma_zalloc(rxb->zone, M_NOWAIT);
-               if (__predict_false(cl == NULL) && fl->zidx != fl->safe_zidx) {
-                       rxb = &sc->sge.rx_buf_info[fl->safe_zidx];
-                       cl = uma_zalloc(rxb->zone, M_NOWAIT);
-                       if (__predict_false(cl == NULL))
+               if (__predict_false(cl == NULL)) {
+                       if (fl->zidx != fl->safe_zidx) {
+                               rxb = &sc->sge.rx_buf_info[fl->safe_zidx];
+                               cl = uma_zalloc(rxb->zone, M_NOWAIT);
+                       }
+                       if (cl == NULL)
                                break;
                }
                fl->cl_allocated++;
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to