Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 24 Apr 2007 04:20:55 GMT
From:      Kip Macy <kmacy@FreeBSD.org>
To:        Perforce Change Reviews <perforce@freebsd.org>
Subject:   PERFORCE change 118700 for review
Message-ID:  <200704240420.l3O4KtU1043613@repoman.freebsd.org>

next in thread | raw e-mail | index | archive | help
http://perforce.freebsd.org/chv.cgi?CH=118700

Change 118700 by kmacy@kmacy_vt-x:opentoe_init on 2007/04/24 04:20:10

	remove bind_ithread hack now that uleintr is in
	offload_enqueue is chaining packets not mbufs
	rss_{hi,lo} have more intuitive names in cxgb

Affected files ...

.. //depot/projects/opentoe/sys/dev/cxgb/cxgb_sge.c#12 edit

Differences ...

==== //depot/projects/opentoe/sys/dev/cxgb/cxgb_sge.c#12 (text+ko) ====

@@ -2031,7 +2031,7 @@
 {
 	m->m_next = NULL;
 	if (q->rx_tail)
-		q->rx_tail->m_next = m;
+		q->rx_tail->m_nextpkt = m;
 	else {
 #ifdef notyet
 		struct sge_qset *qs = rspq_to_qset(q);
@@ -2251,23 +2251,6 @@
 	;
 }
 
-/*
- * This is an awful hack to bind the ithread to CPU 1
- * to work around lack of ithread affinity
- */
-static void
-bind_ithread(int cpu)
-{
-#if 0	
-	KASSERT(cpu < mp_ncpus, ("invalid cpu identifier"));
-	if (mp_ncpus > 1) {
-		mtx_lock_spin(&sched_lock);
-		sched_bind(curthread, cpu);
-		mtx_unlock_spin(&sched_lock);
-	}
-#endif
-}
-
 /**
  *	process_responses - process responses from an SGE response queue
  *	@adap: the adapter
@@ -2300,14 +2283,7 @@
 		printf("next_holdoff=%d\n", rspq->holdoff_tmr);
 		last_holdoff = rspq->holdoff_tmr;
 	}
-#endif	
-	if (pinned[qs->rspq.cntxt_id * adap->params.nports] == 0) {
-		/*
-		 * Assumes that cntxt_id < mp_ncpus
-		 */
-		bind_ithread(qs->rspq.cntxt_id);
-		pinned[qs->rspq.cntxt_id * adap->params.nports] = 1;
-	}
+#endif
 	rspq->next_holdoff = rspq->holdoff_tmr;
 
 	while (__predict_true(budget_left && is_new_response(r, rspq))) {
@@ -2385,8 +2361,11 @@
 				rspq->m = NULL;
 			} else {
 #ifdef notyet
-				m->pkthdr.csum_data = rss_hi;
-				m->m_priority = rss_lo;
+				m->pkthdr.csum_data = rss_csum;
+				/*
+				 * XXX size mismatch
+				 */
+				m->m_priority = rss_hash;
 				
 				rx_offload(&adap->tdev, rspq, m);
 #endif



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