From a3122eeceafae4f37346ec40f24d20706473e962 Mon Sep 17 00:00:00 2001 From: Jacopo Mondi Date: Mon, 13 Dec 2021 15:44:22 +0100 Subject: test: fence: Check write return value The ::write() function used to signal a framebuffer fence in the unit test is marked with the 'warn_unused_result'. When building in debugoptimized mode not checking for the return value causes issues at build time: /test/fence.cpp:254:2: error: ignoring return value of function declared with 'warn_unused_result' attribute Fix that by checking the ::write() return value and emitting an error message in case the write fails. Signed-off-by: Jacopo Mondi Reported-by: Umang Jain Reviewed-by: Laurent Pinchart Tested-by: Laurent Pinchart Reviewed-by: Umang Jain --- test/fence.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'test') diff --git a/test/fence.cpp b/test/fence.cpp index d2dfc9b6..d2865398 100644 --- a/test/fence.cpp +++ b/test/fence.cpp @@ -251,7 +251,12 @@ void FenceTest::requestComplete(Request *request) void FenceTest::signalFence() { uint64_t value = 1; - write(efd2_, &value, sizeof(value)); + int ret; + + ret = write(efd2_, &value, sizeof(value)); + if (ret != sizeof(value)) + cerr << "Failed to signal fence" << endl; + dispatcher_->processEvents(); } -- cgit v1.2.1