qemu.git
7 years agocputlb: Change tlb_flush() argument to CPUState
Andreas Färber [Wed, 4 Sep 2013 00:19:44 +0000 (02:19 +0200)] 
cputlb: Change tlb_flush() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocputlb: Change tlb_flush_page() argument to CPUState
Andreas Färber [Tue, 3 Sep 2013 23:29:02 +0000 (01:29 +0200)] 
cputlb: Change tlb_flush_page() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-microblaze: Replace DisasContext::env field with MicroBlazeCPU
Andreas Färber [Tue, 3 Sep 2013 18:02:48 +0000 (20:02 +0200)] 
target-microblaze: Replace DisasContext::env field with MicroBlazeCPU

This cleans up some mb_env_get_cpu() needed for cpu_abort().

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-cris: Replace DisasContext::env field with CRISCPU
Andreas Färber [Tue, 3 Sep 2013 16:42:27 +0000 (18:42 +0200)] 
target-cris: Replace DisasContext::env field with CRISCPU

This cleans up repeated cris_env_get_cpu() for cpu_abort().

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoexec: Change cpu_abort() argument to CPUState
Andreas Färber [Tue, 3 Sep 2013 15:38:47 +0000 (17:38 +0200)] 
exec: Change cpu_abort() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoexec: Change memory_region_section_get_iotlb() argument to CPUState
Andreas Färber [Tue, 3 Sep 2013 11:32:01 +0000 (13:32 +0200)] 
exec: Change memory_region_section_get_iotlb() argument to CPUState

It no longer needs CPUArchState since moving watchpoints to CPUState.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocputlb: Change tlb_unprotect_code_phys() argument to CPUState
Andreas Färber [Tue, 3 Sep 2013 08:51:26 +0000 (10:51 +0200)] 
cputlb: Change tlb_unprotect_code_phys() argument to CPUState

Note that the argument is unused.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu-exec: Change cpu_resume_from_signal() argument to CPUState
Andreas Färber [Tue, 3 Sep 2013 00:12:23 +0000 (02:12 +0200)] 
cpu-exec: Change cpu_resume_from_signal() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoexec: Change cpu_breakpoint_{insert,remove{,_by_ref,_all}} argument
Andreas Färber [Mon, 2 Sep 2013 15:26:20 +0000 (17:26 +0200)] 
exec: Change cpu_breakpoint_{insert,remove{,_by_ref,_all}} argument

Use CPUState. Allows to clean up CPUArchState in gdbstub.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoexec: Change cpu_watchpoint_{insert,remove{,_by_ref,_all}} argument
Andreas Färber [Mon, 2 Sep 2013 14:57:02 +0000 (16:57 +0200)] 
exec: Change cpu_watchpoint_{insert,remove{,_by_ref,_all}} argument

Use CPUState. This lets us drop a few local env usages.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-ppc: Use PowerPCCPU in PowerPCCPUClass::handle_mmu_fault hook
Andreas Färber [Mon, 2 Sep 2013 12:14:24 +0000 (14:14 +0200)] 
target-ppc: Use PowerPCCPU in PowerPCCPUClass::handle_mmu_fault hook

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotranslate-all: Change tb_flush_jmp_cache() argument to CPUState
Andreas Färber [Sun, 1 Sep 2013 15:52:07 +0000 (17:52 +0200)] 
translate-all: Change tb_flush_jmp_cache() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotranslate-all: Change tb_gen_code() argument to CPUState
Andreas Färber [Sun, 1 Sep 2013 15:43:17 +0000 (17:43 +0200)] 
translate-all: Change tb_gen_code() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotranslate-all: Change cpu_io_recompile() argument to CPUState
Andreas Färber [Sun, 1 Sep 2013 15:21:47 +0000 (17:21 +0200)] 
translate-all: Change cpu_io_recompile() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotranslate-all: Change tb_check_watchpoint() argument to CPUState
Andreas Färber [Sun, 1 Sep 2013 15:12:23 +0000 (17:12 +0200)] 
translate-all: Change tb_check_watchpoint() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotranslate-all: Change cpu_restore_state_from_tb() argument to CPUState
Andreas Färber [Sun, 1 Sep 2013 15:02:58 +0000 (17:02 +0200)] 
translate-all: Change cpu_restore_state_from_tb() argument to CPUState

And normalize the argument order.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotranslate-all: Change cpu_restore_state() argument to CPUState
Andreas Färber [Sun, 1 Sep 2013 14:51:34 +0000 (16:51 +0200)] 
translate-all: Change cpu_restore_state() argument to CPUState

This lets us drop some local variables in tlb_fill() functions.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu-exec: Change cpu_loop_exit() argument to CPUState
Andreas Färber [Tue, 27 Aug 2013 15:52:12 +0000 (17:52 +0200)] 
cpu-exec: Change cpu_loop_exit() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoexec: Change tlb_fill() argument to CPUState
Andreas Färber [Mon, 26 Aug 2013 22:28:06 +0000 (00:28 +0200)] 
exec: Change tlb_fill() argument to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move breakpoints field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 19:22:53 +0000 (21:22 +0200)] 
cpu: Move breakpoints field from CPU_COMMON to CPUState

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 <afaerber@suse.de>
7 years agocpu: Move watchpoint fields from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 16:23:18 +0000 (18:23 +0200)] 
cpu: Move watchpoint fields from CPU_COMMON to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move opaque field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 16:14:44 +0000 (18:14 +0200)] 
cpu: Move opaque field from CPU_COMMON to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move exception_index field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 06:31:06 +0000 (08:31 +0200)] 
cpu: Move exception_index field from CPU_COMMON to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move jmp_env field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 04:22:03 +0000 (06:22 +0200)] 
cpu: Move jmp_env field from CPU_COMMON to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move tb_jmp_cache field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 04:03:38 +0000 (06:03 +0200)] 
cpu: Move tb_jmp_cache field from CPU_COMMON to CPUState

