All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Trevor Gamblin" <tgamblin@baylibre.com>
To: "João Marcos Costa" <joaomarcos.costa@bootlin.com>,
	openembedded-core@lists.openembedded.org
Cc: <thomas.petazzoni@bootlin.com>, <tgamblin@baylibre.com>
Subject: Re: [PATCH v2 1/1] recipes-support/gnutls: disable slow tests for RISCV64
Date: Mon, 28 Sep 2026 13:04:03 -0400	[thread overview]
Message-ID: <DLR4232VEGLK.D4805FQMYD9Z@baylibre.com> (raw)
In-Reply-To: <20260928162614.1685239-2-joaomarcos.costa@bootlin.com>

On Mon Sep 28, 2026 at 12:26 PM EDT, João Marcos Costa wrote:
> Timeouts have been observed (very frequently in the past weeks)
> in the autobuilder, and a bug [1] was filed at Bugzilla.
>
> After some investigation [2] was performed by Trevor Gamblin, we now have a
> list of the most time consuming test cases:
>
> - mini-record-2 (187880ms)
> - record-retvals (145390ms)
> - rng-op-key (63820ms)
> - rng-op-random (63330ms)
> - rng-op-nonce (63260ms)
> - dtls-rehandshake-cert (61360ms)
> - dtls-rehandshake-cert-2 (61230ms)
> - dtls-rehandshake-anon (61030ms)
> - mini-loss-time (60130ms)
> - mini-dtls-hello-verify-48 (30050ms)
> - tls12-ffdhe (27790ms)
> - x509sign-verify-rsa (18460ms)
> - x509sign-verify-ecdsa (16050ms)
>
> Disable them to avoid so many timeouts in qemuriscv64-ptest runs.

Thanks for submitting this. I'm sorry if I wasn't clear before, but I think we
should only disable mini-record-2 and record-retvals to start, since they are
significantly longer than all of the others. If gnutls remains a problem
afterward then the others should be considered, but for now we want to try
retaining the others to maximize test coverage.

