[Qemu-devel] [PATCH 0/3] Intruduce qemu-ga-client

2012-09-14 Thread Ryota Ozaki
This patch series adds qemu-ga-client that is a command line script to use guest agent functions easily. Ryota Ozaki (3): Make negotiation optional in QEMUMonitorProtocol Support settimeout in QEMUMonitorProtocol Add qemu-ga-client script QMP/qemu-ga-client | 299 ++

[Qemu-devel] [PATCH 3/3] Add qemu-ga-client script

2012-09-14 Thread Ryota Ozaki
This is an easy-to-use QEMU guest agent client written in Python. It simply provides commands to call guest agent functions like ping, fsfreeze and shutdown. Additionally, it provides extra useful commands, e.g, cat, ifconfig and reboot, by using guet agent functions. Examples: $ export QGA_CLIE

[Qemu-devel] [PATCH 1/3] Make negotiation optional in QEMUMonitorProtocol

2012-09-14 Thread Ryota Ozaki
This is a preparation for qemu-ga-client which uses QEMUMonitorProtocol class. The class tries to negotiate capabilities on connect, however, qemu-ga doesn't suppose it and fails. This change makes the negotiation optional, though it's still performed by default for compatibility. Signed-off-by:

[Qemu-devel] [PATCH 2/3] Support settimeout in QEMUMonitorProtocol

2012-09-14 Thread Ryota Ozaki
This method is used in the following qemu-ga-client script to implement non-blocking operations. Signed-off-by: Ryota Ozaki --- QMP/qmp.py | 5 + 1 file changed, 5 insertions(+) diff --git a/QMP/qmp.py b/QMP/qmp.py index 5a573e1..33c7d36 100644 --- a/QMP/qmp.py +++ b/QMP/qmp.py @@ -162,3 +1

[Qemu-devel] buildbot failure in qemu on block_openbsd_current

2012-09-14 Thread qemu
The Buildbot has detected a new failure on builder block_openbsd_current while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/block_openbsd_current/builds/336 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: brad_openbsd_cur

Re: [Qemu-devel] [PATCH 6/8] QAPI: add command for live block commit, 'block-commit'

