From ced38b1b0f46f9fbdf9d37644d27bdbd2a29af1d Mon Sep 17 00:00:00 2001 From: Tom Englund Date: Sun, 24 Aug 2025 09:57:37 +0200 Subject: disable buffer readability checks on intel (#11515) * dmabuf: disable buffer read check on intel readability checks directly on buffer fds on some intel laptops is broken, see https://gitlab.freedesktop.org/drm/intel/-/issues/9415 * sync: use rvalue ref in addwaiter doonreadable use rvalue reference in addwaiter and doonreadable, because we store the function in the SReadableWaiter a lot of the time, move it into place there when that happends or let it go out of scope after function call. --- src/helpers/sync/SyncTimeline.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/helpers/sync/SyncTimeline.cpp') diff --git a/src/helpers/sync/SyncTimeline.cpp b/src/helpers/sync/SyncTimeline.cpp index 614a7c5b..9fe2e406 100644 --- a/src/helpers/sync/SyncTimeline.cpp +++ b/src/helpers/sync/SyncTimeline.cpp @@ -64,7 +64,7 @@ std::optional CSyncTimeline::check(uint64_t point, uint32_t flags) { return ret == 0; } -bool CSyncTimeline::addWaiter(const std::function& waiter, uint64_t point, uint32_t flags) { +bool CSyncTimeline::addWaiter(std::function&& waiter, uint64_t point, uint32_t flags) { auto eventFd = CFileDescriptor(eventfd(0, EFD_CLOEXEC)); if (!eventFd.isValid()) { @@ -77,7 +77,7 @@ bool CSyncTimeline::addWaiter(const std::function& waiter, uint64_t poin return false; } - g_pEventLoopManager->doOnReadable(std::move(eventFd), waiter); + g_pEventLoopManager->doOnReadable(std::move(eventFd), std::move(waiter)); return true; } -- cgit v1.2.3