Merge 9febfa7cdc into 48a4939c9f

- In list.h fix a typo in comments.
- In portable/ThirdParty/GCC/Posix/port.c fix compiler warning about
  "gcc -Wwrite-strings".

Signed-off-by: Florian La Roche <Florian.LaRoche@gmail.com>
Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>

examples/cmake_example/main.c

@@ -39,8 +39,6 @@
 #include <queue.h>
 #include <timers.h>
 #include <semphr.h>
-
-/* Standard includes. */
 #include <stdio.h>
 
 /*-----------------------------------------------------------*/
@@ -62,6 +60,32 @@ static void exampleTask( void * parameters )
 }
 /*-----------------------------------------------------------*/
 
+/* Test uxTaskGetAllHandles API before starting the scheduler */
+static void test_uxTaskGetAllHandles(void)
+{
+    UBaseType_t uxCount, uxFilled, i;
+    TaskHandle_t *pxHandles;
+
+    /* First, query the number of tasks (should be 1: exampleTask, before scheduler starts) */
+    uxCount = uxTaskGetAllHandles(NULL, 0);
+    printf("[uxTaskGetAllHandles] Number of tasks: %lu\n", (unsigned long)uxCount);
+
+    if (uxCount > 0) {
+        pxHandles = (TaskHandle_t *)pvPortMalloc(sizeof(TaskHandle_t) * uxCount);
+        if (pxHandles != NULL) {
+            uxFilled = uxTaskGetAllHandles(pxHandles, uxCount);
+            printf("[uxTaskGetAllHandles] Handles returned: %lu\n", (unsigned long)uxFilled);
+            for (i = 0; i < uxFilled; i++) {
+                printf("  Handle[%lu]: %p\n", (unsigned long)i, (void *)pxHandles[i]);
+            }
+            vPortFree(pxHandles);
+        } else {
+            printf("[uxTaskGetAllHandles] pvPortMalloc failed!\n");
+        }
+    }
+}
+/*-----------------------------------------------------------*/
+
 int main( void )
 {
     static StaticTask_t exampleTaskTCB;

include/list.h

@@ -44,7 +44,7 @@
  *
  * In addition to it's value, each list item contains a pointer to the next
  * item in the list (pxNext), a pointer to the list it is in (pxContainer)
- * and a pointer to back to the object that contains it.  These later two
+ * and a pointer back to the object that contains it.  These later two
  * pointers are included for efficiency of list manipulation.  There is
  * effectively a two way link between the object containing the list item and
  * the list item itself.

portable/ThirdParty/GCC/Posix/port.c

@@ -193,7 +193,7 @@ void prvFatalError( const char * pcCall,
 }
 /*-----------------------------------------------------------*/
 
-static void prvPortSetCurrentThreadName( char * pxThreadName )
+static void prvPortSetCurrentThreadName( const char * pxThreadName )
 {
     #ifdef __APPLE__
         pthread_setname_np( pxThreadName );

tasks.c

@@ -4514,6 +4514,73 @@ char * pcTaskGetName( TaskHandle_t xTaskToQuery )
 #endif /* configUSE_TRACE_FACILITY */
 /*----------------------------------------------------------*/
 
+UBaseType_t uxTaskGetAllHandles(TaskHandle_t *pxTaskHandleArray, UBaseType_t uxArraySize)
+{
+    UBaseType_t uxCount = 0, uxQueue;
+    TCB_t *pxTCB;
+    List_t *pxList;
+    ListItem_t *pxIterator, *pxEndMarker;
+
+    vTaskSuspendAll();
+    {
+        /* Ready lists */
+        for (uxQueue = 0; uxQueue < configMAX_PRIORITIES; uxQueue++) {
+            pxList = &pxReadyTasksLists[uxQueue];
+            pxEndMarker = listGET_END_MARKER(pxList);
+            for (pxIterator = listGET_HEAD_ENTRY(pxList);
+                 pxIterator != pxEndMarker;
+                 pxIterator = listGET_NEXT(pxIterator)) {
+                pxTCB = listGET_LIST_ITEM_OWNER(pxIterator);
+                if (uxCount < uxArraySize && pxTaskHandleArray)
+                    pxTaskHandleArray[uxCount] = (TaskHandle_t)pxTCB;
+                uxCount++;
+            }
+        }
+        /* Delayed lists */
+        List_t *delayedLists[] = { pxDelayedTaskList, pxOverflowDelayedTaskList };
+        for (int i = 0; i < 2; i++) {
+            pxList = delayedLists[i];
+            pxEndMarker = listGET_END_MARKER(pxList);
+            for (pxIterator = listGET_HEAD_ENTRY(pxList);
+                 pxIterator != pxEndMarker;
+                 pxIterator = listGET_NEXT(pxIterator)) {
+                pxTCB = listGET_LIST_ITEM_OWNER(pxIterator);
+                if (uxCount < uxArraySize && pxTaskHandleArray)
+                    pxTaskHandleArray[uxCount] = (TaskHandle_t)pxTCB;
+                uxCount++;
+            }
+        }
+#if (INCLUDE_vTaskSuspend == 1)
+        /* Suspended list */
+        pxList = &xSuspendedTaskList;
+        pxEndMarker = listGET_END_MARKER(pxList);
+        for (pxIterator = listGET_HEAD_ENTRY(pxList);
+             pxIterator != pxEndMarker;
+             pxIterator = listGET_NEXT(pxIterator)) {
+            pxTCB = listGET_LIST_ITEM_OWNER(pxIterator);
+            if (uxCount < uxArraySize && pxTaskHandleArray)
+                pxTaskHandleArray[uxCount] = (TaskHandle_t)pxTCB;
+            uxCount++;
+        }
+#endif
+#if (INCLUDE_vTaskDelete == 1)
+        /* Deleted list (waiting for cleanup) */
+        pxList = &xTasksWaitingTermination;
+        pxEndMarker = listGET_END_MARKER(pxList);
+        for (pxIterator = listGET_HEAD_ENTRY(pxList);
+             pxIterator != pxEndMarker;
+             pxIterator = listGET_NEXT(pxIterator)) {
+            pxTCB = listGET_LIST_ITEM_OWNER(pxIterator);
+            if (uxCount < uxArraySize && pxTaskHandleArray)
+                pxTaskHandleArray[uxCount] = (TaskHandle_t)pxTCB;
+            uxCount++;
+        }
+#endif
+    }
+    (void)xTaskResumeAll();
+    return uxCount;
+}
+
 #if ( INCLUDE_xTaskGetIdleTaskHandle == 1 )
 
     #if ( configNUMBER_OF_CORES == 1 )