Calculate TX air time duty cycles #588 -- UNTESTED

src/airtime.cpp

@@ -1,31 +1,80 @@
-/*
+#include "airtime.h"
-We will keep hourly stats for 48 hours and daily stats for 7 days
+#include <Arduino.h>
 
-3600 seconds in an hour, uint16_t per hour
+#define hoursToLog 48
-86400 seconds in a day, uint32_t per day
 
-Memory to store 48 hours = 96 bytes
+// Here for convience and to avoid magic numbers.
-Memory to store 7 days = 28 bytes
+uint16_t secondsPerHour = 3600;
 
+// Don't read out of this directly. Use the helper functions.
+struct airtimeStruct {
+    uint16_t hourTX[hoursToLog];
+    uint16_t hourRX[hoursToLog];
+    uint16_t hourRX_ALL[hoursToLog];
+    uint8_t lastHourIndex;
+} airtimes;
 
-*/
+void logAirtime(reportTypes reportType, uint32_t airtime_ms)
-
-void logTXairtime(uint32_t airtime_ms)
 {
-    /*
+    currentHourIndexReset();
 
-    How many hours since power up?
+    if (reportType == TX_LOG) {
+        airtimes.hourTX[currentHourIndex()] = airtimes.hourTX[currentHourIndex()] + round(airtime_ms / 1000);
+    } else if (reportType == RX_LOG) {
+        airtimes.hourRX[currentHourIndex()] = airtimes.hourRX[currentHourIndex()] + round(airtime_ms / 1000);
+    } else if (reportType == RX_ALL_LOG) {
+        airtimes.hourRX_ALL[currentHourIndex()] = airtimes.hourRX_ALL[currentHourIndex()] + round(airtime_ms / 1000);
+    } else {
+        // Unknown report type
         
-    Millis / 1000 / 3600
+    }
-
-
-
-    */
 }
 
-void logRXairtime()
+// This will let us easily switch away from using millis at some point.
+//  todo: Don't use millis, instead maintain our own count of time since
+//        boot in seconds.
+uint32_t secondsSinceBoot()
 {
-    /*
+    return millis() / 1000;
-
+}
-    */
+
+uint8_t currentHourIndex()
+{
+    // return ((secondsSinceBoot() - (secondsSinceBoot() / (hoursToLog * secondsPerHour))) / secondsPerHour);
+    return ((secondsSinceBoot() / secondsPerHour) % hoursToLog);
+}
+
+// currentHourIndexReset() should be called every time we receive a packet to log (either RX or TX)
+//   and every time we are asked to report on airtime usage.
+void currentHourIndexReset()
+{
+    if (airtimes.lastHourIndex != currentHourIndex()) {
+        airtimes.hourTX[currentHourIndex()] = 0;
+        airtimes.hourRX[currentHourIndex()] = 0;
+        airtimes.hourRX_ALL[currentHourIndex()] = 0;
+
+        airtimes.lastHourIndex = currentHourIndex();
+    }
+}
+
+uint16_t *airtimeReport(reportTypes reportType)
+{
+    static uint16_t array[hoursToLog];
+
+    currentHourIndexReset();
+
+    for (int i = 0; i < hoursToLog; i++) {
+        if (reportType == TX_LOG) {
+            array[i] = airtimes.hourTX[(airtimes.lastHourIndex + i) % hoursToLog];
+        } else if (reportType == RX_LOG) {
+            array[i] = airtimes.hourRX[(airtimes.lastHourIndex + i) % hoursToLog];
+        } else if (reportType == RX_ALL_LOG) {
+            array[i] = airtimes.hourRX_ALL[(airtimes.lastHourIndex + i) % hoursToLog];
+        } else {
+            // Unknown report type
+            return array;
+        }
+    }
+
+    return array;
 }

src/airtime.h

@@ -4,3 +4,13 @@
 #include <Arduino.h>
 #include <functional>
 
+enum reportTypes { TX_LOG, RX_LOG, RX_ALL_LOG };
+
+void logAirtime(reportTypes reportType, uint32_t airtime_ms);
+
+void currentHourIndexReset();
+uint8_t currentHourIndex();
+
+uint32_t secondsSinceBoot();
+
+uint16_t *airtimeReport(reportTypes reportType);

src/mesh/RadioInterface.h

@@ -5,6 +5,7 @@
 #include "MeshTypes.h"
 #include "Observer.h"
 #include "PointerQueue.h"
+#include "airtime.h"
 #include "mesh.pb.h"
 
 #define MAX_TX_QUEUE 16 // max number of packets which can be waiting for transmission

src/mesh/RadioLibInterface.cpp

@@ -88,6 +88,7 @@ ErrorCode RadioLibInterface::send(MeshPacket *p)
     printPacket("enqueuing for send", p);
     uint32_t xmitMsec = getPacketTime(p);
 
+
     DEBUG_MSG("txGood=%d,rxGood=%d,rxBad=%d\n", txGood, rxGood, rxBad);
     ErrorCode res = txQueue.enqueue(p, 0) ? ERRNO_OK : ERRNO_UNKNOWN;
 
@@ -96,6 +97,11 @@ ErrorCode RadioLibInterface::send(MeshPacket *p)
         return res;
     }
 
+    // Count the packet toward our TX airtime utilization.
+    //   We only count it if it can be added to the TX queue.
+    logAirtime(TX_LOG, xmitMsec);
+    
+
     // We want all sending/receiving to be done by our daemon thread, We use a delay here because this packet might have been sent
     // in response to a packet we just received.  So we want to make sure the other side has had a chance to reconfigure its radio
     startTransmitTimer(true);