Re: [Qemu-devel] [PATCH 2/2] ui/spice-core: report version in 'info spice'

2011-08-10 Thread Gerd Hoffmann
On 08/09/11 22:53, Alon Levy wrote: Signed-off-by: Alon Levyal...@redhat.com --- ui/spice-core.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index 3d77c01..342ed6c 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -372,6

Re: [Qemu-devel] [PATCH v5 3/4] block: add block timer and block throttling algorithm

2011-08-10 Thread Zhi Yong Wu
On Tue, Aug 9, 2011 at 11:19 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: Note:      1.) When bps/iops limits are specified to a small value such as 511 bytes/s, this VM will hang up. We are considering how to handle

[Qemu-devel] [PATCH v2] ui/spice-core: report version in info spice/query-spice

2011-08-10 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- ui/spice-core.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index 3d77c01..4fb1b30 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -372,6 +372,10 @@ void

Re: [Qemu-devel] [PATCH 2/2] ui/spice-core: report version in 'info spice'

2011-08-10 Thread Alon Levy
On Wed, Aug 10, 2011 at 08:54:36AM +0200, Gerd Hoffmann wrote: On 08/09/11 22:53, Alon Levy wrote: Signed-off-by: Alon Levyal...@redhat.com --- ui/spice-core.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index

Re: [Qemu-devel] [PATCH 3/3] PPC: SPAPR: Use KVM function for time info

2011-08-10 Thread Alexander Graf
Am 10.08.2011 um 05:35 schrieb David Gibson da...@gibson.dropbear.id.au: On Tue, Aug 09, 2011 at 06:39:30PM +0200, Alexander Graf wrote: One of the things we can't fake on PPC is the timer speed. So we need to extract the frequency information from the host and put it back into the guest

Re: [Qemu-devel] coroutine-win32.c:36: error: thread-local storage not supported for this target

2011-08-10 Thread Paolo Bonzini
On 08/09/2011 06:58 PM, Sebastian Herbszt wrote: CC coroutine-win32.o coroutine-win32.c:36: error: thread-local storage not supported for this target coroutine-win32.c:37: error: thread-local storage not supported for this target make: *** [coroutine-win32.o] Error 1 You need a newer GCC.

Re: [Qemu-devel] Safely reopening image files by stashing fds

2011-08-10 Thread Kevin Wolf
Am 09.08.2011 21:39, schrieb Blue Swirl: On Tue, Aug 9, 2011 at 12:24 PM, Kevin Wolf kw...@redhat.com wrote: Am 09.08.2011 14:00, schrieb Stefan Hajnoczi: On Tue, Aug 09, 2011 at 01:39:13PM +0200, Kevin Wolf wrote: Am 09.08.2011 12:56, schrieb Stefan Hajnoczi: On Tue, Aug 9, 2011 at 11:50 AM,

Re: [Qemu-devel] Bug in recent CFAR patch

2011-08-10 Thread Alexander Graf
Am 10.08.2011 um 07:22 schrieb David Gibson da...@gibson.dropbear.id.au: Hi Alex, I realised there was a bug in the CFAR patch in my recent qemu patch series which breaks the build for ppc32 targets. There is a patch to fix it below, but I'm not sure if you want this standalone, or

Re: [Qemu-devel] [PATCH 2/2] ui/spice-core: report version in 'info spice'

2011-08-10 Thread Gerd Hoffmann
Hi, I think it would also be nice to somehow hint this is the spice sent a second patch (fixing the above), but didn't do any hinting. I'll send a third. Saw v2. I would just stick a version string into the dict instead of the three values for major, minor and micro. version qemu was

[Qemu-devel] [RFC] monitor: fix segfault from uninitialized lexer-emit

