On Tue, Oct 09, 2018 at 04:41:43PM +0200, Ursula Braun wrote:
> Eugene,
>
> we are considering the following patch:
>
> ---
> net/smc/smc_diag.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/net/smc/smc_diag.c b/net/smc/smc_diag.c
> index dbf64a93d68a..371b4cf31fcd 100644
> --- a/net/smc/smc_diag.c
> +++ b/net/smc/smc_diag.c
> @@ -38,6 +38,7 @@ static void smc_diag_msg_common_fill(struct smc_diag_msg
> *r, struct sock *sk)
> {
> struct smc_sock *smc = smc_sk(sk);
>
> + r->diag_family = sk->sk_family;
> if (!smc->clcsock)
> return;
> r->id.idiag_sport = htons(smc->clcsock->sk->sk_num);
> @@ -45,14 +46,12 @@ static void smc_diag_msg_common_fill(struct smc_diag_msg
> *r, struct sock *sk)
> r->id.idiag_if = smc->clcsock->sk->sk_bound_dev_if;
> sock_diag_save_cookie(sk, r->id.idiag_cookie);
> if (sk->sk_protocol == SMCPROTO_SMC) {
> - r->diag_family = PF_INET;
> memset(&r->id.idiag_src, 0, sizeof(r->id.idiag_src));
> memset(&r->id.idiag_dst, 0, sizeof(r->id.idiag_dst));
> r->id.idiag_src[0] = smc->clcsock->sk->sk_rcv_saddr;
> r->id.idiag_dst[0] = smc->clcsock->sk->sk_daddr;
> #if IS_ENABLED(CONFIG_IPV6)
> } else if (sk->sk_protocol == SMCPROTO_SMC6) {
> - r->diag_family = PF_INET6;
> memcpy(&r->id.idiag_src, &smc->clcsock->sk->sk_v6_rcv_saddr,
> sizeof(smc->clcsock->sk->sk_v6_rcv_saddr));
> memcpy(&r->id.idiag_dst, &smc->clcsock->sk->sk_v6_daddr,
> --
>
> Tools would then need to derive the PF_INET/PF_INET6 info from the
> inet_diag_sockid info
> in the smc_diag_msg.
Unfortunately, struct inet_diag_sockid doesn't provide any clue
in order to derive address family.
There's unused (if I haven't missed anything) diag_shutdown field
in struct smc_diag_msg (it looks like that the shutdown state is
reported via a separate attribute in the current implementation),
it probably might be repurposed for providing information about
the underlying socket address family.
> However, this problem is in the mainline kernel since 4.18. So, we are not
> sure if we are allowed
> to change the user interface again.
Well, anything is better that the current state.
