All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] net: fix documentation for kernel_getsockname
@ 2022-02-12 10:29 Alex Maydanik
  2022-02-14 14:10 ` patchwork-bot+netdevbpf
  2022-02-14 20:50 ` net: fix documentation for kernel_getsockname: Build Failure MPTCP CI
  0 siblings, 2 replies; 4+ messages in thread
From: Alex Maydanik @ 2022-02-12 10:29 UTC (permalink / raw)
  Cc: alexander.maydanik, David S. Miller, Jakub Kicinski,
	open list:NETWORKING [GENERAL],
	open list

Fixes return value documentation of kernel_getsockname()
and kernel_getpeername() functions.

The previous documentation wrongly specified that the return
value is 0 in case of success, however sock->ops->getname returns
the length of the address in bytes in case of success.

Signed-off-by: Alex Maydanik <alexander.maydanik@gmail.com>
---
 net/socket.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/net/socket.c b/net/socket.c
index 50cf75730fd7..982eecad464c 100644
--- a/net/socket.c
+++ b/net/socket.c
@@ -3448,7 +3448,7 @@ EXPORT_SYMBOL(kernel_connect);
  *	@addr: address holder
  *
  * 	Fills the @addr pointer with the address which the socket is bound.
- *	Returns 0 or an error code.
+ *	Returns the length of the address in bytes or an error code.
  */
 
 int kernel_getsockname(struct socket *sock, struct sockaddr *addr)
@@ -3463,7 +3463,7 @@ EXPORT_SYMBOL(kernel_getsockname);
  *	@addr: address holder
  *
  * 	Fills the @addr pointer with the address which the socket is connected.
- *	Returns 0 or an error code.
+ *	Returns the length of the address in bytes or an error code.
  */
 
 int kernel_getpeername(struct socket *sock, struct sockaddr *addr)
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] net: fix documentation for kernel_getsockname
  2022-02-12 10:29 [PATCH] net: fix documentation for kernel_getsockname Alex Maydanik
@ 2022-02-14 14:10 ` patchwork-bot+netdevbpf
  2022-02-14 20:50 ` net: fix documentation for kernel_getsockname: Build Failure MPTCP CI
  1 sibling, 0 replies; 4+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-02-14 14:10 UTC (permalink / raw)
  To: Alex Maydanik; +Cc: davem, kuba, netdev, linux-kernel

Hello:

This patch was applied to netdev/net.git (master)
by David S. Miller <davem@davemloft.net>:

On Sat, 12 Feb 2022 12:29:27 +0200 you wrote:
> Fixes return value documentation of kernel_getsockname()
> and kernel_getpeername() functions.
> 
> The previous documentation wrongly specified that the return
> value is 0 in case of success, however sock->ops->getname returns
> the length of the address in bytes in case of success.
> 
> [...]

Here is the summary with links:
  - net: fix documentation for kernel_getsockname
    https://git.kernel.org/netdev/net/c/0fc95dec096c

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: net: fix documentation for kernel_getsockname: Build Failure
  2022-02-12 10:29 [PATCH] net: fix documentation for kernel_getsockname Alex Maydanik
  2022-02-14 14:10 ` patchwork-bot+netdevbpf
@ 2022-02-14 20:50 ` MPTCP CI
  2022-02-14 20:55   ` Mat Martineau
  1 sibling, 1 reply; 4+ messages in thread
From: MPTCP CI @ 2022-02-14 20:50 UTC (permalink / raw)
  To: Alex Maydanik; +Cc: mptcp

Hi Alex,

Thank you for your modifications, that's great!

But sadly, our CI spotted some issues with it when trying to build it.

You can find more details there:

  https://patchwork.kernel.org/project/mptcp/patch/20220212102927.25436-1-alexander.maydanik@gmail.com/
  https://github.com/multipath-tcp/mptcp_net-next/actions/runs/1843188936

Status: failure
Initiator: MPTCPimporter
Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/ec18713d1f5d

Feel free to reply to this email if you cannot access logs, if you need
some support to fix the error, if this doesn't seem to be caused by your
modifications or if the error is a false positive one.

Cheers,
MPTCP GH Action bot

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: net: fix documentation for kernel_getsockname: Build Failure
  2022-02-14 20:50 ` net: fix documentation for kernel_getsockname: Build Failure MPTCP CI
@ 2022-02-14 20:55   ` Mat Martineau
  0 siblings, 0 replies; 4+ messages in thread
From: Mat Martineau @ 2022-02-14 20:55 UTC (permalink / raw)
  To: Alex Maydanik; +Cc: Matthieu Baerts, Davide Caratti, mptcp

On Mon, 14 Feb 2022, MPTCP CI wrote:

> Hi Alex,
>
> Thank you for your modifications, that's great!
>
> But sadly, our CI spotted some issues with it when trying to build it.
>
> You can find more details there:
>
>  https://patchwork.kernel.org/project/mptcp/patch/20220212102927.25436-1-alexander.maydanik@gmail.com/
>  https://github.com/multipath-tcp/mptcp_net-next/actions/runs/1843188936
>
> Status: failure
> Initiator: MPTCPimporter
> Commits: https://github.com/multipath-tcp/mptcp_net-next/commits/ec18713d1f5d
>

Alex,

Sorry about the unexpected email from our CI system. It appears there was
an issue with patchew.org that somehow applied and tagged your patch in
our github repo even though you didn't send to our mailing list.

--
Mat Martineau
Intel

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-02-14 20:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-12 10:29 [PATCH] net: fix documentation for kernel_getsockname Alex Maydanik
2022-02-14 14:10 ` patchwork-bot+netdevbpf
2022-02-14 20:50 ` net: fix documentation for kernel_getsockname: Build Failure MPTCP CI
2022-02-14 20:55   ` Mat Martineau

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.