Skip to contentSkip to conversation

bpftool: Support sendmsg{4,6} attach types

PATCH Applied Andrey Ignatov 4 participants 4 messages May 29, 2018 – May 30, 2018

Subsystems (4)
Networking Network drivers BPF bpftool
lore
Share & tools
lore

Andrey Ignatov

author
#0 lore raw

PATCH 1/1 bpftool: Support sendmsg{4,6} attach types

Add support for recently added BPF_CGROUP_UDP4_SENDMSG and BPF_CGROUP_UDP6_SENDMSG attach types to bpftool, update documentation and bash completion.

Signed-off-by Andrey Ignatov <[email protected]>

---
I'm not sure about "since 4.18" in Documentation part. I can follow-up when the next kernel version is known.

---
 tools/bpf/bpftool/Documentation/bpftool-cgroup.rst | 9 +++++++--
 tools/bpf/bpftool/bash-completion/bpftool          | 5 +++--
 tools/bpf/bpftool/cgroup.c                         | 4 +++-
 3 files changed, 13 insertions(+), 5 deletions(-)
tools/bpf/bpftool/Documentation/bpftool-cgroup.rst +7 -2
index d004f63..7b0e6d4 100644
@@ -27,7 +27,8 @@ MAP COMMANDS
 |
 |	*PROG* := { **id** *PROG_ID* | **pinned** *FILE* | **tag** *PROG_TAG* }
 |	*ATTACH_TYPE* := { **ingress** | **egress** | **sock_create** | **sock_ops** | **device** |
-|		**bind4** | **bind6** | **post_bind4** | **post_bind6** | **connect4** | **connect6** }
+|		**bind4** | **bind6** | **post_bind4** | **post_bind6** | **connect4** | **connect6** |
+|               **sendmsg4** | **sendmsg6** }
 |	*ATTACH_FLAGS* := { **multi** | **override** }
 
 DESCRIPTION

@@ -70,7 +71,11 @@ DESCRIPTION
 		  **post_bind4** return from bind(2) for an inet4 socket (since 4.17);
 		  **post_bind6** return from bind(2) for an inet6 socket (since 4.17);
 		  **connect4** call to connect(2) for an inet4 socket (since 4.17);
-		  **connect6** call to connect(2) for an inet6 socket (since 4.17).
+		  **connect6** call to connect(2) for an inet6 socket (since 4.17);
+		  **sendmsg4** call to sendto(2), sendmsg(2), sendmmsg(2) for an
+		  unconnected udp4 socket (since 4.18);
+		  **sendmsg6** call to sendto(2), sendmsg(2), sendmmsg(2) for an
+		  unconnected udp6 socket (since 4.18).
 
 	**bpftool cgroup detach** *CGROUP* *ATTACH_TYPE* *PROG*
 		  Detach *PROG* from the cgroup *CGROUP* and attach type
tools/bpf/bpftool/bash-completion/bpftool +3 -2
index 7bc198d..1e10833 100644
@@ -407,7 +407,7 @@ _bpftool()
                 attach|detach)
                     local ATTACH_TYPES='ingress egress sock_create sock_ops \
                         device bind4 bind6 post_bind4 post_bind6 connect4 \
-                        connect6'
+                        connect6 sendmsg4 sendmsg6'
                     local ATTACH_FLAGS='multi override'
                     local PROG_TYPE='id pinned tag'
                     case $prev in

@@ -416,7 +416,8 @@ _bpftool()
                             return 0
                             ;;
                         ingress|egress|sock_create|sock_ops|device|bind4|bind6|\
-                        post_bind4|post_bind6|connect4|connect6)
+                        post_bind4|post_bind6|connect4|connect6|sendmsg4|\
+                        sendmsg6)
                             COMPREPLY=( $( compgen -W "$PROG_TYPE" -- \
                                 "$cur" ) )
                             return 0
tools/bpf/bpftool/cgroup.c +3 -1
index 1351bd6..16bee01 100644
@@ -20,7 +20,7 @@
 	"       ATTACH_TYPE := { ingress | egress | sock_create |\n"	       \
 	"                        sock_ops | device | bind4 | bind6 |\n"	       \
 	"                        post_bind4 | post_bind6 | connect4 |\n"       \
-	"                        connect6 }"
+	"                        connect6 | sendmsg4 | sendmsg6 }"
 
 static const char * const attach_type_strings[] = {
 	[BPF_CGROUP_INET_INGRESS] = "ingress",

@@ -34,6 +34,8 @@ static const char * const attach_type_strings[] = {
 	[BPF_CGROUP_INET6_CONNECT] = "connect6",
 	[BPF_CGROUP_INET4_POST_BIND] = "post_bind4",
 	[BPF_CGROUP_INET6_POST_BIND] = "post_bind6",
+	[BPF_CGROUP_UDP4_SENDMSG] = "sendmsg4",
+	[BPF_CGROUP_UDP6_SENDMSG] = "sendmsg6",
 	[__MAX_BPF_ATTACH_TYPE] = NULL,
 };
Signature · 1 line
2.9.5
Reviewed-by on v1 1/1
#1 lore raw
Quoted 5 lines · Andrey Ignatov

Add support for recently added BPF_CGROUP_UDP4_SENDMSG and BPF_CGROUP_UDP6_SENDMSG attach types to bpftool, update documentation and bash completion.

Signed-off-by Andrey Ignatov <[email protected]>
Reviewed-by Jakub Kicinski <[email protected]>
Quoted 2 lines

I'm not sure about "since 4.18" in Documentation part. I can follow-up when the next kernel version is known.

IMHO it's fine, we can follow up if Linus decides to call it something else :)

Thanks!

Acked-by
#2 lore raw
Quoted 9 lines · Jakub Kicinski
Quoted 5 lines · Andrey Ignatov

Add support for recently added BPF_CGROUP_UDP4_SENDMSG and BPF_CGROUP_UDP6_SENDMSG attach types to bpftool, update documentation and bash completion.

Signed-off-by Andrey Ignatov <[email protected]>
Reviewed-by Jakub Kicinski <[email protected]>
Quoted 7 lines
Quoted 2 lines

I'm not sure about "since 4.18" in Documentation part. I can follow-up when the next kernel version is known.

IMHO it's fine, we can follow up if Linus decides to call it something else :)

Thanks!

Acked-by Song Liu <[email protected]>
maintainer Applied
#3 lore raw
Quoted 13 lines · Song Liu
Quoted 12 lines · Jakub Kicinski
Quoted 5 lines · Andrey Ignatov

Add support for recently added BPF_CGROUP_UDP4_SENDMSG and BPF_CGROUP_UDP6_SENDMSG attach types to bpftool, update documentation and bash completion.

Signed-off-by Andrey Ignatov <[email protected]>
Reviewed-by Jakub Kicinski <[email protected]>
Quoted 2 lines

I'm not sure about "since 4.18" in Documentation part. I can follow-up when the next kernel version is known.

Quoted 6 lines
Quoted 4 lines

IMHO it's fine, we can follow up if Linus decides to call it something else :)

Thanks!

Acked-by Song Liu <[email protected]>

Applied to bpf-next, thanks guys!