yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone

Blame SOURCES/kvm-pc-bios-s390-ccw-Fix-inline-assembly-for-older-versi.patch

a83cc2
From c783eab8a3770703a39bbbd7edd23af7b9cb8f14 Mon Sep 17 00:00:00 2001
a83cc2
From: Thomas Huth <thuth@redhat.com>
a83cc2
Date: Wed, 12 May 2021 19:15:48 +0200
a83cc2
Subject: [PATCH 35/39] pc-bios/s390-ccw: Fix inline assembly for older
a83cc2
 versions of Clang
a83cc2
MIME-Version: 1.0
a83cc2
Content-Type: text/plain; charset=UTF-8
a83cc2
Content-Transfer-Encoding: 8bit
a83cc2
a83cc2
RH-Author: Jon Maloy <jmaloy@redhat.com>
a83cc2
RH-MergeRequest: 24: v7:  Add support for building qemu-kvm with clang and safe-stack
a83cc2
RH-Commit: [8/11] 9a69c7705b8128b4098f818c6b672d484e459c83 (jmaloy/qemu-kvm-centos-jon)
a83cc2
RH-Bugzilla: 1939509 1940132
a83cc2
RH-Acked-by: Danilo Cesar Lemes de Paula <ddepaula@redhat.com>
a83cc2
RH-Acked-by: Thomas Huth <thuth@redhat.com>
a83cc2
RH-Acked-by: Cornelia Huck <cohuck@redhat.com>
a83cc2
a83cc2
Clang versions before v11.0 insist on having the %rX or %cX register
a83cc2
names instead of just a number. Since our Travis-CI is currently
a83cc2
still using Clang v6.0, we have to fix this to avoid failing jobs.
a83cc2
a83cc2
Message-Id: <20210512171550.476130-2-thuth@redhat.com>
a83cc2
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
a83cc2
Signed-off-by: Thomas Huth <thuth@redhat.com>
a83cc2
(cherry picked from commit 052b66e7211af64964e005126eaa3c944b296b0e)
a83cc2
Signed-off-by: Jon Maloy <jmaloy@redhat.com>
a83cc2
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
a83cc2
---
a83cc2
 pc-bios/s390-ccw/helper.h   | 2 +-
a83cc2
 pc-bios/s390-ccw/jump2ipl.c | 4 ++--
a83cc2
 pc-bios/s390-ccw/menu.c     | 8 ++++----
a83cc2
 pc-bios/s390-ccw/virtio.c   | 2 +-
a83cc2
 4 files changed, 8 insertions(+), 8 deletions(-)
a83cc2
a83cc2
diff --git a/pc-bios/s390-ccw/helper.h b/pc-bios/s390-ccw/helper.h
a83cc2
index dfcfea0ff0..3d0731c4c6 100644
a83cc2
--- a/pc-bios/s390-ccw/helper.h
a83cc2
+++ b/pc-bios/s390-ccw/helper.h
a83cc2
@@ -31,7 +31,7 @@ static inline void *u32toptr(uint32_t n)
a83cc2
 
a83cc2
 static inline void yield(void)
a83cc2
 {
a83cc2
-    asm volatile ("diag 0,0,0x44"
a83cc2
+    asm volatile ("diag %%r0,%%r0,0x44"
a83cc2
                   : :
a83cc2
                   : "memory", "cc");
a83cc2
 }
a83cc2
diff --git a/pc-bios/s390-ccw/jump2ipl.c b/pc-bios/s390-ccw/jump2ipl.c
a83cc2
index 73e4367e09..78f5f46533 100644
a83cc2
--- a/pc-bios/s390-ccw/jump2ipl.c
a83cc2
+++ b/pc-bios/s390-ccw/jump2ipl.c
a83cc2
@@ -64,8 +64,8 @@ void jump_to_IPL_code(uint64_t address)
a83cc2
      * We use the load normal reset to keep r15 unchanged. jump_to_IPL_2
a83cc2
      * can then use r15 as its stack pointer.
a83cc2
      */
a83cc2
-    asm volatile("lghi 1,1\n\t"
a83cc2
-                 "diag 1,1,0x308\n\t"
a83cc2
+    asm volatile("lghi %%r1,1\n\t"
a83cc2
+                 "diag %%r1,%%r1,0x308\n\t"
a83cc2
                  : : : "1", "memory");
a83cc2
     panic("\n! IPL returns !\n");
a83cc2
 }
a83cc2
diff --git a/pc-bios/s390-ccw/menu.c b/pc-bios/s390-ccw/menu.c
a83cc2
index de8260a5d6..d601952d3e 100644
a83cc2
--- a/pc-bios/s390-ccw/menu.c
a83cc2
+++ b/pc-bios/s390-ccw/menu.c
a83cc2
@@ -36,9 +36,9 @@ static inline void enable_clock_int(void)
a83cc2
     uint64_t tmp = 0;
a83cc2
 
a83cc2
     asm volatile(
a83cc2
-        "stctg      0,0,%0\n"
a83cc2
+        "stctg      %%c0,%%c0,%0\n"
a83cc2
         "oi         6+%0, 0x8\n"
a83cc2
-        "lctlg      0,0,%0"
a83cc2
+        "lctlg      %%c0,%%c0,%0"
a83cc2
         : : "Q" (tmp) : "memory"
a83cc2
     );
a83cc2
 }
a83cc2
@@ -48,9 +48,9 @@ static inline void disable_clock_int(void)
a83cc2
     uint64_t tmp = 0;
a83cc2
 
a83cc2
     asm volatile(
a83cc2
-        "stctg      0,0,%0\n"
a83cc2
+        "stctg      %%c0,%%c0,%0\n"
a83cc2
         "ni         6+%0, 0xf7\n"
a83cc2
-        "lctlg      0,0,%0"
a83cc2
+        "lctlg      %%c0,%%c0,%0"
a83cc2
         : : "Q" (tmp) : "memory"
a83cc2
     );
a83cc2
 }
a83cc2
diff --git a/pc-bios/s390-ccw/virtio.c b/pc-bios/s390-ccw/virtio.c
a83cc2
index ab49840db8..5d2c6e3381 100644
a83cc2
--- a/pc-bios/s390-ccw/virtio.c
a83cc2
+++ b/pc-bios/s390-ccw/virtio.c
a83cc2
@@ -54,7 +54,7 @@ static long kvm_hypercall(unsigned long nr, unsigned long param1,
a83cc2
     register ulong r_param3 asm("4") = param3;
a83cc2
     register long retval asm("2");
a83cc2
 
a83cc2
-    asm volatile ("diag 2,4,0x500"
a83cc2
+    asm volatile ("diag %%r2,%%r4,0x500"
a83cc2
                   : "=d" (retval)
a83cc2
                   : "d" (r_nr), "0" (r_param1), "r"(r_param2), "d"(r_param3)
a83cc2
                   : "memory", "cc");
a83cc2
-- 
a83cc2
2.27.0
a83cc2