linux.git/fs/dlm/lowcomms.c, branch v4.16

dlm: switch to sock_recvmsg()

2017-12-03T01:37:47+00:00

Signed-off-by: Al Viro

DLM: fix NULL pointer dereference in send_to_sock()

2017-09-25T17:45:21+00:00

The writequeue and writequeue_lock member of othercon was not initialized.
If lowcomms_state_change() is called from network layer, othercon->swork
may be scheduled. In this case, send_to_sock() will generate a NULL pointer
reference. We avoid this problem by correctly initializing writequeue and
writequeue_lock member of othercon.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: fix to reschedule rwork

2017-09-25T17:45:21+00:00

When an error occurs in kernel_recvmsg or kernel_sendpage and
close_connection is called and receive work is already scheduled,
receive work is canceled. In that case, the receive work will not
be scheduled forever after reconnection, because CF_READ_PENDING
flag is established.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: fix to use sk_callback_lock correctly

2017-09-25T17:45:21+00:00

In the current implementation, we think that exclusion control between
processing to set the callback function to the connection structure and
processing to refer to the connection structure from the callback function
was not enough. We fix them.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: fix memory leak in tcp_accept_from_sock()

2017-09-25T17:45:21+00:00

The sk member of the socket generated by sock_create_kern() is overwritten
by ops->accept(). So the previous sk will not be released.
We use kernel_accept() instead of sock_create_kern() and ops->accept().

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: use CF_CLOSE flag to stop dlm_send correctly

2017-09-25T17:45:21+00:00

If reconnection fails while executing dlm_lowcomms_stop,
dlm_send will not stop.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: Reanimate CF_WRITE_PENDING flag

2017-09-25T17:45:21+00:00

CF_WRITE_PENDING flag has been reanimated to make dlm_send stop properly
when running dlm_lowcomms_stop.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: close othercon at send/receive error

2017-09-25T17:45:21+00:00

If an error occurs in the sending / receiving process, if othercon
exists, sending / receiving processing using othercon may also result
in an error. We fix to pre-close othercon as well.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: fix to use sock_mutex correctly in xxx_accept_from_sock

2017-09-25T17:45:21+00:00

In the current implementation, we think that exclusion control
for othercon in tcp_accept_from_sock() and sctp_accept_from_sock()
was not enough. We fix them.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland

DLM: fix race condition between dlm_send and dlm_recv

2017-09-25T17:45:21+00:00

When kernel_sendpage(in send_to_sock) and kernel_recvmsg
(in receive_from_sock) return error, close_connection may works at the
same time. At that time, they may wait for each other by cancel_work_sync.

Signed-off-by: Tadashi Miyauchi 
Signed-off-by: Tsutomu Owa 
Signed-off-by: David Teigland