Re: [Qemu-devel] [PATCH v2 3/5] Qemu: do not mark bios readonly

2012-10-30 Thread Jan Kiszka
On 2012-10-31 07:35, Xiao Guangrong wrote: > We can not do that for pflash is a RD device which can not be directly > written, > kvm can not emulate the instruction which implicitly write the memory. > (e.g: > using this area as stack). Isn't enabling ROMD support fo

Re: [Qemu-devel] [PATCH v2 00/19] target-mips: misc fixes and optimizations

2012-10-30 Thread Richard Henderson
On 2012-10-30 11:11, Aurelien Jarno wrote: > Aurelien Jarno (19): > target-mips: correctly restore btarget upon exception > target-mips: do not save CPU state when using retranslation > softfloat: implement fused multiply-add NaN propagation for MIPS > target-mips: use the softfloat floatXX

Re: [Qemu-devel] [PATCH v2 3/5] Qemu: do not mark bios readonly

2012-10-30 Thread Xiao Guangrong
On 10/31/2012 02:03 PM, Jan Kiszka wrote: > On 2012-10-29 09:31, Xiao Guangrong wrote: >> On 10/29/2012 03:44 PM, Jan Kiszka wrote: >>> On 2012-10-29 08:09, Xiao Guangrong wrote: Jan, On 10/26/2012 06:35 PM, Jan Kiszka wrote: > This has two problems: We know it breaks at lea

[Qemu-devel] ISA BIOS mapping for system flash emulation

2012-10-30 Thread Jan Kiszka
Hi Jordan, I was starring at pc_isa_bios_init and wondering why you are creating a copy of the system flash for the low ISA range instead of using an alias here as well, just like old_pc_system_rom_init does. That means the ISA BIOS range can run out of sync when the system flash is updated during

Re: [Qemu-devel] [PATCH v2 3/5] Qemu: do not mark bios readonly

2012-10-30 Thread Jan Kiszka
On 2012-10-29 09:31, Xiao Guangrong wrote: > On 10/29/2012 03:44 PM, Jan Kiszka wrote: >> On 2012-10-29 08:09, Xiao Guangrong wrote: >>> Jan, >>> >>> On 10/26/2012 06:35 PM, Jan Kiszka wrote: >>> This has two problems: We know it breaks at least Win 95 that overwrites its F-segment during

Re: [Qemu-devel] [PATCH 14/22] usb/ehci: add sysbus variant

2012-10-30 Thread Peter Crosthwaite
All good, Needs a hack to get around the dma_memory_foo segfault (attached), but that will come out in the wash with the memory API refactorings. Regards, Peter On Tue, 2012-10-30 at 15:26 +0100, Gerd Hoffmann wrote: > Signed-off-by: Gerd Hoffmann Reviewed-by: Peter Crosthwaite Tested-by: Pet

Re: [Qemu-devel] Ubuntu/Debian Installer + Virtio-SCSI -> Bad ram pointer

2012-10-30 Thread Stefan Hajnoczi
On Tue, Oct 30, 2012 at 8:37 PM, Peter Lieven wrote: > Am 30.10.2012 19:27, schrieb Stefan Hajnoczi: >> Please also post the exact qemu-kvm version you are using. I can see >> it's based on qemu-kvm-1.2.0 but are there any patches applied (e.g. >> distro packages may carry patches so the full pac

[Qemu-devel] [PATCH v4 4/7] TCG: Use gen_opparam_ptr from context instead of global variable.

2012-10-30 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- gen-icount.h |2 +- tcg/tcg-op.h | 254 +- tcg/tcg.c| 36 - 3 files changed, 146 insertions(+), 146 deletions(-) diff --git a/gen-icount.h b/gen-icount.h index 430cb44..248cf5b 100644 --

[Qemu-devel] [PATCH v4 7/7] TCG: Remove unused global variables

2012-10-30 Thread Evgeny Voevodin
From: Evgeny Signed-off-by: Evgeny Voevodin --- tcg/tcg.c |4 tcg/tcg.h |4 translate-all.c |3 --- 3 files changed, 11 deletions(-) diff --git a/tcg/tcg.c b/tcg/tcg.c index 1fd1731..a9c9d6f 100644 --- a/tcg/tcg.c +++ b/tcg/tcg.c @@ -96,10 +96,6 @@ const size_

[Qemu-devel] [PATCH v4 5/7] TCG: Use gen_opc_buf from context instead of global variable.

2012-10-30 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- target-alpha/translate.c |6 ++-- target-arm/translate.c|6 ++-- target-cris/translate.c |9 +++--- target-i386/translate.c |6 ++-- target-lm32/translate.c |9 +++--- target-m68k/translate.c |6 ++--

Re: [Qemu-devel] Ubuntu/Debian Installer + Virtio-SCSI -> Bad ram pointer

2012-10-30 Thread Stefan Hajnoczi
On Tue, Oct 30, 2012 at 10:09 PM, ronnie sahlberg wrote: > About half a year there was an issue where recent kernels had added > support to start using new scsi opcodes, but the qemu functions that > determine "which transfer direction is used for this opcode" had not > yet been updated, so that

[Qemu-devel] [PATCH v4 6/7] TCG: Use gen_opparam_buf from context instead of global variable.

2012-10-30 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- tcg/tcg.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/tcg/tcg.c b/tcg/tcg.c index a7c3832..1fd1731 100644 --- a/tcg/tcg.c +++ b/tcg/tcg.c @@ -298,7 +298,7 @@ void tcg_func_start(TCGContext *s) #endif s->gen_opc_ptr =

[Qemu-devel] [PATCH v4 2/7] tcg/tcg.h: Duplicate global TCG variables in TCGContext

