From owner-freebsd-net@FreeBSD.ORG Wed Nov 17 17:39:16 2010 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 3AF481065912 for ; Wed, 17 Nov 2010 17:39:16 +0000 (UTC) (envelope-from jfvogel@gmail.com) Received: from mail-wy0-f182.google.com (mail-wy0-f182.google.com [74.125.82.182]) by mx1.freebsd.org (Postfix) with ESMTP id BA7728FC12 for ; Wed, 17 Nov 2010 17:39:15 +0000 (UTC) Received: by wyb35 with SMTP id 35so1385066wyb.13 for ; Wed, 17 Nov 2010 09:39:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type; bh=SGYCLIKOKGdXupJAAKLPhQWmUMDTFQv9A5M3nAyJ4Ek=; b=toCIvysM+WOeGag/cBjhCx9yprbXQgkJxdlER/sYm+osz2hyVCYwNp/CAbDJkDRwOs vVZBlV80Nl6fWLH0FgF4E8rlt1CCD57h+FKe9R06JkVCG3MZtvtEf7lNmm2DOAnz+TSa LaBp49DismOMJkbMO6D8KGt66J2jKsKBcILFA= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=lWEAby9fBPTBrUzMdHVCPcpM2e/cnZeHOQCH939i3LVtSE1BtXJ0Zg6WUSu7j8Cy6U Wq0mjBfX3KMwFdgU5pGqRmejZ/5BN/WWSNYJ3HCwhHirtDxZPoBIkxBmco/n5YpMSLgA vRuH51LQ08b+I9YKV+cDNJogIYe8EI3GFwEX8= MIME-Version: 1.0 Received: by 10.216.179.210 with SMTP id h60mr8365046wem.42.1290015554233; Wed, 17 Nov 2010 09:39:14 -0800 (PST) Received: by 10.216.2.206 with HTTP; Wed, 17 Nov 2010 09:39:14 -0800 (PST) In-Reply-To: References: <20101117070422.GA45678@cabstand.com> <4CE3D097.7030204@grosbein.pp.ru> Date: Wed, 17 Nov 2010 09:39:14 -0800 Message-ID: From: Jack Vogel To: Vlad Galu Content-Type: text/plain; charset=ISO-8859-1 X-Content-Filtered-By: Mailman/MimeDel 2.1.5 Cc: freebsd-net@freebsd.org Subject: Re: request for MFC of em/igb drivers X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Nov 2010 17:39:16 -0000 Yes, everyone, I plan on updating all the drivers, there has been no activity because I've tracking down a couple bugs that are tough, involving days of testing to reproduce. I know we're getting close and I appreciate any reports like this before. Stay tuned.... Jack On Wed, Nov 17, 2010 at 5:22 AM, Vlad Galu wrote: > On Wed, Nov 17, 2010 at 1:54 PM, Eugene Grosbein > wrote: > > On 17.11.2010 13:04, Chris Peiffer wrote: > >> > >> Hi, > >> > >> I've been watching the traffic here over the last few months relating > >> to the em and igb Intel ethernet drivers. It seems like there's a big > >> consensus that HEAD has some good new fixes. > >> > >> Looking at this: > >> > >> http://www.freebsd.org/cgi/cvsweb.cgi/src/sys/dev/e1000/if_em.c > >> > >> There's a pretty big commit to HEAD in rev 1.58 on Sep 27 that's > >> marked "MFC: a week" but it doesn't look like anything's been MFC'd > >> since. (And several other revisions that look more experimental have > >> since gone into HEAD.) > >> > >> We've been seeing some weird issues with em devices under high load > >> and if these changes in 1.58 are ready for STABLE we'd love to test > >> them live. > >> > >> So this is a beg... if the relevant committers are out here, can that > >> MFC go through soon? Or if it can't, or I'm reading the logs wrong, > >> please explain. > >> > >> Thanks in advance. > > > > Indeed, in RELENG_8 em(4) is not useable for us because of two flaws: > > > > 1. It panices system due to NULL pointers dereference. > > It needs the following patch taken from CURRENT's igb(4): > > > > --- if_em.c.orig 2010-11-02 15:45:56.000000000 +0600 > > +++ if_em.c 2010-11-08 14:24:46.000000000 +0600 > > @@ -4181,9 +4181,11 @@ > > ifp->if_ierrors++; > > /* Reuse loaded DMA map and just update mbuf chain > */ > > mp = rxr->rx_buffers[i].m_head; > > + if(mp) { > > mp->m_len = mp->m_pkthdr.len = MCLBYTES; > > mp->m_data = mp->m_ext.ext_buf; > > mp->m_next = NULL; > > + } > > if (adapter->max_frame_size <= > > (MCLBYTES - ETHER_ALIGN)) > > m_adj(mp, ETHER_ALIGN); > > > > igb(4) in RELENG_8 has not this fix too and there is a PR: > > http://www.freebsd.org/cgi/query-pr.cgi?pr=150920 > > > > 2. It makes physical link down/up for every vlan creation/deletion > > that leads to 3 seconds of service interruption and > > down/up events for all other vlans based on same parent interface. > > It needs the following patch that's already present in CURRENT > > but not in RELENG_8: > > > > @@ -4342,7 +4344,8 @@ > > em_shadow_vfta[index] |= (1 << bit); > > ++adapter->num_vlans; > > /* Re-init to load the changes */ > > - em_init(adapter); > > + if (ifp->if_capenable & IFCAP_VLAN_HWFILTER) > > + em_init(adapter); > > } > > > > /* > > @@ -4366,7 +4369,8 @@ > > em_shadow_vfta[index] &= ~(1 << bit); > > --adapter->num_vlans; > > /* Re-init to load the changes */ > > - em_init(adapter); > > + if (ifp->if_capenable & IFCAP_VLAN_HWFILTER) > > + em_init(adapter); > > } > > > > static void > > _______________________________________________ > > freebsd-net@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-net > > To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org" > > > > Any plans for bringing in version 2.2.3 of ixgbe(4)? > > > -- > Good, fast & cheap. Pick any two. > _______________________________________________ > freebsd-net@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-net > To unsubscribe, send any mail to "freebsd-net-unsubscribe@freebsd.org" >