mbox series

[0/3] Refix the socket leak in xs_setup_local()

Message ID 20220511132232.4030-1-wanghai38@huawei.com (mailing list archive)
Headers show
Series Refix the socket leak in xs_setup_local() | expand

Message

Wang Hai May 11, 2022, 1:22 p.m. UTC 
Patch1 and patch2 roll back the wrong solution to fix socket leaks.

Patch3 adds safe teardown mechanism to re-fix socket leaks.

Wang Hai (3):
  Revert "SUNRPC: Ensure gss-proxy connects on setup"
  Revert "Revert "SUNRPC: attempt AF_LOCAL connect on setup""
  SUNRPC: Fix local socket leak in xs_setup_local()

 include/linux/sunrpc/clnt.h          |  1 -
 net/sunrpc/auth_gss/gss_rpc_upcall.c |  2 +-
 net/sunrpc/clnt.c                    |  3 ---
 net/sunrpc/xprtsock.c                | 19 ++++++++++++++++++-
 4 files changed, 19 insertions(+), 6 deletions(-)

Comments

Trond Myklebust May 11, 2022, 4:14 p.m. UTC | #1 
On Wed, 2022-05-11 at 21:22 +0800, Wang Hai wrote:
> Patch1 and patch2 roll back the wrong solution to fix socket leaks.
> 
> Patch3 adds safe teardown mechanism to re-fix socket leaks.
> 
> Wang Hai (3):
>   Revert "SUNRPC: Ensure gss-proxy connects on setup"
>   Revert "Revert "SUNRPC: attempt AF_LOCAL connect on setup""
>   SUNRPC: Fix local socket leak in xs_setup_local()
> 
>  include/linux/sunrpc/clnt.h          |  1 -
>  net/sunrpc/auth_gss/gss_rpc_upcall.c |  2 +-
>  net/sunrpc/clnt.c                    |  3 ---
>  net/sunrpc/xprtsock.c                | 19 ++++++++++++++++++-
>  4 files changed, 19 insertions(+), 6 deletions(-)
> 

Thanks, but there is already a fix for this queued up in the linux-next
branch.
Wang Hai May 12, 2022, 1:58 a.m. UTC | #2 
在 2022/5/12 0:14, Trond Myklebust 写道:
> On Wed, 2022-05-11 at 21:22 +0800, Wang Hai wrote:
>> Patch1 and patch2 roll back the wrong solution to fix socket leaks.
>>
>> Patch3 adds safe teardown mechanism to re-fix socket leaks.
>>
>> Wang Hai (3):
>>    Revert "SUNRPC: Ensure gss-proxy connects on setup"
>>    Revert "Revert "SUNRPC: attempt AF_LOCAL connect on setup""
>>    SUNRPC: Fix local socket leak in xs_setup_local()
>>
>>   include/linux/sunrpc/clnt.h          |  1 -
>>   net/sunrpc/auth_gss/gss_rpc_upcall.c |  2 +-
>>   net/sunrpc/clnt.c                    |  3 ---
>>   net/sunrpc/xprtsock.c                | 19 ++++++++++++++++++-
>>   4 files changed, 19 insertions(+), 6 deletions(-)
>>
> Thanks, but there is already a fix for this queued up in the linux-next
> branch.
Thanks, sorry to bother you, I found it.

I only subscribed to the linux-kernel@vger.kernel.org mailing list,
not the linux-nfs@vger.kernel.org mailing list separately. So I
didn't notice it was fixed.