Skip to content

Cross-VM IPC (#916)#1371

Closed
FScholPer wants to merge 257 commits intomainfrom
916-fq-non-ipc-shared-memory-com
Closed

Cross-VM IPC (#916)#1371
FScholPer wants to merge 257 commits intomainfrom
916-fq-non-ipc-shared-memory-com

Conversation

@FScholPer
Copy link
Contributor

@FScholPer FScholPer commented Jul 4, 2025

Added Feature Request description and requirements

Frank Scholter Peres frank.scholter_peres@mercedes-benz.com, Mercedes-Benz Tech Innovation GmbH
Provider Information

@FScholPer FScholPer linked an issue Jul 4, 2025 that may be closed by this pull request
Feature Modification for Cross-VM IPC #916
Open
timkl7
timkl7 suggested changes Jul 7, 2025
View reviewed changes
Copy link
Contributor

@timkl7 timkl7 left a comment

Choose a reason for hiding this comment

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

issue with directory name

@github-actions
Copy link

github-actions bot commented Jul 8, 2025

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

hth9fe
hth9fe reviewed Aug 1, 2025
View reviewed changes
hth9fe
hth9fe reviewed Aug 1, 2025
View reviewed changes
@HartmannNico HartmannNico changed the title docs: added nonipc feature(#916) Cross-VM IPC (#916) Aug 4, 2025
@HartmannNico
Copy link
Contributor

HartmannNico commented Aug 4, 2025

In COM Architecture Meeting on 2025/08/04 we concluded this is a modification of the FR for COM #229. Hence this FR/PR does not constitute a new feature, but creates an updated version of the COM FR.
The description and requiremements from features/nonipc folder shall move into features/ipc.
We use this PR to follow up on this documentation migration appropriately.

hth9fe
hth9fe reviewed Aug 6, 2025
View reviewed changes
Copy link
Contributor

@hth9fe hth9fe left a comment

Choose a reason for hiding this comment

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

minor changes

@AlexanderLanin AlexanderLanin requested a review from Copilot August 6, 2025 14:05

This comment was marked as outdated.

Sign in to view

FScholPer and others added 4 commits August 7, 2025 08:21
@FScholPer @hth9fe
Update docs/features/communication/crossvm/requirements/index.rst
6046bd6 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/requirements/stakeholder/index.rst
3bb7e41 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/features/communication/crossvm/index.rst
9ab19f3 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/features/communication/crossvm/index.rst
a69af69 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@AlexanderLanin AlexanderLanin requested a review from Copilot August 7, 2025 07:14
Copilot AI reviewed Aug 7, 2025
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive documentation for Cross-VM IPC functionality, introducing requirements and specifications for inter-virtual machine communication capabilities.

  • Adds stakeholder requirement for cross-VM communication support
  • Introduces detailed feature requirements for cross-VM communication mechanisms
  • Creates feature documentation with specifications for one-way data sharing and streamed data communication

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.

File Description
docs/requirements/stakeholder/index.rst Adds stakeholder requirement for cross-VM communication
docs/features/communication/index.rst Updates table of contents to include crossvm documentation
docs/features/communication/crossvm/requirements/index.rst Defines detailed functional requirements for cross-VM communication features
docs/features/communication/crossvm/index.rst Provides feature specification, motivation, and implementation details

FScholPer and others added 3 commits August 7, 2025 11:24
@FScholPer
Update docs/requirements/stakeholder/index.rst
ab94351 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Update docs/features/communication/crossvm/index.rst
f61c426 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Update docs/features/communication/crossvm/index.rst
9af771a 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Copy link
Contributor Author

FScholPer commented Aug 7, 2025

Still need the approval. Also all github copilot issues where fixed now

AlexanderLanin
AlexanderLanin previously approved these changes Aug 7, 2025
View reviewed changes
Copy link
Member

@AlexanderLanin AlexanderLanin left a comment

Choose a reason for hiding this comment

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

approving as discussed

crimson11
crimson11 suggested changes Aug 7, 2025
View reviewed changes
FScholPer and others added 18 commits February 10, 2026 17:16
@FScholPer
fixed warning
86b1e20
@FScholPer @hth9fe
Update docs/features/communication/crossvm/index.rst
1919810 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/features/communication/crossvm/requirements/index.rst
a5b34d7 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/requirements/stakeholder/index.rst
a1ca099 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/features/communication/crossvm/index.rst
1674880 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer @hth9fe
Update docs/features/communication/crossvm/index.rst
f1fe0a1 
Co-authored-by: ThomasHahn <83017581+hth9fe@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Update docs/requirements/stakeholder/index.rst
3bd0724 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Update docs/features/communication/crossvm/index.rst
0e49ef4 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Update docs/features/communication/crossvm/index.rst
1a75d41 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
fix: removed cross-vm section
183676f
@FScholPer
fix: removed crossvm index
6c31620
@FScholPer
fix: typo
eba38ae
@FScholPer
fixed: id
9a51b62
@FScholPer
fixed: id #2
cb598fb
@FScholPer
docs: added until flags
179a203
@FScholPer
fix: valid block
fd9954e
@FScholPer @opajonk
Update docs/requirements/stakeholder/index.rst
f2750bc 
Co-authored-by: Oliver Pajonk <oliver.pajonk@elektrobit.com>
Signed-off-by: Frank Scholter Peres(MBTI) <145544737+FScholPer@users.noreply.github.com>
@FScholPer
Merge branch '916-fq-non-ipc-shared-memory-com' of https://github.com…
3fa2489 
…/eclipse-score/score into 916-fq-non-ipc-shared-memory-com
@MaximilianSoerenPollak
Copy link
Contributor

MaximilianSoerenPollak commented Feb 10, 2026

This PR seems to have a lot of commits & changes of the main branch in them.
Would it be possible to rebase this in order to only show the new changes, seems like something went wrong here.

@FScholPer FScholPer closed this Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HartmannNico HartmannNico HartmannNico left review comments

Copilot code review Copilot Copilot left review comments

@AlexanderLanin AlexanderLanin AlexanderLanin left review comments

@LittleHuba LittleHuba LittleHuba left review comments

@johannes-esr johannes-esr johannes-esr left review comments

@antonkri antonkri Awaiting requested review from antonkri antonkri is a code owner

@qor-lb qor-lb Awaiting requested review from qor-lb qor-lb is a code owner

@markert-r markert-r Awaiting requested review from markert-r

@MaximilianSoerenPollak MaximilianSoerenPollak Awaiting requested review from MaximilianSoerenPollak MaximilianSoerenPollak is a code owner

@dcalavrezo-qorix dcalavrezo-qorix Awaiting requested review from dcalavrezo-qorix dcalavrezo-qorix is a code owner

@thilo-schmitt thilo-schmitt Awaiting requested review from thilo-schmitt thilo-schmitt is a code owner

@skappel29 skappel29 Awaiting requested review from skappel29 skappel29 is a code owner

@grandyholger grandyholger Awaiting requested review from grandyholger grandyholger is a code owner

@pahmann pahmann Awaiting requested review from pahmann pahmann is a code owner

@masc2023 masc2023 Awaiting requested review from masc2023 masc2023 is a code owner

@aschemmel-tech aschemmel-tech Awaiting requested review from aschemmel-tech aschemmel-tech is a code owner

@PandaeDo PandaeDo Awaiting requested review from PandaeDo PandaeDo is a code owner

+4 more reviewers

@hth9fe hth9fe hth9fe left review comments

@crimson11 crimson11 crimson11 approved these changes

@opajonk opajonk opajonk approved these changes

@timkl7 timkl7 timkl7 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

feature_request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature Modification for Cross-VM IPC