Clear it on reset.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move icount_decr field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 03:51:49 +0000 (05:51 +0200)] 
cpu: Move icount_decr field from CPU_COMMON to CPUState

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move icount_extra field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 03:39:29 +0000 (05:39 +0200)] 
cpu: Move icount_extra field from CPU_COMMON to CPUState

Reset it.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move can_do_io field from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 03:15:23 +0000 (05:15 +0200)] 
cpu: Move can_do_io field from CPU_COMMON to CPUState

Rename can_do_io() to cpu_can_do_io() and change argument to CPUState.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Move mem_io_{pc,vaddr} fields from CPU_COMMON to CPUState
Andreas Färber [Mon, 26 Aug 2013 01:41:01 +0000 (03:41 +0200)] 
cpu: Move mem_io_{pc,vaddr} fields from CPU_COMMON to CPUState

Reset them.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Turn cpu_handle_mmu_fault() into a CPUClass hook
Andreas Färber [Mon, 26 Aug 2013 01:01:33 +0000 (03:01 +0200)] 
cpu: Turn cpu_handle_mmu_fault() into a CPUClass hook

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 <afaerber@suse.de>
7 years agotarget-m68k: Remove custom qemu_assert() function
Peter Maydell [Wed, 12 Mar 2014 13:24:49 +0000 (13:24 +0000)] 
target-m68k: Remove custom qemu_assert() function

Remove the custom qemu_assert() function defined by target-m68k/translate.c
in favour of either using glib g_assert_not_reached() (for the genuinely
can't-happen cases) or cpu_abort() (for the "this isn't implemented",
in line with other unimplemented cases in the target).

