Commit 0e469023 authored by Jim Harris's avatar Jim Harris Committed by Tomasz Zawadzki
Browse files

sock/posix: return error immediately if epoll_ctl fails 



We do not want to do any further work on adding
the sock to the group if the epoll_ctl (or kevent)
fails.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I44b6dc86ce5676aa1b8d6c50b86f22758e4e37fa
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7594


Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
parent b9563518

module/sock/posix/posix.c

+4 −0
Original line number Diff line number Diff line
@@ -1181,6 +1181,10 @@ posix_sock_group_impl_add_sock(struct spdk_sock_group_impl *_group, struct spdk_ 
	rc = kevent(group->fd, &event, 1, NULL, 0, &ts);

#endif



	if (rc != 0) {

		return rc;

	}



	/* switched from another polling group due to scheduling */

	if (spdk_unlikely(sock->recv_pipe != NULL  &&

			  (spdk_pipe_reader_bytes_available(sock->recv_pipe) > 0))) {