subsys: tracing: Add macros for tracking list operations

[rruuaanng]

Add a set of 'SYS_PORT_TRACK_K_*' wrapper macros for visible access to the global tracking list, and a macro 'SYS_PORT_TRACK_FOREACH' for iterating over the tracking list.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

[ubieda]

@rruuaanng Can you look at the CI failure?

[rruuaanng]

@ubieda It looks like it's not an issue, the warning is caused by my syntax.

/*
  *   SYS_PORT_TRACK_FOREACH(k_mutex, m) {
  *       ...
  *   }
  */
#define SYS_PORT_TRACK_FOREACH(object, var)		\
	for (struct object *var = _track_list_##object	\
               ; var != NULL; var++)

BRACES

include/zephyr/tracing/tracking.h:67  braces {} are required around if/while/for/else

[ubieda]

@ubieda It looks like it's not an issue, the warning is caused by my syntax.

/*
  *   SYS_PORT_TRACK_FOREACH(k_mutex, m) {
  *       ...
  *   }
  */
#define SYS_PORT_TRACK_FOREACH(object, var)		\
	for (struct object *var = _track_list_##object	\
               ; var != NULL; var++)

BRACES

include/zephyr/tracing/tracking.h:67  braces {} are required around if/while/for/else

Try adding it under the ForEachMacros list:

zephyr/.clang-format

Lines 34 to 85 in 0b1cff2

	ForEachMacros:
	- 'ARRAY_FOR_EACH'
	- 'ARRAY_FOR_EACH_PTR'
	- 'FOR_EACH'
	- 'FOR_EACH_FIXED_ARG'
	- 'FOR_EACH_IDX'
	- 'FOR_EACH_IDX_FIXED_ARG'
	- 'FOR_EACH_NONEMPTY_TERM'
	- 'FOR_EACH_FIXED_ARG_NONEMPTY_TERM'
	- 'RB_FOR_EACH'
	- 'RB_FOR_EACH_CONTAINER'
	- 'SYS_DLIST_FOR_EACH_CONTAINER'
	- 'SYS_DLIST_FOR_EACH_CONTAINER_SAFE'
	- 'SYS_DLIST_FOR_EACH_NODE'
	- 'SYS_DLIST_FOR_EACH_NODE_SAFE'
	- 'SYS_SEM_LOCK'
	- 'SYS_SFLIST_FOR_EACH_CONTAINER'
	- 'SYS_SFLIST_FOR_EACH_CONTAINER_SAFE'
	- 'SYS_SFLIST_FOR_EACH_NODE'
	- 'SYS_SFLIST_FOR_EACH_NODE_SAFE'
	- 'SYS_SLIST_FOR_EACH_CONTAINER'
	- 'SYS_SLIST_FOR_EACH_CONTAINER_SAFE'
	- 'SYS_SLIST_FOR_EACH_NODE'
	- 'SYS_SLIST_FOR_EACH_NODE_SAFE'
	- '_WAIT_Q_FOR_EACH'
	- 'Z_FOR_EACH'
	- 'Z_FOR_EACH_ENGINE'
	- 'Z_FOR_EACH_EXEC'
	- 'Z_FOR_EACH_FIXED_ARG'
	- 'Z_FOR_EACH_FIXED_ARG_EXEC'
	- 'Z_FOR_EACH_IDX'
	- 'Z_FOR_EACH_IDX_EXEC'
	- 'Z_FOR_EACH_IDX_FIXED_ARG'
	- 'Z_FOR_EACH_IDX_FIXED_ARG_EXEC'
	- 'Z_GENLIST_FOR_EACH_CONTAINER'
	- 'Z_GENLIST_FOR_EACH_CONTAINER_SAFE'
	- 'Z_GENLIST_FOR_EACH_NODE'
	- 'Z_GENLIST_FOR_EACH_NODE_SAFE'
	- 'STRUCT_SECTION_FOREACH'
	- 'STRUCT_SECTION_FOREACH_ALTERNATE'
	- 'TYPE_SECTION_FOREACH'
	- 'K_SPINLOCK'
	- 'COAP_RESOURCE_FOREACH'
	- 'COAP_SERVICE_FOREACH'
	- 'COAP_SERVICE_FOREACH_RESOURCE'
	- 'HTTP_RESOURCE_FOREACH'
	- 'HTTP_SERVER_CONTENT_TYPE_FOREACH'
	- 'HTTP_SERVICE_FOREACH'
	- 'HTTP_SERVICE_FOREACH_RESOURCE'
	- 'I3C_BUS_FOR_EACH_I3CDEV'
	- 'I3C_BUS_FOR_EACH_I2CDEV'
	- 'MIN_HEAP_FOREACH'

[ubieda]

Keep in mind we'll have to move this to 4.3.0, since 4.2.0 is about to be released now.

[kartben]

Prob better to even make it a request for change then (although I understand your intention was to show your approval in principle) to avoid fly-by merge when merge window re-opens

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[ubieda]

Would you mind rebasing to resolve conflicts?