2012-10-30 Thread Evgeny Voevodin
From: Evgeny Signed-off-by: Evgeny Signed-off-by: Evgeny Voevodin --- tcg/tcg.h |6 ++ 1 file changed, 6 insertions(+) diff --git a/tcg/tcg.h b/tcg/tcg.h index a6c9256..b229061 100644 --- a/tcg/tcg.h +++ b/tcg/tcg.h @@ -431,6 +431,12 @@ struct TCGContext { int temps_in_use;

[Qemu-devel] [PATCH v4 0/7] TCG global variables clean-up

2012-10-30 Thread Evgeny Voevodin
This set of patches moves global variables to tcg_ctx: gen_opc_ptr gen_opparam_ptr gen_opc_buf gen_opparam_buf Build tested for all targets. Execution tested on Exynos4210 target. After this patchset was aplied, I noticed 0.7% speed-up of code generation. Probably, this is due to better data cach

Re: [Qemu-devel] [PATCH v12 09/14] target-mips: Add ASE DSP bit/manipulation instructions

2012-10-30 Thread Richard Henderson
On 2012-10-31 01:44, Peter Maydell wrote: > On 30 October 2012 15:34, Jia Liu wrote: >> On Mon, Oct 29, 2012 at 9:40 PM, Jovanovic, Petar wrote: imm = (int16_t)(imm << 6) >> 6; >>> >>> result of a bitwise shift of a signed type and a negative vlaue is >>> implementation-defined, so you can n

[Qemu-devel] [PATCH v4 3/7] TCG: Use gen_opc_ptr from context instead of global variable.

2012-10-30 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- target-alpha/translate.c |8 ++--- target-arm/translate.c|8 ++--- target-cris/translate.c | 10 +++--- target-i386/translate.c |8 ++--- target-lm32/translate.c | 10 +++--- target-m68k/translate.c |8 ++-

Re: [Qemu-devel] [RFC 7/7] target-alpha: Implement CPU reset

2012-10-30 Thread Richard Henderson
On 2012-10-31 14:04, Andreas Färber wrote: > +/* CPUClass::reset() */ > +static void alpha_cpu_reset(CPUState *s) > +{ > +AlphaCPU *cpu = ALPHA_CPU(s); > +AlphaCPUClass *acc = ALPHA_CPU_GET_CLASS(cpu); > +CPUAlphaState *env = &cpu->env; > + > +if (qemu_loglevel_mask(CPU_LOG_RESET))

Re: [Qemu-devel] [PATCH 6/7] target-alpha: Avoid leaking the alarm timer over reset

2012-10-30 Thread Richard Henderson
On 2012-10-31 14:04, Andreas Färber wrote: > Move the timer from CPUAlphaState to AlphaCPU to avoid the pointer being > zero'ed once we implement reset. Would cause a segfault in > sys_helper.c:helper_set_alarm(). > > This also simplifies timer initialization in Typhoon. > > Signed-off-by: Andrea

Re: [Qemu-devel] [PATCH 5/7] alpha: Pass AlphaCPU array to Typhoon

2012-10-30 Thread Richard Henderson
On 2012-10-31 14:04, Andreas Färber wrote: > Also store it in TyphoonCchip. > > Signed-off-by: Andreas Färber > --- > hw/alpha_dp264.c | 18 +- > hw/alpha_sys.h |2 +- > hw/alpha_typhoon.c | 29 - Acked-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH 4/7] target-alpha: Let cpu_alpha_init() return AlphaCPU

2012-10-30 Thread Richard Henderson
On 2012-10-31 14:04, Andreas Färber wrote: > Replace cpu_init() macro with inline function for backwards > compatibility. > > Signed-off-by: Andreas Färber > --- > target-alpha/cpu.h | 13 +++-- > target-alpha/translate.c |4 ++-- Acked-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH 3/7] target-alpha: Add support for -cpu ?

2012-10-30 Thread Richard Henderson
On 2012-10-31 14:04, Andreas Färber wrote: > Implement alphabetical listing of CPU subclasses. > > Signed-off-by: Andreas Färber There doesn't seem to be anything alpha-specific about this. Does it really need to be replicated? That said, Acked-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH 2/7] target-alpha: Turn CPU definitions into subclasses

