[Qemu-devel] [Bug 1224444] Re: virtio-serial loses writes when used over virtio-mmio

2013-09-13 Thread Richard Jones
I can reproduce this bug on a second ARM machine which doesn't have KVM (ie. using TCG). Note it's still linked to virtio-mmio. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/122 Title: virtio-

Re: [Qemu-devel] [PATCH] sh4: Fix serial line access for Linux kernels later than 3.2

2013-09-13 Thread Guenter Roeck
On 09/08/2013 12:39 AM, Guenter Roeck wrote: With Linux kernel version 3.3 or later, qemu fails with the following message: sh_serial: unsupported read from 0x18 Aborted Reported-and-analyzed-by: Rob Landley Signed-off-by: Guenter Roeck ping ... --- See http://lists.nongnu.org/archive/

Re: [Qemu-devel] [PATCH 4/5] hw: arm_gic: Support setting/getting binary point reg

2013-09-13 Thread Christoffer Dall
On Fri, Sep 06, 2013 at 03:41:04PM +0100, Peter Maydell wrote: > On 23 August 2013 21:10, Christoffer Dall wrote: > > Add a binary_point field to the gic emulation structure and support > > setting/getting this register now when we have it. We don't actually > > support interrupt grouping yet, oh

Re: [Qemu-devel] [PATCH 0/4] lsi: small cleanup and add 53C810 variant

2013-09-13 Thread Peter Maydell
On 13 September 2013 20:50, Hervé Poussineau wrote: > However, there is some precedence in QEMU codebase, where for example > Intel 486 CPU can execute cmov instructions (like cmov) Poor choice of example, since that's just a bug and the fix is hopefully going to get committed in the not too dist

Re: [Qemu-devel] Tracking down a bug in CIL + tcg + kernel >= 3.2

2013-09-13 Thread Gabriel Kerneis
On Fri, Sep 13, 2013 at 08:38:58AM +0100, Gabriel Kerneis wrote: > But I'm nonetheless hitting a bug in CIL simplifications and I have a hard > time tracking it down. Fixed! https://github.com/kerneis/cil/commit/640994197bed274c373d585decbe9ebe2073f014 Thanks to everybody who helped, -- Gabriel

[Qemu-devel] [Bug 1225187] [NEW] qemu hangs in windows 7 host with -serial pipe:windbg

