If there are still mbufs on a ring when we're freeing it,

it'd be a good idea to free them too.

ok dlg@

diff --git a/sys/dev/pci/if_bnxt.c b/sys/dev/pci/if_bnxt.c
index f60b3ad..dfd9392 100644 (file)
--- a/sys/dev/pci/if_bnxt.c
+++ b/sys/dev/pci/if_bnxt.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: if_bnxt.c,v 1.40 2024/01/04 07:08:47 jmatthew Exp $   */
+/*     $OpenBSD: if_bnxt.c,v 1.41 2024/01/09 04:29:46 jmatthew Exp $   */
 /*-
  * Broadcom NetXtreme-C/E network driver.
  *
@@ -742,6 +742,8 @@ bnxt_free_slots(struct bnxt_softc *sc, struct bnxt_slot *slots, int allocated,
        while (i-- > 0) {
                bs = &slots[i];
                bus_dmamap_destroy(sc->sc_dmat, bs->bs_map);
+               if (bs->bs_m != NULL)
+                       m_freem(bs->bs_m);
        }
        free(slots, M_DEVBUF, total * sizeof(*bs));
 }