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 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.