This has the benefit of silencing some clang warnings about
variables used while uninitialized (which are emitted because
clang can't figure out that qemu_assert(0, something) never
returns.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Factor out cpu_generic_init()
Andreas Färber [Tue, 4 Mar 2014 02:17:10 +0000 (03:17 +0100)] 
cpu: Factor out cpu_generic_init()

All targets using it gain the ability to set -cpu name,key=value,...
options via the default TYPE_CPU CPUClass::parse_features() implementation.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Implement CPUClass::parse_features() for the rest of CPUs
Andreas Färber [Mon, 3 Mar 2014 22:33:51 +0000 (23:33 +0100)] 
cpu: Implement CPUClass::parse_features() for the rest of CPUs

CPUs who do not provide their own implementation of feature parsing
will treat each option as a QOM property and set it to the supplied
value.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-sparc: Defer SPARCCPU feature inference to QOM realize
Andreas Färber [Tue, 4 Mar 2014 00:26:33 +0000 (01:26 +0100)] 
target-sparc: Defer SPARCCPU feature inference to QOM realize

Gets it out of cpu_sparc_register() and aligns with target-arm.

Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-sparc: Implement CPUClass::parse_features() for SPARCCPU
Andreas Färber [Mon, 3 Mar 2014 23:38:58 +0000 (00:38 +0100)] 
target-sparc: Implement CPUClass::parse_features() for SPARCCPU

Factor cpu_model parsing out of cpu_sparc_find_by_name() by passing
cpu_sparc_find_by_name() the name portion only and calling
CPUClass::parse_features() from cpu_sparc_register() afterwards.

Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-sparc: Use error_report() for CPU error reporting
Andreas Färber [Mon, 3 Mar 2014 22:47:54 +0000 (23:47 +0100)] 
target-sparc: Use error_report() for CPU error reporting

Replace non-debug fprintf() with error_report().

Tested-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Introduce CPUClass::parse_features() hook
Andreas Färber [Mon, 3 Mar 2014 22:19:19 +0000 (23:19 +0100)] 
cpu: Introduce CPUClass::parse_features() hook

Adapt the X86CPU implementation to suit the generic hook.
This involves a cleanup of error handling to cope with NULL errp.

Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: X86CPU model subclasses
Eduardo Habkost [Mon, 10 Feb 2014 10:21:30 +0000 (08:21 -0200)] 
target-i386: X86CPU model subclasses

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 creating objects using the appropriate class name,
without having to restart QEMU.

This also allows us to eliminate the qdev_prop_set_globals_for_type()
hack to set CPU-model-specific global properties.

Instead of creating separate class_init functions for each class, I just
used class_data to store a pointer to the X86CPUDefinition struct for
each CPU model. This should make the patch shorter and easier to review.
Later we can gradually convert each X86CPUDefinition field to lists of
per-class property defaults.

The "host" CPU model is special, as the feature flags depend on KVM
being initialized. So it has its own class_init and instance_init
function, and feature flags are set on instance_init instead of
class_init.

Signed-off-by: Andreas Färber <afaerber@suse.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Tested-by: Eduardo Habkost <ehabkost@redhat.com>
[AF: Limit the host CPU type to CONFIG_KVM as build fix]
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Prepare CPUClass::class_by_name for X86CPU
Andreas Färber [Mon, 10 Feb 2014 21:02:44 +0000 (22:02 +0100)] 
target-i386: Prepare CPUClass::class_by_name for X86CPU

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Tested-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Enable x2apic by default on KVM
Eduardo Habkost [Wed, 19 Feb 2014 14:58:12 +0000 (11:58 -0300)] 
target-i386: Enable x2apic by default on KVM

When on KVM mode, enable x2apic by default on all CPU models.

Normally we try to keep the CPU model definitions as close as the real
CPUs as possible, but x2apic can be emulated by KVM without host CPU
support for x2apic, and it improves performance by reducing APIC access
overhead. x2apic emulation is available on KVM since 2009 (Linux
2.6.32-rc1), there's no reason for not enabling x2apic by default when
running KVM.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Introduce x86_cpu_compat_disable_kvm_features()
Eduardo Habkost [Wed, 19 Feb 2014 14:58:11 +0000 (11:58 -0300)] 
target-i386: Introduce x86_cpu_compat_disable_kvm_features()

Instead of the feature-specific disable_kvm_pv_eoi() function, create a
more general function that can be used to disable other feature bits in
machine-type compat code.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Make kvm_default_features an array
Eduardo Habkost [Wed, 19 Feb 2014 14:58:10 +0000 (11:58 -0300)] 
target-i386: Make kvm_default_features an array

We will later make the KVM-specific code affect other feature words,
too.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Don't declare variables in the middle of blocks
Eduardo Habkost [Wed, 19 Feb 2014 19:39:21 +0000 (16:39 -0300)] 
target-i386: Don't declare variables in the middle of blocks

Some of my recent changes introduced variable declarations in the middle
of code blocks.

Fix the code so that it compiles without warnings when using
-Wdeclaration-after-statement.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Rename x86_def_t to X86CPUDefinition
Eduardo Habkost [Thu, 30 Jan 2014 19:48:58 +0000 (17:48 -0200)] 
target-i386: Rename x86_def_t to X86CPUDefinition

As the new X86CPU subclass code is going to change lots of the code
invoving x86_def_t, let's rename the struct to match coding style first.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Call x86_cpu_load_def() earlier
Eduardo Habkost [Thu, 30 Jan 2014 19:48:57 +0000 (17:48 -0200)] 
target-i386: Call x86_cpu_load_def() earlier

As we will initialize the X86CPU fields on instance_init eventually,
move the code that initializes the X86CPU data based on the CPU model
name closer to the object_new() call.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Rename cpu_x86_register() to x86_cpu_load_def()
Eduardo Habkost [Thu, 30 Jan 2014 19:48:56 +0000 (17:48 -0200)] 
target-i386: Rename cpu_x86_register() to x86_cpu_load_def()

There isn't any kind of "registration" involved in cpu_x86_register()
anymore: it is simply looking up a CPU model name and loading the model
definition data into the X86CPU object. Rename it to x86_cpu_load_def()
to reflect what it does.

Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Turn cpu_has_work() into a CPUClass hook
Andreas Färber [Sun, 25 Aug 2013 16:53:55 +0000 (18:53 +0200)] 
cpu: Turn cpu_has_work() into a CPUClass hook

Default to false.

Tidy variable naming and inline cast uses while at it.

Tested-by: Jia Liu <proljc@gmail.com> (or32)
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-xtensa: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 19:02:29 +0000 (20:02 +0100)] 
target-xtensa: Clean up ENV_GET_CPU() usage

Commits a00817cc4c18b7872e92765a4736fb2227cc237b and
fdfba1a298ae26dd44bcfdb0429314139a0bc55a added usages of ENV_GET_CPU()
macro in target-specific code.

Use xtensa_env_get_cpu() instead.

Cc: Max Filippov <jcmvbkbc@gmail.com>
Cc: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-unicore32: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 18:58:13 +0000 (19:58 +0100)] 
target-unicore32: Clean up ENV_GET_CPU() usage

Commit fdfba1a298ae26dd44bcfdb0429314139a0bc55a added a usage of
ENV_GET_CPU() macro in target-specific code.

Use uc32_env_get_cpu() instead.

Cc: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-sparc: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 18:51:00 +0000 (19:51 +0100)] 
target-sparc: Clean up ENV_GET_CPU() usage

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 <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-s390x: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 18:40:08 +0000 (19:40 +0100)] 
target-s390x: Clean up ENV_GET_CPU() usage

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 <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-ppc: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 18:29:41 +0000 (19:29 +0100)] 
target-ppc: Clean up ENV_GET_CPU() usage

Commits fdfba1a298ae26dd44bcfdb0429314139a0bc55a,
ab1da85791340e504d10487e1add81b9988afa98,
f606604f1c10b60ef294f1b9b229426521a365e3 and
2c17449b3022ca9623c4a7e2a504a4150ac4ad30 added usages of ENV_GET_CPU()
macro in target-specific code.

Use ppc_env_get_cpu() instead.

Cc: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-i386: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 18:15:27 +0000 (19:15 +0100)] 
target-i386: Clean up ENV_GET_CPU() usage

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 <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-arm: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 18:10:29 +0000 (19:10 +0100)] 
target-arm: Clean up ENV_GET_CPU() usage

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 <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotarget-alpha: Clean up ENV_GET_CPU() usage
Andreas Färber [Sun, 9 Mar 2014 17:58:57 +0000 (18:58 +0100)] 
target-alpha: Clean up ENV_GET_CPU() usage

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 <edgar.iglesias@xilinx.com>
Cc: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agocpu: Don't clear cpu->exit_request on reset
Edgar E. Iglesias [Thu, 13 Feb 2014 05:07:14 +0000 (15:07 +1000)] 
cpu: Don't clear cpu->exit_request on reset

