All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Färber" <afaerber@suse.de>
To: Paolo Bonzini <pbonzini@redhat.com>, qemu-devel@nongnu.org
Cc: Claudio Fontana <claudio.fontana@huawei.com>,
	qemu-stable <qemu-stable@nongnu.org>,
	Aurelien Jarno <aurelien@aurel32.net>,
	Richard Henderson <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH] tcg/aarch64: Fix tcg_out_qemu_{ld, st} for linux-user
Date: Tue, 1 Sep 2015 18:31:54 +0200	[thread overview]
Message-ID: <55E5D2FA.30908@suse.de> (raw)
In-Reply-To: <55E15650.7020802@redhat.com>

[-- Attachment #1: Type: text/plain, Size: 5799 bytes --]

Am 29.08.2015 um 08:50 schrieb Paolo Bonzini:
> On 28/08/2015 01:47, Andreas Färber wrote:
>> The argument order for the !CONFIG_SOFTMMU case was jumbled up since
>> ffc6372851d8631a9f9fa56ec613b3244dc635b9 ("tcg/aarch64: use 32-bit
>> offset for 32-bit user-mode emulation"), regressing from -rc2 to v2.4.0.
>> Fix their order to avoid segfaults, e.g., in openSUSE's GNU coreutils 8.24.
> 
> How does -d out_asm change?  The patch was tested on aarch64.

Breaking stderr log attached. Diff below.

ffc6372851d8631a9f9fa56ec613b3244dc635b9^ vs.
ffc6372851d8631a9f9fa56ec613b3244dc635b9:

@@ -53,7 +53,7 @@
 0x006cd950:  52800000      mov w0, #0x0
 0x006cd954:  147ffefd      b #+0x1fffbf4 (addr 0x26cd548)
 0x006cd958:  d2800260      mov x0, #0x13
-0x006cd95c:  f2ae81c0      movk x0, #0x740e, lsl #16
+0x006cd95c:  f2b4fc60      movk x0, #0xa7e3, lsl #16
 0x006cd960:  f2c07fe0      movk x0, #0x3ff, lsl #32
 0x006cd964:  147ffef9      b #+0x1fffbe4 (addr 0x26cd548)

@@ -65,53 +65,53 @@
 0x006cd980:  f9009e74      str x20, [x19, #312]
 0x006cd984:  d1018294      sub x20, x20, #0x60 (96)
 0x006cd988:  f9409675      ldr x21, [x19, #296]
-0x006cd98c:  f83f6a95      str x21, [x20, xzr]
+0x006cd98c:  f8346bf5      str x21, [sp, x20]
 0x006cd990:  91002294      add x20, x20, #0x8 (8)
 0x006cd994:  f9409a75      ldr x21, [x19, #304]
-0x006cd998:  f83f6a95      str x21, [x20, xzr]
+0x006cd998:  f8346bf5      str x21, [sp, x20]
 0x006cd99c:  d1002294      sub x20, x20, #0x8 (8)
 0x006cd9a0:  f9009e74      str x20, [x19, #312]
 0x006cd9a4:  aa1403f5      mov x21, x20
 0x006cd9a8:  f9009675      str x21, [x19, #296]
 0x006cd9ac:  91014294      add x20, x20, #0x50 (80)
 0x006cd9b0:  f9408e76      ldr x22, [x19, #280]
-0x006cd9b4:  f83f6a96      str x22, [x20, xzr]
+0x006cd9b4:  f8346bf6      str x22, [sp, x20]
 0x006cd9b8:  91002294      add x20, x20, #0x8 (8)
 0x006cd9bc:  f9409276      ldr x22, [x19, #288]
-0x006cd9c0:  f83f6a96      str x22, [x20, xzr]
+0x006cd9c0:  f8346bf6      str x22, [sp, x20]
 0x006cd9c4:  9101e2b4      add x20, x21, #0x78 (120)
 0x006cd9c8:  f9008e74      str x20, [x19, #280]
 0x006cd9cc:  910042b6      add x22, x21, #0x10 (16)
 0x006cd9d0:  f9406e77      ldr x23, [x19, #216]
-0x006cd9d4:  f83f6ad7      str x23, [x22, xzr]
+0x006cd9d4:  f8366bf7      str x23, [sp, x22]
 0x006cd9d8:  910022d6      add x22, x22, #0x8 (8)
 0x006cd9dc:  f9407277      ldr x23, [x19, #224]
-0x006cd9e0:  f83f6ad7      str x23, [x22, xzr]
+0x006cd9e0:  f8366bf7      str x23, [sp, x22]
 0x006cd9e4:  910082b6      add x22, x21, #0x20 (32)
 0x006cd9e8:  f9407677      ldr x23, [x19, #232]
-0x006cd9ec:  f83f6ad7      str x23, [x22, xzr]
+0x006cd9ec:  f8366bf7      str x23, [sp, x22]
 0x006cd9f0:  910022d6      add x22, x22, #0x8 (8)
 0x006cd9f4:  f9407a77      ldr x23, [x19, #240]
-0x006cd9f8:  f83f6ad7      str x23, [x22, xzr]
+0x006cd9f8:  f8366bf7      str x23, [sp, x22]
 0x006cd9fc:  f9402276      ldr x22, [x19, #64]
 0x006cda00:  f9006e76      str x22, [x19, #216]
 0x006cda04:  910a8296      add x22, x20, #0x2a0 (672)
 0x006cda08:  f9002676      str x22, [x19, #72]
 0x006cda0c:  9100c2b7      add x23, x21, #0x30 (48)
 0x006cda10:  f9407e78      ldr x24, [x19, #248]
-0x006cda14:  f83f6af8      str x24, [x23, xzr]
+0x006cda14:  f8376bf8      str x24, [sp, x23]
 0x006cda18:  910022f7      add x23, x23, #0x8 (8)
 0x006cda1c:  f9408278      ldr x24, [x19, #256]
-0x006cda20:  f83f6af8      str x24, [x23, xzr]
+0x006cda20:  f8376bf8      str x24, [sp, x23]
 0x006cda24:  910102b5      add x21, x21, #0x40 (64)
 0x006cda28:  f9408677      ldr x23, [x19, #264]
-0x006cda2c:  f83f6ab7      str x23, [x21, xzr]
+0x006cda2c:  f8356bf7      str x23, [sp, x21]
 0x006cda30:  910022b5      add x21, x21, #0x8 (8)
 0x006cda34:  f9408a77      ldr x23, [x19, #272]
-0x006cda38:  f83f6ab7      str x23, [x21, xzr]
+0x006cda38:  f8356bf7      str x23, [sp, x21]
 0x006cda3c:  91010294      add x20, x20, #0x40 (64)
 0x006cda40:  f9002274      str x20, [x19, #64]
-0x006cda44:  f83f6a9f      str xzr, [x20, xzr]
+0x006cda44:  f8346bff      str xzr, [sp, x20]
 0x006cda48:  91002294      add x20, x20, #0x8 (8)
 0x006cda4c:  f9002274      str x20, [x19, #64]
 0x006cda50:  cb160295      sub x21, x20, x22
@@ -136,7 +136,7 @@
 0x006cda9c:  f2c00814      movk x20, #0x40, lsl #32
 0x006cdaa0:  f900a274      str x20, [x19, #320]
 0x006cdaa4:  d2801000      mov x0, #0x80
-0x006cdaa8:  f2ae81c0      movk x0, #0x740e, lsl #16
+0x006cdaa8:  f2b4fc60      movk x0, #0xa7e3, lsl #16
 0x006cdaac:  f2c07fe0      movk x0, #0x3ff, lsl #32
 0x006cdab0:  147ffea6      b #+0x1fffa98 (addr 0x26cd548)
 0x006cdab4:  14000000      b #+0x0 (addr 0x6cdab4)
@@ -145,135024 +145,12 @@
 0x006cdac0:  f2c00814      movk x20, #0x40, lsl #32
 0x006cdac4:  f900a274      str x20, [x19, #320]
 0x006cdac8:  d2801020      mov x0, #0x81
-0x006cdacc:  f2ae81c0      movk x0, #0x740e, lsl #16
+0x006cdacc:  f2b4fc60      movk x0, #0xa7e3, lsl #16
 0x006cdad0:  f2c07fe0      movk x0, #0x3ff, lsl #32
 0x006cdad4:  147ffe9d      b #+0x1fffa74 (addr 0x26cd548)
 0x006cdad8:  d2801060      mov x0, #0x83
-0x006cdadc:  f2ae81c0      movk x0, #0x740e, lsl #16
+0x006cdadc:  f2b4fc60      movk x0, #0xa7e3, lsl #16
 0x006cdae0:  f2c07fe0      movk x0, #0x3ff, lsl #32
 0x006cdae4:  147ffe99      b #+0x1fffa64 (addr 0x26cd548)

-OUT: [size=180]
-0x006cdaf0:  b85fc274      ldur w20, [x19, #-4]
-0x006cdaf4:  35000514      cbnz w20, #+0xa0 (addr 0x6cdb94)
[...]
+qemu: uncaught target signal 11 (Segmentation fault) - core dumped

Regards,
Andreas

-- 
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)

[-- Attachment #2: aarch64-post.log --]
[-- Type: text/x-log, Size: 7232 bytes --]

host mmap_min_addr=0x10000
guest_base  0x0
start            end              size             prot
0000004000000000-000000400001e000 000000000001e000 r-x
000000400001e000-000000400002d000 000000000000f000 ---
000000400002d000-0000004000030000 0000000000003000 rw-
0000004000030000-0000004000040000 0000000000010000 ---
0000004000040000-0000004000840000 0000000000800000 rw-
0000004000840000-000000400085f000 000000000001f000 r-x
000000400085f000-000000400086e000 000000000000f000 ---
000000400086e000-0000004000871000 0000000000003000 rw-
start_brk   0x0000000000000000
end_code    0x000000400001d2fc
start_code  0x0000004000000000
start_data  0x000000400002d358
end_data    0x000000400002e630
start_stack 0x000000400083f490
brk         0x000000400002f378
entry       0x0000004000840d80
PROLOGUE: [size=72]
0x026cd520:  a9ba7bfd      stp x29, x30, [sp, #-96]!
0x026cd524:  910003fd      mov x29, sp
0x026cd528:  a90153f3      stp x19, x20, [sp, #16]
0x026cd52c:  a9025bf5      stp x21, x22, [sp, #32]
0x026cd530:  a90363f7      stp x23, x24, [sp, #48]
0x026cd534:  a9046bf9      stp x25, x26, [sp, #64]
0x026cd538:  a90573fb      stp x27, x28, [sp, #80]
0x026cd53c:  d11203ff      sub sp, sp, #0x480 (1152)
0x026cd540:  aa0003f3      mov x19, x0
0x026cd544:  d61f0020      br x1
0x026cd548:  911203ff      add sp, sp, #0x480 (1152)
0x026cd54c:  a94153f3      ldp x19, x20, [sp, #16]
0x026cd550:  a9425bf5      ldp x21, x22, [sp, #32]
0x026cd554:  a94363f7      ldp x23, x24, [sp, #48]
0x026cd558:  a9446bf9      ldp x25, x26, [sp, #64]
0x026cd55c:  a94573fb      ldp x27, x28, [sp, #80]
0x026cd560:  a8c67bfd      ldp x29, x30, [sp], #96
0x026cd564:  d65f03c0      ret

OUT: [size=72]
0x006cd920:  b85fc274      ldur w20, [x19, #-4]
0x006cd924:  350001b4      cbnz w20, #+0x34 (addr 0x6cd958)
0x006cd928:  f9409e74      ldr x20, [x19, #312]
0x006cd92c:  f9002274      str x20, [x19, #64]
0x006cd930:  d281b114      mov x20, #0xd88
0x006cd934:  f2a01094      movk x20, #0x84, lsl #16
0x006cd938:  f2c00814      movk x20, #0x40, lsl #32
0x006cd93c:  f9009a74      str x20, [x19, #304]
0x006cd940:  d2887414      mov x20, #0x43a0
0x006cd944:  f2a01094      movk x20, #0x84, lsl #16
0x006cd948:  f2c00814      movk x20, #0x40, lsl #32
0x006cd94c:  f900a274      str x20, [x19, #320]
0x006cd950:  52800000      mov w0, #0x0
0x006cd954:  147ffefd      b #+0x1fffbf4 (addr 0x26cd548)
0x006cd958:  d2800260      mov x0, #0x13
0x006cd95c:  f2b4fc60      movk x0, #0xa7e3, lsl #16
0x006cd960:  f2c07fe0      movk x0, #0x3ff, lsl #32
0x006cd964:  147ffef9      b #+0x1fffbe4 (addr 0x26cd548)

OUT: [size=376]
0x006cd970:  b85fc274      ldur w20, [x19, #-4]
0x006cd974:  35000b34      cbnz w20, #+0x164 (addr 0x6cdad8)
0x006cd978:  f9409e74      ldr x20, [x19, #312]
0x006cd97c:  d1120294      sub x20, x20, #0x480 (1152)
0x006cd980:  f9009e74      str x20, [x19, #312]
0x006cd984:  d1018294      sub x20, x20, #0x60 (96)
0x006cd988:  f9409675      ldr x21, [x19, #296]
0x006cd98c:  f8346bf5      str x21, [sp, x20]
0x006cd990:  91002294      add x20, x20, #0x8 (8)
0x006cd994:  f9409a75      ldr x21, [x19, #304]
0x006cd998:  f8346bf5      str x21, [sp, x20]
0x006cd99c:  d1002294      sub x20, x20, #0x8 (8)
0x006cd9a0:  f9009e74      str x20, [x19, #312]
0x006cd9a4:  aa1403f5      mov x21, x20
0x006cd9a8:  f9009675      str x21, [x19, #296]
0x006cd9ac:  91014294      add x20, x20, #0x50 (80)
0x006cd9b0:  f9408e76      ldr x22, [x19, #280]
0x006cd9b4:  f8346bf6      str x22, [sp, x20]
0x006cd9b8:  91002294      add x20, x20, #0x8 (8)
0x006cd9bc:  f9409276      ldr x22, [x19, #288]
0x006cd9c0:  f8346bf6      str x22, [sp, x20]
0x006cd9c4:  9101e2b4      add x20, x21, #0x78 (120)
0x006cd9c8:  f9008e74      str x20, [x19, #280]
0x006cd9cc:  910042b6      add x22, x21, #0x10 (16)
0x006cd9d0:  f9406e77      ldr x23, [x19, #216]
0x006cd9d4:  f8366bf7      str x23, [sp, x22]
0x006cd9d8:  910022d6      add x22, x22, #0x8 (8)
0x006cd9dc:  f9407277      ldr x23, [x19, #224]
0x006cd9e0:  f8366bf7      str x23, [sp, x22]
0x006cd9e4:  910082b6      add x22, x21, #0x20 (32)
0x006cd9e8:  f9407677      ldr x23, [x19, #232]
0x006cd9ec:  f8366bf7      str x23, [sp, x22]
0x006cd9f0:  910022d6      add x22, x22, #0x8 (8)
0x006cd9f4:  f9407a77      ldr x23, [x19, #240]
0x006cd9f8:  f8366bf7      str x23, [sp, x22]
0x006cd9fc:  f9402276      ldr x22, [x19, #64]
0x006cda00:  f9006e76      str x22, [x19, #216]
0x006cda04:  910a8296      add x22, x20, #0x2a0 (672)
0x006cda08:  f9002676      str x22, [x19, #72]
0x006cda0c:  9100c2b7      add x23, x21, #0x30 (48)
0x006cda10:  f9407e78      ldr x24, [x19, #248]
0x006cda14:  f8376bf8      str x24, [sp, x23]
0x006cda18:  910022f7      add x23, x23, #0x8 (8)
0x006cda1c:  f9408278      ldr x24, [x19, #256]
0x006cda20:  f8376bf8      str x24, [sp, x23]
0x006cda24:  910102b5      add x21, x21, #0x40 (64)
0x006cda28:  f9408677      ldr x23, [x19, #264]
0x006cda2c:  f8356bf7      str x23, [sp, x21]
0x006cda30:  910022b5      add x21, x21, #0x8 (8)
0x006cda34:  f9408a77      ldr x23, [x19, #272]
0x006cda38:  f8356bf7      str x23, [sp, x21]
0x006cda3c:  91010294      add x20, x20, #0x40 (64)
0x006cda40:  f9002274      str x20, [x19, #64]
0x006cda44:  f8346bff      str xzr, [sp, x20]
0x006cda48:  91002294      add x20, x20, #0x8 (8)
0x006cda4c:  f9002274      str x20, [x19, #64]
0x006cda50:  cb160295      sub x21, x20, x22
0x006cda54:  f10002bf      cmp x21, #0x0 (0)
0x006cda58:  1a9f07f7      cset w23, ne
0x006cda5c:  b9020e77      str w23, [x19, #524]
0x006cda60:  d360feb7      lsr x23, x21, #32
0x006cda64:  b9020a77      str w23, [x19, #520]
0x006cda68:  eb16029f      cmp x20, x22
0x006cda6c:  1a9f37f7      cset w23, hs
0x006cda70:  b9020277      str w23, [x19, #512]
0x006cda74:  ca1402b5      eor x21, x21, x20
0x006cda78:  ca160294      eor x20, x20, x22
0x006cda7c:  8a1402b4      and x20, x21, x20
0x006cda80:  d360fe94      lsr x20, x20, #32
0x006cda84:  b9020674      str w20, [x19, #516]
0x006cda88:  b9420e74      ldr w20, [x19, #524]
0x006cda8c:  35000154      cbnz w20, #+0x28 (addr 0x6cdab4)
0x006cda90:  14000000      b #+0x0 (addr 0x6cda90)
0x006cda94:  d2887b94      mov x20, #0x43dc
0x006cda98:  f2a01094      movk x20, #0x84, lsl #16
0x006cda9c:  f2c00814      movk x20, #0x40, lsl #32
0x006cdaa0:  f900a274      str x20, [x19, #320]
0x006cdaa4:  d2801000      mov x0, #0x80
0x006cdaa8:  f2b4fc60      movk x0, #0xa7e3, lsl #16
0x006cdaac:  f2c07fe0      movk x0, #0x3ff, lsl #32
0x006cdab0:  147ffea6      b #+0x1fffa98 (addr 0x26cd548)
0x006cdab4:  14000000      b #+0x0 (addr 0x6cdab4)
0x006cdab8:  d2887a14      mov x20, #0x43d0
0x006cdabc:  f2a01094      movk x20, #0x84, lsl #16
0x006cdac0:  f2c00814      movk x20, #0x40, lsl #32
0x006cdac4:  f900a274      str x20, [x19, #320]
0x006cdac8:  d2801020      mov x0, #0x81
0x006cdacc:  f2b4fc60      movk x0, #0xa7e3, lsl #16
0x006cdad0:  f2c07fe0      movk x0, #0x3ff, lsl #32
0x006cdad4:  147ffe9d      b #+0x1fffa74 (addr 0x26cd548)
0x006cdad8:  d2801060      mov x0, #0x83
0x006cdadc:  f2b4fc60      movk x0, #0xa7e3, lsl #16
0x006cdae0:  f2c07fe0      movk x0, #0x3ff, lsl #32
0x006cdae4:  147ffe99      b #+0x1fffa64 (addr 0x26cd548)

qemu: uncaught target signal 11 (Segmentation fault) - core dumped

  reply	other threads:[~2015-09-01 16:32 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-27 23:47 [Qemu-devel] [PATCH] tcg/aarch64: Fix tcg_out_qemu_{ld, st} for linux-user Andreas Färber
2015-08-28  4:30 ` Richard Henderson
2015-08-28 16:23   ` Andreas Färber
2015-08-29  5:33     ` Richard Henderson
2015-08-29  6:50 ` Paolo Bonzini
2015-09-01 16:31   ` Andreas Färber [this message]
2015-09-01 20:09     ` Richard Henderson
2015-09-02  9:02     ` Paolo Bonzini
2015-09-02 14:26       ` Richard Henderson
2015-09-02 14:38       ` Andreas Färber

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=55E5D2FA.30908@suse.de \
    --to=afaerber@suse.de \
    --cc=aurelien@aurel32.net \
    --cc=claudio.fontana@huawei.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-stable@nongnu.org \
    --cc=rth@twiddle.net \
    /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.