Skip site navigation (1)Skip section navigation (2) 
Date:      Tue, 22 Jul 2014 02:02:39 +0000 (UTC)
From:      Navdeep Parhar <np@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r268971 - head/sys/dev/cxgbe
Message-ID:  <201407220202.s6M22d34080222@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: np
Date: Tue Jul 22 02:02:39 2014
New Revision: 268971
URL: http://svnweb.freebsd.org/changeset/base/268971

Log:
  Simplify r267600, there's no need to distinguish between allocated and
  inlined mbufs.
  
  MFC after:	1 week

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

Modified: head/sys/dev/cxgbe/adapter.h
==============================================================================
--- head/sys/dev/cxgbe/adapter.h	Tue Jul 22 00:42:55 2014	(r268970)
+++ head/sys/dev/cxgbe/adapter.h	Tue Jul 22 02:02:39 2014	(r268971)
@@ -290,8 +290,7 @@ struct cluster_metadata {
 
 struct fl_sdesc {
 	caddr_t cl;
-	uint8_t nimbuf;		/* # of inline mbufs with ref on the cluster */
-	uint8_t nembuf;		/* # of allocated mbufs with ref */
+	uint16_t nmbuf;	/* # of driver originated mbufs with ref on cluster */
 	struct cluster_layout cll;
 };
 

Modified: head/sys/dev/cxgbe/t4_sge.c
==============================================================================
--- head/sys/dev/cxgbe/t4_sge.c	Tue Jul 22 00:42:55 2014	(r268970)
+++ head/sys/dev/cxgbe/t4_sge.c	Tue Jul 22 02:02:39 2014	(r268971)
@@ -1559,7 +1559,7 @@ get_scatter_segment(struct adapter *sc, 
 		/* copy data to mbuf */
 		bcopy(payload, mtod(m, caddr_t), len);
 
-	} else if (sd->nimbuf * MSIZE < cll->region1) {
+	} else if (sd->nmbuf * MSIZE < cll->region1) {
 
 		/*
 		 * There's spare room in the cluster for an mbuf.  Create one
@@ -1567,14 +1567,14 @@ get_scatter_segment(struct adapter *sc, 
 		 */
 
 		MPASS(clm != NULL);
-		m = (struct mbuf *)(sd->cl + sd->nimbuf * MSIZE);
+		m = (struct mbuf *)(sd->cl + sd->nmbuf * MSIZE);
 		/* No bzero required */
 		if (m_init(m, NULL, 0, M_NOWAIT, MT_DATA, flags | M_NOFREE))
 			return (NULL);
 		fl->mbuf_inlined++;
 		m_extaddref(m, payload, padded_len, &clm->refcount, rxb_free,
 		    swz->zone, sd->cl);
-		sd->nimbuf++;
+		sd->nmbuf++;
 
 	} else {
 
@@ -1591,7 +1591,7 @@ get_scatter_segment(struct adapter *sc, 
 		if (clm != NULL) {
 			m_extaddref(m, payload, padded_len, &clm->refcount,
 			    rxb_free, swz->zone, sd->cl);
-			sd->nembuf++;
+			sd->nmbuf++;
 		} else {
 			m_cljset(m, sd->cl, swz->type);
 			sd->cl = NULL;	/* consumed, not a recycle candidate */
@@ -3253,7 +3253,7 @@ refill_fl(struct adapter *sc, struct sge
 
 		if (sd->cl != NULL) {
 
-			if (sd->nimbuf + sd->nembuf == 0) {
+			if (sd->nmbuf == 0) {
 				/*
 				 * Fast recycle without involving any atomics on
 				 * the cluster's metadata (if the cluster has
@@ -3312,8 +3312,7 @@ recycled:
 #endif
 			clm->refcount = 1;
 		}
-		sd->nimbuf = 0;
-		sd->nembuf = 0;
+		sd->nmbuf = 0;
 recycled_fast:
 		fl->pending++;
 		fl->needed--;
@@ -3382,7 +3381,7 @@ free_fl_sdesc(struct adapter *sc, struct
 
 		cll = &sd->cll;
 		clm = cl_metadata(sc, fl, cll, sd->cl);
-		if (sd->nimbuf + sd->nembuf == 0 ||
+		if (sd->nmbuf == 0 ||
 		    (clm && atomic_fetchadd_int(&clm->refcount, -1) == 1)) {
 			uma_zfree(sc->sge.sw_zone_info[cll->zidx].zone, sd->cl);
 		}

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201407220202.s6M22d34080222>