Merge pull request #48 from hit9/dev-c-perf

(wip) v1.1.0: C standard mode performance improvements

Author: hit9

benchmark/stm32/README.rst

@@ -13,7 +13,7 @@ Benchmark result
 
 * Microcontroller: `STM32F103ZE <https://www.st.com/en/microcontrolles-microprocessors/stm32f103ze.html>`_,
   Arm Cortex-M3 MCU, 72 MHz
-* bitproto file: `drone.bitproto <drone.bitproto>`_.
+* bitproto file: `drone.bitproto <drone.bitproto>`_,  with a `100bytes` length message.
 
 Standard Mode
 ^^^^^^^^^^^^^
@@ -30,16 +30,16 @@ the optimization mode flag ``-O``, that is to use `the bitproto c lib <../../lib
      - Decode cost per call
    * - No ``-O``
      - 10000
-     - 170μs
-     - 163μs
+     - 144μs
+     - 143μs
    * - ``gcc -O2``
      - 10000
-     - 135μs
-     - 131μs
+     - 140μs
+     - 143μs
    * - ``gcc -O3``
      - 10000
-     - 127μs
-     - 119μs
+     - 128μs
+     - 123μs
 
 Optimization Mode
 ^^^^^^^^^^^^^^^^^
@@ -56,9 +56,9 @@ the optimization mode flag ``-O``, that is not to use `the bitproto c lib <../..
      - Decode cost per call
    * - No ``-O``
      - 10000
-     - 9μs
-     - 9μs
+     - 15μs
+     - 15μs
    * - ``gcc -O2``
      - 10000
-     - 9μs
-     - 9μs
+     - 15μs
+     - 15μs

benchmark/stm32/bpbench/Core/Src/main.c

@@ -21,6 +21,7 @@
 #include "main.h"
 
 #include <stdarg.h>
+#include <stdbool.h>
 #include <stdio.h>
 
 #include "drone_bp.h"
@@ -65,6 +66,10 @@ static void MX_USART1_UART_Init(void);
 /* Private user code ---------------------------------------------------------*/
 /* USER CODE BEGIN 0 */
 
+void soft_assert(bool v) {
+    if (!v) simple_printf("assert failed !!!!!!!!!!!!!!!!!!!!\r\n");
+}
+
 // Printf to uart1.
 void simple_printf(const char *fmt, ...) {
     char buf[512] = {0};
@@ -132,6 +137,51 @@ void bench() {
     bench_decode(n, s);
 }
 
+void simple_test() {
+    struct Drone drone = {0};
+
+    drone.status = DRONE_STATUS_RISING;
+    drone.position.longitude = 2000;
+    drone.position.latitude = 2000;
+    drone.position.altitude = 1080;
+    drone.flight.acceleration[0] = -1001;
+    drone.power.is_charging = false;
+    drone.propellers[0].direction = ROTATING_DIRECTION_CLOCK_WISE;
+    drone.pressure_sensor.pressures[0] = -11;
+
+    unsigned char s[BYTES_LENGTH_DRONE] = {0};
+
+    EncodeDrone(&drone, s);
+
+    unsigned char s_expected[BYTES_LENGTH_DRONE] = {
+        0x82, 0x3E, 0x00, 0x00, 0x80, 0x3E, 0x00, 0x00, 0xC0, 0x21, 0x00, 0x00,
+        0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+        0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+        0xB8, 0xE0, 0xFF, 0xFF, 0x07, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+        0x00, 0x20, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+        0x50, 0xFF, 0xFF, 0x0F, 0x00, 0x00, 0x00};
+
+    for (int i = 0; i < BYTES_LENGTH_DRONE; i++)
+        soft_assert(s_expected[i] == s[i]);
+
+    struct Drone drone_new = {0};
+    DecodeDrone(&drone_new, s);
+
+    soft_assert(drone.status == drone_new.status);
+    soft_assert(drone.position.longitude == drone_new.position.longitude);
+    soft_assert(drone.position.latitude == drone_new.position.latitude);
+    soft_assert(drone.position.altitude == drone_new.position.altitude);
+    soft_assert(drone.flight.acceleration[0] ==
+                drone_new.flight.acceleration[0]);
+    soft_assert(drone.power.is_charging == drone_new.power.is_charging);
+    soft_assert(drone.propellers[0].direction ==
+                drone_new.propellers[0].direction);
+    soft_assert(drone.pressure_sensor.pressures[0] ==
+                drone_new.pressure_sensor.pressures[0]);
+
+    simple_printf("simple test finished\r\n");
+}
+
 /* USER CODE END 0 */
 
 /**
@@ -172,6 +222,8 @@ int main(void) {
 
     /* Infinite loop */
     /* USER CODE BEGIN WHILE */
+    simple_test();
+
     while (1) {
         /* USER CODE END WHILE */
         HAL_GPIO_TogglePin(GPIOB, GPIO_PIN_5);

benchmark/stm32/drone.bitproto

@@ -83,6 +83,10 @@ message Flight {
     TernaryInt32 acceleration = 3;
 }
 
+message PressureSensor {
+    int24[2] pressures = 1;
+}
+
 message Drone {
     DroneStatus status = 1;
     Position position = 2;
@@ -91,4 +95,8 @@ message Drone {
     Power power = 5;
     Network network = 6;
     LandingGear landing_gear = 7;
+    PressureSensor pressure_sensor = 8;
+    // 67 bytes => 100bytes
+    // For optimiation mode, remove the '
+    byte[33] bytes = 9;
 }