cpu->exit_request is part of the execution environment and should
not be cleared when a CPU resets.

Otherwise, we might deadlock QEMU if a CPU resets while there is
I/O going on.

Signed-off-by: Edgar E. Iglesias <edgar.iglesias@xilinx.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoMerge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging
Peter Maydell [Thu, 13 Mar 2014 15:33:04 +0000 (15:33 +0000)] 
Merge remote-tracking branch 'remotes/stefanha/tags/block-pull-request' into staging

Block pull request

# gpg: Signature made Thu 13 Mar 2014 13:50:49 GMT using RSA key ID 81AB73C8
# gpg: Good signature from "Stefan Hajnoczi <stefanha@redhat.com>"
# gpg:                 aka "Stefan Hajnoczi <stefanha@gmail.com>"
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: 8695 A8BF D3F9 7CDA AC35  775A 9CA4 ABB3 81AB 73C8

* remotes/stefanha/tags/block-pull-request: (24 commits)
  block/raw-win32: bdrv_parse_filename() for hdev
  block/raw-posix: Strip protocol prefix on creation
  block/raw-posix: bdrv_parse_filename() for cdrom
  block/raw-posix: bdrv_parse_filename() for floppy
  block/raw-posix: bdrv_parse_filename() for hdev
  qemu-io: Fix warnings from static code analysis
  block: Unlink temporary file
  qcow2: Don't write with BDRV_O_INCOMING
  qcow2: Keep option in qcow2_invalidate_cache()
  qmp: add query-iothreads command
  iothread: stash thread ID away
  dataplane: replace internal thread with IOThread
  iothread: add "iothread" qdev property type
  qdev: make get_pointer() handle temporary strings
  iothread: add I/O thread object
  aio: add aio_context_acquire() and aio_context_release()
  rfifolock: add recursive FIFO lock
  object: add object_get_canonical_path_component()
  block: Rewrite the snapshot authorization mechanism for block filters.
  iotests: Test corruption during COW request
  ...

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
7 years agooslib-posix: Fix build on FreeBSD
Andreas Färber [Thu, 13 Mar 2014 13:27:59 +0000 (14:27 +0100)] 
oslib-posix: Fix build on FreeBSD

Commit 10f5bff622cad71645e22c027b77ac31e51008ef (util: Split out
exec_dir from os_find_datadir) moved code from os-posix.c to
util/oslib-posix.c but forgot to move a FreeBSD #include alongside,
needed for CTL_KERN among others.

Cc: Fam Zheng <famz@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <andreas.faerber@web.de>
Message-id: 1394717279-23406-1-git-send-email-andreas.faerber@web.de
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
7 years agoblock/raw-win32: bdrv_parse_filename() for hdev
Max Reitz [Fri, 7 Mar 2014 23:39:45 +0000 (00:39 +0100)] 
block/raw-win32: bdrv_parse_filename() for hdev

The "host_device" protocol driver should strip the "host_device:" prefix
from filenames if present.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock/raw-posix: Strip protocol prefix on creation
Max Reitz [Fri, 7 Mar 2014 23:39:44 +0000 (00:39 +0100)] 
block/raw-posix: Strip protocol prefix on creation

The hdev_create() implementation in block/raw-posix.c is used by the
"host_device", "host_cdrom" and "host_floppy" protocol block drivers
together. Thus, any of the associated prefixes may occur and exactly one
should should be stripped, if it does (thus,
"host_device:host_cdrom:/dev/cdrom" is not shortened to "/dev/cdrom").

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock/raw-posix: bdrv_parse_filename() for cdrom
Max Reitz [Fri, 7 Mar 2014 23:39:43 +0000 (00:39 +0100)] 
block/raw-posix: bdrv_parse_filename() for cdrom

The "host_cdrom" protocol drivers should strip the "host_cdrom:" prefix
from filenames if present.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock/raw-posix: bdrv_parse_filename() for floppy
Max Reitz [Fri, 7 Mar 2014 23:39:42 +0000 (00:39 +0100)] 
block/raw-posix: bdrv_parse_filename() for floppy

The "host_floppy" protocol driver should strip the "host_floppy:" prefix
from filenames if present.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock/raw-posix: bdrv_parse_filename() for hdev
Max Reitz [Fri, 7 Mar 2014 23:39:41 +0000 (00:39 +0100)] 
block/raw-posix: bdrv_parse_filename() for hdev

The "host_device" protocol driver should strip the "host_device:" prefix
from filenames if present.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqemu-io: Fix warnings from static code analysis
Stefan Weil [Wed, 5 Mar 2014 21:23:00 +0000 (22:23 +0100)] 
qemu-io: Fix warnings from static code analysis

Smatch complains about several global symbols which should be local.

Add the missing 'static' attributes and move the 'extern' declaration
of variable qemuio_misalign to qemu-io.h. This variable also changes
the type from 'int' to 'bool' which better fits documents its use.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock: Unlink temporary file
Max Reitz [Sat, 15 Feb 2014 17:03:21 +0000 (18:03 +0100)] 
block: Unlink temporary file

If the image file cannot be opened and was created as a temporary file,
it should be deleted; thus, in this case, we should jump to the
"unlink_and_fail" label and not just to "fail".

