qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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 :|



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