[Qemu-devel] [Bug 1689499] Re: copy-storage-all/inc does not easily converge with load going on

2017-07-08 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1689499 Title: copy-storag

Re: [Qemu-devel] [PATCH v3 1/4] ACPI: Add APEI GHES Table Generation support

2017-07-08 Thread gengdongjiu
Laszlo, thanks for your clear and detailed answer. I completely understand what you mean. 2017-07-07 17:43 GMT+08:00, Laszlo Ersek : > On 07/07/17 10:32, gengdongjiu wrote: >> Hi Laszlo, >>sorry for my late response. >> >> On 2017/6/3 20:01, Laszlo Ersek wrote: >>> On 05/22/17 16:23, Laszlo

[Qemu-devel] Wiki page for CRIS

2017-07-08 Thread Programmingkid
I have recently made a wiki page for the CRIS emulator in QEMU. I have supplied as much information as I could but there is still a need for more. If anyone knows what the suggested command-line should be or knows of more information which should be included please contact me and I will be glad

Re: [Qemu-devel] [Bug 1703147] [NEW] Xfer:features:read truncating xml sent to gdb frontends

2017-07-08 Thread Peter Maydell
On 8 July 2017 at 22:19, Duane Voth wrote: > Around line 1326 in gdbstub.c: > > if (len > (MAX_PACKET_LENGTH - 5) / 2) > len = (MAX_PACKET_LENGTH - 5) / 2; > > is truncating processor reg description xml files longer than 2045 > bytes. Deleting these lines works for my

[Qemu-devel] [Bug 1703147] [NEW] Xfer:features:read truncating xml sent to gdb frontends

2017-07-08 Thread Duane Voth
Public bug reported: Around line 1326 in gdbstub.c: if (len > (MAX_PACKET_LENGTH - 5) / 2) len = (MAX_PACKET_LENGTH - 5) / 2; is truncating processor reg description xml files longer than 2045 bytes. Deleting these lines works for my immediate need, but they seem to

Re: [Qemu-devel] [Qemu-arm] [PATCH 3/3] linux-user/sh4: Reduce TARGET_VIRT_ADDR_SPACE_BITS to 31

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/07/2017 11:50 PM, Richard Henderson wrote: The real kernel has TASK_SIZE as 0x7c00, due to quirks with a couple of SH parts. But nominally user-space is limited to 2GB. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/cpu.h | 6 +- 1 fil

Re: [Qemu-devel] [Qemu-arm] [PATCH 1/3] tcg: Fix off-by-one in assert in page_set_flags

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/07/2017 11:50 PM, Richard Henderson wrote: Most of the users of page_set_flags offset (page, page + len) as the end points. One might consider this an error, since the other users do supply an endpoint as the last byte of the region. However, the first thing that page_set_flags does is ro

Re: [Qemu-devel] [PATCH v2 20/27] target/sh4: Unify code for CHECK_FPU_ENABLED

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:21 PM, Richard Henderson wrote: We do not need to emit N copies of raising an exception. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/translate.c | 24 ++-- 1 file changed, 14 insertions(+), 10 deletions(-) dif

Re: [Qemu-devel] [PATCH v2 21/27] target/sh4: Tidy misc illegal insn checks

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:21 PM, Richard Henderson wrote: Now that we have a do_illegal label, use goto in order to self-document the forcing of the exception. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/translate.c | 22 +- 1 file chan

Re: [Qemu-devel] [PATCH v2 19/27] target/sh4: Unify code for CHECK_PRIVILEGED

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:21 PM, Richard Henderson wrote: We do not need to emit N copies of raising an exception. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/translate.c | 14 -- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/t

Re: [Qemu-devel] [PATCH v2 18/27] target/sh4: Unify code for CHECK_NOT_DELAY_SLOT

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:21 PM, Richard Henderson wrote: We do not need to emit N copies of raising an exception. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/translate.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/targe

Re: [Qemu-devel] [PATCH v2 12/27] target/sh4: Pass DisasContext to fpr64 routines

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:20 PM, Richard Henderson wrote: Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/translate.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/target/sh4/translate.c b/target/sh4/translate.

Re: [Qemu-devel] [PATCH v2 11/27] target/sh4: Unify cpu_fregs into FREG

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:20 PM, Richard Henderson wrote: We were treating FREG as an index and REG as a TCGv. Making FREG return a TCGv is both less confusing and a step toward cleaner banking of cpu_fregs. Signed-off-by: Richard Henderson still: Reviewed-by: Philippe Mathieu-Daudé --- target/s

Re: [Qemu-devel] [PATCH v2 05/27] target/sh4: Adjust TB_FLAG_PENDING_MOVCA

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:20 PM, Richard Henderson wrote: Don't leave an unused bit after DELAY_SLOT_MASK. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/cpu.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/target/sh4/cpu.h b/target

Re: [Qemu-devel] [PATCH v2 04/27] target/sh4: Keep env->flags clean

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:20 PM, Richard Henderson wrote: If we mask off any out-of-band bits before we assign to the variable, then we don't need to clean it up when reading. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/cpu.h | 2 +- target/sh4/cpu.c | 2 +

Re: [Qemu-devel] [PATCH v2 03/27] target/sh4: Introduce TB_FLAG_ENVFLAGS_MASK

2017-07-08 Thread Philippe Mathieu-Daudé
On 07/06/2017 11:20 PM, Richard Henderson wrote: We'll be putting more things into this bitmask soon. Let's have a name that covers all possible uses. Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé --- target/sh4/cpu.h | 4 +++- target/sh4/translate.c | 4 ++-

Re: [Qemu-devel] [Qemu-arm] [RFC PATCH] target/arm: ensure eret exits the run-loop

2017-07-08 Thread Philippe Mathieu-Daudé
Hi Alex, On 07/07/2017 03:52 PM, Richard Henderson wrote: On 07/07/2017 08:29 AM, Alex Bennée wrote: Naming all of these different exit conditions is certainly non-trivial. Given the variation of usage this is something that should probably be done after Lluís common run loop goes in and we c