libraries/openamp_arduino: Optimize Open-AMP RPMSG buffer configuration.

iabdalkader · iabdalkader · commit b874b41562b8 · 2024-01-15T15:05:01.000+02:00
Reduce buffer size to 512 and increase available buffers to 64. The
increased number of buffers minimizes the chances of blocking while
waiting for a free buffer.

Signed-off-by: iabdalkader &lt;i.abdalkader@gmail.com&gt;
diff --git a/libraries/openamp_arduino/src/openamp_conf.h b/libraries/openamp_arduino/src/openamp_conf.h
@@ -158,7 +158,7 @@ extern int __OPENAMP_region_end__[];
 #define VRING_BUFF_ADDRESS      (SHM_START_ADDRESS + 0x2000)
 #define VRING_BUFF_SIZE         (SHM_SIZE - 0x2000)
 #define VRING_ALIGNMENT         32
-#define VRING_NUM_BUFFS         16   /* number of rpmsg buffers */
+#define VRING_NUM_BUFFS         64   /* number of rpmsg buffers */
 
 /* Fixed parameter */
 #define NUM_RESOURCE_ENTRIES    2
diff --git a/variants/GENERIC_STM32H747_M4/conf/mbed_app.json b/variants/GENERIC_STM32H747_M4/conf/mbed_app.json
@@ -22,7 +22,7 @@
         "VIRTIO_DEVICE_ONLY",
         "NO_ATOMIC_64_SUPPORT",
         "METAL_MAX_DEVICE_REGIONS=2",
-        "RPMSG_BUFFER_SIZE=2048"
+        "RPMSG_BUFFER_SIZE=512"
       ]
     }
   }
diff --git a/variants/GENERIC_STM32H747_M4/defines.txt b/variants/GENERIC_STM32H747_M4/defines.txt
@@ -52,7 +52,7 @@
 -DMETAL_MAX_DEVICE_REGIONS=2
 -DNO_ATOMIC_64_SUPPORT
 -DNO_VTOR_RELOCATE
--DRPMSG_BUFFER_SIZE=2048
+-DRPMSG_BUFFER_SIZE=512
 -DSTM32H747xx
 -DTARGET_CORDIO
 -DTARGET_CORTEX
diff --git a/variants/GIGA/conf/mbed_app.json b/variants/GIGA/conf/mbed_app.json
@@ -16,7 +16,7 @@
         "VIRTIO_DRIVER_ONLY",
         "NO_ATOMIC_64_SUPPORT",
         "METAL_MAX_DEVICE_REGIONS=2",
-        "RPMSG_BUFFER_SIZE=2048"
+        "RPMSG_BUFFER_SIZE=512"
       ]
     }
   }
diff --git a/variants/GIGA/defines.txt b/variants/GIGA/defines.txt
@@ -53,7 +53,7 @@
 -DMETAL_MAX_DEVICE_REGIONS=2
 -DNO_ATOMIC_64_SUPPORT
 -DQSPI_NO_SAMPLE_SHIFT
--DRPMSG_BUFFER_SIZE=2048
+-DRPMSG_BUFFER_SIZE=512
 -DSTM32H747xx
 -DTARGET_CORDIO
 -DTARGET_CORTEX
diff --git a/variants/NICLA_VISION/conf/mbed_app.json b/variants/NICLA_VISION/conf/mbed_app.json
@@ -17,7 +17,7 @@
         "VIRTIO_DRIVER_ONLY",
         "NO_ATOMIC_64_SUPPORT",
         "METAL_MAX_DEVICE_REGIONS=2",
-        "RPMSG_BUFFER_SIZE=100"
+        "RPMSG_BUFFER_SIZE=512"
       ]
     }
   }
diff --git a/variants/NICLA_VISION/defines.txt b/variants/NICLA_VISION/defines.txt
@@ -54,7 +54,7 @@
 -DMETAL_MAX_DEVICE_REGIONS=2
 -DNO_ATOMIC_64_SUPPORT
 -DQSPI_NO_SAMPLE_SHIFT
--DRPMSG_BUFFER_SIZE=2048
+-DRPMSG_BUFFER_SIZE=512
 -DSTM32H747xx
 -DT1oI2C
 -DT1oI2C_UM11225
