Several suggestions were put forward by reviewers(Joao, Raghu, Olivier) during the review of hafnium patches. This ticket captures these suggestions to be fixed/implemented in the next iteration.
# List ItemManage the partition runtime model upon completion of initialization by secondary execution contexts: https://review.trustedfirmware.org/c/hafnium/hafnium/+/15655/comment/9cb4e08b_0c9c5089/
2. Declutter the member fields of vcpu struct that are specific to arm aarch64/secure state: https://review.trustedfirmware.org/c/hafnium/hafnium/+/15656/comment/3d2b3990_455839cd/
3. Pass locked structures to newly created helper functions such as plat_ffa_check_runtime_state_transition():
https://review.trustedfirmware.org/c/hafnium/hafnium/+/15657/comment/6708ec9b_e1b7e5ae/
https://review.trustedfirmware.org/c/hafnium/hafnium/+/15657/comment/f6917756_6fc31ff9/
https://review.trustedfirmware.org/c/hafnium/hafnium/+/15660/comment/6539f1e4_29cd1912/
https://review.trustedfirmware.org/c/hafnium/hafnium/+/15662/comment/f6d82d82_b41779ef/
4. Revisit the need for implicit_completion_signal in the design of virtual secure interrupt signaling/completion:
https://review.trustedfirmware.org/c/hafnium/hafnium/+/15671/comment/84eb76f8_5ff0fee3/