Re: [Qemu-devel] [PATCH] add support for ATA_PASSTHROUGH_xx scsi command

2012-08-02 Thread Paolo Bonzini
Il 03/08/2012 06:02, Cong Meng ha scritto: > Correct the command names of opcode 0x85 and 0xa1, and calculate > their xfer size from CDB. > > ChangeLog: > v2: For opcode 0xa1 on TYPE_ROM device, do not calc the xfer size > v3: Complete xfer calculation of all situations > > Signed-off-by: Cong Me

Re: [Qemu-devel] Cirrus bugs vs endian: how two bugs cancel each other out

2012-08-02 Thread Alon Levy
On Wed, Aug 01, 2012 at 02:22:37PM -0500, Anthony Liguori wrote: > Andreas Färber writes: > > > Am 30.07.2012 18:19, schrieb Alon Levy: > >> On Mon, Jul 30, 2012 at 09:54:27PM +1000, Benjamin Herrenschmidt wrote: > >>> On Mon, 2012-07-30 at 14:25 +0300, Avi Kivity wrote: > >>> > [...] why no

[Qemu-devel] [PATCH v2][RFC] Add compare subcommand for qemu-img

2012-08-02 Thread Miroslav Rezanina
This is second version of patch adding compare subcommand that compares two images. Compare has following criteria: - only data part is compared - unallocated sectors are not read - in case of different image size, exceeding part of bigger disk has to be zeroed/unallocated to compare rest -

Re: [Qemu-devel] [PATCH v3] memory: add -machine dump-guest-core=on|off

2012-08-02 Thread Markus Armbruster
Jason Baron writes: > Add a new '[,dump-guest-core=on|off]' option to the '-machine' option. When > 'dump-guest-core=off' is specified, guest memory is omitted from the core > dump. > The default behavior continues to be to include guest memory when a core dump > is > triggered. In my testing,

Re: [Qemu-devel] [PATCH 1/6 v11] docs: spec for add-cow file format

2012-08-02 Thread Dong Xu Wang
On Thu, Aug 2, 2012 at 6:44 PM, Stefan Hajnoczi wrote: > On Thu, Aug 2, 2012 at 8:09 AM, Dong Xu Wang > wrote: >> On Wed, Aug 1, 2012 at 9:55 PM, Stefan Hajnoczi wrote: >>> On Tue, Jul 31, 2012 at 5:51 PM, Dong Xu Wang >>> wrote: +test.raw may be larger than ubuntu.img, in that case, the

[Qemu-devel] [PATCH 3/3] Enable kvm emulated watchdog

2012-08-02 Thread Bharat Bhushan
Enable the KVM emulated watchdog if KVM supports (use the capability enablement in watchdog handler). Also watchdog exit (KVM_EXIT_WATCHDOG) handling is added. Watchdog state machine is cleared whenever VM state changes to running. This is to handle the cases like return from debug halt etc. Signe

[Qemu-devel] [PATCH 2/3] Registered timer reset handler

2012-08-02 Thread Bharat Bhushan
Reset qemu timers when guest reset. Signed-off-by: Bharat Bhushan --- hw/ppc_booke.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/hw/ppc_booke.c b/hw/ppc_booke.c index d51e7fa..837a5b6 100644 --- a/hw/ppc_booke.c +++ b/hw/ppc_booke.c @@ -231,6 +231,16 @@ v

[Qemu-devel] [PATCH 1/3] Synchronize the linux-headers

2012-08-02 Thread Bharat Bhushan
This also includes the changes for watchdog emulation Signed-off-by: Bharat Bhushan --- linux-headers/asm-powerpc/kvm_para.h |7 --- linux-headers/asm-s390/kvm.h |2 +- linux-headers/asm-s390/kvm_para.h|2 +- linux-headers/asm-x86/kvm.h |1 + linux-header

[Qemu-devel] [Bug 812398] Re: powerpc 7450 MMU initialization broken

2012-08-02 Thread Samuel Bronson
** Tags added: mmu ppc -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/812398 Title: powerpc 7450 MMU initialization broken Status in QEMU: New Bug description: The 7540 family of PPCs' MMU can

[Qemu-devel] [RFC 11/19] rename x86_def_t to X86CPUDefinition

2012-08-02 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 43 ++- 1 file changed, 22 insertions(+), 21 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index c96300e..4f67ecc 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -222,8 +222

[Qemu-devel] [PATCH v6 2/7] monitor: rename keyname '<' to 'less'