2013-09-13 Thread alex
Public bug reported: Execution line: qemu-system-i386.exe -m 512 c:\Disks\Qemu_XP_en.vhd -serial pipe:windbg It waits for the pipe. Execute windbg c:\WINDDK\7600.16385.1\Debuggers\windbg.exe -k com:pipe,port=\\.\pipe\windbg,resets=0,reconnect GUI black screen shown. QEMU hangs. qemu v1.5.3 (c

[Qemu-devel] [Bug 1180777] Re: Windows 7 VM freeze on Ubuntu 12.04 KVM

2013-09-13 Thread Vasile Dumitrescu
I also see these EXACT symptoms, using kvm (VM managed through livirt virsh) on Debian x64 host, guest is Windows 8, RedHat VirtIo network driver. rgds ** Also affects: debian Importance: Undecided Status: New -- You received this bug notification because you are a member of qemu- dev

Re: [Qemu-devel] [PATCH v2 0/4] avoid CONFIG_NO_* using rules.mak logic/string fns

2013-09-13 Thread Paolo Bonzini
Il 13/09/2013 19:25, Peter Maydell ha scritto: > This is based on a set of patches by Ákos Kovács which were > lurking at the beginning of his KConfig patchset but which > I think make a nice standalone cleanup. Since I basically > rewrote all of patch 1&2 in review comments I figured it was > easi

Re: [Qemu-devel] [PATCH v9 5/8] module: implement module loading

2013-09-13 Thread Richard Henderson
On 09/13/2013 09:20 AM, Michael Tokarev wrote: > 13.09.2013 18:52, Richard Henderson wrote: > >> Also, separate the whitelists by type. I.e. > > What for? We are talking about a function whose interface is to only load modules of a particular type, right? Given that we know the set of all modu

Re: [Qemu-devel] [PATCH 08/12] qemu-img: conditionally zero out target on convert

2013-09-13 Thread Eric Blake
On 09/13/2013 01:48 PM, Peter Lieven wrote: +flags = BDRV_O_RDWR | BDRV_O_UNMAP; >>> I think this should be a new command-line flag. >> I agree - while 'sparse by default' may be reasonable, it is also >> feasible to want a mode that guarantees expansion rather than unmapped >> or sparse.

[Qemu-devel] [PATCH 3/4] lsi: ignore write accesses to CTEST0 registers

2013-09-13 Thread Hervé Poussineau
53C895A datasheet says that this register is read/write, and that the value returned on read access is dependant of DMA FIFO state. However, nothing is said for written value. 53C810A datasheet gives more insight about this register: "This was a general purpose read/write register in previous SYM5

[Qemu-devel] [PATCH 2/4] lsi: check ssid versus sdid only if ssid is valid

2013-09-13 Thread Hervé Poussineau
This prevents some (invalid) error messages on console. Signed-off-by: Hervé Poussineau --- hw/scsi/lsi53c895a.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/scsi/lsi53c895a.c b/hw/scsi/lsi53c895a.c index ca01e86..764feaa 100644 --- a/hw/scsi/lsi53c895a.c +++ b/hw/

[Qemu-devel] [PATCH 4/4] lsi: add 53C810 variant

2013-09-13 Thread Hervé Poussineau
Currently, treat it exactly as a 53C895A. 53C895A is a 53C810 with more capabilities, so this should work. However, this lets us test different code paths on Linux, which don't use lastest features if it detect a 810, or on some OSes which only support 810 and not 895A (like very old Windows NT ve

[Qemu-devel] [PATCH 1/4] lsi: use constant name instead of its value

2013-09-13 Thread Hervé Poussineau
Signed-off-by: Hervé Poussineau --- hw/scsi/lsi53c895a.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/scsi/lsi53c895a.c b/hw/scsi/lsi53c895a.c index 0c36842..ca01e86 100644 --- a/hw/scsi/lsi53c895a.c +++ b/hw/scsi/lsi53c895a.c @@ -1521,7 +1521,7 @@ static uint8_t

[Qemu-devel] [PATCH 0/4] lsi: small cleanup and add 53C810 variant

2013-09-13 Thread Hervé Poussineau
Hi, This small patchset fixes a few issues I encountered while trying to add support for the IBM RS/6000 40p. Patches 1 to 3 are small cleanups. Patch 4 may be more controversial, as it adds support for an older (albeit compatible) SCSI adapter, without removing emulation of newer features. Howev

Re: [Qemu-devel] [PATCH 08/12] qemu-img: conditionally zero out target on convert

2013-09-13 Thread Peter Lieven
Am 13.09.2013 20:25, schrieb Eric Blake: > On 09/13/2013 04:36 AM, Paolo Bonzini wrote: >> Il 13/09/2013 12:25, Peter Lieven ha scritto: >>> if the target has_zero_init = 0, but supports efficiently >>> writing zeroes by unmapping we call bdrv_zeroize to >>> avoid fully allocating the target. this

Re: [Qemu-devel] [PATCH 3/5] hw: arm_gic: Keep track of SGI sources

2013-09-13 Thread Christoffer Dall
On Fri, Sep 06, 2013 at 03:08:16PM +0100, Peter Maydell wrote: > On 23 August 2013 21:10, Christoffer Dall wrote: > > Right now the arm gic emulation doesn't keep track of the source of an > > SGI (which apparently Linux guests don't use, or they're fine with > > assuming CPU 0 always). > > > > Ad

Re: [Qemu-devel] [PATCH 5/6] QMP: Update qmp-spec.txt

2013-09-13 Thread Eric Blake
On 09/13/2013 11:54 AM, Luiz Capitulino wrote: > On Wed, 11 Sep 2013 17:27:58 -0600 > Eric Blake wrote: > >> On 09/11/2013 02:52 PM, Luiz Capitulino wrote: >>> Simplify the text, fix some of the examples. >>> >>> - QEMU Monitor Protocol Specification - Version 0.1 >>> +

[Qemu-devel] [PATCH 4/6] QMP: Update README file

2013-09-13 Thread Luiz Capitulino
Drop unneeded info, fix some of the examples and rename QEMU Monitor Protocol to QEMU Machine Protocol. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- docs/qmp/README | 99 - 1 file changed, 49 insertions(+), 50 deletions(-) d

Re: [Qemu-devel] [PATCH 06/12] iscsi: add .bdrv_get_info

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block/iscsi.c | 24 > 1 file changed, 24 insertions(+) > > + > +bdi->discard_zeroes = iscsilun->lbprz; > +bdi->discard_write_zeroes = (bs->open_flags & BDRV_O_UNMAP) && Why tw

Re: [Qemu-devel] [PATCH 03/12] block: introduce BDRV_REQ_MAY_UNMAP in bdrv_co_write_zeroes

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > the BDRV_REQ_MAY_UNMAP flag is used to indicate that block driver > is allowed to optimze a write zeroes request by unmapping (discarding) s/optimze/optimize/ > blocks if it is guaranteed that the result will read back as > zeroes. > > the flag is on

Re: [Qemu-devel] [PATCH 11/12] block/get_block_status: avoid redundant callouts on raw devices

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > if a raw device like an iscsi target or host device is used > the current implementation makes a second call out to get > the block status of bs->file. however, the raw driver already > has called bdrv_get_block_status on bs->file. > > Signed-off-by: P

Re: [Qemu-devel] [PATCH 6/6] QMP: qmp-events.txt: minor fixes

2013-09-13 Thread Luiz Capitulino
On Wed, 11 Sep 2013 17:30:48 -0600 Eric Blake wrote: > On 09/11/2013 02:52 PM, Luiz Capitulino wrote: > > Signed-off-by: Luiz Capitulino > > --- > > docs/qmp/qmp-events.txt | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/docs/qmp/qmp-events.txt b/docs/qmp/qmp

Re: [Qemu-devel] [PULL 15/33] snapshot: distinguish id and name in snapshot delete

2013-09-13 Thread Eric Blake
On 09/13/2013 05:50 AM, Kevin Wolf wrote: > From: Wenchao Xia > > Snapshot creation actually already distinguish id and name since it take > a structured parameter *sn, but delete can't. Later an accurate delete > is needed in qmp_transaction abort and blockdev-snapshot-delete-sync, > so change i

Re: [Qemu-devel] [PATCH 09/12] block/get_block_status: set *pnum = 0 on error

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > if the call is invoked through bdrv_is_allocated the caller might > expect *pnum = 0 on error. however, a new implementation of > bdrv_get_block_status might only return a negative exit value on > error while keeping *pnum untouched. > > Signed-off-by:

[Qemu-devel] [PATCH 5/6] QMP: Update qmp-spec.txt

2013-09-13 Thread Luiz Capitulino
Simplify the text, fix some of the examples. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- docs/qmp/qmp-spec.txt | 65 ++- 1 file changed, 28 insertions(+), 37 deletions(-) diff --git a/docs/qmp/qmp-spec.txt b/docs/qmp/qmp-spec.txt i

[Qemu-devel] [PATCH 1/6] QMP: add scripts/qmp

2013-09-13 Thread Luiz Capitulino
Populate it with all scripts stored in QMP/. Also fixes trailing whitespaces in qmp-shell and qmp.py. Signed-off-by: Luiz Capitulino --- {QMP => scripts/qmp}/qemu-ga-client | 0 {QMP => scripts/qmp}/qmp| 0 {QMP => scripts/qmp}/qmp-shell | 2 +- {QMP => scripts/qmp}/qmp.py

Re: [Qemu-devel] [PATCH 6/6] QMP: qmp-events.txt: minor fixes

2013-09-13 Thread Eric Blake
On 09/13/2013 12:01 PM, Luiz Capitulino wrote: > On Wed, 11 Sep 2013 17:30:48 -0600 > Eric Blake wrote: > > >> and it's probably worth splitting SPICE_CONNECTED and >> SPICE_DISCONNECTED into separate sections (matching VNC_CONNECTED and >> VNC_DISCONNECTED). > > This probably better done in a

Re: [Qemu-devel] [PATCH v2 0/6] QMP: re-organize documentation

2013-09-13 Thread Eric Blake
On 09/13/2013 12:11 PM, Luiz Capitulino wrote: > The QMP dir is storing QMP docs and scripts. This series moves the scripts to > scripts/qmp/ and the docs to docs/qmp/. Also, the docs are updated. > > v2 > > - Set git diff.renames to true > - Fix trailing whitespaces > - Other minor fixes > >

[Qemu-devel] [PATCH 6/6] QMP: qmp-events.txt: alphabetical order fix and other minor changes

2013-09-13 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino --- docs/qmp/qmp-events.txt | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/docs/qmp/qmp-events.txt b/docs/qmp/qmp-events.txt index 4b24ec9..6b87e97 100644 --- a/docs/qmp/qmp-events.txt +++ b/docs/qmp/qmp-even

Re: [Qemu-devel] [PATCH 07/12] block: introduce bdrv_zeroize

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > this patch adds a call to completely zero out a block device. > the operation is speed up by checking the block status and s/speed/sped/ > only writing zeroes to the device if they currently do not > return zeroes. optionally the zero writing can be s

Re: [Qemu-devel] [RFC PATCH] spapr: support time base offset migration

2013-09-13 Thread Alexander Graf
On 13.09.2013, at 00:20, David Gibson wrote: > On Mon, Sep 09, 2013 at 08:06:53AM +0200, Alexander Graf wrote: >> >> >> Am 09.09.2013 um 07:58 schrieb Alexey Kardashevskiy : >> >>> On 09/09/2013 03:50 PM, Alexander Graf wrote: Am 09.09.2013 um 04:40 schrieb Alexey Kardashevski

[Qemu-devel] [PATCH 2/6] QMP: fix qmp-commands.txt generation path

2013-09-13 Thread Luiz Capitulino
This file should be generated in the BUILD_DIR, as all other docs. Signed-off-by: Luiz Capitulino Reviewed-by: Eric Blake --- Makefile | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index 362fe3e..aca548d 100644 --- a/Makefile +++ b/Makefile @@ -65

Re: [Qemu-devel] [PATCH 10/12] block/get_block_status: avoid segfault if there is no backing_hd

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Eric Blake > > diff --git a/block.c b/block.c > index 4922fb9..cf9db62 100644 > --- a/block.c > +++ b/block.c > @@ -3156,7 +3156,7

[Qemu-devel] [PATCH 3/6] QMP: QMP/ -> docs/qmp/

2013-09-13 Thread Luiz Capitulino
Signed-off-by: Luiz Capitulino --- {QMP => docs/qmp}/README | 0 {QMP => docs/qmp}/qmp-events.txt | 0 {QMP => docs/qmp}/qmp-spec.txt | 0 3 files changed, 0 insertions(+), 0 deletions(-) rename {QMP => docs/qmp}/README (100%) rename {QMP => docs/qmp}/qmp-events.txt (100%) rename {QM

Re: [Qemu-devel] [PATCH 04/12] iscsi: add .bdrv_co_write_zeroes

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block/iscsi.c | 78 > + > 1 file changed, 78 insertions(+) > > +if (iscsilun->zeroblock == NULL) { > +iscsilun->zeroblock = g_malloc(iscsilun-

Re: [Qemu-devel] [PATCH 6/6] QMP: qmp-events.txt: minor fixes

2013-09-13 Thread Luiz Capitulino
On Fri, 13 Sep 2013 12:16:04 -0600 Eric Blake wrote: > On 09/13/2013 12:01 PM, Luiz Capitulino wrote: > > On Wed, 11 Sep 2013 17:30:48 -0600 > > Eric Blake wrote: > > > > > >> and it's probably worth splitting SPICE_CONNECTED and > >> SPICE_DISCONNECTED into separate sections (matching VNC_CON

[Qemu-devel] [PATCH v2 0/6] QMP: re-organize documentation

2013-09-13 Thread Luiz Capitulino
The QMP dir is storing QMP docs and scripts. This series moves the scripts to scripts/qmp/ and the docs to docs/qmp/. Also, the docs are updated. v2 - Set git diff.renames to true - Fix trailing whitespaces - Other minor fixes Luiz Capitulino (6): QMP: add scripts/qmp QMP: fix qmp-command

Re: [Qemu-devel] [PATCH 08/12] qemu-img: conditionally zero out target on convert

2013-09-13 Thread Eric Blake
On 09/13/2013 04:36 AM, Paolo Bonzini wrote: > Il 13/09/2013 12:25, Peter Lieven ha scritto: >> if the target has_zero_init = 0, but supports efficiently >> writing zeroes by unmapping we call bdrv_zeroize to >> avoid fully allocating the target. this currently >> is designed especially for iscsi.

Re: [Qemu-devel] [PULL 12/33] qemu-iotests: add tests for runtime fd passing via SCM rights

2013-09-13 Thread Eric Blake
On 09/13/2013 05:50 AM, Kevin Wolf wrote: > From: Wenchao Xia > > This case will test whether the monitor can receive fd at runtime. > To verify better, additional monitor is created to see if qemu > can handler two monitor instances correctly. > > Signed-off-by: Wenchao Xia > Reviewed-by: Stef

Re: [Qemu-devel] [PATCH 5/6] QMP: Update qmp-spec.txt

2013-09-13 Thread Luiz Capitulino
On Wed, 11 Sep 2013 17:27:58 -0600 Eric Blake wrote: > On 09/11/2013 02:52 PM, Luiz Capitulino wrote: > > Simplify the text, fix some of the examples. > > > > Signed-off-by: Luiz Capitulino > > --- > > docs/qmp/qmp-spec.txt | 65 > > ++- > > 1 f

Re: [Qemu-devel] [PATCH 1/6] QMP: add scripts/qmp

2013-09-13 Thread Luiz Capitulino
On Wed, 11 Sep 2013 16:13:32 -0600 Eric Blake wrote: > On 09/11/2013 04:08 PM, Eric Blake wrote: > > On 09/11/2013 02:52 PM, Luiz Capitulino wrote: > >> Populate it with all scripts stored in QMP/. > >> > >> Signed-off-by: Luiz Capitulino > > > Reviewed-by: Eric Blake > > > > Repeat after me:

[Qemu-devel] [PATCH v2 1/4] rules.mak: New logical functions for handling y/n values

2013-09-13 Thread Peter Maydell
Add new logical functions for handling y/n values like those we use in CONFIG_FOO variables: lnot : logical NOT land : logical AND lor : logical OR lxor : logical XOR leqv : logical equality, inverse of lxor lif : like Make's $(if) but with an eq-like test Based on an idea by Ákos Kovács .

[Qemu-devel] [PATCH v2 4/4] default-configs/: CONFIG_GDBSTUB_XML removed

2013-09-13 Thread Peter Maydell
From: Ákos Kovács Makefile.target: Build gdbstub-xml.o only when TARGET_XML_FILES is not empty. Signed-off-by: Ákos Kovács Reviewed-by: Peter Maydell --- Makefile.target |2 +- default-configs/arm-linux-user.mak|2 -- default-configs/arm-softmmu.mak

[Qemu-devel] [PATCH v2 0/4] avoid CONFIG_NO_* using rules.mak logic/string fns

2013-09-13 Thread Peter Maydell
This is based on a set of patches by Ákos Kovács which were lurking at the beginning of his KConfig patchset but which I think make a nice standalone cleanup. Since I basically rewrote all of patch 1&2 in review comments I figured it was easier to just write it as a new patch. Patch 3 and 4 are Áko

[Qemu-devel] [PATCH v2 2/4] rules.mak: New string testing functions

2013-09-13 Thread Peter Maydell
Add new string testing functions which return a y/n result: eq : are two strings equal (ignoring leading/trailing space)? ne : are two strings unequal? isempty : is a string empty? notempty : is a string non-empty? Based on an idea by Ákos Kovács . Signed-off-by: Peter Maydell --- rules.mak

Re: [Qemu-devel] [PATCH] integrator: fix Linux boot failure by emulating dbg

2013-09-13 Thread Peter Maydell
On 13 September 2013 16:20, Peter Maydell wrote: > On 13 September 2013 15:58, wrote: >> + * This code is licensed under the GPL. > > Would be nice to give a specific version of the GPL... > ("2" or "2 or later" are your two basic options.) ...but I forgot to mention that "2-or-later" is strong

Re: [Qemu-devel] [PATCH 02/12] block: add flags to bdrv_*_write_zeroes

2013-09-13 Thread Eric Blake
On 09/13/2013 04:24 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block-migration.c |2 +- > block.c | 20 +++- > block/backup.c|3 ++- > block/qcow2.c |2 +- > block/qed.c |3 ++- >

[Qemu-devel] [PATCH v2 3/4] Makefile.target: CONFIG_NO_* variables removed

2013-09-13 Thread Peter Maydell
From: Ákos Kovács CONFIG_NO_* variables replaced with the lnot logical function Signed-off-by: Ákos Kovács [PMM: fixed a few CONFIG_NO_* uses that were missed] Signed-off-by: Peter Maydell --- Makefile.target |8 ++-- hw/pci/Makefile.objs |2 +- target-arm/Makefile.

[Qemu-devel] [PATCH] linux-user: correct how SOL_SOCKET is converted from target to host and back

2013-09-13 Thread Petar Jovanovic
From: Petar Jovanovic Previous implementation does not take into account that SOL_SOCKET constant can be arch specific. This change fixes some issues with sendmsg/recvmsg. Signed-off-by: Petar Jovanovic --- linux-user/syscall.c | 18 +- 1 file changed, 13 insertions(+), 5 del

Re: [Qemu-devel] [PATCH] linux-user: correct how SOL_SOCKET is converted from target to host and back

2013-09-13 Thread Peter Maydell
On 13 September 2013 18:27, Petar Jovanovic wrote: > From: Petar Jovanovic > > Previous implementation does not take into account that SOL_SOCKET constant > can be arch specific. This change fixes some issues with sendmsg/recvmsg. > > Signed-off-by: Petar Jovanovic Reviewed-by: Peter Maydell

Re: [Qemu-devel] [PATCH v2] target-mips: fix 34Kf configuration for DSP ASE

2013-09-13 Thread Maciej W. Rozycki
On Sat, 3 Aug 2013, Aurelien Jarno wrote: > > 34Kf core does support DSP ASE. > > CP0_Config3 configuration for 34Kf and description are wrong. > > > > Please refer to MIPS32(R) 34Kf(TM) Processor Core Datasheet > > > > Signed-off-by: Yongbok Kim > > --- > > changes from v1: > > make status.MX

Re: [Qemu-devel] [PATCH] target-mips: fix decoding of microMIPS POOL32Axf instructions

2013-09-13 Thread Maciej W. Rozycki
On Mon, 5 Aug 2013, Leon Alrae wrote: > Just to make sure that we are refering to the same thing :) > For MFHI: > page no. 303 in MIPS Architecture for Programmers Volume II-B: The > microMIPS32 Instruction Set (document MD00764) > page no. 140 in MIPS Architecture for Programmers Volume IV-e: The

[Qemu-devel] qemu aarch64 kvm control: branch for testing

2013-09-13 Thread Peter Maydell
I've been asked to put together a git branch with the latest versions of various dependencies (mach-virt, cpu-host, Mian's patchset) for aarch64 kvm control support in qemu. You can find it at: git://git.linaro.org/people/pmaydell/qemu-arm.git aarch64-kvm pointy-clicky version at: https://git

Re: [Qemu-devel] [PATCH 01/12] block: make BdrvRequestFlags public

2013-09-13 Thread Eric Blake
On 09/13/2013 04:24 AM, Peter Lieven wrote: > Signed-off-by: Peter Lieven > --- > block.c |5 - > include/block/block.h |5 + > 2 files changed, 5 insertions(+), 5 deletions(-) Reviewed-by: Eric Blake -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt v

Re: [Qemu-devel] [PATCH v9 5/8] module: implement module loading

2013-09-13 Thread Michael Tokarev
13.09.2013 18:52, Richard Henderson wrote: > Also, separate the whitelists by type. I.e. What for? Thanks, /mjt

Re: [Qemu-devel] [PATCH 0/8] [PATCH RFC v3] s390 cpu hotplug

2013-09-13 Thread Andreas Färber
Am 13.09.2013 17:01, schrieb Jason J. Herne: > On 09/05/2013 10:06 AM, Andreas Färber wrote: >> Am 05.09.2013 15:10, schrieb Alexander Graf: >>> On 05.09.2013, at 15:05, Andreas Färber wrote: Am 05.09.2013 14:54, schrieb Alexander Graf: > Very simple and clean patch set. I don't think it d

Re: [Qemu-devel] [PATCH v4 24/24] arm11mpcore: Split off RealView MPCore

2013-09-13 Thread Andreas Färber
Am 13.09.2013 17:33, schrieb Peter Maydell: > On 11 September 2013 15:37, Andreas Färber wrote: >> Signed-off-by: Andreas Färber >> --- >> hw/cpu/Makefile.objs | 1 + >> hw/cpu/arm11mpcore.c | 121 - >> hw/cpu/realview_mpcore.c | 139 >>

Re: [Qemu-devel] [PATCH v4 00/24] arm: ARM11MPCore+A9MPCore+A15MPCore QOM'ification

2013-09-13 Thread Peter Maydell
On 11 September 2013 15:37, Andreas Färber wrote: > From: Andreas Färber > > Hello Peter, > > This series fully QOM'ifies A9MPCore so that it can be embedded for Tegra2. > It goes on to do the same for A15MPCore, which had previously been taken as > template for Cortex-A57 by John Rigby, and sinc

Re: [Qemu-devel] [PATCH 5/8] [PATCH RFC v3] s390-qemu: cpu hotplug - ipi_states enhancements

2013-09-13 Thread Jason J. Herne
On 09/05/2013 08:01 AM, Andreas Färber wrote: Am 01.08.2013 16:12, schrieb Jason J. Herne: From: "Jason J. Herne" Modify s390_cpu_addr2state to allow fetching state information for cpu addresses above smp_cpus. Hotplug requires this capability. Also add s390_cpu_set_state function to allow m

Re: [Qemu-devel] [PATCH 6/8] [PATCH RFC v3] s390-qemu: cpu hotplug - s390 cpu init improvements for hotplug

2013-09-13 Thread Jason J. Herne
On 09/05/2013 08:28 AM, Andreas Färber wrote: Am 01.08.2013 16:12, schrieb Jason J. Herne: From: "Jason J. Herne" s390_new_cpu is created to encapsulate the creation of a new QOM S390CPU object given a cpuid and a model string. All actual cpu initialization code is moved from b

Re: [Qemu-devel] [PULL 33/42] block: return BDRV_BLOCK_ZERO past end of backing file

2013-09-13 Thread Kevin Wolf
Am 13.09.2013 um 10:25 hat Paolo Bonzini geschrieben: > Il 13/09/2013 09:33, Peter Lieven ha scritto: > > On 06.09.2013 17:39, Stefan Hajnoczi wrote: > >> From: Paolo Bonzini > >> > >> If the sectors are unallocated and we are past the end of the > >> backing file, they will read as zero. > >> > >

Re: [Qemu-devel] [PATCH 7/8] [PATCH RFC v3] s390-qemu: cpu hotplug - Implement hot_add_cpu hook

2013-09-13 Thread Jason J. Herne
On 09/05/2013 08:38 AM, Andreas Färber wrote: Am 01.08.2013 16:12, schrieb Jason J. Herne: From: "Jason J. Herne" Implement hot_add_cpu for S390 to allow hot plugging of cpus. Signed-off-by: Jason J. Herne --- hw/s390x/s390-virtio-ccw.c |3 +++ target-s390x/cpu.c | 32 ++

Re: [Qemu-devel] [PATCH] integrator: fix Linux boot failure by emulating dbg

2013-09-13 Thread Peter Maydell
On 13 September 2013 15:58, wrote: > From: Alex Bennée > > Commit 9b8c69243 broke the ability to boot the kernel as the value > returned by unassigned_mem_read returned non-zero and left the kernel > looping forever waiting for it to change (see integrator_led_set in > the kernel code). > > Rely

[Qemu-devel] [PATCH] .travis.yml: basic compile and check recipes

2013-09-13 Thread alex . bennee
From: Alex Bennée While QEMU already has various continuous integration set-ups in buildbot and commercial Jenkins setups there is some value in supporting travis-ci as well. It is well integrated into GitHub work flow and will trigger on all branch pushes and pull requests. This makes it easier

Re: [Qemu-devel] [PATCH v4 24/24] arm11mpcore: Split off RealView MPCore

2013-09-13 Thread Peter Maydell
On 11 September 2013 15:37, Andreas Färber wrote: > Signed-off-by: Andreas Färber > --- > hw/cpu/Makefile.objs | 1 + > hw/cpu/arm11mpcore.c | 121 - > hw/cpu/realview_mpcore.c | 139 > +++ > 3 files c

[Qemu-devel] [PATCH] integrator: fix Linux boot failure by emulating dbg

2013-09-13 Thread alex . bennee
From: Alex Benn??e Commit 9b8c69243 broke the ability to boot the kernel as the value returned by unassigned_mem_read returned non-zero and left the kernel looping forever waiting for it to change (see integrator_led_set in the kernel code). Relying on a varying implementation detail is incorrec

Re: [Qemu-devel] [PATCH 4/8] [PATCH RFC v3] s390-qemu: cpu hotplug - Storage key global access

2013-09-13 Thread Jason J. Herne
On 09/05/2013 07:46 AM, Andreas Färber wrote: Am 01.08.2013 16:12, schrieb Jason J. Herne: From: "Jason J. Herne" diff --git a/hw/s390x/s390-virtio.c b/hw/s390x/s390-virtio.c index 439d732..21e9124 100644 --- a/hw/s390x/s390-virtio.c +++ b/hw/s390x/s390-virtio.c @@ -123,6 +123,18 @@ static voi

Re: [Qemu-devel] [PATCH 01/47] rules.mak: New logical functions

2013-09-13 Thread Paolo Bonzini
Il 13/09/2013 17:02, Peter Maydell ha scritto: > On 13 September 2013 15:55, Paolo Bonzini wrote: >> Il 13/09/2013 15:43, Peter Maydell ha scritto: >>> On 25 August 2013 23:58, Ákos Kovács wrote: +eq = $(if $(subst $2,,$1)$(subst $1,,$2),n,y) +ne = $(if $(subst $2,,$1)$(subst $1,,$2),y,

Re: [Qemu-devel] [PATCH 01/47] rules.mak: New logical functions

2013-09-13 Thread Peter Maydell
On 13 September 2013 15:55, Paolo Bonzini wrote: > Il 13/09/2013 15:43, Peter Maydell ha scritto: >> On 25 August 2013 23:58, Ákos Kovács wrote: >>> +eq = $(if $(subst $2,,$1)$(subst $1,,$2),n,y) >>> +ne = $(if $(subst $2,,$1)$(subst $1,,$2),y,n) >> >> These give the wrong answer for comparisons

Re: [Qemu-devel] [PATCH 0/8] [PATCH RFC v3] s390 cpu hotplug

2013-09-13 Thread Jason J. Herne
On 09/05/2013 10:06 AM, Andreas Färber wrote: Am 05.09.2013 15:10, schrieb Alexander Graf: On 05.09.2013, at 15:05, Andreas Färber wrote: Am 05.09.2013 14:54, schrieb Alexander Graf: Very simple and clean patch set. I don't think it deserves the RFC tag. Negative, see my review. If you want

Re: [Qemu-devel] [PATCH 01/47] rules.mak: New logical functions

2013-09-13 Thread Paolo Bonzini
Il 13/09/2013 15:43, Peter Maydell ha scritto: > On 25 August 2013 23:58, Ákos Kovács wrote: >> lnot, land, lor, lif, eq, ne, isempty, notempty functions added >> Example usage: >> obj-$(call lor,$(CONFIG_LINUX),$(CONFIG_BSD)) += feature.o >> >> Signed-off-by: Ákos Kovács > > Hi; I like the

Re: [Qemu-devel] [PATCH v9 5/8] module: implement module loading

2013-09-13 Thread Richard Henderson
On 09/13/2013 02:59 AM, Fam Zheng wrote: > +const char *module_whitelist[] = { > +CONFIG_MODULE_WHITELIST > +}; static const char * const module_whitelist[] = ... > +switch (type) { > +case MODULE_LOAD_BLOCK: > +path = CONFIG_MODDIR "/block/"; > +break; >

Re: [Qemu-devel] [PATCH 21/47] hw/char/Kconfig: Add Kconfig file

2013-09-13 Thread Paolo Bonzini
Il 13/09/2013 16:00, Andreas Färber ha scritto: >> > I have no objection to hw/ipack, but I have a question. Would you >> > follow the SCSI/USB model (with devices under hw/ipack, also followed >> > for IndustryPack in the Linux kernel) or the virtio model (where the >> > device remains under hw/c

Re: [Qemu-devel] [RFC v5 1/5] hw/arm: add very initial support for Canon DIGIC SoC

2013-09-13 Thread Andreas Färber
Am 07.09.2013 09:04, schrieb Antony Pavlov: > DIGIC is Canon Inc.'s name for a family of SoC > for digital cameras and camcorders. > > There is no publicly available specification for > DIGIC chips. All information about DIGIC chip > internals is based on reverse engineering efforts > made by CHDK

Re: [Qemu-devel] [PATCH] seccomp: adding times() to the whitelist

2013-09-13 Thread Paul Moore
On Wednesday, September 04, 2013 10:11:10 AM Paul Moore wrote: > On Wednesday, September 04, 2013 09:25:08 AM Eduardo Otubo wrote: > > This was causing Qemu process to hang when using -sandbox on. > > > > Related RHBZ: https://bugzilla.redhat.com/show_bug.cgi?id=1004175 > > > > Signed-off-by: Edu

Re: [Qemu-devel] [RFC v5 0/5] hw/arm: add initial support for Canon DIGIC SoC

2013-09-13 Thread Antony Pavlov
On Sat, 7 Sep 2013 11:04:22 +0400 Antony Pavlov wrote: ping > [RFC v5 1/5] hw/arm: add very initial support for Canon DIGIC SoC > [RFC v5 2/5] hw/arm/digic: prepare DIGIC-based boards support > [RFC v5 3/5] hw/arm/digic: add timer support > [RFC v5 4/5] hw/arm/digic: add UART support > [RFC v5 5

[Qemu-devel] [PATCH 0/3] avoid CONFIG_NO_* using rules.mak logical functions

2013-09-13 Thread Peter Maydell
This is based on a set of patches by Ákos Kovács which were lurking at the beginning of his KConfig patchset but which I think make a nice standalone cleanup. Since I basically rewrote all of patch 1 in review comments I figured it was easier to just write it as a new patch. Patch 2 and 3 are Ákos'

Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks

2013-09-13 Thread Max Reitz
On 2013-09-13 14:29, Eric Blake wrote: On 09/13/2013 04:23 AM, Max Reitz wrote: […] I think, we should first take care of the command line interface and about QMP later (that is, if you agree on generally allowing modification of the qcow2 runtime options through QMP). There, we could offer both

[Qemu-devel] [PATCH 1/3] rules.mak: New logical functions for handling y/n values

2013-09-13 Thread Peter Maydell
Add new logical functions for handling y/n values like those we use in CONFIG_FOO variables: lnot : logical NOT land : logical AND lor : logical OR eq : equality ne : non-equality lif : like Make's $(if) but with an eq-like test and a couple of utility functions: isempty : true if argument

[Qemu-devel] [PATCH 2/3] Makefile.target: CONFIG_NO_* variables removed

2013-09-13 Thread Peter Maydell
From: Ákos Kovács CONFIG_NO_* variables replaced with the lnot logical function Signed-off-by: Ákos Kovács [PMM: fixed a few CONFIG_NO_* uses that were missed] Signed-off-by: Peter Maydell --- Makefile.target |8 ++-- hw/pci/Makefile.objs |2 +- target-arm/Makefile.

[Qemu-devel] PING Re: [PATCH v3] target-i386: Fix segment cache dump

2013-09-13 Thread Tobias Markus
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 PING (Mail resent - sorry, Thunderbird wrapped line in the first mail) PS: Sorry for the two wrapped lines below ---, but probably not worth resending the patch just for that. On 08/25/2013 12:20 PM, Tobias Markus wrote: > When in Long Mode, cpu_x86_

Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks

2013-09-13 Thread Eric Blake
On 09/13/2013 08:29 AM, Max Reitz wrote: > Another idea: Instead of providing an integer for "shorthand" > manipulations, what do you think of a string parameter (such as -o > cache=foo right now, although I do know -o cache isn't even document > anymore and provided only for compatibility reasons,

[Qemu-devel] [PATCH 3/3] default-configs/: CONFIG_GDBSTUB_XML removed

2013-09-13 Thread Peter Maydell
From: Ákos Kovács Makefile.target: Build gdbstub-xml.o only when TARGET_XML_FILES is not empty. Signed-off-by: Ákos Kovács Reviewed-by: Peter Maydell --- Makefile.target |2 +- default-configs/arm-linux-user.mak|2 -- default-configs/arm-softmmu.mak

Re: [Qemu-devel] [PATCH 21/47] hw/char/Kconfig: Add Kconfig file

2013-09-13 Thread Andreas Färber
Am 27.08.2013 00:40, schrieb Paolo Bonzini: > Il 26/08/2013 19:15, Andreas Färber ha scritto: PCI devices are generally configurable, so you need to add prompts to them. >> IndustryPack is really misplaced in hw/char/ and I believe I posted >> patches to remedy that and let one actually find i

[Qemu-devel] PING Re: [PATCH v3] target-i386: Fix segment cache dump

2013-09-13 Thread Tobias Markus
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 08/25/2013 12:20 PM, Tobias Markus wrote: > When in Long Mode, cpu_x86_seg_cache() logs "DS16" because the > Default operation size bit (D/B bit) is not set for Long Mode Data > Segments since there are only Data Segments in Long Mode and no > expli

Re: [Qemu-devel] [PATCH v9 5/8] module: implement module loading

2013-09-13 Thread Daniel P. Berrange
On Fri, Sep 13, 2013 at 05:59:13PM +0800, Fam Zheng wrote: > +void module_load(module_load_type type) > +{ > +#ifdef CONFIG_MODULES > +const char *path; > +char *fname = NULL; > +const char **mp; > +const char *module_whitelist[] = { > +CONFIG_MODULE_WHITELIST > +}; > +

[Qemu-devel] [PATCH v2 uq/master 1/2] x86: fix migration from pre-version 12

2013-09-13 Thread Paolo Bonzini
On KVM, the KVM_SET_XSAVE would be executed with a 0 xstate_bv, and not restore anything. Since FP and SSE data are always valid, set them in xstate_bv at reset time. In fact, that value is the same that KVM_GET_XSAVE returns on pre-XSAVE hosts. Signed-off-by: Paolo Bonzini --- target-i386/cpu

[Qemu-devel] [PATCH v2 uq/master 2/2] x86: cpuid: reconstruct leaf 0Dh data

2013-09-13 Thread Paolo Bonzini
The data in leaf 0Dh depends on information from other feature bits. Instead of passing it blindly from the host, compute it based on whether these feature bits are enabled. Signed-off-by: Paolo Bonzini --- target-i386/cpu.c | 63 +++- 1 file chang

[Qemu-devel] [PATCH v2 uq/master 0/2] KVM: issues with XSAVE support

2013-09-13 Thread Paolo Bonzini
This series fixes two migration bugs concerning KVM's XSAVE ioctls. The second right now is only a theoretical problem, since the only XSAVE-specific state is AVX and all machines with XSAVE also have AVX. In the future, this will ensure that machines with MPX or AVX-512 do not expose those XSAVE s

Re: [Qemu-devel] [PATCH v4] libxl: usb2 and usb3 controller support for upstream qemu

2013-09-13 Thread Ian Campbell
On Fri, 2013-09-13 at 15:42 +0200, Fabio Fantoni wrote: > >>> +case 2: > >>> +flexarray_vappend(dm_args, > >>> "-device","ich9-usb-ehci1,id=usb," > >>> +"bus=pci.0,addr=0x1d.0x7", NULL); > >>> +for (i = 1; i < 4; i++) > >>> +

Re: [Qemu-devel] [PATCH 12/12] block/get_block_status: fix BDRV_BLOCK_ZERO for unallocated blocks

2013-09-13 Thread Eric Blake
On 09/13/2013 04:25 AM, Peter Lieven wrote: > this patch does 2 things: > a) only do additional call outs if BDRV_BLOCK_ZERO is not already set. > b) use bdi.discard_zeroes to return the zero state of an unallocated block. >the callout to bdrv_has_zero_init() is only valid right after bdrv_crea

Re: [Qemu-devel] [PATCH 01/47] rules.mak: New logical functions

2013-09-13 Thread Peter Maydell
On 25 August 2013 23:58, Ákos Kovács wrote: > lnot, land, lor, lif, eq, ne, isempty, notempty functions added > Example usage: > obj-$(call lor,$(CONFIG_LINUX),$(CONFIG_BSD)) += feature.o > > Signed-off-by: Ákos Kovács Hi; I like the general idea here but I think there are some issues with y

Re: [Qemu-devel] [PATCH v4] libxl: usb2 and usb3 controller support for upstream qemu

2013-09-13 Thread Fabio Fantoni
Il 13/09/2013 13:59, Ian Campbell ha scritto: On Mon, 2013-07-15 at 11:11 +0200, Fabio Fantoni wrote: Stefano or Anthony -- your input as qemu-xen types would be appreciated here on this patch. +case 2: +flexarray_vappend(dm_args, "-device","ich9-usb-ehci1,id=usb,"

Re: [Qemu-devel] MSI-X doesn't work when running Windows as guest

2013-09-13 Thread Andreas Färber
Am 13.09.2013 14:31, schrieb Michael S. Tsirkin: > On Fri, Sep 13, 2013 at 03:22:01PM +0300, Michael S. Tsirkin wrote: >> On Fri, Sep 13, 2013 at 01:14:43AM -0300, Eduardo Habkost wrote: >>> On Fri, Sep 13, 2013 at 12:03:40AM +0300, Michael S. Tsirkin wrote: On Thu, Sep 12, 2013 at 04:45:01PM

Re: [Qemu-devel] [PATCH 05/12] block: add logical block provisioning information to BlockDriverInfo

2013-09-13 Thread Paolo Bonzini
Il 13/09/2013 14:22, Peter Lieven ha scritto: > Am 13.09.2013 13:45, schrieb Paolo Bonzini: >> Il 13/09/2013 12:44, Peter Lieven ha scritto: >>> On 13.09.2013 12:34, Paolo Bonzini wrote: Il 13/09/2013 12:25, Peter Lieven ha scritto: > +/* maximum number of sectors that can be discarded

Re: [Qemu-devel] [PATCH v2 07/47] hw/arm/Makefile.objs: CONFIG_* created for each board

2013-09-13 Thread Peter Maydell
On 26 August 2013 10:40, Ákos Kovács wrote: > The new CONFIG_* definitions added to the default-configs/arm-softmmu.mak > Signed-off-by: Ákos Kovács Just noticed this, but please don't send just one updated patch out of a series. If you need to send out an updated v2/v3/etc, resend the whole ser

Re: [Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks

2013-09-13 Thread Max Reitz
On 2013-09-13 14:29, Eric Blake wrote: On 09/13/2013 04:23 AM, Max Reitz wrote: The more interesting part is that adding an option always needs thought because once it is exposed, it's an API that is set in stone. And I'm also not sure what the best command line and QMP representations of a bitm

Re: [Qemu-devel] Wiki vandalism

2013-09-13 Thread Anthony Liguori
On Fri, Sep 13, 2013 at 1:06 AM, Stefan Weil wrote: > Am 13.09.2013 07:57, schrieb Michal Privoznik: >> So I've just got another trivial patch ACKed. Yay! So I wanted to check >> it out in the next-trivial branch. Therefore I went on the qemu-trivial >> wiki and found two things: >> >> 1) Plenty o

Re: [Qemu-devel] MSI-X doesn't work when running Windows as guest

2013-09-13 Thread Michael S. Tsirkin
On Fri, Sep 13, 2013 at 03:22:01PM +0300, Michael S. Tsirkin wrote: > On Fri, Sep 13, 2013 at 01:14:43AM -0300, Eduardo Habkost wrote: > > On Fri, Sep 13, 2013 at 12:03:40AM +0300, Michael S. Tsirkin wrote: > > > On Thu, Sep 12, 2013 at 04:45:01PM -0300, Eduardo Habkost wrote: > > > > On Thu, Sep 1

  1   2   3   >