From owner-freebsd-current@freebsd.org Sat Jan 9 08:40:16 2016 Return-Path: Delivered-To: freebsd-current@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id E6ECBA68364 for ; Sat, 9 Jan 2016 08:40:16 +0000 (UTC) (envelope-from ohartman@zedat.fu-berlin.de) Received: from outpost1.zedat.fu-berlin.de (outpost1.zedat.fu-berlin.de [130.133.4.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id A28BE1192; Sat, 9 Jan 2016 08:40:16 +0000 (UTC) (envelope-from ohartman@zedat.fu-berlin.de) Received: from inpost2.zedat.fu-berlin.de ([130.133.4.69]) by outpost.zedat.fu-berlin.de (Exim 4.85) with esmtp (envelope-from ) id <1aHp4A-0037ZK-11>; Sat, 09 Jan 2016 09:40:06 +0100 Received: from f052010042.adsl.alicedsl.de ([78.52.10.42] helo=thor.walstatt.dynvpn.de) by inpost2.zedat.fu-berlin.de (Exim 4.85) with esmtpsa (envelope-from ) id <1aHp49-003Csd-Ng>; Sat, 09 Jan 2016 09:40:05 +0100 Date: Sat, 9 Jan 2016 09:40:00 +0100 From: "O. Hartmann" To: Eric Joyner Cc: Sean Bruno , "Pieper, Jeffrey E" , Hans Petter Selasky , freebsd-current Subject: Re: CURRENT: net/igb broken Message-ID: <20160109094000.72891d91.ohartman@zedat.fu-berlin.de> In-Reply-To: References: <20150915125618.0f752596@freyja.zeit4.iv.bundesimmobilien.de> <55F7FCB5.5090809@selasky.org> <2A35EA60C3C77D438915767F458D65688077F87B@ORSMSX111.amr.corp.intel.com> <20150917133509.6c87f5b6@freyja.zeit4.iv.bundesimmobilien.de> <20150917133712.0ffd01ee@freyja.zeit4.iv.bundesimmobilien.de> <55FC40CB.7090504@freebsd.org> <55FDA830.7030307@freebsd.org> <20150921105848.386d6ae0@freyja.zeit4.iv.bundesimmobilien.de> <20150922082323.6f861448@freyja.zeit4.iv.bundesimmobilien.de> <5601B591.6000005@freebsd.org> <20151002094721.1f890e19@freyja.zeit4.iv.bundesimmobilien.de> <560EA859.2030500@freebsd.org> <20151005072355.5bbadc0b@freyja.zeit4.iv.bundesimmobilien.de> Organization: FU Berlin X-Mailer: Claws Mail 3.13.1 (GTK+ 2.24.29; amd64-portbld-freebsd11.0) MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; boundary="Sig_/kIvam9VLACcQp6c_enfLO2e"; protocol="application/pgp-signature" X-Originating-IP: 78.52.10.42 X-ZEDAT-Hint: A X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 09 Jan 2016 08:40:17 -0000 --Sig_/kIvam9VLACcQp6c_enfLO2e Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Am Fri, 08 Jan 2016 20:08:39 +0000 Eric Joyner schrieb: > Does your i210 now work with the reverted version of igb? I didn't get a > chance to follow up on this earlier. >=20 > Also, can you give us the device ID for the device? There are a couple > versions of the i210 hardware. >=20 > - Eric Not yet, since the box is in the lab and I have access to it earliest on Mo= nday - but of course, I will provide the informations as soon I have access. oh >=20 > On Sun, Oct 4, 2015 at 10:23 PM O. Hartmann > wrote: >=20 > > On Fri, 2 Oct 2015 08:52:57 -0700 > > Sean Bruno wrote: > > =20 > > > -----BEGIN PGP SIGNED MESSAGE----- > > > Hash: SHA512 > > > > > > > > > > > > On 10/02/15 00:47, O. Hartmann wrote: =20 > > > > On Thu, 01 Oct 2015 15:39:11 +0000 Eric Joyner > > > > wrote: > > > > =20 > > > >> Oliver, > > > >> > > > >> did you try Sean's suggestion? > > > >> > > > >> - Eric > > > >> > > > >> On Tue, Sep 22, 2015 at 1:10 PM Sean Bruno > > > >> wrote: > > > >> =20 > > > > > > > > > > > > On 09/21/15 23:23, O. Hartmann wrote: =20 > > > >>>>> On Mon, 21 Sep 2015 21:13:18 +0000 Eric Joyner > > > >>>>> wrote: > > > >>>>> =20 > > > >>>>>> If you do a diff between r288057 and r287761, there are > > > >>>>>> no differences between the sys/dev/e1000, sys/modules/em, > > > >>>>>> and sys/modules/igb directories. Are you sure r287761 > > > >>>>>> actually works? =20 > > > >>>>> > > > >>>>> I'm quite sure r287761 works (and r287762 doesn't), double > > > >>>>> checked this this morning again. I also checked r288093 and > > > >>>>> it is still not working. > > > >>>>> > > > >>>>> The ensure that I'm not the culprit and stupid here: > > > >>>>> > > > >>>>> I use a NanoBSD environment and the only thing that gets > > > >>>>> exchanged, is the underlying OS/OS revision. The > > > >>>>> configuration always stays the same. The base system for > > > >>>>> all of my tests is built from a clean source - (deleted > > > >>>>> obj/ dir, clean, fresh build into obj/ for every test I > > > >>>>> ran). > > > >>>>> > > > >>>>> I realised a funny thing. Playing around with > > > >>>>> enabling/disabling TSO (I have been told that could be the > > > >>>>> culprit in an earlier Email from this list) with the > > > >>>>> commend sequence: > > > >>>>> > > > >>>>> ifconfig igb1 down ifconfig igb1 -tso ifconfig igb1 up > > > >>>>> ifconfig igb1 down ifconfig igb1 tso ifconfig igb1 up . . > > > >>>>> . > > > >>>>> > > > >>>>> while a ping is pinging in the background a remote host > > > >>>>> connected to that specific interface, the ping does work > > > >>>>> for a while and dies then after a round trip of roughly 10 > > > >>>>> - 20. I can reproduce this. > > > >>>>> > > > >>>>> is that observation of any help? > > > >>>>> > > > >>>>> Regards, > > > >>>>> > > > >>>>> oh > > > >>>>> =20 > > > >>>>>> > > > >>>>>> On Mon, Sep 21, 2015 at 1:58 AM O. Hartmann > > > >>>>>> wrote: > > > >>>>>> =20 > > > >>>>>>> On Sat, 19 Sep 2015 11:23:44 -0700 Sean Bruno > > > >>>>>>> wrote: > > > >>>>>>> =20 > > > >>>>> > > > >>>>> > > > >>>>> On 09/18/15 10:20, Eric Joyner wrote: =20 > > > >>>>>>>>>> He has an i210 -- he would want to revert > > > >>>>>>>>>> e1000_i210.[ch], too. > > > >>>>>>>>>> > > > >>>>>>>>>> Sorry for the thrash Sean -- it sounds like it > > > >>>>>>>>>> would be a good idea for you should revert this > > > >>>>>>>>>> patch, and Jeff and I can go look at trying these > > > >>>>>>>>>> shared code updates and igb changes internally > > > >>>>>>>>>> again. We at Intel really could've done a better > > > >>>>>>>>>> job of making sure these changes worked across a > > > >>>>>>>>>> wider variety of devices. > > > >>>>>>>>>> > > > >>>>>>>>>> - Eric =20 > > > >>>>> > > > >>>>> I've reverted the changes to head. I'll reopen the reviews > > > >>>>> and we can proceed from there. > > > >>>>> > > > >>>>> sean > > > >>>>> > > > >>>>> =20 > > > >>>>>>>>>> > > > >>>>>>>>>> On Fri, Sep 18, 2015 at 9:50 AM Sean Bruno > > > >>>>>>>>>> > > > > >>>>>>>>>> wrote: > > > >>>>>>>>>> > > > >>>>>>>>>> =20 > > > >>>>>>>>>>> > > > >>>>>>>>>>> r287762 broke the system =20 > > > >>>>>>>>>> > > > >>>>>>>>>> > > > >>>>>>>>>> Before I revert this changeset *again* can you > > > >>>>>>>>>> test revert r287762 from if_igb.c, e1000_82575.c > > > >>>>>>>>>> and e1000_82575.h *only* > > > >>>>>>>>>> > > > >>>>>>>>>> That narrows down the change quite a bit. > > > >>>>>>>>>> > > > >>>>>>>>>> sean =20 > > > > [...] > > =20 > > > >>>>>>> =20 > > > >>>>>>>>>> =20 > > > I'm now on r288057 on that specific machine, supposedly =20 > > > >>>>>>> reverted changes that seemingly has been identified as > > > >>>>>>> the culprit. Still NO change in behaviour! > > > >>>>>>> > > > >>>>>>> r287761 works with the same configuration on igb > > > >>>>>>> (i210), any further does not. Not ping/connect from the > > > >>>>>>> outside, no ping/connect from the inside. Tried > > > >>>>>>> different protocols (SAMBA, ssh, LDAP, DNS). Affected > > > >>>>>>> is/are only boxes with the igb driver and i210 chipset > > > >>>>>>> (we do not have other chips covered by igb). > > > >>>>>>> > > > >>>>>>> Regards, Oliver =20 > > > > [...] > > =20 > > > > > > > > For my entertainment (and HPS's), can you run HEAD and revert > > > > r287775? > > > > > > > > sean =20 > > > > [...] > > =20 > > > > I did as suggested: > > > > > > > > checking out the most recent HEAD of CURRENT this morning, which > > > > is/was for me r288474. I applied then "svn merge -c -287775 .", > > > > which reverted(?) only r287775, which is something with > > > > tcp_output.c or so. I did not remember. > > > > =20 > > > > > > Thanks. This is what I intended. > > > > > > =20 > > > > I recompiled a fresh world (cleaning up /usr/obj completely by > > > > deleting the folder) and try running the target system with the > > > > created image. > > > > > > > > Result: the same as >r287761, it doesn't work. I reverted back to > > > > r287761, which works for me on the specific target hardware > > > > (Fujitsu Primergy RX 1330 M1). > > > > =20 > > > > > > What's really confusing me is that I've reverted r287762 and you are > > > still having problems. =20 > > > > It is confusing me also. I'm about to walk through the commits to check > > whether > > there is another possibility of influence - say: changes in the way thi= ngs > > work > > due to configuration et cetera. Due to the fact I use a NanoBSD image on > > that > > very specific system, the configuration always is the very same but the > > underlying OS changes with the revision. > > > > An observation I made is also very strange: on most recent CURRENT > > flapping the > > state of the igb network interface by bringing it up and down repeatedl= y, > > I get > > sometimes, not always and reproducable, a connection - pings go through > > for a > > couple of pakets, but not more than 10 in the tests I ran so far. > > =20 > > > > > > Can you set bootverbose (boot_verbose=3D"YES" in loader.conf) with the > > > current version of -CURRENT and post the dmesg somewhere for me to > > > look at? =20 > > > > Yes, of course, but in worst case I can do this not before Wednesday si= nce > > we > > have to perform some tests on that specific system today and Tuesday and > > I'm > > now with the working revision r287761. It's a bit complicated, die to t= he > > fact > > the system is isolated from the internet so far and I have to pull the > > dmesg > > and save it to a flash drive and this I have to do on-site, and I'm not > > on-site > > at the moment. > > =20 > > > > > > sean =20 > > > > Oliver > > =20 --Sig_/kIvam9VLACcQp6c_enfLO2e Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJWkMdgAAoJEOgBcD7A/5N8M8sIANEfD6Bcu8a+U/VmnT7QtoJH MzlFsZaCkWIvIGZrtWGndOV4npEpiCKTpMRqP/bKvWk9ZOTQZU8+iHCvrrlGTakN LLAqU2gahC0n40qURJs9DX6IWqDhpOF6yMIj+ArNBt6RApXH/S6zhmeIJ2hESvPx Fpkpr6i3XsGEwHJ+AtByqnooInp5qfeD82en8vhTQsMvm+aK+NfdpjwZbzl/MrlR FDhTRlUA77xL5oThQr1W47usEHxhZLW+mJwksN6K3MCWy91H5kt/tKqyYhVBRcR3 tYuGCTm3vk7Vnc2k/NPJXXR/UUa4AYYEnz7aI4lLbG9jdfheTVRXIunRubjQLhk= =kqB4 -----END PGP SIGNATURE----- --Sig_/kIvam9VLACcQp6c_enfLO2e--