[PATCH 2/2] bpf: support new commands BPF_LINK_{CREATE,UPDATE}

Paul Chaignon paul at isovalent.com
Mon Apr 13 09:28:21 UTC 2020


* xlat/bpf_commands.in (BPF_LINK_CREATE): New constant introduced by Linux
commit v5.7-rc1~146^2~10^2^2~3.
(BPF_LINK_UPDATE): New constant introduced by Linux commit
v5.7-rc1~146^2~10^2^2~2.
* bpf.c (BEGIN_BPF_CMD_DECODER(BPF_LINK_CREATE),
BEGIN_BPF_CMD_DECODER(BPF_LINK_UPDATE)): Decode the new commands.
(SYS_FUNC(bpf)): Decode the new commands.
* bpf_attr.h (BPF_LINK_CREATE_struct): New structure introduced by Linux
commit v5.7-rc1~146^2~10^2^2~3.
(BPF_LINK_UPDATE_struct): New structure introduced by Linux commit
v5.7-rc1~146^2~10^2^2~2.
(BPF_LINK_CREATE_struct_size, BPF_LINK_UPDATE_struct_size): New macros.
* NEWS: Mention this.
* tests/bpf.c (BPF_LINK_CREATE_checks, BPF_LINK_UPDATE_checks): Tests for
the new commands.

Signed-off-by: Paul Chaignon <paul at cilium.io>
---
 NEWS                 |  2 ++
 bpf.c                | 23 +++++++++++++++++++++
 bpf_attr.h           | 22 ++++++++++++++++++++
 tests/bpf.c          | 48 ++++++++++++++++++++++++++++++++++++++++++++
 xlat/bpf_commands.in |  2 ++
 5 files changed, 97 insertions(+)

diff --git a/NEWS b/NEWS
index 6292e1c3..51617029 100644
--- a/NEWS
+++ b/NEWS
@@ -2,6 +2,8 @@ Noteworthy changes in release ?.? (????-??-??)
 ==============================================
 
 * Improvements
+  * Implemented decoding of BPF_LINK_CREATE and BPF_LINK_UPDATE bpf
+    syscall commands.
   * Updated lists of BPF_* constants.
 
 Noteworthy changes in release 5.6 (2020-04-07)
diff --git a/bpf.c b/bpf.c
index 0a47766e..f01a7d52 100644
--- a/bpf.c
+++ b/bpf.c
@@ -1038,6 +1038,27 @@ BEGIN_BPF_CMD_DECODER(BPF_MAP_DELETE_BATCH)
 }
 END_BPF_CMD_DECODER(0)
 
