Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 1 Jun 2011 20:00:25 +0000 (UTC)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r222583 - head/sys/net
Message-ID:  <201106012000.p51K0PsO084767@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rwatson
Date: Wed Jun  1 20:00:25 2011
New Revision: 222583
URL: http://svn.freebsd.org/changeset/base/222583

Log:
  Add an optional netisr dispatch point at ether_input(), but set the
  default dispatch method to NETISR_DISPATCH_DIRECT in order to force
  direct dispatch.  This adds a fairly negligble overhead without
  changing default behavior, but in the future will allow deferred or
  hybrid dispatch to other worker threads before link layer processing
  has taken place.
  
  For example, this could allow redistribution using RSS hashes
  without ethernet header cache line hits, if the NIC was unable to
  adequately implement load balancing to too small a number of input
  queues -- perhaps due to hard queueset counts of 1, 3, or 8, but in
  a modern system with 16-128 threads.  This can happen on highly
  threaded systems, where you want want an ithread per core,
  redistributing work to other queues, but also on virtualised systems
  where hardware hashing is (or is not) available, but only a single
  queue has been directed to one VCPU on a VM.
  
  Note: this adds a previously non-present assertion about the
  equivalence of the ifnet from which the packet is received, and the
  ifnet stamped in the mbuf header.  I believe this assertion to
  generally be true, but we'll find out soon -- if it's not, we might
  have to add additional overhead in some cases to add an m_tag with
  the originating ifnet pointer stored in it.
  
  Reviewed by:    bz
  MFC after:      3 weeks
  Sponsored by:   Juniper Networks, Inc.

Modified:
  head/sys/net/if_ethersubr.c

Modified: head/sys/net/if_ethersubr.c
==============================================================================
--- head/sys/net/if_ethersubr.c	Wed Jun  1 19:44:52 2011	(r222582)
+++ head/sys/net/if_ethersubr.c	Wed Jun  1 20:00:25 2011	(r222583)
@@ -561,7 +561,7 @@ ether_ipfw_chk(struct mbuf **m0, struct 
  * mbuf chain m with the ethernet header at the front.
  */
 static void
-ether_input(struct ifnet *ifp, struct mbuf *m)
+ether_input_internal(struct ifnet *ifp, struct mbuf *m)
 {
 	struct ether_header *eh;
 	u_short etype;
@@ -755,6 +755,46 @@ ether_input(struct ifnet *ifp, struct mb
 }
 
 /*
+ * Ethernet input dispatch; by default, direct dispatch here regardless of
+ * global configuration.
+ */
+static void
+ether_nh_input(struct mbuf *m)
+{
+
+	ether_input_internal(m->m_pkthdr.rcvif, m);
+}
+
+static struct netisr_handler	ether_nh = {
+	.nh_name = "ether",
+	.nh_handler = ether_nh_input,
+	.nh_proto = NETISR_ETHER,
+	.nh_policy = NETISR_POLICY_SOURCE,
+	.nh_dispatch = NETISR_DISPATCH_DIRECT,
+};
+
+static void
+ether_init(__unused void *arg)
+{
+
+	netisr_register(&ether_nh);
+}
+SYSINIT(ether, SI_SUB_INIT_IF, SI_ORDER_ANY, ether_init, NULL);
+
+static void
+ether_input(struct ifnet *ifp, struct mbuf *m)
+{
+
+	/*
+	 * We will rely on rcvif being set properly in the deferred context,
+	 * so assert it is correct here.
+	 */
+	KASSERT(m->m_pkthdr.rcvif == ifp, ("%s: ifnet mismatch", __func__));
+
+	netisr_dispatch(NETISR_ETHER, m);
+}
+
+/*
  * Upper layer processing for a received Ethernet packet.
  */
 void



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