Re: [PATCH bpf-next] bpf: Fix a few typos in BPF helpers documentation

[Jesper Dangaard Brouer <jbrouer@redhat.com>]

On 25/08/2022 11.26, Quentin Monnet wrote:
> Address a few typos in the documentation for the BPF helper functions.
> They were reported by Jakub [0], who ran spell checkers on the generated
> man page [1].
> 
> Sync-up the UAPI header with its version in tools/.
> 
> [0]https://lore.kernel.org/linux-man/d22dcd47-023c-8f52-d369-7b5308e6c842@gmail.com/T/#mb02e7d4b7fb61d98fa914c77b581184e9a9537af
> [1]https://lore.kernel.org/linux-man/eb6a1e41-c48e-ac45-5154-ac57a2c76108@gmail.com/T/#m4a8d1b003616928013ffcd1450437309ab652f9f
> 
> Cc: Alejandro Colomar<alx.manpages@gmail.com>
> Cc: Jakub Wilk<jwilk@jwilk.net>
> Cc: Jesper Dangaard Brouer<brouer@redhat.com>
> Cc:linux-man@vger.kernel.org
> Reported-by: Jakub Wilk<jwilk@jwilk.net>
> Signed-off-by: Quentin Monnet<quentin@isovalent.com>
> ---
>   include/uapi/linux/bpf.h       | 16 ++++++++--------
>   tools/include/uapi/linux/bpf.h | 18 +++++++++---------
>   2 files changed, 17 insertions(+), 17 deletions(-)
> 
> diff --git a/include/uapi/linux/bpf.h b/include/uapi/linux/bpf.h
> index 644600dbb114..e4d3810990be 100644
> --- a/include/uapi/linux/bpf.h
> +++ b/include/uapi/linux/bpf.h
> @@ -4728,12 +4728,12 @@ union bpf_attr {
>    *
>    *		The argument*len_diff*  can be used for querying with a planned
>    *		size change. This allows to check MTU prior to changing packet
> - *		ctx. Providing an*len_diff*  adjustment that is larger than the
> + *		ctx. Providing a*len_diff*  adjustment that is larger than the
>    *		actual packet size (resulting in negative packet size) will in
> - *		principle not exceed the MTU, why it is not considered a
> - *		failure.  Other BPF-helpers are needed for performing the
> - *		planned size change, why the responsability for catch a negative
> - *		packet size belong in those helpers.
> + *		principle not exceed the MTU, which is why it is not considered
> + *		a failure.  Other BPF helpers are needed for performing the
> + *		planned size change, therefore the responsibility for catching
> + *		a negative packet size belongs in those helpers.
>    *
>    *		Specifying*ifindex*  zero means the MTU check is performed
>    *		against the current net device.  This is practical if this isn't

Thanks for improving these formulations.

Acked-by: Jesper Dangaard Brouer <brouer@redhat.com>