Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CHANGELOG for 4.0.0 #6341

Merged
merged 1 commit into from
Oct 20, 2023
Merged

Conversation

jforissier
Copy link
Contributor

@jforissier jforissier commented Oct 4, 2023

Update CHANGELOG for 4.0.0 and collect Tested-by tags.

The 4.0.0-rc1 tag will be created on Friday October 13th (!) and the 4.0.0 release is expected on October 20th.

I will hold off merging the pull requests with "[v4]" in their subject [1] until the end of next week (just before -rc1), even the ones that have the proper review tags etc., so that people have more opportunities to comment.

[1] https://github.com/OP-TEE/optee_os/pulls?q=is%3Apr+is%3Aopen+in%3Atitle+%5Bv4%5D

@jneuhauser
Copy link
Contributor

Tested-by: Johann Neuhauser <[email protected]> (stm32mp1-157C_DHCOM_PDK2)

Tested stm32mp157c-dhcom-pdk2 using latest OP-TEE developer build:

...
NOTICE:  BL2: v2.9(release):v2.9
...
I/TC: OP-TEE version: 3.22.0-229-g97ea199a2ae8 (gcc version 11.3.1 20220712 (Arm GNU Toolchain 11.3.Rel1)) #1 Fri Oct  6 11:28:31 UTC 2023 arm
...
# xtest
...
+-----------------------------------------------------
28130 subtests of which 0 failed
105 test cases of which 0 failed
0 test cases were skipped
TEE test application done!

@ldts
Copy link
Contributor

ldts commented Oct 15, 2023

With 4.0.0-rc1

Tested-by: Jorge Ramirez-Ortiz <[email protected]> (versal)

+-----------------------------------------------------
31360 subtests of which 2 failed
134 test cases of which 1 failed
0 test cases were skipped
TEE test application done!

Failing test [still needs PLM fix]

regression_4005.7 AE case 6 algo 0x40000810 line 2591
regression_4000.c:606:

Reason:
PLM firmware does not accept unaligned payloads (AES-GCM)

Trace information:
E/TC:? 0 update_payload:479 Versal AES payload length not word aligned (len = 0)

@ichergui
Copy link

Tested-by: Ilies CHERGUI <[email protected]> (QEMUv8)

  • Tested with QEMUv8 by using the latest OP-TEE changes
....
NOTICE:  Booting Trusted Firmware
NOTICE:  BL1: v2.9(release):v2.9
NOTICE:  BL1: Built : 16:55:22, Oct 15 2023
WARNING: Firmware Image Package header check failed.
NOTICE:  BL1: Booting BL2
NOTICE:  BL2: v2.9(release):v2.9
NOTICE:  BL2: Built : 16:55:24, Oct 15 2023
WARNING: Firmware Image Package header check failed.
WARNING: Firmware Image Package header check failed.
WARNING: Firmware Image Package header check failed.
WARNING: Firmware Image Package header check failed.
NOTICE:  BL1: Booting BL31
NOTICE:  BL31: v2.9(release):v2.9
NOTICE:  BL31: Built : 16:55:26, Oct 15 2023
...
I/TC: OP-TEE version: 4.0.0-rc1 (gcc version 11.3.1 20220712 (Arm GNU Toolchain 11.3.Rel1)) #1 Sun Oct 15 15:54:57 UTC 2023 aarch64
...

# xtest
Run test suite with level=0
...
+-----------------------------------------------------
32119 subtests of which 0 failed
134 test cases of which 0 failed
0 test cases were skipped
TEE test application done!
# 

@clementfaure
Copy link
Contributor

Hi,
For 4.0.0-rc1

Tested-by: Clement Faure <[email protected]> (imx-mx6dlsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx6qsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx6sllevk)
Tested-by: Clement Faure <[email protected]> (imx-mx6sxsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx6ulevk)
Tested-by: Clement Faure <[email protected]> (imx-mx6ullevk)
Tested-by: Clement Faure <[email protected]> (imx-mx6ulzevk)
Tested-by: Clement Faure <[email protected]> (imx-mx7dsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx7ulpevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mmevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mnevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mqevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mpevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8qmmek)
Tested-by: Clement Faure <[email protected]> (imx-mx8qxpmek)
Tested-by: Clement Faure <[email protected]> (imx-mx8ulpevk)
Tested-by: Clement Faure <[email protected]> (imx-mx93evk)

@gyuri-szing
Copy link
Contributor

gyuri-szing commented Oct 17, 2023

Tested-by: György Szing <[email protected]> (FVP_Base_RevC-2xAEMvA)

For 4.0.0-rc1.

