Yeah,

Internally we have the ixl and iavf drivers split up so that they no longer
share source and have independent code bases, but that change hasn't made
it upstream yet. Doing that would fix the dependency issue, so I'll see
about doing that ASAP, possibly for 13.0. I recently came back from a long
vacation, so I have to catch up on all of these things. :(

- Eric

On Sat, Feb 6, 2021 at 10:32 AM Kyle Evans <[email protected]> wrote:

> On Sat, Feb 6, 2021 at 12:30 PM Kyle Evans <[email protected]> wrote:
> >
> > On Thu, Feb 4, 2021 at 5:34 PM Eric Joyner <[email protected]> wrote:
> > >
> > > The branch main has been updated by erj:
> > >
> > > URL:
> https://cgit.FreeBSD.org/src/commit/?id=7d4dceec103039e2b2fa90793ceeb71a8d6684aa
> > >
> > > commit 7d4dceec103039e2b2fa90793ceeb71a8d6684aa
> > > Author:     Krzysztof Galazka <[email protected]>
> > > AuthorDate: 2021-02-03 23:22:55 +0000
> > > Commit:     Eric Joyner <[email protected]>
> > > CommitDate: 2021-02-04 23:33:42 +0000
> > >
> > >     ixl(4): Fix VLAN HW filtering
> > >
> > >     X700 family of controllers has limited number of available VLAN
> > >     HW filters. Driver did not handle properly a case when user
> > >     assigned more VLANs to the interface which had all filters
> > >     already in use. Fix that by disabling HW filtering when
> > >     it is impossible to create filters for all requested VLANs.
> > >     Keep track of registered VLANs using bitstring to be able
> > >     to re-enable HW filtering when number of requested VLANs
> > >     drops below the limit.
> > >
> > >     Also switch all allocations to use M_IXL malloc type
> > >     to ease detecting memory leaks in the driver.
> > >
> >
> > Hi,
> >
> > This adds a dependency on `device ixl` for `device iavf` since M_IXL
> > is defined in ixl_pf_main.c, but I think that particular part should
> > be partially reverted anyways. There's an  M_IAVF already that's more
> > appropriate for i40e_osdep.c -- proposed patch here:
> >
> https://people.freebsd.org/~kevans/0001-ixl-switch-i40e_osdep.c-allocations-to-M_IAVF.patch
> >
>
> Ah crud, nope. I missed that i40e_osdep is pulled in for ixl as well. :-(
>
_______________________________________________
[email protected] mailing list
https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main
To unsubscribe, send any mail to "[email protected]"

Reply via email to