downloading logs over blefs

[jmlich]

I am wondering how can I debug code with sealed device. The logging is done mostly via NRF_LOG_INFO.

I would like to try to redefine NRF_LOG_INFO to store messages into file system and then download them via blefs. Have already someone tried that?

The other idea is to implement UART via BLE.

[vchigrin]

I did some (very dirty) hacks that implemented this idea and helped me with debugging on sealed PineTime.
https://github.com/vchigrin/InfiniTime/tree/logging
If somebody interested in them, I can make kind of PR, (guess there are necessary some build flag to enable logging to filesystem, and some bugs should be addressed....).

If anybody chose to use these changes "as is", be very careful and do NOT "mark as verified" firmware on your smartwatch.
I almost bricked my PineTime because filesystem got damaged after long run of InfiniTime with these changes (I guess there are some issue with task synchronization).

[mark9064]

Nice stuff! Yes LittleFS currently isn't threadsafe. Enabling it isn't too hard though

--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -890,6 +896,7 @@ target_compile_options(lvgl PRIVATE
         )
 
 # LITTLEFS_SRC
+add_definitions(-DLFS_THREADSAFE)
 add_library(littlefs STATIC ${LITTLEFS_SRC})
 target_include_directories(littlefs SYSTEM PUBLIC . ../)
 target_include_directories(littlefs SYSTEM PUBLIC ${INCLUDES_FROM_LIBS})
 
--- a/src/components/fs/FS.cpp
+++ b/src/components/fs/FS.cpp
@@ -13,6 +13,8 @@ FS::FS(Pinetime::Drivers::SpiNorFlash& driver)
       .prog = SectorProg,
       .erase = SectorErase,
       .sync = SectorSync,
+      .lock = LockFilesystem,
+      .unlock = UnlockFilesystem,
 
       .read_size = 16,
       .prog_size = 8,
@@ -109,6 +111,16 @@ lfs_ssize_t FS::GetFSSize() {
   return lfs_fs_size(&lfs);
 }
 
+int FS::LockFilesystem(const struct lfs_config* c) {
+  Pinetime::Controllers::FS& lfs = *(static_cast<Pinetime::Controllers::FS*>(c->context));
+  return xSemaphoreTake(lfs.fsMutex, portMAX_DELAY) == pdTRUE ? 0 : -1;
+}
+
+int FS::UnlockFilesystem(const struct lfs_config* c) {
+  Pinetime::Controllers::FS& lfs = *(static_cast<Pinetime::Controllers::FS*>(c->context));
+  return xSemaphoreGive(lfs.fsMutex) == pdTRUE ? 0 : -1;
+}
+
 /*
 
     ----------- Interface between littlefs and SpiNorFlash -----------

--- a/src/components/fs/FS.h
+++ b/src/components/fs/FS.h
@@ -3,6 +3,8 @@
 #include <cstdint>
 #include "drivers/SpiNorFlash.h"
 #include <littlefs/lfs.h>
+#include <FreeRTOS.h>
+#include <semphr.h>
 
 namespace Pinetime {
   namespace Controllers {
@@ -71,12 +73,15 @@ namespace Pinetime {
       bool resourcesValid = false;
       const struct lfs_config lfsConfig;
 
+      SemaphoreHandle_t fsMutex = xSemaphoreCreateMutex();
       lfs_t lfs;
 
       static int SectorSync(const struct lfs_config* c);
       static int SectorErase(const struct lfs_config* c, lfs_block_t block);
       static int SectorProg(const struct lfs_config* c, lfs_block_t block, lfs_off_t off, const void* buffer, lfs_size_t size);
       static int SectorRead(const struct lfs_config* c, lfs_block_t block, lfs_off_t off, void* buffer, lfs_size_t size);
+      static int LockFilesystem(const struct lfs_config* c);
+      static int UnlockFilesystem(const struct lfs_config* c);
     };
   }
 }

Sorry for the bad formatting, copied and pasted from terminal but hopefully you get the idea
Not sure if this solves your problem but maybe it'll help