diff mbox series

ip.7: Document IP_LOCAL_PORT_RANGE socket option

Message ID 20230201123634.284689-1-jakub@cloudflare.com (mailing list archive)
State Not Applicable
Delegated to: Stephen Hemminger
Headers show
Series ip.7: Document IP_LOCAL_PORT_RANGE socket option | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Jakub Sitnicki Feb. 1, 2023, 12:36 p.m. UTC
Linux commit 91d0b78c5177 ("inet: Add IP_LOCAL_PORT_RANGE socket option")
introduced a new socket option available for AF_INET and AF_INET6 sockets.

Option will be available starting from Linux 6.3. Document it.

Signed-off-by: Jakub Sitnicki <jakub@cloudflare.com>
---
Submitting this man page update as the author of the feature.

We did a technical review of the man page text together with the code [1].

[1]: https://lore.kernel.org/all/20221221-sockopt-port-range-v6-0-be255cc0e51f@cloudflare.com/
---
 man7/ip.7 | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

Comments

Stephen Hemminger Feb. 3, 2023, 6:58 p.m. UTC | #1
On Wed,  1 Feb 2023 13:36:34 +0100
Jakub Sitnicki <jakub@cloudflare.com> wrote:

> Linux commit 91d0b78c5177 ("inet: Add IP_LOCAL_PORT_RANGE socket option")
> introduced a new socket option available for AF_INET and AF_INET6 sockets.
> 
> Option will be available starting from Linux 6.3. Document it.
> 
> Signed-off-by: Jakub Sitnicki <jakub@cloudflare.com>
> ---
> Submitting this man page update as the author of the feature.
> 
> We did a technical review of the man page text together with the code [1].
> 
> [1]: https://lore.kernel.org/all/20221221-sockopt-port-range-v6-0-be255cc0e51f@cloudflare.com/
> ---
>  man7/ip.7 | 21 +++++++++++++++++++++
>  1 file changed, 21 insertions(+)

Marking this patch as not applicable in netdev patchwork since it is for documentation
and I assume linux-man has its own tracking.
Simon Horman Feb. 3, 2023, 7:09 p.m. UTC | #2
On Wed, Feb 01, 2023 at 01:36:34PM +0100, Jakub Sitnicki wrote:
> Linux commit 91d0b78c5177 ("inet: Add IP_LOCAL_PORT_RANGE socket option")
> introduced a new socket option available for AF_INET and AF_INET6 sockets.
> 
> Option will be available starting from Linux 6.3. Document it.
> 
> Signed-off-by: Jakub Sitnicki <jakub@cloudflare.com>
> ---
> Submitting this man page update as the author of the feature.
> 
> We did a technical review of the man page text together with the code [1].
> 
> [1]: https://lore.kernel.org/all/20221221-sockopt-port-range-v6-0-be255cc0e51f@cloudflare.com/

Reviewed-by: Simon Horman <simon.horman@corigine.com>
Alejandro Colomar Feb. 5, 2023, 1:16 p.m. UTC | #3
Hi Jakub,

On 2/1/23 13:36, Jakub Sitnicki wrote:
> Linux commit 91d0b78c5177 ("inet: Add IP_LOCAL_PORT_RANGE socket option")
> introduced a new socket option available for AF_INET and AF_INET6 sockets.
> 
> Option will be available starting from Linux 6.3. Document it.
> 
> Signed-off-by: Jakub Sitnicki <jakub@cloudflare.com>
> ---
> Submitting this man page update as the author of the feature.
> 
> We did a technical review of the man page text together with the code [1].

The formatting LGTM.  Could you please resend when it arrives to Linus's tree?

Cheers,

Alex

> 
> [1]: https://lore.kernel.org/all/20221221-sockopt-port-range-v6-0-be255cc0e51f@cloudflare.com/
> ---
>   man7/ip.7 | 21 +++++++++++++++++++++
>   1 file changed, 21 insertions(+)
> 
> diff --git a/man7/ip.7 b/man7/ip.7
> index f69af1b32..f166c3b57 100644
> --- a/man7/ip.7
> +++ b/man7/ip.7
> @@ -439,6 +439,27 @@ and
>   .B IP_TOS
>   are ignored.
>   .TP
> +.BR IP_LOCAL_PORT_RANGE " (since Linux 6.3)"
> +Set or get the per-socket default local port range. This option can be used to
> +clamp down the global local port range, defined by the
> +.I ip_local_port_range
> +.I /proc
> +interface described below, for a given socket.
> +.IP
> +The option takes an
> +.I uint32_t
> +value with the high 16 bits set to the upper range bound, and the low 16 bits
> +set to the lower range bound. Range bounds are inclusive. The 16-bit values
> +should be in host byte order.
> +.IP
> +The lower bound has to be less than the upper bound when both bounds are not
> +zero. Otherwise, setting the option fails with EINVAL.
> +.IP
> +If either bound is outside of the global local port range, or is zero, then that
> +bound has no effect.
> +.IP
> +To reset the setting, pass zero as both the upper and the lower bound.
> +.TP
>   .BR IP_MSFILTER " (since Linux 2.4.22 / 2.5.68)"
>   This option provides access to the advanced full-state filtering API.
>   Argument is an
Jakub Sitnicki Feb. 6, 2023, 10:42 a.m. UTC | #4
On Sun, Feb 05, 2023 at 02:16 PM +01, Alejandro Colomar wrote:
> [[PGP Signed Part:Undecided]]
> Hi Jakub,
>
> On 2/1/23 13:36, Jakub Sitnicki wrote:
>> Linux commit 91d0b78c5177 ("inet: Add IP_LOCAL_PORT_RANGE socket option")
>> introduced a new socket option available for AF_INET and AF_INET6 sockets.
>> Option will be available starting from Linux 6.3. Document it.
>> Signed-off-by: Jakub Sitnicki <jakub@cloudflare.com>
>> ---
>> Submitting this man page update as the author of the feature.
>> We did a technical review of the man page text together with the code [1].
>
> The formatting LGTM.  Could you please resend when it arrives to Linus's tree?

Thanks for the review, Alex. Will do.

-Jakub
diff mbox series

Patch

diff --git a/man7/ip.7 b/man7/ip.7
index f69af1b32..f166c3b57 100644
--- a/man7/ip.7
+++ b/man7/ip.7
@@ -439,6 +439,27 @@  and
 .B IP_TOS
 are ignored.
 .TP
+.BR IP_LOCAL_PORT_RANGE " (since Linux 6.3)"
+Set or get the per-socket default local port range. This option can be used to
+clamp down the global local port range, defined by the
+.I ip_local_port_range
+.I /proc
+interface described below, for a given socket.
+.IP
+The option takes an
+.I uint32_t
+value with the high 16 bits set to the upper range bound, and the low 16 bits
+set to the lower range bound. Range bounds are inclusive. The 16-bit values
+should be in host byte order.
+.IP
+The lower bound has to be less than the upper bound when both bounds are not
+zero. Otherwise, setting the option fails with EINVAL.
+.IP
+If either bound is outside of the global local port range, or is zero, then that
+bound has no effect.
+.IP
+To reset the setting, pass zero as both the upper and the lower bound.
+.TP
 .BR IP_MSFILTER " (since Linux 2.4.22 / 2.5.68)"
 This option provides access to the advanced full-state filtering API.
 Argument is an