Change MessageQueueId to handle that internally #205

New Issue

gaisser · 2020-09-22T15:54:15+02:00

gaisser commented

2020-09-22 15:54:15 +02:00

The current FreeRTOS Version uses a QueueHandle_t pointer to handles its queues, but the FSFW does only support Numeric Queue Identifiers. This is handle by a dirty hack which casts the QueueHandle_t to a void pointer which is 32 bit on a 32 Bit System but brakes on any others.

To fix that, the FSFW should use a map internally to handle the conversion of numeric to OSAL Type.

The current FreeRTOS Version uses a QueueHandle_t pointer to handles its queues, but the FSFW does only support Numeric Queue Identifiers. This is handle by a dirty hack which casts the QueueHandle_t to a void pointer which is 32 bit on a 32 Bit System but brakes on any others. To fix that, the FSFW should use a map internally to handle the conversion of numeric to OSAL Type.

gaisser added the

feature

label 2020-09-22 15:54:15 +02:00

gaisser referenced this issue

2020-09-22 15:54:35 +02:00

MessageQueueId_t NO_QUEUE = 0 #94

gaisser referenced this issue

2020-10-27 15:04:02 +01:00

QueueMapManager #128

gaisser referenced this issue

2020-12-01 15:52:47 +01:00

QueueMapManager #128

gaisser added this to the ASTP 1.0.0 Local pools milestone 2020-12-01 15:56:44 +01:00

gaisser commented

2021-04-20 18:00:52 +02:00

Currently, I would prefer a internal solution of the OSAL like the current Host OSAL:
c50796b785/osal/host/MessageQueue.cpp (L11)

Therefore, this only relevant for FreeRTOS.

Currently, I would prefer a internal solution of the OSAL like the current Host OSAL: https://egit.irs.uni-stuttgart.de/fsfw/fsfw/src/commit/c50796b785cbed04a0b30937f94870f5c2d28f52/osal/host/MessageQueue.cpp#L11 Therefore, this only relevant for FreeRTOS.

gaisser modified the milestone from ASTP 1.0.0 Local pools to ASTP 1.1.0

2021-04-20 18:14:01 +02:00

muellerr was assigned by gaisser

2021-05-11 14:46:01 +02:00

muellerr referenced a pull request that will close this issue

2021-05-27 13:44:27 +02:00

FreeRTOS Queue Map Manager #423

gaisser closed this issue

2021-06-08 14:03:46 +02:00

Sign in to join this conversation.