From 74c0deed3f97ccbf129f2d76ddc0740a5b7e8530 Mon Sep 17 00:00:00 2001
From: Nick Bolton <nick@symless.com>
Date: Mon, 9 Sep 2024 13:33:57 +0100
Subject: [PATCH] Move some EI server log messages to higher log levels (#7497)

---
 src/lib/platform/EiEventQueueBuffer.cpp |  4 ++--
 src/lib/platform/EiScreen.cpp           | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/lib/platform/EiEventQueueBuffer.cpp b/src/lib/platform/EiEventQueueBuffer.cpp
index a7c4efe80..dc5d84939 100644
--- a/src/lib/platform/EiEventQueueBuffer.cpp
+++ b/src/lib/platform/EiEventQueueBuffer.cpp
@@ -91,7 +91,7 @@ void EiEventQueueBuffer::waitForEvent(double timeout_in_ms) {
     if (pfds[PIPEFD].revents & POLLIN) {
       char buf[64];
       auto result = read(pipe_r_, buf, sizeof(buf)); // discard
-      LOG_DEBUG("event queue read result: %d", result);
+      LOG_DEBUG2("event queue read result: %d", result);
     }
   }
   Thread::testCancel();
@@ -133,7 +133,7 @@ bool EiEventQueueBuffer::addEvent(uint32_t dataID) {
 
   // tickle the pipe so our read thread wakes up
   auto result = write(pipe_w_, "!", 1);
-  LOG_DEBUG("event queue write result: %d", result);
+  LOG_DEBUG2("event queue write result: %d", result);
 
   return true;
 }
diff --git a/src/lib/platform/EiScreen.cpp b/src/lib/platform/EiScreen.cpp
index 2bd00aafc..acd972000 100644
--- a/src/lib/platform/EiScreen.cpp
+++ b/src/lib/platform/EiScreen.cpp
@@ -88,7 +88,7 @@ EiScreen::EiScreen(bool is_primary, IEventQueue *events, bool use_portal)
     // Note: socket backend does not support reconnections
     auto rc = ei_setup_backend_socket(ei_, nullptr);
     if (rc != 0) {
-      LOG_DEBUG("ei init error: %s", strerror(-rc));
+      LOG_ERR("ei init error: %s", strerror(-rc));
       throw std::runtime_error("failed to init ei context");
     }
   }
@@ -115,7 +115,7 @@ void EiScreen::handle_ei_log_event(
     ei_log_context *context) {
   switch (priority) {
   case EI_LOG_PRIORITY_DEBUG:
-    LOG_DEBUG("ei: %s", message);
+    LOG_DEBUG1("ei: %s", message);
     break;
   case EI_LOG_PRIORITY_INFO:
     LOG_INFO("ei: %s", message);
@@ -670,7 +670,7 @@ void EiScreen::on_motion_event(ei_event *event) {
     auto pixel_dx = static_cast<std::int32_t>(buffer_dx);
     auto pixel_dy = static_cast<std::int32_t>(buffer_dy);
     if (pixel_dx || pixel_dy) {
-      LOG_DEBUG("event: motion on secondary x=%d y=%d", pixel_dx, pixel_dy);
+      LOG_DEBUG1("event: motion on secondary x=%d y=%d", pixel_dx, pixel_dy);
       sendEvent(
           events_->forIPrimaryScreen().motionOnSecondary(),
           MotionInfo::alloc(pixel_dx, pixel_dy));
@@ -769,10 +769,10 @@ void EiScreen::handleSystemEvent(const Event &sysevent, void *) {
     case EI_EVENT_FRAME:
       break;
     case EI_EVENT_DEVICE_START_EMULATING:
-      LOG_DEBUG("device %s starts emulating", ei_device_get_name(device));
+      LOG_DEBUG("device %s started emulating", ei_device_get_name(device));
       break;
     case EI_EVENT_DEVICE_STOP_EMULATING:
-      LOG_DEBUG("device %s stops emulating", ei_device_get_name(device));
+      LOG_DEBUG("device %s stopped emulating", ei_device_get_name(device));
       break;
     case EI_EVENT_KEYBOARD_KEY:
       on_key_event(event);