AoU: ETAS_RTACAR_SM_OS_184¶
This AoU checks that calls to SuspendAllInterrupts() and ResumeAllInterrupts() are correctly paired and properly
nested along all feasible execution paths, according to the ETAS_RTACAR_SM_OS_184 requirement.
The validation procedure simulates a push/pop stack of SuspendAllInterrupts calls and verifies that:
Every
SuspendAllInterrupts()call is matched by a correspondingResumeAllInterrupts()call.The nesting of suspend/resume pairs is strictly respected along all execution paths.
No path reaches a function exit point with an unbalanced (open) suspend region.
Rationale for the validation
The validation process follows these steps:
Generation of the base code call graph
Collection of source code files to be analyzed, as specified by the compile_commands.json file.
Generation of the Abstract Syntax Tree (AST) for each source file.
Processing of the AST to identify function implementations and related calls.
Resolution of all the calls to the linkable target implementation.
Inspection of the complete call graph to detect restricted regions.
Detect the start point of each restricted region.
Detect the end point of each restricted region.
Inspection of the call graph contained within the restricted regions.
Identify all function calls within the restricted regions.
Check each function call against the list of prohibited calls.
Known Limitations¶
During the generation of the call graph and the collection of source code files to be analyzed, files with the following extensions are ignored:
.cpp
.hpp
.s
.S
.asm
.inc
The generation of the call graph is based on static analysis of the source code. This means that the tool analyzes the code without executing it, which can lead to certain limitations:
It may not capture all dynamic behaviors, such as those resulting from runtime conditions or external inputs.
It cannot account for all possible execution paths, especially in the presence of complex control flows.
The accuracy of the call graph is dependent on the completeness and correctness of the source code being analyzed.
Important
Link Os as external library functions
This behavior is important for maintaining the integrity of the analysis process. The tool expects that the functions implementing the OS are linked as external library functions, and therefore does not expect the source code for these functions to be available within the analyzed base code.
The tool’s ability to detect function calls is limited by the information available from the source code, and it’s possible that the tool encounters limitations:
Important
Handling of External Functions
The tool may be unable to resolve function calls if their implementations are not available in the analyzed source code, such as functions provided by the OS, RTE, or external libraries.
Important
Analysis Limitations
The tool’s static analysis may not detect all function calls, particularly in cases involving complex code structures or the use of function pointers.
When the start or the end point of a restricted region is contained inside a conditional construct, it may lead to uncertainties in the identification of the restricted region due to the potential for the condition to affect the execution flow.
Example: Restricted region within a conditional construct
Consider the following C code snippet, where the start of a restricted region is inside an if statement:
void example_function(void)
{
if (condition)
{
// Start of restricted region
startRegion();
}
// End of restricted region
endRegion();
}
This example illustrates how the static analysis tool may struggle to accurately identify the boundaries of restricted regions, when they are nested within conditional constructs.
The tool may not be able to determine all possible execution paths, leading to uncertainty in identifying the exact boundaries of the restricted region. A result indicating a human review may be reported if the analysis cannot definitively establish the boundaries.
Example: Restricted region within the same conditional construct
Consider the following C code snippet, where both the start and end points of a restricted region are inside the if statement:
void example_function(void)
{
if (condition)
{
// Start of restricted region
startRegion();
// End of restricted region
endRegion();
}
}
In this case, the static analysis tool may have an easier time determining the boundaries of the restricted region, as both the start and end points are clearly defined within the same conditional construct.
The static analysis tool may leverage this information to more accurately identify the restricted region and its boundaries, potentially reducing the need for human review.
When the tool detects a limitation in its analysis, it shall report the issue to the user for manual review, indicating the specific code section that requires attention. Please refer to the OS reviews section for more information on how to handle such cases.
Important
Os API call region delimiters
The tool detects the start of a critical region when a call to the Os_SuspendAllInterrupts() function is made. The end of the critical region is detected when a corresponding call to Os_ResumeAllInterrupts() is made.
Function macro delimiters¶
The following couples of function macros are detected as critical region delimiters:
SuspendAllInterrupts() / ResumeAllInterrupts()
Rte_SuspendAllInterrupts() / Rte_ResumeAllInterrupts()
Rte_GetResource() / Rte_ReleaseResource()
When the region starts/ends due to a function macro call, the tool will consider the entire macro body as part of the region delimiter, so no further analysis will be performed within the expansion code of the macros.
Attention
The tool recognizes only the listed macros as critical region delimiters. In case of usage of unsupported macros, or new macros mapped to the supported ones, the detection process will be downgraded to the analysis of the call graph considering the full expansion of the macro.
Note
Macro Delimiters verification
Before considering a macro as region delimiter, a check to the macro expansion code is performed: only when, inside the body of the macro expansion, a call to the expected Os API delimiter functions is found, the macro itself is considered as region delimiter. This approach helps to cover cases where the macros are defined by invoking functions that act on different primitives than the ones expected by the check.
Rule Id |
ETAS_RTACAR_SM_OS_184 |
|---|---|
Title |
SuspendAllInterrupts/ResumeAllInterrupts calls must be properly paired and nested. |
Description |
This rule verifies that each call to SuspendAllInterrupts is properly matched with a corresponding call to ResumeAllInterrupts, and that the nesting of suspend/resume pairs is correct along all feasible execution paths. |
Message Description¶
Id |
id_000 |
Message |
Issue detected while executing the verification. |
Kind |
Open |
Level |
None |
Description |
An internal exception occurred during the execution. This can happen if there are insufficient resources available on the host machine. For more details, please consult the RTA-SQF log file. |
How To Fix |
Restart the command to exclude any internal exceptions. Alternatively, please contact ETAS GmbH support for assistance. When reporting the error, include a sample configuration and command line that reproduces the issue. |
Id |
id_001 |
Message |
The SuspendAllInterrupts/ResumeAllInterrupts nesting was checked and proper nesting was found. |
Kind |
Pass |
Level |
None |
Description |
The SuspendAllInterrupts/ResumeAllInterrupts calls are correctly paired and nested along all analyzed execution paths. |
How To Fix |
Everything is in order. No action is required. |
Id |
id_002 |
Message |
The SuspendAllInterrupts/ResumeAllInterrupts nesting was checked and improper nesting was found. |
Kind |
Review |
Level |
None |
Description |
The analysis detected an unbalanced or improperly nested SuspendAllInterrupts/ResumeAllInterrupts pair on one or more execution paths. |
How To Fix |
Please review the code to ensure every SuspendAllInterrupts() call is matched by a corresponding ResumeAllInterrupts() call along all execution paths. |
Id |
id_003 |
Message |
The SuspendAllInterrupts/ResumeAllInterrupts nesting could not be determined due to the presence of a conditional statement. The tool does not perform data flow analysis, manual review is required. |
Kind |
Review |
Level |
None |
Description |
A conditional statement obscures the boundaries of the SuspendAllInterrupts/ResumeAllInterrupts section. The tool cannot statically resolve the nesting. |
How To Fix |
Please manually review the code to verify that SuspendAllInterrupts/ResumeAllInterrupts calls are correctly paired along all conditional execution paths. |
Id |
id_004 |
Message |
The SuspendAllInterrupts/ResumeAllInterrupts nesting could not be fully inspected due to the presence of unresolved call targets. |
Kind |
Review |
Level |
None |
Description |
The call graph contains unresolved call targets (e.g., function pointers or indirect calls) that prevent complete nesting analysis. |
How To Fix |
Please manually review the affected code path to ensure SuspendAllInterrupts/ResumeAllInterrupts calls are correctly paired. |
Id |
id_005 |
Message |
Usage of SuspendAllInterrupts/ResumeAllInterrupts Os APIs was not found. |
Kind |
Pass |
Level |
None |
Description |
No SuspendAllInterrupts or ResumeAllInterrupts API calls were detected in the analyzed artifacts. |
How To Fix |
Everything is in order. No action is required. |
Id |
id_006 |
Message |
The analysis of the source file is not supported due to the file extension. The file was skipped and shall be reviewed manually. |
Kind |
Review |
Level |
None |
Description |
The source file has an extension that is not supported by the analysis tool and was skipped. |
How To Fix |
Please manually review the skipped file to ensure SuspendAllInterrupts/ResumeAllInterrupts calls are correctly paired. |