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
| ||
|
Date: Tue, 24 Mar 2020 12:40:16 -0700 From: Jakub Kicinski <kuba@...nel.org> To: Toke Høiland-Jørgensen <toke@...hat.com> Cc: netdev@...r.kernel.org, bpf@...r.kernel.org, Alexei Starovoitov <ast@...nel.org>, Daniel Borkmann <daniel@...earbox.net>, Martin KaFai Lau <kafai@...com>, Song Liu <songliubraving@...com>, Yonghong Song <yhs@...com>, Andrii Nakryiko <andriin@...com>, "David S. Miller" <davem@...emloft.net>, Jesper Dangaard Brouer <brouer@...hat.com>, John Fastabend <john.fastabend@...il.com>, Lorenz Bauer <lmb@...udflare.com>, Andrey Ignatov <rdna@...com> Subject: Re: [PATCH bpf-next v3 1/4] xdp: Support specifying expected existing program when attaching XDP On Tue, 24 Mar 2020 19:12:53 +0100 Toke Høiland-Jørgensen wrote: > From: Toke Høiland-Jørgensen <toke@...hat.com> > > While it is currently possible for userspace to specify that an existing > XDP program should not be replaced when attaching to an interface, there is > no mechanism to safely replace a specific XDP program with another. > > This patch adds a new netlink attribute, IFLA_XDP_EXPECTED_ID, which can be > set along with IFLA_XDP_FD. If set, the kernel will check that the program > currently loaded on the interface matches the expected one, and fail the > operation if it does not. This corresponds to a 'cmpxchg' memory operation. > Setting the new attribute with a negative value means that no program is > expected to be attached, which corresponds to setting the UPDATE_IF_NOEXIST > flag. > > A new companion flag, XDP_FLAGS_EXPECT_ID, is also added to explicitly > request checking of the EXPECTED_ID attribute. This is needed for userspace > to discover whether the kernel supports the new attribute. > > Signed-off-by: Toke Høiland-Jørgensen <toke@...hat.com> Reviewed-by: Jakub Kicinski <kuba@...nel.org> Thanks!
Powered by blists - more mailing lists