Skip to content

Conversation

@jmi-reds
Copy link
Contributor

@jmi-reds jmi-reds commented Jan 26, 2026

Fix bugs in AVZ introduced during so3 kernel updates done to support MUSL libc.

Also stored a capsule ID in AVZ.

Only tested on virt64 target, #198 has to be fixed for rpi4_64

clemdiep and others added 5 commits January 22, 2026 14:58
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds jmi-reds linked an issue Jan 26, 2026 that may be closed by this pull request
Copy link
Contributor

@clemdiep clemdiep left a comment

Choose a reason for hiding this comment

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

After some digging up on the get_ME_state double definition, it seems that it's come from so3/include/log.h which include soo/soo.h and soo/dev/vlogs.h if CONFIG_SOO is defined. As vlogs is only used for the logs when CONFIG_VLOGS_FRONTEND is defined, the ifdef can be changed to use it, fixing issue with multiple definition of get_ME_state.

In any case, I still think it's a good idea to rename the get_ME_state in avz to differentiate from the one in SOO. As well for the set_ME_state which take the slot ID too in avz. Maybe, instead of avz_get_ME_state we could name it get_ME_slot_state ?

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds
Copy link
Contributor Author

Based on @clemdiep review and discussion with @daniel-rossier , I've reverted the modification of get_ME_state() function and updated the logs.h file

@jmi-reds jmi-reds merged commit f7a3905 into main Jan 30, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ avz ] Fix issues due to kernel modifications

4 participants