2012-10-30 Thread Richard Henderson
On 2012-10-31 14:03, Andreas Färber wrote: > +static const AlphaCPUInfo alpha_cpus[] = { > +{ .name = "ev4", .initfn = ev4_cpu_initfn }, > +{ .name = "ev5", .initfn = ev5_cpu_initfn }, > +{ .name = "ev56",.initfn = ev56_cpu_initfn }, > +{ .name = "pca56", .initfn = pca

Re: [Qemu-devel] [RFC] Configure option to turn on all debug printerfy

2012-10-30 Thread Peter Crosthwaite
On Wed, Oct 31, 2012 at 1:22 PM, Andreas Färber wrote: > Am 31.10.2012 04:10, schrieb Peter Crosthwaite: >> So in a couple of cases now I have done work on mature device models >> that include #ifdef'd debug printfery that is broken, and have >> submitted (trivial) patches to fix. Whats happening

Re: [Qemu-devel] [PATCH 1/5] disas: avoid using cpu_single_env

2012-10-30 Thread guanxuetao
> Pass around CPUState instead of using global cpu_single_env. > > Signed-off-by: Blue Swirl For unicore32 part: Acked-by: Guan Xuetao > --- > disas.c | 149 > ++--- > disas.h |3 +- > qemu-log.h

Re: [Qemu-devel] [PATCH 0/5] cpu_single_env cleanup

2012-10-30 Thread Andreas Färber
Am 28.10.2012 16:03, schrieb Blue Swirl: > In some cases it's pretty easy to avoid using global > cpu_single_env since a local version is available. > > Blue Swirl (5): > disas: avoid using cpu_single_env > kvm: avoid using cpu_single_env > target-unicore32: avoid using cpu_single_env > ta

[Qemu-devel] [PATCH 2/7] target-alpha: Turn CPU definitions into subclasses

2012-10-30 Thread Andreas Färber
Make TYPE_ALPHA_CPU abstract and default to creating type "ev67". Signed-off-by: Andreas Färber --- target-alpha/cpu.c | 157 +- target-alpha/translate.c | 49 +++ 2 Dateien geändert, 163 Zeilen hinzugefügt(+), 43 Zeilen entfernt(-

[Qemu-devel] [PATCH] Fix off-by-1 error in RAM migration code

2012-10-30 Thread David Gibson
The code for migrating (or savevm-ing) memory pages starts off by creating a dirty bitmap and filling it with 1s. Except, actually, because bit addresses are 0-based it fills every bit except bit 0 with 1s and puts an extra 1 beyond the end of the bitmap, potentially corrupting unrelated memory.

Re: [Qemu-devel] [PATCH 5/5] target-i386: avoid using cpu_single_env

2012-10-30 Thread Andreas Färber
Am 28.10.2012 16:03, schrieb Blue Swirl: > Pass around CPUState instead of using global cpu_single_env. > > Signed-off-by: Blue Swirl > --- > target-i386/translate.c | 440 > --- > 1 files changed, 222 insertions(+), 218 deletions(-) Reviewed-by: An

[Qemu-devel] [PATCH 6/7] target-alpha: Avoid leaking the alarm timer over reset

2012-10-30 Thread Andreas Färber
Move the timer from CPUAlphaState to AlphaCPU to avoid the pointer being zero'ed once we implement reset. Would cause a segfault in sys_helper.c:helper_set_alarm(). This also simplifies timer initialization in Typhoon. Signed-off-by: Andreas Färber --- hw/alpha_typhoon.c|3 +-- targ

[Qemu-devel] [PATCH 3/7] target-alpha: Add support for -cpu ?

2012-10-30 Thread Andreas Färber
Implement alphabetical listing of CPU subclasses. Signed-off-by: Andreas Färber --- target-alpha/cpu.c | 41 + target-alpha/cpu.h |4 +++- 2 Dateien geändert, 44 Zeilen hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/target-alpha/cpu.c b/target-alp

Re: [Qemu-devel] [PATCH 4/5] target-xtensa: avoid using cpu_single_env

2012-10-30 Thread Andreas Färber
Am 28.10.2012 16:03, schrieb Blue Swirl: > Pass around CPUState instead of using global cpu_single_env. > > Signed-off-by: Blue Swirl > --- > target-xtensa/translate.c | 10 +- > 1 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Andreas Färber ditto about "CPUState". An

Re: [Qemu-devel] [PATCH 3/5] target-unicore32: avoid using cpu_single_env

2012-10-30 Thread Andreas Färber
Am 28.10.2012 16:03, schrieb Blue Swirl: > Pass around CPUState instead of using global cpu_single_env. > > Signed-off-by: Blue Swirl > --- > target-unicore32/softmmu.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) Reviewed-by: Andreas Färber with the usual remark on "CP

Re: [Qemu-devel] [PATCH v3 00/35] postcopy live migration

2012-10-30 Thread Isaku Yamahata
On Tue, Oct 30, 2012 at 06:53:31PM +, Benoit Hudzia wrote: > Hi Isaku, > > > Are you going to be at the KVM forum ( i think you have a presentation there). > It would be nice if we could meet in order to see if we can synch our efforts > . Yes, definitively. > As you know we have been deve

Re: [Qemu-devel] [PATCH 2/5] kvm: avoid using cpu_single_env

2012-10-30 Thread Andreas Färber
Am 28.10.2012 16:03, schrieb Blue Swirl: > Pass around CPUState instead of using global cpu_single_env. > > Signed-off-by: Blue Swirl > --- > target-i386/kvm.c | 21 +++-- > 1 files changed, 11 insertions(+), 10 deletions(-) If you fix CPUState -> CPU*State or CPUArchState abo

Re: [Qemu-devel] [RFC] Configure option to turn on all debug printerfy

2012-10-30 Thread Andreas Färber
Am 31.10.2012 04:10, schrieb Peter Crosthwaite: > So in a couple of cases now I have done work on mature device models > that include #ifdef'd debug printfery that is broken, and have > submitted (trivial) patches to fix. Whats happening is tree wide or > automated changes (changing types of variab

[Qemu-devel] [PATCH 5/7] alpha: Pass AlphaCPU array to Typhoon

2012-10-30 Thread Andreas Färber
Also store it in TyphoonCchip. Signed-off-by: Andreas Färber --- hw/alpha_dp264.c | 18 +- hw/alpha_sys.h |2 +- hw/alpha_typhoon.c | 29 - 3 Dateien geändert, 26 Zeilen hinzugefügt(+), 23 Zeilen entfernt(-) diff --git a/hw/alpha_dp264.c

Re: [Qemu-devel] [PATCH v3 0/6] TCG global variables clean-up

2012-10-30 Thread Evgeny Voevodin
On 10/30/2012 10:46 PM, Blue Swirl wrote: On Mon, Oct 29, 2012 at 9:14 AM, Evgeny Voevodin wrote: This set of patches moves global variables to tcg_ctx: gen_opc_ptr gen_opparam_ptr gen_opc_buf gen_opparam_buf Patches don't apply, please rebase. Ok. When I sent they applied correctly. I'll re

Re: [Qemu-devel] [PULL] QOM CPUState patch queue 2012-10-31

2012-10-30 Thread Andreas Färber
Am 31.10.2012 01:59, schrieb Andreas Färber: > Hello Anthony, > > This is my current QOM CPUState queue. Please pull. > > This completes Igor's first step of x86 CPU hotplug roadmap: > http://wiki.qemu.org/Features/CPUHotplug > CPU-as-a-device is still under review and blocking CPU properties/sub

[Qemu-devel] [RFC] Configure option to turn on all debug printerfy

2012-10-30 Thread Peter Crosthwaite
So in a couple of cases now I have done work on mature device models that include #ifdef'd debug printfery that is broken, and have submitted (trivial) patches to fix. Whats happening is tree wide or automated changes (changing types of variables etc) is causing the debug printfery to break, usuall

[Qemu-devel] [RFC 7/7] target-alpha: Implement CPU reset

2012-10-30 Thread Andreas Färber
The parent_reset class field was already prepared but unused so far. No guarantees that this actually does The Right Thing, more fields may need to be moved within CPUAlphaState or to AlphaCPU. Signed-off-by: Andreas Färber --- target-alpha/cpu.c | 28 1 Datei geä

[Qemu-devel] [PATCH 4/7] target-alpha: Let cpu_alpha_init() return AlphaCPU

2012-10-30 Thread Andreas Färber
Replace cpu_init() macro with inline function for backwards compatibility. Signed-off-by: Andreas Färber --- target-alpha/cpu.h | 13 +++-- target-alpha/translate.c |4 ++-- 2 Dateien geändert, 13 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-) diff --git a/target-alpha/cpu.h b/

[Qemu-devel] [FYI 1/7] target-alpha: Use consistent include paths

2012-10-30 Thread Andreas Färber
Part of "cpus: Pass CPUState to [qemu_]cpu_has_work()" in pull request for master. Signed-off-by: Andreas Färber --- target-alpha/cpu.c |2 +- 1 Datei geändert, 1 Zeile hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/target-alpha/cpu.c b/target-alpha/cpu.c index 62d2a66..11a19eb 100644 ---

[Qemu-devel] [PATCH 0/7] target-alpha: More CPU QOM'ifications

2012-10-30 Thread Andreas Färber
Hello Richard, This series introduces CPU subclasses and implements -cpu ? support and CPU reset. The PATCHes are intended for v1.3; the reset RFC can well be postponed if more work is needed, it demonstrates what the preceding patch is about. Regards, Andreas Cc: Richard Henderson Andreas Fä

[Qemu-devel] [PATCH] block: vpc support for ~2 TB disks

2012-10-30 Thread Charles Arnold
The VHD specification allows for up to a 2 TB disk size. The current implementation in qemu emulates EIDE and ATA-2 hardware which only allows for up to 127 GB. This disk size limitation can be overridden by allowing up to 255 heads instead of the normal 4 bit limitation of 16. Doing so allows di

[Qemu-devel] [PATCH 06/35] target-i386: Pass X86CPU to cpu_x86_load_seg_cache_sipi()

2012-10-30 Thread Andreas Färber
Simplifies the call in apic_sipi() again and needed for moving halted field to CPUState. Signed-off-by: Andreas Färber Reviewed-by: Igor Mammedov --- hw/apic.c |2 +- target-i386/cpu.h |4 +++- 2 Dateien geändert, 4 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/hw/ap

[Qemu-devel] [PATCH 16/35] cpus: Pass CPUState to qemu_tcg_cpu_thread_fn

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed except for iterating the CPUs. Needed for qemu_tcg_init_vcpu(). KVM and dummy threads still need CPUArchState for cpu_single_env. Signed-off-by: Andreas Färber --- cpus.c |6 +++--- 1 Datei geändert, 3 Zeilen hinzugefügt(+), 3 Zeilen entfernt(-) diff --git

[Qemu-devel] [PATCH 07/35] cpus: Pass CPUState to qemu_cpu_is_self()

2012-10-30 Thread Andreas Färber
Change return type to bool, move to include/qemu/cpu.h and add documentation. Signed-off-by: Andreas Färber Reviewed-by: Igor Mammedov [AF: Updated new caller qemu_in_vcpu_thread()] --- cpus.c | 12 +--- exec.c |3 ++- hw/apic.c |6 -- incl

[Qemu-devel] [PATCH 33/35] cpu: Move thread_id to CPUState

2012-10-30 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-defs.h |1 - cpus.c | 11 ++- exec.c |5 - include/qemu/cpu.h |1 + 4 Dateien geändert, 11 Zeilen hinzugefügt(+), 7 Zeilen entfernt(-) diff --git a/cpu-defs.h b/cpu-defs.h index b30a8e9..3669241 10064

[Qemu-devel] [PATCH 10/35] cpu: Move stop field to CPUState

2012-10-30 Thread Andreas Färber
Change its type to bool. Signed-off-by: Andreas Färber --- cpu-defs.h |1 - cpus.c | 27 ++- include/qemu/cpu.h |2 ++ 3 Dateien geändert, 20 Zeilen hinzugefügt(+), 10 Zeilen entfernt(-) diff --git a/cpu-defs.h b/cpu-defs.h index 3b8bc20..7a

[Qemu-devel] [PATCH 12/35] cpu: Move stopped field to CPUState

2012-10-30 Thread Andreas Färber
Change its type to bool. Signed-off-by: Andreas Färber --- cpu-defs.h |1 - cpus.c | 30 ++ hw/ppce500_spin.c |3 ++- include/qemu/cpu.h |2 ++ 4 Dateien geändert, 22 Zeilen hinzugefügt(+), 14 Zeilen entfernt(-) diff --git a/cpu-def

Re: [Qemu-devel] [PATCH 3/5] target-unicore32: avoid using cpu_single_env

2012-10-30 Thread guanxuetao
> Pass around CPUState instead of using global cpu_single_env. > > Signed-off-by: Blue Swirl Acked-by: Guan Xuetao > --- > target-unicore32/softmmu.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/target-unicore32/softmmu.c b/target-unicore32/softmmu.c > i

[Qemu-devel] [PATCH 04/35] target-i386: Inline APIC cpu_env property setting

2012-10-30 Thread Andreas Färber
This prepares for changing the variable type from void*. Signed-off-by: Andreas Färber Reviewed-by: Igor Mammedov --- hw/apic_common.c |1 - target-i386/cpu.c |5 - 2 Dateien geändert, 4 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/hw/apic_common.c b/hw/apic_common.c i

[Qemu-devel] [PATCH 24/35] xtensa_pic: Pass XtensaCPU to xtensa_ccompare_cb()

2012-10-30 Thread Andreas Färber
Needed for changing cpu_has_work() argument type to CPUState. Signed-off-by: Andreas Färber Acked-by: Max Filippov --- hw/xtensa_pic.c |7 +-- 1 Datei geändert, 5 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/hw/xtensa_pic.c b/hw/xtensa_pic.c index 653ded6..8b9c051 100644 --

[Qemu-devel] [PULL] QOM CPUState patch queue 2012-10-31

2012-10-30 Thread Andreas Färber
Hello Anthony, This is my current QOM CPUState queue. Please pull. This completes Igor's first step of x86 CPU hotplug roadmap: http://wiki.qemu.org/Features/CPUHotplug CPU-as-a-device is still under review and blocking CPU properties/subclasses; I hope to get that in during the Soft Freeze if no

[Qemu-devel] [PATCH 25/35] target-ppc: Pass PowerPCCPU to powerpc_excp()

2012-10-30 Thread Andreas Färber
Needed for changing cpu_ppc_hypercall() argument type to PowerPCCPU. Signed-off-by: Andreas Färber --- target-ppc/excp_helper.c | 36 1 Datei geändert, 20 Zeilen hinzugefügt(+), 16 Zeilen entfernt(-) diff --git a/target-ppc/excp_helper.c b/target-ppc/excp_

[Qemu-devel] [PATCH 02/35] target-i386: If x86_cpu_realize() failed, report error and do cleanup

2012-10-30 Thread Andreas Färber
From: Igor Mammedov Signed-off-by: Igor Mammedov Signed-off-by: Andreas Färber --- target-i386/helper.c |9 +++-- 1 Datei geändert, 7 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/target-i386/helper.c b/target-i386/helper.c index c5d42c5..0424ccf 100644 --- a/target-i386/he

[Qemu-devel] [PATCH 27/35] spapr: Pass PowerPCCPU to spapr_hypercall()

2012-10-30 Thread Andreas Färber
Needed for changing the hypercall handlers' argument type to PowerPCCPU. Signed-off-by: Andreas Färber --- hw/spapr.c |2 +- hw/spapr.h |2 +- hw/spapr_hcall.c |4 +++- target-ppc/kvm.c |3 ++- 4 Dateien geändert, 7 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-) diff --

Re: [Qemu-devel] [PATCH v11] kvm: notify host when the guest is panicked

2012-10-30 Thread Sasha Levin
On Tue, Oct 30, 2012 at 9:48 PM, Wen Congyang wrote: > At 10/31/2012 09:12 AM, Marcelo Tosatti Wrote: >> It has been asked earlier why a simple virtio device is not usable >> for this (with no response IIRC). > > 1. We can't use virtio device when the kernel is booting. So the issue here is the s

[Qemu-devel] [PATCH 34/35] target-i386: Pass X86CPU to kvm_get_mp_state()

2012-10-30 Thread Andreas Färber
Needed for moving halted field to CPUState. Signed-off-by: Andreas Färber --- target-i386/kvm.c |9 + 1 Datei geändert, 5 Zeilen hinzugefügt(+), 4 Zeilen entfernt(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 64b837b..7da816f 100644 --- a/target-i386/kvm.c +++ b/target

[Qemu-devel] [PATCH 20/35] cpus: Pass CPUState to qemu_cpu_kick()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed there. Signed-off-by: Andreas Färber --- cpus.c | 13 + exec.c |2 +- hw/ppc.c |4 ++-- hw/ppce500_spin.c |2 +- hw/spapr_rtas.c|5 - hw/sun4m.c |2 +- hw/sun4u.c |2

Re: [Qemu-devel] [PATCH 0/8] add paravirtualization hwrng support (v2)

2012-10-30 Thread Anthony Liguori
"H. Peter Anvin" writes: > On 10/30/2012 04:02 PM, Anthony Liguori wrote: >> >> My take away from all of the various discussions on what the Right Way to >> use virtio-rng is: >> >> 1) /dev/random should always be used as the entropy source (I've left it >> configurable though) >> >> 2)

[Qemu-devel] [PATCH 35/35] target-i386: Pass X86CPU to kvm_handle_halt()

2012-10-30 Thread Andreas Färber
Needed for moving interrupt_request and halted fields to CPUState. Signed-off-by: Andreas Färber --- target-i386/kvm.c |7 +-- 1 Datei geändert, 5 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 7da816f..9ccbcb5 100644 --- a/target-i3

[Qemu-devel] [PATCH 08/35] cpus: Pass CPUState to qemu_cpu_kick_thread()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed there. Signed-off-by: Andreas Färber Reviewed-by: Igor Mammedov --- cpus.c |9 - 1 Datei geändert, 4 Zeilen hinzugefügt(+), 5 Zeilen entfernt(-) diff --git a/cpus.c b/cpus.c index 1f3ac91..3946d49 100644 --- a/cpus.c +++ b/cpus.c @@ -844,9 +844,8 @@

[Qemu-devel] [PATCH 09/35] cpu: Move created field to CPUState

2012-10-30 Thread Andreas Färber
Change its type to bool. Signed-off-by: Andreas Färber --- cpu-defs.h |1 - cpus.c | 13 +++-- include/qemu/cpu.h |2 ++ 3 Dateien geändert, 9 Zeilen hinzugefügt(+), 7 Zeilen entfernt(-) diff --git a/cpu-defs.h b/cpu-defs.h index a796577..3b8bc20 100644 ---

[Qemu-devel] [PATCH 32/35] cpus: Pass CPUState to run_on_cpu()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed. Move the declaration to include/qemu/cpu.h and add documentation. Signed-off-by: Andreas Färber --- cpu-all.h|1 - cpus.c |3 +-- hw/kvm/apic.c|2 +- hw/kvmvapic.c|6 -- hw/ppce500_spin.c|2 +

[Qemu-devel] [PATCH 11/35] ppce500_spin: Store PowerPCCPU in SpinKick

2012-10-30 Thread Andreas Färber
Needed for moving stopped field to CPUState. Signed-off-by: Andreas Färber --- hw/ppce500_spin.c |6 +++--- 1 Datei geändert, 3 Zeilen hinzugefügt(+), 3 Zeilen entfernt(-) diff --git a/hw/ppce500_spin.c b/hw/ppce500_spin.c index 55aa9dc..04e7e65 100644 --- a/hw/ppce500_spin.c +++ b/hw/ppce5

[Qemu-devel] [PATCH 31/35] target-i386: Pass X86CPU to cpu_x86_inject_mce()

2012-10-30 Thread Andreas Färber
Needed for changing run_on_cpu() argument to CPUState. Signed-off-by: Andreas Färber --- monitor.c|6 -- target-i386/cpu.h|2 +- target-i386/helper.c |3 ++- target-i386/kvm.c|2 +- 4 Dateien geändert, 8 Zeilen hinzugefügt(+), 5 Zeilen entfernt(-) diff --

[Qemu-devel] [PATCH 03/35] target-i386: Initialize APIC at CPU level

2012-10-30 Thread Andreas Färber
From: Igor Mammedov (L)APIC is a part of cpu [1] so move APIC initialization inside of x86_cpu object. Since cpu_model and override flags currently specify whether APIC should be created or not, APIC creation&initialization is moved into x86_cpu_apic_init() which is called from x86_cpu_realize().

[Qemu-devel] [PATCH 15/35] cpu: Move halt_cond to CPUState

2012-10-30 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-defs.h |1 - cpus.c | 22 -- include/qemu/cpu.h |1 + 3 Dateien geändert, 13 Zeilen hinzugefügt(+), 11 Zeilen entfernt(-) diff --git a/cpu-defs.h b/cpu-defs.h index 83bf108..76c76f6 100644 --- a/cpu-defs.h ++

[Qemu-devel] [PATCH 13/35] cpus: Pass CPUState to cpu_is_stopped()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed there. Also change the return type to bool. Signed-off-by: Andreas Färber --- cpu-all.h |1 - cpus.c |4 +--- include/qemu/cpu.h | 11 +++ target-i386/kvm.c |4 ++-- 4 Dateien geändert, 14 Zeilen hinzugefügt(+), 6 Zeil

[Qemu-devel] [PATCH 19/35] target-ppc: Rename kvm_kick_{env => cpu} and pass PowerPCCPU

2012-10-30 Thread Andreas Färber
Needed for changing qemu_cpu_kick() argument type to CPUState. Signed-off-by: Andreas Färber --- target-ppc/kvm.c |8 ++-- 1 Datei geändert, 6 Zeilen hinzugefügt(+), 2 Zeilen entfernt(-) diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c index 5cbe98a..d7d8e8f 100644 --- a/target-ppc/kvm.

[Qemu-devel] [PATCH 14/35] cpus: Pass CPUState to cpu_can_run()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed there. Also change its return type to bool. Signed-off-by: Andreas Färber --- cpus.c | 14 ++ 1 Datei geändert, 6 Zeilen hinzugefügt(+), 8 Zeilen entfernt(-) diff --git a/cpus.c b/cpus.c index 0721a96..45877ee 100644 --- a/cpus.c +++ b/cpus.c @@ -

Re: [Qemu-devel] [PATCH v11] kvm: notify host when the guest is panicked

2012-10-30 Thread Wen Congyang
At 10/31/2012 09:12 AM, Marcelo Tosatti Wrote: > On Thu, Oct 25, 2012 at 11:42:32AM +0800, Hu Tao wrote: >> We can know the guest is panicked when the guest runs on xen. >> But we do not have such feature on kvm. >> >> Another purpose of this feature is: management app(for example: >> libvirt) can

Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Handle new kernel uapi/ directories

2012-10-30 Thread Marcelo Tosatti
On Tue, Oct 30, 2012 at 07:33:16PM +0100, Jan Kiszka wrote: > On 2012-10-30 10:48, Peter Maydell wrote: > > Ping? > > > > (patchwork url http://patchwork.ozlabs.org/patch/193125/) > > Should obviously go in soon. > > Jan Applied to uq/master, thanks.

Re: [Qemu-devel] [PATCH v11] kvm: notify host when the guest is panicked

2012-10-30 Thread Marcelo Tosatti
On Thu, Oct 25, 2012 at 11:42:32AM +0800, Hu Tao wrote: > We can know the guest is panicked when the guest runs on xen. > But we do not have such feature on kvm. > > Another purpose of this feature is: management app(for example: > libvirt) can do auto dump when the guest is panicked. If managemen

Re: [Qemu-devel] [PATCH 29/35] cpus: Pass CPUState to [qemu_]cpu_has_work()

2012-10-30 Thread Richard Henderson
On 2012-10-31 12:00, Andreas Färber wrote: > target-alpha/cpu.c |2 +- > target-alpha/cpu.h |4 +++- Acked-by: Richard Henderson r~

[Qemu-devel] [PATCH 01/35] target-i386: cpu_x86_register(): report error from property setter

2012-10-30 Thread Andreas Färber
From: Igor Mammedov Signed-off-by: Igor Mammedov Reviewed-by: Eduardo Habkost Signed-off-by: Andreas Färber --- target-i386/cpu.c |3 ++- 1 Datei geändert, 2 Zeilen hinzugefügt(+), 1 Zeile entfernt(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index d4f2e65..f3a3121 100644 --- a/

Re: [Qemu-devel] [PATCH 0/8] add paravirtualization hwrng support (v2)

2012-10-30 Thread H. Peter Anvin
On 10/30/2012 04:02 PM, Anthony Liguori wrote: My take away from all of the various discussions on what the Right Way to use virtio-rng is: 1) /dev/random should always be used as the entropy source (I've left it configurable though) 2) I think the Right Way to configure virtio-rng is

Re: [Qemu-devel] [PATCH 1/4 v3] vmware_vga: Coding style cleanup

2012-10-30 Thread BALATON Zoltan
On Tue, 30 Oct 2012, Blue Swirl wrote: The patches look OK. But they don't apply anymore, please rebase. Sorry, I did not notice that as they merged all right for me. I've sent a rebased set to the list now. Thanks, BALATON Zoltan

[Qemu-devel] [PATCH 4/4 v4] vmware_vga: Allow simple drivers to work without using the fifo

2012-10-30 Thread BALATON Zoltan
Postpone stopping the dirty log to the point where the command fifo is configured to allow drivers which don't use the fifo to work too. (Without this the picture rendered into the vram never got to the screen and the DIRECT_VRAM option meant to support this case was removed a year ago.) Signed-o

[Qemu-devel] [PATCH 3/4 v4] vmware_vga: Return a value for FB_SIZE before the device is enabled

2012-10-30 Thread BALATON Zoltan
According to the documentation drivers using this device should read FB_SIZE before enabling the device to know what memory to map. This would not work if we return 0 before enabled. The docs also mention reading SVGA_REG_DEPTH but not writing it. (Only SVGA_REG_BITS_PER_PIXEL can be written but w

[Qemu-devel] [PATCH 2/4 v4] vmware_vga: Remove duplicated info from local state

2012-10-30 Thread BALATON Zoltan
Removed info from vmsvga_state that is available from elsewhere and thus was duplicated here unnecessarily. Signed-off-by: BALATON Zoltan --- console.h | 20 +++ hw/vmware_vga.c | 156 +++ 2 files changed, 84 insertions(+), 92 del

[Qemu-devel] [PATCH 1/4 v4] vmware_vga: Coding style cleanup

2012-10-30 Thread BALATON Zoltan
Fix coding style as suggested by checkpatch.pl Signed-off-by: BALATON Zoltan --- hw/vmware_vga.c | 283 ++- 1 file changed, 156 insertions(+), 127 deletions(-) v4: rebased to apply to current diff --git a/hw/vmware_vga.c b/hw/vmware_vga.c

[Qemu-devel] [PATCH 30/35] target-i386: Pass X86CPU to kvm_mce_inject()

2012-10-30 Thread Andreas Färber
Needed for changing cpu_x86_inject_mce() argument to X86CPU. Signed-off-by: Andreas Färber [AF: Rebased onto hwaddr] --- target-i386/kvm.c |8 +--- 1 Datei geändert, 5 Zeilen hinzugefügt(+), 3 Zeilen entfernt(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index a3491a4..5bf2f89c

[Qemu-devel] [PATCH 28/35] spapr: Pass PowerPCCPU to hypercalls

2012-10-30 Thread Andreas Färber
Needed for changing cpu_has_work() argument type to CPUState, used in h_cede(). Signed-off-by: Andreas Färber --- hw/spapr.h |2 +- hw/spapr_hcall.c | 38 +- hw/spapr_iommu.c |2 +- hw/spapr_llan.c | 10 +- hw/spapr_vio.c | 10 ++

[Qemu-devel] [PATCH 17/35] cpus: Pass CPUState to qemu_tcg_init_vcpu()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed. Signed-off-by: Andreas Färber --- cpus.c |7 ++- 1 Datei geändert, 2 Zeilen hinzugefügt(+), 5 Zeilen entfernt(-) diff --git a/cpus.c b/cpus.c index 068fa12..5f91523 100644 --- a/cpus.c +++ b/cpus.c @@ -991,11 +991,8 @@ void resume_all_vcpus(void) }

[Qemu-devel] [PATCH 26/35] target-ppc: Pass PowerPCCPU to cpu_ppc_hypercall

2012-10-30 Thread Andreas Färber
Adapt emulate_spapr_hypercall() accordingly. Needed for changing spapr_hypercall() argument type to PowerPCCPU. Signed-off-by: Andreas Färber --- hw/spapr.c |4 +++- target-ppc/cpu.h |2 +- target-ppc/excp_helper.c |4 ++-- 3 Dateien geändert, 6 Zeilen hinzugef

[Qemu-devel] [PATCH 21/35] cpu: Move queued_work_{first, last} to CPUState

2012-10-30 Thread Andreas Färber
Signed-off-by: Andreas Färber --- cpu-defs.h |1 - cpus.c | 19 ++- include/qemu/cpu.h |1 + 3 Dateien geändert, 11 Zeilen hinzugefügt(+), 10 Zeilen entfernt(-) diff --git a/cpu-defs.h b/cpu-defs.h index 76c76f6..b30a8e9 100644 --- a/cpu-defs.h +++ b

[Qemu-devel] [PATCH 05/35] apic: Store X86CPU in APICCommonState

2012-10-30 Thread Andreas Färber
Prepares for using a link<> property to connect APIC with CPU and for changing the CPU APIs to CPUState. Resolve Coding Style warnings by moving the closing parenthesis of foreach_apic() macro to next line. Signed-off-by: Andreas Färber Reviewed-by: Igor Mammedov --- hw/apic.c | 38

[Qemu-devel] [PATCH 18/35] ppc: Pass PowerPCCPU to {ppc6xx, ppc970, power7, ppc40x, ppce500}_set_irq()

2012-10-30 Thread Andreas Färber
Needed for changing qemu_cpu_kick() argument type to CPUState and for moving halted field into CPUState. Signed-off-by: Andreas Färber --- hw/ppc.c | 55 +++ 1 Datei geändert, 35 Zeilen hinzugefügt(+), 20 Zeilen entfernt(-) diff --git a/hw/p

[Qemu-devel] [PATCH 29/35] cpus: Pass CPUState to [qemu_]cpu_has_work()

2012-10-30 Thread Andreas Färber
For target-mips also change the return type to bool. Make include paths for cpu-qom.h consistent for alpha and unicore32. Signed-off-by: Andreas Färber [AF: Updated new target-openrisc function accordingly] --- cpu-all.h |2 -- cpu-exec.c |8 +++- cpus.c

[Qemu-devel] [PATCH 23/35] cpus: Pass CPUState to qemu_wait_io_event_common()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed there. Signed-off-by: Andreas Färber --- cpus.c | 12 +--- 1 Datei geändert, 5 Zeilen hinzugefügt(+), 7 Zeilen entfernt(-) diff --git a/cpus.c b/cpus.c index e40823c..6baf2bc 100644 --- a/cpus.c +++ b/cpus.c @@ -687,10 +687,8 @@ static void flush_queue

[Qemu-devel] [PATCH 22/35] cpus: Pass CPUState to flush_queued_work()

2012-10-30 Thread Andreas Färber
CPUArchState is no longer needed there. Signed-off-by: Andreas Färber --- cpus.c |5 ++--- 1 Datei geändert, 2 Zeilen hinzugefügt(+), 3 Zeilen entfernt(-) diff --git a/cpus.c b/cpus.c index 307c1f2..e40823c 100644 --- a/cpus.c +++ b/cpus.c @@ -670,9 +670,8 @@ void run_on_cpu(CPUArchState *e

Re: [Qemu-devel] 64-on-32 TCG broken

2012-10-30 Thread Aurelien Jarno
On Tue, Oct 30, 2012 at 11:24:34PM +0100, Stefan Weil wrote: > Am 30.10.2012 09:15, schrieb Paolo Bonzini: > >Il 29/10/2012 19:29, Aurelien Jarno ha scritto: > >>On Mon, Oct 29, 2012 at 06:53:14PM +0100, Paolo Bonzini wrote: > Known-good commit: 8473f377393219390ea6f2d8d450a2b054bb823e > Kn

[Qemu-devel] [PATCH 5/8] rng-egd: introduce EGD compliant RNG backend

2012-10-30 Thread Anthony Liguori
This backend talks EGD to a CharDriverState. A typical way to invoke this would be: qemu -chardev socket,host=localhost,port=1024,id=chr0 \ -object rng-egd,chardev=chr0,id=egd0 \ -device virtio-rng-pci,rng=egd0 Signed-off-by: Anthony Liguori --- backends/Makefile.objs | 2 +- backe

[Qemu-devel] [PATCH 4/8] rng-random: add an RNG backend that uses /dev/random (v2)

2012-10-30 Thread Anthony Liguori
The filename can be overridden but it expects a non-blocking source of entropy. A typical invocation would be: qemu -object rng-random,id=rng0 -device virtio-rng-pci,rng=rng0 This can also be used with /dev/urandom by using the command line: qemu -object rng-random,filename=/dev/urandom,id=rng0

[Qemu-devel] [PATCH 8/8] virtio-rng-pci: create a default backend if none exists

2012-10-30 Thread Anthony Liguori
This allows you to specify: $ qemu -device virtio-rng-pci And things will Just Work with a reasonable default. Signed-off-by: Anthony Liguori --- hw/virtio-pci.c | 13 + hw/virtio-rng.h | 2 ++ 2 files changed, 15 insertions(+) diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c ind

[Qemu-devel] [PATCH 7/8] virtio-rng: add rate limiting support

2012-10-30 Thread Anthony Liguori
This adds parameters to virtio-rng-pci to allow rate limiting the entropy a guest receives. An example command line: $ qemu -device virtio-rng-pci,max-bytes=1024,period=1000 Would limit entropy collection to 1Kb/s. Signed-off-by: Anthony Liguori --- hw/virtio-pci.c | 7 +++ hw/virtio-rng

[Qemu-devel] [PATCH] target-alpha: Use TCG_CALL_NO_WG

2012-10-30 Thread Richard Henderson
Mark helper functions that raise exceptions, but otherwise do not change TCG register state, with TCG_CALL_NO_WG. Signed-off-by: Richard Henderson --- target-alpha/helper.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/target-alpha/helper.h b/target-a

Re: [Qemu-devel] 64-on-32 TCG broken

2012-10-30 Thread Aurelien Jarno
On Tue, Oct 30, 2012 at 11:24:34PM +0100, Stefan Weil wrote: > Am 30.10.2012 09:15, schrieb Paolo Bonzini: > >Il 29/10/2012 19:29, Aurelien Jarno ha scritto: > >>On Mon, Oct 29, 2012 at 06:53:14PM +0100, Paolo Bonzini wrote: > Known-good commit: 8473f377393219390ea6f2d8d450a2b054bb823e > Kn

  1   2   3   4   >