Trevor
>
> [1] https://bugzilla.yoctoproject.org/show_bug.cgi?id=16162
> [2] https://bugzilla.yoctoproject.org/show_bug.cgi?id=16162#c19
>
> Signed-off-by: João Marcos Costa <joaomarcos.costa@bootlin.com>
> ---
>  ...m-disable-slow-test-cases-in-RISCV64.patch | 109 ++++++++++++++++++
>  meta/recipes-support/gnutls/gnutls_3.8.13.bb  |   4 +
>  2 files changed, 113 insertions(+)
>  create mode 100644 meta/recipes-support/gnutls/gnutls/0001-tests-Makefile.am-disable-slow-test-cases-in-RISCV64.patch
>
> diff --git a/meta/recipes-support/gnutls/gnutls/0001-tests-Makefile.am-disable-slow-test-cases-in-RISCV64.patch b/meta/recipes-support/gnutls/gnutls/0001-tests-Makefile.am-disable-slow-test-cases-in-RISCV64.patch
> new file mode 100644
> index 0000000000..6dbd165658
> --- /dev/null
> +++ b/meta/recipes-support/gnutls/gnutls/0001-tests-Makefile.am-disable-slow-test-cases-in-RISCV64.patch
> @@ -0,0 +1,109 @@
> +From a27faed63722263f5ae8e2b7fac564de3a65fda1 Mon Sep 17 00:00:00 2001
> +From: =?UTF-8?q?Jo=C3=A3o=20Marcos=20Costa?= <joaomarcos.costa@bootlin.com>
> +Date: Mon, 28 Sep 2026 15:56:37 +0000
> +Subject: [PATCH] tests/Makefile.am: disable slow test cases in RISCV64
> +MIME-Version: 1.0
> +Content-Type: text/plain; charset=UTF-8
> +Content-Transfer-Encoding: 8bit
> +
> +Timeouts have been observed (very frequently in the past weeks)
> +in the autobuilder, and a bug [1] was filed at Bugzilla.
> +
> +After some investigation [2] was performed by Trevor Gamblin, we now have a
> +list of the most time consuming test cases:
> +
> +- mini-record-2 (187880ms)
> +- record-retvals (145390ms)
> +- rng-op-key (63820ms)
> +- rng-op-random (63330ms)
> +- rng-op-nonce (63260ms)
> +- dtls-rehandshake-cert (61360ms)
> +- dtls-rehandshake-cert-2 (61230ms)
> +- dtls-rehandshake-anon (61030ms)
> +- mini-loss-time (60130ms)
> +- mini-dtls-hello-verify-48 (30050ms)
> +- tls12-ffdhe (27790ms)
> +- x509sign-verify-rsa (18460ms)
> +- x509sign-verify-ecdsa (16050ms)
> +
> +Disable them to avoid so many timeouts in qemuriscv64-ptest runs.
> +
> +Upstream-Status: Inappropriate [oe-core specific]
> +
> +[1] https://bugzilla.yoctoproject.org/show_bug.cgi?id=16162
> +[2] https://bugzilla.yoctoproject.org/show_bug.cgi?id=16162#c19
> +
> +Signed-off-by: João Marcos Costa <joaomarcos.costa@bootlin.com>
> +---
> + tests/Makefile.am | 20 ++++++++++----------
> + 1 file changed, 10 insertions(+), 10 deletions(-)
> +
> +diff --git a/tests/Makefile.am b/tests/Makefile.am
> +index f84b2f1..b03e570 100644
> +--- a/tests/Makefile.am
> ++++ b/tests/Makefile.am
> +@@ -161,7 +161,7 @@ ctests += tls13-early-start
> + 
> + ctests += tls13/no-auto-send-ticket
> + 
> +-ctests += mini-record-2 simple gnutls_hmac_fast set_pkcs12_cred cert certuniqueid tls-neg-ext-key \
> ++ctests += simple gnutls_hmac_fast set_pkcs12_cred cert certuniqueid tls-neg-ext-key \
> + 	 mpi certificate_set_x509_crl dn parse_ca x509-dn x509-dn-decode record-sizes \
> + 	 hostname-check cve-2008-4989 pkcs12_s2k chainverify missingissuer missingissuer_aia record-sizes-range \
> + 	 crq_key_id x509sign-verify sign-verify cve-2009-1415 cve-2009-1416		\
> +@@ -172,13 +172,13 @@ ctests += mini-record-2 simple gnutls_hmac_fast set_pkcs12_cred cert certuniquei
> + 	 rng-fork mini-eagain-dtls resume-dtls empty_retrieve_function \
> + 	 tls13-rehandshake-cert gnutls_ext_raw_parse handshake-large-cert \
> + 	 x509cert x509cert-tl x509cert-ct infoaccess mini-dtls-hello-verify sign-verify-ed25519-rfc8080 \
> +-	 trustdb-tofu dtls-rehandshake-anon mini-alpn mini-dtls-large \
> ++	 trustdb-tofu mini-alpn mini-dtls-large \
> + 	 mini-termination mini-x509-cas mini-x509-2 pkcs12_simple \
> +-	 mini-emsgsize-dtls chainverify-unsorted mini-overhead tls12-ffdhe \
> ++	 mini-emsgsize-dtls chainverify-unsorted mini-overhead \
> + 	 mini-dtls-heartbeat mini-x509-callbacks key-openssl priorities priorities-groups	\
> + 	 gnutls_x509_privkey_import gnutls_x509_crt_list_import time x509-server-verify \
> + 	 sign-verify-ext4 tls-neg-ext4-key resume-lifetime \
> +-	 mini-dtls-srtp rsa-encrypt-decrypt mini-loss-time gnutls-strcodes \
> ++	 mini-dtls-srtp rsa-encrypt-decrypt gnutls-strcodes \
> + 	 mini-record mini-dtls-record handshake-timeout mini-record-range \
> + 	 cert-status rsa-psk global-init sec-params sign-verify-data \
> + 	 fips-test fips-override-test mini-global-load name-constraints x509-extensions \
> +@@ -192,16 +192,16 @@ ctests += mini-record-2 simple gnutls_hmac_fast set_pkcs12_cred cert certuniquei
> + 	 rsa-rsa-oaep rsa-rsa-pss \
> + 	 mini-dtls-fork mini-key-material x509cert-invalid \
> + 	 tls-ext-register tls-supplemental mini-dtls0-9 duplicate-extensions \
> +-	 record-retvals mini-server-name tls-etm tls-force-etm x509-cert-callback alerts \
> ++	 mini-server-name tls-etm tls-force-etm x509-cert-callback alerts \
> + 	 client-sign-md5-rep tls12-invalid-key-exchanges session-rdn-read \
> + 	 tls13-cert-key-exchange x509-cert-callback-ocsp gnutls_ocsp_resp_list_import2 \
> + 	 server-sign-md5-rep privkey-keygen mini-tls-nonblock no-signal pkcs7-gen dtls-etm \
> +-	 x509sign-verify-rsa x509sign-verify-ecdsa x509sign-verify-gost \
> ++	 x509sign-verify-gost \
> + 	 cipher-alignment oids atfork prf psk-file priority-init2 post-client-hello-change-prio \
> + 	 status-request status-request-ok rfc7633-missing sign-verify-ext \
> +-	 fallback-scsv pkcs8-key-decode urls dtls-rehandshake-cert rfc7633-ok \
> ++	 fallback-scsv pkcs8-key-decode urls rfc7633-ok \
> + 	 key-usage-rsa key-usage-ecdhe-rsa mini-session-verify-function auto-verify \
> +-	 record-timeouts mini-dtls-hello-verify-48 set-default-prio \
> ++	 record-timeouts set-default-prio \
> + 	 tls12-anon-upgrade tlsext-decoding rsa-psk-cb gnutls-ids \
> + 	 rehandshake-switch-cert rehandshake-switch-cert-allow rehandshake-switch-cert-client \
> + 	 rehandshake-switch-cert-client-allow handshake-versions dtls-handshake-versions \
> +@@ -214,7 +214,7 @@ ctests += mini-record-2 simple gnutls_hmac_fast set_pkcs12_cred cert certuniquei
> + 	 set_x509_key_file_der set_x509_pkcs12_key crt_apis tls12-cert-key-exchange \
> + 	 tls11-cert-key-exchange tls10-cert-key-exchange ssl30-cert-key-exchange \
> + 	 dtls12-cert-key-exchange dtls10-cert-key-exchange x509-cert-callback-legacy \
> +-	 keylog-env ssl2-hello tlsfeature-ext dtls-rehandshake-cert-2 dtls-session-ticket-lost \
> ++	 keylog-env ssl2-hello tlsfeature-ext dtls-session-ticket-lost \
> + 	 tlsfeature-crt dtls-rehandshake-cert-3 resume-with-false-start \
> + 	 set_x509_key_file_ocsp client-fastopen rng-sigint srp \
> + 	 safe-renegotiation/srn0 safe-renegotiation/srn1 safe-renegotiation/srn2 \
> +@@ -230,7 +230,7 @@ ctests += mini-record-2 simple gnutls_hmac_fast set_pkcs12_cred cert certuniquei
> + 	 rng-no-onload dtls1-2-mtu-check crl_apis cert_verify_inv_utf8 no-extensions \
> + 	 no-status-request hostname-check-utf8 pkcs8-key-decode-encrypted priority-mix pkcs7 \
> + 	 send-data-before-handshake recv-data-before-handshake crt_inv_write \
> +-	 x509sign-verify-error rng-op-nonce rng-op-random rng-op-key x509-dn-decode-compat \
> ++	 x509sign-verify-error x509-dn-decode-compat \
> + 	 ip-check mini-x509-ipaddr trust-store base64-raw random-art dhex509self \
> + 	 dss-sig-val sign-pk-api tls-session-ext-override record-pad \
> + 	 tls13-server-kx-neg gnutls_ext_raw_parse_dtls key-export-pkcs8 \
> diff --git a/meta/recipes-support/gnutls/gnutls_3.8.13.bb b/meta/recipes-support/gnutls/gnutls_3.8.13.bb
> index 9c8c994bdc..ff1c3e661e 100644
> --- a/meta/recipes-support/gnutls/gnutls_3.8.13.bb
> +++ b/meta/recipes-support/gnutls/gnutls_3.8.13.bb
> @@ -27,6 +27,10 @@ SRC_URI = "https://www.gnupg.org/ftp/gcrypt/gnutls/v${SHRT_VER}/gnutls-${PV}.tar
>             file://0001-tests-mini-dtls-framents-link-to-gnulib.patch \
>             "
>  
> +SRC_URI:append:riscv64 = "\
> +    file://0001-tests-Makefile.am-disable-slow-test-cases-in-RISCV64.patch \
> +"
> +
>  SRC_URI[sha256sum] = "ffed8ec1bf09c2426d4f14aae377de4753b53e537d685e604e99a8b16ca9c97e"
>  
>  inherit autotools texinfo pkgconfig gettext lib_package gtk-doc ptest




  reply	other threads:[~2026-09-28 18:56 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-28 16:26 [PATCH v2 0/1] recipes-support/gnutls: disable slow tests for RISCV64 João Marcos Costa
2026-09-28 16:26 ` [PATCH v2 1/1] " João Marcos Costa
2026-09-28 17:04   ` Trevor Gamblin [this message]
2026-09-28 19:09     ` [OE-core] " Joao Marcos Costa
2026-09-28 20:10       ` Trevor Gamblin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DLR4232VEGLK.D4805FQMYD9Z@baylibre.com \
    --to=tgamblin@baylibre.com \
    --cc=joaomarcos.costa@bootlin.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.