[Qemu-devel] [Bug 591666] Re: broken pci_add auto storage

2011-07-07 Thread Pierce Liu
** Attachment added: guest write data into vda https://bugs.launchpad.net/qemu/+bug/591666/+attachment/2194879/+files/2-write2virtDisk.png -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/591666

[Qemu-devel] [Bug 591666] Re: broken pci_add auto storage

2011-07-07 Thread Pierce Liu
** Attachment added: Host get the expect string helloHpervisor https://bugs.launchpad.net/qemu/+bug/591666/+attachment/2194881/+files/3.getStringOnHypervisor.png -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Qemu-devel] [Bug 591666] Re: broken pci_add auto storage

2011-07-07 Thread Pierce Liu
** Attachment added: The disk device on host including virtio block --vda https://bugs.launchpad.net/qemu/+bug/591666/+attachment/2194878/+files/1-vdaOnGuest.png -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

[Qemu-devel] [Bug 591666] Re: broken pci_add auto storage

2011-07-07 Thread Pierce Liu
If configure guest kernel correctly, it seems that virtio for block is OK! Could you help or send me your configure file to have a second check Thanks version info === QEMU 0.14.50 monitor host kernel: 2.6.39 guest kernel: 2.6.32 config file for kernel info

[Qemu-devel] [Bug 591666] Re: broken pci_add auto storage

2011-07-07 Thread Pierce Liu
** Attachment added: Just in case for use, the config for host https://bugs.launchpad.net/qemu/+bug/591666/+attachment/2194877/+files/h-config.gz -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Gerd Hoffmann
Hi, We don't have a hard policy about not merging devices that don't support migration. Since migration must be supported forever, I'd rather see a device get some solid testing before it starts doing live migration. That said, we should probably do this consciously by explicitly marking

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Gerd Hoffmann
On 07/06/11 19:13, Anthony Liguori wrote: On 07/06/2011 11:04 AM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e. devices which used to not save state before do now. We already have one case in tree: usb. qemu 0.14

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Avi Kivity
On 07/07/2011 10:14 AM, Gerd Hoffmann wrote: Can't we just implicitly fail migration whenever there's a device in the tree that doesn't have VMSTATE? There are cases where the device doesn't need to save state, so that alone doesn't cut it. It should then say so by having an empty VMSTATE

Re: [Qemu-devel] [PATCH] qxl: add defines from latest spice-protocol.

2011-07-07 Thread Gerd Hoffmann
On 07/06/11 14:19, Alon Levy wrote: From: Gerd Hoffmannkra...@redhat.com Allows to build with older spice-protocol versions. Given that we can't get away with changing the protocol only but require a new libspices-server for the new features this is pointless ... cheers, Gerd

Re: [Qemu-devel] [PATCH] qxl: move qemu_spice_add_memslot call out of qxl_add_memslot

