commit 3c4029f7d9170c77322e2f83ecf5b073e126d1e5
author Austin Schuh <austin.schuh@bluerivertech.com> Thu Apr 13 12:09:55 2023 -0700
committer Austin Schuh <austin.linux@gmail.com> Fri May 05 18:05:17 2023 -0700
tree 50ab5160e0ef2e8706c1dca6609a0e791a5921b2
parent 71e7214f017d799cb18cdcd2cfb723b39e4e95b5

Fix sync mode in the logging backend

This flushes data out as we go and teaches the kernel that it can drop
it from the cache.  This keeps the kernel cache pretty free.

We were only syncing the first page and never flushing...  Whops.  This
now flushes.  Confirmed with strace (And I really don't know how to
write a reliable test for it...)

Change-Id: I939791b41b04bae9bc53857efa585d0cbb318924
Signed-off-by: Austin Schuh <austin.schuh@bluerivertech.com>

aos/events/logging/log_backend.cc

diff --git a/aos/events/logging/log_backend.cc b/aos/events/logging/log_backend.cc
index a8b7a43..4b170d3 100644
--- a/aos/events/logging/log_backend.cc
+++ b/aos/events/logging/log_backend.cc
@@ -242,8 +242,8 @@
                     total_write_bytes_ - last_synced_bytes_,
                     POSIX_FADV_DONTNEED);
 
-      last_synced_bytes_ = total_write_bytes_;
     }
+    last_synced_bytes_ = total_write_bytes_;
   }
 
   const auto end = aos::monotonic_clock::now();