Skip to content

Rework/move psasim #10511

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.

Sign up for GitHub

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

Jump to bottom
Open
minosgalanakis wants to merge 3 commits into
base: development
Choose a base branch
from
Open

Rework/move psasim #10511

minosgalanakis wants to merge 3 commits into from

Conversation

@minosgalanakis
Copy link
Contributor

@minosgalanakis minosgalanakis commented Nov 17, 2025 •
edited
Loading

Description

Part of the work required by #10487. This is the PR that removes the code from the MbedTLS side, and changes the test makefiles to point at the new location.

PR checklist

Please remove the segment/s on either side of the | symbol as appropriate, and add any relevant link/s to the end of the line.
If the provided content is part of the present PR remove the # symbol.

  • changelog provided | not required because:
  • development PR provided This is it
  • TF-PSA-Crypto PR provided # | not required because:
  • framework PR provided Move psasim into framework mbedtls-framework#238
  • 3.6 PR not required because: Psasim is not present on 3.6
  • tests provided. This is a testing framework PR

@minosgalanakis minosgalanakis added size-s Estimated task size: small (~2d) needs-ci Needs to pass CI tests component-test Test framework and CI scripts priority-medium Medium priority - this can be reviewed as time permits labels Nov 17, 2025
@minosgalanakis minosgalanakis force-pushed the rework/move-psasim branch 4 times, most recently from 336fe81 to 69edec3 Compare December 4, 2025 11:00
@minosgalanakis minosgalanakis added needs-review Every commit must be reviewed by at least two team members, needs-backports Backports are missing or are pending review and approval. needs-preceding-pr Requires another PR to be merged first needs-reviewer This PR needs someone to pick it up for review and removed needs-ci Needs to pass CI tests needs-backports Backports are missing or are pending review and approval. labels Dec 4, 2025
@minosgalanakis
Test Makefiles: Updated location of psasim
272baa4 
Signed-off-by: Minos Galanakis <minos.galanakis@arm.com>
@minosgalanakis
Moved psa-client-server/psasim to framework
77e93fb 
Signed-off-by: Minos Galanakis <minos.galanakis@arm.com>
@minosgalanakis
Updated framework pointer
af6ad03 
Signed-off-by: Minos Galanakis <minos.galanakis@arm.com>
@minosgalanakis minosgalanakis mentioned this pull request Dec 24, 2025
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ronald-cron-arm ronald-cron-arm Awaiting requested review from ronald-cron-arm

@bensze01 bensze01 Awaiting requested review from bensze01

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

component-test Test framework and CI scripts needs-preceding-pr Requires another PR to be merged first needs-review Every commit must be reviewed by at least two team members, needs-reviewer This PR needs someone to pick it up for review priority-medium Medium priority - this can be reviewed as time permits size-s Estimated task size: small (~2d)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@minosgalanakis