2011-07-07 Thread Gerd Hoffmann
Hi, -static void qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta) +static void qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta, +QXLDevMemSlot *memslot) -qxl_add_memslot(d, val, 0); +qxl_add_memslot(d, val,

Re: [Qemu-devel] [PATCH] spice: lock spice worker calls

2011-07-07 Thread Gerd Hoffmann
On 07/06/11 14:19, Alon Levy wrote: From: Gerd Hoffmannkra...@redhat.com ... so we can call them from a thread. Still needed? I don't think so. cheers, Gerd

Re: [Qemu-devel] [PATCH] qxl: add defines from latest spice-protocol.

2011-07-07 Thread Alon Levy
On Thu, Jul 07, 2011 at 09:33:05AM +0200, Gerd Hoffmann wrote: On 07/06/11 14:19, Alon Levy wrote: From: Gerd Hoffmannkra...@redhat.com Allows to build with older spice-protocol versions. Given that we can't get away with changing the protocol only but require a new libspices-server for

Re: [Qemu-devel] [PATCH] qxl: async I/O

2011-07-07 Thread Gerd Hoffmann
Hi, +void qxl_spice_update_area_async(PCIQXLDevice *qxl, uint32_t surface_id, + struct QXLRect *area, struct QXLRect *dirty_rects, + uint32_t num_dirty_rects, uint32_t clear_dirty_region, + int async) +{ +

Re: [Qemu-devel] [PATCH] spice: lock spice worker calls

2011-07-07 Thread Alon Levy
On Thu, Jul 07, 2011 at 09:40:15AM +0200, Gerd Hoffmann wrote: On 07/06/11 14:19, Alon Levy wrote: From: Gerd Hoffmannkra...@redhat.com ... so we can call them from a thread. Still needed? I don't think so. Will drop (I wasn't sure we won't want this at a distant future, so I kept it).

Re: [Qemu-devel] [PATCH v2] XBRLE page delta compression for live migration of large memory apps

2011-07-07 Thread Stefan Hajnoczi
On Wed, Jul 06, 2011 at 02:01:58PM +0200, Shribman, Aidan wrote: XBRLE does not suite all scenarios but has been proven highly beneficial in for VMs running payloads such as: SAP ERP systems; VLC transcoding; LMbench memory write benchmarks. Another way of looking at this patch is as a

Re: [Qemu-devel] SSH console for qemu

2011-07-07 Thread Nikita A Menkovich
Seems no, I think, that the best way is running some kind of ssh daemon, that connects to console (/dev/vcsa1 for example), that listen on port and send to output console events on boot, and allow to work with server with simple ssh client. Also important feature will be saving history of output.

Re: [Qemu-devel] [PATCH] qxl: move qemu_spice_add_memslot call out of qxl_add_memslot

2011-07-07 Thread Alon Levy
On Thu, Jul 07, 2011 at 09:39:34AM +0200, Gerd Hoffmann wrote: Hi, -static void qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta) +static void qxl_add_memslot(PCIQXLDevice *d, uint32_t slot_id, uint64_t delta, +QXLDevMemSlot *memslot) -

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Markus Armbruster
Anthony Liguori anth...@codemonkey.ws writes: On 07/06/2011 11:04 AM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e. devices which used to not save state before do now. We already have one case in tree: usb. qemu

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Alexander Graf
On 07.07.2011, at 11:23, Markus Armbruster wrote: Anthony Liguori anth...@codemonkey.ws writes: On 07/06/2011 11:04 AM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e. devices which used to not save state

Re: [Qemu-devel] [SeaBIOS] [PATCH V5 0/9] Add TPM support to SeaBIOS

2011-07-07 Thread Stefan Berger
On 07/06/2011 06:58 PM, Kevin O'Connor wrote: On Wed, Jul 06, 2011 at 12:31:58PM -0400, Stefan Berger wrote: The following set of patches add TPM and Trusted Computing support to SeaBIOS. In particular the patches add: - a TPM driver for the Qemu's TPM TIS emulation (not yet in Qemu git) -

Re: [Qemu-devel] [PATCH] Add tee option to qemu char device

2011-07-07 Thread Stefano Stabellini
On Thu, 7 Jul 2011, Chunyan Liu wrote: In previous thread Support logging xen-guest console, it's considered that adding a tee option to char layer is a more generic way and makes more sense. http://lists.nongnu.org/archive/html/qemu-devel/2011-06/msg03011.html Following is an

Re: [Qemu-devel] [PATCH v2 2/6] Add copy and constant propagation.

2011-07-07 Thread Kirill Batuzov
On Fri, 10 Jun 2011, Richard Henderson wrote: +/* Do copy propagation */ +if (!(def-flags (TCG_OPF_CALL_CLOBBER | TCG_OPF_SIDE_EFFECTS))) { Why are you suppressing copy propagation in this way? I see no reason for it. This was suggested by Aurelien Jarno. I do not

[Qemu-devel] [PATCH v3 0/6] Implement constant folding and copy propagation in TCG

2011-07-07 Thread Kirill Batuzov
This series implements some basic machine-independent optimizations. They simplify code and allow liveness analysis do it's work better. Suppose we have following ARM code: movwr12, #0xb6db movtr12, #0xdb6d In TCG before optimizations we'll have: movi_i32 tmp8,$0xb6db mov_i32

[Qemu-devel] [PATCH v3 6/6] Do constant folding for unary operations.

2011-07-07 Thread Kirill Batuzov
Perform constant folding for NOT and EXT{8,16,32}{S,U} operations. Signed-off-by: Kirill Batuzov batuz...@ispras.ru --- tcg/optimize.c | 59 1 files changed, 59 insertions(+), 0 deletions(-) diff --git a/tcg/optimize.c b/tcg/optimize.c

[Qemu-devel] [PATCH v3 2/6] Add copy and constant propagation.

2011-07-07 Thread Kirill Batuzov
Make tcg_constant_folding do copy and constant propagation. It is a preparational work before actual constant folding. Signed-off-by: Kirill Batuzov batuz...@ispras.ru --- tcg/optimize.c | 182 +++- 1 files changed, 180 insertions(+), 2

[Qemu-devel] [PATCH v3 4/6] Do constant folding for boolean operations.

2011-07-07 Thread Kirill Batuzov
Perform constant folding for AND, OR, XOR operations. Signed-off-by: Kirill Batuzov batuz...@ispras.ru --- tcg/optimize.c | 37 + 1 files changed, 37 insertions(+), 0 deletions(-) diff --git a/tcg/optimize.c b/tcg/optimize.c index 42a1bda..c469952 100644

[Qemu-devel] [PATCH v3 3/6] Do constant folding for basic arithmetic operations.

2011-07-07 Thread Kirill Batuzov
Perform actual constant folding for ADD, SUB and MUL operations. Signed-off-by: Kirill Batuzov batuz...@ispras.ru --- tcg/optimize.c | 125 1 files changed, 125 insertions(+), 0 deletions(-) diff --git a/tcg/optimize.c b/tcg/optimize.c

[Qemu-devel] [PATCH v3 1/6] Add TCG optimizations stub

2011-07-07 Thread Kirill Batuzov
Added file tcg/optimize.c to hold TCG optimizations. Function tcg_optimize is called from tcg_gen_code_common. It calls other functions performing specific optimizations. Stub for constant folding was added. Signed-off-by: Kirill Batuzov batuz...@ispras.ru --- Makefile.target |2 +-

Re: [Qemu-devel] [PATCH v3 0/6] Implement constant folding and copy propagation in TCG

2011-07-07 Thread Peter Maydell
On 7 July 2011 13:37, Kirill Batuzov batuz...@ispras.ru wrote: ARM guests for 254.gap and 255.vortex and x86_64 guest for 252.eon does not work under QEMU for some unrelated reason. If you can provide a binary and a command line for these I can have a look at what's going on with the failing

[Qemu-devel] [PATCH v3 5/6] Do constant folding for shift operations.

2011-07-07 Thread Kirill Batuzov
Perform constant forlding for SHR, SHL, SAR, ROTR, ROTL operations. Signed-off-by: Kirill Batuzov batuz...@ispras.ru --- tcg/optimize.c | 72 1 files changed, 72 insertions(+), 0 deletions(-) diff --git a/tcg/optimize.c b/tcg/optimize.c

Re: [Qemu-devel] [SeaBIOS] [PATCH V5 0/9] Add TPM support to SeaBIOS

2011-07-07 Thread Kevin O'Connor
On Thu, Jul 07, 2011 at 07:48:29AM -0400, Stefan Berger wrote: On 07/06/2011 06:58 PM, Kevin O'Connor wrote: BTW, I don't think patch 7 or 9 really make sense to integrate in the official version of SeaBIOS. Also, in patch 8, I'd prefer to see all new fw_cfg entries use the romfile mechanism.

[Qemu-devel] [PATCH 2/3] virtio-console: Add some trace events

2011-07-07 Thread Amit Shah
Add some trace events for messages passed between the char layer and the virtio-serial bus. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-console.c |9 - trace-events|5 + 2 files changed, 13 insertions(+), 1 deletions(-) diff --git

[Qemu-devel] [PATCH 1/3] virtio-serial-bus: Add trace events

2011-07-07 Thread Amit Shah
Add some trace events for messages passed between the guest and host. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-serial-bus.c |7 +++ trace-events |6 ++ 2 files changed, 13 insertions(+), 0 deletions(-) diff --git a/hw/virtio-serial-bus.c

[Qemu-devel] [PULL] virtio-serial: trace events, trivial fix

2011-07-07 Thread Amit Shah
Hello, This series adds some trace events to virtio-serial-bus.c and virtio-console.c. There's also one trivial patch to remove a trailing \n from an error_report() string. Note: some mirrors may not yet have received the update. The following changes since commit

[Qemu-devel] [PATCH 3/3] virtio-serial-bus: Fix trailing \n in error_report string

2011-07-07 Thread Amit Shah
Markus fixed offenders in the file but one instance sneaked in via another patch. Fix it. Signed-off-by: Amit Shah amit.s...@redhat.com --- hw/virtio-serial-bus.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c index

Re: [Qemu-devel] [PATCH] Add tee option to qemu char device

2011-07-07 Thread Alexander Graf
On 07/07/2011 10:24 AM, Chunyan Liu wrote: In previous thread Support logging xen-guest console, it's considered that adding a tee option to char layer is a more generic way and makes more sense. http://lists.nongnu.org/archive/html/qemu-devel/2011-06/msg03011.html Following is an

[Qemu-devel] [PATCH v2 1/9] exec: add endian specific phys ld/st functions

2011-07-07 Thread Alexander Graf
Device code some times needs to access physical memory and does that through the ld./st._phys functions. However, these are the exact same functions that the CPU uses to access memory, which means they will be endianness swapped depending on the target CPU. However, devices don't know about the

[Qemu-devel] [PATCH 1/1] virtio-console: Prevent abort()s in case of host chardev close

2011-07-07 Thread Amit Shah
A host chardev could close just before the guest sends some data to be written. This will cause an -EPIPE error. This shouldn't be propagated to virtio-serial-bus. Ideally we should close the port once -EPIPE is received, but since the chardev interface doesn't return such meaningful values to

[Qemu-devel] [PATCH 1/3] qemu: Add strtosz_suffix_unit function

2011-07-07 Thread Joerg Roedel
This function does the same as the strtosz_suffix function except that it allows to specify the unit to which the k/M/B/T suffixes apply. This function will be used later to parse the tsc-frequency from the command-line. Signed-off-by: Joerg Roedel joerg.roe...@amd.com --- cutils.c | 16

[Qemu-devel] [PATCH 0/3][uq/master] Basic TSC-Scaling support v2

2011-07-07 Thread Joerg Roedel
Hi Avi, Marcelo, here is v2 of the patches to support setting the guests tsc-frequency from the qemu command line. This version addresses the comment from Avi on the first version. To reflect that units can be given to the frequency, the parameter was renamed from tsc_khz to tsc_freq. Thanks,

Re: [Qemu-devel] [PATCH v3 0/6] Implement constant folding and copy propagation in TCG

2011-07-07 Thread Kirill Batuzov
On Thu, 7 Jul 2011, Peter Maydell wrote: On 7 July 2011 13:37, Kirill Batuzov batuz...@ispras.ru wrote: ARM guests for 254.gap and 255.vortex and x86_64 guest for 252.eon does not work under QEMU for some unrelated reason. If you can provide a binary and a command line for these I can

Re: [Qemu-devel] [PATCH v2 1/3] spapr: proper qdevification

2011-07-07 Thread Alexander Graf
On 05/26/2011 11:52 AM, Paolo Bonzini wrote: Right now the spapr devices cannot be instantiated with -device, because the IRQs need to be passed to the spapr_*_create functions. Do this instead in the bus's init wrapper. This is particularly important with the conversion from scsi-disk to

Re: [Qemu-devel] [PATCH v5 05/18] qapi: add QMP input visitor

2011-07-07 Thread Michael Roth
On 07/07/2011 09:32 AM, Luiz Capitulino wrote: On Tue, 5 Jul 2011 08:02:32 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: A type of Visiter class that is used to walk a qobject's structure and assign each entry to the corresponding native C type. Command marshaling function will use this

[Qemu-devel] [PATCH 3/3] qemu-x86: Set tsc_khz in kvm when supported

2011-07-07 Thread Joerg Roedel
Make use of the KVM_TSC_CONTROL feature if available. Signed-off-by: Joerg Roedel joerg.roe...@amd.com --- target-i386/kvm.c | 18 +- 1 files changed, 17 insertions(+), 1 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 10fb2c4..923d2d5 100644 ---

Re: [Qemu-devel] [QAPI+QGA 2/3] QAPI code generation infrastructure v5

2011-07-07 Thread Luiz Capitulino
On Tue, 5 Jul 2011 08:02:27 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: This is Set 2/3 of the QAPI+QGA patchsets. These patches apply on top of master (set1 merged), and can also be obtained from: git://repo.or.cz/qemu/mdroth.git qapi-backport-set2-v5 This doesn't build due to a

Re: [Qemu-devel] [PATCH v5 05/18] qapi: add QMP input visitor

2011-07-07 Thread Luiz Capitulino
On Tue, 5 Jul 2011 08:02:32 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: A type of Visiter class that is used to walk a qobject's structure and assign each entry to the corresponding native C type. Command marshaling function will use this to pull out QMP command parameters recieved

Re: [Qemu-devel] [PATCH v3 0/3] spapr qdevification

2011-07-07 Thread Alexander Graf
On 07/06/2011 01:32 PM, Paolo Bonzini wrote: On 06/06/2011 04:16 PM, Paolo Bonzini wrote: This series fixes some problems with spapr's qdev interface. Patch 1 is the important one, which makes it possible to use -device to create vio devices. The other two are cosmetic. v1-v2:

Re: [Qemu-devel] [PATCH v5 04/18] qapi: add QAPI visitor core

2011-07-07 Thread Luiz Capitulino
On Tue, 5 Jul 2011 08:02:31 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: Base definitions/includes for Visiter interface used by generated visiter/marshalling code. Includes a GenericList type. Our lists require an embedded element. Since these types are generated, if you want to

Re: [Qemu-devel] [QAPI+QGA 2/3] QAPI code generation infrastructure v5

2011-07-07 Thread Michael Roth
On 07/07/2011 09:37 AM, Luiz Capitulino wrote: On Tue, 5 Jul 2011 08:02:27 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: This is Set 2/3 of the QAPI+QGA patchsets. These patches apply on top of master (set1 merged), and can also be obtained from: git://repo.or.cz/qemu/mdroth.git

Re: [Qemu-devel] [SeaBIOS] [PATCH V5 0/9] Add TPM support to SeaBIOS

2011-07-07 Thread Stefan Berger
On 07/07/2011 08:43 AM, Kevin O'Connor wrote: On Thu, Jul 07, 2011 at 07:48:29AM -0400, Stefan Berger wrote: On 07/06/2011 06:58 PM, Kevin O'Connor wrote: BTW, I don't think patch 7 or 9 really make sense to integrate in the official version of SeaBIOS. Also, in patch 8, I'd prefer to see all

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Anthony Liguori
On 07/07/2011 04:23 AM, Markus Armbruster wrote: Anthony Liguorianth...@codemonkey.ws writes: On 07/06/2011 11:04 AM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e. devices which used to not save state before do

Re: [Qemu-devel] [PATCH v5 04/18] qapi: add QAPI visitor core

2011-07-07 Thread Michael Roth
On 07/07/2011 09:32 AM, Luiz Capitulino wrote: On Tue, 5 Jul 2011 08:02:31 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: Base definitions/includes for Visiter interface used by generated visiter/marshalling code. Includes a GenericList type. Our lists require an embedded element. Since

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Gerd Hoffmann
Hi, Not so fast :) I agree that throwing away unrecognized migration data is unsafe, and should not be done. Now let me present my little problem. I'm working on making migration preserve tray status: open/closed, locked/unlocked. For ide-cd, I can stick a subsection ide_drive/tray_state

[Qemu-devel] [Bug 568614] Re: x86_64 host curses interface: spacing/garbling

2011-07-07 Thread Andrzej Zaborowski
This is probably the source of the problem. As you say it'd be best to use chtype directly if it can be done cleanly, unfortunately it looks like it'll add a curses specific snippet in console.h, but so be it. The only other option is to add a conversion step in curses.c and give up zero-copy

[Qemu-devel] [PATCH] Error: Fix build when qemu-common.h is not included

2011-07-07 Thread Luiz Capitulino
Commit e4ea5e2d0e0e4c5188ab45b66f3195062ae059dc added the use of the macro GCC_FMT_ATTR to error.h, however qemu-common.h is not included by error.h This will cause a build error when files including error.h don't include qemu-common.h. Not an issue today because the only file including it is

[Qemu-devel] [PATCH] qemu-options.hx: Document werror and rerror -drive options

2011-07-07 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qemu-options.hx |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/qemu-options.hx b/qemu-options.hx index e6d7adc..5f16a78 100644 --- a/qemu-options.hx +++ b/qemu-options.hx @@ -160,6 +160,11 @@ an untrusted

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Paolo Bonzini
On 07/07/2011 01:02 PM, Alexander Graf wrote: I'd guess the best would be to have a special VMSTATE that means broken old version doesn't send a section which we can set for special -M? No, the best would be to have a serious migration format, based for example on ASN.1 which Michael Tsirkin

Re: [Qemu-devel] [PATCH] Error: Fix build when qemu-common.h is not included

2011-07-07 Thread Michael Roth
On 07/07/2011 11:02 AM, Luiz Capitulino wrote: Commit e4ea5e2d0e0e4c5188ab45b66f3195062ae059dc added the use of the macro GCC_FMT_ATTR to error.h, however qemu-common.h is not included by error.h This will cause a build error when files including error.h don't include qemu-common.h. Not an

Re: [Qemu-devel] KVM call agenda for June 28

2011-07-07 Thread Stefan Hajnoczi
On Tue, Jul 5, 2011 at 7:18 PM, Marcelo Tosatti mtosa...@redhat.com wrote: On Tue, Jul 05, 2011 at 04:37:08PM +0100, Stefan Hajnoczi wrote: On Tue, Jul 5, 2011 at 3:32 PM, Marcelo Tosatti mtosa...@redhat.com wrote: On Tue, Jul 05, 2011 at 04:39:06PM +0300, Dor Laor wrote: On 07/05/2011 03:58

Re: [Qemu-devel] [PATCH 3/3] hw/omap_gpio.c: Convert to qdev

2011-07-07 Thread andrzej zaborowski
Hi, On 5 July 2011 15:19, Peter Maydell peter.mayd...@linaro.org wrote: On 4 July 2011 23:39, andrzej zaborowski balr...@gmail.com wrote:  I'd also prefer that omap2_gpio_init remains a function that does all the qdev magic in omap_gpio.c and with the current signature I'm not convinced about

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Anthony Liguori
On 07/06/2011 06:32 PM, Alexander Graf wrote: On 06.07.2011, at 22:01, Anthony Liguori wrote: On 07/06/2011 12:28 PM, Avi Kivity wrote: On 07/06/2011 07:04 PM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e.

[Qemu-devel] [PATCH v2] qxl: use update_area_async in qxl-render

2011-07-07 Thread Alon Levy
--- hw/qxl-render.c |4 ++-- hw/qxl.c| 14 +- hw/qxl.h|3 --- 3 files changed, 7 insertions(+), 14 deletions(-) diff --git a/hw/qxl-render.c b/hw/qxl-render.c index 60b822d..8d847d0 100644 --- a/hw/qxl-render.c +++ b/hw/qxl-render.c @@ -124,8 +124,8 @@ void

[Qemu-devel] [PATCH v2] qxl: only disallow specific io's in vga mode

2011-07-07 Thread Alon Levy
Since the driver is still in operation even after moving to UNDEFINED, i.e. by destroying primary in any way. --- hw/qxl.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/hw/qxl.c b/hw/qxl.c index 395d994..e57ccf2 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -1167,8 +1167,9

[Qemu-devel] [PATCH v2] qxl: bump pci rev

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com Inform guest drivers about the new features I/O commands we have now (async commands, S3 support). Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/hw/qxl.c

[Qemu-devel] [PATCH v2] qxl: make qxl_guest_bug take variable arguments

2011-07-07 Thread Alon Levy
--- hw/qxl.c | 18 +++--- hw/qxl.h |2 +- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/hw/qxl.c b/hw/qxl.c index 0d62036..935bac0 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -124,11 +124,15 @@ static void qxl_reset_memslots(PCIQXLDevice *d); static void

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Anthony Liguori
On 07/07/2011 02:19 AM, Gerd Hoffmann wrote: On 07/06/11 19:13, Anthony Liguori wrote: On 07/06/2011 11:04 AM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e. devices which used to not save state before do now. We

[Qemu-devel] [PATCH v2] qxl: add QXL_IO_FLUSH_{SURFACES, RELEASE} for guest S3S4 support

2011-07-07 Thread Alon Levy
Add two new IOs. QXL_IO_FLUSH_SURFACES - equivalent to update area for all surfaces, used to reduce vmexits from NumSurfaces to 1 on guest S3, S4 and resolution change (windows driver implementation is such that this is done on each of those occasions). QXL_IO_FLUSH_RELEASE - used to ensure

Re: [Qemu-devel] migration: new sections and backward compatibility.

2011-07-07 Thread Markus Armbruster
Anthony Liguori anth...@codemonkey.ws writes: On 07/07/2011 04:23 AM, Markus Armbruster wrote: Anthony Liguorianth...@codemonkey.ws writes: On 07/06/2011 11:04 AM, Gerd Hoffmann wrote: Hi folks, We'll need to figure a sane way to handle migration to older versions with new sections, i.e.

[Qemu-devel] [PATCH v2] qxl: error handling fixes and cleanups.

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com Add qxl_guest_bug() function which is supposed to be called in case sanity checks of guest requests fail. It raises an error IRQ and logs a message in case guest debugging is enabled. Make PANIC_ON() abort instead of exit. That macro should be used for

[Qemu-devel] [PATCH v2] spice: add worker wrapper functions.

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com Add wrapper functions for all spice worker calls. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl-render.c|4 +- hw/qxl.c | 32 +- ui/spice-display.c | 94 ---

[Qemu-devel] [PATCH v2] qxl: fix surface tracking locking

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com Surface tracking needs proper locking since it is used from vcpu and spice worker threads, add it. Also reset the surface counter when zapping all surfaces. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c | 13 - hw/qxl.h |2

[Qemu-devel] [PATCH v2] qxl: add io_port_to_string

2011-07-07 Thread Alon Levy
--- hw/qxl.c | 62 +- 1 files changed, 61 insertions(+), 1 deletions(-) diff --git a/hw/qxl.c b/hw/qxl.c index 9116c99..f3312f0 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -407,6 +407,65 @@ static const char *qxl_mode_to_string(int mode)

Re: [Qemu-devel] [PATCH 3/3] hw/omap_gpio.c: Convert to qdev

2011-07-07 Thread Peter Maydell
On 7 July 2011 16:29, andrzej zaborowski balr...@gmail.com wrote:  Optimally we should pass the clock as a pointer or name, not only the baudrate (clocks can be stopped, reparented, or have frequency changed dynamically -- in fact the OS will most likely manipulate the uart source clock

[Qemu-devel] [PATCH v2] spice: add qemu_spice_display_init_common

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com Factor out SimpleSpiceDisplay initialization into qemu_spice_display_init_common() and call it from both qxl.c (for vga mode) and spice-display.c Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c |7 +-- ui/spice-display.c | 17

[Qemu-devel] [PATCH v2] qxl: async I/O

2011-07-07 Thread Alon Levy
Some of the QXL port i/o commands are waiting for the spice server to complete certain actions. Add async versions for these commands, so we don't block the vcpu while the spice server processses the command. Instead the qxl device will raise an IRQ when done. The async command processing relies

Re: [Qemu-devel] [PATCH] Error: Fix build when qemu-common.h is not included

2011-07-07 Thread Stefan Weil
Am 07.07.2011 18:17, schrieb Michael Roth: On 07/07/2011 11:02 AM, Luiz Capitulino wrote: Commit e4ea5e2d0e0e4c5188ab45b66f3195062ae059dc added the use of the macro GCC_FMT_ATTR to error.h, however qemu-common.h is not included by error.h This will cause a build error when files including

Re: [Qemu-devel] [PATCH] Implement the global timer present in ARM MPCore chips.

2011-07-07 Thread Christopher Harvey
On Wed, Jul 06, 2011 at 10:58:09PM +0100, Peter Maydell wrote: On 6 July 2011 21:04, Christopher Harvey char...@matrox.com wrote: Below are some random comments noticed on a first-pass. +/* Global timer data */ +static uint64_t global_timer_count = 0; +static uint64_t

[Qemu-devel] [PATCH v2] spice/qxl: move worker wrappers

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com Move the wrapper functions which are used by qxl only to qxl.c. Rename them from qemu_spice_* to qxl_spice_*. Also pass in a qxl state pointer instead of a SimpleSpiceDisplay pointer. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl-render.c|

[Qemu-devel] Upstream Qemu With Xen configuration problem

2011-07-07 Thread Daniel Castro
Greetings, I am trying to configure Xen4.2 unstable with upstream qemu. I am running on a small problem when running the configuration of qemu, Qemu Config can not find Xen. I have debugged the Configuration script and the temp c file that qemu uses to detect xen version are returning empty

[Qemu-devel] [PATCH v2] async + suspend reworked

2011-07-07 Thread Alon Levy
Everything is based on spice.v38 from git://anongit.freedesktop.org/spice/qemu v1-v2 changes: dropped wlock dropped oom_async update_area_async used in qxl-render added async_lock async_complete handles completion of io, not at dispatcher call time Git trees:

[Qemu-devel] [PATCH v2] qxl: use QXL_REVISION_*

2011-07-07 Thread Alon Levy
--- hw/qxl.c | 24 +++- 1 files changed, 11 insertions(+), 13 deletions(-) diff --git a/hw/qxl.c b/hw/qxl.c index 4007847..54ee5f5 100644 --- a/hw/qxl.c +++ b/hw/qxl.c @@ -1551,7 +1551,6 @@ static DisplayChangeListener display_listener = { static int

[Qemu-devel] [PATCH v2] qxl: remove qxl_destroy_primary()

2011-07-07 Thread Alon Levy
From: Gerd Hoffmann kra...@redhat.com We'll have to move qemu_spice_destroy_primary_surface() out of qxl_destroy_primary(). That makes the function pretty pointless, so zap it and open code the two lines instead. Signed-off-by: Gerd Hoffmann kra...@redhat.com --- hw/qxl.c | 28

Re: [Qemu-devel] [PATCH] Error: Fix build when qemu-common.h is not included

2011-07-07 Thread Luiz Capitulino
On Thu, 07 Jul 2011 18:50:28 +0200 Stefan Weil w...@mail.berlios.de wrote: Am 07.07.2011 18:17, schrieb Michael Roth: On 07/07/2011 11:02 AM, Luiz Capitulino wrote: Commit e4ea5e2d0e0e4c5188ab45b66f3195062ae059dc added the use of the macro GCC_FMT_ATTR to error.h, however qemu-common.h is

Re: [Qemu-devel] [libvirt-users] SSH console for qemu

2011-07-07 Thread Eric Blake
On 07/07/2011 02:36 AM, Nikita A Menkovich wrote: Seems no, I think, that the best way is running some kind of ssh daemon, that connects to console (/dev/vcsa1 for example), that listen on port and send to output console events on boot, and allow to work with server with simple ssh client.

[Qemu-devel] [Bug 568614] Re: x86_64 host curses interface: spacing/garbling

2011-07-07 Thread Devin J. Pohly
How about using CONFIG_CURSES? If we --enable-curses, then we know we have curses.h and chtype. If we --disable-curses, then we don't care. Patch attached. It's a no-op if curses is disabled, and it fixes the issue on my machine with curses enabled. I had to undef the color constants from

Re: [Qemu-devel] Upstream Qemu With Xen configuration problem

2011-07-07 Thread John Baboval
I find I have to point at the src directories, and not the dist directory. My extra-cflags looks something like this: --extra-cflags=-I /path/to/xen/tools/include -L /path/to/xen/tools/libxc -I /path/to/xen/tools/libxc -L /path/to/xen/tools/xenstore -I /path/to/xen/tools/xenstore' -John

Re: [Qemu-devel] SSH console for qemu

2011-07-07 Thread Stefan Hajnoczi
On Thu, Jul 7, 2011 at 9:36 AM, Nikita A Menkovich menkov...@gmail.com wrote: Seems no, I think, that the best way is running some kind of ssh daemon, that connects to console (/dev/vcsa1 for example), that listen on port and send to output console events on boot, and allow to work with server

Re: [Qemu-devel] [libvirt-users] SSH console for qemu

2011-07-07 Thread Nikita A Menkovich
I want to launch very minimal ssh daemon (that listen on selected port and allows only input commands and read output from console) that launch when guest started, and this not run inside a guest. Thanks to the link for patch On 7 July 2011 22:19, Eric Blake ebl...@redhat.com wrote: On

Re: [Qemu-devel] [PATCH v2] XBRLE page delta compression for live migration of large memory apps

2011-07-07 Thread Stefan Hajnoczi
On Wed, Jul 6, 2011 at 1:01 PM, Shribman, Aidan aidan.shrib...@sap.com wrote: Subject: [PATCH v2] XBRLE page delta compression for live migration of large memory apps From: Aidan Shribman aidan.shrib...@sap.com Work is based on research results published VEE 2011: Evaluation of Delta

Re: [Qemu-devel] [PATCH 2/3] Add fno-strict-overflow

2011-07-07 Thread Raghavendra D Prabhu
* On Tue, Jul 05, 2011 at 09:30:44PM +0100, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Jul 5, 2011 at 4:36 PM, Raghavendra D Prabhu raghu.prabh...@gmail.com wrote: * On Mon, Jul 04, 2011 at 11:38:30PM +0100, Peter Maydell peter.mayd...@linaro.org wrote: On 4 July 2011 23:00,

[Qemu-devel] [PATCH v2 0/4] ppc: booke206: KVM MMU API and info tlb

2011-07-07 Thread Scott Wood
Scott Wood (4): kvm: ppc: Update KVM headers for MMU API kvm: ppc: booke206: use MMU API ppc: booke206: use MAV=2.0 TSIZE definition, fix 4G pages ppc: booke206: add info tlb support hmp-commands.hx |2 +- hw/ppce500_mpc8544ds.c |4 +-

[Qemu-devel] [PATCH 1/4] kvm: ppc: Update KVM headers for MMU API

2011-07-07 Thread Scott Wood
Signed-off-by: Scott Wood scottw...@freescale.com --- linux-headers/asm-powerpc/kvm.h | 39 +-- linux-headers/linux/kvm.h | 27 --- 2 files changed, 57 insertions(+), 9 deletions(-) diff --git a/linux-headers/asm-powerpc/kvm.h

Re: [Qemu-devel] SeaBIOS error with Juniper FreeBSD kernel

2011-07-07 Thread Kevin O'Connor
On Thu, Jul 07, 2011 at 05:45:02PM +0200, Bjørn Mork wrote: It's been a while with little work and little progress on my side... But I looked at this again today, and found that it may be related to the SMBIOS table being allocated with malloc_high(). Does that make sense? Anyway, the

[Qemu-devel] [PATCH v2 2/4] kvm: ppc: booke206: use MMU API

2011-07-07 Thread Scott Wood
Share the TLB array with KVM. This allows us to set the initial TLB both on initial boot and reset, is useful for debugging, and could eventually be used to support migration. Signed-off-by: Scott Wood scottw...@freescale.com --- v2: - rebase on top of current tree, remove now-unneeded ifdefs

[Qemu-devel] [PATCH 3/4] ppc: booke206: use MAV=2.0 TSIZE definition, fix 4G pages

2011-07-07 Thread Scott Wood
This definition is backward compatible with MAV=1.0 as long as the guest does not set reserved bits in MAS1/MAS4. Also, fix the shift in booke206_tlb_to_page_size -- it's the base that should be able to hold a 4G page size, not the shift count. Signed-off-by: Scott Wood scottw...@freescale.com

[Qemu-devel] [PATCH v2 4/4] ppc: booke206: add info tlb support

2011-07-07 Thread Scott Wood
Signed-off-by: Scott Wood scottw...@freescale.com --- v2: Remove redundant valid? comment hmp-commands.hx |2 +- monitor.c |5 ++- target-ppc/cpu.h|2 + target-ppc/helper.c | 88 +++ 4 files changed, 94

Re: [Qemu-devel] KVM call agenda for June 28

2011-07-07 Thread Marcelo Tosatti
On Tue, Jul 05, 2011 at 04:39:06PM +0300, Dor Laor wrote: On 07/05/2011 03:58 PM, Marcelo Tosatti wrote: On Tue, Jul 05, 2011 at 01:40:08PM +0100, Stefan Hajnoczi wrote: On Tue, Jul 5, 2011 at 9:01 AM, Dor Laordl...@redhat.com wrote: I tried to re-arrange all of the requirements and use cases

Re: [Qemu-devel] Qemu performance

2011-07-07 Thread Lê Đức Tài
On 30.06.2011, at 15:24, Alexander Graf ag...@suse.de wrote: On 30.06.2011, at 04:11, Lê Đức Tài letai_d...@yahoo.com.vn wrote: Thank for your answer. Beside nbench, I'm also using Dhrystone to measure the guest cpu performance. The performance does not much diffetence too.