Testing vas focusing on SPMC functionality. xtest:ffa_spmc and TS tests were executed. For a more detailed test report see https://developer.trustedfirmware.org/w/trusted-services/op-tee-spmc/release-testing-4-0/

FVP version:

/src$ /fvp/Base_RevC_AEMvA_pkg/models/Linux64_GCC-9.3/FVP_Base_RevC-2xAEMvA --version

Fast Models [11.18.16 (Jun 16 2022)]
Copyright 2000-2022 ARM Limited.
All Rights Reserved.

SPMC test results:

#/mnt/host/out/linux-arm-ffa-user/load_module.sh
[    4.688854] arm_ffa_user: loading out-of-tree module taints kernel.

#echo ==========x-test=========

==========x-test=========
#
#xtest -t ffa_spmc
Run test suite with level=0

TEE test application started over default TEE instance
######################################################
#
# ffa_spmc
#
######################################################
 
* ffa_spmc_1001 Test FF-A communication
o ffa_spmc_1001.1 SP1 comms check
  ffa_spmc_1001.1 OK
o ffa_spmc_1001.2 Sp2 comms check
  ffa_spmc_1001.2 OK
o ffa_spmc_1001.3 SP to SP messaging check
  ffa_spmc_1001.3 OK
  ffa_spmc_1001 OK
 
* ffa_spmc_1002 Test FF-A memory: share memory from Normal World to SP
o ffa_spmc_1002.1 Test memory set-up
  ffa_spmc_1002.1 OK
o ffa_spmc_1002.2 Test retrieve memory second time
  ffa_spmc_1002.2 OK
o ffa_spmc_1002.3 Test accessing memory
  ffa_spmc_1002.3 OK
o ffa_spmc_1002.4 Test relinquish memory
  ffa_spmc_1002.4 OK
o ffa_spmc_1002.5 Test incorrect reclaim
  ffa_spmc_1002.5 OK
o ffa_spmc_1002.6 Test relinquish memory second time
  ffa_spmc_1002.6 OK
o ffa_spmc_1002.7 Test correct reclaim
  ffa_spmc_1002.7 OK
o ffa_spmc_1002.8 Test retrieve invalid memory region
  ffa_spmc_1002.8 OK
  ffa_spmc_1002 OK
 
* ffa_spmc_1003 Test FF-A memory: SP to SP
  ffa_spmc_1003 OK
 
* ffa_spmc_1004 Test FF-A memory: Access and flags
o ffa_spmc_1004.1 Test sharing with exc access
  ffa_spmc_1004.1 OK
o ffa_spmc_1004.2 Test sharing with incorrect access
  ffa_spmc_1004.2 OK
  ffa_spmc_1004 OK
+-----------------------------------------------------
Result of testsuite ffa_spmc:
ffa_spmc_1001 OK
ffa_spmc_1002 OK
ffa_spmc_1003 OK
ffa_spmc_1004 OK
+-----------------------------------------------------
39 subtests of which 0 failed
4 test cases of which 0 failed
0 test cases were skipped
TEE test application done!

@jenswi-linaro
Copy link
Contributor

Tested-by: Jens Wiklander <[email protected]> (Juno)

@jforissier
Copy link
Contributor Author

For 4.0.0-rc1:

Tested-by: Jerome Forissier <[email protected]> (vexpress-qemu_armv8a)

Tested with the following Makefile: https://github.com/jforissier/optee_release_tests/blob/2a24f95bcbabbaf99e6aaad763379d99ee7118e9/qemuv8/Makefile

Issues found:

@jforissier
Copy link
Contributor Author

For 4.0.0-rc1:

Tested-by: Jerome Forissier <[email protected]> (rockchip-rk3399) (Rockpi4B)

@jforissier
Copy link
Contributor Author

For 4.0.0-rc1:

Tested-by: Jerome Forissier <[email protected]> (vexpress-qemu_virt)

Tested with the following Makefile: https://github.com/jforissier/optee_release_tests/blob/810f8fa49b77d682555f0b11958720907371ab8f/qemu/Makefile

@ricardosalveti
Copy link
Contributor

For 4.0.0-rc1:

Tested-by: Ricardo Salveti <[email protected]> (ZynqMP)

+-----------------------------------------------------
32174 subtests of which 0 failed
133 test cases of which 0 failed
0 test cases were skipped
TEE test application done!

@ricardosalveti
Copy link
Contributor

For 4.0.0-rc1:

Tested-by: Ricardo Salveti <[email protected]> (k3-am62x)

+-----------------------------------------------------
32154 subtests of which 0 failed
133 test cases of which 0 failed
0 test cases were skipped
TEE test application done!

@etienne-lms
Copy link
Contributor

