lkml.org 
[lkml]   [2022]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] bpf: Add SO_REUSEPORT to bpf_{g,s}etsockopt documentation
From
On 10/7/22 10:55 AM, Rongfeng Ji wrote:
> SO_REUSEPORT has been supported by bpf_{g,s}etsockopt for a long time.
> It is added at the beginning of the optname list under level
> SOL_SOCKET because it is supported by both helper functions.
>
> Signed-off-by: Rongfeng Ji <SikoJobs@outlook.com>
> ---
> include/uapi/linux/bpf.h | 7 ++++---
> tools/include/uapi/linux/bpf.h | 7 ++++---
> 2 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h
> index 51b9aa640ad2..ccae9cb833b8 100644
> --- a/include/uapi/linux/bpf.h
> +++ b/include/uapi/linux/bpf.h
> @@ -2574,9 +2574,9 @@ union bpf_attr {
> * It supports the following *level*\ s:
> *
> * * **SOL_SOCKET**, which supports the following *optname*\ s:
> - * **SO_RCVBUF**, **SO_SNDBUF**, **SO_MAX_PACING_RATE**,
> - * **SO_PRIORITY**, **SO_RCVLOWAT**, **SO_MARK**,
> - * **SO_BINDTODEVICE**, **SO_KEEPALIVE**.
> + * **SO_REUSEPORT**, **SO_RCVBUF**, **SO_SNDBUF**,
> + * **SO_MAX_PACING_RATE**, **SO_PRIORITY**, **SO_RCVLOWAT**,
> + * **SO_MARK**, **SO_BINDTODEVICE**, **SO_KEEPALIVE**.

More options has recently been added to SOL_SOCKET, IPPROTO_TCP, and
IPPROTO_IPV6. Could you take this chance to also add them together?

Also, it seems the optnames here is not in any particular order. Please append
the new ones to the end to avoid shifting code churn like the above.

> * * **IPPROTO_TCP**, which supports the following *optname*\ s:
> * **TCP_CONGESTION**, **TCP_BPF_IW**,
> * **TCP_BPF_SNDCWND_CLAMP**, **TCP_SAVE_SYN**,
> @@ -2802,6 +2802,7 @@ union bpf_attr {
> * This helper actually implements a subset of **getsockopt()**.
> * It supports the following *level*\ s:
> *
> + * * **SOL_SOCKET**, which supports *optname* **SO_REUSEPORT**.

For bpf_getsockopt(), it supports all optnames in bpf_setsockopt() with a few
exceptions. The exceptions should be the TCP_BPF_* which is Set only. The
TCP_SAVED_SYNC is Get only. Please check.

The doc for bpf_getsockopt() could be simplified to "... the same set of
bpf_setsockopt's optnames is supported. The exceptions are... TCP_BPF_* which
is bpf_setsockopt() only.... TCP_SAVED_SYNC is bpf_getsockopt() only..."

Please tag it as bpf-next and also v2 in the next revision:
Documentation/bpf/bpf_devel_QA.rst (Q: How do I indicate which tree....)


\
 
 \ /
  Last update: 2022-10-11 02:19    [W:0.058 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site