txm_module_manager_absolute_load create wrong data start pointer

[amgross]

Describe the bug
In _txm_module_manager_absolute_load There is:

threadx/common_modules/module_manager/src/txm_module_manager_absolute_load.c

Line 314 in 7ad78c4

module_instance -> txm_module_instance_data_start = (VOID *) (((CHAR *) module_preamble->txm_module_preamble_code_size) + sizeof(TXM_MODULE_PREAMBLE));

This causes the txm_module_instance_data_start to be defined as size of struct + size of code, which not make sense to me, as it not address (and this variable should be address, as it used for things like MPU and checking if things located in memory area).

Expected behavior
txm_module_instance_data_start should contain the module data memory pointer

Impact
In edge cases like usage of MPU or objects that located in this memory things may fail

[fdesbiens]

Thank you for the bug report @amgross. We will discuss it.

[amgross]

I would just want to add about my MPU point, that probably (although not checked) the current MPU code won't work in current absolute load.
This is because in different than the other loads here the main data is separated from the start stop+callback threads stack (main data is in absolute place and stacks allocated on byte pool in runtime).
I would consider make it similar to all other loads and but the stacks in the beginning of the (absolute located) main data

[fdesbiens]

@eclipse-threadx/iot-threadx-committers and @eclipse-threadx/iot-threadx-contributors: Please provide your feedback.