You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@teaclave.apache.org by "ivila (via GitHub)" <gi...@apache.org> on 2024/10/24 08:29:27 UTC

[I] Make Panic Handler Optional [incubator-teaclave-trustzone-sdk]

ivila opened a new issue, #145:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/145

   In no-std branch, teaclave SDK define a panic handler, which stop ta developer from setting an custom one (for example, we would like to log the panic information before calling TEE_Panic).
   https://github.com/apache/incubator-teaclave-trustzone-sdk/blob/3177d4fa06f77afb351d0a19b8aa51b4a50fa1c8/optee-utee/src/lib.rs#L36
   
   I think we should add a feature flag to let developer choose if they really need to disable the default one, just like this issue: [rust-osdev/uefi-rs: Make panic handler optional](https://github.com/rust-osdev/uefi-rs/issues/177)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org


Re: [I] Make Panic Handler Optional [incubator-teaclave-trustzone-sdk]

Posted by "ivila (via GitHub)" <gi...@apache.org>.
ivila commented on issue #145:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/145#issuecomment-2440280016

   Done


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org


Re: [I] Make Panic Handler Optional [incubator-teaclave-trustzone-sdk]

Posted by "ivila (via GitHub)" <gi...@apache.org>.
ivila closed issue #145: Make Panic Handler Optional
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/145


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org


Re: [I] Make Panic Handler Optional [incubator-teaclave-trustzone-sdk]

Posted by "b49020 (via GitHub)" <gi...@apache.org>.
b49020 commented on issue #145:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/145#issuecomment-2434776682

   Looks reasonable to me, feel free to create corresponding PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org


Re: [I] Make Panic Handler Optional [incubator-teaclave-trustzone-sdk]

Posted by "ivila (via GitHub)" <gi...@apache.org>.
ivila commented on issue #145:
URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/145#issuecomment-2436709022

   @b49020 Thanks for your patience, a PR has already created.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@teaclave.apache.org
For additional commands, e-mail: dev-help@teaclave.apache.org