Commit b63dadd6 authored by Daniel Borkmann's avatar Daniel Borkmann Committed by Martin KaFai Lau
Browse files

bpf, tcx: Get rid of tcx_link_const 



Small clean up to get rid of the extra tcx_link_const() and only retain
the tcx_link().

Signed-off-by: default avatarDaniel Borkmann <daniel@iogearbox.net>
Link: https://lore.kernel.org/r/20231023185015.21152-1-daniel@iogearbox.net


Signed-off-by: default avatarMartin KaFai Lau <martin.lau@kernel.org>
parent 69a19170

include/net/tcx.h

+1 −6
Original line number Diff line number Diff line
@@ -38,16 +38,11 @@ static inline struct tcx_entry *tcx_entry(struct bpf_mprog_entry *entry) 
	return container_of(bundle, struct tcx_entry, bundle);

}



static inline struct tcx_link *tcx_link(struct bpf_link *link)

static inline struct tcx_link *tcx_link(const struct bpf_link *link)

{

	return container_of(link, struct tcx_link, link);

}



static inline const struct tcx_link *tcx_link_const(const struct bpf_link *link)

{

	return tcx_link((struct bpf_link *)link);

}



void tcx_inc(void);

void tcx_dec(void);

kernel/bpf/tcx.c

+2 −2
Original line number Diff line number Diff line
@@ -250,7 +250,7 @@ static void tcx_link_dealloc(struct bpf_link *link) 


static void tcx_link_fdinfo(const struct bpf_link *link, struct seq_file *seq)

{

	const struct tcx_link *tcx = tcx_link_const(link);

	const struct tcx_link *tcx = tcx_link(link);

	u32 ifindex = 0;



	rtnl_lock();
@@ -267,7 +267,7 @@ static void tcx_link_fdinfo(const struct bpf_link *link, struct seq_file *seq) 
static int tcx_link_fill_info(const struct bpf_link *link,

			      struct bpf_link_info *info)

{

	const struct tcx_link *tcx = tcx_link_const(link);

	const struct tcx_link *tcx = tcx_link(link);

	u32 ifindex = 0;



	rtnl_lock();