From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Anthony Harivel <aharivel@redhat.com>
Cc: pbonzini@redhat.com, mtosatti@redhat.com, qemu-devel@nongnu.org,
vchundur@redhat.com, rjarry@redhat.com
Subject: Re: [PATCH v4 2/3] tools: build qemu-vmsr-helper
Date: Thu, 28 Mar 2024 10:46:24 +0000 [thread overview]
Message-ID: <ZgVKgARidIFQZF5M@redhat.com> (raw)
In-Reply-To: <D05B8SENZ5MO.52MU1G4SOBY0@fedora>
On Thu, Mar 28, 2024 at 11:27:50AM +0100, Anthony Harivel wrote:
> Hi Daniel,
>
> My apologies for all the missed feedback in v2.
> I'll be more organized for my next iteration.
>
> For this specific comment below, I would like to make sure I'm testing
> the right way.
>
> > > diff --git a/meson.build b/meson.build
> > > index b375248a7614..376da49b60ab 100644
> > > --- a/meson.build
> > > +++ b/meson.build
> > > @@ -4052,6 +4052,11 @@ if have_tools
> > > dependencies: [authz, crypto, io, qom, qemuutil,
> > > libcap_ng, mpathpersist],
> > > install: true)
> > > +
> > > + executable('qemu-vmsr-helper', files('tools/i386/qemu-vmsr-helper.c'),
> > > + dependencies: [authz, crypto, io, qom, qemuutil,
> > > + libcap_ng, mpathpersist],
> > > + install: true)
> > > endif
> >
> > Missed feedback from v2 saying this must /only/ be built
> > on x86 architectures. It fails to build on others due
> > to the ASM usage eg
> >
> > https://gitlab.com/berrange/qemu/-/jobs/6445384073
> >
>
> To recreate your build system, I need to, for example, compile with the
> following configuration for arm64 (aarch64):
>
> ../configure --enable-werror --disable-docs --enable-fdt=system
> --disable-user --cross-prefix=aarch64-linux-gnu-
> --target-list-exclude="arm-softmmu cris-softmmu i386-softmmu
> microblaze-softmmu mips-softmmu mipsel-softmmu mips64-softmmu
> ppc-softmmu riscv32-softmmu sh4-softmmu sparc-softmmu xtensa-softmmu"
>
> This is cross-compiling on x86 right?
> Because on my laptop I've got the following error:
>
> WARNING: unrecognized host CPU, proceeding with 'uname -m' output 'x86_64'
> python determined to be '/usr/bin/python3'
> python version: Python 3.12.2
> mkvenv: Creating non-isolated virtual environment at 'pyvenv'
> mkvenv: checking for meson>=0.63.0
>
> ERROR: Unrecognized host OS (uname -s reports 'Linux')
>
> It looks like it wants to build natively on aarch64.
> Maybe I need to create a VM with aarch64 Debian and compile natively?
> Might take a long time but I'm not sure this is the best way.
You can do this all with our Debian cross build containers locally
Either build the container fresh
podman build --tag qemuarm64 -f tests/docker/dockerfiles/debian-arm64-cross.docker
Or pull down the one from CI
podman pull registry.gitlab.com/qemu-project/qemu/qemu/debian-arm64-cross
Inside the container you just need to clone your QEMU repo and then run
./configure $QEMU_CONFIGURE_OPTS --target-list=x86_64-softmmu
$QEMU_CONFIGURE_OPTS expands to the args needed for a cross-compile,
as it set by the container itself.
Alternatively fork the project on gitlab, and then push your branch
to your fork, while requesting CI
git push -o ci.variable=QEMU_CI=1 <gitlab remote> <branch name>
then in the gitlab UI, you can press play on the individual jobs
you want to test. See docs/devel/testing.rst for info on that.
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 :|
next prev parent reply other threads:[~2024-03-28 10:46 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-18 15:12 [PATCH v4 0/3] Add support for the RAPL MSRs series Anthony Harivel
2024-03-18 15:12 ` [PATCH v4 1/3] qio: add support for SO_PEERCRED for socket channel Anthony Harivel
2024-03-20 17:18 ` Daniel P. Berrangé
2024-03-18 15:12 ` [PATCH v4 2/3] tools: build qemu-vmsr-helper Anthony Harivel
2024-03-21 11:33 ` Daniel P. Berrangé
2024-03-28 10:27 ` Anthony Harivel
2024-03-28 10:46 ` Daniel P. Berrangé [this message]
2024-03-18 15:12 ` [PATCH v4 3/3] Add support for RAPL MSRs in KVM/Qemu Anthony Harivel
2024-03-21 13:35 ` Daniel P. Berrangé
2024-04-05 8:33 ` Anthony Harivel
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=ZgVKgARidIFQZF5M@redhat.com \
--to=berrange@redhat.com \
--cc=aharivel@redhat.com \
--cc=mtosatti@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=rjarry@redhat.com \
--cc=vchundur@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).