Re: [Qemu-devel] [PATCH v16 00/14] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD

2014-03-09 Thread Benoît Canet
The Monday 10 Mar 2014 à 14:38:12 (+0800), Fam Zheng wrote : > On Mon, 03/10 06:13, Benoît Canet wrote: > > The Friday 07 Mar 2014 à 16:20:42 (+0800), Fam Zheng wrote : > > > This series adds for point-in-time snapshot NBD exporting based on > > > blockdev-backup (variant of drive-backup with exist

Re: [Qemu-devel] [PATCH v16 00/14] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD

2014-03-09 Thread Fam Zheng
On Mon, 03/10 06:13, Benoît Canet wrote: > The Friday 07 Mar 2014 à 16:20:42 (+0800), Fam Zheng wrote : > > This series adds for point-in-time snapshot NBD exporting based on > > blockdev-backup (variant of drive-backup with existing device as target). > > > > We get a thin point-in-time snapshot

Re: [Qemu-devel] [PATCH v3 1/3] loader: rename in_ram/has_mr

2014-03-09 Thread Jeff E. Nelson
On 03/09/2014 02:06 PM, Michael S. Tsirkin wrote: we put copy of ROMs in MR for migration. but the name rom_in_ram makes one think we load it in guest RAM. Rename has_mr to make intent clearer. Signed-off-by: Michael S. Tsirkin --- include/hw/loader.h | 2 +- hw/core/loader.c| 6 +++---

Re: [Qemu-devel] [PATCH v16 00/14] Drop in_use from BlockDriverState and enable point-in-time snapshot exporting over NBD

2014-03-09 Thread Benoît Canet
The Friday 07 Mar 2014 à 16:20:42 (+0800), Fam Zheng wrote : > This series adds for point-in-time snapshot NBD exporting based on > blockdev-backup (variant of drive-backup with existing device as target). > > We get a thin point-in-time snapshot by COW mechanism of drive-backup, and > export it t

Re: [Qemu-devel] [PULL] migration patches

2014-03-09 Thread Amit Shah
Hi Peter, On (Sat) 08 Mar 2014 [12:19:10], Peter Maydell wrote: > On 6 March 2014 20:03, Amit Shah wrote: > > Hi Juan, > > > > Here's a compilation of migration-related patches from the list that > > I've reviewed. > > > > For the first patch, I picked Markus's over yours, just because it's > > b

[Qemu-devel] [PATCH qom-cpu v2 00/40] QOM CPUState, part 13: Emptying CPU_COMMON

2014-03-09 Thread Andreas Färber
Hello, This series aggressively decimizes CPU_COMMON fields and then begins with some follow-up cleanups in core CPU code. Paolo has suggested we handle cputlb.c not as "common code" but as code to be inlined into target code. I have therefore dropped some cputlb.c changes that seemed mostly rele

[Qemu-devel] [PATCH qom-cpu v2 03/40] target-i386: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits fdfba1a298ae26dd44bcfdb0429314139a0bc55a, f606604f1c10b60ef294f1b9b229426521a365e3 and 2c17449b3022ca9623c4a7e2a504a4150ac4ad30 added usages of ENV_GET_CPU() macro in target-specific code. Use x86_env_get_cpu() or reuse existing X86CPU variable instead. Cc: Edgar E. Iglesias Cc: Peter Ma

[Qemu-devel] [PATCH qom-cpu v2 05/40] target-s390x: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits f606604f1c10b60ef294f1b9b229426521a365e3, 2c17449b3022ca9623c4a7e2a504a4150ac4ad30 and 5ce5944dc0ffdc43c11b5cad11e526f699aabe4c added usages of ENV_GET_CPU() macro in target-specific code. Use s390_env_get_cpu() instead. Cc: Edgar E. Iglesias Cc: Peter Maydell Signed-off-by: Andreas Fär

[Qemu-devel] [PATCH qom-cpu v2 37/40] cputlb: Change tlb_flush_page() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cputlb.c | 4 +-- exec.c | 7 ++ include/exec/exec-all.h | 4 +-- target-alpha/sys_helper.c| 2 +- target-arm/helper.c | 14 --- target-cris/mmu.c| 3 ++- target-cris/op_help

[Qemu-devel] [PATCH qom-cpu v2 30/40] exec: Change cpu_breakpoint_{insert, remove{, _by_ref, _all}} argument

2014-03-09 Thread Andreas Färber
Use CPUState. Allows to clean up CPUArchState in gdbstub. Signed-off-by: Andreas Färber --- exec.c | 20 gdbstub.c | 20 include/exec/cpu-all.h | 15 --- include/qom/cpu.h | 15 +++ linux-user/mai

[Qemu-devel] [PATCH qom-cpu v2 25/40] translate-all: Change cpu_io_recompile() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- include/exec/exec-all.h | 2 +- include/exec/softmmu_template.h | 4 ++-- translate-all.c | 4 ++-- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h index cf5cd71..727dc3c 100

[Qemu-devel] [PATCH qom-cpu v2 28/40] target-ppc: Use PowerPCCPU in PowerPCCPUClass::handle_mmu_fault hook

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- target-ppc/cpu-qom.h| 8 +--- target-ppc/mmu-hash32.c | 5 +++-- target-ppc/mmu-hash32.h | 2 +- target-ppc/mmu-hash64.c | 5 +++-- target-ppc/mmu-hash64.h | 2 +- target-ppc/mmu_helper.c | 2 +- 6 files changed, 14 insertions(+), 10 deletions(-) diff --

[Qemu-devel] [PATCH qom-cpu v2 22/40] translate-all: Change cpu_restore_state() argument to CPUState

2014-03-09 Thread Andreas Färber
This lets us drop some local variables in tlb_fill() functions. Signed-off-by: Andreas Färber --- hw/i386/kvmvapic.c| 2 +- include/exec/exec-all.h | 2 +- target-alpha/helper.c | 2 +- target-alpha/mem_helper.c | 7 ++- target-arm/op_helper.c| 2 +-

[Qemu-devel] [PATCH qom-cpu v2 21/40] cpu-exec: Change cpu_loop_exit() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-exec.c| 14 ++ exec.c| 2 +- include/exec/exec-all.h | 2 +- target-alpha/helper.c | 4 ++-- target-alpha/mem_helper.c | 4 ++-- target-arm/op_helper.c| 6 +++--- target-cr

[Qemu-devel] [PATCH qom-cpu v2 01/40] target-alpha: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits 2c17449b3022ca9623c4a7e2a504a4150ac4ad30, fdfba1a298ae26dd44bcfdb0429314139a0bc55a, ab1da85791340e504d10487e1add81b9988afa98 and f606604f1c10b60ef294f1b9b229426521a365e3 added usages of ENV_GET_CPU() macro in target-specific code. Use alpha_env_get_cpu() instead. Cc: Edgar E. Iglesias Cc

[Qemu-devel] [PATCH qom-cpu v2 17/40] cpu: Move opaque field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- bsd-user/main.c | 2 +- gdbstub.c| 3 +-- include/exec/cpu-defs.h | 3 --- include/qom/cpu.h| 3 +++ linux-user/elfload.c | 10 ++ linux-user/linuxload.c | 3 +-- linux-user/m68k/target_

[Qemu-devel] [PATCH qom-cpu v2 40/40] user-exec: Change exception_action() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- user-exec.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/user-exec.c b/user-exec.c index 3b795c1..bc58056 100644 --- a/user-exec.c +++ b/user-exec.c @@ -38,11 +38,12 @@ //#define DEBUG_SIGNAL -static void exception_action(C

[Qemu-devel] [PATCH qom-cpu v2 15/40] cpu: Move jmp_env field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-exec.c | 8 +--- include/exec/cpu-defs.h | 2 -- include/qom/cpu.h | 2 ++ user-exec.c | 3 ++- 4 files changed, 9 insertions(+), 6 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index dd8da53..3e17ff5 100644 --- a/cpu-e

[Qemu-devel] [PATCH qom-cpu v2 39/40] cputlb: Change tlb_set_page() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cputlb.c | 4 ++-- include/exec/exec-all.h| 2 +- target-alpha/helper.c | 2 +- target-arm/helper.c| 2 +- target-cris/helper.c | 2 +- target-i386/helper.c | 2 +- target-lm32/helper.c | 6 +++--- target-m68k/

[Qemu-devel] [PATCH qom-cpu v2 13/40] cpu: Move icount_decr field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-exec.c| 4 ++-- cpus.c| 13 ++--- include/exec/cpu-defs.h | 20 include/exec/gen-icount.h | 6 -- include/qom/cpu.h | 19 +++ qom/cpu.c | 1 + tr

[Qemu-devel] [PATCH qom-cpu v2 31/40] cpu-exec: Change cpu_resume_from_signal() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-exec.c | 4 +--- exec.c | 2 +- hw/i386/kvmvapic.c | 2 +- include/exec/exec-all.h | 2 +- target-i386/helper.c| 2 +- target-lm32/helper.c| 2 +- target-xtensa/helper.c | 2 +- translate-all.c | 6 +++--- u

[Qemu-devel] [PATCH qom-cpu v2 38/40] cputlb: Change tlb_flush() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cputlb.c | 6 +++--- exec.c | 4 +--- hw/sh4/sh7750.c| 2 +- include/exec/exec-all.h| 4 ++-- target-alpha/cpu.c | 2 +- target-alpha/sys_helper.c

[Qemu-devel] [PATCH qom-cpu v2 12/40] cpu: Move icount_extra field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Reset it. Signed-off-by: Andreas Färber --- cpu-exec.c | 10 +- cpus.c | 14 +++--- include/exec/cpu-defs.h | 1 - include/qom/cpu.h | 2 ++ qom/cpu.c | 1 + 5 files changed, 15 insertions(+), 13 deletions(-) diff --git a/cpu-

[Qemu-devel] [PATCH qom-cpu v2 35/40] target-cris: Replace DisasContext::env field with CRISCPU

2014-03-09 Thread Andreas Färber
This cleans up repeated cris_env_get_cpu() for cpu_abort(). Signed-off-by: Andreas Färber --- target-cris/translate.c | 16 target-cris/translate_v10.c | 16 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/target-cris/translate.c b/target-cr

[Qemu-devel] [PATCH qom-cpu v2 36/40] target-microblaze: Replace DisasContext::env field with MicroBlazeCPU

2014-03-09 Thread Andreas Färber
This cleans up some mb_env_get_cpu() needed for cpu_abort(). Signed-off-by: Andreas Färber --- target-microblaze/translate.c | 87 +-- 1 file changed, 43 insertions(+), 44 deletions(-) diff --git a/target-microblaze/translate.c b/target-microblaze/transla

[Qemu-devel] [PATCH qom-cpu v2 24/40] translate-all: Change tb_check_watchpoint() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- exec.c | 2 +- translate-all.c | 4 ++-- translate-all.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/exec.c b/exec.c index d4ce76a..dfa43a5 100644 --- a/exec.c +++ b/exec.c @@ -1599,7 +1599,7 @@ static void check_watchpoint(int

[Qemu-devel] [PATCH qom-cpu v2 11/40] cpu: Move can_do_io field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Rename can_do_io() to cpu_can_do_io() and change argument to CPUState. Signed-off-by: Andreas Färber --- cpus.c | 2 +- include/exec/cpu-defs.h | 1 - include/exec/exec-all.h | 21 + include/exec/gen-icount.h | 4 ++-- include

[Qemu-devel] [PATCH qom-cpu v2 27/40] translate-all: Change tb_flush_jmp_cache() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cputlb.c | 2 +- include/exec/cputlb.h | 2 +- translate-all.c | 3 +-- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/cputlb.c b/cputlb.c index 0eb1801..6b2cdb2 100644 --- a/cputlb.c +++ b/cputlb.c @@ -106,7 +106,7 @@ void tlb_

[Qemu-devel] [PATCH qom-cpu v2 33/40] exec: Change memory_region_section_get_iotlb() argument to CPUState

2014-03-09 Thread Andreas Färber
It no longer needs CPUArchState since moving watchpoints to CPUState. Signed-off-by: Andreas Färber --- cputlb.c | 2 +- exec.c| 3 +-- include/exec/cputlb.h | 2 +- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/cputlb.c b/cputlb.c index 343ede2..8a0

[Qemu-devel] [PATCH qom-cpu v2 10/40] cpu: Move mem_io_{pc, vaddr} fields from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Reset them. Signed-off-by: Andreas Färber --- exec.c | 9 + hw/i386/kvmvapic.c | 2 +- include/exec/cpu-defs.h | 7 --- include/exec/softmmu_template.h | 8 include/qom/cpu.h | 8 qom/cpu.c

[Qemu-devel] [PATCH qom-cpu v2 32/40] cputlb: Change tlb_unprotect_code_phys() argument to CPUState

2014-03-09 Thread Andreas Färber
Note that the argument is unused. Signed-off-by: Andreas Färber --- cputlb.c | 2 +- include/exec/cputlb.h | 2 +- translate-all.c | 6 +++--- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/cputlb.c b/cputlb.c index 6b2cdb2..343ede2 100644 --- a/cputlb.c +++ b/

[Qemu-devel] [PATCH qom-cpu v2 26/40] translate-all: Change tb_gen_code() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-exec.c | 4 ++-- exec.c | 2 +- hw/i386/kvmvapic.c | 2 +- include/exec/exec-all.h | 2 +- translate-all.c | 9 + 5 files changed, 10 insertions(+), 9 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index

[Qemu-devel] [PATCH qom-cpu v2 20/40] exec: Change tlb_fill() argument to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- include/exec/exec-all.h | 2 +- include/exec/softmmu_template.h | 8 target-alpha/mem_helper.c | 8 +--- target-arm/op_helper.c | 12 +++- target-cris/op_helper.c | 7 --- target-i386/mem_helper.c

[Qemu-devel] [PATCH qom-cpu v2 29/40] exec: Change cpu_watchpoint_{insert, remove{, _by_ref, _all}} argument

2014-03-09 Thread Andreas Färber
Use CPUState. This lets us drop a few local env usages. Signed-off-by: Andreas Färber --- exec.c| 34 +- gdbstub.c | 17 + include/exec/cpu-all.h| 6 -- include/qom/cpu.h | 7 +++ linux-user

[Qemu-devel] [PATCH qom-cpu v2 09/40] cpu: Turn cpu_handle_mmu_fault() into a CPUClass hook

2014-03-09 Thread Andreas Färber
Note that while such functions may exist both for *-user and softmmu, only *-user uses the CPUState hook, while softmmu reuses the prototype for calling it directly. Signed-off-by: Andreas Färber --- include/qom/cpu.h | 3 +++ target-alpha/cpu.c| 4 +++- target-alpha/cp

[Qemu-devel] [PATCH qom-cpu v2 14/40] cpu: Move tb_jmp_cache field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Clear it on reset. Signed-off-by: Andreas Färber --- cpu-exec.c | 6 -- cputlb.c| 2 +- include/exec/cpu-defs.h | 4 include/qom/cpu.h | 4 qom/cpu.c | 1 + translate-all.c | 15 ++- 6 files changed, 16 inser

[Qemu-devel] [PATCH qom-cpu v2 18/40] cpu: Move watchpoint fields from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-exec.c | 5 +++-- exec.c | 33 - gdbstub.c | 8 include/exec/cpu-defs.h | 10 -- include/qom/cpu.h | 10 ++ linux-user/main.c | 5 +++-- target

[Qemu-devel] [PATCH qom-cpu v2 23/40] translate-all: Change cpu_restore_state_from_tb() argument to CPUState

2014-03-09 Thread Andreas Färber
And normalize the argument order. Signed-off-by: Andreas Färber --- translate-all.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/translate-all.c b/translate-all.c index 70d8229..adcef83 100644 --- a/translate-all.c +++ b/translate-all.c @@ -197,10 +197,10 @

[Qemu-devel] [PATCH qom-cpu v2 04/40] target-ppc: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits fdfba1a298ae26dd44bcfdb0429314139a0bc55a, ab1da85791340e504d10487e1add81b9988afa98, f606604f1c10b60ef294f1b9b229426521a365e3 and 2c17449b3022ca9623c4a7e2a504a4150ac4ad30 added usages of ENV_GET_CPU() macro in target-independent code. Use ppc_env_get_cpu() instead. Cc: Edgar E. Iglesias C

[Qemu-devel] [PATCH qom-cpu v2 08/40] target-xtensa: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits a00817cc4c18b7872e92765a4736fb2227cc237b and fdfba1a298ae26dd44bcfdb0429314139a0bc55a added usages of ENV_GET_CPU() macro in target-specific code. Use xtensa_env_get_cpu() instead. Cc: Max Filippov Cc: Edgar E. Iglesias Cc: Peter Maydell Signed-off-by: Andreas Färber --- target-xtens

[Qemu-devel] [PATCH qom-cpu v2 02/40] target-arm: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits ab1da85791340e504d10487e1add81b9988afa98, fdfba1a298ae26dd44bcfdb0429314139a0bc55a, 2c17449b3022ca9623c4a7e2a504a4150ac4ad30 added usages of ENV_GET_CPU() macro to target-specific code. Use arm_env_get_cpu() instead and enforce separating variable declarations. Cc: Edgar E. Iglesias Cc:

[Qemu-devel] [PATCH qom-cpu v2 19/40] cpu: Move breakpoints field from CPU_COMMON to CPUState

2014-03-09 Thread Andreas Färber
Most targets were using offsetof(CPUFooState, breakpoints) to determine how much of CPUFooState to clear on reset. Use the next field after CPU_COMMON instead, if any, or sizeof(CPUFooState) otherwise. Signed-off-by: Andreas Färber --- exec.c| 21 + in

[Qemu-devel] [PATCH qom-cpu v2 06/40] target-sparc: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commits fdfba1a298ae26dd44bcfdb0429314139a0bc55a, 2c17449b3022ca9623c4a7e2a504a4150ac4ad30 and f606604f1c10b60ef294f1b9b229426521a365e3 added usages of ENV_GET_CPU() macro in target-specific code. Use sparc_env_get_cpu() instead and reuse the variables. Cc: Edgar E. Iglesias Cc: Peter Maydell S

[Qemu-devel] [PATCH qom-cpu v2 07/40] target-unicore32: Clean up ENV_GET_CPU() usage

2014-03-09 Thread Andreas Färber
Commit fdfba1a298ae26dd44bcfdb0429314139a0bc55a added a usage of ENV_GET_CPU() macro in target-specific code. Use uc32_env_get_cpu() instead. Cc: Edgar E. Iglesias Cc: Peter Maydell Signed-off-by: Andreas Färber --- target-unicore32/softmmu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [Qemu-devel] Call for testing QEMU aarch64-linux-user emulation

2014-03-09 Thread Peter Maydell
On 9 March 2014 23:37, Dann Frazier wrote: > Also - I've found an issue with running OpenJDK in the latest upstream git: > > root@server-75e0210e-4f99-4c86-9277-3201ab7b6afd:/root# java > # > [thread 274902467056 also had an error]# A fatal error has been > detected by the Java Runtime Environment

[Qemu-devel] [Bug 1289527] Re: qemu-aarch64-static: java dies with SIGILL

2014-03-09 Thread dann frazier
** Also affects: qemu Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1289527 Title: qemu-aarch64-static: java dies with SIGILL Status in QEMU: New Stat

Re: [Qemu-devel] QEMU: SMBIOS: Build full smbios tables

2014-03-09 Thread Gabriel L. Somlo
On Sun, Mar 09, 2014 at 08:18:11PM +0100, Laszlo Ersek wrote: > On 03/09/14 03:57, Gabriel L. Somlo wrote: > > I'd like to get the rest of what SeaBIOS builds by default (types 3+) > > in before the 2.0 freeze (BTW, how much longer do I have for that?), > > The hard freeze is on March 12th.

Re: [Qemu-devel] Call for testing QEMU aarch64-linux-user emulation

2014-03-09 Thread Dann Frazier
On Tue, Feb 25, 2014 at 1:39 AM, Alex Bennée wrote: > > Dann Frazier writes: > >> On Mon, Feb 17, 2014 at 6:40 AM, Alex Bennée wrote: >>> Hi, >> >> Thanks to all involved for your work here! >> >>> After a solid few months of work the QEMU master branch [1] has now reached >>> instruction featur

[Qemu-devel] [Bug 1289898] Re: qemu-system-ppc64 easily cause file corruption

2014-03-09 Thread wzis
I found a CentOS 6.4 machine which has QEMU 1.5.3, and copied the ppcrhel5.img to that machine, and tested, found it's reliable, and no such file and filesystem easy corruption issue. So, it's the 1.6.2 and 1.7.0 have the problem. -- You received this bug notification because you are a member of

[Qemu-devel] [Bug 1289898] Re: qemu-system-ppc64 easily cause file corruption

2014-03-09 Thread wzis
I used the following way to start the VM: #!/bin/bash ifconfig -a|grep tap0 >/dev/null 2>&1 || qemu-ifup tap0 qemu-system-ppc64 -hda ppcrhel5.img -cpu POWER7 -machine type=pseries,usb=off -m 768 -nographic -net nic -net tap,ifname=tap0,script=no qemu-ifdown tap0 -- You received this bug notifi

Re: [Qemu-devel] [PATCH 00/11] tcg-sparc updates

2014-03-09 Thread Mark Cave-Ayland
On 05/03/14 18:11, Richard Henderson wrote: Rescued from a branch from September. I can't recall if I ever actually posted these back then, so I'll wait a week before asking for a pull. Re-tested on v8plus. The machine doesn't have 64-bit libraries installed, so I'm a bit stuck with that. r

Re: [Qemu-devel] [PATCH v5] target-sparc: Add and use CPU_FEATURE_CASA

2014-03-09 Thread Mark Cave-Ayland
On 09/03/14 11:02, Sebastian Huber wrote: The LEON3 processor has support for the CASA instruction which is normally only available for SPARC V9 processors. Binutils 2.24 and GCC 4.9 will support this instruction for LEON3. GCC uses it to generate C11 atomic operations. The CAS synthetic instru

Re: [Qemu-devel] [PATCH RFC 0/2] qemu-arg: general purpose argument parser

2014-03-09 Thread Leandro Dorileo
Hi Andreas, On Sun, Mar 09, 2014 at 05:32:17PM +0100, Andreas Färber wrote: > Am 08.03.2014 19:47, schrieb Leandro Dorileo: > > The following patchset introduces a general purpose argument parser and > > migrates > > qemu-img to make use of it. qemu-img is just the first user of it, if we > > s

[Qemu-devel] [Bug 1289898] Re: qemu-system-ppc64 easily cause file corruption

2014-03-09 Thread Paolo Bonzini
Hi, any chance you could try the latest snapshot from git? It has a lot of PPC64 changes. Also, please pass your QEMU command line. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1289898 Title: qem

[Qemu-devel] [PULL v3 03/14] virtio-net: remove function calls from assert

2014-03-09 Thread Michael S. Tsirkin
From: Joel Stanley peer_{de,at}tach were called from inside assert(). We don't support building without NDEBUG but it's not tidy. Rearrange to attach peer outside assert calls. Signed-off-by: Joel Stanley Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/net/virtio-net

[Qemu-devel] [Bug 1251470] Re: Guest not working in KVM mode but does in TCG mode

2014-03-09 Thread Paolo Bonzini
I'll download FreeBSD 9.2 and debug it. In the meanwhile, does it work with an additional option "-cpu Nehalem"? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1251470 Title: Guest not working in KV

[Qemu-devel] [Bug 1252270] Re: installing NT4 on MIPS Magnum/Jazz asserts

2014-03-09 Thread Paolo Bonzini
** Changed in: qemu Status: New => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1252270 Title: installing NT4 on MIPS Magnum/Jazz asserts Status in QEMU: Fix Committed Bug

[Qemu-devel] [PULL v3 02/14] acpi-test-data: update expected files

2014-03-09 Thread Michael S. Tsirkin
Signed-off-by: Michael S. Tsirkin --- tests/acpi-test-data/pc/DSDT | Bin 4582 -> 4485 bytes tests/acpi-test-data/pc/SSDT | Bin 2200 -> 2275 bytes tests/acpi-test-data/q35/DSDT | Bin 7438 -> 7383 bytes tests/acpi-test-data/q35/SSDT | Bin 475 -> 564 bytes 4 files changed, 0 insertions(+), 0 d

[Qemu-devel] [PULL v3 06/14] Rework --name to use QemuOpts

2014-03-09 Thread Michael S. Tsirkin
From: "Dr. David Alan Gilbert" Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Alex Bennée Acked-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek --- vl.c | 52 +++- 1 file changed, 39 insertions(+), 13 deletions(-) diff --git a/vl.c b/vl.c

[Qemu-devel] [PULL v3 07/14] Add 'debug-threads' suboption to --name

2014-03-09 Thread Michael S. Tsirkin
From: "Dr. David Alan Gilbert" Add flag storage to qemu-thread-* to store the namethreads flag Signed-off-by: Dr. David Alan Gilbert Acked-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek --- include/qemu/thread.h| 1 + util/qemu-thread-posix.c | 7 +++ util/qemu-thread-win32.c | 8 +

[Qemu-devel] [PULL v3 13/14] pckbd: return 'keyboard enabled' on read input port command

2014-03-09 Thread Michael S. Tsirkin
From: Hervé Poussineau Bit 7 of Input Port is the keyboard inhibit switch. 0 means keyboard inhibited, while 1 means keyboard enabled. Incidentaly, this also fixes an error encountered while booting an Award BIOS: "Keyboard is locked out - Unlock the key". Signed-off-by: Hervé Poussineau Revie

[Qemu-devel] [PULL v3 09/14] MAINTAINERS: drop an out of date address

2014-03-09 Thread Michael S. Tsirkin
Gleb's address seems to be out of date. Since it stayed like that for a while now, I'm guessing he's no longer interested in getting mail. Signed-off-by: Michael S. Tsirkin --- MAINTAINERS | 2 -- 1 file changed, 2 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 7efaccf..62e7683 1006

[Qemu-devel] [PULL v3 12/14] pam: partly fix write-only mode

2014-03-09 Thread Michael S. Tsirkin
From: Hervé Poussineau In write-only mode, writes are forwarded to RAM, while reads should not be handled (ie should return 0xff). Assume that in this mode, no read access is ever done, as they shouldn't give any sensible result. So, in write-only mode, alias PAM region to RAM, instead of PCI me

[Qemu-devel] [PULL v3 14/14] qemu: x86: ignore ioapic polarity

2014-03-09 Thread Michael S. Tsirkin
From: "Gabriel L. Somlo" Both QEMU and KVM have already accumulated a significant number of optimizations based on the hard-coded assumption that ioapic polarity will always use the ActiveHigh convention, where the logical and physical states of level-triggered irq lines always match (i.e., activ

[Qemu-devel] [PULL v3 10/14] acpi-test: retain both asl and aml files on failure

2014-03-09 Thread Michael S. Tsirkin
From: Marcel Apfelbaum Updated the error message while at it. Signed-off-by: Marcel Apfelbaum Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/acpi-test.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/tests/acpi-test.c b/tests

[Qemu-devel] [PULL v3 11/14] acpi-test: issue errors instead of warnings when possible

2014-03-09 Thread Michael S. Tsirkin
From: Marcel Apfelbaum If the expected (offline) acpi tables loaded correctly, it is safe to assume the iasl installation is OK and issue an error if the actual tables failed to load. Signed-off-by: Marcel Apfelbaum Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/

[Qemu-devel] [PULL v3 05/14] PCIE: fix regression with coldplugged multifunction device

2014-03-09 Thread Michael S. Tsirkin
From: Igor Mammedov PCIE is causing asserts each time a multifunction device is added on command line (coldplug). This is caused by commit a66e657e18cd9b70e9f57ae5512c07faf2bc508f pci/pcie: convert PCIE hotplug to use hotplug-handler API QEMU abort is caused by misplaced assertion, which sho

[Qemu-devel] [PULL v3 04/14] memory_region_present: return false if address is not found in child MemoryRegion

2014-03-09 Thread Michael S. Tsirkin
From: Igor Mammedov Windows XP shows COM2 port as non functional in "Device Manager" although no COM2 port backing device is present in QEMU. This regression is really due to 3bb28b7208b349e7a1b326e3c6ef9efac1d462bf? memory: Provide separate handling of unassigned io ports accesses That is

[Qemu-devel] [PULL v3 08/14] Add a 'name' parameter to qemu_thread_create

2014-03-09 Thread Michael S. Tsirkin
From: "Dr. David Alan Gilbert" If enabled, set the thread name at creation (on GNU systems with pthread_set_np) Fix up all the callers with a thread name Signed-off-by: Dr. David Alan Gilbert Acked-by: Michael S. Tsirkin Reviewed-by: Laszlo Ersek --- include/qemu/thread.h | 2 +-

[Qemu-devel] [PULL v3 01/14] acpi-build: append description for non-hotplug

2014-03-09 Thread Michael S. Tsirkin
As reported in http://article.gmane.org/gmane.comp.emulators.qemu/253987 Mac OSX actually requires describing all occupied slots in ACPI - even if hotplug isn't enabled. I didn't expect this so I dropped description of all non hotpluggable slots from ACPI. As a result: before commit 99fd437dee4686

[Qemu-devel] [PULL v3 00/14] acpi,pc,pci,virtio,memory bug fixes

2014-03-09 Thread Michael S. Tsirkin
Changes from v2: fix for OSx by Gabriel fixed build on systems without IASL The following changes since commit f53f3d0a00b6df39ce8dfca942608e5b6a9a4f71: Merge remote-tracking branch 'remotes/kvaneesh/for-upstream' into staging (2014-03-08 12:38:43 +) are available in the git repos

Re: [Qemu-devel] QEMU: SMBIOS: Build full smbios tables

2014-03-09 Thread Laszlo Ersek
On 03/09/14 03:57, Gabriel L. Somlo wrote: > I'll send the current set of seven patches, which basically replace > the way types 0 and 1 are handled with building full tables, and adds > the option to request that a type 2 table be built. > > I'd like to get the rest of what SeaBIOS builds by def

Re: [Qemu-devel] [Qemu-ppc] [PATCH v2 2/2] Fix return value of vga initlization on ppc

2014-03-09 Thread Paolo Bonzini
Il 08/03/2014 14:26, Alexey Kardashevskiy ha scritto: > I think that after this patch, "-nodefaults -device VGA" will get a USB > controller that it didn't get before. I suspect what was meant by "the machine not aware of the graphics device" is that the guest won't work with VGA and without k

[Qemu-devel] [PATCH v3 2/3] pc: avoid duplicate names for ROM MRs

2014-03-09 Thread Michael S. Tsirkin
Since commit 04920fc0faa4760f9c4fc0e73b992b768099be70 loader: store FW CFG ROM files in RAM RAM MRs including ROM files in FW CFGs are created and named using the file basename. This becomes problematic if these names are supplied by user, since the basename might not be unique. There are two

[Qemu-devel] [PATCH v3 3/3] pc: option rom migration compatibility with 1.7

2014-03-09 Thread Michael S. Tsirkin
when using 1.7 machine types, enable option ROMs in RAM to match that version. Signed-off-by: Michael S. Tsirkin --- hw/i386/pc_piix.c | 1 + hw/i386/pc_q35.c | 1 + 2 files changed, 2 insertions(+) diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c index fb2d636..5e1d2d3 100644 --- a/hw/i386/

[Qemu-devel] [PATCH v3 1/3] loader: rename in_ram/has_mr

2014-03-09 Thread Michael S. Tsirkin
we put copy of ROMs in MR for migration. but the name rom_in_ram makes one think we load it in guest RAM. Rename has_mr to make intent clearer. Signed-off-by: Michael S. Tsirkin --- include/hw/loader.h | 2 +- hw/core/loader.c| 6 +++--- hw/i386/pc_piix.c | 2 +- hw/i386/pc_q35.c| 2 +-

Re: [Qemu-devel] [PATCH] qdev: Fix bus dependency of DeviceState::hotpluggable getter

2014-03-09 Thread Michael S. Tsirkin
On Sun, Mar 09, 2014 at 05:48:57PM +0100, Andreas Färber wrote: > Am 09.03.2014 17:40, schrieb Michael S. Tsirkin: > > On Fri, Mar 07, 2014 at 07:13:18PM +0100, Andreas Färber wrote: > >> Commit 1a37eca107cece3ed454bae29eef0bd1fac4a244 (qdev: add > >> "hotpluggable" property to Device) added a prop

Re: [Qemu-devel] [PATCH] New feature - RFC3931 L2TPv3 network transport using static Ethernet over L2TPv3 tunnels

2014-03-09 Thread Anton Ivanov
+ return -1; +} + +freeaddrinfo(result); + +memset(&hints, 0, sizeof(hints)); + +if (s->ipv6) { + hints.ai_family = AF_INET6; +} else { + hints.ai_family = AF_INET; +} +if (s->udp) { + hints.ai_socktype = SOCK_DGRAM; + hints.ai_protocol

Re: [Qemu-devel] [PATCH] qdev: Fix bus dependency of DeviceState::hotpluggable getter

2014-03-09 Thread Andreas Färber
Am 09.03.2014 17:40, schrieb Michael S. Tsirkin: > On Fri, Mar 07, 2014 at 07:13:18PM +0100, Andreas Färber wrote: >> Commit 1a37eca107cece3ed454bae29eef0bd1fac4a244 (qdev: add >> "hotpluggable" property to Device) added a property "hotpluggable" to >> each device, with its getter accessing parent_

Re: [Qemu-devel] [PATCH] qdev: Fix bus dependency of DeviceState::hotpluggable getter

2014-03-09 Thread Michael S. Tsirkin
On Fri, Mar 07, 2014 at 07:13:18PM +0100, Andreas Färber wrote: > Commit 1a37eca107cece3ed454bae29eef0bd1fac4a244 (qdev: add > "hotpluggable" property to Device) added a property "hotpluggable" to > each device, with its getter accessing parent_bus->allow_hotplug. > > Add a NULL check. > > Cc: Ig

Re: [Qemu-devel] [PATCH RFC 0/2] qemu-arg: general purpose argument parser

2014-03-09 Thread Andreas Färber
Am 08.03.2014 19:47, schrieb Leandro Dorileo: > The following patchset introduces a general purpose argument parser and > migrates > qemu-img to make use of it. qemu-img is just the first user of it, if we see a > good feedback here I move forward and migrate all the other possible users. Why? :)

Re: [Qemu-devel] [PATCH qom-cpu 0/6] cpu: Unifying features parsing

2014-03-09 Thread Andreas Färber
Am 08.03.2014 21:50, schrieb Mark Cave-Ayland: > On 04/03/14 20:32, Andreas Färber wrote: > >> Am 04.03.2014 03:55, schrieb Andreas Färber: >>> Hello, >>> >>> Prompted by Alexey's desire for tweakable PowerPCCPU properties but >>> also by >>> Peter's wish for ARMCPU properties, this series sets ou

Re: [Qemu-devel] [PATCH qom-cpu 0/6] cpu: Unifying features parsing

2014-03-09 Thread Andreas Färber
Am 05.03.2014 12:27, schrieb Alexey Kardashevskiy: > On 03/05/2014 07:30 PM, Andreas Färber wrote: >> Am 05.03.2014 03:50, schrieb Alexey Kardashevskiy: >>> On 03/04/2014 01:55 PM, Andreas Färber wrote: Hello, Prompted by Alexey's desire for tweakable PowerPCCPU properties but also b

Re: [Qemu-devel] [PATCH qom-cpu 1/6] cpu: Introduce CPUClass::parse_features() hook

2014-03-09 Thread Andreas Färber
Am 05.03.2014 23:31, schrieb Eduardo Habkost: > On Wed, Mar 05, 2014 at 05:57:10PM +0100, Igor Mammedov wrote: >> On Wed, 05 Mar 2014 17:06:15 +0100 >> Andreas Färber wrote: >> >>> Am 05.03.2014 16:04, schrieb Igor Mammedov: On Tue, 4 Mar 2014 03:55:44 +0100 Andreas Färber wrote:

Re: [Qemu-devel] [PATCH] configure: don't modify .status on error

2014-03-09 Thread Peter Maydell
On 9 March 2014 15:41, Michael S. Tsirkin wrote: > ./configure --help > make > > will try to re-run configure with --help > which isn't what was intended. > > The reason is that config.status was written > even on configure error. > Defer writing config.status until configure > has completed succe

Re: [Qemu-devel] [PATCH qom-cpu v10 2/2] target-i386: X86CPU model subclasses

2014-03-09 Thread Andreas Färber
Am 04.03.2014 20:53, schrieb Eduardo Habkost: > On Tue, Feb 25, 2014 at 08:36:27AM +0100, Andreas Färber wrote: >> From: Eduardo Habkost >> >> Register separate QOM types for each x86 CPU model. >> >> This will allow management code to more easily probe what each CPU model >> provides, by simply c

[Qemu-devel] [PATCH 04/16] target-arm: A64: Add FSQRT to C3.6.17 (two misc)

2014-03-09 Thread Peter Maydell
From: Alex Bennée Implement FSQRT in the two-reg-misc category. GCC uses this instruction form. Signed-off-by: Alex Bennée Signed-off-by: Peter Maydell --- target-arm/translate-a64.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/target-arm/translate-a64.c b

Re: [Qemu-devel] [PATCH qom-cpu 1/6] cpu: Introduce CPUClass::parse_features() hook

2014-03-09 Thread Andreas Färber
Am 04.03.2014 03:55, schrieb Andreas Färber: > Adapt the X86CPU implementation to suit the generic hook. > This involves a cleanup of error handling to cope with NULL errp. > > Signed-off-by: Andreas Färber > --- > include/qom/cpu.h | 3 +++ > target-i386/cpu.c | 36 +---

[Qemu-devel] trying to kill a VM

2014-03-09 Thread Alexander Binun
Hello Friends, I have done a lot of tricks trying to kill a VM - so far in vain :-) As you told me I have reached the VCPU of the VM to be killed: struct kvm_vcpu *cpu = (struct kvm_vcpu*)vcpu; struct pid *vcpu_pid = cpu->pid; struct task_struct* task = pid_task(vcpu_pid,PIDTYPE_PID); Then I

[Qemu-devel] [PATCH] configure: don't modify .status on error

2014-03-09 Thread Michael S. Tsirkin
./configure --help make will try to re-run configure with --help which isn't what was intended. The reason is that config.status was written even on configure error. Defer writing config.status until configure has completed successfully. Signed-off-by: Michael S. Tsirkin --- configure | 27 +++

[Qemu-devel] [PATCH 15/16] target-arm: A64: Implement FRINT*

2014-03-09 Thread Peter Maydell
Implement the FRINT* round-to-integral operations from the 2-reg-misc category. Signed-off-by: Peter Maydell --- target-arm/translate-a64.c | 37 ++--- 1 file changed, 34 insertions(+), 3 deletions(-) diff --git a/target-arm/translate-a64.c b/target-arm/translate

[Qemu-devel] [PATCH 14/16] target-arm: A64: Implement SRI

2014-03-09 Thread Peter Maydell
Implement SRI (shift right and insert). Signed-off-by: Peter Maydell --- target-arm/translate-a64.c | 57 +++--- 1 file changed, 49 insertions(+), 8 deletions(-) diff --git a/target-arm/translate-a64.c b/target-arm/translate-a64.c index 5f560eb..6337d6f 1

[Qemu-devel] [PATCH 05/16] target-arm: A64: Add remaining CLS/Z vector ops

2014-03-09 Thread Peter Maydell
From: Alex Bennée Implement the CLS, CLZ operations in the 2-reg-misc category. Signed-off-by: Alex Bennée Signed-off-by: Peter Maydell --- target-arm/helper-a64.c| 5 + target-arm/helper-a64.h| 1 + target-arm/translate-a64.c | 36 +++- 3 files c

[Qemu-devel] [PATCH 06/16] target-arm: A64: Saturating and narrowing shift ops

2014-03-09 Thread Peter Maydell
From: Alex Bennée This implements a the remaining [US][R]SHR[R][2] opcodes which are saturating and narrowing shift right operations. These are used in things like libav. Note signed shifts can have an "unsigned" saturating narrow operation which will floor negative values. Signed-off-by: Alex B

[Qemu-devel] [PATCH 16/16] exec-all.h: Increase MAX_OP_PER_INSTR for ARM A64 decoder

2014-03-09 Thread Peter Maydell
The ARM A64 decoder's worst case number of TCG ops per instruction is 266 (for insn 0x4c80, a post-indexed ST4 multiple-structures store). Raise the MAX_OP_PER_INSTR define accordingly. Signed-off-by: Peter Maydell --- include/exec/exec-all.h| 2 +- target-arm/translate-a64.c | 12 +

[Qemu-devel] [PATCH 13/16] target-arm: A64: Add FRECPX (reciprocal exponent)

2014-03-09 Thread Peter Maydell
From: Alex Bennée These are fairly simple exponent only estimation functions using helpers. Signed-off-by: Alex Bennée Signed-off-by: Peter Maydell --- target-arm/helper-a64.c| 59 ++ target-arm/helper-a64.h| 2 ++ target-arm/translate-a64.c | 70 +

[Qemu-devel] [PATCH 09/16] target-arm: A64: Implement FCVT[NMAPZ][SU] SIMD instructions

2014-03-09 Thread Peter Maydell
Implement the floating-point-to-integer conversion instructions FCVT[NMAPZ][SU] in the 2-reg-misc and scalar-2-reg-misc categories. Signed-off-by: Peter Maydell --- target-arm/translate-a64.c | 188 - 1 file changed, 169 insertions(+), 19 deletions(-)

[Qemu-devel] [PATCH 03/16] target-arm: A64: Add last AdvSIMD Integer to FP ops

2014-03-09 Thread Peter Maydell
From: Alex Bennée This adds the remaining [US]CVTF operations to the SIMD shift-immediate, scalar-shift-immediate, two-reg-misc and scalar-two-reg-misc groups of opcodes. Signed-off-by: Alex Bennée [PMM: added scalar 2-misc and scalar-shift-imm encodings] Signed-off-by: Peter Maydell --- targ

[Qemu-devel] [PATCH 00/16] A64 Neon patches: sixth set

2014-03-09 Thread Peter Maydell
This is the sixth set of patches implementing more of the Neon instructions for A64 (assuming I haven't lost count...) PMULL is a fixed version of the patch from last time round which doesn't pointlessly pass cpu_env to the helper; the rest are new. Remaining unimplemented usermode instructions:

[Qemu-devel] [PATCH 01/16] target-arm: A64: Implement PMULL instruction

2014-03-09 Thread Peter Maydell
Implement the PMULL instruction; this is the last unimplemented insn in the three-reg-diff group. Note that PMULL with size 3 is considered part of the AES part of the crypto extensions (see the ID_AA64ISAR0_EL1 register definition in the v8 ARM ARM), so it isn't necessary to burn an extra feature

  1   2   >