2011-08-10 Thread Alon Levy
The following triggers a call to monitor_control_read before monitor_control_event has been called. This patch is misplaced but fixes the problem by moving the json_message_parser_init call before setting the chardev handlers. (echo '{execute: qmp_capabilities}'; echo '{execute:query-spice}';

[Qemu-devel] [PATCH v3] ui/spice-core: report version in info spice/query-spice

2011-08-10 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- ui/spice-core.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index 3d77c01..f22f1d6 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -372,6 +372,10 @@ void

Re: [Qemu-devel] [PATCH 2/2] ui/spice-core: report version in 'info spice'

2011-08-10 Thread Alon Levy
On Wed, Aug 10, 2011 at 10:10:59AM +0200, Gerd Hoffmann wrote: Hi, I think it would also be nice to somehow hint this is the spice sent a second patch (fixing the above), but didn't do any hinting. I'll send a third. Saw v2. I would just stick a version string into the dict instead

Re: [Qemu-devel] [PATCH] Correctly assign PCI domain numbers

2011-08-10 Thread Michael S. Tsirkin
On Thu, Aug 04, 2011 at 07:00:38PM +1000, David Gibson wrote: On Wed, Aug 03, 2011 at 04:28:33PM +0300, Michael S. Tsirkin wrote: On Tue, Aug 02, 2011 at 12:15:22AM +1000, David Gibson wrote: On Mon, Aug 01, 2011 at 05:03:18PM +0300, Michael S. Tsirkin wrote: On Mon, Aug 01, 2011 at

Re: [Qemu-devel] [PATCH] Permit -mem-path without sync mmu

2011-08-10 Thread Avi Kivity
On 08/10/2011 08:10 AM, David Gibson wrote: On Mon, Aug 08, 2011 at 11:24:09AM +0300, Avi Kivity wrote: On 08/08/2011 09:03 AM, David Gibson wrote: Second, if userspace qemu passing hugepages to kvm can cause (host) kernel memory corruption, that is clearly a host kernel bug. So am I

Re: [Qemu-devel] build failure with coroutine-gthread

2011-08-10 Thread Stefan Hajnoczi
On Tue, Aug 09, 2011 at 05:28:17PM +, Blue Swirl wrote: On Mon, Aug 8, 2011 at 9:29 AM, Stefan Hajnoczi stefa...@gmail.com wrote: On Mon, Aug 8, 2011 at 10:01 AM, Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com wrote:  LINK  qemu-ga coroutine-gthread.o: In function

Re: [Qemu-devel] [PATCH v5 1/4] block: add the command line support

2011-08-10 Thread Stefan Hajnoczi
On Wed, Aug 10, 2011 at 01:20:22PM +0800, Zhi Yong Wu wrote: On Tue, Aug 9, 2011 at 8:25 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: Signed-off-by: Zhi Yong Wu wu...@linux.vnet.ibm.com ---  Makefile.objs   |  

Re: [Qemu-devel] [PATCH v5 2/4] block: add the block queue support

2011-08-10 Thread Stefan Hajnoczi
On Wed, Aug 10, 2011 at 01:54:33PM +0800, Zhi Yong Wu wrote: On Tue, Aug 9, 2011 at 8:49 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: +BlockDriverAIOCB *qemu_block_queue_enqueue(BlockQueue *queue, +              

Re: [Qemu-devel] [PATCH v5 0/4] The intro of QEMU block I/O throttling

2011-08-10 Thread Stefan Hajnoczi
On Wed, Aug 10, 2011 at 01:09:27PM +0800, Zhi Yong Wu wrote: On Tue, Aug 9, 2011 at 8:08 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote:  Makefile.objs     |    2 +-  block.c           |  347

[Qemu-devel] [PATCH STABLE] build: Move QEMU_INCLUDES before QEMU_CFLAGS

2011-08-10 Thread Jiri Denemark
This patch fixes build when any of the include paths from QEMU_CFLAGS contains a header file with similar name to a header file in qemu sources. I hit it with error.h included by qapi/qapi-types-core.h. GCC decided to use /usr/include/alsa/error.h instead of qemu's error.h. Signed-off-by: Jiri

Re: [Qemu-devel] [PATCH v5 3/4] block: add block timer and block throttling algorithm

2011-08-10 Thread Stefan Hajnoczi
On Wed, Aug 10, 2011 at 7:57 AM, Zhi Yong Wu zwu.ker...@gmail.com wrote: On Tue, Aug 9, 2011 at 11:19 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Tue, Aug 9, 2011 at 5:17 AM, Zhi Yong Wu wu...@linux.vnet.ibm.com wrote: Note:      1.) When bps/iops limits are specified to a small value

[Qemu-devel] [PATCH] PPC: Fix for the gdb single step problem on an rfi instruction

2011-08-10 Thread Sebastian Bauer
When using gdb to single step a ppc interrupt routine, the execution flow passes the rfi instruction without actually returning from the interrupt. The patch fixes this by avoiding to update the nip when the debug exception is raised and a previous POWERPC_EXCP_SYNC was set. The latter is the

Re: [Qemu-devel] [PATCH STABLE] build: Move QEMU_INCLUDES before QEMU_CFLAGS

2011-08-10 Thread Michael Roth
On 08/10/2011 05:04 AM, Jiri Denemark wrote: This patch fixes build when any of the include paths from QEMU_CFLAGS contains a header file with similar name to a header file in qemu sources. I hit it with error.h included by qapi/qapi-types-core.h. GCC decided to use /usr/include/alsa/error.h

[Qemu-devel] [Bug 823902] [NEW] multithreaded ARM seg/longjmp causes uninitialized stack frame due to0d10193870b5a81c3bce13a602a5403c3a55cf6c

2011-08-10 Thread Dr. David Alan Gilbert
Public bug reported: Hi, I've got an ARM multithreaded test program that I wrote as a gcc testcase (attached) that fails on QEmu, firefox from Ubuntu ARM maverick also fails in the same way. The failure is either a seg fault or '*** longjmp causes uninitialized stack frame ***:

Re: [Qemu-devel] [PATCH STABLE] build: Move QEMU_INCLUDES before QEMU_CFLAGS

2011-08-10 Thread Avi Kivity
On 08/10/2011 01:04 PM, Jiri Denemark wrote: This patch fixes build when any of the include paths from QEMU_CFLAGS contains a header file with similar name to a header file in qemu sources. I hit it with error.h included by qapi/qapi-types-core.h. GCC decided to use /usr/include/alsa/error.h

[Qemu-devel] [Bug 823902] Re: multithreaded ARM seg/longjmp causes uninitialized stack frame due to0d10193870b5a81c3bce13a602a5403c3a55cf6c

2011-08-10 Thread Dr. David Alan Gilbert
Test built from source from this set of patches: http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02235.html ** Attachment added: Multithreaded armv7 test binary, uses ldrexd/strexd https://bugs.launchpad.net/qemu/+bug/823902/+attachment/2265952/+files/di-ll64-syncmulti-armv7 -- You received

[Qemu-devel] [Bug 823902] Re: multithreaded ARM seg/longjmp causes uninitialized stack frame due to0d10193870b5a81c3bce13a602a5403c3a55cf6c

2011-08-10 Thread Dr. David Alan Gilbert
Note this needs the 64bit ARM sync patches for gcc to build ** Attachment added: Multithreaded test source https://bugs.launchpad.net/qemu/+bug/823902/+attachment/2265957/+files/di-sync-multithread.c -- You received this bug notification because you are a member of qemu- devel-ml, which is

[Qemu-devel] [Bug 823902] Re: multithreaded ARM seg/longjmp causes uninitialized stack frame due to0d10193870b5a81c3bce13a602a5403c3a55cf6c

2011-08-10 Thread Peter Maydell
If you roll back to commit 2b41f10e186ccb4f0058815161586f8d6d006ea3 what is the pass/fail rate? That ought to separate out new bugs caused by recent commits (including the 0d101 change which is definitely wrong since it assumes cpu_single_env is only being used by one thread) from random other

Re: [Qemu-devel] [RFC] postcopy livemigration proposal

2011-08-10 Thread Avi Kivity
On 08/09/2011 05:33 AM, Isaku Yamahata wrote: On Mon, Aug 08, 2011 at 03:38:54PM +0300, Avi Kivity wrote: On 08/08/2011 06:24 AM, Isaku Yamahata wrote: This mail is on Yabusame: Postcopy Live Migration for Qemu/KVM on which we'll give a talk at KVM-forum. The purpose of this mail is to

[Qemu-devel] [Bug 823902] Re: multithreaded ARM seg/longjmp causes uninitialized stack frame due to0d10193870b5a81c3bce13a602a5403c3a55cf6c

2011-08-10 Thread Dr. David Alan Gilbert
2b41f... is a disaster on 64bit - 1 out of 10 pass; most of the others fail with: qemu/user-exec.c:99: handle_cpu_signal: Assertion `({ unsigned long __guest = (unsigned long)(address) - guest_base; __guest (1ul 32); })' failed. which I think is a segfault at host address 0 or there about.

Re: [Qemu-devel] [PATCH STABLE] build: Move QEMU_INCLUDES before QEMU_CFLAGS

2011-08-10 Thread Jiri Denemark
On Wed, Aug 10, 2011 at 16:04:15 +0300, Avi Kivity wrote: On 08/10/2011 01:04 PM, Jiri Denemark wrote: This patch fixes build when any of the include paths from QEMU_CFLAGS contains a header file with similar name to a header file in qemu sources. I hit it with error.h included by

Re: [Qemu-devel] [PATCH STABLE] build: Move QEMU_INCLUDES before QEMU_CFLAGS

2011-08-10 Thread Avi Kivity
On 08/10/2011 05:02 PM, Jiri Denemark wrote: Shouldn't we use -isystem instead of -I for system headers? That would be ideal but unfortunately QEMU_CFLAGS also contains output of pkg-config --cflags which uses -I for header paths. We would need to convert them to -isystem. And is -isystem

[Qemu-devel] [PATCH] monitor: fix build breakage for !CONFIG_VNC

2011-08-10 Thread Jamie Iles
Commit c62f6d1 (monitor: fix build breakage with --disable-vnc) conditionalised some VNC setup code but left an unused variable. Move the variable into the conditional code to fix the build breakage. Cc: Luiz Capitulino lcapitul...@redhat.com Cc: Markus Armbruster arm...@redhat.com

[Qemu-devel] Multithreaded ARM regression from 0d101938

2011-08-10 Thread David Gilbert
Hi, I've just filed bug 823902 ( https://bugs.launchpad.net/qemu/+bug/823902 ) which is a mutlthreaded user mode ARM crash that goes away if I revert 0d101938 ( tcg: Reload local variables after return from longjmp ). It's actually a bit more complicated than that, in that: 1) It fails

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Shribman, Aidan
-Original Message- From: Anthony Liguori [mailto:anth...@codemonkey.ws] Sent: Monday, August 08, 2011 7:56 PM To: Avi Kivity Cc: Blue Swirl; Stefan Hajnoczi; Shribman, Aidan; qemu-devel Developers Subject: Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

[Qemu-devel] Memory API feature page

2011-08-10 Thread Avi Kivity
While the memory API is not a feature, it deserves a feature page, so I can put the TODO list there: http://wiki.qemu.org/Features/Memory_API Please consider helping out. Device conversions are easy, if boring, and there's no end to them. There is a lot of more interesting work on the

Re: [Qemu-devel] [PATCH 2/6] Implement POWER7's CFAR in TCG

2011-08-10 Thread Alexander Graf
On 08/04/2011 09:02 AM, David Gibson wrote: This patch implements support for the CFAR SPR on POWER7 (Come From Address Register), which snapshots the PC value at the time of a branch or an rfid. The latest powerpc-next kernel also catches it and can show it in xmon or in the signal frames.

Re: [Qemu-devel] [PATCH 2/2] ui/spice-core: report version in 'info spice'

2011-08-10 Thread Gerd Hoffmann
Hi, Argh. Sent v3 with the three ints. Do you prefer a single version? Yea, would be great. thanks, Gerd

Re: [Qemu-devel] [PATCH v2 0/4] usb/hid: bugfixes, more on usb and hid split

2011-08-10 Thread Gerd Hoffmann
On 08/09/11 23:54, Michael Walle wrote: This USB patchset moves the HID VM state stuff from usb-hid.c to hid.c, so it can be reused by other devices. changes v1-v2: - don't reorder vmstate field to be backward compatible Michael Walle (4): hid: register kbd hander in init() hid:

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Avi Kivity
On 08/10/2011 06:07 PM, Shribman, Aidan wrote: XBZRLE will very rarely (if at all) degrade live-migration as it runs at ~2 GB/s or 16 Gbps. Additionally XBZRLE could get even faster by using 128bit registers instead of the 64bit registers used currently. IMO XBZRLE could safely be used by

Re: [Qemu-devel] [PATCH] qxl: allowing the command rings to be not empty when spice worker is stopped RHBZ #728984

2011-08-10 Thread Gerd Hoffmann
On 08/09/11 15:12, Yonit Halperin wrote: same as 8927cfbba232e28304734f7afd463c1b84134031, but for qxl_check_state, that was triggered by qxl_pre_load (which calls qxl_hard_reset, which calls qxl_soft_reset), and caused the migration target to crash. Added to spice patch queue. thanks,

Re: [Qemu-devel] pseries machine updates

2011-08-10 Thread Alexander Graf
On 08/04/2011 09:02 AM, David Gibson wrote: Hi Alex, Here's another batch of assorted updates for the pseries machine. Looks pretty nice. Please update patch 2/6 with the bug you found and the whitespace problems. I'll put the others into my tree already. Alex

Re: [Qemu-devel] [PATCH 2/2] i_generation / st_gen support for handle based fs driver

2011-08-10 Thread Stefan Hajnoczi
On Fri, Aug 5, 2011 at 1:53 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Fri, Aug 5, 2011 at 12:32 PM, Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com wrote: On Fri, 5 Aug 2011 10:24:42 +0100, Stefan Hajnoczi stefa...@gmail.com wrote: On Fri, Aug 5, 2011 at 7:40 AM, Aneesh Kumar K.V

Re: [Qemu-devel] [PATCH 2/2] i_generation / st_gen support for handle based fs driver

2011-08-10 Thread Stefan Hajnoczi
On Wed, Aug 10, 2011 at 4:17 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Fri, Aug 5, 2011 at 1:53 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Fri, Aug 5, 2011 at 12:32 PM, Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com wrote: On Fri, 5 Aug 2011 10:24:42 +0100, Stefan Hajnoczi

Re: [Qemu-devel] [PATCH 13/16] usb-hid: add hid_has_events()

2011-08-10 Thread Gerd Hoffmann
Hi, - -/* Force poll routine to be run and grab input the first time. */ -us-changed = 1; USB tablet does not work on the winxp guest. I think this code can't be removed. Attached patch should fix that. cheers, Gerd commit 21635e121ae0f0ab7874152a7c2f96e9d8cd642f Author: Gerd

Re: [Qemu-devel] [PATCH 3/6] pseries: Add real mode debugging hcalls

2011-08-10 Thread Alexander Graf
On 08/04/2011 09:02 AM, David Gibson wrote: From: Ben Herrenschmidtb...@kernel.crashing.org PAPR systems support several hypercalls intended for use in real mode debugging tools. These implement reads and writes to arbitrary guest physical addresses. This is useful for real mode software

Re: [Qemu-devel] [ANNOUNCE] QEMU 0.15.0 Release

2011-08-10 Thread Rick Vernam
On Tuesday 09 August 2011 21:34:10 Natalia Portillo wrote: Hi, El 08/08/2011, a las 20:16, Anthony Liguori escribió: Hi, On behalf of the entire QEMU team, I'm please to announce the release of QEMU 0.15.0. This is the first release of the 0.15 branch and is intended for production

[Qemu-devel] [PATCH v4] ui/spice-core: report compiled-version in info spice/query-spice

2011-08-10 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- ui/spice-core.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index 3d77c01..b9d1cd4 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -372,6 +372,8 @@ void

[Qemu-devel] [PATCH v5] ui/spice-core: report compiled-version in info spice/query-spice

2011-08-10 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- ui/spice-core.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index 3d77c01..768378e 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -372,6 +372,8 @@ void

Re: [Qemu-devel] [PATCH v4] ui/spice-core: report compiled-version in info spice/query-spice

2011-08-10 Thread Alon Levy
On Wed, Aug 10, 2011 at 06:21:16PM +0300, Alon Levy wrote: Signed-off-by: Alon Levy al...@redhat.com NACK, next version uses snprintf. --- ui/spice-core.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index

Re: [Qemu-devel] pseries machine updates

2011-08-10 Thread Alexander Graf
On 08/10/2011 05:16 PM, Alexander Graf wrote: On 08/04/2011 09:02 AM, David Gibson wrote: Hi Alex, Here's another batch of assorted updates for the pseries machine. Looks pretty nice. Please update patch 2/6 with the bug you found and the whitespace problems. I'll put the others into my

Re: [Qemu-devel] [PATCH v5] ui/spice-core: report compiled-version in info spice/query-spice

2011-08-10 Thread Gerd Hoffmann
+char version_string[20]; // 12 = |255.255.255\0| is the max checkpatch reports: ERROR: do not use C99 // comments #19: FILE: ui/spice-core.c:393: +char version_string[20]; // 12 = |255.255.255\0| is the max cheers, Gerd

Re: [Qemu-devel] [PATCH 1/2] qxl: unbreak after memory API conversion

2011-08-10 Thread Gerd Hoffmann
On 08/09/11 22:53, Alon Levy wrote: Break is only noticable with newer spice-server library (0.8.2 release or 0.9.0 and newer on master branch). ioport_write's val was changed from uint32_t to uint64_t, this broke two printfs. Use PRId64 instead of %d. Added to the spice patch queue. thanks,

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Anthony Liguori
On 08/10/2011 10:12 AM, Avi Kivity wrote: On 08/10/2011 06:07 PM, Shribman, Aidan wrote: XBZRLE will very rarely (if at all) degrade live-migration as it runs at ~2 GB/s or 16 Gbps. Additionally XBZRLE could get even faster by using 128bit registers instead of the 64bit registers used

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Avi Kivity
On 08/10/2011 06:58 PM, Anthony Liguori wrote: I don't think we should couple the two features together. ASN.1 is orthogonal to capabilities. Capabilities are a hard requirement before merging any new type of compression algorithm IMO. Right now we have capabilties in the form of -help

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Anthony Liguori
On 08/10/2011 11:08 AM, Avi Kivity wrote: On 08/10/2011 06:58 PM, Anthony Liguori wrote: I don't think we should couple the two features together. ASN.1 is orthogonal to capabilities. Capabilities are a hard requirement before merging any new type of compression algorithm IMO. Right now

Re: [Qemu-devel] [PATCH 21/24] isa: add isa_address_space()

2011-08-10 Thread Richard Henderson
On 08/08/2011 10:07 AM, Avi Kivity wrote: A helper that returns the address space used by ISA devices. Useful for getting rid of isa_mem_base, multiple ISA buses, or ISA buses behind bridges. Signed-off-by: Avi Kivity a...@redhat.com --- hw/isa-bus.c |6 ++ hw/isa.h |1 +

Re: [Qemu-devel] [PATCH 21/24] isa: add isa_address_space()

2011-08-10 Thread Richard Henderson
On 08/10/2011 09:24 AM, Richard Henderson wrote: Of course, as far as I can see, this variable is only used by the VGA devices. Surely we can arrange to pass down some address space during setup of the VGA? ... Which seems to be what you've done in patch 23. So what's the point of this

[Qemu-devel] [PATCH vLast] ui/spice-core: report compiled-version in info spice/query-spice

2011-08-10 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- ui/spice-core.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/ui/spice-core.c b/ui/spice-core.c index 3d77c01..8bb62ea 100644 --- a/ui/spice-core.c +++ b/ui/spice-core.c @@ -372,6 +372,8 @@ void

Re: [Qemu-devel] [PATCH 21/24] isa: add isa_address_space()

2011-08-10 Thread Avi Kivity
On 08/10/2011 07:24 PM, Richard Henderson wrote: @@ -202,4 +203,9 @@ static char *isabus_get_fw_dev_path(DeviceState *dev) return strdup(path); } +MemoryRegion *isa_address_space(ISADevice *dev) +{ +return get_system_memory(); +} + This does not help get rid of

Re: [Qemu-devel] [PATCH 21/24] isa: add isa_address_space()

2011-08-10 Thread Avi Kivity
On 08/10/2011 07:26 PM, Richard Henderson wrote: On 08/10/2011 09:24 AM, Richard Henderson wrote: Of course, as far as I can see, this variable is only used by the VGA devices. Surely we can arrange to pass down some address space during setup of the VGA? ... Which seems to be what

Re: [Qemu-devel] [PATCH 21/24] isa: add isa_address_space()

2011-08-10 Thread Avi Kivity
On 08/10/2011 07:35 PM, Avi Kivity wrote: On 08/10/2011 07:26 PM, Richard Henderson wrote: On 08/10/2011 09:24 AM, Richard Henderson wrote: Of course, as far as I can see, this variable is only used by the VGA devices. Surely we can arrange to pass down some address space during setup of

[Qemu-devel] [PULL] seabios update

2011-08-10 Thread Gerd Hoffmann
Hi, This pull brings the latest seabios bits to master, featuring AHCI support with all bells and whistles. So you can finally boot from your AHCI disk, and bootorder works too! The pci device handling got a major update too. Most visible is the two-pass ressource allocation, which will

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Avi Kivity
On 08/10/2011 07:23 PM, Anthony Liguori wrote: Right now we have capabilties in the form of -help output. If -help says -no-xzbrle disable xzbrle support (or -migration-compression xzbrle=off, or something) that's sufficient for management tools. This is static, not dynamic. You may

[Qemu-devel] [PATCH V6 0/9] Add TPM support to SeaBIOS

2011-08-10 Thread Stefan Berger
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) - ACPI support for the TPM device (SSDT table) - ACPI support for measurement logging (TCPA table) - Support for

[Qemu-devel] [PATCH V6 6/9] Add measurement code to the BIOS

2011-08-10 Thread Stefan Berger
This patch adds invocactions of functions that measure various parts of the code and data through various parts of the BIOS code. It follows TCG specifications on what needs to be measured. It also adds the implementation of the called functions. Reference for what needs to be measured can be

[Qemu-devel] [PATCH V6 8/9] Support for Qemu-provided measurements

2011-08-10 Thread Stefan Berger
This patch adds support for measurements provided by Qemu via the firmware interface. In the case where Qemu was started with the -kernel, -initrd and -append command lines, Qemu hashes the kernel and initrd files as well as the command line parameters and provides the measurements to SeaBIOS via

[Qemu-devel] [Bug 824074] [NEW] Provide runtime option to expose the supported list of keymaps for vnc

2011-08-10 Thread Jorge Manuel B. S. Vicetto
Public bug reported: As discussed in the ganeti group[1], I'm opening this bug to request that qemu provides a runtime command or switch to list the supported keymaps for vnc. [1] - http://groups.google.com/group/ganeti/browse_thread/thread/dd524f5311d8d79e ** Affects: qemu Importance:

[Qemu-devel] fedora 16 build failure

2011-08-10 Thread Gerd Hoffmann
CCui/vnc-tls.o ui/vnc-tls.c: In function ‘vnc_tls_client_setup’: ui/vnc-tls.c:313:9: error: ‘gnutls_kx_set_priority’ is deprecated (declared at /usr/include/gnutls/compat.h:342) [-Werror=deprecated-declarations] ui/vnc-tls.c:320:9: error: ‘gnutls_certificate_type_set_priority’ is

[Qemu-devel] openbsd build failure (+warnings)

2011-08-10 Thread Gerd Hoffmann
LINK qemu-ga /usr/local/lib/libglib-2.0.so.2600.0: warning: vsprintf() is often misused, please use vsnprintf() /usr/local/lib/libglib-2.0.so.2600.0: warning: strcpy() is almost always misused, please use strlcpy() /usr/local/lib/libcurl.so.17.0: warning: sprintf() is often misused, please

Re: [Qemu-devel] Safely reopening image files by stashing fds

2011-08-10 Thread Blue Swirl
On Wed, Aug 10, 2011 at 7:58 AM, Kevin Wolf kw...@redhat.com wrote: Am 09.08.2011 21:39, schrieb Blue Swirl: On Tue, Aug 9, 2011 at 12:24 PM, Kevin Wolf kw...@redhat.com wrote: Am 09.08.2011 14:00, schrieb Stefan Hajnoczi: On Tue, Aug 09, 2011 at 01:39:13PM +0200, Kevin Wolf wrote: Am

[Qemu-devel] openbsd build failure (stable branch)

2011-08-10 Thread Gerd Hoffmann
LINK i386-bsd-user/qemu-i386 /usr/bin/ld:/home/buildbot/slave-spunk/openbsd/build/x86_64.ld:41: syntax error collect2: ld returned 1 exit status gmake[1]: *** [qemu-i386] Error 1 gmake: *** [subdir-i386-bsd-user] Error 2

Re: [Qemu-devel] build failure with coroutine-gthread

2011-08-10 Thread Blue Swirl
On Wed, Aug 10, 2011 at 9:14 AM, Stefan Hajnoczi stefa...@linux.vnet.ibm.com wrote: On Tue, Aug 09, 2011 at 05:28:17PM +, Blue Swirl wrote: On Mon, Aug 8, 2011 at 9:29 AM, Stefan Hajnoczi stefa...@gmail.com wrote: On Mon, Aug 8, 2011 at 10:01 AM, Aneesh Kumar K.V

[Qemu-devel] [PATCH V6 3/9] Add public get_rsdp function

2011-08-10 Thread Stefan Berger
This patch adds a global get_rsdp() function call and refactors find_resume_vector() to call it. Signed-off-by: Stefan Berger stef...@linux.vnet.ibm.com --- src/acpi.c | 12 ++-- src/acpi.h |1 + 2 files changed, 11 insertions(+), 2 deletions(-) Index: seabios/src/acpi.c

Re: [Qemu-devel] [PATCH 2/2] i_generation / st_gen support for handle based fs driver

2011-08-10 Thread Aneesh Kumar K.V
On Wed, 10 Aug 2011 16:17:22 +0100, Stefan Hajnoczi stefa...@gmail.com wrote: On Fri, Aug 5, 2011 at 1:53 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Fri, Aug 5, 2011 at 12:32 PM, Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com wrote: On Fri, 5 Aug 2011 10:24:42 +0100, Stefan Hajnoczi

[Qemu-devel] [PATCH V6 9/9] Optional tests for the TIS interface

2011-08-10 Thread Stefan Berger
This patch adds an optional test suite (CONFIG_TIS_TEST) for the TIS interface to SeaBIOS. If compiled into the BIOS, it can be invoked through the TPM-specific menu item 8. 1. Enable TPM 2. Disable TPM 3. Activate TPM 4. Deactivate TPM 5. Clear ownership 6. Allow installation of owner 7. Prevent

[Qemu-devel] Fwd: VNC server running on `127.0.0.1:5900'

2011-08-10 Thread Mulyadi Santosa
-- Forwarded message -- From: Mulyadi Santosa mulyadi.sant...@gmail.com Date: Thu, Aug 11, 2011 at 00:42 Subject: Re: [Qemu-devel] VNC server running on `127.0.0.1:5900' To: Nithish R nithi...@gmail.com On Wed, Aug 10, 2011 at 20:33, Nithish R nithi...@gmail.com wrote: make

[Qemu-devel] [PATCH V6 4/9] Implementation of the TCG BIOS extensions

2011-08-10 Thread Stefan Berger
This patch implements the main part of the TCG BIOS extensions. It provides the following functionality: - initialization of the TCPA ACPI table used for logging of measurements - initialization of the TPM by sending a sequence of commands to it - proper setup of the TPM once the BIOS hands over

[Qemu-devel] [PATCH V6 5/9] Support for BIOS interrupt handler

2011-08-10 Thread Stefan Berger
This patch implements the TCG BIOS interrupt handler 1ah. It is for example used by trusted grub. This patch adds an implementation of SHA1 (following NIST specs., IETF RFC 3147 and Wikipedia) for speeding up measurements of code. Trusted Grub for example makes use of this interface and measures

[Qemu-devel] [PATCH V6 7/9] Add a menu for TPM control

2011-08-10 Thread Stefan Berger
This patch provides an addtional menu entry that enables the user to control certain aspects of the TPM. If a working TPM has been detected, the top level BIOS menu will look like this: Press F12 for boot menu. Press F11 to TPM menu. Upon pressing F11 the TPM menu will be shown: 1. Enable TPM

[Qemu-devel] [PATCH V6 1/9] Add an implementation of a TPM TIS driver

2011-08-10 Thread Stefan Berger
This patch adds an implementation of a TPM TIS driver for the TPM TIS emulation supported by Qemu (patches posted, not in git yet). Usage of the driver is broken up into several functions. The driver is cleanly separated from the rest of the code through an interface holding pointers to the

[Qemu-devel] [PATCH V6 2/9] Provide ACPI SSDT table for TPM device + S3 resume support

2011-08-10 Thread Stefan Berger
This patch provides ACPI support for the TPM device. It probes for the TPM device and only if a TPM device is found then the TPM's SSDT and TCPA table are created. This patch also connects them to the RSDT. Since the logging area in the TCPA table requires 64kb, the memory reserved for ACPI

[Qemu-devel] Making QEMU ARM work with Mac OS X

2011-08-10 Thread Sherjil Ozair
Hello everyone, I was just trying to see if I could make qemu work on mac os x, but I don't see any pre-compiled binaries. How should I proceed ? Should I try to compile it myself ? If yes, then what would I need to do that ? Else, is there any place where the pre-compiled binaries are

[Qemu-devel] [PATCH STABLE] qapi: fix build issue due to missing newline in generated header

2011-08-10 Thread Michael Roth
Fixes a build issue on RHEL5, and potentially other distros, where gcc will generate an error due to us not writing a trailing \n when generating *qmp-commands.h Signed-off-by: Michael Roth mdr...@linux.vnet.ibm.com --- scripts/qapi-commands.py |2 +- 1 files changed, 1 insertions(+), 1

Re: [Qemu-devel] [PATCH vLast] ui/spice-core: report compiled-version in info spice/query-spice

2011-08-10 Thread Gerd Hoffmann
On 08/10/11 18:31, Alon Levy wrote: Signed-off-by: Alon Levyal...@redhat.com Queued up. thanks, Gerd

Re: [Qemu-devel] openbsd build failure (stable branch)

2011-08-10 Thread Blue Swirl
On Wed, Aug 10, 2011 at 5:21 PM, Gerd Hoffmann kra...@redhat.com wrote:  LINK  i386-bsd-user/qemu-i386 /usr/bin/ld:/home/buildbot/slave-spunk/openbsd/build/x86_64.ld:41: syntax error collect2: ld returned 1 exit status gmake[1]: *** [qemu-i386] Error 1 gmake: *** [subdir-i386-bsd-user] Error

Re: [Qemu-devel] openbsd build failure (+warnings)

2011-08-10 Thread Blue Swirl
On Wed, Aug 10, 2011 at 5:19 PM, Gerd Hoffmann kra...@redhat.com wrote:  LINK  qemu-ga /usr/local/lib/libglib-2.0.so.2600.0: warning: vsprintf() is often misused, please use vsnprintf() /usr/local/lib/libglib-2.0.so.2600.0: warning: strcpy() is almost always misused, please use strlcpy()

Re: [Qemu-devel] [PATCH 21/24] isa: add isa_address_space()

2011-08-10 Thread Avi Kivity
On 08/10/2011 07:26 PM, Richard Henderson wrote: On 08/10/2011 09:24 AM, Richard Henderson wrote: Of course, as far as I can see, this variable is only used by the VGA devices. Surely we can arrange to pass down some address space during setup of the VGA? ... Which seems to be what

[Qemu-devel] [Bug 789831] Re: kvm locks up when out of disk space

2011-08-10 Thread Bug Watch Updater
** Changed in: qemu-kvm (Debian) Status: Unknown = Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/789831 Title: kvm locks up when out of disk space Status in QEMU: Invalid

Re: [Qemu-devel] [PATCH v4] XBZRLE delta for live migration of large memory apps

2011-08-10 Thread Anthony Liguori
On 08/10/2011 11:40 AM, Avi Kivity wrote: On 08/10/2011 07:23 PM, Anthony Liguori wrote: Right now we have capabilties in the form of -help output. If -help says -no-xzbrle disable xzbrle support (or -migration-compression xzbrle=off, or something) that's sufficient for management tools.

[Qemu-devel] [PATCH V7 10/13] Encrypt state blobs using AES CBC encryption

2011-08-10 Thread Stefan Berger
This patch adds encryption of the individual state blobs that are written into the block storage. The 'directory' at the beginnig of the block storage is not encrypted. The encryption support added in this patch would also work if QCoW2 was not to be used as the (only) image file format to store

[Qemu-devel] [PATCH V7 02/13] Add TPM (frontend) hardware interface (TPM TIS) to Qemu

2011-08-10 Thread Stefan Berger
This patch adds the main code of the TPM frontend driver, the TPM TIS interface, to Qemu. The code is largely based on the previous implementation for Xen but has been significantly extended to meet the standard's requirements, such as the support for changing of localities and all the

[Qemu-devel] [PATCH V7 08/13] Introduce file lock for the block layer

2011-08-10 Thread Stefan Berger
This patch introduces file locking via fcntl() for the block layer so that concurrent access to files shared by 2 Qemu instances, for example via NFS, can be serialized. This feature is useful primarily during initial phases of VM migration where the target machine's TIS driver validates the block

Re: [Qemu-devel] [PATCH v4 23/39] lsi53c895a: convert to memory API

2011-08-10 Thread Gerhard Wiesinger
Hello Avi, qemu/qemu-kvm doesn't boot any more with LSI option rom (-option-rom 8xx_64.rom). Guess it comes from changes of the memory API. Also latest git version from seabios. Sorry, no further details currently available. Ciao, Gerhard -- http://www.wiesinger.com/ On Mon, 8 Aug 2011,

[Qemu-devel] [PATCH V7 12/13] Support for taking measurements when kernel etc. are passed to Qemu

2011-08-10 Thread Stefan Berger
This patch adds support for hashing the kernel and initrd as well as the command line parameters in the case that Qemu was provided the -kernel, -initrd and -apppend command line parameters. The hashes are then passed to SeaBIOS for logging. Typically SeaBIOS would take those measurements

[Qemu-devel] [PATCH V7 13/13] Add a TPM backend null driver implementation

2011-08-10 Thread Stefan Berger
This patch adds a TPM null driver implementation acting as a backend for the TIS hardware emulation. The NULL driver responds to all commands with a TPM fault response. To use this null driver, use either -tpm null or -tpmdev null,id=tpm0 -device tpm-tis,tpmdev=tpm0 as parameters on the

[Qemu-devel] [PATCH V7 07/13] Implementation of the libtpms-based backend

2011-08-10 Thread Stefan Berger
This patch provides the glue for the TPM TIS interface (frontend) to the libtpms that provides the actual TPM functionality. Some details: This part of the patch provides support for the spawning of a thread that will interact with the libtpms-based TPM. It expects a signal from the frontend to

[Qemu-devel] [PATCH V7 04/13] Add tpm_tis driver to build process

2011-08-10 Thread Stefan Berger
The TPM interface (tpm_tis) needs to be explicitly enabled via ./configure --enable-tpm. This patch also restricts the building of the TPM support to i386 and x86_64 targets since only there it is currently supported. This prevents that one will end up with support for a frontend but no available

[Qemu-devel] [PATCH 1/8] Move vm_state_notify() prototype from cpus.h to sysemu.h

2011-08-10 Thread Luiz Capitulino
It's where all the state handling functions prototypes are located. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- cpus.h |1 - sysemu.h |1 + 2 files changed, 1 insertions(+), 1 deletions(-) diff --git a/cpus.h b/cpus.h index f42b54e..5885885 100644 --- a/cpus.h +++

[Qemu-devel] [PATCH v2 0/8]: Introduce the RunState type

2011-08-10 Thread Luiz Capitulino
It replaces the VMSTOP macros and allows us to drop some global variables. Additionally, the problem with issuing 'cont' when the VM is in bad state is addressed and we make the current state available in QMP and HMP. changelog - v2 o Rename the new type from QemuState to RunState

[Qemu-devel] [PATCH 8/8] HMP: info status: Print the VM state

2011-08-10 Thread Luiz Capitulino
Today our printf format for the info status command is: VM status: %s Where the string can be running, running (single step mode) or paused. This commit extends it to: VM status: %s (%s) The second string corresponds to the status field as returned by the query-status QMP command and it's

  1   2   >