svn commit: r323961 - head/sys/dev/ti

Alan Cox alc at FreeBSD.org
Sun Sep 24 02:51:01 UTC 2017


Author: alc
Date: Sun Sep 24 02:50:59 2017
New Revision: 323961
URL: https://svnweb.freebsd.org/changeset/base/323961

Log:
  Since the page "frame" doesn't belong to a vm object, it can't be paged
  out.  Since it can't be paged out, it is never actually enqueued in a
  paging queue.  Nonetheless, passing PQ_INACTIVE to vm_page_unwire()
  creates the appearance that the page "frame" is being enqueued in the
  inactive queue.  As of r288122, we can avoid this false impression by
  passing PQ_NONE.
  
  MFC after:	1 week

Modified:
  head/sys/dev/ti/if_ti.c

Modified: head/sys/dev/ti/if_ti.c
==============================================================================
--- head/sys/dev/ti/if_ti.c	Sun Sep 24 00:14:48 2017	(r323960)
+++ head/sys/dev/ti/if_ti.c	Sun Sep 24 02:50:59 2017	(r323961)
@@ -1621,7 +1621,7 @@ ti_newbuf_jumbo(struct ti_softc *sc, int idx, struct m
 			}
 			sf[i] = sf_buf_alloc(frame, SFB_NOWAIT);
 			if (sf[i] == NULL) {
-				vm_page_unwire(frame, PQ_INACTIVE);
+				vm_page_unwire(frame, PQ_NONE);
 				vm_page_free(frame);
 				device_printf(sc->ti_dev, "buffer allocation "
 				    "failed -- packet dropped!\n");


More information about the svn-src-head mailing list