[libcamera-devel] [PATCH v3 01/11] libcamera: Print Timer identifier

Jacopo Mondi jacopo at jmondi.org
Wed Dec 1 00:36:24 CET 2021


The Timer debug output does not report to which timer a condition refers
to. Fix that by printing the Timer address.

Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>
---
 src/libcamera/base/event_dispatcher_poll.cpp | 3 ++-
 src/libcamera/base/timer.cpp                 | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/libcamera/base/event_dispatcher_poll.cpp b/src/libcamera/base/event_dispatcher_poll.cpp
index 8ee22d5adcc4..20cc6e617189 100644
--- a/src/libcamera/base/event_dispatcher_poll.cpp
+++ b/src/libcamera/base/event_dispatcher_poll.cpp
@@ -214,7 +214,8 @@ int EventDispatcherPoll::poll(std::vector<struct pollfd> *pollfds)
 			timeout = { 0, 0 };
 
 		LOG(Event, Debug)
-			<< "timeout " << timeout.tv_sec << "."
+			<< "timeout " << nextTimer << " "
+			<< timeout.tv_sec << "."
 			<< std::setfill('0') << std::setw(9)
 			<< timeout.tv_nsec;
 	}
diff --git a/src/libcamera/base/timer.cpp b/src/libcamera/base/timer.cpp
index 9c54352d46bd..187336e3a1a4 100644
--- a/src/libcamera/base/timer.cpp
+++ b/src/libcamera/base/timer.cpp
@@ -96,7 +96,7 @@ void Timer::start(std::chrono::milliseconds duration)
 void Timer::start(std::chrono::steady_clock::time_point deadline)
 {
 	if (Thread::current() != thread()) {
-		LOG(Timer, Error) << "Timer can't be started from another thread";
+		LOG(Timer, Error) << "Timer " << this << " << can't be started from another thread";
 		return;
 	}
 
@@ -128,7 +128,7 @@ void Timer::stop()
 		return;
 
 	if (Thread::current() != thread()) {
-		LOG(Timer, Error) << "Timer can't be stopped from another thread";
+		LOG(Timer, Error) << "Timer " << this << " can't be stopped from another thread";
 		return;
 	}
 
-- 
2.33.1



More information about the libcamera-devel mailing list