Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-05-03 Thread Eric Joyner
This fixes the amd64 LINT build error, but not the others. I'm looking at fixing the others now. On Tue, May 2, 2017 at 3:58 PM Eric Joyner wrote: > I fixed the netmap build for amd64 at least with r317711. I'm running a > tinderbox build on it now, though, so I'll see if that fixes it. > > On F

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-05-02 Thread Eric Joyner
I fixed the netmap build for amd64 at least with r317711. I'm running a tinderbox build on it now, though, so I'll see if that fixes it. On Fri, Apr 28, 2017 at 1:40 PM Eric Joyner wrote: > I notified Jeb (the original patch submitter) about that netmap symbol > issue, but we've both been distra

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-04-28 Thread Eric Joyner
I notified Jeb (the original patch submitter) about that netmap symbol issue, but we've both been distracted with other development. I'll work on updating it. - Eric On Thu, Apr 27, 2017 at 3:06 PM Ngie Cooper wrote: > On Thu, Apr 27, 2017 at 1:07 PM, Ed Maste wrote: > > On 15 March 2017 at 17

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-04-27 Thread Ngie Cooper
On Thu, Apr 27, 2017 at 1:07 PM, Ed Maste wrote: > On 15 March 2017 at 17:20, Eric Joyner wrote: >> Author: erj >> Date: Wed Mar 15 21:20:17 2017 >> New Revision: 315333 >> URL: https://svnweb.freebsd.org/changeset/base/315333 >> >> Log: >> ixgbe(4): Update to 3.2.11-k > > This broke tinderbox

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-04-27 Thread Ed Maste
On 15 March 2017 at 17:20, Eric Joyner wrote: > Author: erj > Date: Wed Mar 15 21:20:17 2017 > New Revision: 315333 > URL: https://svnweb.freebsd.org/changeset/base/315333 > > Log: > ixgbe(4): Update to 3.2.11-k This broke tinderbox on many architectures: ia64 GENERIC and powerpc GENERIC64: /

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Cy Schubert
In message , 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_netma

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Eric Joyner
I think I've got the i386 error fixed in r315434; I have Jeb looking at fixing the netmap issue. On Thu, Mar 16, 2017 at 1:52 PM Ryan Stone wrote: > On Thu, Mar 16, 2017 at 2:59 PM, Eric Joyner wrote: > > I don't think the PCI_IOV stuff should be left in there. At least from > what I gathered i

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Ryan Stone
On Thu, Mar 16, 2017 at 2:59 PM, Eric Joyner wrote: > 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. > It shouldn't hurt to leave it in, though. It should all be compiled under #ifdef PCI_IOV and just be compiled

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Peter Jeremy
On 2017-Mar-15 21:20:17 +, Eric Joyner wrote: >Author: erj >Date: Wed Mar 15 21:20:17 2017 >New Revision: 315333 >URL: https://svnweb.freebsd.org/changeset/base/315333 > >Log: > ixgbe(4): Update to 3.2.11-k I'll add that this broke the ixv compile on my i386 VM: ===> ixv (all) cc -O2 -pipe

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Eric Joyner
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 P

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Eric Joyner
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. 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.

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Ngie Cooper
> On Mar 16, 2017, at 09:29, Eric Joyner 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. > > The LINT build error looks like a netmap linking issue (probably due to Jeb > moving/copying n

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-16 Thread Cy Schubert
In message <62c8a9fd-1ee0-4def-a28a-843dbbb49...@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 Joy

Re: svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-15 Thread Ngie Cooper (yaneurabeya)
> On Mar 15, 2017, at 14:20, Eric Joyner wrote: > > Author: erj > Date: Wed Mar 15 21:20:17 2017 > New Revision: 315333 > URL: https://svnweb.freebsd.org/changeset/base/315333 > > Log: > ixgbe(4): Update to 3.2.11-k > > Includes: > > 1. Support for X553 (Denverton) Ethernet devices. > 2.

svn commit: r315333 - in stable/10/sys: conf dev/ixgbe modules/ix modules/ixv

2017-03-15 Thread Eric Joyner
Author: erj Date: Wed Mar 15 21:20:17 2017 New Revision: 315333 URL: https://svnweb.freebsd.org/changeset/base/315333 Log: ixgbe(4): Update to 3.2.11-k Includes: 1. Support for X553 (Denverton) Ethernet devices. 2. Initialization code refactoring. 3. ixgbe-specific netmap code move