{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":22514248,"defaultBranch":"master","name":"optee_os","ownerLogin":"jforissier","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2014-08-01T14:48:31.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/8288195?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1726833343.0","currentOid":""},"activityList":{"items":[{"before":null,"after":"75fa14dfb8782d997968f3f8375dce12d6a0c59d","ref":"refs/heads/stack-prot-fix","pushedAt":"2024-09-20T11:55:43.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ta: apply __no_stack_protector to __ta_entry() or __ta_entry_c()\n\nApply the __no_stack_protector attribute to the first C function called\nfollowing the TA entry point (i.e., __ta_entry(), or for the special\ncase of ARM32, __ta_entry_c()). This is required because\n__stack_chk_guard is initialized in this very function, therefore stack\nprotection cannot be assumed to be functional at this point.\n\nFixes a TA crash on QEMUv7 with Clang 18.1.7 [1].\n\nLink: https://github.com/OP-TEE/optee_os/issues/7047 [1]\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"ta: apply __no_stack_protector to __ta_entry() or __ta_entry_c()"}},{"before":"8d0ad8a860671f718395b3fec0c442e86f5de824","after":null,"ref":"refs/heads/ci-juno-mbedtls","pushedAt":"2024-08-19T16:13:32.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"}},{"before":"4a783f24c578f245633ee7272934b53c0b8bd0be","after":null,"ref":"refs/heads/misc-fixes","pushedAt":"2024-08-19T15:36:42.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"}},{"before":"212035681088f31a56a16fc6b3078d85d9f62d50","after":"8d0ad8a860671f718395b3fec0c442e86f5de824","ref":"refs/heads/ci-juno-mbedtls","pushedAt":"2024-08-19T14:55:43.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: add build with Mbed TLS and crypto extensions\n\nBuild for QEMUv8 with Mbed TLS as the core crypto library and\ncryptographic extensions enabled. This would have caught the issue fixed\nin commit 021fee0affe5 (\"core: mbedtls: Fix build\").\n\nSigned-off-by: Jerome Forissier \nReviewed-by: Jens Wiklander ","shortMessageHtmlLink":"ci: add build with Mbed TLS and crypto extensions"}},{"before":"d6d40b9b23a9708000a992835a8e80007e52846f","after":"212035681088f31a56a16fc6b3078d85d9f62d50","ref":"refs/heads/ci-juno-mbedtls","pushedAt":"2024-08-19T13:45:14.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: add build with Mbed TLS and crypto extensions\n\nBuild for QEMUv8 with Mbed TLS as the core crypto library and\ncryptographic extensions enabled. This would have caught the issue fixed\nin commit 021fee0affe5 (\"core: mbedtls: Fix build\").\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"ci: add build with Mbed TLS and crypto extensions"}},{"before":null,"after":"d6d40b9b23a9708000a992835a8e80007e52846f","ref":"refs/heads/ci-juno-mbedtls","pushedAt":"2024-08-19T13:08:57.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: add build for Juno with Mbed TLS\n\nBuild for the Juno platform with Mbed TLS as the core crypto library.\nThis would have caught the issue fixed in commit 021fee0affe5 (\"core:\nmbedtls: Fix build\").\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"ci: add build for Juno with Mbed TLS"}},{"before":"47c40216d670ec74ea598093a5a672a24383fca2","after":"4a783f24c578f245633ee7272934b53c0b8bd0be","ref":"refs/heads/misc-fixes","pushedAt":"2024-08-16T07:23:19.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"kernel/link.mk: fix missing build number in version string on first build\n\nFix an issue with the build number in the version string. While at it,\nfactor out the duplicated code into mk/macros.mk.\n\nBefore:\n\n $ rm -rf out/\n $ make out/arm-plat-vexpress/core/version.o\n UPD out/arm-plat-vexpress/core/.buildcount\n GEN out/arm-plat-vexpress/core/version.o\ncat: out/arm-plat-vexpress/core/.buildcount: No such file or directory\n\nIn addition to the error message, note the missing build number after the\nhash sign:\n\n $ strings out/arm-plat-vexpress/core/version.o | grep UTC\n 4.3.0-48-g9c97e7d52 (gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)) # Wed Aug 14 16:17:07 UTC 2024 arm\n\nAfter:\n\n $ rm -rf out/\n $ make out/arm-plat-vexpress/core/version.o\n UPD out/arm-plat-vexpress/core/.buildcount\n GEN out/arm-plat-vexpress/core/version.o\n $ strings out/arm-plat-vexpress/core/version.o | grep UTC\n 4.3.0-48-g9c97e7d52-dev (gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)) #1 Wed Aug 14 16:17:24 UTC 2024 arm\n\nSigned-off-by: Jerome Forissier \nReviewed-by: Jens Wiklander ","shortMessageHtmlLink":"kernel/link.mk: fix missing build number in version string on first b…"}},{"before":"83719b3538b72bb776e5b94b69611c4cf35f7de4","after":"ec11c428864b8abaad7528e7aa78a1ba81c3da7a","ref":"refs/heads/test","pushedAt":"2024-08-16T07:20:37.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"[TESTING] Use build.git PR 777\n\nMake sure CI runs OK with build.git PR 777 \"qemuv8: fix to enable Rust\nby default on x86_64 hosts only\".\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"[TESTING] Use build.git PR 777"}},{"before":null,"after":"47c40216d670ec74ea598093a5a672a24383fca2","ref":"refs/heads/misc-fixes","pushedAt":"2024-08-14T16:37:46.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"kernel/link.mk: fix missing build number in version string on first build\n\nFix an issue with the build number in the version string. While at it,\nfactor out the duplicated code into mk/macros.mk.\n\nBefore:\n\n $ rm -rf out/\n $ make out/arm-plat-vexpress/core/version.o\n UPD out/arm-plat-vexpress/core/.buildcount\n GEN out/arm-plat-vexpress/core/version.o\ncat: out/arm-plat-vexpress/core/.buildcount: No such file or directory\n\nIn addition to the error message, note the missing build number after the\nhash sign:\n\n $ strings out/arm-plat-vexpress/core/version.o | grep UTC\n 4.3.0-48-g9c97e7d52 (gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)) # Wed Aug 14 16:17:07 UTC 2024 arm\n\nAfter:\n\n $ rm -rf out/\n $ make out/arm-plat-vexpress/core/version.o\n UPD out/arm-plat-vexpress/core/.buildcount\n GEN out/arm-plat-vexpress/core/version.o\n $ strings out/arm-plat-vexpress/core/version.o | grep UTC\n 4.3.0-48-g9c97e7d52-dev (gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)) #1 Wed Aug 14 16:17:24 UTC 2024 arm\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"kernel/link.mk: fix missing build number in version string on first b…"}},{"before":"47c40216d670ec74ea598093a5a672a24383fca2","after":null,"ref":"refs/heads/dtc-fixes","pushedAt":"2024-08-14T16:37:42.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"}},{"before":"9c97e7d52e78da4023600652028a967ec970e209","after":"47c40216d670ec74ea598093a5a672a24383fca2","ref":"refs/heads/dtc-fixes","pushedAt":"2024-08-14T16:37:21.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"kernel/link.mk: fix missing build number in version string on first build\n\nFix an issue with the build number in the version string. While at it,\nfactor out the duplicated code into mk/macros.mk.\n\nBefore:\n\n $ rm -rf out/\n $ make out/arm-plat-vexpress/core/version.o\n UPD out/arm-plat-vexpress/core/.buildcount\n GEN out/arm-plat-vexpress/core/version.o\ncat: out/arm-plat-vexpress/core/.buildcount: No such file or directory\n\nIn addition to the error message, note the missing build number after the\nhash sign:\n\n $ strings out/arm-plat-vexpress/core/version.o | grep UTC\n 4.3.0-48-g9c97e7d52 (gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)) # Wed Aug 14 16:17:07 UTC 2024 arm\n\nAfter:\n\n $ rm -rf out/\n $ make out/arm-plat-vexpress/core/version.o\n UPD out/arm-plat-vexpress/core/.buildcount\n GEN out/arm-plat-vexpress/core/version.o\n $ strings out/arm-plat-vexpress/core/version.o | grep UTC\n 4.3.0-48-g9c97e7d52-dev (gcc version 11.4.0 (Ubuntu 11.4.0-1ubuntu1~22.04)) #1 Wed Aug 14 16:17:24 UTC 2024 arm\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"kernel/link.mk: fix missing build number in version string on first b…"}},{"before":null,"after":"9c97e7d52e78da4023600652028a967ec970e209","ref":"refs/heads/dtc-fixes","pushedAt":"2024-08-14T16:02:21.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"mk/compile.mk: separate rule to produce preprocessed dts files\n\nWhen compiling a .dts into a .dtb, there is a pre-preprocessing phase\nwhere the C preprocessor is run on the .dts to produce a .pre.dts, then\nthe device tree compiler is called to generate the .dtb. The two phases\nare currently bundled into a single makefile recipe, which is not ideal.\n\nExtract the preprocessing commands into a separate recipe to address\nthis concern. This allows to do for example:\n\n make PLATFORM=stm32mp1-157A_DK1 \\\n out/arm-plat-stm32mp1/core/arch/arm/dts/stm32mp157a-dk1.pre.dts\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"mk/compile.mk: separate rule to produce preprocessed dts files"}},{"before":"435c45938c129d3887ac53117947f83c4a297c8e","after":"efb12db9725a1ee32e2f946c83a75935c5bfc392","ref":"refs/heads/ci-cache@v4","pushedAt":"2024-07-30T11:45:55.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: upgrade actions/cache@v3 to v4\n\nUpgrade the \"cache\" action to address the CI warning:\n\n\"\"\"\nThe following actions uses Node.js version which is deprecated and\nwill be forced to run on node20: actions/cache@v3. For more info: [1]\n\"\"\"\n\nLink: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/ [1]\nSigned-off-by: Jerome Forissier \nReviewed-by: Joakim Bech ","shortMessageHtmlLink":"ci: upgrade actions/cache@v3 to v4"}},{"before":null,"after":"435c45938c129d3887ac53117947f83c4a297c8e","ref":"refs/heads/ci-cache@v4","pushedAt":"2024-07-30T10:56:56.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: upgrade actions/cache@v3 to v4\n\nUpgrade the \"cache\" action to address the CI warning:\n\n\"\"\"\nThe following actions uses Node.js version which is deprecated and\nwill be forced to run on node20: actions/cache@v3. For more info: [1]\n\"\"\"\n\nLink: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/ [1]\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"ci: upgrade actions/cache@v3 to v4"}},{"before":null,"after":"9dca1658f78acd17510b2468bd30525d74f7212d","ref":"refs/heads/test-mbedtls-expmod-regression-fix-9281","pushedAt":"2024-07-23T12:59:35.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libtomcrypt: test MBed TLS PR#9281\n\nMake LibTomcrypt use the more a secure but slower modular exponentiation\nalgorithm introduced in MBed TLS v3.6.0 when dealing with secret data\n(private keys) and the faster but less secure legacy one when dealing\nwith public data.\n\nOnly RSA is covered here but other algorithms that use mp_exptmod()\nshould be modified, too.\n\nTest results: \"time xtest 4011\" on QEMUv7:\n\n1. optee_os master @ (mbedtls_mpi_exp_mod() v3.5.2): 14.5s\n2. (1) plus revert \"libmbedtls: bignum: restore mbedtls_mpi_exp_mod()\n from v3.5.2\" (i.e., mbedtls_mpi_exp_mod() v3.6.0): 8m 26.5s\n3. this commit (i.e., MBedTLS PR#9281 plus the required LibTomcrypt\n changes): 5m 58s\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"libtomcrypt: test MBed TLS PR#9281"}},{"before":"b2b4562ba6b7f4574e2d06d60c15970b77621265","after":null,"ref":"refs/heads/hisi-trng-license","pushedAt":"2024-07-02T10:35:45.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"}},{"before":"71b81b68b5edd98efdda35ec021ee316d0a26c36","after":"b2b4562ba6b7f4574e2d06d60c15970b77621265","ref":"refs/heads/hisi-trng-license","pushedAt":"2024-07-02T09:03:36.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"core: drivers: relicense hisi_trng.c as BSD-2-Clause\n\nThe file core/drivers/hisi_trng.c should not have been committed with a\nGPL-2.0 license. Relicense it as BSD-2-Clause.\n\nSigned-off-by: Jerome Forissier \nReported-by: Sahil Malhotra \nCloses: https://github.com/OP-TEE/optee_os/issues/6834\nAcked-by: loubaihui \nAcked-by: leisen ","shortMessageHtmlLink":"core: drivers: relicense hisi_trng.c as BSD-2-Clause"}},{"before":"d340fd9e0a468faed7f3a4ecd5593b45cb08654a","after":null,"ref":"refs/heads/clang-fixes","pushedAt":"2024-06-18T08:48:55.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"}},{"before":"c5dfc63ca73f9cf22e61f438e4f9d8a5a123ce33","after":"d340fd9e0a468faed7f3a4ecd5593b45cb08654a","ref":"refs/heads/clang-fixes","pushedAt":"2024-06-17T08:45:18.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libutils, zlib: fix Clang warnings\n\nClang 18.1.6 reports the following warnings:\n\n CC out/arm/ldelf-lib/libutils/isoc/bget_malloc.o\n In file included from lib/libutils/isoc/bget_malloc.c:127:\n lib/libutils/isoc/bget.c:607:7: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C23 [-Wdeprecated-non-prototype]\n 607 | void *bget(requested_align, hdr_size, requested_size, poolset)\n | ^\n\nAnd same with lib/zlib/{adler32.c,inffast.c,inflate.c,zutil.c}.\n\nIn addition, zutil.c causes:\n\n CC out/arm/core/lib/zlib/zutil.o\ncore/lib/zlib/zutil.c:28:33: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes]\n 28 | const char * ZEXPORT zlibVersion()\n | ^\n | void\n\nAdd -Wno-deprecated-non-prototype to libutils' bget_malloc.c to silence\nthe first series, and simply remove -Wstrict-prototypes (added by\ndefault by mk/compile.mk) when building zlib.\n\nSigned-off-by: Jerome Forissier \nAcked-by: Joakim Bech \nAcked-by: Jens Wiklander ","shortMessageHtmlLink":"libutils, zlib: fix Clang warnings"}},{"before":"a8f676916fa1aa59d70ebce6cbf8282ffc25a849","after":"c5dfc63ca73f9cf22e61f438e4f9d8a5a123ce33","ref":"refs/heads/clang-fixes","pushedAt":"2024-06-14T19:20:52.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libutils, zlib: fix Clang warnings\n\nClang 18.1.6 reports the following warnings:\n\n CC out/arm/ldelf-lib/libutils/isoc/bget_malloc.o\n In file included from lib/libutils/isoc/bget_malloc.c:127:\n lib/libutils/isoc/bget.c:607:7: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C23 [-Wdeprecated-non-prototype]\n 607 | void *bget(requested_align, hdr_size, requested_size, poolset)\n | ^\n\nAnd same with lib/zlib/{adler32.c,inffast.c,inflate.c,zutil.c}.\n\nIn addition, zutil.c causes:\n\n CC out/arm/core/lib/zlib/zutil.o\ncore/lib/zlib/zutil.c:28:33: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes]\n 28 | const char * ZEXPORT zlibVersion()\n | ^\n | void\n\nAdd -Wno-deprecated-non-prototype to libutils' bget_malloc.c to silence\nthe first series, and simply remove -Wstrict-prototypes (added by\ndefault by mk/compile.mk) when building zlib.\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"libutils, zlib: fix Clang warnings"}},{"before":"0c4c7a0408a344ad673a5bbc26abd1912fe0568e","after":"a8f676916fa1aa59d70ebce6cbf8282ffc25a849","ref":"refs/heads/clang-fixes","pushedAt":"2024-06-14T16:44:14.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libutils, zlib: fix Clang warnings\n\nClang 18.1.6 reports the following warnings:\n\n CC out/arm/ldelf-lib/libutils/isoc/bget_malloc.o\n In file included from lib/libutils/isoc/bget_malloc.c:127:\n lib/libutils/isoc/bget.c:607:7: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C23 [-Wdeprecated-non-prototype]\n 607 | void *bget(requested_align, hdr_size, requested_size, poolset)\n | ^\n\nAnd same with lib/zlib/{adler32.c,inffast.c,inflate.c,zutil.c}.\n\nIn addition, zutil.c causes:\n\n CC out/arm/core/lib/zlib/zutil.o\ncore/lib/zlib/zutil.c:28:33: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes]\n 28 | const char * ZEXPORT zlibVersion()\n | ^\n | void\n\nAdd -Wno-deprecated-non-prototype to libutils' bget_malloc.c to silence\nthe first series, and simply remove -Wstrict-prototypes (added by\ndefault by mk/compile.mk) when building zlib.\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"libutils, zlib: fix Clang warnings"}},{"before":null,"after":"0c4c7a0408a344ad673a5bbc26abd1912fe0568e","ref":"refs/heads/clang-fixes","pushedAt":"2024-06-14T16:04:21.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"arm64.h: fix compile error with Clang\n\nClang 18.1.6 fails to compile OP-TEE OS with the following error:\n\n CC out/arm/core/arch/arm/kernel/vfp.o\n In file included from core/arch/arm/kernel/vfp.c:6:\n In file included from core/arch/arm/include/arm.h:137:\n core/arch/arm/include/arm64.h:455:1: error: expected readable system register\n 455 | DEFINE_U32_REG_READWRITE_FUNCS(fpcr)\n | ^\n core/arch/arm/include/arm64.h:436:3: note: expanded from macro 'DEFINE_U32_REG_READWRITE_FUNCS'\n 436 | DEFINE_U32_REG_READ_FUNC(reg) \\\n | ^\n core/arch/arm/include/arm64.h:430:3: note: expanded from macro 'DEFINE_U32_REG_READ_FUNC'\n 430 | DEFINE_REG_READ_FUNC_(reg, uint32_t, reg)\n | ^\n core/arch/arm/include/arm64.h:417:15: note: expanded from macro 'DEFINE_REG_READ_FUNC_'\n 417 | asm volatile(\"mrs %0, \" #asmreg : \"=r\" (val64)); \\\n | ^\n :1:10: note: instantiated into assembly here\n 1 | mrs x8, fpcr\n | ^\n\n...and similar ones for fpcr write, as well as fpsr read and write.\n\nClang 12.0.0 does not have any problem with this code which makes me\nthink that it's a Clang/LLVM issue.\n\nWork around the problem by using the coded system register identifiers\nS3_3_c4_c4_0 and S3_3_c4_c4_1 instead of fpcr and fpsr, respectively.\nThe values 3-3-4-4-0 and 3-3-4-4-1 are taken from the Arm ARM sections\nC.5.2.8 and C.5.2.9.\n\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"arm64.h: fix compile error with Clang"}},{"before":"fe2c4f91786c50368304431dc7b54c80c8bb6cf2","after":null,"ref":"refs/heads/buildroot-build-root","pushedAt":"2024-06-13T13:52:47.000Z","pushType":"branch_deletion","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"}},{"before":"54130dc20173ada02deb534df2201658e8dedace","after":"fe2c4f91786c50368304431dc7b54c80c8bb6cf2","ref":"refs/heads/buildroot-build-root","pushedAt":"2024-06-13T11:29:00.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: set FORCE_UNSAFE_CONFIGURE=1\n\nThe newer version of Buildroot recently selected in the OP-TEE manifest\n[1] does not allow building as root by default (apparently this is not\na Buildroot policy but rather defined in some packages included in\nBuildroot). While this makes sense in general, we could not care less in\na CI container. Therefore set the appropriate environment variable to\nbypass this check. We could as well change the Docker image to build as\na non-root user but let's just pick the easiest way for now.\n\nLink: https://github.com/OP-TEE/manifest/pull/281 [1]\nSigned-off-by: Jerome Forissier \nAcked-by: Jens Wiklander ","shortMessageHtmlLink":"ci: set FORCE_UNSAFE_CONFIGURE=1"}},{"before":null,"after":"54130dc20173ada02deb534df2201658e8dedace","ref":"refs/heads/buildroot-build-root","pushedAt":"2024-06-13T09:54:11.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"ci: set FORCE_UNSAFE_CONFIGURE=1\n\nThe newer version of Buildroot recently selected in the OP-TEE manifest\n[1] does not allow building as root by default (apparently this is not\na Buildroot policy but rather defined in some packages included in\nBuildroot). While this makes sense in general, we could not care less in\na CI container. Therefore set the appropriate environment variable to\nbypass this check. We could as well change the Docker image to build as\na non-root user but let's just pick the easiest way for now.\n\nLink: https://github.com/OP-TEE/manifest/pull/281 [1]\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"ci: set FORCE_UNSAFE_CONFIGURE=1"}},{"before":"f147fc17b69efc9f22d122466d7b6b9678a6ad61","after":"d3a67bc7a8735c95ff6847fac9f4c43e03fffac0","ref":"refs/heads/mbedtls-3.6-regression","pushedAt":"2024-06-05T17:02:19.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libmbedtls: bignum: restore mbedtls_mpi_exp_mod() from v3.5.2\n\nThe implementation of mbedtls_mpi_exp_mod() in Mbed TLS v3.6.0\nintroduces a large performance regression in \"xtest 4011\" on QEMUv7\n(32-bit). One iteration of the test used to take 1.4 second on my\nmachine but the newer implementation now needs 23 seconds. To make\nmatters worse, xtest 4011 runs ten iterations so in reality it is\n14 seconds vs. almost 4 minutes for the whole test.\nRevert mbedtls_mpi_exp_mod() to the v3.5.2 implementation to gain the\nperformance back. The upstream commit that changed the algorithm is\n[2]. Note that some mpi_* static functions have been made non static\nand renamed mbedtls_mpi_* in the current Mbed TLS so the code from\nv3.5.2 is modified accordingly.\n\nLink: https://optee.readthedocs.io/en/latest/building/devices/qemu.html#qemu-v7 [1]\nLink: https://github.com/Mbed-TLS/mbedtls/commit/1ba40585f9446372f69f19693c20f593283930af [2]\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"libmbedtls: bignum: restore mbedtls_mpi_exp_mod() from v3.5.2"}},{"before":"8a52564318bb8026ce59739e29907eb8d66ede57","after":"f147fc17b69efc9f22d122466d7b6b9678a6ad61","ref":"refs/heads/mbedtls-3.6-regression","pushedAt":"2024-06-05T15:45:15.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libmbedtls: bignum: restore mbedtls_mpi_exp_mod() from v3.5.2\n\nThe implementation of mbedtls_mpi_exp_mod() in Mbed TLS v3.6.0\nintroduces a large performance regression in \"xtest 4011\" on QEMUv7\n(32-bit). One iteration of the test used to take 1.4 second on my\nmachine but the newer implementation now needs 23 seconds. To make\nmatters worse, xtest 4011 runs ten iterations so in reality it is\n14 seconds vs. almost 4 minutes for the whole test.\nRevert mbedtls_mpi_exp_mod() to the v3.5.2 implementation to gain the\nperformance back. The upstream commit that changed the algorithm is\n[2]. Note that some mpi_* static functions have been made non static\nand renamed mbedtls_mpi_* in the current Mbed TLS so the code from\nv3.5.2 is modified accordingly.\n\nLink: https://optee.readthedocs.io/en/latest/building/devices/qemu.html#qemu-v7 [1]\nLink: https://github.com/Mbed-TLS/mbedtls/commit/1ba40585f9446372f69f19693c20f593283930af [2]\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"libmbedtls: bignum: restore mbedtls_mpi_exp_mod() from v3.5.2"}},{"before":null,"after":"8a52564318bb8026ce59739e29907eb8d66ede57","ref":"refs/heads/mbedtls-3.6-regression","pushedAt":"2024-06-05T15:43:23.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"libmbedtls: bignum: restore mbedtls_mpi_exp_mod() from v3.5.2\n\nThe implementation of mbedtls_mpi_exp_mod() in Mbed TLS v3.6.0\nintroduces a large performance regression in \"xtest 4011\" on QEMUv7\n(32-bit). One iteration of the test used to take 1.4 second on my\nmachine but the newer implementation now needs 23 seconds. To make\nmatters worse, xtest 4011 runs ten iterations so in reality it is\n14 seconds vs. almost 4 minutes for the whole test.\nRevert mbedtls_mpi_exp_mod() to the v3.5.2 implementation to gain the\nperformance back.\nNote that some mpi_* static functions have been made non static and\nrenamed mbedtls_mpi_* in the current Mbed TLS so the code from v3.5.2\nis modified accordingly.\n\nLink: https://optee.readthedocs.io/en/latest/building/devices/qemu.html#qemu-v7 [1]\nSigned-off-by: Jerome Forissier ","shortMessageHtmlLink":"libmbedtls: bignum: restore mbedtls_mpi_exp_mod() from v3.5.2"}},{"before":"dabbd641129df06f999035852b5542490af769fd","after":"71b81b68b5edd98efdda35ec021ee316d0a26c36","ref":"refs/heads/hisi-trng-license","pushedAt":"2024-06-03T15:39:40.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"core: drivers: relicense hisi_trng.c as BSD-2-Clause\n\nThe file core/drivers/hisi_trng.c should not have been committed with a\nGPL-2.0 license. Relicense it as BSD-2-Clause.\n\nSigned-off-by: Jerome Forissier \nReported-by: Sahil Malhotra \nCloses: https://github.com/OP-TEE/optee_os/issues/6834","shortMessageHtmlLink":"core: drivers: relicense hisi_trng.c as BSD-2-Clause"}},{"before":null,"after":"dabbd641129df06f999035852b5542490af769fd","ref":"refs/heads/hisi-trng-license","pushedAt":"2024-06-03T15:39:01.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"jforissier","name":"Jérôme Forissier","path":"/jforissier","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/8288195?s=80&v=4"},"commit":{"message":"core: drivers: relicense hisi_trng.c as BSD-2-Clause\n\nThe file core/drivers/hisi_trng.c should not have been committed with a\nGPL-2.0 license. Relicense it as BSD-2-Clause.\n\nSigned-off-by: Jerome Forissier \nReported-by: Sahil Malhotra ","shortMessageHtmlLink":"core: drivers: relicense hisi_trng.c as BSD-2-Clause"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yMFQxMTo1NTo0My4wMDAwMDBazwAAAAS757iq","startCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wOS0yMFQxMTo1NTo0My4wMDAwMDBazwAAAAS757iq","endCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wNi0wM1QxNTozOTowMS4wMDAwMDBazwAAAARbHeod"}},"title":"Activity · jforissier/optee_os"}