aboutsummaryrefslogtreecommitdiff
path: root/applications/message_handler_openamp
AgeCommit message (Collapse)Author
2024-04-22Add missing mutex/semaphore destroy overridesJonny Svärd
Override NPU driver weak function prototypes to properly free up resources after use. Change-Id: I4f3141d6106e5feeb1523452ca5fe2dbe66903be Signed-off-by: Jonny Svärd <jonny.svaerd@arm.com>
2024-04-19Fix rpmsg not being zero initializedMikael Olsson
The rpmsg struct used in response messages is not zero initialized so later response messages may contain old data. To resolve this, the rpmsg struct is now zero initialized for each new message. Change-Id: I116fa443eca7b6ec3d8817909fd3c59bd69427cb Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-04-16Fix zero init of class members in message handlerMikael Olsson
There are instances of class members not being zero initialized in the message handler. To avoid possible issues with using the class members when they have an intermediate value, the constructors have been updated to zero initialize all the member variables. Change-Id: Ib539410a8f1486f8155272731f5cb7006c22ac7a Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-04-02Fix semaphore clean up in RProc destructorMikael Olsson
The semaphore created in the RProc constructor is never cleaned up. To resolve this, the destructor will now delete the semaphore. Change-Id: Ib641a87129bacf3d7ba097c4e89aa77fa255e2f8 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-04-02Fix Rpmsg constructor not initializing membersMikael Olsson
The constructor for the Rpmsg class does not zero-initialize its member variables. This causes issues with the remoteproc library because it thinks function pointers in the variables are set to valid functions when they are actually just uninitialized values and using these function pointer causes crashes and unexpected behavior. To resolve this, the Rpmsg class constructor has been changed to zero-initialize all its member variables. Change-Id: Ic34fd52bc39ed3018e5f67474949f10adc8cb7a8 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-03-21Change tensor arena size to be a power of twoMikael Olsson
The size of the memory used for the message handler's tensor arena will always be rounded up to the next power of two when the Linux kernel allocates it. Therefore, to align the tensor arena size with how the memory is allocated and be able to use the full size allocated, the tensor arena size must now be a power of two. Change-Id: Ib67e77d1ffeb233ad12762a56ddbc20d26c71052 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-02-13Update rpmsg type names in message handlerMikael Olsson
With the restructuring of the Linux kernel driver source files, some macros and types in the rpmsg header was renamed to better namespace their usage. The message handler openamp application has been updated accordingly. Change-Id: Ibfc11ced58cddd23463bb2060d658cc9b374e035 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-02-12Update kernel rpmsg header path in message handlerMikael Olsson
The source files for the Linux kernel driver has been restructured and the common rpmsg header file has been renamed and moved to an include folder. The message handler openamp application has been updated accordingly to use the new header file. Change-Id: If1c4f5f80dbd87ee1b14103620b1183f6a3e3826 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-01-16Remove cancel error for inference not in queueMikael Olsson
Currently if the inference to cancel is not found in the inference queue, the message_handler_openamp will response with an error. The inference may already be running and therefore not be in the queue anymore or the inference is already done and the response is in the mailbox queue. In either case, responding with an error when the inference is not in the queue isn't useful so the check has been removed. The Linux kernel driver knows what inferences that are inflight and will track what inferences that have been cancelled and handle it accordingly. Change-Id: I780e0e08a68e458221649bc2cecf7d665adc3c33 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2024-01-16Remove designated initializer usage in remoteprocMikael Olsson
Designated initializer is not part of the C++ standard until C++20 and requires a compiler extension to work in earlier versions. To ensure the message_handler_openamp can be compiled with compilers without this extension the usage has been removed. Change-Id: I97c90df6691302cc470b8538f515a1b1ceb385b9 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2023-12-21Change to accumulate PMU values in message handlerMikael Olsson
Currently the message_handler_openamp overwrites the PMU values in the response message each time operators are offloaded the NPU in an inference request. To instead get the total PMU values for all the times the NPU was used during the inference, the PMU values are now accumulated instead. Change-Id: I7fb8697e441b362f2f6da8a26536ba51d17c6b4c Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2023-12-19Add app support for core_driver inference timeoutJonny Svärd
Update example applications to support new core_driver inference timeout feature. Change-Id: Ib7283421aa8db9a3b5f1dd6d1bd44803491f0399 Signed-off-by: Jonny Svärd <jonny.svaerd@arm.com>
2023-12-15Update message_handler_openamp to use NPU profilerMikael Olsson
The message_handler_openamp application now uses the NPU profiler to use a common way to collect profiling data and report it. Change-Id: Ia2a860c790facc5193aca065636c58e1b5bbef22 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2023-09-28Remove address translation for buffer and carveoutMikael Olsson
Address translation for inference buffers and carveout areas have been removed because the Linux Kernel NPU driver now provides device addresses for them. Change-Id: I71a133ba5ecbb94aa2c225d259e75229b6dbce5b Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2023-09-28Add resource table entry for address translationMikael Olsson
To avoid having to use a hardcoded memory map to translate between the Linux physical addresses and device addresses, a new entry type has been added to the resource table where the host can provide the memory map to use for the translation. Change-Id: I668bba3aeac27c358647c969c5504326e1c91d24 Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2023-09-21Set mailbox interrupt priority in message handlerMikael Olsson
When the mailbox interrupt is triggered, the mailbox interrupt handler will invoke the remote processor mailbox callback, which in turn will use FreeRTOS's xSemaphoreGiveFromISR to wake up a task to handle the new message. Until now, the interrupt priority for the mailbox interrupt has been left at its default value but this stopped working since FreeRTOS kernel V10.6.0 because since that version, the ISR functions will only work from an interrupt with the same or lower priority than configMAX_SYSCALL_INTERRUPT_PRIORITY in the FreeRTOS configuration. Therefore, a priority is now set for the mailbox interrupt that is compatible with the FreeRTOS configuration used. Change-Id: I0186fdc9951dfa73a2692ba95530094abb0e4d4a Signed-off-by: Mikael Olsson <mikael.olsson@arm.com>
2023-06-15Add trace resource entry in firmware's resource tableLedion Daja
Patch adds an optional trace resource entry which can be enabled via build flags. This patch also changes the resource table into a plain C struct, which allows for link time assignment of trace resource address and size. Change-Id: I6ca3260eb9b13a9bf511c95cd54c57d554c0d997
2023-03-14Fix SEI CERT coding standard warningsLedion Daja
Change-Id: I47fdf29a50440c82fd9ccf111a846bf87e8a9e29
2023-02-10Fix size of TFLM arenaKristofer Jonsson
The TFLM arena size was incorrectly surrounded by sizeof(), which caused the arena size to be allocated to only 4 bytes. Change-Id: Iea8c76e0ce434973c9d0a7f696cfb51af49a48fd
2023-02-08Updating message handler firmware23.02-rc1Kristofer Jonsson
The 'message handler' firmware was based on a custom interface between Linux and the firmware. Because the kernel driver has been converted into a rpmsg driver, the 'message handler' application has been updated into an OpenAMP based firmware. Change-Id: I1339180c4f53cbad42501a2827863b7b49561ff4