Reported-by: Benoît Canet <benoit@irqsave.net>
Signed-off-by: Max Reitz <mreitz@redhat.com>
Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqcow2: Don't write with BDRV_O_INCOMING
Kevin Wolf [Tue, 11 Mar 2014 14:15:03 +0000 (15:15 +0100)] 
qcow2: Don't write with BDRV_O_INCOMING

qcow2_open() causes writes when repairing an image with the dirty flag
set and when clearing autoclear flags. It shouldn't do this when another
qemu instance is still actively working on this image file.

One effect of the bug is that images may have a cleared dirty flag while
the migration source host still has it in use with lazy refcounts
enabled, so refcounts are not accurate and the dirty flag must remain
set.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqcow2: Keep option in qcow2_invalidate_cache()
Kevin Wolf [Tue, 11 Mar 2014 16:42:41 +0000 (17:42 +0100)] 
qcow2: Keep option in qcow2_invalidate_cache()

Instead of manually building a list of all options from BDRVQcowState
values just reuse the options that were used to open the image.
qcow2_open() won't fully use all of the options in the QDict, but that's
okay.

This fixes all of the driver-specific options in qcow2, except for
lazy-refcounts, which was special cased before.

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqmp: add query-iothreads command
Stefan Hajnoczi [Thu, 27 Feb 2014 10:48:42 +0000 (11:48 +0100)] 
qmp: add query-iothreads command

The "query-iothreads" command returns a list of information about
iothreads.  See the patch for API documentation.

Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoiothread: stash thread ID away
Stefan Hajnoczi [Thu, 27 Feb 2014 10:48:41 +0000 (11:48 +0100)] 
iothread: stash thread ID away

Keep the thread ID around so we can report it via QMP.

There's only one problem: qemu_get_thread_id() (gettid() wrapper on
Linux) must be called from the thread itself.  There is no way to get
the thread ID outside the thread.

This patch uses a condvar to wait for iothread_run() to populate the
thread_id inside the thread.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agodataplane: replace internal thread with IOThread
Stefan Hajnoczi [Mon, 3 Mar 2014 10:30:08 +0000 (11:30 +0100)] 
dataplane: replace internal thread with IOThread

Today virtio-blk dataplane uses a 1:1 device-per-thread model.  Now that
IOThreads have been introduced we can generalize this to N:M devices per
threads.

This patch drops thread code from dataplane in favor of running inside
an IOThread AioContext.

As a bonus we solve the case where a guest keeps submitting I/O requests
while dataplane is trying to stop.  Previously the dataplane thread
would continue to process requests until the request gave it a break.
Now we can shut down in bounded time thanks to
aio_context_acquire/release.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoiothread: add "iothread" qdev property type
Stefan Hajnoczi [Mon, 3 Mar 2014 10:30:07 +0000 (11:30 +0100)] 
iothread: add "iothread" qdev property type

Add a "iothread" qdev property type so devices can be hooked up to an
IOThread from the comand-line:

  qemu -object iothread,id=iothread0 \
       -device some-device,x-iothread=iothread0

Note that Paolo Bonzini <pbonzini@redhat.com> has suggested using QOM
links instead.  This way the relationship between the objects is
reflected in QOM.  There are currently shortcomings of
object_property_add_link() which prevent this use case.  I will attempt
to fix them and move to QOM links in a separate series.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqdev: make get_pointer() handle temporary strings
Igor Mammedov [Mon, 3 Mar 2014 10:30:06 +0000 (11:30 +0100)] 
qdev: make get_pointer() handle temporary strings

get_pointer()'s print() callback might return a heap allocated
string, to avoid adding dedicated get_pointer_foo for this case
convert current print() callbacks to return temporary heap
allocated string and make get_pointer() free it.

Reviewed-by: Andreas Färber <afaerber@suse.de>
Signed-off-by: Igor Mammedov <imammedo@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoiothread: add I/O thread object
Stefan Hajnoczi [Mon, 3 Mar 2014 10:30:05 +0000 (11:30 +0100)] 
iothread: add I/O thread object

This is a stand-in for Michael Roth's QContext.  I expect this to be
replaced once QContext is completed.

The IOThread object is an AioContext event loop thread.  This patch adds
the concept of multiple event loop threads, allowing users to define
them.

When SMP guests run on SMP hosts it makes sense to instantiate multiple
IOThreads.  This spreads event loop processing across multiple cores.
Note that additional patches are required to actually bind a device to
an IOThread.

[Andreas Färber <afaerber@suse.de> pointed out that the embedded parent
object instance should be called "parent_obj" and have a newline
afterwards.  This patch has been changed to reflect this.
-- Stefan]

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoaio: add aio_context_acquire() and aio_context_release()
Stefan Hajnoczi [Mon, 3 Mar 2014 10:30:04 +0000 (11:30 +0100)] 
aio: add aio_context_acquire() and aio_context_release()

It can be useful to run an AioContext from a thread which normally does
not "own" the AioContext.  For example, request draining can be
implemented by acquiring the AioContext and looping aio_poll() until all
requests have been completed.

The following pattern should work:

  /* Event loop thread */
  while (running) {
      aio_context_acquire(ctx);
      aio_poll(ctx, true);
      aio_context_release(ctx);
  }

  /* Another thread */
  aio_context_acquire(ctx);
  bdrv_read(bs, 0x1000, buf, 1);
  aio_context_release(ctx);

This patch implements aio_context_acquire() and aio_context_release().

