This is an automatic generated email to let you know that the following patch 
were queued at the 
http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:

Subject: v4l2-compliance: improve select() check in captureBufs()
Author:  Hans Verkuil <hverkuil-ci...@xs4all.nl>
Date:    Sat Dec 4 12:13:30 2021 +0100

This test checks that at least one of the read or write fds are
set upon return from select(). But it is possible while streaming
that only an event is returned. In that case this check would fail.

Add a check for an event (exception) as well.

This should solve a regular occuring failure in the compliance test.

Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 utils/v4l2-compliance/v4l2-test-buffers.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

---

http://git.linuxtv.org/cgit.cgi/v4l-utils.git/commit/?id=d95200f6b2e12ddac8f9b94ef9fd5ad197897923
diff --git a/utils/v4l2-compliance/v4l2-test-buffers.cpp 
b/utils/v4l2-compliance/v4l2-test-buffers.cpp
index fbf926532e3f..3122f150f937 100644
--- a/utils/v4l2-compliance/v4l2-test-buffers.cpp
+++ b/utils/v4l2-compliance/v4l2-test-buffers.cpp
@@ -990,7 +990,8 @@ static int captureBufs(struct node *node, struct node 
*node_m2m_cap, const cv4l_
                        fail_on_test(ret == 0);
                        fail_on_test(ret < 0);
                        fail_on_test(!FD_ISSET(node->g_fd(), &rfds) &&
-                                    !FD_ISSET(node->g_fd(), &wfds));
+                                    !FD_ISSET(node->g_fd(), &wfds) &&
+                                    !FD_ISSET(node->g_fd(), &efds));
                        can_read = FD_ISSET(node->g_fd(), &rfds);
                        have_event = FD_ISSET(node->g_fd(), &efds);
                } else if (pollmode == POLL_MODE_EPOLL) {

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to