On Mon, Aug 05, 2019 at 09:28:14AM +0200, Michael Braun wrote:
> Currently, there is no vlan information (e.g. when used with a vlan aware
> bridge) passed to userspache, HWHEADER will contain an 08 00 (ip) suffix
> even for tagged ip packets.
>
> Therefore, add an extra netlink attribute that passes the vlan tag to
> userspace. Userspace might need to handle PCP/DEI included in this field.
>
> Signed-off-by: Michael Braun <michael-dev@xxxxxxxxxxxxx>
> ---
> include/uapi/linux/netfilter/nfnetlink_log.h | 1 +
> net/netfilter/nf_log_common.c | 2 ++
> net/netfilter/nfnetlink_log.c | 6 ++++++
> 3 files changed, 9 insertions(+)
>
> diff --git a/include/uapi/linux/netfilter/nfnetlink_log.h b/include/uapi/linux/netfilter/nfnetlink_log.h
> index 20983cb195a0..d15f74d47f48 100644
> --- a/include/uapi/linux/netfilter/nfnetlink_log.h
> +++ b/include/uapi/linux/netfilter/nfnetlink_log.h
> @@ -54,6 +54,7 @@ enum nfulnl_attr_type {
> NFULA_HWLEN, /* hardware header length */
> NFULA_CT, /* nf_conntrack_netlink.h */
> NFULA_CT_INFO, /* enum ip_conntrack_info */
> + NFULA_VLAN_TAG, /* __u16 vlan tag */
Nitpick: Would you rename this to NFULA_VLAN_TCI? Both fields in the
vlan header refer to tag, either for protocol or control.
Thanks.
