[PATCH net 2/2] vsock: small cleanup in vsock_poll()

From: Stefano Garzarella
Date: Tue Aug 11 2020 - 05:55:33 EST


This patch combines nested if statements in a single one to reduce
the indentation in vsock_poll().
It also combines an if nested in the else branch.

The behavior isn't changed.

Signed-off-by: Stefano Garzarella <sgarzare@xxxxxxxxxx>
---
net/vmw_vsock/af_vsock.c | 27 ++++++++++++---------------
1 file changed, 12 insertions(+), 15 deletions(-)

diff --git a/net/vmw_vsock/af_vsock.c b/net/vmw_vsock/af_vsock.c
index 9e93bc201cc0..2c80dc14fa60 100644
--- a/net/vmw_vsock/af_vsock.c
+++ b/net/vmw_vsock/af_vsock.c
@@ -1032,21 +1032,18 @@ static __poll_t vsock_poll(struct file *file, struct socket *sock,
}

/* Connected sockets that can produce data can be written. */
- if (transport && sk->sk_state == TCP_ESTABLISHED) {
- if (!(sk->sk_shutdown & SEND_SHUTDOWN)) {
- bool space_avail_now = false;
- int ret = transport->notify_poll_out(
- vsk, 1, &space_avail_now);
- if (ret < 0) {
- mask |= EPOLLERR;
- } else {
- if (space_avail_now)
- /* Remove EPOLLWRBAND since INET
- * sockets are not setting it.
- */
- mask |= EPOLLOUT | EPOLLWRNORM;
-
- }
+ if (transport && sk->sk_state == TCP_ESTABLISHED &&
+ !(sk->sk_shutdown & SEND_SHUTDOWN)) {
+ bool space_avail_now = false;
+ int ret = transport->notify_poll_out(vsk, 1,
+ &space_avail_now);
+ if (ret < 0) {
+ mask |= EPOLLERR;
+ } else if (space_avail_now) {
+ /* Remove EPOLLWRBAND since INET
+ * sockets are not setting it.
+ */
+ mask |= EPOLLOUT | EPOLLWRNORM;
}
}

--
2.26.2