| Message ID | 20230808134049.1407498-2-leitao@debian.org (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | io_uring: Initial support for {s,g}etsockopt commands | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Guessing tree name failed - patch did not apply |
| bpf/vmtest-bpf-PR | success | PR summary |
| bpf/vmtest-bpf-VM_Test-1 | success | Logs for ShellCheck |
| bpf/vmtest-bpf-VM_Test-2 | success | Logs for build for aarch64 with gcc |
| bpf/vmtest-bpf-VM_Test-3 | pending | Logs for build for s390x with gcc |
| bpf/vmtest-bpf-VM_Test-4 | success | Logs for build for x86_64 with gcc |
| bpf/vmtest-bpf-VM_Test-5 | success | Logs for build for x86_64 with llvm-16 |
| bpf/vmtest-bpf-VM_Test-6 | success | Logs for set-matrix |
On Tue, 8 Aug 2023 06:40:41 -0700 Breno Leitao <leitao@debian.org> wrote: > Exposing function sock_use_custom_sol_socket(), so it could be used by > io_uring subsystem. > > This function will be used in the function io_uring_cmd_setsockopt() in > the coming patch, so, let's move it to the socket.h header file. Hi, this description doesn't seem to match the code change below... Hugo Villeneuve. > Signed-off-by: Breno Leitao <leitao@debian.org> > --- > include/linux/net.h | 5 +++++ > net/socket.c | 5 ----- > 2 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/include/linux/net.h b/include/linux/net.h > index 41c608c1b02c..14a956e4530e 100644 > --- a/include/linux/net.h > +++ b/include/linux/net.h > @@ -355,4 +355,9 @@ u32 kernel_sock_ip_overhead(struct sock *sk); > #define MODULE_ALIAS_NET_PF_PROTO_NAME(pf, proto, name) \ > MODULE_ALIAS("net-pf-" __stringify(pf) "-proto-" __stringify(proto) \ > name) > + > +static inline bool sock_use_custom_sol_socket(const struct socket *sock) > +{ > + return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); > +} > #endif /* _LINUX_NET_H */ > diff --git a/net/socket.c b/net/socket.c > index 1dc23f5298ba..8df54352af83 100644 > --- a/net/socket.c > +++ b/net/socket.c > @@ -2216,11 +2216,6 @@ SYSCALL_DEFINE4(recv, int, fd, void __user *, ubuf, size_t, size, > return __sys_recvfrom(fd, ubuf, size, flags, NULL, NULL); > } > > -static bool sock_use_custom_sol_socket(const struct socket *sock) > -{ > - return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); > -} > - > /* > * Set a socket option. Because we don't know the option lengths we have > * to pass the user mode parameter for the protocols to sort out. > -- > 2.34.1 >
Hello Hugo, On Tue, Aug 08, 2023 at 12:13:23PM -0400, Hugo Villeneuve wrote: > On Tue, 8 Aug 2023 06:40:41 -0700 > Breno Leitao <leitao@debian.org> wrote: > > > Exposing function sock_use_custom_sol_socket(), so it could be used by > > io_uring subsystem. > > > > This function will be used in the function io_uring_cmd_setsockopt() in > > the coming patch, so, let's move it to the socket.h header file. > > Hi, > this description doesn't seem to match the code change below... I re-read the patch comment and it seems to match what the code does, so, probably this description only makes sense to me (?). That said, hat have you understood from reading the description above? Thanks for the review, > > --- > > include/linux/net.h | 5 +++++ > > net/socket.c | 5 ----- > > 2 files changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/include/linux/net.h b/include/linux/net.h > > index 41c608c1b02c..14a956e4530e 100644 > > --- a/include/linux/net.h > > +++ b/include/linux/net.h > > @@ -355,4 +355,9 @@ u32 kernel_sock_ip_overhead(struct sock *sk); > > #define MODULE_ALIAS_NET_PF_PROTO_NAME(pf, proto, name) \ > > MODULE_ALIAS("net-pf-" __stringify(pf) "-proto-" __stringify(proto) \ > > name) > > + > > +static inline bool sock_use_custom_sol_socket(const struct socket *sock) > > +{ > > + return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); > > +} > > #endif /* _LINUX_NET_H */ > > diff --git a/net/socket.c b/net/socket.c > > index 1dc23f5298ba..8df54352af83 100644 > > --- a/net/socket.c > > +++ b/net/socket.c > > @@ -2216,11 +2216,6 @@ SYSCALL_DEFINE4(recv, int, fd, void __user *, ubuf, size_t, size, > > return __sys_recvfrom(fd, ubuf, size, flags, NULL, NULL); > > } > > > > -static bool sock_use_custom_sol_socket(const struct socket *sock) > > -{ > > - return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); > > -} > > - > > /* > > * Set a socket option. Because we don't know the option lengths we have > > * to pass the user mode parameter for the protocols to sort out. > > -- > > 2.34.1 > >
On Tue, 8 Aug 2023 10:21:03 -0700 Breno Leitao <leitao@debian.org> wrote: > Hello Hugo, > > On Tue, Aug 08, 2023 at 12:13:23PM -0400, Hugo Villeneuve wrote: > > On Tue, 8 Aug 2023 06:40:41 -0700 > > Breno Leitao <leitao@debian.org> wrote: > > > > > Exposing function sock_use_custom_sol_socket(), so it could be used by > > > io_uring subsystem. > > > > > > This function will be used in the function io_uring_cmd_setsockopt() in > > > the coming patch, so, let's move it to the socket.h header file. > > > > Hi, > > this description doesn't seem to match the code change below... > > I re-read the patch comment and it seems to match what the code does, > so, probably this description only makes sense to me (?). > > That said, hat have you understood from reading the description above? > socket.h > Thanks for the review, Hi Breno, your comments says "move it to the socket.h header file" but it seems to be moved to the net.h header file? Hugo Villeneuve > > > --- > > > include/linux/net.h | 5 +++++ > > > net/socket.c | 5 ----- > > > 2 files changed, 5 insertions(+), 5 deletions(-) > > > > > > diff --git a/include/linux/net.h b/include/linux/net.h > > > index 41c608c1b02c..14a956e4530e 100644 > > > --- a/include/linux/net.h > > > +++ b/include/linux/net.h > > > @@ -355,4 +355,9 @@ u32 kernel_sock_ip_overhead(struct sock *sk); > > > #define MODULE_ALIAS_NET_PF_PROTO_NAME(pf, proto, name) \ > > > MODULE_ALIAS("net-pf-" __stringify(pf) "-proto-" __stringify(proto) \ > > > name) > > > + > > > +static inline bool sock_use_custom_sol_socket(const struct socket *sock) > > > +{ > > > + return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); > > > +} > > > #endif /* _LINUX_NET_H */ > > > diff --git a/net/socket.c b/net/socket.c > > > index 1dc23f5298ba..8df54352af83 100644 > > > --- a/net/socket.c > > > +++ b/net/socket.c > > > @@ -2216,11 +2216,6 @@ SYSCALL_DEFINE4(recv, int, fd, void __user *, ubuf, size_t, size, > > > return __sys_recvfrom(fd, ubuf, size, flags, NULL, NULL); > > > } > > > > > > -static bool sock_use_custom_sol_socket(const struct socket *sock) > > > -{ > > > - return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); > > > -} > > > - > > > /* > > > * Set a socket option. Because we don't know the option lengths we have > > > * to pass the user mode parameter for the protocols to sort out. > > > -- > > > 2.34.1 > > >
Breno Leitao <leitao@debian.org> writes: > Hello Hugo, > > On Tue, Aug 08, 2023 at 12:13:23PM -0400, Hugo Villeneuve wrote: >> On Tue, 8 Aug 2023 06:40:41 -0700 >> Breno Leitao <leitao@debian.org> wrote: >> >> > Exposing function sock_use_custom_sol_socket(), so it could be used by >> > io_uring subsystem. >> > >> > This function will be used in the function io_uring_cmd_setsockopt() in >> > the coming patch, so, let's move it to the socket.h header file. >> >> Hi, >> this description doesn't seem to match the code change below... > > I re-read the patch comment and it seems to match what the code does, > so, probably this description only makes sense to me (?). > > That said, hat have you understood from reading the description above? The comment states the function prototype is moving to socket.h, but the patch puts it in net.h. Cheers, Jeff > > Thanks for the review, > >> > --- >> > include/linux/net.h | 5 +++++ >> > net/socket.c | 5 ----- >> > 2 files changed, 5 insertions(+), 5 deletions(-) >> > >> > diff --git a/include/linux/net.h b/include/linux/net.h >> > index 41c608c1b02c..14a956e4530e 100644 >> > --- a/include/linux/net.h >> > +++ b/include/linux/net.h >> > @@ -355,4 +355,9 @@ u32 kernel_sock_ip_overhead(struct sock *sk); >> > #define MODULE_ALIAS_NET_PF_PROTO_NAME(pf, proto, name) \ >> > MODULE_ALIAS("net-pf-" __stringify(pf) "-proto-" __stringify(proto) \ >> > name) >> > + >> > +static inline bool sock_use_custom_sol_socket(const struct socket *sock) >> > +{ >> > + return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); >> > +} >> > #endif /* _LINUX_NET_H */ >> > diff --git a/net/socket.c b/net/socket.c >> > index 1dc23f5298ba..8df54352af83 100644 >> > --- a/net/socket.c >> > +++ b/net/socket.c >> > @@ -2216,11 +2216,6 @@ SYSCALL_DEFINE4(recv, int, fd, void __user *, ubuf, size_t, size, >> > return __sys_recvfrom(fd, ubuf, size, flags, NULL, NULL); >> > } >> > >> > -static bool sock_use_custom_sol_socket(const struct socket *sock) >> > -{ >> > - return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); >> > -} >> > - >> > /* >> > * Set a socket option. Because we don't know the option lengths we have >> > * to pass the user mode parameter for the protocols to sort out. >> > -- >> > 2.34.1 >> >
On Tue, Aug 08, 2023 at 01:46:47PM -0400, Hugo Villeneuve wrote: > On Tue, 8 Aug 2023 10:21:03 -0700 > Breno Leitao <leitao@debian.org> wrote: > > > Hello Hugo, > > > > On Tue, Aug 08, 2023 at 12:13:23PM -0400, Hugo Villeneuve wrote: > > > On Tue, 8 Aug 2023 06:40:41 -0700 > > > Breno Leitao <leitao@debian.org> wrote: > > > > > > > Exposing function sock_use_custom_sol_socket(), so it could be used by > > > > io_uring subsystem. > > > > > > > > This function will be used in the function io_uring_cmd_setsockopt() in > > > > the coming patch, so, let's move it to the socket.h header file. > > > > > > Hi, > > > this description doesn't seem to match the code change below... > > > > I re-read the patch comment and it seems to match what the code does, > > so, probably this description only makes sense to me (?). > > > > That said, hat have you understood from reading the description above? > > socket.h > > Thanks for the review, > > Hi Breno, > your comments says "move it to the socket.h header file" but it seems > to be moved to the net.h header file? Gotcha. Thanks. I will update.
diff --git a/include/linux/net.h b/include/linux/net.h index 41c608c1b02c..14a956e4530e 100644 --- a/include/linux/net.h +++ b/include/linux/net.h @@ -355,4 +355,9 @@ u32 kernel_sock_ip_overhead(struct sock *sk); #define MODULE_ALIAS_NET_PF_PROTO_NAME(pf, proto, name) \ MODULE_ALIAS("net-pf-" __stringify(pf) "-proto-" __stringify(proto) \ name) + +static inline bool sock_use_custom_sol_socket(const struct socket *sock) +{ + return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); +} #endif /* _LINUX_NET_H */ diff --git a/net/socket.c b/net/socket.c index 1dc23f5298ba..8df54352af83 100644 --- a/net/socket.c +++ b/net/socket.c @@ -2216,11 +2216,6 @@ SYSCALL_DEFINE4(recv, int, fd, void __user *, ubuf, size_t, size, return __sys_recvfrom(fd, ubuf, size, flags, NULL, NULL); } -static bool sock_use_custom_sol_socket(const struct socket *sock) -{ - return test_bit(SOCK_CUSTOM_SOCKOPT, &sock->flags); -} - /* * Set a socket option. Because we don't know the option lengths we have * to pass the user mode parameter for the protocols to sort out.
Exposing function sock_use_custom_sol_socket(), so it could be used by io_uring subsystem. This function will be used in the function io_uring_cmd_setsockopt() in the coming patch, so, let's move it to the socket.h header file. Signed-off-by: Breno Leitao <leitao@debian.org> --- include/linux/net.h | 5 +++++ net/socket.c | 5 ----- 2 files changed, 5 insertions(+), 5 deletions(-)