Note that existing aio_poll() callers do not need to worry about
acquiring and releasing - it is only needed when multiple threads will
call aio_poll() on the same AioContext.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agorfifolock: add recursive FIFO lock
Stefan Hajnoczi [Mon, 3 Mar 2014 10:30:03 +0000 (11:30 +0100)] 
rfifolock: add recursive FIFO lock

QemuMutex does not guarantee fairness and cannot be acquired
recursively:

Fairness means each locker gets a turn and the scheduler cannot cause
starvation.

Recursive locking is useful for composition, it allows a sequence of
locking operations to be invoked atomically by acquiring the lock around
them.

This patch adds RFifoLock, a recursive lock that guarantees FIFO order.
Its first user is added in the next patch.

RFifoLock has one additional feature: it can be initialized with an
optional contention callback.  The callback is invoked whenever a thread
must wait for the lock.  For example, it can be used to poke the current
owner so that they release the lock soon.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoobject: add object_get_canonical_path_component()
Stefan Hajnoczi [Mon, 3 Mar 2014 10:30:02 +0000 (11:30 +0100)] 
object: add object_get_canonical_path_component()

It is often useful to find an object's child property name.  Also use
this new function to simplify the implementation of
object_get_canonical_path().

Reviewed-by: Andreas Färber <afaerber@suse.de>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock: Rewrite the snapshot authorization mechanism for block filters.
Benoît Canet [Mon, 3 Mar 2014 18:11:34 +0000 (19:11 +0100)] 
block: Rewrite the snapshot authorization mechanism for block filters.

This patch keep the recursive way of doing things but simplify it by giving
two responsabilities to all block filters implementors.

They will need to do two things:

-Set the is_filter field of their block driver to true.

-Implement the bdrv_recurse_is_first_non_filter method of their block driver like
it is done on the Quorum block driver. (block/quorum.c)

[Paolo Bonzini <pbonzini@redhat.com> pointed out that this patch changes
the semantics of blkverify, which now recurses down both bs->file and
s->test_file.
-- Stefan]

Reported-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Benoit Canet <benoit@irqsave.net>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoiotests: Test corruption during COW request
Max Reitz [Mon, 10 Mar 2014 22:44:09 +0000 (23:44 +0100)] 
iotests: Test corruption during COW request

Extend test file 060 by a test case for corruption occuring concurrently
to a COW request. QEMU should not crash but rather return an appropriate
error message.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock: bs->drv may be NULL in bdrv_debug_resume()
Max Reitz [Mon, 10 Mar 2014 22:44:08 +0000 (23:44 +0100)] 
block: bs->drv may be NULL in bdrv_debug_resume()

Currently, bdrv_debug_resume() requires every bs->drv in the BDS stack
to be NULL until a bs->drv with an implementation of bdrv_debug_resume()
is found. For a normal function, this would be fine, but this is a
function for debugging purposes and should therefore allow intermediate
BDS not to have a driver (i.e., be "ejected"). Otherwise, it is hard to
debug such situations.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqcow2: Check bs->drv in copy_sectors()
Max Reitz [Mon, 10 Mar 2014 22:44:07 +0000 (23:44 +0100)] 
qcow2: Check bs->drv in copy_sectors()

Before dereferencing bs->drv for a call to its member bdrv_co_readv(),
copy_sectors() should check whether that pointer is indeed valid, since
it may have been set to NULL by e.g. a concurrent write triggering the
corruption prevention mechanism.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoblock: Update image size in bdrv_invalidate_cache()
Kevin Wolf [Tue, 11 Mar 2014 09:58:39 +0000 (10:58 +0100)] 
block: Update image size in bdrv_invalidate_cache()

After migration has completed, we call bdrv_invalidate_cache() so that
drivers which cache some data drop their stale copy of the data and
reread it from the image file to get a new version of data that the
source modified while the migration was running.

Reloading metadata from the image file is useless, though, if the size
of the image file stays stale (this is a value that is cached for all
image formats in block.c). Reads from (meta)data after the old EOF
return only zeroes, causing image corruption.

