Skip to content

Feature FMEA/DFA for baselibs#2777

Merged
PandaeDo merged 2 commits intomainfrom
aschemmel-tech-baselibs-safety-analysis
Apr 20, 2026
Merged

Feature FMEA/DFA for baselibs#2777
PandaeDo merged 2 commits intomainfrom
aschemmel-tech-baselibs-safety-analysis

Conversation

@aschemmel-tech
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

⚠️ Docs-as-Code version mismatch detected
Please check the CI build logs for details and align the documentation version with the Bazel dependency.

@github-actions
Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-baselibs-safety-analysis branch 2 times, most recently from fb28f1c to ca3a65c Compare April 17, 2026 14:23
@aschemmel-tech aschemmel-tech marked this pull request as ready for review April 17, 2026 14:23
@aschemmel-tech aschemmel-tech requested review from a team, PandaeDo and masc2023 as code owners April 17, 2026 14:23
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

DFA document is still draft, where the dfa need are already valid?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the hint, after consulting the guideline, I set the needs which are "not sufficient" to invalid.

@aschemmel-tech aschemmel-tech force-pushed the aschemmel-tech-baselibs-safety-analysis branch from ca3a65c to e612595 Compare April 20, 2026 06:26
Copy link
Copy Markdown
Contributor

@PandaeDo PandaeDo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't get the FMEA approach for the moment. Looking forward for your explaination in the meeting.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would expect :security: YES because the architecture is also security related.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, this came from error in template - @PandaeDo will correct the templates

* - UI_01_04
- Deadlocks
- yes
- shared memory or file access may lead to deadlock, link to need
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the link to the need

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

* - UI_01_06
- Blocking of execution
- yes
- As baselibs are in context of a application, they may block execution, link to need
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the link to the need

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

* - UI_01_10
- CPU time depletion
- yes
- Baselibs may deplete or suffer from depletion, link to need
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the link to the need

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

* - UI_01_02
- Read/Write access to memory allocated to another software element
- yes
- As baselibs are in context of a application, they may access their memory, link to need
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the link to the need

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

:failure_effect: memory of using component may be corrupted leading to safety requirement violation
:mitigation_issue: https://github.com/eclipse-score/score/issues/2816
:sufficient: no
:status: invalid
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the status correct? Until the mitigation is open it's sufficent to show this with the attribute sufficient.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, change to valid and also the doc status. Error in process reqs (mitigation) - @PandaeDo will correct

Comment thread docs/features/baselibs/docs/safety_analysis/dfa.rst
:failure_effect: Using application is blocked from execution and thus cannot fulfill its safety function
:mitigation_issue: https://github.com/eclipse-score/score/issues/2816
:sufficient: no
:status: invalid
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the status correct? Until the mitigation is open it's sufficent to show this with the attribute sufficient.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

:status: draft
:status: valid
:safety: ASIL_B
:security: NO
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same question here as in DFA. Would expect :security: yes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Copy link
Copy Markdown
Contributor

@PandaeDo PandaeDo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@PandaeDo PandaeDo merged commit f3bed3a into main Apr 20, 2026
8 checks passed
@PandaeDo PandaeDo deleted the aschemmel-tech-baselibs-safety-analysis branch April 20, 2026 13:04
@github-project-automation github-project-automation Bot moved this from Backlog to Done in S-CORE Roadmap Apr 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants