Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
ivila merged PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179 -- 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-unsub

Re: [I] Systest broken [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
ivila closed issue #168: Systest broken URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/168 -- 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 unsubscrib

Re: [apache/incubator-teaclave-sgx-sdk] Fix panic with misaligned pointer dereference in `u_sgxfs_open_ocall` (PR #463)

2025-04-16 Thread volcano
volcano0dr left a comment (apache/incubator-teaclave-sgx-sdk#463) The address `0x7ffeb7fdbd25` doesn't look right. I think we should debug why it's not 8-byte aligned. -- Reply to this email directly or view it on GitHub: https://github.com/apache/incubator-teaclave-sgx-sdk/pull/463#issuecommen

[apache/incubator-teaclave-sgx-sdk] Fix panic with misaligned pointer dereference in `u_sgxfs_open_ocall` (PR #463)

2025-04-16 Thread private-yusuke
When a program calls `sgx_tprotected_fs::write`, the following error occurs during the execution: ``` thread 'main' panicked at rust-sgx-sdk/sgx_urts/src/ocall/sgxfile.rs:56:5: misaligned pointer dereference: address must be a multiple of 0x8 but is 0x7ffeb7fdbd25 ``` To avoid this panic happe

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046559685 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
ivila commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046557447 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC_Con

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046540649 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046540649 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046540649 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
ivila commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046513152 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC_Con

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046496148 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046496148 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046496148 ## optee-teec/src/context.rs: ## @@ -57,11 +57,7 @@ impl Context { /// ``` pub fn new() -> Result { // define an empty TEEC

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
ivila commented on PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#issuecomment-2808788112 > BTW could the ARM64 image also help on #160? I'm currently working on this (stuck on the QEMU aarch64 memory expansion issue). You can review my chang

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#issuecomment-2808766768 BTW could the ARM64 image also help on https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/160? -- This is an automated message from the Apache Git

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#issuecomment-2808758265 After the tiny comments resolved please feel free to add: `Reviewed-by: Yuan Zhuang ` -- This is an automated message from the Apache Git Service. To respond

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046348812 ## .github/workflows/ci.yml: ## @@ -211,6 +211,28 @@ jobs: make -j3 toolchains && make -j`nproc` RUST_ENABLE=y COMPILE_S

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
ivila commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046319991 ## .github/workflows/ci.yml: ## @@ -211,6 +211,28 @@ jobs: make -j3 toolchains && make -j`nproc` RUST_ENABLE=y COMPILE_S_USE

Re: [PR] Fix teec ctest zc [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on code in PR #179: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/pull/179#discussion_r2046311740 ## .github/workflows/ci.yml: ## @@ -211,6 +211,28 @@ jobs: make -j3 toolchains && make -j`nproc` RUST_ENABLE=y COMPILE_S

Re: [I] [TODO] Update for CI checking script in OP-TEE [incubator-teaclave-trustzone-sdk]

2025-04-16 Thread via GitHub
DemesneGH commented on issue #174: URL: https://github.com/apache/incubator-teaclave-trustzone-sdk/issues/174#issuecomment-2808686302 Please refer to: https://github.com/apache/incubator-teaclave-trustzone-sdk/blob/main/docs/expanding-ta-secure-memory-on-qemuv8.md -- This is an automated