test: add Management Mode service group coverage#101
Open
slingappa wants to merge 1 commit into
Open
Conversation
Add a dedicated Management Mode test scenario and register it in the test object list. The scenario covers notification unsupported behavior, attribute reporting for the MM shared-memory region, registered-GUID communication, callback-observed shared-memory request and response side effects, unknown GUID error handling, and short request rejection. Signed-off-by: Subrahmanya Lingappa <subrahmanya.lingappa@qti.qualcomm.com>
This was referenced Jun 15, 2026
| @@ -0,0 +1,316 @@ | |||
| // SPDX-License-Identifier: BSD-2-Clause | |||
| /* | |||
| * Copyright (c) 2024 Ventana Micro Systems Inc. | |||
Collaborator
There was a problem hiding this comment.
Year and Org are outdated
| }; | ||
|
|
||
| static enum rpmi_error test_mm_active_cb(struct rpmi_shmem *shmem, | ||
| rpmi_uint16_t req_datalen, |
Collaborator
There was a problem hiding this comment.
indentation looks out of sync here and several other places in the file below.
Collaborator
There was a problem hiding this comment.
With commit 9836b30 integrated, please cross check the test cases execution wrt impact on expected response data length and value
| .cleanup = test_scenario_default_cleanup, | ||
|
|
||
| .num_tests = 5, | ||
| .tests = { |
Collaborator
There was a problem hiding this comment.
The following may be put inside separate struct if that helps to reduce indentation levels.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR adds Management Mode service-group test coverage.
Stack
Depends on: PR-10
Changes
Validation
make LIBRPMI_TEST=y CROSS_COMPILE= && make check CROSS_COMPILE=