+BEGIN_BPF_CMD_DECODER(BPF_LINK_CREATE)
+{
+	PRINT_FIELD_FD("{link_create={", attr, prog_fd, tcp);
+	PRINT_FIELD_FD(", ", attr, target_fd, tcp);
+	PRINT_FIELD_XVAL(", ", attr, attach_type, bpf_attach_type, "BPF_???");
+	PRINT_FIELD_X(", ", attr, flags);
+	tprints("}");
+}
+END_BPF_CMD_DECODER(RVAL_DECODED | RVAL_FD)
+
+BEGIN_BPF_CMD_DECODER(BPF_LINK_UPDATE)
+{
+	PRINT_FIELD_FD("{link_update={", attr, link_fd, tcp);
+	PRINT_FIELD_FD(", ", attr, new_prog_fd, tcp);
+	PRINT_FIELD_FLAGS(", ", attr, flags, bpf_attach_flags,
+			  "BPF_F_???");
+	PRINT_FIELD_FD(", ", attr, old_prog_fd, tcp);
+	tprints("}");
+}
+END_BPF_CMD_DECODER(RVAL_DECODED)
+
 SYS_FUNC(bpf)
 {
 	static const bpf_cmd_decoder_t bpf_cmd_decoders[] = {
@@ -1069,6 +1090,8 @@ SYS_FUNC(bpf)
 		BPF_CMD_ENTRY(BPF_MAP_LOOKUP_AND_DELETE_BATCH),
 		BPF_CMD_ENTRY(BPF_MAP_UPDATE_BATCH),
 		BPF_CMD_ENTRY(BPF_MAP_DELETE_BATCH),
+		BPF_CMD_ENTRY(BPF_LINK_CREATE),
+		BPF_CMD_ENTRY(BPF_LINK_UPDATE),
 	};
 
 	const unsigned int cmd = tcp->u_arg[0];
diff --git a/bpf_attr.h b/bpf_attr.h
index 6b66fb06..bed011bb 100644
--- a/bpf_attr.h
+++ b/bpf_attr.h
@@ -385,4 +385,26 @@ struct BPF_MAP_LOOKUP_BATCH_struct /* batch */ {
 # define BPF_MAP_DELETE_BATCH_struct BPF_MAP_LOOKUP_BATCH_struct
 # define BPF_MAP_DELETE_BATCH_struct_size BPF_MAP_LOOKUP_BATCH_struct_size
 
+struct BPF_LINK_CREATE_struct /* link_create */ {
+	uint32_t prog_fd;
+	uint32_t target_fd;
+	uint32_t attach_type;
+	uint32_t flags;
+};
+
+# define BPF_LINK_CREATE_struct_size \
+	sizeof(struct BPF_LINK_CREATE_struct)
+# define expected_BPF_LINK_CREATE_struct_size 16
+
+struct BPF_LINK_UPDATE_struct /* link_update */ {
+	uint32_t link_fd;
+	uint32_t new_prog_fd;
+	uint32_t flags;
+	uint32_t old_prog_fd;
+};
+
+# define BPF_LINK_UPDATE_struct_size \
+	sizeof(struct BPF_LINK_UPDATE_struct)
+# define expected_BPF_LINK_UPDATE_struct_size 16
+
 #endif /* !STRACE_BPF_ATTR_H */
diff --git a/tests/bpf.c b/tests/bpf.c
index 0d45f887..54d41ffb 100644
--- a/tests/bpf.c
+++ b/tests/bpf.c
@@ -83,6 +83,8 @@ union bpf_attr_data {
 	BPF_ATTR_DATA_FIELD(BPF_MAP_LOOKUP_BATCH);
 	BPF_ATTR_DATA_FIELD(BPF_MAP_UPDATE_BATCH);
 	BPF_ATTR_DATA_FIELD(BPF_MAP_DELETE_BATCH);
+	BPF_ATTR_DATA_FIELD(BPF_LINK_CREATE);
+	BPF_ATTR_DATA_FIELD(BPF_LINK_UPDATE);
 	char char_data[256];
 };
 
@@ -1388,6 +1390,50 @@ static const struct bpf_attr_check BPF_MAP_DELETE_BATCH_checks[] = {
 	}
 };
 
+static const struct bpf_attr_check BPF_LINK_CREATE_checks[] = {
+	{
+		.data = { .BPF_LINK_CREATE_data = { .prog_fd = -1, .target_fd = -2 } },
+		.size = offsetofend(struct BPF_LINK_CREATE_struct, flags),
+		.str = "link_create={prog_fd=-1, target_fd=-2"
+		       ", attach_type=BPF_CGROUP_INET_INGRESS, flags=0}"
+	},
+	{
+		.data = { .BPF_LINK_CREATE_data = {
+			.prog_fd = -1,
+			.target_fd = 0xdeadbeef,
+			.attach_type = 5,
+			.flags = 4
+		} },
+		.size = offsetofend(struct BPF_LINK_CREATE_struct, flags),
+		.str = "link_create={prog_fd=-1, target_fd=-559038737"
+		       ", attach_type=BPF_SK_SKB_STREAM_VERDICT, flags=0x4}"
+	}
+};
+
+static const struct bpf_attr_check BPF_LINK_UPDATE_checks[] = {
+	{
+		.data = { .BPF_LINK_UPDATE_data = {
+			.link_fd = -1,
+			.new_prog_fd = -2,
+			.old_prog_fd = -3
+		} },
+		.size = offsetofend(struct BPF_LINK_UPDATE_struct, old_prog_fd),
+		.str = "link_update={link_fd=-1, new_prog_fd=-2, flags=0"
+		       ", old_prog_fd=-3}"
+	},
+	{
+		.data = { .BPF_LINK_UPDATE_data = {
+			.link_fd = -1,
+			.new_prog_fd = 0xdeadbeef,
+			.flags = 4,
+			.old_prog_fd = 0xdeadf00d
+		} },
+		.size = offsetofend(struct BPF_LINK_UPDATE_struct, old_prog_fd),
+		.str = "link_update={link_fd=-1, new_prog_fd=-559038737"
+		       ", flags=BPF_F_REPLACE, old_prog_fd=-559026163}"
+	}
+};
+
 
 #define CHK(cmd_) \
 	{ \
@@ -1427,6 +1473,8 @@ main(void)
 		CHK(BPF_MAP_LOOKUP_BATCH),
 		CHK(BPF_MAP_UPDATE_BATCH),
 		CHK(BPF_MAP_DELETE_BATCH),
+		CHK(BPF_LINK_CREATE),
+		CHK(BPF_LINK_UPDATE),
 	};
 
 	page_size = get_page_size();
diff --git a/xlat/bpf_commands.in b/xlat/bpf_commands.in
index 2b95264b..1ff80f41 100644
--- a/xlat/bpf_commands.in
+++ b/xlat/bpf_commands.in
@@ -27,3 +27,5 @@ BPF_MAP_LOOKUP_BATCH 24
 BPF_MAP_LOOKUP_AND_DELETE_BATCH 25
 BPF_MAP_UPDATE_BATCH 26
 BPF_MAP_DELETE_BATCH 27
+BPF_LINK_CREATE 28
+BPF_LINK_UPDATE 29
-- 
2.20.1



More information about the Strace-devel mailing list