diff --git a/variants/OPTA/conf/mbed_app.json b/variants/OPTA/conf/mbed_app.json
@@ -16,7 +16,7 @@
         "VIRTIO_DRIVER_ONLY",
         "NO_ATOMIC_64_SUPPORT",
         "METAL_MAX_DEVICE_REGIONS=2",
-        "RPMSG_BUFFER_SIZE=2048"
+        "RPMSG_BUFFER_SIZE=512"
       ]
     }
   }
diff --git a/variants/OPTA/defines.txt b/variants/OPTA/defines.txt
@@ -53,7 +53,7 @@
 -DMETAL_MAX_DEVICE_REGIONS=2
 -DNO_ATOMIC_64_SUPPORT
 -DQSPI_NO_SAMPLE_SHIFT
--DRPMSG_BUFFER_SIZE=2048
+-DRPMSG_BUFFER_SIZE=512
 -DSTM32H747xx
 -DTARGET_CORDIO
 -DTARGET_CORTEX
diff --git a/variants/PORTENTA_H7_M7/conf/mbed_app.json b/variants/PORTENTA_H7_M7/conf/mbed_app.json
@@ -22,7 +22,7 @@
         "VIRTIO_DRIVER_ONLY",
         "NO_ATOMIC_64_SUPPORT",
         "METAL_MAX_DEVICE_REGIONS=2",
-        "RPMSG_BUFFER_SIZE=2048",
+        "RPMSG_BUFFER_SIZE=512",
         "LSE_STARTUP_TIMEOUT=200"
       ]
     }
diff --git a/variants/PORTENTA_H7_M7/defines.txt b/variants/PORTENTA_H7_M7/defines.txt
@@ -55,7 +55,7 @@
 -DMETAL_MAX_DEVICE_REGIONS=2
 -DNO_ATOMIC_64_SUPPORT
 -DQSPI_NO_SAMPLE_SHIFT
--DRPMSG_BUFFER_SIZE=2048
+-DRPMSG_BUFFER_SIZE=512
 -DSTM32H747xx
 -DTARGET_CORDIO
 -DTARGET_CORTEX
diff --git a/variants/PORTENTA_X8/conf/mbed_app.json b/variants/PORTENTA_X8/conf/mbed_app.json
@@ -16,7 +16,7 @@
         "VIRTIO_DEVICE_ONLY",
         "NO_ATOMIC_64_SUPPORT",
         "METAL_MAX_DEVICE_REGIONS=2",
-        "RPMSG_BUFFER_SIZE=2048"
+        "RPMSG_BUFFER_SIZE=512"
       ]
     }
   }
diff --git a/variants/PORTENTA_X8/defines.txt b/variants/PORTENTA_X8/defines.txt
@@ -44,7 +44,7 @@
 -DMETAL_MAX_DEVICE_REGIONS=2
 -DNO_ATOMIC_64_SUPPORT
 -DNO_VTOR_RELOCATE
--DRPMSG_BUFFER_SIZE=2048
+-DRPMSG_BUFFER_SIZE=512
 -DSTM32H747xx
 -DTARGET_CORTEX
 -DTARGET_CORTEX_M

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`	`"VIRTIO_DEVICE_ONLY",`
`23`	`23`	`"NO_ATOMIC_64_SUPPORT",`
`24`	`24`	`"METAL_MAX_DEVICE_REGIONS=2",`
`25`		`- "RPMSG_BUFFER_SIZE=2048"`
	`25`	`+ "RPMSG_BUFFER_SIZE=512"`
`26`	`26`	`]`
`27`	`27`	`}`
`28`	`28`	`}`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`	`"VIRTIO_DRIVER_ONLY",`
`17`	`17`	`"NO_ATOMIC_64_SUPPORT",`
`18`	`18`	`"METAL_MAX_DEVICE_REGIONS=2",`
`19`		`- "RPMSG_BUFFER_SIZE=2048"`
	`19`	`+ "RPMSG_BUFFER_SIZE=512"`
`20`	`20`	`]`
`21`	`21`	`}`
`22`	`22`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@`
`17`	`17`	`"VIRTIO_DRIVER_ONLY",`
`18`	`18`	`"NO_ATOMIC_64_SUPPORT",`
`19`	`19`	`"METAL_MAX_DEVICE_REGIONS=2",`
`20`		`- "RPMSG_BUFFER_SIZE=100"`
	`20`	`+ "RPMSG_BUFFER_SIZE=512"`
`21`	`21`	`]`
`22`	`22`	`}`
`23`	`23`	`}`