2012-08-02 Thread Amos Kong
There are many maps of keycode 0x56 in pc-bios/keymaps/* pc-bios/keymaps/common:less 0x56 pc-bios/keymaps/common:greater 0x56 shift pc-bios/keymaps/common:bar 0x56 altgr pc-bios/keymaps/common:brokenbar 0x56 shift altgr This patch just renames '<' to 'less', QAPI would add new variable by

[Qemu-devel] [PATCH v6 3/7] hmp: rename arguments

2012-08-02 Thread Amos Kong
Rename 'string' to 'keys', rename 'hold_time' to 'hold-time'. Signed-off-by: Amos Kong --- hmp-commands.hx |2 +- monitor.c | 14 +++--- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/hmp-commands.hx b/hmp-commands.hx index 4041a9b..2891d48 100644 --- a/hmp-co

[Qemu-devel] [RFC 08/19] i386: extract CPU model lookup to a separate function

2012-08-02 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 24 +--- 1 file changed, 17 insertions(+), 7 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index dd2dc45..e5be586 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1231,16 +1231,10 @@ error:

[Qemu-devel] [RFC 10/19] move CPU object creation to cpu.c

2012-08-02 Thread Eduardo Habkost
cpu.c handles the class registration and parsing of the CPU model string, so moving it there will make it easier to refactor the CPU model string parsing and CPU object creation. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c| 17 + target-i386/cpu.h| 1 + target-i

[Qemu-devel] [PATCH v2 0/2] trivial fix of qemu-sockets.c

2012-08-02 Thread Amos Kong
Those patches fix trivial issues which were found in the second review. Amos Kong (2): remove unused include of error.h socket: clean up redundant assignment qemu-sockets.c |1 - qemu_socket.h |1 - 2 files changed, 0 insertions(+), 2 deletions(-) Changes from v1: - drop wrong patc

[Qemu-devel] [Bug 501141] Re: qemu powerpc target crashes on "hello world"

2012-08-02 Thread Samuel Bronson
** Tags added: ppc -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/501141 Title: qemu powerpc target crashes on "hello world" Status in QEMU: New Bug description: Downloaded qemu and associated

[Qemu-devel] [RFC 19/19] HACK: late CPU class initialization

2012-08-02 Thread Eduardo Habkost
This is just a hack to make the experimental CPU class code work without removing the cpudef support. After we remove the cpudef support, we can simply use type_init() again, and register only the builtin CPU models as CPU classes. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 7 ++

[Qemu-devel] [RFC 00/19] i386 CPU code cleanup + CPU model classes

2012-08-02 Thread Eduardo Habkost
Hi, I started looking at how the current CPU models could be converted to QOM classes, and ended up implementing this. It's completely experimental and untested, and the final step involves a hack, that is necessary until we remove support for the cpudef config section. The first 7 patches are pu

[Qemu-devel] [RFC 06/19] i386: cpu: extract the full feature list before parsing it

2012-08-02 Thread Eduardo Habkost
The parsing of the feature list string will be moved to a separate function, so first extract the full feature list, then parse it. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c in

[Qemu-devel] [RFC 04/19] rename cpu_x86_find_by_name to x86_cpu_build_from_name

2012-08-02 Thread Eduardo Habkost
The function does a lot more than just "finding" the CPU model, so name it accordingly. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 1154a34..be04e4b 100644 --- a/target-i

[Qemu-devel] [PATCH v6 5/7] add the QKeyCode enum and the key_defs table

2012-08-02 Thread Amos Kong
key_defs[] in monitor.c is a mapping table of keys and keycodes, this patch added a QKeyCode enum and a new key_defs table, Key's index in the enmu is same as keycode's index in new key_defs[]. Signed-off-by: Amos Kong --- input.c | 146 +

[Qemu-devel] [PATCH] add support for ATA_PASSTHROUGH_xx scsi command

2012-08-02 Thread Cong Meng
Correct the command names of opcode 0x85 and 0xa1, and calculate their xfer size from CDB. ChangeLog: v2: For opcode 0xa1 on TYPE_ROM device, do not calc the xfer size v3: Complete xfer calculation of all situations Signed-off-by: Cong Meng --- hw/scsi-bus.c | 90

[Qemu-devel] [RFC 09/19] i386: reorder object setup on cpu_x86_init()

2012-08-02 Thread Eduardo Habkost
The object creation will be moved to cpu.c, with all the rest of the CPU model handling. Signed-off-by: Eduardo Habkost --- target-i386/helper.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/target-i386/helper.c b/target-i386/helper.c index b748d90..5a7eb56 100644 -

[Qemu-devel] [RFC 14/19] extract CPU object field initialization from cpu_x86_register()

2012-08-02 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 27 --- 1 file changed, 20 insertions(+), 7 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 5efbe41..f30e621 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1411,17 +1411,12 @@ void x8

[Qemu-devel] [RFC 17/19] kill cpu_x86_build_from_name()

2012-08-02 Thread Eduardo Habkost
We will reorder some code in the CPU object initialization, so move all the cpu_x86_build_from_name() code inside cpu_x86_create(). Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 48 ++-- 1 file changed, 18 insertions(+), 30 deletions(-) diff

[Qemu-devel] [RFC 16/19] kill cpu_x86_register()

2012-08-02 Thread Eduardo Habkost
We will need to reorder parts the CPU object creation code, so move cpu_x86_register() code into the cpu_x86_create() function to make it possible. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 25 - target-i386/cpu.h | 1 - 2 files changed, 8 insertions(+), 18

[Qemu-devel] [PATCH v6 4/7] qapi: generate list struct and visit_list for enum

2012-08-02 Thread Amos Kong
Currently, if define an 'enum' and use it in one command's data, List struct for enum could not be generated, but it's used in qmp function. For example: KeyCodesList could not be generated. >>> qapi-schema.json: { 'enum': 'KeyCodes', 'data': [ 'shift', 'alt' ... ] } { 'command': 'sendkey', 'd

[Qemu-devel] [RFC 03/19] i386: x86_def_t: rename 'flags' field

2012-08-02 Thread Eduardo Habkost
The field is being used for a single purpose: to indicate if the CPU model is a builtin one. So, instead of an opaque and confusing 'flags' field name, use a boolean 'is_builtin' field to indicate if the model is a built-in one. No behavior change, just a field rename. Signed-off-by: Eduardo Habk

[Qemu-devel] [PATCH v2 1/2] remove unused include of error.h

2012-08-02 Thread Amos Kong
Signed-off-by: Amos Kong --- qemu_socket.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/qemu_socket.h b/qemu_socket.h index 4689ff3..1a2f517 100644 --- a/qemu_socket.h +++ b/qemu_socket.h @@ -27,7 +27,6 @@ int inet_aton(const char *cp, struct in_addr *ia); #endif /* !

[Qemu-devel] [RFC 15/19] cpu_x86_create: move error handling to end of function

2012-08-02 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index f30e621..7821331 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1489,12 +1489,15 @@ X86CPU *cpu_x86_create(const

[Qemu-devel] [RFC 13/19] move X86CPUDefinition to cpu-qom.h

2012-08-02 Thread Eduardo Habkost
From: Eduardo Habkost The struct will be used by X86CPUClass, too. Signed-off-by: Eduardo Habkost --- target-i386/cpu-qom.h | 20 target-i386/cpu.c | 19 --- 2 files changed, 20 insertions(+), 19 deletions(-) diff --git a/target-i386/cpu-qom.h b/target

[Qemu-devel] [RFC 05/19] cpu_x86_build_from_name: use strtok_r()

2012-08-02 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index be04e4b..154a01f 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1083,8 +1083,9 @@ static int cpu_x86_build_from_n

[Qemu-devel] [RFC 12/19] create struct X86CPUModelTableEntry

2012-08-02 Thread Eduardo Habkost
X86CPUDefinition will be reused in other places, so remove the fields that are specific to the CPU model name table ('name', 'is_builtin', and 'next') and put them into a separate struct. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 132 +

[Qemu-devel] [RFC 18/19] register a class for each CPU model

2012-08-02 Thread Eduardo Habkost
The trick here is to replace only the cpu_x86_find_by_name() logic and nothing else. So, the only difference in relation to the previous code is that instead of looking at the CPU model table on cpu_x86_create()/cpu_x86_find_by_name(), we just use the right CPU class, that will already contain the

[Qemu-devel] [RFC 07/19] i386: cpu: extract parsing of feature strings to separate function

2012-08-02 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 62 ++- 1 file changed, 39 insertions(+), 23 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 7af80a0..dd2dc45 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@

[Qemu-devel] [Bug 788697] Re: [PowerPC] [patch] mtmsr does not preserve high bits of MSR

2012-08-02 Thread Samuel Bronson
** Tags added: ppc ** Tags added: ppc64 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/788697 Title: [PowerPC] [patch] mtmsr does not preserve high bits of MSR Status in QEMU: New Bug descripti

[Qemu-devel] [Bug 1031698] Re: Missing PPC TRACE exception on a branch

2012-08-02 Thread Samuel Bronson
** Tags added: ppc -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1031698 Title: Missing PPC TRACE exception on a branch Status in QEMU: New Bug description: I am using qemu-1.1.1-1 to emulate

[Qemu-devel] [RFC 02/19] x86_cpudef_setup: coding style change

2012-08-02 Thread Eduardo Habkost
Make source code lines shorter. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 15 +-- 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 9414d75..d546534 100644 --- a/target-i386/cpu.c +++ b/target-i386/cpu.c @@ -1526,20

[Qemu-devel] [RFC 01/19] target-i386/cpu.c: coding style fixes

2012-08-02 Thread Eduardo Habkost
Changes to make checkpatch.pl happy: - Use "sizeof(...)" instead of "sizeof (...)"; - Spaces around ":"; - Use spaces instead of tabs; - Braces on if statements. Signed-off-by: Eduardo Habkost --- target-i386/cpu.c | 66 ++- 1 file changed, 36

[Qemu-devel] [Bug 636446] Re: prep ppc "machine" no more working

2012-08-02 Thread Samuel Bronson
** Tags added: ppc -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/636446 Title: prep ppc "machine" no more working Status in QEMU: Confirmed Bug description: I have tried qemu from 0.11 to lat

Re: [Qemu-devel] [PATCH 1/2] Allow QEMUMachine to override reset sequencing

2012-08-02 Thread David Gibson
On Wed, Aug 01, 2012 at 09:37:39PM -0500, Anthony Liguori wrote: > David Gibson writes: > > > At present the qemu_system_reset() function always performs the same basic > > actions on all machines. This includes running all the reset handler > > hooks, however the order in which they run is not

Re: [Qemu-devel] [PATCH 1/2] Allow QEMUMachine to override reset sequencing

2012-08-02 Thread David Gibson
On Thu, Aug 02, 2012 at 05:17:27PM +0200, Paolo Bonzini wrote: > Il 02/08/2012 05:50, Benjamin Herrenschmidt ha scritto: > > > >> > This sort of stuff is either > >> > (1) hard coded in a firmware/flashrom or (2) built dynamically in > >> > firmware. Let's not pretend like we're doing this becau

[Qemu-devel] [PATCH v2 2/2] socket: clean up redundant assignment

2012-08-02 Thread Amos Kong
Signed-off-by: Amos Kong --- qemu-sockets.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/qemu-sockets.c b/qemu-sockets.c index 668fa93..beb2bb6 100644 --- a/qemu-sockets.c +++ b/qemu-sockets.c @@ -284,7 +284,6 @@ int inet_connect_opts(QemuOpts *opts, Error **errp)

[Qemu-devel] [Bug 608107] Re: ppc fails to clear MSR_POW when incurring exception

2012-08-02 Thread Samuel Bronson
** Tags added: ppc -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/608107 Title: ppc fails to clear MSR_POW when incurring exception Status in QEMU: Confirmed Bug description: QEMU VERSION: 0.1

[Qemu-devel] [Bug 393569] Re: qemu cannot load multiple initramfs archives

2012-08-02 Thread Samuel Bronson
** Tags added: initramfs initrd -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/393569 Title: qemu cannot load multiple initramfs archives Status in QEMU: Confirmed Bug description: According t

Re: [Qemu-devel] [Qemu-trivial] [RESEND PATCH 1/3] socket: remove redundant check

2012-08-02 Thread Amos Kong
- Original Message - > Stefan Hajnoczi writes: > > > On Wed, Aug 01, 2012 at 04:59:01PM +0800, Amos Kong wrote: > >> It's aleady in the end of loop, error should be set. > >> > >> Signed-off-by: Amos Kong > >> --- > >> qemu-sockets.c |4 +--- > >> 1 files changed, 1 insertions(+),

[Qemu-devel] [PATCH v6 7/7] qapi: convert sendkey

2012-08-02 Thread Amos Kong
Convert 'sendkey' to use QAPI. QAPI passes key's index of mapping table to qmp_send_key(), not keycode. So we use two functions to convert key/code to index of key_defs, and 'index' will be converted to 'keycode' insider qmp_send_key(). For qmp, QAPI would check invalid key and raise error. For h

[Qemu-devel] [PATCH v6 6/7] add functions to convert key/code to index of mapping table

2012-08-02 Thread Amos Kong
Those two help functions will return Q_KEY_CODE_MAX if the code/key is invalid. Signed-off-by: Amos Kong --- console.h |5 + input.c | 38 ++ 2 files changed, 43 insertions(+), 0 deletions(-) diff --git a/console.h b/console.h index 4334db5..c702b

[Qemu-devel] [PATCH v6 1/7] fix doc of using raw values with sendkey

2012-08-02 Thread Amos Kong
(qemu) sendkey a (qemu) sendkey 0x1e (qemu) sendkey #0x1e unknown key: '#0x1e' The last command doesn't work, '#' is not requested before raw values. And the raw value in decimal format is also not supported. Signed-off-by: Amos Kong --- hmp-commands.hx |6 +++--- 1 files changed, 3 insert

[Qemu-devel] [PATCH v6 0/7] convert sendkey to qapi

2012-08-02 Thread Amos Kong
This series converted 'sendkey' command to qapi. The raw value in hexadecimal format is not supported by 'send-key' of qmp. Amos Kong (7): fix doc of using raw values with sendkey monitor: rename keyname '<' to 'less' hmp: rename arguments qapi: generate list struct and visit_list for enum

Re: [Qemu-devel] [PATCH 2/2] pseries: Use new hook to correct reset sequence

2012-08-02 Thread David Gibson
On Thu, Aug 02, 2012 at 02:40:19PM -0500, Anthony Liguori wrote: > Andreas Färber writes: > > > Am 02.08.2012 20:29, schrieb Anthony Liguori: > >> Andreas Färber writes: > >> > >>> Anthony was favoring moving reset code out of machines and expressed > >>> dislike for looping through CPUs, which

Re: [Qemu-devel] [PATCH 2/2] pseries: Use new hook to correct reset sequence

2012-08-02 Thread David Gibson
On Thu, Aug 02, 2012 at 05:44:49PM +0200, Andreas Färber wrote: > Am 02.08.2012 04:10, schrieb David Gibson: > > A number of things need to occur during reset of the PAPR paravirtualized > > platform in a specific order. For example, the hash table needs to be > > cleared before the CPUs are reset

Re: [Qemu-devel] [PATCH 1/2] Allow QEMUMachine to override reset sequencing

2012-08-02 Thread David Gibson
On Thu, Aug 02, 2012 at 06:00:06PM +0300, Lluís Vilanova wrote: > David Gibson writes: > [...] > > diff --git a/vl.c b/vl.c > > index 9fea320..ac47a7c 100644 > > --- a/vl.c > > +++ b/vl.c > > @@ -1410,6 +1410,14 @@ void qemu_system_reset(bool report) > > cpu_synchronize_all_post_reset(); > >

Re: [Qemu-devel] [PATCH v5 6/6] qapi: convert sendkey

2012-08-02 Thread Amos Kong
- Original Message - > On Thu, 26 Jul 2012 12:49:01 +0800 > Amos Kong wrote: > > > Convert 'sendkey' to use QAPI. > > > > Keys' indexes in the enmu are same as keycodes' indexes in the > > key_defs[], index_from_code() and index_from_key() will return > > Q_KEY_CODE_MAX if the code/key i

Re: [Qemu-devel] [PATCH 4/4] Enable kvm emulated watchdog

2012-08-02 Thread Bhushan Bharat-R65777
> -Original Message- > From: Wood Scott-B07421 > Sent: Friday, August 03, 2012 2:02 AM > To: Bhushan Bharat-R65777 > Cc: Wood Scott-B07421; Alexander Graf; qemu-...@nongnu.org List; kvm- > p...@vger.kernel.org; qemu-devel qemu-devel; KVM list > Subject: Re: [PATCH 4/4] Enable kvm emulated

[Qemu-devel] KVM Forum 2012 Call For Participation

2012-08-02 Thread Chris Wright
For some reason I'm not seeing this on the qemu list, so here's a fwd - Forwarded message from KVM Forum 2012 Program Committee - Date: Fri, 27 Jul 2012 16:31:45 -0700 From: KVM Forum 2012 Program Committee To: k...@vger.kernel.org, libvir-l...@redhat.com, qemu-devel@nongnu.org,

Re: [Qemu-devel] [PATCH v5 6/6] block: Enable qemu_open/close to work with fd sets

2012-08-02 Thread Corey Bryant
On 07/23/2012 09:14 AM, Corey Bryant wrote: On 07/23/2012 09:08 AM, Corey Bryant wrote: When qemu_open is passed a filename of the "/dev/fdset/nnn" format (where nnn is the fdset ID), an fd with matching access mode flags will be searched for within the specified monitor fd set. If the fd i

Re: [Qemu-devel] [PATCH 01/18] smbios: Add a function to directly add an entry

2012-08-02 Thread Anthony Liguori
Corey Minyard writes: > On 08/02/2012 01:32 PM, Anthony Liguori wrote: >> Corey Minyard writes: >> >>> On 08/01/2012 09:40 PM, Anthony Liguori wrote: Corey Minyard writes: > On 08/01/2012 08:15 PM, Kevin O'Connor wrote: > Well, I should also probably add the ACPI name space de

Re: [Qemu-devel] [PATCH 1/2] Allow QEMUMachine to override reset sequencing

2012-08-02 Thread Anthony Liguori
Benjamin Herrenschmidt writes: > On Thu, 2012-08-02 at 17:17 +0200, Paolo Bonzini wrote: >> >> On x86 we do not pre-load the kernel. Neither the kernel/initramfs >> and >> the option ROM that loads the kernel are written in memory, they are >> passed to the guest via fw_cfg. Then the option RO

Re: [Qemu-devel] [PATCH v2 3/6] qapi: avoid reserved keywords

2012-08-02 Thread Luiz Capitulino
On Thu, 02 Aug 2012 10:32:59 +0200 Markus Armbruster wrote: > Blue Swirl writes: > > > Clang compiler complained about use of reserved word 'restrict' in SLIRP > > and QAPI. > > > > Prefix C keywords with "q_", adjust SLIRP accordingly. > > > > Signed-off-by: Blue Swirl > > I like this soluti

Re: [Qemu-devel] [PATCH V6 0/3] Show backing file depth in HMP and QMP

2012-08-02 Thread Eric Blake
On 08/02/2012 02:22 AM, Benoît Canet wrote: > In some setups many backing files and snapshot are chained. > This lead to the formation of huge trees of snapshots all depending > on a common ancestor. > > Hence if something bad happen to this common ancestor all the snapshot > of the tree will be b

Re: [Qemu-devel] [PATCH 1/2] Allow QEMUMachine to override reset sequencing

2012-08-02 Thread Benjamin Herrenschmidt
On Thu, 2012-08-02 at 17:17 +0200, Paolo Bonzini wrote: > > On x86 we do not pre-load the kernel. Neither the kernel/initramfs > and > the option ROM that loads the kernel are written in memory, they are > passed to the guest via fw_cfg. Then the option ROM is loaded by the > BIOS, and loads the

Re: [Qemu-devel] [PATCH V6 0/3] Show backing file depth in HMP and QMP

2012-08-02 Thread Luiz Capitulino
On Thu, 2 Aug 2012 10:22:46 +0200 "Benoît Canet" wrote: > In some setups many backing files and snapshot are chained. > This lead to the formation of huge trees of snapshots all depending > on a common ancestor. > > Hence if something bad happen to this common ancestor all the snapshot > of the

Re: [Qemu-devel] [PATCH] configure: Don't implicitly hardcode list of KVM architectures

2012-08-02 Thread Marcelo Tosatti
On Thu, Aug 02, 2012 at 05:53:37PM +0200, Alexander Graf wrote: > > On 02.08.2012, at 17:46, Andreas Färber wrote: > > > Am 02.08.2012 17:41, schrieb Peter Maydell: > >> Ping? > > > > Alex? Christian? > > Looks good to me, and should probably go in through the kvm queue. Avi, > Marcelo? Sure,

Re: [Qemu-devel] [v2 Patch 8/9]block: Cmd "block_set_hostcache" for dynamic cache change

2012-08-02 Thread Luiz Capitulino
On Thu, 02 Aug 2012 00:14:47 +0530 Supriya Kannery wrote: > New command "block_set_hostcache" added for dynamically changing > host pagecache setting of a block device. > > Usage: > block_set_hostcache > = block device > = on/off > > Example: > (qemu) block_set_hostcache ide0-hd0 o

Re: [Qemu-devel] [PATCH 4/4] Enable kvm emulated watchdog

2012-08-02 Thread Scott Wood
On 08/02/2012 10:56 AM, Bhushan Bharat-R65777 wrote: > > >> -Original Message- >> From: Wood Scott-B07421 >> Sent: Wednesday, August 01, 2012 11:31 PM >> To: Bhushan Bharat-R65777 >> Cc: Alexander Graf; qemu-...@nongnu.org List; kvm-...@vger.kernel.org; qemu- >> devel qemu-devel; KVM list

Re: [Qemu-devel] [v2 Patch 1/9]block: Framework for reopening image files safely

2012-08-02 Thread Luiz Capitulino
On Tue, 31 Jul 2012 03:04:22 +0530 Supriya Kannery wrote: > Struct BDRVReopenState along with three reopen related functions > introduced for handling reopening of images safely. This can be > extended by each of the block drivers to reopen respective > image files. > > Signed-off-by: Supriya Ka

Re: [Qemu-devel] [GIT PULL] tcm_vhost: Initial merge of vhost level target fabric driver

2012-08-02 Thread Nicholas A. Bellinger
Hi Linus, Ping on the initial tcm_vhost merge for-3.6..? I know it's been a busier than usual merge window, but hopefully this one is still in your PULL queue.. Otherwise if there is something else that you'd like to see different from this PULL request, please let us know. Thank you! --nab O

Re: [Qemu-devel] [PATCH v5 6/6] qapi: convert sendkey

2012-08-02 Thread Luiz Capitulino
On Thu, 26 Jul 2012 12:49:01 +0800 Amos Kong wrote: > Convert 'sendkey' to use QAPI. > > Keys' indexes in the enmu are same as keycodes' indexes in the > key_defs[], index_from_code() and index_from_key() will return > Q_KEY_CODE_MAX if the code/key is invalid. > > For qmp, QAPI would check inv

[Qemu-devel] [PATCH v3] memory: add -machine dump-guest-core=on|off

2012-08-02 Thread Jason Baron
Add a new '[,dump-guest-core=on|off]' option to the '-machine' option. When 'dump-guest-core=off' is specified, guest memory is omitted from the core dump. The default behavior continues to be to include guest memory when a core dump is triggered. In my testing, this brought the core dump size down

Re: [Qemu-devel] [PATCH 2/2] pseries: Use new hook to correct reset sequence

2012-08-02 Thread Anthony Liguori
Andreas Färber writes: > Am 02.08.2012 20:29, schrieb Anthony Liguori: >> Andreas Färber writes: >> >>> Anthony was favoring moving reset code out of machines and expressed >>> dislike for looping through CPUs, which my above patch took into >>> account. The ordering issue between CPU and devic

Re: [Qemu-devel] [PATCH 04/47] block: add block_job_query

2012-08-02 Thread Jeff Cody
On 07/31/2012 04:50 AM, Paolo Bonzini wrote: > Il 31/07/2012 10:47, Kevin Wolf ha scritto: >> Why did you convert the initialisation to separate statement? If you >> really want to do this, I think using g_new0 would be safer now, but I >> actually like compound literals better. >>

Re: [Qemu-devel] [RFC 00/12] Qorum disk image corruption resiliency

2012-08-02 Thread Benoît Canet
> I'm not entirely sure I understand the use-case all that well. > > Wouldn't the more typical approach be RAID-5 and the use of parity > instead of relying on voting? > > Quorum doesn't work well with an odd number of disks whereas RAID-5 > does. You also get significantly more usable disk spac

Re: [Qemu-devel] [PATCH 01/18] smbios: Add a function to directly add an entry

2012-08-02 Thread Corey Minyard
On 08/02/2012 01:32 PM, Anthony Liguori wrote: Corey Minyard writes: On 08/01/2012 09:40 PM, Anthony Liguori wrote: Corey Minyard writes: On 08/01/2012 08:15 PM, Kevin O'Connor wrote: Well, I should also probably add the ACPI name space definition for this information, too, and the SMBIOS

[Qemu-devel] [PATCH v2 4/4] vfio: Enable vfio-pci and mark supported

2012-08-02 Thread Alex Williamson
Signed-off-by: Alex Williamson --- MAINTAINERS |5 + configure | 12 hw/i386/Makefile.objs |1 + 3 files changed, 18 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2d219d2..9680d69 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -460,6

[Qemu-devel] [PATCH v2 2/4] Update Linux kernel headers

2012-08-02 Thread Alex Williamson
Based on Linux as of 1a9b4993. Note that vfio.h isn't yet an installed header at this commit, but will be fixed soon. Signed-off-by: Alex Williamson --- linux-headers/asm-s390/kvm.h |2 linux-headers/asm-s390/kvm_para.h |2 linux-headers/asm-x86/kvm.h |1 linux-header

[Qemu-devel] [PATCH v2 1/4] Update kernel header script to include vfio

2012-08-02 Thread Alex Williamson
Signed-off-by: Alex Williamson --- scripts/update-linux-headers.sh |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/update-linux-headers.sh b/scripts/update-linux-headers.sh index 9d2a4bc..270d32b 100755 --- a/scripts/update-linux-headers.sh +++ b/scripts/update-lin

[Qemu-devel] [PATCH v2 0/4] VFIO-based PCI device assignment for QEMU 1.2

2012-08-02 Thread Alex Williamson
v2: - Update kernel header update script - Pull in all changes as a result of that - Note that vfio.h is non yet and installed header, but I'll fix that on my next kernel vfio pull request. - New tag for v2 as noted below. Thanks, Alex VFIO kernel support was just merged into Linux, so I

Re: [Qemu-devel] [PATCH 2/2] pseries: Use new hook to correct reset sequence

2012-08-02 Thread Andreas Färber
Am 02.08.2012 20:29, schrieb Anthony Liguori: > Andreas Färber writes: > >> Anthony was favoring moving reset code out of machines and expressed >> dislike for looping through CPUs, which my above patch took into >> account. The ordering issue between CPU and devices is still unsolved there. >> >

Re: [Qemu-devel] [PATCH 01/18] smbios: Add a function to directly add an entry

2012-08-02 Thread Anthony Liguori
Corey Minyard writes: > On 08/01/2012 09:40 PM, Anthony Liguori wrote: >> Corey Minyard writes: >> >>> On 08/01/2012 08:15 PM, Kevin O'Connor wrote: >>> Well, I should also probably add the ACPI name space definition for this >>> information, too, and the SMBIOS information is not capable of pas

Re: [Qemu-devel] [PATCH 2/2] pseries: Use new hook to correct reset sequence

2012-08-02 Thread Anthony Liguori
Andreas Färber writes: > Am 02.08.2012 04:10, schrieb David Gibson: >> A number of things need to occur during reset of the PAPR paravirtualized >> platform in a specific order. For example, the hash table needs to be >> cleared before the CPUs are reset, so that they initialize their register >

Re: [Qemu-devel] [RFC 00/12] Qorum disk image corruption resiliency

2012-08-02 Thread Anthony Liguori
Benoît Canet writes: > This patchset create a block driver implementing a qorum using three qemu disk > images. Writes are mirrored on the three files. > For the reading part the three files are read at the same time and a vote is > done to determine which is the majoritary qiov version. It then

Re: [Qemu-devel] [PATCH v2] memory: add cmd line option, to omit guest memory from qemu core dump

2012-08-02 Thread Luiz Capitulino
On Wed, 01 Aug 2012 13:21:57 -0500 Anthony Liguori wrote: > > diff --git a/qemu-config.c b/qemu-config.c > > index 5c3296b..5c2c9a9 100644 > > --- a/qemu-config.c > > +++ b/qemu-config.c > > @@ -595,6 +595,10 @@ static QemuOptsList qemu_machine_opts = { > > .name = "dt_compatible", >

[Qemu-devel] [PATCH 2/2] configure: Fix set-but-not-used warning in Xen 4.1 probe

2012-08-02 Thread Peter Maydell
The Xen 4.1 probe never uses the return value from xc_interface_open(), so was provoking a compiler warning on newer gcc. Fix by not bothering to put the return value anywhere. Signed-off-by: Peter Maydell --- configure |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/con

[Qemu-devel] [PATCH 0/2] configure: fix minor issues in Xen checks

2012-08-02 Thread Peter Maydell
These patches fix some minor issues in the Xen checks: * the use of subshells meant that the (not yet committed) "bail out on -Werror failure" check in do_cc didn't actually cause the configure run to stop * the 4.1 probe code had a set-but-unused-variable warning Blue: with these two I'm

[Qemu-devel] [PATCH 1/2] configure: Don't run Xen compile checks in subshells

2012-08-02 Thread Peter Maydell
The Xen compile checks are currently run inside subshells. This is unnecessary and has the effect that if do_cc() exits with an error message then this only causes the subshell to exit, not the whole of configure, which is confusing. Remove the subshells, changing: if ( cat ; compile_prog ) ; the

Re: [Qemu-devel] [PATCH v1 00/34]: add new error format

2012-08-02 Thread Markus Armbruster
Luiz Capitulino writes: > Quick summary first, long introduction afterwards. > >>From the rfc: > > o rebased on top of master > o dropped patch "qapi: qapi.py: allow the "'" character be escaped" > o split patch "qerror: drop qerror_abort()" into two patches > o fixed tcp_start_outgoing_migra

Re: [Qemu-devel] [PATCH 33/34] error: drop error_get_qobject()/error_set_qobject()

2012-08-02 Thread Markus Armbruster
Luiz Capitulino writes: > They are not used anymore. Also drops error_int.h. Squash into PATCH 32?

Re: [Qemu-devel] [PATCH 31/34] error, qerror: pass desc string to error calls

2012-08-02 Thread Markus Armbruster
Luiz Capitulino writes: > This commit changes all QERR_ macros to contain a human message (ie. > the desc string found in qerr_table[]) instead of a json dictionary > in string format. > > Before this commit, error_set() and qerror_report() would receive > a json dictionary in string format and b

Re: [Qemu-devel] [PATCH 29/34] qmp: switch to the new error format on the wire

2012-08-02 Thread Luiz Capitulino
On Thu, 02 Aug 2012 19:12:03 +0200 Markus Armbruster wrote: > Luiz Capitulino writes: > > > IMPORTANT: this BREAKS QMP's compatibility for the error response. > > > > This commit changes QMP's wire protocol to make use of the simpler > > error format introduced by previous commits. > > > > Ther

Re: [Qemu-devel] [PATCH 29/34] qmp: switch to the new error format on the wire

2012-08-02 Thread Markus Armbruster
Luiz Capitulino writes: > IMPORTANT: this BREAKS QMP's compatibility for the error response. > > This commit changes QMP's wire protocol to make use of the simpler > error format introduced by previous commits. > > There are two important (and mostly incompatible) changes: > > 1. Almost all erro

Re: [Qemu-devel] [PATCH 17/34] qerror: drop QERR_SOCKET_CONNECT_IN_PROGRESS

2012-08-02 Thread Luiz Capitulino
On Thu, 2 Aug 2012 11:54:11 -0500 Michael Roth wrote: > On Wed, Aug 01, 2012 at 10:02:37PM -0300, Luiz Capitulino wrote: > > This error is currently returned by inet_connect_opts(), however > > it causes the follow spurious message on HMP: > > > > (qemu) migrate tcp:0: > > migrate: C

Re: [Qemu-devel] [PATCH 27/34] qerror: add proper ErrorClass value for QERR_ macros

2012-08-02 Thread Markus Armbruster
Luiz Capitulino writes: > This commit replaces the place holder value for the ErrorClass > argument with a proper ErrorClass value for all QERR_ macros. > > All current errors are mapped to GenericError, except for errors > CommandNotFound, DeviceNotActive, DeviceNotFound, KVMMissingCap and > Mig

Re: [Qemu-devel] [PATCH 10/11] Add XBZRLE statistics

2012-08-02 Thread Orit Wasserman
On 08/02/2012 05:01 PM, Eric Blake wrote: > On 08/02/2012 06:44 AM, Orit Wasserman wrote: >> Signed-off-by: Benoit Hudzia >> Signed-off-by: Petter Svard >> Signed-off-by: Aidan Shribman >> Signed-off-by: Orit Wasserman >> Signed-off-by: Juan Quintela >> --- > > >> >> +6. Migration is being

Re: [Qemu-devel] [PATCH 08/11] Add migrate_set_cachesize command

2012-08-02 Thread Orit Wasserman
On 08/02/2012 04:58 PM, Eric Blake wrote: > On 08/02/2012 06:44 AM, Orit Wasserman wrote: >> Change XBZRLE cache size in bytes (the size should be a power of 2, it will >> be >> rounded down to the nearest power of 2). >> If XBZRLE cache size is too small there will be many cache miss. >> >> New q

[Qemu-devel] [PATCH v3 00/10] Remove periodic wakeup from RTC timer

2012-08-02 Thread Paolo Bonzini
The current RTC emulation has two timers firing every second, one on each edge of the UIP bit. This will prevent CPUs from staying at deep C-states. Intel's measurements from previous submissions show the C6 residency reduced by 6% when running 64 idle guests. The following patches remove the tw

Re: [Qemu-devel] [PATCH 26/34] error, qerror: add ErrorClass argument to error functions

2012-08-02 Thread Markus Armbruster
Luiz Capitulino writes: > The new argument is added to functions qerror_report() and error_set(). > It's stored in Error and QError. qerror_report_err() is also updated to > take care of it. > > The QERR_ macros are changed to contain a place holder value for the > new argument, so that the value

Re: [Qemu-devel] [PATCH 17/34] qerror: drop QERR_SOCKET_CONNECT_IN_PROGRESS

2012-08-02 Thread Michael Roth
On Wed, Aug 01, 2012 at 10:02:37PM -0300, Luiz Capitulino wrote: > This error is currently returned by inet_connect_opts(), however > it causes the follow spurious message on HMP: > > (qemu) migrate tcp:0: > migrate: Connection can not be completed immediately > (qemu) > > But mig

[Qemu-devel] [PATCH v3 08/10] RTC: Do not fire timer periodically to catch next alarm

2012-08-02 Thread Paolo Bonzini
This patch limits further the usage of a periodic timer. It computes the time of the next alarm, and uses it to skip all intermediate occurrences of the timer. Cc: Yang Zhang Signed-off-by: Paolo Bonzini --- hw/mc146818rtc.c | 117 +++--- 1 file

[Qemu-devel] [PATCH v3 09/10] RTC: Get and set time without going through s->current_tm

2012-08-02 Thread Paolo Bonzini
This patch makes rtc_set_time and rtc_set_cmos work without reading s->current_tm. In the case of rtc_set_time I introduce a new function that retrieves the time and stores into a given struct tm (not hard-coded to s->current_tm). In the case of rtc_set_cmos, the current time is similarly taken f

  1   2   3   >