mbox series

[net-next,v2,0/2] Separate SMC parameter settings from TCP sysctls

Message ID 1663667542-119851-1-git-send-email-guwen@linux.alibaba.com (mailing list archive)
Headers show
Series Separate SMC parameter settings from TCP sysctls | expand

Message

Wen Gu Sept. 20, 2022, 9:52 a.m. UTC 
SMC shares some sysctls with TCP, but considering the difference
between these two protocols, it may not be very suitable for SMC
to reuse TCP parameter settings in some cases, such as keepalive
time or buffer size.

So this patch set aims to introduce some SMC specific sysctls to
independently and flexibly set the parameters that suit SMC.

v2->v1:
- Use proc_dointvec_jiffies as proc_handler and allow value 0 to
  disable TEST_LINK.

Tony Lu (1):
  net/smc: Unbind r/w buffer size from clcsock and make them tunable

Wen Gu (1):
  net/smc: Introduce a specific sysctl for TEST_LINK time

 Documentation/networking/smc-sysctl.rst | 25 +++++++++++++++++++++++++
 include/net/netns/smc.h                 |  3 +++
 net/smc/af_smc.c                        |  5 ++---
 net/smc/smc_core.c                      |  8 ++++----
 net/smc/smc_llc.c                       |  2 +-
 net/smc/smc_llc.h                       |  1 +
 net/smc/smc_sysctl.c                    | 30 ++++++++++++++++++++++++++++++
 7 files changed, 66 insertions(+), 8 deletions(-)

Comments

Wenjia Zhang Sept. 20, 2022, 2 p.m. UTC | #1 
On 20.09.22 11:52, Wen Gu wrote:
> SMC shares some sysctls with TCP, but considering the difference
> between these two protocols, it may not be very suitable for SMC
> to reuse TCP parameter settings in some cases, such as keepalive
> time or buffer size.
> 
> So this patch set aims to introduce some SMC specific sysctls to
> independently and flexibly set the parameters that suit SMC.
> 
> v2->v1:
> - Use proc_dointvec_jiffies as proc_handler and allow value 0 to
>    disable TEST_LINK.
> 
> Tony Lu (1):
>    net/smc: Unbind r/w buffer size from clcsock and make them tunable
> 
> Wen Gu (1):
>    net/smc: Introduce a specific sysctl for TEST_LINK time
> 
>   Documentation/networking/smc-sysctl.rst | 25 +++++++++++++++++++++++++
>   include/net/netns/smc.h                 |  3 +++
>   net/smc/af_smc.c                        |  5 ++---
>   net/smc/smc_core.c                      |  8 ++++----
>   net/smc/smc_llc.c                       |  2 +-
>   net/smc/smc_llc.h                       |  1 +
>   net/smc/smc_sysctl.c                    | 30 ++++++++++++++++++++++++++++++
>   7 files changed, 66 insertions(+), 8 deletions(-)
> 
Looks good. Thank you!

For the series:
Acked-by: Wenjia Zhang <wenjia@linux.ibm.com>
patchwork-bot+netdevbpf@kernel.org Sept. 22, 2022, 11:20 a.m. UTC | #2 
Hello:

This series was applied to netdev/net-next.git (master)
by Paolo Abeni <pabeni@redhat.com>:

On Tue, 20 Sep 2022 17:52:20 +0800 you wrote:
> SMC shares some sysctls with TCP, but considering the difference
> between these two protocols, it may not be very suitable for SMC
> to reuse TCP parameter settings in some cases, such as keepalive
> time or buffer size.
> 
> So this patch set aims to introduce some SMC specific sysctls to
> independently and flexibly set the parameters that suit SMC.
> 
> [...]

Here is the summary with links:
  - [net-next,v2,1/2] net/smc: Introduce a specific sysctl for TEST_LINK time
    https://git.kernel.org/netdev/net-next/c/77eee3251431
  - [net-next,v2,2/2] net/smc: Unbind r/w buffer size from clcsock and make them tunable
    https://git.kernel.org/netdev/net-next/c/0227f058aa29

You are awesome, thank you!