2012-09-14 Thread Jeff Cody
On 09/14/2012 09:05 PM, Eric Blake wrote: > On 09/14/2012 07:41 AM, Jeff Cody wrote: >> The command for live block commit is added, which has the following >> arguments: >> >> device: the block device to perform the commit on (mandatory) >> base: the base image to commit into; optional (if not sp

[Qemu-devel] [PATCH] SCSI: Standard INQUIRY data should report HiSup flag as set.

2012-09-14 Thread Ronnie Sahlberg
QEMU as far as I know only reports LUN numbers using the modes that are described in SAM4. As such, since all LUN numbers generated by the SCSI emulation in QEMU follow SAM4, we should set the HiSup bit in the standard INQUIRY data to indicate such. >From SAM4: 4.6.3 LUNs overview All LUN form

[Qemu-devel] [PATCH] SCSI disk, set HISUP bit in INQ data

2012-09-14 Thread Ronnie Sahlberg
Paolo, List Please find a trivial patch that make SCSI-DISK set the HISUP bit in the INQ data. Since I think all LUN numbers are reported as SAM4 describes, we should set this bit. Its already sed in SCSI-BUS regards ronnie sahlberg

[Qemu-devel] buildbot failure in qemu on block_mingw32

2012-09-14 Thread qemu
The Buildbot has detected a new failure on builder block_mingw32 while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/block_mingw32/builds/325 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: kraxel_rhel61 Build Reason: The

Re: [Qemu-devel] [PATCH 6/8] QAPI: add command for live block commit, 'block-commit'

2012-09-14 Thread Eric Blake
On 09/14/2012 07:41 AM, Jeff Cody wrote: > The command for live block commit is added, which has the following > arguments: > > device: the block device to perform the commit on (mandatory) > base: the base image to commit into; optional (if not specified, > it is the underlying original

[Qemu-devel] buildbot failure in qemu on default_openbsd_current

2012-09-14 Thread qemu
The Buildbot has detected a new failure on builder default_openbsd_current while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/default_openbsd_current/builds/374 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: brad_openbsd

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Isaku Yamahata
On Fri, Sep 14, 2012 at 03:01:55PM -0400, Jason Baron wrote: > On Sat, Sep 15, 2012 at 02:34:18AM +0900, Isaku Yamahata wrote: > > On Fri, Sep 14, 2012 at 11:23:29AM -0400, Jason Baron wrote: > > > On Sat, Sep 15, 2012 at 12:14:01AM +0900, Isaku Yamahata wrote: > > > > On Thu, Sep 13, 2012 at 04:12

[Qemu-devel] [Bug 1033727] Re: USB passthrough doesn't work anymore with qemu-kvm 1.1.1

2012-09-14 Thread Klaus Hochlehnert
I just tested with qemu-kvm 1.2.0 But basically still the same problem. Device shows up in the guest, but the software can't access it anymore. Just the guest isn't unresponsive anymore. And the dmesg message happens now only once in a while (but always two very close together): [ 4066.239358] u

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

2012-09-14 Thread Alex Williamson
Enabled for all softmmu guests supporting PCI on Linux hosts. Note that currently only x86 hosts have the kernel side VFIO IOMMU support for this. PPC (g3beige) is the only non-x86 guest known to work. ARM (veratile) hangs in firmware, others untested. Signed-off-by: Alex Williamson --- MAINT

Re: [Qemu-devel] [libvirt] [PATCH v2 1/4] config: Introduce for SPICE graphics

2012-09-14 Thread Eric Blake
[adding qemu] On 09/14/2012 11:47 AM, Daniel P. Berrange wrote: > On Fri, Sep 14, 2012 at 07:34:50PM +0200, Michal Privoznik wrote: >> With this element users will control how SPICE >> server behaves upon migration. For now, there's >> just one attribute 'seamless' turning seamless >> migration on

Re: [Qemu-devel] [PATCH v5 0/4] VFIO-based PCI device assignment

2012-09-14 Thread Alex Williamson
On Fri, 2012-09-14 at 17:01 -0600, Alex Williamson wrote: > Same goodness as v4, plus: > > - Addressed comments by Blue Swirl (thanks for the review) >(hopefully w/o breaking anything wrt slow bar endianness) > - Fixed a couple checkpatch warnings that snuck in > > BTW, this works fine with

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

2012-09-14 Thread Alex Williamson
Based on Linux as of 1a95620. Signed-off-by: Alex Williamson --- linux-headers/linux/vfio.h | 368 1 file changed, 368 insertions(+) create mode 100644 linux-headers/linux/vfio.h diff --git a/linux-headers/linux/vfio.h b/linux-headers/linux/vfio.h

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

2012-09-14 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 a639c5b..605102f 100755 --- a/scripts/update-linux-headers.sh +++ b/scripts/update-lin

[Qemu-devel] [PATCH v5 0/4] VFIO-based PCI device assignment

2012-09-14 Thread Alex Williamson
Same goodness as v4, plus: - Addressed comments by Blue Swirl (thanks for the review) (hopefully w/o breaking anything wrt slow bar endianness) - Fixed a couple checkpatch warnings that snuck in BTW, this works fine with Jason's Q35 patches though we will need to add INTx routing support for

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 21:51, Blue Swirl ha scritto: > On Fri, Sep 14, 2012 at 1:17 PM, Paolo Bonzini wrote: >> Hi all, >> >> here is a proposal for moving around 150 C files currently in the >> toplevel directory to separate, well-delimited subdirectories. Header >> files would be moved for now in includ

Re: [Qemu-devel] [PATCH 2/8] block: add live block commit functionality

2012-09-14 Thread Jeff Cody
On 09/14/2012 02:23 PM, Eric Blake wrote: > On 09/14/2012 10:07 AM, Jeff Cody wrote: >>> Question: is it valid to have a qcow2 file whose size is smaller than >>> it's backing image? >> >> I don't think so... however: >> >>> Suppose I have base[1M] <- mid[2M] <- top[3M] <- >>> active[3M], and requ

Re: [Qemu-devel] [PATCH] slirp: Remove unused return value of tftp_send_next_block

2012-09-14 Thread Hervé Poussineau
Jan Kiszka a écrit : No caller actually makes use of this value, so let's simplify the code. Signed-off-by: Jan Kiszka Acked-by: Hervé Poussineau --- Applies on top of http://thread.gmane.org/gmane.comp.emulators.qemu/169909 slirp/tftp.c | 14 +- 1 files changed, 5 inserti

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Blue Swirl
On Fri, Sep 14, 2012 at 1:17 PM, Paolo Bonzini wrote: > Hi all, > > here is a proposal for moving around 150 C files currently in the > toplevel directory to separate, well-delimited subdirectories. Header > files would be moved for now in include/, preparing for subsequent > reorganization of he

Re: [Qemu-devel] qemu-system-i386 vs qemu-system-x86_64 ?

2012-09-14 Thread Blue Swirl
On Fri, Sep 14, 2012 at 7:39 AM, Michael Tokarev wrote: > What's the difference between the two except that the > latter adds some more instructions (actually whole new > subsytem) to the former? Why do we need -i386, what > -x86_64 does not do which does -i386? i386 should not allow executing x

Re: [Qemu-devel] [PATCH v2] m68k: implement move to/from usp register instruction

2012-09-14 Thread Richard Henderson
On 09/13/2012 04:37 PM, g...@snapgear.com wrote: > From: Greg Ungerer > > Fill out the code support for the move to/from usp instructions. They are > being decoded, but there is no code to support their actions. So add it. > > Current versions of Linux running on the ColdFire 5208 use these inst

Re: [Qemu-devel] [PATCH 24/25] Add a fallback bios file search, if -L fails.

2012-09-14 Thread Blue Swirl
On Thu, Sep 13, 2012 at 8:12 PM, Jason Baron wrote: > If -L is specified, and qemu does not find the bios file in , then > the search fails. Add infrastructure such that the search will continue in > the default paths, if not found in the -L path. > > Signed-off-by: Jason Baron > --- > vl.c |

Re: [Qemu-devel] [PATCH 8/9] MicroBlaze: Add a config that is dynamically set up by a device tree file.

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 12:17 AM, Peter Crosthwaite wrote: > On Tue, 2012-09-11 at 19:27 +, Blue Swirl wrote: >> On Mon, Sep 10, 2012 at 12:20 AM, wrote: >> > From: Chris Wulff >> > >> > +/* >> > + * Xilinx ethernetlite device >> > + */ >> > +static void xilinx_ethlite_probe(void *fdt, int

Re: [Qemu-devel] [PATCH] tci: Support deposit operations

2012-09-14 Thread Stefan Weil
Am 10.09.2012 16:08, schrieb Richard Henderson: On Sat, 2012-09-08 at 16:12 +0200, Stefan Weil wrote: +tmp16 = *tb_ptr++; +tmp8 = *tb_ptr++; +tmp32 = (((1<< tmp8) - 1)<< tmp16); +tci_write_reg32(t0, (t1& ~tmp32) | ((t2<< tmp16)& tmp32)); Use

Re: [Qemu-devel] [PATCH V2 1/6] libqblock API design

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 8:19 AM, Kevin Wolf wrote: > Am 11.09.2012 22:28, schrieb Blue Swirl: >> On Mon, Sep 10, 2012 at 8:26 AM, Wenchao Xia >> wrote: >>> This patch contains the major APIs in the library. >>> Important APIs: >>> 1 QBroker. These structure was used to retrieve errors, every

Re: [Qemu-devel] [PATCH 12/25] q35: Re-base q35 to 1.2

2012-09-14 Thread Blue Swirl
On Thu, Sep 13, 2012 at 8:12 PM, Jason Baron wrote: > Rebase q35 to 1.2 - memory api updates, acpi updates, qom... > > Signed-off-by: Jason Baron > --- > hw/acpi_ich9.c | 65 > hw/acpi_ich9.h |9 +- > hw/pc.h|2 + > hw/pc_piix.c |4 +- > hw/pc_q35.c| 189 +++

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Jason Baron
On Sat, Sep 15, 2012 at 02:34:18AM +0900, Isaku Yamahata wrote: > On Fri, Sep 14, 2012 at 11:23:29AM -0400, Jason Baron wrote: > > On Sat, Sep 15, 2012 at 12:14:01AM +0900, Isaku Yamahata wrote: > > > On Thu, Sep 13, 2012 at 04:12:34PM -0400, Jason Baron wrote: > > > > 2) hotplug > > > > I've added

Re: [Qemu-devel] [PATCH 23/25] q35: add acpi-based pci hotplug.

2012-09-14 Thread Blue Swirl
On Thu, Sep 13, 2012 at 8:12 PM, Jason Baron wrote: > Add piix style acpi hotplug to q35. > > Signed-off-by: Jason Baron > --- > hw/acpi_ich9.c | 173 > +++- > hw/acpi_ich9.h | 10 +++ > 2 files changed, 182 insertions(+), 1 deletions(-) >

Re: [Qemu-devel] [PATCH 07/25] pc/piix_pci: factor out smram/pam logic

2012-09-14 Thread Blue Swirl
On Thu, Sep 13, 2012 at 8:12 PM, Jason Baron wrote: > From: Isaku Yamahata > > Factor out smram/pam logic for later use. > Which will be used by q35 too. > > [jba...@redhat.com: changes for updated memory API] > Signed-off-by: Isaku Yamahata > Signed-off-by: Jason Baron > --- > hw/i386/Makefil

Re: [Qemu-devel] [PATCH 2/8] block: add live block commit functionality

2012-09-14 Thread Eric Blake
On 09/14/2012 10:07 AM, Jeff Cody wrote: >> Question: is it valid to have a qcow2 file whose size is smaller than >> it's backing image? > > I don't think so... however: > >> Suppose I have base[1M] <- mid[2M] <- top[3M] <- >> active[3M], and request to commit top into base. This bdrv_truncate(

Re: [Qemu-devel] [PATCH 02/10] spice: split qemu_spice_create_update

2012-09-14 Thread Blue Swirl
On Thu, Sep 13, 2012 at 8:45 AM, Gerd Hoffmann wrote: > Creating one function which creates a single update for a given > rectangle. And one (for now) pretty simple wrapper around it to > queue up screen updates for the dirty region. > > [ v2: also update bounding box ] > > Signed-off-by: Gerd Ho

Re: [Qemu-devel] [PATCH V2 2/6] libqblock type and structure defines

2012-09-14 Thread Blue Swirl
On Thu, Sep 13, 2012 at 3:49 AM, Eric Blake wrote: > On 09/12/2012 09:33 PM, Eric Blake wrote: >>> OK ,then I think >>> #if __GNUC__ >= 4 >>> >>> #else >>> [warn name space pollution may happen] >>> #endif >>> would be better. >> >> It may be shorter, but it is definitely not better, at le

Re: [Qemu-devel] [PATCH] Basic support for ARM A15 "architectured" (cp15) timers

2012-09-14 Thread Peter Maydell
On 14 September 2012 18:26, Blue Swirl wrote: > On Wed, Sep 12, 2012 at 11:49 AM, Daniel Forsgren >> +++ qemu-linaro-1.1.50-2012.07-modified/target-arm/helper.c 2012-09-12 >> 13:15:45.544424842 +0200 >> @@ -1012,9 +1012,11 @@ void register_cp_regs_for_features(ARMCP >> if (arm_feature(en

Re: [Qemu-devel] [PATCH V2 2/6] libqblock type and structure defines

2012-09-14 Thread Blue Swirl
On Tue, Sep 11, 2012 at 10:52 PM, Eric Blake wrote: > On 09/11/2012 02:31 PM, Blue Swirl wrote: >> On Mon, Sep 10, 2012 at 8:26 AM, Wenchao Xia >> wrote: >>> This patch contains type and defines used in APIs, one file for public >>> usage >>> by user, one for libqblock internal usage. >>> > >

Re: [Qemu-devel] [PATCH] w64: Fix calls of TCG helper functions with 5 arguments

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 7:39 PM, Stefan Weil wrote: > Am 12.09.2012 21:14, schrieb Aurelien Jarno: > >> On Wed, Sep 12, 2012 at 07:12:47PM +0100, Peter Maydell wrote: >>> >>> On 12 September 2012 19:03, Stefan Weil wrote: diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c i

Re: [Qemu-devel] [PATCH] configure: fix "--target-list=, , ..." option

2012-09-14 Thread Stefan Weil
Am 14.09.2012 14:53, schrieb Peter Maydell: On 12 September 2012 14:20, Laurent Desnogues wrote: Sorry, I had missed this patch... On Tue, Sep 11, 2012 at 9:02 PM, Eduardo Habkost wrote: commit 66d5499b3754b83c09487259c08fe2ce73188a59 broke the support for comma-separated target lists on th

[Qemu-devel] [PATCH] slirp: Remove unused return value of tftp_send_next_block

2012-09-14 Thread Jan Kiszka
No caller actually makes use of this value, so let's simplify the code. Signed-off-by: Jan Kiszka --- Applies on top of http://thread.gmane.org/gmane.comp.emulators.qemu/169909 slirp/tftp.c | 14 +- 1 files changed, 5 insertions(+), 9 deletions(-) diff --git a/slirp/tftp.c b/sli

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Isaku Yamahata
On Fri, Sep 14, 2012 at 11:23:29AM -0400, Jason Baron wrote: > On Sat, Sep 15, 2012 at 12:14:01AM +0900, Isaku Yamahata wrote: > > On Thu, Sep 13, 2012 at 04:12:34PM -0400, Jason Baron wrote: > > > 2) hotplug > > > I've added piix acpi style hotplug to ich9. > > > > What's the point of this? > > I

Re: [Qemu-devel] [PATCH 4/5] QAPI: Introduce memchar-read QMP command

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 11:57 AM, Lei Li wrote: > Signed-off-by: Lei Li > --- > hmp-commands.hx | 25 + > hmp.c| 18 ++ > hmp.h|1 + > qapi-schema.json | 27 +++ > qemu-char.c | 48 +

Re: [Qemu-devel] [PATCH] Basic support for ARM A15 "architectured" (cp15) timers

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 11:49 AM, Daniel Forsgren wrote: > This patch adds basic support for the "architected" timers (i.e. cp15) > found in A15. It's enough to allow Linux to boot, using arch_timer for > the tick. However - it is not a complete model of the timer block at > large, it is not that

Re: [Qemu-devel] [PATCH 3/5] QAPI: Introduce memchar-write QMP command

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 11:57 AM, Lei Li wrote: > Signed-off-by: Lei Li > --- > hmp-commands.hx | 23 ++ > hmp.c| 19 +++ > hmp.h|1 + > qapi-schema.json | 69 > ++ > qemu-char.c

Re: [Qemu-devel] [PATCH 5/5] HMP: Introduce console command

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 11:57 AM, Lei Li wrote: > Signed-off-by: Lei Li > --- > hmp.c | 42 ++ > monitor.c | 18 ++ > monitor.h |2 ++ > 3 files changed, 62 insertions(+), 0 deletions(-) > > diff --git a/hmp.c b/hmp.c > index 43

Re: [Qemu-devel] [PATCH V2 3/6] libqblock error handling

2012-09-14 Thread Blue Swirl
On Wed, Sep 12, 2012 at 2:58 AM, Wenchao Xia wrote: > 于 2012-9-12 4:32, Blue Swirl 写道: > >> On Mon, Sep 10, 2012 at 8:26 AM, Wenchao Xia >> wrote: >>> >>>This patch contains error handling APIs, which user could call them to >>> get error details. >>> >>> Signed-off-by: Wenchao Xia >>> --- >

[Qemu-devel] [PATCH] configure: Allow builds without any system or user emulation

2012-09-14 Thread Stefan Weil
The old code aborted configure when no emulation target was selected. Even after removing the 'exit 1', it tried to read from STDIN when QEMU was configured with configure' '--disable-user' '--disable-system' This is fixed here. Signed-off-by: Stefan Weil --- This patch can be applied afte

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 18:09, Kevin Wolf ha scritto: > Cleaning up the main directory is probably a good idea. However, we > should be aware that renaming comes with a cost: Some git commands on a > single file need additional options (git log --follow) to work well > across renames, for others you need to

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 16:37, Stefan Weil ha scritto: > Here are some more ideas: > > * Move the target-xxx directories (currently 15) to target/xxx. Yes, of course. > * Add host/xxx for host specific files (xxx = posix, wxx). > Here the currently used win32 might be replaced by either > wxx or by wi

Re: [Qemu-devel] [PATCH 02/25] pci: add opaque argument to pci_map_irq_fn

2012-09-14 Thread Alex Williamson
On Thu, 2012-09-13 at 16:12 -0400, Jason Baron wrote: > From: Isaku Yamahata > > Pass opaque argument to pci_map_irq_fn like pci_set_irq_fn. > ICH9 irq routing is not static, but configurable by chipset configuration > registers, so the corresponding irq mapping function of pci_map_irq_fn > needs

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Kevin Wolf
Am 14.09.2012 15:17, schrieb Paolo Bonzini: > Hi all, > > here is a proposal for moving around 150 C files currently in the > toplevel directory to separate, well-delimited subdirectories. Header > files would be moved for now in include/, preparing for subsequent > reorganization of headers. >

Re: [Qemu-devel] [PATCH 2/8] block: add live block commit functionality

2012-09-14 Thread Jeff Cody
On 09/14/2012 11:45 AM, Eric Blake wrote: > On 09/14/2012 07:41 AM, Jeff Cody wrote: >> This adds the live commit coroutine. This iteration focuses on the >> commit only below the active layer, and not the active layer itself. >> >> The behaviour is similar to block streaming; the sectors are walk

Re: [Qemu-devel] [PATCH 4/8] qerror: new error for live block commit, QERR_TOP_NOT_FOUND

2012-09-14 Thread Eric Blake
On 09/14/2012 07:41 AM, Jeff Cody wrote: > Signed-off-by: Jeff Cody > --- > qerror.h | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/qerror.h b/qerror.h > index d0a76a4..7396184 100644 > --- a/qerror.h > +++ b/qerror.h > @@ -219,6 +219,9 @@ void assert_no_error(Error *err); > #defin

Re: [Qemu-devel] apparently missing yet another notify_event()

2012-09-14 Thread Michael Tokarev
On 14.09.2012 18:47, Michael Tokarev wrote: [] >> >> qemu-kvm -nographic -kernel /boot/vmlinuz-$(uname -r) -append console=ttyS0 >> -serial pty >> >> This hangs till I send a char to the pty. > > And it is even _more_ twisted than that. > > It depends on the timing. If I connect to the pty "to

Re: [Qemu-devel] [PATCH 2/8] block: add live block commit functionality

2012-09-14 Thread Eric Blake
On 09/14/2012 07:41 AM, Jeff Cody wrote: > This adds the live commit coroutine. This iteration focuses on the > commit only below the active layer, and not the active layer itself. > > The behaviour is similar to block streaming; the sectors are walked > through, and anything that exists above 'b

Re: [Qemu-devel] [PATCH 1/8] block: add support functions for live commit, to find and delete images.

2012-09-14 Thread Jeff Cody
On 09/14/2012 11:23 AM, Eric Blake wrote: > On 09/14/2012 07:41 AM, Jeff Cody wrote: >> Add bdrv_find_overlay(), and bdrv_drop_intermediate(). >> >> bdrv_find_overlay(): given 'bs' and the active (topmost) BDS of an image >> chain, >> find the image that is the immediate top o

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Kevin Wolf
Am 14.09.2012 16:12, schrieb Alexander Graf: > > On 14.09.2012, at 16:08, Jason Baron wrote: > >> On Fri, Sep 14, 2012 at 03:56:17PM +0200, Alexander Graf wrote: >>> On 14.09.2012, at 15:50, Jason Baron wrote: >>> On Fri, Sep 14, 2012 at 12:29:17AM +0200, Alexander Graf wrote: > On 13.09

Re: [Qemu-devel] [PATCH 1/8] block: add support functions for live commit, to find and delete images.

2012-09-14 Thread Eric Blake
On 09/14/2012 07:41 AM, Jeff Cody wrote: > Add bdrv_find_overlay(), and bdrv_drop_intermediate(). > > bdrv_find_overlay(): given 'bs' and the active (topmost) BDS of an image > chain, > find the image that is the immediate top of 'bs' > > bdrv_drop_intermediate(): >

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Jason Baron
On Sat, Sep 15, 2012 at 12:14:01AM +0900, Isaku Yamahata wrote: > On Thu, Sep 13, 2012 at 04:12:34PM -0400, Jason Baron wrote: > > 2) hotplug > > I've added piix acpi style hotplug to ich9. > > What's the point of this? > Its design is ad-hoc and shpc/pcie hotplug are available. The point was to

Re: [Qemu-devel] [PATCH 11/25] q35: Introduce q35 pc based chipset emulator

2012-09-14 Thread Jason Baron
On Fri, Sep 14, 2012 at 03:26:30PM +0300, Michael S. Tsirkin wrote: > I think it's best to smash the following patch into this one > otherwise review becomes painful. Some short notes: > ok. > > --- > > hw/acpi_ich9.c| 315 ++ > > hw/acpi_ich9.h| 53 +++ > > h

Re: [Qemu-devel] [PULL 00/14] usb patch queue

2012-09-14 Thread Hans de Goede
Hi, On 09/14/2012 10:27 AM, Gerd Hoffmann wrote: Are any of the bugfixes worth to apply to previous or new -stable series? I know usbredir configure fix is needed, anything else? Gerd Hoffmann (2): usb-host: allow emulated (non-async) control requests without USBPacket This.

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Isaku Yamahata
On Thu, Sep 13, 2012 at 04:12:34PM -0400, Jason Baron wrote: > 2) hotplug > I've added piix acpi style hotplug to ich9. What's the point of this? Its design is ad-hoc and shpc/pcie hotplug are available. -- yamahata

Re: [Qemu-devel] apparently missing yet another notify_event()

2012-09-14 Thread Michael Tokarev
On 14.09.2012 18:47, Michael Tokarev wrote: [] >> qemu-kvm -nographic -kernel /boot/vmlinuz-$(uname -r) -append console=ttyS0 >> -serial pty >> >> This hangs till I send a char to the pty. > > And it is even _more_ twisted than that. > > It depends on the timing. If I connect to the pty "too s

Re: [Qemu-devel] apparently missing yet another notify_event()

2012-09-14 Thread Michael Tokarev
On 14.09.2012 18:17, Michael Tokarev wrote: > On 04.09.2012 10:58, Michael Tokarev wrote: >> On 04.09.2012 10:53, Paolo Bonzini wrote: >>> Il 03/09/2012 20:13, Michael Tokarev ha scritto: >> [] qemu -nographic -kernel /boot/vmlinuz-$(uname -r) -append console=ttyS0 -serial pty

Re: [Qemu-devel] [libvirt] [PATCH v4 0/5] Per-guest configurable user/group for QEMU processes

2012-09-14 Thread Corey Bryant
On 09/14/2012 09:51 AM, Daniel P. Berrange wrote: On Fri, Sep 14, 2012 at 09:31:26AM -0400, Corey Bryant wrote: On 09/14/2012 04:40 AM, Daniel P. Berrange wrote: On Tue, Sep 11, 2012 at 02:13:38PM -0400, Corey Bryant wrote: Are there any other requirements that need to be taken care of to

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Stefan Weil
Am 14.09.2012 15:17, schrieb Paolo Bonzini: Hi all, here is a proposal for moving around 150 C files currently in the toplevel directory to separate, well-delimited subdirectories. Header files would be moved for now in include/, preparing for subsequent reorganization of headers. Usually the

Re: [Qemu-devel] [PATCH 18/25] q35: Fix irr initialization for slots 25..31

2012-09-14 Thread Jason Baron
On Fri, Sep 14, 2012 at 09:05:10AM +0200, Paolo Bonzini wrote: > Il 13/09/2012 22:12, Jason Baron ha scritto: > > From: Isaku Yamahata > > > > This was totally off: The CC registers are 16 bit (stored as little > > endian), their offsets run in reverse order, and D26IR as well as D25IR > > have 4

Re: [Qemu-devel] [PATCH 16/25] pci: Add class 0xc05 as 'SMBus'

2012-09-14 Thread Jason Baron
On Fri, Sep 14, 2012 at 09:04:06AM +0200, Paolo Bonzini wrote: > Il 13/09/2012 22:12, Jason Baron ha scritto: > > From: Jan Kiszka > > > > Signed-off-by: Jan Kiszka > > Signed-off-by: Jason Baron > > --- > > hw/pci.c |1 + > > 1 files changed, 1 insertions(+), 0 deletions(-) > > > > diff

Re: [Qemu-devel] apparently missing yet another notify_event()

2012-09-14 Thread Michael Tokarev
On 04.09.2012 10:58, Michael Tokarev wrote: > On 04.09.2012 10:53, Paolo Bonzini wrote: >> Il 03/09/2012 20:13, Michael Tokarev ha scritto: > [] >>> qemu -nographic -kernel /boot/vmlinuz-$(uname -r) -append console=ttyS0 >>> -serial pty >>> >>> This will hang with 100% CPU usage until something i

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Alexander Graf
On 14.09.2012, at 16:08, Jason Baron wrote: > On Fri, Sep 14, 2012 at 03:56:17PM +0200, Alexander Graf wrote: >> On 14.09.2012, at 15:50, Jason Baron wrote: >> >>> On Fri, Sep 14, 2012 at 12:29:17AM +0200, Alexander Graf wrote: On 13.09.2012, at 22:12, Jason Baron wrote: > Hi, >>>

Re: [Qemu-devel] [PATCH 11/25] q35: Introduce q35 pc based chipset emulator

2012-09-14 Thread Jason Baron
On Fri, Sep 14, 2012 at 09:37:03AM +0200, Gerd Hoffmann wrote: > On 09/13/12 22:12, Jason Baron wrote: > > +if (ram_size >= 0xe000) { > > +above_4g_mem_size = ram_size - 0xe000; > > +below_4g_mem_size = 0xe000; > > We should make the hole larger to get more pci i/o

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Jason Baron
On Fri, Sep 14, 2012 at 03:56:17PM +0200, Alexander Graf wrote: > On 14.09.2012, at 15:50, Jason Baron wrote: > > > On Fri, Sep 14, 2012 at 12:29:17AM +0200, Alexander Graf wrote: > >> On 13.09.2012, at 22:12, Jason Baron wrote: > >> > >>> Hi, > >>> > >>> Qemu bits for q35 support, I'm posting t

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 15:44, Luiz Capitulino ha scritto: >> > qapi: >> > error.c > error.c isn't tied to a subsystem, it's really "qemu wide". If it > must reside in dir (ie. it can't be in the top dir), then I'd > suggest to put it in qmp. It's either qapi/ or util/. I'd rather avoid util/ as a dumping

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Peter Maydell
On 14 September 2012 14:55, Paolo Bonzini wrote: > Il 14/09/2012 15:48, Peter Maydell ha scritto: >> ...oh, and it would be good to be sure we've fixed all the problems >> with dependencies that tend to cause 'make' to fail to DTRT going >> backwards and forwards across a file-move commit, before

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 15:47, Daniel P. Berrange ha scritto: > On Fri, Sep 14, 2012 at 03:17:52PM +0200, Paolo Bonzini wrote: >> Hi all, >> >> here is a proposal for moving around 150 C files currently in the >> toplevel directory to separate, well-delimited subdirectories. Header >> files would be moved f

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Alexander Graf
On 14.09.2012, at 15:50, Jason Baron wrote: > On Fri, Sep 14, 2012 at 12:29:17AM +0200, Alexander Graf wrote: >> On 13.09.2012, at 22:12, Jason Baron wrote: >> >>> Hi, >>> >>> Qemu bits for q35 support, I'm posting the seabios changes separately. The >>> patches require '-M pc_q35' and -L 'seab

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 15:48, Peter Maydell ha scritto: >> > here is a proposal for moving around 150 C files currently in the >> > toplevel directory to separate, well-delimited subdirectories. > ...oh, and it would be good to be sure we've fixed all the problems > with dependencies that tend to cause 'mak

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Paolo Bonzini
Il 14/09/2012 15:36, Peter Maydell ha scritto: > On 14 September 2012 14:17, Paolo Bonzini wrote: >> here is a proposal for moving around 150 C files currently in the >> toplevel directory to separate, well-delimited subdirectories. > > No general objection (though some specific comments below).

Re: [Qemu-devel] [libvirt] [PATCH v4 0/5] Per-guest configurable user/group for QEMU processes

2012-09-14 Thread Daniel P. Berrange
On Fri, Sep 14, 2012 at 09:31:26AM -0400, Corey Bryant wrote: > > > On 09/14/2012 04:40 AM, Daniel P. Berrange wrote: > >On Tue, Sep 11, 2012 at 02:13:38PM -0400, Corey Bryant wrote: > >>Are there any other requirements that need to be taken care of to > >>enable execution of QEMU guests under se

Re: [Qemu-devel] [PATCH 00/25] q35 series take #1

2012-09-14 Thread Jason Baron
On Fri, Sep 14, 2012 at 12:29:17AM +0200, Alexander Graf wrote: > On 13.09.2012, at 22:12, Jason Baron wrote: > > > Hi, > > > > Qemu bits for q35 support, I'm posting the seabios changes separately. The > > patches require '-M pc_q35' and -L 'seabios dir with q35 changes' on the > > qemu command

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Peter Maydell
On 14 September 2012 14:17, Paolo Bonzini wrote: > here is a proposal for moving around 150 C files currently in the > toplevel directory to separate, well-delimited subdirectories. ...oh, and it would be good to be sure we've fixed all the problems with dependencies that tend to cause 'make' to

[Qemu-devel] [PATCH 4/8] qerror: new error for live block commit, QERR_TOP_NOT_FOUND

2012-09-14 Thread Jeff Cody
Signed-off-by: Jeff Cody --- qerror.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/qerror.h b/qerror.h index d0a76a4..7396184 100644 --- a/qerror.h +++ b/qerror.h @@ -219,6 +219,9 @@ void assert_no_error(Error *err); #define QERR_TOO_MANY_FILES \ ERROR_CLASS_GENERIC_ERROR, "Too m

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Daniel P. Berrange
On Fri, Sep 14, 2012 at 03:17:52PM +0200, Paolo Bonzini wrote: > Hi all, > > here is a proposal for moving around 150 C files currently in the > toplevel directory to separate, well-delimited subdirectories. Header > files would be moved for now in include/, preparing for subsequent > reorganizat

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Luiz Capitulino
On Fri, 14 Sep 2012 15:17:52 +0200 Paolo Bonzini wrote: > Hi all, > > here is a proposal for moving around 150 C files currently in the > toplevel directory to separate, well-delimited subdirectories. Header > files would be moved for now in include/, preparing for subsequent > reorganization o

[Qemu-devel] [PATCH 7/8] qemu-iotests: add initial tests for live block commit

2012-09-14 Thread Jeff Cody
Derived from the streaming test cases (030), this adds the following tests: 1. For the following image chain, commit [mid] into [backing], and use qemu-io to verify [backing] has its original data, as well as the data from [mid] [backing] <-- [mid] <-- [test] 2. Verifies that 'b

[Qemu-devel] [PATCH 01/11] sheepdog: fix savevm and loadvm

2012-09-14 Thread Kevin Wolf
From: MORITA Kazutaka This patch sets data to be sent to Sheepdog correctly and fixes savevm and loadvm operations on a Sheepdog image. Signed-off-by: MORITA Kazutaka Signed-off-by: Kevin Wolf --- block/sheepdog.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/bloc

[Qemu-devel] [PATCH 5/8] block: helper function, to find the base image of a chain

2012-09-14 Thread Jeff Cody
This is a simple helper function, that will return the base image of a given image chain. Signed-off-by: Jeff Cody --- block.c | 16 block.h | 1 + 2 files changed, 17 insertions(+) diff --git a/block.c b/block.c index 9549eca..f19c1dd 100644 --- a/block.c +++ b/block.c @@ -31

[Qemu-devel] [PATCH 1/8] block: add support functions for live commit, to find and delete images.

2012-09-14 Thread Jeff Cody
Add bdrv_find_overlay(), and bdrv_drop_intermediate(). bdrv_find_overlay(): given 'bs' and the active (topmost) BDS of an image chain, find the image that is the immediate top of 'bs' bdrv_drop_intermediate(): Given 3 BDS (active, top, base), delete images

[Qemu-devel] [PATCH 8/8] block: after creating a live snapshot, make old image read-only

2012-09-14 Thread Jeff Cody
Currently, after a live snapshot of a drive, the image that has been 'demoted' to be below the new active layer remains r/w. This patch reopens it read-only. Note that we do not check for error on the reopen(), because we will not abort the snapshots if the reopen fails. This patch depends on the

[Qemu-devel] [PATCH 6/8] QAPI: add command for live block commit, 'block-commit'

2012-09-14 Thread Jeff Cody
The command for live block commit is added, which has the following arguments: device: the block device to perform the commit on (mandatory) base: the base image to commit into; optional (if not specified, it is the underlying original image) top:the top image of the commit - all dat

[Qemu-devel] [PATCH 2/8] block: add live block commit functionality

2012-09-14 Thread Jeff Cody
This adds the live commit coroutine. This iteration focuses on the commit only below the active layer, and not the active layer itself. The behaviour is similar to block streaming; the sectors are walked through, and anything that exists above 'base' is committed back down into base. At the end,

[Qemu-devel] [PATCH 3/8] blockdev: rename block_stream_cb to a generic block_job_cb

2012-09-14 Thread Jeff Cody
Signed-off-by: Jeff Cody --- blockdev.c | 6 +++--- trace-events | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/blockdev.c b/blockdev.c index 7c83baa..9caa16f 100644 --- a/blockdev.c +++ b/blockdev.c @@ -1063,12 +1063,12 @@ static QObject *qobject_from_block_job(BlockJo

[Qemu-devel] [PATCH 0/8] Live block commit

2012-09-14 Thread Jeff Cody
This series adds the basic case, of a live commit between two images below the active layer, e.g.: [base] <--- [snp-1] <--- [snp-2] <--- [snp-3] <--- [active] can be collapsed down via commit, into: [base] <--- [active] or, [base] <--- [snp-1] <--- [active], [base] <--- [snp-3] <--- [active],

Re: [Qemu-devel] directory hierarchy

2012-09-14 Thread Peter Maydell
On 14 September 2012 14:17, Paolo Bonzini wrote: > here is a proposal for moving around 150 C files currently in the > toplevel directory to separate, well-delimited subdirectories. No general objection (though some specific comments below). However I think it would be helpful if you could provid

[Qemu-devel] [PATCH 09/11] qemu-img: Add json output option to the info command.

2012-09-14 Thread Kevin Wolf
From: Benoît Canet This option --output=[human|json] make qemu-img info output on human or JSON representation at the choice of the user. example: { "snapshots": [ { "vm-clock-nsec": 637102488, "name": "vm-20120821145509", "date-sec": 1345553709,

Re: [Qemu-devel] [libvirt] [PATCH v4 0/5] Per-guest configurable user/group for QEMU processes

2012-09-14 Thread Corey Bryant
On 09/14/2012 04:40 AM, Daniel P. Berrange wrote: On Tue, Sep 11, 2012 at 02:13:38PM -0400, Corey Bryant wrote: Are there any other requirements that need to be taken care of to enable execution of QEMU guests under separate unprivileged user IDs (ie. DAC isolation)? At this point, this patch

Re: [Qemu-devel] [PATCH] configure: fix "--target-list=, , ..." option

2012-09-14 Thread Daniel P. Berrange
On Fri, Sep 14, 2012 at 01:53:11PM +0100, Peter Maydell wrote: > On 12 September 2012 14:20, Laurent Desnogues > wrote: > > Sorry, I had missed this patch... > > > > On Tue, Sep 11, 2012 at 9:02 PM, Eduardo Habkost > > wrote: > >> commit 66d5499b3754b83c09487259c08fe2ce73188a59 broke the support

[Qemu-devel] [PATCH 10/11] Don't require encryption password for 'qemu-img info' command

2012-09-14 Thread Kevin Wolf
From: "Daniel P. Berrange" The encryption password is only required if I/O is going to be performed on a disk image. The 'qemu-img info' command merely reports metadata, so it should not ask for a decryption password Signed-off-by: Daniel P. Berrange Signed-off-by: Kevin Wolf --- qemu-img.c |

Re: [Qemu-devel] [PATCH] Revert "Add ability to disable build of all targets"

2012-09-14 Thread Peter Maydell
On 14 September 2012 14:20, Anthony Liguori wrote: > This reverts commit 66d5499b3754b83c09487259c08fe2ce73188a59. > > This commit broke --target-list="x86_64-softmmu" Actually that still works fine. What doesn't work is passing several comma-specified target names to --target-list. > and the fi

  1   2   3   >