Date: Thu, 16 Mar 2017 17:57:53 -0700 From: Cy Schubert <Cy.Schubert@komquats.com> To: Eric Joyner <erj@freebsd.org> Cc: Ngie Cooper <yaneurabeya@gmail.com>, Cy Schubert <Cy.Schubert@komquats.com>, src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org Subject: Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv Message-ID: <201703170057.v2H0vr29087029@slippy.cwsent.com> In-Reply-To: Message from Eric Joyner <erj@freebsd.org> of "Thu, 16 Mar 2017 18:59:31 -0000." <CA%2Bb0zg-Kw8WAUacohh9iHzdLDmb-ZQemJHJFq1qbeR-kZ-aerw@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
In message <CA+b0zg-Kw8WAUacohh9iHzdLDmb-ZQemJHJFq1qbeR-kZ-aerw@mail.gmail.c om> , Eric Joyner writes: > --94eb2c18d092e83944054adda9b6 > Content-Type: text/plain; charset=UTF-8 > > Ngie, > > It looks like what's in the commit is using an older version of netmap, so > I think we're okay on that. I think the naming conflict may be due to > splitting the ixgbe_netmap.c file into ixgbe_netmap.c & ixv_netmap.c, and > not having the functions in ixv_netmap.c be renamed. > > I don't think the PCI_IOV stuff should be left in there. At least from what > I gathered in D4767, it's probably abandoned in 10. > > Cy, > > Are you using special options in your kernel config file? "device ixgbe" > works for me, so I want to assume you have "device netmap" in your config > file, unless you're missing INET and/or INET6. > > On Thu, Mar 16, 2017 at 10:00 AM Ngie Cooper <yaneurabeya@gmail.com> wrote: > > > > > On Mar 16, 2017, at 09:29, Eric Joyner <erj@freebsd.org> wrote: > > > > I'm looking at this now. > > > > Cy, in your case it sounds like the "device ixgbe" option is broken. I'll > > work on getting that to build again. I don't specify ix or ixgbe. Just a module build. I disconnected it to allow the build to finish. > > > > The LINT build error looks like a netmap linking issue (probably due to > > Jeb moving/copying netmap stuff into the driver folder), so that'll be fun > > to root cause. > > > > > > Hi Eric, > > One sandtrap to be aware of is r307394 (and corresponding commits) > > not being MFCed, along with a host of PCI-IOV, etc changes not being easily > > backportable. > > If you can, look for commits from me in the r29xxxx series citing > > ix_crcstrip (it was a backport from bz@ to unbreak LINT on head that also > > fixed stable/10). > > Thanks! > > -Ngie > > > > On Thu, Mar 16, 2017 at 7:10 AM Cy Schubert <Cy.Schubert@komquats.com> > > wrote: > > > > In message <62C8A9FD-1EE0-4DEF-A28A-843DBBB4972A@gmail.com>, "Ngie Cooper > > (yane > > urabeya)" writes: > > > > > > --Apple-Mail=_BD9D96B3-A5E1-4DF6-8A1A-EEC15F03AFC5 > > > Content-Transfer-Encoding: quoted-printable > > > Content-Type: text/plain; > > > charset=us-ascii > > > > > > > > > > On Mar 15, 2017, at 14:20, Eric Joyner <erj@FreeBSD.org> wrote: > > > >=20 > > > > Author: erj > > > > Date: Wed Mar 15 21:20:17 2017 > > > > New Revision: 315333 > > > > URL: https://svnweb.freebsd.org/changeset/base/315333 > > > >=20 > > > > Log: > > > > ixgbe(4): Update to 3.2.11-k > > > >=20 > > > > Includes: > > > >=20 > > > > 1. Support for X553 (Denverton) Ethernet devices. > > > > 2. Initialization code refactoring. > > > > 3. ixgbe-specific netmap code moved to ixgbe folder > > > > (but not removed from dev/netmap/). > > > > 4. VF driver full split. Some cleanup can be had due to redundant = > > > code, > > > > but this split goes inline with the refactoring. > > > >=20 > > > > NOTE: This is a direct commit to stable/10. Newer branches will = > > > receive an > > > > iflib-converted version of ixgbe, with these updates included, = > > > instead. > > > >=20 > > > > Differential Revision: https://reviews.freebsd.org/D9310 > > > > Submitted by: Jeb Cramer (jeb.j.cramer@intel.com) > > > > Tested by: Jeffrey Pieper (jeffrey.e.pieper@intel.com) > > > > Sponsored by: Intel Corporation > > > > > > Hi Eric, > > > This commit broke amd64/LINT, further (there are other issues = > > > preventing it from passing): = > > > https://ci.freebsd.org/job/FreeBSD-stable-10-amd64-LINT/543/console . > > > > It also broke buildkernel while building the ixgbe module in my > > stable10/amd64 VM. > > > > > > -- > > Cheers, > > Cy Schubert <Cy.Schubert@cschubert.com> > > FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org > > > > The need of the many outweighs the greed of the few. > > > > > > -- Cheers, Cy Schubert <Cy.Schubert@cschubert.com> FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org The need of the many outweighs the greed of the few.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201703170057.v2H0vr29087029>