For 4.0.0-rc1 + optee_os @ 117fe69
Tested-by: Etienne Carriere <[email protected]> (stm32mp1-135F_DK)
Tested-by: Etienne Carriere <[email protected]> (stm32mp1-157C_DK2)
Tested-by: Etienne Carriere <[email protected]> (stm32mp1-157C_EV1)

@jenswi-linaro
Copy link
Contributor

Tested-by: Jens Wiklander <[email protected]> (imx-mx8mqevk)

@jenswi-linaro
Copy link
Contributor

Tested-by: Jens Wiklander <[email protected]> (FVP)
Tested-by: Jens Wiklander <[email protected]> (Hikey)

Copy link
Contributor

@jbech-linaro jbech-linaro left a comment

Choose a reason for hiding this comment

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

NOTICE:  BL1: v2.6(debug):v2.6
NOTICE:  BL1: Built : 14:13:13, Oct 19 2023
NOTICE:  BL2: v2.6(debug):v2.6
NOTICE:  BL2: Built : 14:13:13, Oct 19 2023
NOTICE:  BL31: v2.6(debug):v2.6
NOTICE:  BL31: Built : 14:13:13, Oct 19 2023
...
I/TC: OP-TEE version: 4.0.0-rc1-10-g117fe695c (gcc version 11.3.1 20220712 (Arm GNU Toolchain 11.3.Rel1)) #1 Thu Oct 19 12:12:56 UTC 2023 aarch64
...
U-Boot 2021.10 (Oct 19 2023 - 14:25:03 +0200)
...
[    0.000000] Linux version 5.17.14-v8 (jyx@pop-os) (aarch64-linux-gnu-gcc (Arm GNU Toolchain 11.3.Rel1) 11.3.1 20220712, GNU ld (Arm GNU Toolchain 11.3.Rel1) 2.38.20220708) #1 SMP PREEMPT Thu Oct 19 14:12:57 CEST 2023
# xtest -l 15
...
+-----------------------------------------------------
48076 subtests of which 0 failed
105 test cases of which 0 failed
0 test cases were skipped
TEE test application done!

Tested-by: Joakim Bech <[email protected]> (Rpi3B)

Update CHANGELOG for 4.0.0 and collect Tested-by tags.

Tested-by: Johann Neuhauser <[email protected]>  (stm32mp1-157C_DHCOM_PDK2)
Tested-by: Jorge Ramirez-Ortiz <[email protected]> (versal)
Tested-by: Ilies CHERGUI <[email protected]>  (QEMUv8)
Tested-by: Clement Faure <[email protected]> (imx-mx6dlsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx6qsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx6sllevk)
Tested-by: Clement Faure <[email protected]> (imx-mx6sxsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx6ulevk)
Tested-by: Clement Faure <[email protected]> (imx-mx6ullevk)
Tested-by: Clement Faure <[email protected]> (imx-mx6ulzevk)
Tested-by: Clement Faure <[email protected]> (imx-mx7dsabresd)
Tested-by: Clement Faure <[email protected]> (imx-mx7ulpevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mmevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mnevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mqevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8mpevk)
Tested-by: Clement Faure <[email protected]> (imx-mx8qmmek)
Tested-by: Clement Faure <[email protected]> (imx-mx8qxpmek)
Tested-by: Clement Faure <[email protected]> (imx-mx8ulpevk)
Tested-by: Clement Faure <[email protected]> (imx-mx93evk)
Tested-by: György Szing <[email protected]>  (FVP_Base_RevC-2xAEMvA)
Tested-by: Jens Wiklander <[email protected]> (Juno)
Tested-by: Jerome Forissier <[email protected]> (vexpress-qemu_armv8a)
Tested-by: Jerome Forissier <[email protected]> (rockchip-rk3399) (Rockpi4B)
Tested-by: Jerome Forissier <[email protected]> (vexpress-qemu_virt)
Tested-by: Ricardo Salveti <[email protected]> (ZynqMP)
Tested-by: Ricardo Salveti <[email protected]> (k3-am62x)
Tested-by: Etienne Carriere <[email protected]> (stm32mp1-135F_DK)
Tested-by: Etienne Carriere <[email protected]> (stm32mp1-157C_DK2)
Tested-by: Etienne Carriere <[email protected]> (stm32mp1-157C_EV1)
Tested-by: Jens Wiklander <[email protected]> (imx-mx8mqevk)
Tested-by: Jens Wiklander <[email protected]> (FVP)
Tested-by: Jens Wiklander <[email protected]> (Hikey)
Tested-by: Joakim Bech <[email protected]> (Rpi3B)
Signed-off-by: Jerome Forissier <[email protected]>
@jforissier jforissier merged commit 2a5b1d1 into OP-TEE:master Oct 20, 2023
14 of 15 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.

10 participants