From: Vasiliy Ulyanov <vulyanov(a)suse.de>

Define the target machine architecture in lowercase.

The name of the executable qemu-system-* is defined from the build flags
and should be in lowercase:

    ( "qemu-system-" ARCH ),

I.e. qemu-system-x86_64 instead of qemu-system-X86_64. Otherwise, the
exec call will fail.

Signed-off-by: Vasiliy Ulyanov <vulyanov(a)suse.de>
Signed-off-by: Dario Faggioli <dfaggioli(a)suse.com>
---
 Makefile |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Makefile b/Makefile
index c47a5f6..4ccca67 100644
--- a/Makefile
+++ b/Makefile
@@ -15,10 +15,10 @@ RLIMIT_STACK_VAL := 1024
 endif
 
 # Get 'uname -m'-like architecture description for target
-TARGET_ARCH := $(shell $(CC) -dumpmachine | cut -f1 -d- | tr [a-z] [A-Z])
-TARGET_ARCH := $(shell echo $(TARGET_ARCH) | sed 's/POWERPC/PPC/')
+TARGET_ARCH := $(shell $(CC) -dumpmachine | cut -f1 -d- | tr [A-Z] [a-z])
+TARGET_ARCH := $(shell echo $(TARGET_ARCH) | sed 's/powerpc/ppc/')
 
-AUDIT_ARCH := $(shell echo $(TARGET_ARCH) | sed 's/^ARM.*/ARM/')
+AUDIT_ARCH := $(shell echo $(TARGET_ARCH) | tr [a-z] [A-Z] | sed
's/^ARM.*/ARM/')
 AUDIT_ARCH := $(shell echo $(AUDIT_ARCH) | sed 's/I[456]86/I386/')
 AUDIT_ARCH := $(shell echo $(AUDIT_ARCH) | sed 's/PPC64/PPC/')
 AUDIT_ARCH := $(shell echo $(AUDIT_ARCH) | sed 's/PPCLE/PPC64LE/')
@@ -90,7 +90,7 @@ docdir		?= $(datarootdir)/doc/passt
 mandir		?= $(datarootdir)/man
 man1dir		?= $(mandir)/man1
 
-ifeq ($(TARGET_ARCH),X86_64)
+ifeq ($(TARGET_ARCH),x86_64)
 BIN := passt passt.avx2 pasta pasta.avx2 qrap
 else
 BIN := passt pasta qrap

On Fri, 23 Sep 2022 11:56:23 +0200
Dario Faggioli <dfaggioli(a)suse.com> wrote:

...
  Hello everyone!
 
 So, while playing with out passt package for openSUSE, my colleague
 Vasiliy found out an issue in how passt assumes the qemu binary to be
 called.
 
 In fact, in openSUSE, while we have a package that allows people to
 launch qemu by invoking `qemu-kvm`, we don't install it by default, and
 we therefore only have the standard upstream names in the filesystem,
 which have the ARCH part in lowercase. 
Ah, right. I never noticed because 'kvm' or 'qemu-kvm' were always
there in the environments I use, and the openSUSE test script in the CI:

  https://passt.top/passt/tree/test/distro/opensuse

explicitly passes the path to the qemu binary to qrap.

By the way, for context, as the name might suggest: I'm quite eager to
get rid of qrap, and it looks like we're close. Once native AF_UNIX
support lands in qemu with (further revisions of) Laurent's patchset:

  https://patchwork.kernel.org/project/qemu-devel/list/?series=676445

we can simply tell qemu to connect to passt, without starting it with a
wrapper.

...
  This patch, which we're currently carrying as a
downstream one in our
 package, should fix the problem.
 
 Thanks and Regards,
 Dario
 ---
 Vasiliy Ulyanov (1):
       Fix the name of the qemu-system-* executable 
For that patch,

Reviewed-by: Stefano Brivio <sbrivio(a)redhat.com>

And while at it: you might want to subscribe to this list:

  https://lists.passt.top/

if you're interested. Right now I'm approving your postings manually as
you're not subscribed, so they are always a bit delayed.

-- 
Stefano