netdev - Re: [PATCH v2 1/2] tools/include: Update if

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <e126f901-f3ce-1325-b3c1-9d325bbc8249@iogearbox.net>
Date:   Mon, 27 Sep 2021 19:02:09 +0200
From:   Daniel Borkmann <daniel@...earbox.net>
To:     Ian Rogers <irogers@...gle.com>,
        Alexei Starovoitov <ast@...nel.org>,
        Martin KaFai Lau <kafai@...com>,
        Song Liu <songliubraving@...com>, Yonghong Song <yhs@...com>,
        Andrii Nakryiko <andrii@...nel.org>,
        John Fastabend <john.fastabend@...il.com>,
        KP Singh <kpsingh@...omium.org>,
        "David S. Miller" <davem@...emloft.net>,
        Igor Russkikh <irusskikh@...vell.com>,
        Mark Starovoytov <mstarovoitov@...vell.com>,
        Horatiu Vultur <horatiu.vultur@...rochip.com>,
        Antoine Tenart <antoine.tenart@...tlin.com>,
        Toke Høiland-Jørgensen <toke@...hat.com>,
        Nikolay Aleksandrov <nikolay@...ulusnetworks.com>,
        Roopa Prabhu <roopa@...ulusnetworks.com>,
        Jiri Pirko <jiri@...lanox.com>,
        Vlad Buslov <vladbu@...lanox.com>,
        Ido Schimmel <idosch@...lanox.com>,
        Alexandre Cassen <acassen@...il.com>,
        linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
        bpf@...r.kernel.org
Subject: Re: [PATCH v2 1/2] tools/include: Update if_link.h and netlink.h

On 9/23/21 12:29 AM, Ian Rogers wrote:
> Sync the uAPI headers so that userspace and the kernel match. These
> changes make the tools version match the updates to the files in the
> kernel directory that were updated by commits:
> 
> if_link.h:
> 8f4c0e01789c hsr: enhance netlink socket interface to support PRP
> 427f0c8c194b macvlan: Add nodst option to macvlan type source
> 583be982d934 mctp: Add device handling and netlink interface
> c7fa1d9b1fb1 net: bridge: mcast: dump ipv4 querier state
> 2dba407f994e net: bridge: multicast: make tracked EHT hosts limit configurable
> b6e5d27e32ef net: ethernet: rmnet: Add support for MAPv5 egress packets
> 14452ca3b5ce net: qualcomm: rmnet: Export mux_id and flags to netlink
> 78a3ea555713 net: remove comments on struct rtnl_link_stats
> 0db0c34cfbc9 net: tighten the definition of interface statistics
> 571912c69f0e net: UDP tunnel encapsulation module for tunnelling different protocols like MPLS, IP, NSH etc.
> 00e77ed8e64d rtnetlink: add IFLA_PARENT_[DEV|DEV_BUS]_NAME
> 829eb208e80d rtnetlink: add support for protodown reason
> 
> netlink.h:
> d07dcf9aadd6 netlink: add infrastructure to expose policies to userspace
> 44f3625bc616 netlink: export policy in extended ACK
> d409989b59ad netlink: simplify NLMSG_DATA with NLMSG_HDRLEN
> a85cbe6159ff uapi: move constants from <linux/kernel.h> to <linux/const.h>
> 
> v2. Is a rebase and sync to the latest versions. A list of changes
>      computed via diff and blame was added to the commit message as suggested
>      in:
> https://lore.kernel.org/lkml/20201015223119.1712121-1-irogers@google.com/
> 
> Signed-off-by: Ian Rogers <irogers@...gle.com>

With both patches applied to bpf-next, this would break our CI:

[...]
   CC       bench.o
   CC       bench_count.o
   CC       bench_rename.o
   CC       bench_trigger.o
   CC       bench_ringbufs.o
   BINARY   bench
   BINARY   xdpxceiver
xdpxceiver.c: In function ‘testapp_invalid_desc’:
xdpxceiver.c:1223:41: warning: implicit declaration of function ‘ARRAY_SIZE’ [-Wimplicit-function-declaration]
  1223 |  pkt_stream_generate_custom(test, pkts, ARRAY_SIZE(pkts));
       |                                         ^~~~~~~~~~
/usr/bin/ld: /tmp/ccoQONpi.o: in function `testapp_invalid_desc':
/root/daniel/bpf/tools/testing/selftests/bpf/xdpxceiver.c:1223: undefined reference to `ARRAY_SIZE'
collect2: error: ld returned 1 exit status
make: *** [Makefile:166: /root/daniel/bpf/tools/testing/selftests/bpf/xdpxceiver] Error 1

Please take a look and submit v3 of the series with the build issue fixed.

Thanks,
Daniel