From: "Marc-André Lureau" <marcandre.lureau@redhat.com>
To: "Daniel P. Berrangé" <berrange@redhat.com>
Cc: qemu-devel@nongnu.org, "Jiaxun Yang" <jiaxun.yang@flygoat.com>,
"Ed Maste" <emaste@freebsd.org>, "Thomas Huth" <thuth@redhat.com>,
"Li-Wen Hsu" <lwhsu@freebsd.org>,
"Yonggang Luo" <luoyonggang@gmail.com>,
"Warner Losh" <imp@bsdimp.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Kohei Tokunaga" <ktokunaga.mail@gmail.com>,
"Kyle Evans" <kevans@freebsd.org>
Subject: Re: [PATCH 02/24] scripts/archive-source: speed up downloading subprojects
Date: Mon, 22 Sep 2025 10:59:11 +0400 [thread overview]
Message-ID: <CAMxuvawKG1h2mv3sHvm-e1n2M8N_DOpLxJROxCi4QkN-RHqkgQ@mail.gmail.com> (raw)
In-Reply-To: <aM1fjf1tuzuBXudQ@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 2611 bytes --]
Hi
On Fri, Sep 19, 2025 at 5:50 PM Daniel P. Berrangé <berrange@redhat.com>
wrote:
> On Fri, Sep 19, 2025 at 05:32:56PM +0400, marcandre.lureau@redhat.com
> wrote:
> > From: Marc-André Lureau <marcandre.lureau@redhat.com>
> >
> > Running meson on each subproject is quite slow.
>
> Why is this faster ? Does it make meson do parallel downloads
> if you list then all at once ?
>
It doesn't look like it does it in paralllel, yet it is x times faster...
time sh -c 'for s in keycodemapdb libvfio-user berkeley-softfloat-3
berkeley-testfloat-3 anyhow-1-rs arbitrary-int-1-rs bilge-0.2-rs
bilge-impl-0.2-rs either-1-rs foreign-0.3-rs itertools-0.11-rs libc-0.2-rs
proc-macro2-1-rs proc-macro-error-1-rs proc-macro-error-attr-1-rs
quote-1-rs syn-2-rs unicode-ident-1-rs; do
meson subprojects download $s >/dev/null
done'
sh -c 5.27s user 0.33s system 99% cpu 5.613 total
time sh -c 'meson subprojects download keycodemapdb libvfio-user
berkeley-softfloat-3 berkeley-testfloat-3 anyhow-1-rs arbitrary-int-1-rs
bilge-0.2-rs bilge-impl-0.2-rs either-1-rs foreign-0.3-rs itertools-0.11-rs
libc-0.2-rs proc-macro2-1-rs proc-macro-error-1-rs
proc-macro-error-attr-1-rs quote-1-rs syn-2-rs unicode-ident-1-rs'
sh -c 0.32s user 0.01s system 99% cpu 0.339 total
>
> >
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> > ---
> > scripts/archive-source.sh | 5 +++--
> > 1 file changed, 3 insertions(+), 2 deletions(-)
> >
> > diff --git a/scripts/archive-source.sh b/scripts/archive-source.sh
> > index 035828c532..85cd0a63a0 100755
> > --- a/scripts/archive-source.sh
> > +++ b/scripts/archive-source.sh
> > @@ -77,9 +77,10 @@ function subproject_dir() {
> > git archive --format tar "$(tree_ish)" > "$tar_file"
> > test $? -ne 0 && error "failed to archive qemu"
> >
> > +meson subprojects download $subprojects
> > +test $? -ne 0 && error "failed to download subprojects $subprojects"
> > +
> > for sp in $subprojects; do
> > - meson subprojects download $sp
> > - test $? -ne 0 && error "failed to download subproject $sp"
> > tar --append --file "$tar_file" --exclude=.git
> subprojects/"$(subproject_dir $sp)"
> > test $? -ne 0 && error "failed to append subproject $sp to
> $tar_file"
> > done
> > --
> > 2.51.0
> >
>
> With regards,
> Daniel
> --
> |: https://berrange.com -o-
> https://www.flickr.com/photos/dberrange :|
> |: https://libvirt.org -o-
> https://fstop138.berrange.com :|
> |: https://entangle-photo.org -o-
> https://www.instagram.com/dberrange :|
>
>
[-- Attachment #2: Type: text/html, Size: 4157 bytes --]
next prev parent reply other threads:[~2025-09-22 6:59 UTC|newest]
Thread overview: 59+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-19 13:32 [PATCH 00/24] CI/build-sys fixes to enable Rust more widely marcandre.lureau
2025-09-19 13:32 ` [PATCH 01/24] gitlab-ci: fix 'needs' property type must be array marcandre.lureau
2025-09-19 14:05 ` Daniel P. Berrangé
2025-09-19 13:32 ` [PATCH 02/24] scripts/archive-source: speed up downloading subprojects marcandre.lureau
2025-09-19 13:50 ` Daniel P. Berrangé
2025-09-22 6:59 ` Marc-André Lureau [this message]
2025-09-22 8:08 ` Daniel P. Berrangé
2025-09-22 8:30 ` Paolo Bonzini
2025-09-19 13:32 ` [PATCH 03/24] scripts/archive-source: silence subprojects downloads marcandre.lureau
2025-09-19 13:49 ` Daniel P. Berrangé
2025-09-19 13:32 ` [PATCH 04/24] configure: fix rust meson configuration marcandre.lureau
2025-09-19 14:06 ` Daniel P. Berrangé
2025-09-19 13:32 ` [PATCH 05/24] configure: set the bindgen cross target marcandre.lureau
2025-09-19 15:08 ` Paolo Bonzini
2025-09-19 13:33 ` [PATCH 06/24] RFC: configure: use the same machine as the host triple by default marcandre.lureau
2025-09-19 15:11 ` Paolo Bonzini
2025-09-19 22:27 ` Paolo Bonzini
2025-09-19 13:33 ` [PATCH 07/24] tests/docker/common: print errors to stderr marcandre.lureau
2025-09-19 14:07 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 08/24] tests/docker: use fully qualified image name for emsdk marcandre.lureau
2025-09-19 14:08 ` Daniel P. Berrangé
2025-09-21 3:19 ` Kohei Tokunaga
2025-09-19 13:33 ` [PATCH 09/24] tests/docker/common: print meson log on configure failure marcandre.lureau
2025-09-19 14:09 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 10/24] lcitool: update, switch to f41 marcandre.lureau
2025-09-19 14:10 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 11/24] lcitool/qemu: include libclang-rt for TSAN marcandre.lureau
2025-09-19 14:10 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 12/24] lcitool/alpine: workaround bindgen issue marcandre.lureau
2025-09-19 14:10 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 13/24] tests/lcitool: add missing rust-std dep marcandre.lureau
2025-09-19 14:11 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 14/24] tests: move mips to debian-legacy-test-cross marcandre.lureau
2025-09-19 14:21 ` Daniel P. Berrangé
2025-09-22 8:38 ` Marc-André Lureau
2025-09-22 9:06 ` Daniel P. Berrangé
2025-09-22 9:44 ` Alex Bennée
2025-09-22 15:10 ` Paolo Bonzini
2025-09-22 16:40 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 15/24] tests/lcitool: update to debian13 marcandre.lureau
2025-09-19 14:21 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 16/24] build-sys: cfi_debug and safe_stack are not compatible marcandre.lureau
2025-09-19 14:25 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 17/24] tests/docker: add ENABLE_RUST environment marcandre.lureau
2025-09-19 14:37 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 18/24] tests/lcitool: enable rust & refresh marcandre.lureau
2025-09-19 14:39 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 19/24] configure: set the meson executable suffix/ext marcandre.lureau
2025-09-19 14:41 ` Daniel P. Berrangé
2025-09-22 10:34 ` Marc-André Lureau
2025-09-22 10:40 ` Daniel P. Berrangé
2025-09-22 10:45 ` Marc-André Lureau
2025-09-19 13:33 ` [PATCH 20/24] tests/freebsd: enable Rust marcandre.lureau
2025-09-19 14:41 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 21/24] meson: rust-bindgen limit allowlist-file to srcdir/include marcandre.lureau
2025-09-19 13:33 ` [PATCH 22/24] WIP: tests/docker: add rust to debian-legacy-test-cross marcandre.lureau
2025-09-19 14:43 ` Daniel P. Berrangé
2025-09-19 13:33 ` [PATCH 23/24] WIP: gitlab-ci: enable rust for msys2-64bit marcandre.lureau
2025-09-19 13:33 ` [PATCH 24/24] WIP: cirrus/macos: enable Rust marcandre.lureau
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=CAMxuvawKG1h2mv3sHvm-e1n2M8N_DOpLxJROxCi4QkN-RHqkgQ@mail.gmail.com \
--to=marcandre.lureau@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=berrange@redhat.com \
--cc=emaste@freebsd.org \
--cc=imp@bsdimp.com \
--cc=jiaxun.yang@flygoat.com \
--cc=kevans@freebsd.org \
--cc=ktokunaga.mail@gmail.com \
--cc=luoyonggang@gmail.com \
--cc=lwhsu@freebsd.org \
--cc=pbonzini@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=thuth@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).