On Sat, Jun 03, 2017 at 05:05:26PM -0500, Michael Roth wrote:
> Quoting David Gibson (2017-06-02 02:29:48)
> > Currently we only have a single QOM type for all DRCs, but lots of
> > places where we switch behaviour based on the DRC's PAPR defined type.
> > This is a poor use of our existing type sy
On Sat, Jun 03, 2017 at 05:24:23PM -0500, Michael Roth wrote:
> Quoting David Gibson (2017-06-02 02:29:50)
> > Currently the sPAPRMachineState contains a of sPAPRConfigureConnector
>
> "contains a list"?
Ta, corrected.
> > structures which store intermediate state for the ibm,configure-connector
On 05/26/2017 01:57 AM, Michael S. Tsirkin wrote:
I think that's a very valid point. Linux isn't currently optimized to
handle packets in device BAR.
There are several issues here and you do need to address them in the
kernel, no way around that:
1. lots of drivers set protection to
v
On Fri, Jun 02, 2017 at 10:15:25AM +0200, Greg Kurz wrote:
> On Fri, 2 Jun 2017 12:00:07 +1000
> David Gibson wrote:
>
> > On Thu, Jun 01, 2017 at 03:09:15PM +0200, Greg Kurz wrote:
> > > On Thu, 1 Jun 2017 13:59:14 +0200
> > > Cédric Le Goater wrote:
> > >
> > > > On 06/01/2017 08:52 AM, Dav
From: Thomas Huth
Add a link to the GPLv2 and a link to the LICENSE file in the
QEMU repository to fix the two TODO items in this appendix.
Signed-off-by: Thomas Huth
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Michael Tokarev
---
qemu-doc.texi | 8
1 file changed, 4 insertions(+),
From: Juan Quintela
All the file is surounded already by #ifndef CONFIG_USER_ONLY.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
Reviewed-by: Laurent Vivier
Signed-off-by: Michael Tokarev
---
include/exec/memory.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/inclu
From: Luc MICHEL
The cp15, CRn=15, opc1=0, CRm=5, opc2=0 instruction invalidates all the
data cache on the cortex-r5. Implementing it as a NOP.
Signed-off-by: Luc MICHEL
Signed-off-by: Michael Tokarev
---
target/arm/cpu.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/arm/cpu.c
From: Paolo Bonzini
Use sizeof instead of ARRAY_SIZE, fixing -Wmemset-elt-size with recent
GCC versions.
Signed-off-by: Paolo Bonzini
Reviewed-by: Laurent Vivier
Signed-off-by: Michael Tokarev
---
hw/timer/altera_timer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/
From: Thomas Huth
When you currently try to run a test directly from the command line
without setting the QTEST_QEMU_BINARY environment variable first,
you are presented with an unhelpful assertion message like this:
ERROR:tests/libqtest.c:163:qtest_init_without_qmp_handshake:
assertion failed
From: Thomas Huth
The target-specific code in nmi.c has been removed with this commit:
commit f7e981f29548fe4af7812f5920304fe607e5bf0d
nmi: remove x86 specific nmi handling
Signed-off-by: Thomas Huth
Reviewed-by: Juan Quintela
Signed-off-by: Michael Tokarev
---
hw/core/Makef
Trivial patches for 04 Jun 2017. Please consider applying.
/mjt
The following changes since commit c6e84fbd447a51e1161d74d71566a5f67b47eac5:
Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging
(2017-06-02 17:46:22 +0100)
are available in the git repository at:
git://
From: Max Filippov
Replace malloc/free/sprintf with g_string/g_string_printf/g_string_free.
Replace g_malloc with g_new when allocating the MemoryRegion to get more
type safety.
Suggested-by: Alex Bennée
Signed-off-by: Max Filippov
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Michael To
On 06/03/2017 06:49 AM, Laszlo Ersek wrote:
On 06/02/17 11:55, Marc-André Lureau wrote:
Hi
On Fri, Jun 2, 2017 at 1:46 PM Marc-André Lureau
wrote:
Hi
On Thu, Jun 1, 2017 at 10:19 PM Eric Blake wrote:
On 06/01/2017 01:06 PM, Laszlo Ersek wrote:
On 06/01/17 15:03, Marc-André Lureau wrot
From: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Acked-by: Aurelien Jarno
Signed-off-by: Michael Tokarev
---
include/hw/mips/mips.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/hw/mips/mips.h b/include/hw/mips/mips.h
index e0065ce808..16412dc150 100644
--- a/i
From: John Snow
To quiet patchew, add an assert for fwrite's return value.
Signed-off-by: John Snow
Reviewed-by: Jeff Cody
Reviewed-by: Laurent Vivier
Signed-off-by: Michael Tokarev
---
tests/ide-test.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/tests/ide-te
From: Thomas Huth
The qemu-ga description is currently a subsection of the Disk Images
chapter - which does not make much sense since the qemu-ga is not
directly related to disk images. So let's move this information
into a separate chapter instead.
Signed-off-by: Thomas Huth
Reviewed-by: Stefa
From: Marc-André Lureau
There is no need to duplicate a fixed string.
Signed-off-by: Marc-André Lureau
Reviewed-by: Eric Blake
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Michael Tokarev
---
qga/commands-posix.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/q
From: Suraj Jitindar Singh
The help text for the thread sub option of the accel option is missing
a newline at the end. This is annoying as it makes it hard to see the
help text for the next option.
Add the new line so that the following option help text (-smp) is
displayed on a new line rather
From: Marc-André Lureau
Spotted by ASAN.
Signed-off-by: Marc-André Lureau
Signed-off-by: Michael Tokarev
---
memory_mapping.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/memory_mapping.c b/memory_mapping.c
index 6a39d71da2..a5d38552a6 100644
--- a/memory_mapping.c
+++ b/memory_mapping
From: Eric Blake
In the process of getting rid of docs/qmp-commands.txt, we managed
to regress on some of the text that changed after the point where
the move was first branched and when the move actually occurred.
For example, commit 3282eca for blockdev-snapshot re-added the
extra "options" lay
From: Markus Armbruster
Resynchronize the table of default device suppressions with vl.c's
default_list[].
Signed-off-by: Markus Armbruster
Reviewed-by: Thomas Huth
Signed-off-by: Michael Tokarev
---
docs/qdev-device-use.txt | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
dif
From: Markus Armbruster
Suggested-by: Thomas Huth
Signed-off-by: Markus Armbruster
Reviewed-by: Thomas Huth
Signed-off-by: Michael Tokarev
---
docs/bootindex.txt | 2 +-
docs/qdev-device-use.txt | 4 ++--
qemu-options.hx | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
From: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alistair Francis
Signed-off-by: Michael Tokarev
---
hw/core/register.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/core/register.c b/hw/core/register.c
index da38ef3a54..900294b9
From: Kamil Rytarowski
NetBSD ships with traditional BSD curses with compatibility with ncurses.
qemu works nicely with the basesystem version of curses(3) from NetBSD.
The only mismatch between curses(3) and ncurses is the lack of
curses_version() in the NetBSD version. This function is used so
From: Kamil Rytarowski
Currently ivshmem requires eventfd() which is Linux specific.
Do not and build it unconditionally on every Linux/BSD/Solaris.
This patch indirectly fixes build failure on NetBSD, where these tools
additionally require -lrt for shm_open(3). In future there should be
added s
From: Mao Zhongyi
lsi_mem_read/write() always return 0 about which their
callers actually don't care. Change the function type
to void.
Signed-off-by: Mao Zhongyi
Signed-off-by: Michael Tokarev
---
hw/scsi/lsi53c895a.c | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --g
From: Eric Blake
Use the correct command name.
Signed-off-by: Eric Blake
Signed-off-by: Michael Tokarev
---
qapi/block-core.json | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/qapi/block-core.json b/qapi/block-core.json
index ea0b3e8b13..309b1df214 100644
--- a/qapi/block
From: Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé
Reviewed-by: Alex Bennée
Signed-off-by: Michael Tokarev
---
hw/sparc64/sun4u.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/sparc64/sun4u.c b/hw/sparc64/sun4u.c
index 18b8f8bcba..69f565db25 100644
---
On 05/29/2017 05:12 AM, David Hildenbrand wrote:
We have to make the address in the old PSW point at the next
instruction, as addressing exceptions are suppressing and not
nullifying.
I assume that there are a lot of other broken cases (as most instructions
we care about are suppressing) - all t
0/qemu.git tags/pull-s390-20170604
for you to fetch changes up to 2be5fdbe14165d47a88054c9d117801e09239098:
target/s390x: addressing exceptions are suppressing (2017-06-04 10:23:15
-0700)
Queu
From: Thomas Huth
TEST BLOCK was likely once used to execute basic memory
tests, but nowadays it's just a (slow) way to clear a page.
Reviewed-by: Aurelien Jarno
Reviewed-by: David Hildenbrand
Reviewed-by: Richard Henderson
Signed-off-by: Thomas Huth
Message-Id: <1495128400-23759-1-git-send-
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 0c6a0d9..e3325a4 100644
--- a/target/s39
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 29 +
target/s390x/translate.c | 1 -
2 files changed, 17 insertions(+), 13 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 78a9ac1..5
This will avoid needing forward declarations in following patches.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 161 +++---
1 file changed, 81 insertions(+), 80 deletions(-)
diff --git a/target/s390x/mem_he
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 5 +++--
target/s390x/translate.c | 1 -
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 33d83e5..af2
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 7 ---
target/s390x/translate.c | 1 -
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 2acc984..4
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 31 ++-
target/s390x/translate.c | 1 -
2 files changed, 18 insertions(+), 14 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 90b62fa.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 30 ++
target/s390x/translate.c | 1 -
2 files changed, 18 insertions(+), 13 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index b71437a..
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 31 ++-
target/s390x/translate.c | 1 -
2 files changed, 18 insertions(+), 14 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 7d6133b.
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 7 ---
target/s390x/translate.c | 1 -
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 49cfc9b..8
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 11 ++-
target/s390x/translate.c | 1 -
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index f5a3
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 25 +++--
target/s390x/translate.c | 1 -
2 files changed, 15 insertions(+), 11 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 0701e10..b37a9
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 44
target/s390x/translate.c | 1 -
2 files changed, 24 insertions(+), 21 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
i
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 9 +
target/s390x/translate.c | 1 -
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index d4ee364.
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 3 ++-
target/s390x/translate.c | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 88e817a..2acc9
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 28
target/s390x/translate.c | 1 -
2 files changed, 16 insertions(+), 13 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 11 ++-
target/s390x/translate.c | 1 -
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 5068
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 9 +
target/s390x/translate.c | 1 -
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 68e3817.
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 6 +++---
target/s390x/translate.c | 1 -
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 8a095ad..f5
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 3 +++
target/s390x/translate.c | 1 -
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index b64c04e..7df2e5
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 3 ++-
target/s390x/translate.c | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 3e75cae..33d83
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 10 ++
target/s390x/translate.c | 2 --
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 614c
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 15 ---
target/s390x/translate.c | 1 -
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 2 +-
target/s390x/mem_helper.c | 9 +
target/s390x/translate.c | 3 +--
3 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/target/s390x/helper.h b/target/s
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Reviewed-by: Philippe Mathieu-Daudé
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-2-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/translate.c | 5 +
1 file changed, 1 insertion(+), 4 deletions
Fix saving exception_index around mmu_translate; eliminate a dead store.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 6 +++---
target/s390x/translate.c | 1 -
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/target/s390x/mem_helper
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 8 ++--
target/s390x/translate.c | 2 --
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 17d8257.
Use this saved value instead of recomputing from next_pc difference.
Reviewed-by: David Hildenbrand
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/translate.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/target/s390x/translate.c b/t
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 5 +++--
target/s390x/translate.c | 1 -
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 1c36a47..e4b
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-6-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/insn-data.def | 3 +++
target/s390x/translate.c | 10 ++
2 files changed, 13 insertions(+)
Also provide the cross-cpu tlb flushing required by the PoO.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 2 +-
target/s390x/insn-data.def | 2 +-
target/s390x/mem_helper.c | 32
target/s390x/translate.c | 42
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 3 ++-
target/s390x/translate.c | 1 -
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index e4bfc6e..88e81
Previously, helper_ex would construct the insn and then implement
the insn via direct calls other helpers. This was sufficient to
boot Linux but that is all.
It is easy enough to go the whole nine yards by stashing state for
EXECUTE within the cpu, and then rely on a new TB to be created
that pro
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-7-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 5 +
target/s390x/mem_helper.c | 37 ++
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/insn-data.def | 1 +
target/s390x/translate.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/target/s390x/insn-data.def b/target/s390x/insn-data.def
index 4c91f30..f818437 100644
--- a/target/s390x/insn-data.d
From: Aurelien Jarno
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-3-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/cpu.h| 2 ++
target/s390x/mem_helper.c | 17 ++---
target/s390x/mmu_helper.c | 2 --
3 files changed, 12 insertio
From: Aurelien Jarno
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-8-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 13 +
target/s390x/translate.c |
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 8
target/s390x/translate.c | 1 -
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index ff12777..
This split will be required for implementing EXECUTE properly.
Do this now as a separate step to aid comparison of before and
after TB listings.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 54 ---
targe
Reviewed-by: Thomas Huth
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 1 -
target/s390x/translate.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index 7df2e53..a8c85c9 100644
--- a/tar
From: Aurelien Jarno
Improve fix_address to also handle the 24-bit mode. Rename fix_address
to wrap_address to better explain what is changed.
Replace the calls to get_address with x2 = 0 and b2 = 0 by
call to wrap_address, leading to the removal of this function. Rename
get_address_31fix into g
From: Aurelien Jarno
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-9-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 12
target/s390x/translate.c | 6
While the previous patch is required for proper conformance,
the vast majority of target insns are MVC and XC for implementing
memmove and memset respectively. The next most common are CLC,
TR, and SVC.
Implementing these (and a few others for which we already have
an implementation) directly is
(1) The OR of the low bits or R1 into INSN were not being done
consistently; it was forgotten along all but the SVC path.
(2) The setting of ILEN was wrong on SVC path for EXRL.
(3) The data load for ICM read too much.
Fix these by consolidating data load at the beginning, using
get_ilen to contro
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-16-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 37 ++---
1 file changed, 30 insertions(+), 7 deletions
From: Aurelien Jarno
And at the same time make IPTE SMP aware.
Reviewed-by: Thomas Huth
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-4-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 2 +-
target/s390x
From: Aurelien Jarno
These functions differ from COMPARE by generating an exception for a
QNaN input. Use the non quiet version of floatXX_compare.
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-10-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/fpu_he
The PoO specifies that when R1==0, no ORing into the insn
loaded from storage takes place. Load a zero for this case.
Reviewed-by: Aurelien Jarno
Signed-off-by: Richard Henderson
---
target/s390x/insn-data.def | 4 ++--
target/s390x/translate.c | 14 +-
2 files changed, 15 inser
From: Aurelien Jarno
adj_len_to_page doesn't return the correct result when the address
is already page aligned and the length is bigger than a page. Fix that.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-20-aurel...@aurel32.net>
Signed-off-by:
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-11-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 12
From: Aurelien Jarno
As CLCL and CLCLE mostly differ by their operands, use a common do_clcl
helper. Another difference is that CLCL is not interruptible.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-19-aurel...@aurel32.net>
Signed-off-by: Rich
From: Aurelien Jarno
As MVCL and MVCLE only differ by their operands, use a common
do_mvcl helper. Optimize it calling fast_memmove and fast_memset.
Correctly write back addresses. Check that r1 and r2/r3 registers
are even.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-
From: Thomas Huth
Currently we only present the plain z900 feature bits to the guest,
but QEMU already emulates some additional features (but not all of
the next CPU generation, so we can not use the next CPU level as
default yet). Since newer Linux kernels are checking the feature bits
and refus
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-12-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 13
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-5-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/insn-data.def | 3 +++
target/s390x/translate.c | 10 ++
2 files changed, 13 insertions(+)
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-28-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 3 +++
target/s390x/mem_helper.c | 23 +++
From: Aurelien Jarno
Use a common helper with PACK ASCII as the differences are limited to
the stride of the source operand.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-25-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s39
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-13-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 4
target/s390x/mem_helper.c | 31 ++
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-14-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 13
From: Aurelien Jarno
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-29-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 9 +
target/s390x/mem_helper.c | 45 +
From: Aurelien Jarno
For that we need to make program_interrupt available to qemu-user.
Fortunately there is almost nothing to change as both kvm_enabled and
CONFIG_KVM evaluate to false in that case.
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.2772
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-17-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/mem_helper.c | 47 +--
1 file changed, 33 insertions(+), 1
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-23-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 47
From: David Hildenbrand
We have to make the address in the old PSW point at the next
instruction, as addressing exceptions are suppressing and not
nullifying.
I assume that there are a lot of other broken cases (as most instructions
we care about are suppressing) - all trigger_pgm_exception() sp
From: Aurelien Jarno
There are multiple issues with the COMPARE LOGICAL LONG EXTENDED
instruction:
- The test between the two operands is inverted, leading to an inversion
of the cc values 1 and 2.
- The address and length of an operand continue to be decreased after
reaching the end of this
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-26-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 51
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-24-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 35
From: Aurelien Jarno
Reviewed-by: Richard Henderson
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-27-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/helper.h | 1 +
target/s390x/insn-data.def | 2 ++
target/s390x/mem_helper.c | 49
From: Aurelien Jarno
Signed-off-by: Aurelien Jarno
Message-Id: <20170531220129.27724-30-aurel...@aurel32.net>
Signed-off-by: Richard Henderson
---
target/s390x/cpu_models.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/target/s390x/cpu_models.c b/target/s390x/cpu_models.c
index e5e005a
On 06/02/2017 04:39 PM, Emilio G. Cota wrote:
+aligned = (void *)ROUND_UP((uintptr_t)s->code_gen_ptr, 64);
I would prefer that this and
+} QEMU_ALIGNED(64);
this both use a define. We may well have to adjust this for different hosts.
In particular I'm thinking of PPC64 which would pre
1 - 100 of 134 matches
Mail list logo
