-
Notifications
You must be signed in to change notification settings - Fork 98
Results comp FMEA #2879
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Results comp FMEA #2879
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -18,33 +18,131 @@ FMEA (Failure Modes and Effects Analysis) | |||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. document:: result FMEA | ||||||||||||||||||||||||||||||||||||||||||||||
| :id: doc__result_fmea | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: draft | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: valid | ||||||||||||||||||||||||||||||||||||||||||||||
| :safety: ASIL_B | ||||||||||||||||||||||||||||||||||||||||||||||
| :security: NO | ||||||||||||||||||||||||||||||||||||||||||||||
| :realizes: wp__sw_component_fmea | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. note:: Use the content of the document to describe e.g. why a fault model is not applicable for the diagram. | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Failure Mode List | ||||||||||||||||||||||||||||||||||||||||||||||
| ----------------- | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. code-block:: rst | ||||||||||||||||||||||||||||||||||||||||||||||
| Fault Models for sequence diagrams | ||||||||||||||||||||||||||||||||||||||||||||||
| .. list-table:: Fault Models for sequence diagrams | ||||||||||||||||||||||||||||||||||||||||||||||
| :header-rows: 1 | ||||||||||||||||||||||||||||||||||||||||||||||
| :widths: 10,20,10,20 | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| * - ID | ||||||||||||||||||||||||||||||||||||||||||||||
| - Failure Mode | ||||||||||||||||||||||||||||||||||||||||||||||
| - Applicability | ||||||||||||||||||||||||||||||||||||||||||||||
| - Rationale | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_01 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message is not received (is a subset/more precise description of MF_01_05) | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - If set result was not received before the get value/error are called, this will lead to an exception/terminate. In case of the get value user defined defaults are provided. | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_02 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message received too late (only relevant if delay is a realistic fault) | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - Do not see this as a problem for result lib, would lead to the same consideration as in MF_01_01 | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_03 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message received too early (usually not a problem) | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - No problem for result lib | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_04 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message not received correctly by all recipients (different messages or messages partly lost). Only relevant if the same message goes to multiple recipients. | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - No multiple recipients (maybe from different threads?) | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_05 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message is corrupted | ||||||||||||||||||||||||||||||||||||||||||||||
| - yes | ||||||||||||||||||||||||||||||||||||||||||||||
| - Error message string is destroyed before accessing it by the user - see :need:`comp_saf_fmea__result__error_message_unavail` | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_06 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message is not sent | ||||||||||||||||||||||||||||||||||||||||||||||
| - yes | ||||||||||||||||||||||||||||||||||||||||||||||
| - Value or error are not returned - see :need:`comp_saf_fmea__result__no_return` | ||||||||||||||||||||||||||||||||||||||||||||||
| * - MF_01_07 | ||||||||||||||||||||||||||||||||||||||||||||||
| - message is unintended sent | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - not applicable for a library | ||||||||||||||||||||||||||||||||||||||||||||||
| * - CO_01_01 | ||||||||||||||||||||||||||||||||||||||||||||||
| - minimum constraint boundary is violated | ||||||||||||||||||||||||||||||||||||||||||||||
| - yes | ||||||||||||||||||||||||||||||||||||||||||||||
| - Used enum types may not match - see :need:`comp_saf_fmea__result__enum_type_mismatch` | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
| * - CO_01_02 | ||||||||||||||||||||||||||||||||||||||||||||||
| - maximum constraint boundary is violated | ||||||||||||||||||||||||||||||||||||||||||||||
| - yes | ||||||||||||||||||||||||||||||||||||||||||||||
| - same as above | ||||||||||||||||||||||||||||||||||||||||||||||
| * - EX_01_01 | ||||||||||||||||||||||||||||||||||||||||||||||
| - Process calculates wrong result(s) (is a subset/more precise description of MF_01_05 or MF_01_04). This failure mode is related to the analysis if e.g. internal safety mechanisms are required (level 2 function, plausibility check of the output, …) because of the size / complexity of the feature. | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - Due to low complexity of the component this error is completely eliminated by testing. Low complex architecture according to criteria in :need:`gd_chklst__arch_inspection_checklist` ARC_03_03 and design complexity below numbers as in :need:`gd_req__impl_complexity_analysis` | ||||||||||||||||||||||||||||||||||||||||||||||
| * - EX_01_02 | ||||||||||||||||||||||||||||||||||||||||||||||
| - processing too slow (only relevant if timing is considered) | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - Due to the small functionality, being too slow is no likely issue. | ||||||||||||||||||||||||||||||||||||||||||||||
| * - EX_01_03 | ||||||||||||||||||||||||||||||||||||||||||||||
| - processing too fast (only relevant if timing is considered) | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - Get functions only deliver data when called, no "too fast" is possible. | ||||||||||||||||||||||||||||||||||||||||||||||
| * - EX_01_04 | ||||||||||||||||||||||||||||||||||||||||||||||
| - loss of execution | ||||||||||||||||||||||||||||||||||||||||||||||
| - yes | ||||||||||||||||||||||||||||||||||||||||||||||
| - Loss of execution leads to the same error as MF_01_06 | ||||||||||||||||||||||||||||||||||||||||||||||
| * - EX_01_05 | ||||||||||||||||||||||||||||||||||||||||||||||
| - processing changes to arbitrary process | ||||||||||||||||||||||||||||||||||||||||||||||
| - no | ||||||||||||||||||||||||||||||||||||||||||||||
| - Not a problem of result lib as this is a libray and not a process | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
| * - EX_01_06 | ||||||||||||||||||||||||||||||||||||||||||||||
| - processing is not complete (infinite loop) | ||||||||||||||||||||||||||||||||||||||||||||||
| - yes | ||||||||||||||||||||||||||||||||||||||||||||||
| - User gives back a function as return which induces stop of user execution - see :need:`comp_saf_fmea__result__stop_user` | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| FMEA | ||||||||||||||||||||||||||||||||||||||||||||||
| ---- | ||||||||||||||||||||||||||||||||||||||||||||||
| For all identified applicable failure initiators, the FMEA is performed in the following section. | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. comp_saf_fmea:: Result Enum Type Mismatch | ||||||||||||||||||||||||||||||||||||||||||||||
| :violates: comp_arc_dyn__baselibs__result | ||||||||||||||||||||||||||||||||||||||||||||||
| :id: comp_saf_fmea__result__enum_type_mismatch | ||||||||||||||||||||||||||||||||||||||||||||||
| :fault_id: CO_01_01 | ||||||||||||||||||||||||||||||||||||||||||||||
| :failure_effect: User would understand a wrong error type (based on different error domains) | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigation_issue: https://github.com/eclipse-score/score/issues/2880 | ||||||||||||||||||||||||||||||||||||||||||||||
| :sufficient: no | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: valid | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| Only if the user would use the error information not only for debug reasons but for selecting the | ||||||||||||||||||||||||||||||||||||||||||||||
| type of error reaction this error may have an error impact. We need to make the user aware of this. | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. comp_saf_fmea:: Result Error Message Unavailability | ||||||||||||||||||||||||||||||||||||||||||||||
| :violates: comp_arc_dyn__baselibs__result | ||||||||||||||||||||||||||||||||||||||||||||||
| :id: comp_saf_fmea__result__error_message_unavail | ||||||||||||||||||||||||||||||||||||||||||||||
| :fault_id: MF_01_05 | ||||||||||||||||||||||||||||||||||||||||||||||
| :failure_effect: Accessing error message could result in undefined behaviour | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigated_by: aou_req__result__resource_lifetime | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigation_issue: https://github.com/eclipse-score/score/issues/2880 | ||||||||||||||||||||||||||||||||||||||||||||||
| :sufficient: no | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: valid | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| The linked AoU cares about unavailability of other return objects, but also the error message may be unavailable. | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. comp_saf_fmea:: <Title> | ||||||||||||||||||||||||||||||||||||||||||||||
| :violates: <Component architecture> | ||||||||||||||||||||||||||||||||||||||||||||||
| :id: comp_saf_fmea__<Component>__<Element descriptor> | ||||||||||||||||||||||||||||||||||||||||||||||
| :fault_id: <ID from fault model :need:`gd_guidl__fault_models`> | ||||||||||||||||||||||||||||||||||||||||||||||
| :failure_effect: "description of failure effect of the fault model on the element" | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigated_by: <ID from Component Requirement | ID from AoU Component Requirement> | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigation_issue: <ID from Issue Tracker> | ||||||||||||||||||||||||||||||||||||||||||||||
| :sufficient: <yes|no> | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: <valid|invalid> | ||||||||||||||||||||||||||||||||||||||||||||||
| .. comp_saf_fmea:: Result No Return | ||||||||||||||||||||||||||||||||||||||||||||||
| :violates: comp_arc_dyn__baselibs__result | ||||||||||||||||||||||||||||||||||||||||||||||
| :id: comp_saf_fmea__result__no_return | ||||||||||||||||||||||||||||||||||||||||||||||
| :fault_id: MF_01_06 | ||||||||||||||||||||||||||||||||||||||||||||||
| :failure_effect: Accessing value object could result in undefined behaviour (e.g. usage of wrong value) | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigated_by: aou_req__result__value_handling, aou_req__result__error_reaction | ||||||||||||||||||||||||||||||||||||||||||||||
| :sufficient: yes | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: valid | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. correction: effect is not UB, but deterministic termination. also I don't think
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. note:: argument is inside the 'content'. Therefore content is mandatory | ||||||||||||||||||||||||||||||||||||||||||||||
| If a value or a error is not returned this will be noticed by the user and reacted upon. This is ensured | ||||||||||||||||||||||||||||||||||||||||||||||
| additionally by the provided AoU. | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| .. attention:: | ||||||||||||||||||||||||||||||||||||||||||||||
| The above directive must be updated according to your component FMEA. | ||||||||||||||||||||||||||||||||||||||||||||||
| .. comp_saf_fmea:: Result Stop User | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Now when I think about this one it seems too generic. Any library accepting callbacks might suffer from this.
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would keep - we have a generic (platform level) AoU to cover this. |
||||||||||||||||||||||||||||||||||||||||||||||
| :violates: comp_arc_dyn__baselibs__result | ||||||||||||||||||||||||||||||||||||||||||||||
| :id: comp_saf_fmea__result__stop_user | ||||||||||||||||||||||||||||||||||||||||||||||
| :fault_id: EX_01_06 | ||||||||||||||||||||||||||||||||||||||||||||||
| :failure_effect: User could be stopped by a function provided as a result from another user | ||||||||||||||||||||||||||||||||||||||||||||||
| :mitigated_by: aou_req__platform__flow_monitoring | ||||||||||||||||||||||||||||||||||||||||||||||
| :sufficient: yes | ||||||||||||||||||||||||||||||||||||||||||||||
| :status: valid | ||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||
| - The above "code-block" directive must be updated | ||||||||||||||||||||||||||||||||||||||||||||||
| - Fill in all the needed information in the <brackets> | ||||||||||||||||||||||||||||||||||||||||||||||
| Stopping its own execution has to be managed by the user via program flow monitoring, see AoU. | ||||||||||||||||||||||||||||||||||||||||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
In that case I would propose some changes that provide a bit more clarity:
Suggested change
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ok |
||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok