Commit b9563518 authored Apr 23, 2021 by Jim Harris Committed by Tomasz Zawadzki Apr 26, 2021

thread: do not ignore read() return



This fixes unused-result warning which prevents
building with -Werror.

Fixes: 924d4bf3 ("poller: add busy wait mechanism for intr")

Signed-off-by: Jim Harris <james.r.harris@intel.com>
Change-Id: I074949b65e68405033322af34d90e96a896c8b8c
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7593


Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Xiaodong Liu <xiaodong.liu@intel.com>
Reviewed-by:  <dongx.yi@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot

parent 9a387721

lib/thread/thread.c

+2 −1

Original line number	Diff line number	Diff line
		@@ -1119,6 +1119,7 @@ busy_poller_set_interrupt_mode(struct spdk_poller poller, void cb_arg, bool in
		{
		int busy_efd = poller->interruptfd;
		uint64_t notify = 1;
		int rc __attribute__((unused));

		assert(busy_efd >= 0);

		@@ -1129,7 +1130,7 @@ busy_poller_set_interrupt_mode(struct spdk_poller poller, void cb_arg, bool in
		}
		} else {
		/* Read on eventfd will clear its level triggering. */
		read(busy_efd, &notify, sizeof(notify));
		rc = read(busy_efd, &notify, sizeof(notify));
		}
		}

Admin message