We need to update bs->total_sectors in all layers that could potentially
have changed their size (i.e. backing files are not a concern - if they
are changed, we're in bigger trouble)

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoqcow2-refcount: Sanitize refcount table entry
Max Reitz [Fri, 7 Mar 2014 22:10:12 +0000 (23:10 +0100)] 
qcow2-refcount: Sanitize refcount table entry

When reading the refcount table entry in get_refcount(), only bits which
are actually significant for the refcount block offset should be taken
into account.

Signed-off-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
7 years agoMerge remote-tracking branch 'remotes/afaerber/tags/ppc-for-2.0' into staging
Peter Maydell [Thu, 13 Mar 2014 13:19:46 +0000 (13:19 +0000)] 
Merge remote-tracking branch 'remotes/afaerber/tags/ppc-for-2.0' into staging

PowerPC queue for 2.0-rc0

* QEMUMachine include cleanup
* SLOF update
* XICS reset fix
* sPAPR PCI host bridge refactorings

# gpg: Signature made Thu 13 Mar 2014 02:50:51 GMT using RSA key ID 3E7E013F
# gpg: Good signature from "Andreas Färber <afaerber@suse.de>"
# gpg:                 aka "Andreas Färber <afaerber@suse.com>"

* remotes/afaerber/tags/ppc-for-2.0:
  spapr-pci: Convert fprintf() to error_report()
  spapr-pci: Convert to QOM realize
  xics-kvm: Fix reset function
  pseries: Update SLOF firmware image to qemu-slof-20140304
  Move QEMUMachine typedef to qemu/typedefs.h
  Revert "KVM: Split QEMUMachine typedef into separate header"

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
7 years agoMerge remote-tracking branch 'remotes/afaerber/tags/qom-devices-for-peter' into staging
Peter Maydell [Thu, 13 Mar 2014 12:32:47 +0000 (12:32 +0000)] 
Merge remote-tracking branch 'remotes/afaerber/tags/qom-devices-for-peter' into staging

QOM/QTest infrastructure fixes and device conversions

* QTest cleanups and test cases for some virtio devices
* QTest for sPAPR PCI host bridge
* qom-test now tests reading all properties beneath /machine
* QOM API leak fixes
* QOM cleanups for SSI devices
* QOM conversion of QEMUMachine
* QOM realize for buses
* sPAPR PCI bus name change

# gpg: Signature made Thu 13 Mar 2014 00:22:40 GMT using RSA key ID 3E7E013F
# gpg: Good signature from "Andreas Färber <afaerber@suse.de>"
# gpg:                 aka "Andreas Färber <afaerber@suse.com>"

* remotes/afaerber/tags/qom-devices-for-peter: (31 commits)
  libqtest: Fix possible deadlock in qtest initialization
  pci: Move VMState registration/unregistration to QOM realize/unrealize
  qdev: Realize buses on device realization
  qdev: Prepare realize/unrealize hooks for BusState
  tests: Add spapr-pci-host-bridge qtest
  virtio-serial-port: Convert to QOM realize/unrealize
  virtio-console: QOM cast cleanup for VirtConsole
  tests: Add virtio-console qtest
  tests: Add virtio-serial qtest
  tests: Add virtio-scsi qtest
  tests: Add virtio-rng qtest
  tests: Add virtio-balloon qtest
  tests: Add virtio-blk qtest
  tests: Clean up IndustryPack TPCI200 gcov paths
  qom-test: Test QOM properties
  hw/boards: Convert current_machine to MachineState
  vl: Use MachineClass instead of global QEMUMachine list
  hw/core: Introduce QEMU machine as QOM object
  qdev-monitor-test: Don't test human-readable error message
  qdev-monitor-test: Simplify using g_assert_cmpstr()
  ...

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
7 years agolibqtest: Avoid inline varargs functions
Peter Maydell [Thu, 13 Mar 2014 11:24:15 +0000 (11:24 +0000)] 
libqtest: Avoid inline varargs functions

Older versions of gcc (eg 4.6) can't handle varargs functions declared
inline for anything other than completely trivial uses, and complain:

tests/qom-test.c: In function 'qmp': tests/libqtest.h:359:60: sorry,
unimplemented: function 'qmp' can never be inlined because it uses
variable argument lists

Avoid this problem by putting the functions into libqtest.c instead
of using inline definitions in libqtest.h.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Andreas Färber <afaerber@suse.de>
7 years agospapr-pci: Convert fprintf() to error_report()
Alexey Kardashevskiy [Thu, 21 Nov 2013 04:08:58 +0000 (15:08 +1100)] 
spapr-pci: Convert fprintf() to error_report()

Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Reviewed-by: Mike Day <ncmike@ncultra.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agospapr-pci: Convert to QOM realize
Alexey Kardashevskiy [Thu, 21 Nov 2013 04:08:55 +0000 (15:08 +1100)] 
spapr-pci: Convert to QOM realize

This converts the old-style SysBusDevice::init() callback to a new-style
DeviceClass::realize() callback.

As a part of conversion, this replaces fprintf(stderr) with error_setg()
as realize() does not "return" any value, instead it puts the extended
error into **errp.

Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Reviewed-by: Mike Day <ncmike@ncultra.org>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoxics-kvm: Fix reset function
Alexey Kardashevskiy [Thu, 13 Feb 2014 01:08:35 +0000 (12:08 +1100)] 
xics-kvm: Fix reset function

Currently interrupt priorities are set to 0 (highest) at the very
beginning of the guest execution which is not correct and makes the guest
produce random interrupt error messages such as:
"Interrupt 0x1001 (real) is invalid, disabling it".
This also prevents interrupt states from correct migration.

This initializes priority to 0xFF as the emulated XICS does.

Suggested-by: Paul Mackerras <paulus@samba.org>
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agopseries: Update SLOF firmware image to qemu-slof-20140304
Alexey Kardashevskiy [Mon, 10 Mar 2014 12:07:58 +0000 (23:07 +1100)] 
pseries: Update SLOF firmware image to qemu-slof-20140304

The changelog is:
  > version: update to 20140304
  > Introduce dummy console device
  > vio-vscsi: Fix CRQ allocation alignment
  > version: update to 20140204
  > virtio-9p: disable unused structure
  > Make "boot net:dhcp" boot from IPv4 only
  > Fix virtio device shutdown
  > Change shutdown method name for virtio-scsi
  > Add support for 64bit LE ABI v1 and v2 support
  > Change representation of string environment variable
  > cas: return error when unknown node found
  > version: update
  > Reset obp-tftp arguments before parsing
  > Enable seamless netboot on IPv6 network
  > Fix shutdown for virtio devices
  > Fix zero checksum in UDP header
  > Handle router advertisement message properly
  > [oex]hci_exit: Check before freeing/unmapping memory
  > Work around missing sc 1 traps on pHyp
  > fix print_version() to return where it came from
  > usb-xhci: memory freeing and using returns as bool uniformly
  > Output banner and initial display output in VNC window
  > use VERSION file to generate FW version
  > cas: remove warning
  > Add support for loading little endian ELF binaries.
  > Add bswap_{16,32,64}p
  > dhcpv6 and other minor net-snk fixes
  > Fix missing drop in virtio-fs setup-alias
  > Find next available alias name
  > SLOF does not exit if given 1KB disk
  > boot: enable support for bootindex
  > pci-properties: add properties to enable hotplug for spapr
  > e1000: remember node handle
  > Increase quiesce tokens array size
  > virtio: timeout after 5sec
  > Enable IPv6 support in dns
  > usb-ohci: fix warnings
  > Add ipv6 support in net-snk
  > ipv4: fix frame overwriting following arp_send_request
  > e1000: fix SLOF_dma_map_out arguments
  > Maintain single global packet buffer for tftp
  > Increase virtio-net receive queue size
  > Increase veth receive queue size
  > Fix dprintf macros at various points
  > usb-ohci: rewrite done_head processing code
  > boot: add net in default boot order
  > block 0 address in the allocator
  > scsi: make-media-alias fix
  > usb-xhci: add xhci host controller support
  > usb-xhci: add xhci support
  > Avoid veth read/write calls with zero length buffer
  > boot: include other aliases
  > usb-core: disable xhci

Cc: Andreas Färber <afaerber@suse.de>
Cc: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
7 years agoMove QEMUMachine typedef to qemu/typedefs.h
Andreas Färber [Thu, 13 Mar 2014 02:40:13 +0000 (03:40 +0100)] 
Move QEMUMachine typedef to qemu/typedefs.h

As reported in commit 9c06a1f79f959fffd09bfb7efc3d76051a6cd2da, xen.h is
not self-contained with regards to its use of QEMUMachine. Fix this.

Reported-by: Alexander Graf <agraf@suse.de>
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoRevert "KVM: Split QEMUMachine typedef into separate header"
Andreas Färber [Thu, 13 Mar 2014 02:29:51 +0000 (03:29 +0100)] 
Revert "KVM: Split QEMUMachine typedef into separate header"

This reverts commit 9c06a1f79f959fffd09bfb7efc3d76051a6cd2da.
The new header sysemu/qemumachine.h is undesired.

Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agolibqtest: Fix possible deadlock in qtest initialization
Marcel Apfelbaum [Tue, 11 Mar 2014 13:00:34 +0000 (15:00 +0200)] 
libqtest: Fix possible deadlock in qtest initialization

'socket_accept' waits for QEMU to init its unix socket.
If QEMU encounters an error during command line parsing,
it can exit before initializing the communication channel.

Using a timeout for sockets fixes the issue.

Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Marcel Apfelbaum <marcel.a@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agopci: Move VMState registration/unregistration to QOM realize/unrealize
Bandan Das [Mon, 25 Nov 2013 22:48:42 +0000 (17:48 -0500)] 
pci: Move VMState registration/unregistration to QOM realize/unrealize

Use the realize and unrealize hooks to register and unregister
vmstate_pcibus respectively.

Relocate some stuff to avoid forward declarations.

Signed-off-by: Bandan Das <bsd@redhat.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
[AF: Keep using PCI_BUS() cast macro]
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoqdev: Realize buses on device realization
Bandan Das [Wed, 12 Mar 2014 20:02:12 +0000 (21:02 +0100)] 
qdev: Realize buses on device realization

Integrate (un)realization of child buses with realization/unrealization
of the device hosting them. Code in device_unparent() is reordered for
unrealization of buses to work as part of device unrealization.

That way no changes need to be made to bus instantiation.

Signed-off-by: Bandan Das <bsd@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agoqdev: Prepare realize/unrealize hooks for BusState
Bandan Das [Mon, 25 Nov 2013 22:48:40 +0000 (17:48 -0500)] 
qdev: Prepare realize/unrealize hooks for BusState

Add a "realized" property calling realize/unrealize hooks as for devices.

Signed-off-by: Bandan Das <bsd@redhat.com>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotests: Add spapr-pci-host-bridge qtest
Alexey Kardashevskiy [Mon, 10 Feb 2014 03:52:56 +0000 (14:52 +1100)] 
tests: Add spapr-pci-host-bridge qtest

This adds a test whether sPAPR PHB can be added via the command line.

Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agovirtio-serial-port: Convert to QOM realize/unrealize
Andreas Färber [Fri, 7 Jun 2013 17:02:12 +0000 (19:02 +0200)] 
virtio-serial-port: Convert to QOM realize/unrealize

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agovirtio-console: QOM cast cleanup for VirtConsole
Andreas Färber [Fri, 7 Jun 2013 17:10:02 +0000 (19:10 +0200)] 
virtio-console: QOM cast cleanup for VirtConsole

Introduce type constant, cast macro and rename parent field.

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotests: Add virtio-console qtest
Andreas Färber [Fri, 21 Feb 2014 16:49:12 +0000 (17:49 +0100)] 
tests: Add virtio-console qtest

Signed-off-by: Andreas Färber <afaerber@suse.de>
7 years agotests: Add virtio-serial qtest
Andreas Färber [Fri, 21 Feb 2014 16:36:57 +0000 (17:36 +0100)] 
tests: Add virtio-serial qtest

Signed-off-by: Andreas Färber <afaerber@suse.de>