Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 2 Sep 2025 22:52:14 +0800
From:      Zhenlei Huang <zlei@FreeBSD.org>
To:        "src-committers@freebsd.org" <src-committers@FreeBSD.org>, "dev-commits-src-all@freebsd.org" <dev-commits-src-all@FreeBSD.org>, "dev-commits-src-branches@freebsd.org" <dev-commits-src-branches@FreeBSD.org>
Cc:        Justin Hibbits <jhibbits@freebsd.org>
Subject:   Re: git: 135478abe3bc - stable/14 - ifnet: Don't include ethernet.h twice
Message-ID:  <1D876234-4AC4-4AD0-9ED1-CECC9BEE8DB7@FreeBSD.org>
In-Reply-To: <202509011336.581Dah0q051386@gitrepo.freebsd.org>
References:  <202509011336.581Dah0q051386@gitrepo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Oops, this breaks LINT-NOIP build. The if_etherbpfmtap() in if.c =
requires ETHER_BPF_MTAP which is defined in ethernet.h .

Hi Justin,

I'd like to MFC commit [1] rather than reverting this, since the =
preferred ones bpf_mtap_if() and ether_bpf_mtap_if() have been in 14.0 =
[2].

Any objection ?

1. 2a3716432d20 IfAPI: Retire if_etherbpfmtap() and if_bpfmtap()
2. 950cc1f44fbd bpf: Add "_if" tap APIs

Best regards,
Zhenlei

> On Sep 1, 2025, at 9:36 PM, Zhenlei Huang <zlei@FreeBSD.org> wrote:
>=20
> The branch stable/14 has been updated by zlei:
>=20
> URL: =
https://cgit.FreeBSD.org/src/commit/?id=3D135478abe3bcac1c5195b7e4d75e9f2d=
6ab40221
>=20
> commit 135478abe3bcac1c5195b7e4d75e9f2d6ab40221
> Author:     Zhenlei Huang <zlei@FreeBSD.org>
> AuthorDate: 2025-08-28 04:29:06 +0000
> Commit:     Zhenlei Huang <zlei@FreeBSD.org>
> CommitDate: 2025-09-01 13:29:42 +0000
>=20
>    ifnet: Don't include ethernet.h twice
>=20
>    MFC after:      3 days
>=20
>    (cherry picked from commit =
393356f25fb8b76e38b10347f0ad40d4a23372ba)
> ---
> sys/net/if.c | 1 -
> 1 file changed, 1 deletion(-)
>=20
> diff --git a/sys/net/if.c b/sys/net/if.c
> index 39854489f5f7..07369c4512cf 100644
> --- a/sys/net/if.c
> +++ b/sys/net/if.c
> @@ -76,7 +76,6 @@
> #include <vm/uma.h>
>=20
> #include <net/bpf.h>
> -#include <net/ethernet.h>
> #include <net/if.h>
> #include <net/if_arp.h>
> #include <net/if_clone.h>






Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1D876234-4AC4-4AD0-9ED1-CECC9BEE8DB7>