[Qemu-devel] [PATCH] PPC: Fix dcbz for linux-user on 970

2013-04-26 Thread Alexander Graf
The default with linux-user for dcbz on 970 is to emulate 32 byte clears. However, redoing the dcbzl support we added a check to not honor the bit in HID5 that sets this. Remove the #ifdef check on linux user, so that we get 32 byte clears again. Reported-by: Riku Voipio Signed-off-by: Alexander

Re: [Qemu-devel] [PATCH] ppc: initialize GPRs as per epapr

2013-04-26 Thread Bhushan Bharat-R65777
> -Original Message- > From: Alexander Graf [mailto:ag...@suse.de] > Sent: Friday, April 26, 2013 11:51 AM > To: Bhushan Bharat-R65777 > Cc: qemu-...@nongnu.org; qemu-devel@nongnu.org; Wood Scott-B07421; Bhushan > Bharat-R65777; Yoder Stuart-B08248 > Subject: Re: [PATCH] ppc: initialize G

Re: [Qemu-devel] [PATCH 04/15] target-i386: introduce apic-id property

2013-04-26 Thread Igor Mammedov
On Thu, 25 Apr 2013 17:36:45 -0300 Eduardo Habkost wrote: > On Thu, Apr 25, 2013 at 04:05:26PM +0200, Igor Mammedov wrote: > [...] > > +static void x86_cpuid_set_apic_id(Object *obj, Visitor *v, void *opaque, > > + const char *name, Error **errp) > > +{ > > +X

Re: [Qemu-devel] [PATCH v2 05/10] target-ppc: emulate prtyw and prtyd instructions

2013-04-26 Thread Alexander Graf
On 20.04.2013, at 20:56, Aurelien Jarno wrote: > Needed for Power ISA version 2.05 compliance. > > Reviewed-by: Richard Henderson > Signed-off-by: Aurelien Jarno > --- > target-ppc/translate.c | 38 ++ > 1 file changed, 38 insertions(+) > > diff --git a/ta

Re: [Qemu-devel] [PATCH v2 00/10] target-ppc: emulate Power ISA 2.05 instructions

2013-04-26 Thread Alexander Graf
On 20.04.2013, at 20:56, Aurelien Jarno wrote: > qemu-system-ppc64 -M pseries clamis to emulate a POWER7 CPU, > corresponding to an ISA 2.06 compliant CPU, while QEMU emulates > something like an ISA 2.04 CPU. > > Given that glibc and GCC start to use some instructions like cmpb and > fcpsgn, th

Re: [Qemu-devel] [PATCH for-1.5] qmp: add query-drive-mirror-capabilities

2013-04-26 Thread Paolo Bonzini
Il 25/04/2013 14:26, Luiz Capitulino ha scritto: > That's a good point, although I wonder if a command could have a new > capability that's not mapped to a new argument. IOW, I'd expect most/all > new capabilities to always be mapped to new arguments. A new enum value would also be a new capabilit

Re: [Qemu-devel] [PATCH] rbd: Fix use after free in rbd_open()

2013-04-26 Thread Stefan Hajnoczi
On Thu, Apr 25, 2013 at 03:59:27PM +0200, Kevin Wolf wrote: > Commit 56d1b4d2 frees the QemuOpts for the driver-specific options > immediately, even though it still needs the filename string that is > contained there. This doesn't work. Move the deletion of the QemuOpts to > the end of the function

Re: [Qemu-devel] [PATCH 1/4] Fix vmw_pvscsi.c for iommu support changes

2013-04-26 Thread Paolo Bonzini
Il 26/04/2013 08:02, David Gibson ha scritto: > vmw_pvscsi.c directly calls pci_dma_sglist_init() instead of using the > helper for PCI devices, which means it was broken by Paolo Bonzini's > recent addition of iommu support to the memory API. This fixes it. > > Signed-off-by: David Gibson > ---

Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion

2013-04-26 Thread Paolo Bonzini
Il 26/04/2013 08:02, David Gibson ha scritto: > At the moment, vfio maintains a global list of containers that are assumed > to be more or less interchangeable, since they are all set up with a > MemoryListener to have all of system memory mapped. However, that only > makes sense if all the contai

Re: [Qemu-devel] [PATCH] ich9: kill cmos_s3

2013-04-26 Thread Paolo Bonzini
Il 24/04/2013 20:49, Andreas Färber ha scritto: > Am 24.04.2013 13:30, schrieb Paolo Bonzini: >> Il 24/04/2013 12:37, Hu Tao ha scritto: >>> Signed-off-by: Hu Tao >> >> I had exactly the same patch in my 1.6 queue. :) >> >> Reviewed-by: Paolo Bonzini > > The commit message fails to explain *why*

Re: [Qemu-devel] [PULL 0/9] SCSI updates for 2013-04-13

2013-04-26 Thread Paolo Bonzini
Il 24/04/2013 18:56, Stefan Weil ha scritto: > Am 24.04.2013 10:19, schrieb Paolo Bonzini: >> Il 24/04/2013 06:56, Stefan Weil ha scritto: >>> Hi Paolo, the latest QEMU build is broken when CONFIG_VHOST_SCSI is >>> undefined. >> --verbose, please. :) > > Sorry, I thought it was obvious. The error

[Qemu-devel] [PATCH 1/3] virtio-pci: properly validate address before accessing config

2013-04-26 Thread Jason Wang
There are several several issues in the current checking: - The check was based on the minus of unsigned values which can overflow - It was done after .{set|get}_config() which can lead crash when config_len is zero since vdev->config is NULL Fix this by: - Validate the address in virtio_pci_c

[Qemu-devel] [PATCH 2/3] virtio-ccw: check config length before accessing it

2013-04-26 Thread Jason Wang
virtio-rng-ccw has zero config length, so we need validate the config length before trying to access it. Otherwise we may crash since vdev->config is NULL. Cc: Cornelia Huck Cc: Richard Henderson Cc: Alexander Graf Signed-off-by: Jason Wang --- hw/s390x/virtio-ccw.c |4 ++-- 1 files chang

Re: [Qemu-devel] [PATCH 4/5] vfio: Move container list to DMAContext

2013-04-26 Thread Paolo Bonzini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Il 25/04/2013 08:36, David Gibson ha scritto: > On Wed, Apr 24, 2013 at 06:33:33PM +0200, Paolo Bonzini wrote: >> Il 24/04/2013 17:12, Alex Williamson ha scritto: > At the moment, vfio maintains a global list of containers > that are assumed to

[Qemu-devel] [PATCH 3/3] s390-virtio-bus: sync config only when config_len is not zero

2013-04-26 Thread Jason Wang
virtio-rng-s390 has zero config length, so no need to sync its config otherwise qemu will crash since vdev->config is NULL. Cc: Alexander Graf Cc: Richard Henderson Signed-off-by: Jason Wang --- hw/s390x/s390-virtio-bus.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --gi

Re: [Qemu-devel] [PATCH 4/5] vfio: Move container list to DMAContext

2013-04-26 Thread Alexey Kardashevskiy
On 04/26/2013 06:44 PM, Paolo Bonzini wrote: Il 25/04/2013 08:36, David Gibson ha scritto: On Wed, Apr 24, 2013 at 06:33:33PM +0200, Paolo Bonzini wrote: Il 24/04/2013 17:12, Alex Williamson ha scritto: At the moment, vfio maintains a global list of containers that are assumed to be more or l

Re: [Qemu-devel] [PATCH 4/5] vfio: Move container list to DMAContext

2013-04-26 Thread Paolo Bonzini
Il 26/04/2013 10:46, Alexey Kardashevskiy ha scritto: > On 04/26/2013 06:44 PM, Paolo Bonzini wrote: >> Il 25/04/2013 08:36, David Gibson ha scritto: >>> On Wed, Apr 24, 2013 at 06:33:33PM +0200, Paolo Bonzini wrote: Il 24/04/2013 17:12, Alex Williamson ha scritto: >>> At the moment, vfio

Re: [Qemu-devel] [PATCH 04/15] target-i386: introduce apic-id property

2013-04-26 Thread Eduardo Habkost
On Fri, Apr 26, 2013 at 09:32:59AM +0200, Igor Mammedov wrote: > On Thu, 25 Apr 2013 17:36:45 -0300 > Eduardo Habkost wrote: > > > On Thu, Apr 25, 2013 at 04:05:26PM +0200, Igor Mammedov wrote: > > [...] > > > +static void x86_cpuid_set_apic_id(Object *obj, Visitor *v, void *opaque, > > > +

Re: [Qemu-devel] [PATCH 4/5] vfio: Move container list to DMAContext

2013-04-26 Thread Alexey Kardashevskiy
On 04/26/2013 06:52 PM, Paolo Bonzini wrote: Il 26/04/2013 10:46, Alexey Kardashevskiy ha scritto: On 04/26/2013 06:44 PM, Paolo Bonzini wrote: Il 25/04/2013 08:36, David Gibson ha scritto: On Wed, Apr 24, 2013 at 06:33:33PM +0200, Paolo Bonzini wrote: Il 24/04/2013 17:12, Alex Williamson ha

Re: [Qemu-devel] [PATCH v2 0/4] sheepdog: support online snapshot from qemu-img

2013-04-26 Thread Stefan Hajnoczi
On Fri, Apr 26, 2013 at 01:19:50AM +0900, MORITA Kazutaka wrote: > v2: > - rename reload_vdi_obj to reload_inode and use it from >sd_snapshot_goto (Yuan) > - use g_free instead of free (Stefan) > - fix typo in the comment (Stefan) > - add coroutine_fn marker to resend_aioreq > > Currently,

Re: [Qemu-devel] [sheepdog] [PATCH v4] sheepdog: fix loadvm operation

2013-04-26 Thread Liu Yuan
On 04/25/2013 09:06 PM, MORITA Kazutaka wrote: > At Thu, 25 Apr 2013 20:49:39 +0800, > Liu Yuan wrote: >> >> From: Liu Yuan >> >> Currently the 'loadvm' opertaion works as following: >> 1. switch to the snapshot >> 2. mark current working VDI as a snapshot >> 3. rely on sd_create_branch to create

Re: [Qemu-devel] [PATCH 4/5] vfio: Move container list to DMAContext

2013-04-26 Thread Paolo Bonzini
Il 26/04/2013 10:56, Alexey Kardashevskiy ha scritto: > On 04/26/2013 06:52 PM, Paolo Bonzini wrote: >> Il 26/04/2013 10:46, Alexey Kardashevskiy ha scritto: >>> On 04/26/2013 06:44 PM, Paolo Bonzini wrote: Il 25/04/2013 08:36, David Gibson ha scritto: > On Wed, Apr 24, 2013 at 06:33:33PM

Re: [Qemu-devel] [RFC PATCH v5 01/14] util: introduce gsource event abstraction

2013-04-26 Thread Stefan Hajnoczi
On Fri, Apr 26, 2013 at 10:47:22AM +0800, Liu Ping Fan wrote: > +GPollFD *events_source_add_gfd(EventsGSource *src, int fd) > +{ > +GPollFD *retfd; > + > +retfd = g_slice_alloc(sizeof(GPollFD)); > +retfd->events = 0; > +retfd->fd = fd; > +src->pollfds_list = g_list_append(src->p

[Qemu-devel] [PATCH v3 0/2] i2c: Add AT24Cxx EEPROM model

2013-04-26 Thread Jan Kiszka
Changes in v3: - rebased over current master - addressed review comments by Andreas Färber and Peter Crosthwaite - better QOM compliance - table-driven device parametrization I also briefly looked into a libqos-based test case, but the omap wrapper just gives me libi2c-omap.c:79:omap_

[Qemu-devel] [PATCH v3 2/2] Add AT24Cxx I2C EEPROM device model

2013-04-26 Thread Jan Kiszka
This implements I2C EEPROMs of the AT24Cxx series. Sizes from 1Kbit to 1024Kbit are supported. Each EEPROM is backed by a block device. Its size can be explicitly specified by the "size" property (required for sizes < 512, the blockdev sector size) or is derived from the size of the backing block d

[Qemu-devel] [PATCH v3 1/2] i2c: Introduce device address mask

2013-04-26 Thread Jan Kiszka
Some devices react on multiple addresses. To emulate this, we could register them multiple times, but that is cumbersome. The AT24C16, e.g. listens on 8 different addresses. Instead, introduce a device address mask that is applied on the transmitted address before matching it against the stored on

Re: [Qemu-devel] [RFC PATCH v5 04/14] net: port vde onto GSource

2013-04-26 Thread Stefan Hajnoczi
On Fri, Apr 26, 2013 at 10:47:25AM +0800, Liu Ping Fan wrote: > +static gboolean vde_handler(gpointer data) > +{ > +EventGSource *nsrc = (EventGSource *)data; > + > +if (nsrc->gfd.revents & G_IO_IN) { The VDE file descriptor is a socket. Please use the full G_IO_IN | G_IO_HUP | G_IO_ERR s

Re: [Qemu-devel] [PATCH v4 3/7] hw/acpi: extract standard table headers as a standalone structure

2013-04-26 Thread Laszlo Ersek
On 04/25/13 20:47, Anthony Liguori wrote: > Laszlo Ersek writes: >> +typedef struct acpi_table_std_header { >> +char sig[4]; /* ACPI signature (4 ASCII characters) */ >> +uint32_t length; /* Length of table, in bytes, including >> header */ >> +uint8_t revision;

[Qemu-devel] [PATCH 0/7] qapi and snapshot code clean up in block layer

2013-04-26 Thread Wenchao Xia
These patches are the common part of my hmp/qmp block query series and Pavel's qmp snapshot command converion series. It mainly does following things: 1 move snapshot related code to block/snapshot.c, qmp and info dumping code to block/qapi.c. 2 better bdrv_snapshot_find(). 3 better info dumping fu

[Qemu-devel] [PATCH 1/7] block: drop bs_snapshots global variable

2013-04-26 Thread Wenchao Xia
From: Stefan Hajnoczi The bs_snapshots global variable points to the BlockDriverState which will be used to save vmstate. This is really a savevm.c concept but was moved into block.c:bdrv_snapshots() when it became clear that hotplug could result in a dangling pointer. While auditing the block

[Qemu-devel] [PATCH 2/7] block: move bdrv_snapshot_find() to block/snapshot.c

2013-04-26 Thread Wenchao Xia
This patch adds block/snapshot.c and then moves the function there. It also fixes small code style errors reported by check script. Signed-off-by: Wenchao Xia Reviewed-by: Eric Blake Reviewed-by: Kevin Wolf --- block/Makefile.objs |1 + block/snapshot.c | 48

[Qemu-devel] [PATCH 3/7] block: move snapshot code in block.c to block/snapshot.c

2013-04-26 Thread Wenchao Xia
All snapshot related code, except bdrv_snapshot_dump(), is moved to block/snapshot.c. bdrv_snapshot_dump() will be moved to another file later. It also fixes small code style errors reported by check script. Signed-off-by: Wenchao Xia --- block.c | 105 -

[Qemu-devel] [PATCH 4/7] block: distinguish id and name in bdrv_find_snapshot()

2013-04-26 Thread Wenchao Xia
To make it clear about id and name in searching, the API is changed a bit to distinguish them, and caller can choose to search by id or name. If not found, *errp will be set to tip why. Note that the caller logic is changed a bit: 1) In del_existing_snapshots() called by do_savevm(), it travers tw

[Qemu-devel] [PATCH 6/7] block: move qmp and info dump related code to block/qapi.c

2013-04-26 Thread Wenchao Xia
This patch is a pure code move patch, except following modification: 1 get_human_readable_size() is changed to static function. 2 dump_human_image_info() is renamed to bdrv_image_info_dump(). 3 in qmp_query_block() and qmp_query_blockstats, use bdrv_next(bs) instead of direct traverse of global arr

[Qemu-devel] [PATCH 7/7] block: dump to monitor for bdrv_snapshot_dump() and bdrv_image_info_dump()

2013-04-26 Thread Wenchao Xia
This patch introduce a new print function, which will output message to monitor when it present. With it, bdrv_snapshot_dump() need no more buffer and can avoid string truncation, bdrv_image_info_dump() can also be used by hmp code later, besides qemu-img code. Signed-off-by: Wenchao Xia --- blo

[Qemu-devel] [PATCH 5/7] block: move collect_snapshots() and collect_image_info() to block/qapi.c

2013-04-26 Thread Wenchao Xia
This patch adds block/qapi.c and moves the functions there. To avoid conflict and tip better, macro in header file is BLOCK_QAPI_H instead of QAPI_H. Signed-off-by: Wenchao Xia Reviewed-by: Eric Blake Reviewed-by: Kevin Wolf --- block/Makefile.objs |2 +- block/qapi.c | 107 +

Re: [Qemu-devel] [PATCH 04/15] target-i386: introduce apic-id property

2013-04-26 Thread Igor Mammedov
On Fri, 26 Apr 2013 05:53:55 -0300 Eduardo Habkost wrote: > On Fri, Apr 26, 2013 at 09:32:59AM +0200, Igor Mammedov wrote: > > On Thu, 25 Apr 2013 17:36:45 -0300 > > Eduardo Habkost wrote: > > > > > On Thu, Apr 25, 2013 at 04:05:26PM +0200, Igor Mammedov wrote: > > > [...] > > > > +static void

Re: [Qemu-devel] [PATCH v2 05/10] target-ppc: emulate prtyw and prtyd instructions

2013-04-26 Thread Aurelien Jarno
On Fri, Apr 26, 2013 at 09:50:31AM +0200, Alexander Graf wrote: > > On 20.04.2013, at 20:56, Aurelien Jarno wrote: > > > Needed for Power ISA version 2.05 compliance. > > > > Reviewed-by: Richard Henderson > > Signed-off-by: Aurelien Jarno > > --- > > target-ppc/translate.c | 38

Re: [Qemu-devel] [PATCH v4 4/7] hw/acpi: export default ACPI headers using the type just introduced

2013-04-26 Thread Laszlo Ersek
On 04/25/13 20:49, Anthony Liguori wrote: > Laszlo Ersek writes: > >> This enables reuse when preparing per-table fw_cfg blobs later. >> >> Signed-off-by: Laszlo Ersek >> Acked-by: Michael S. Tsirkin >> --- >> include/hw/acpi/acpi.h |2 ++ >> hw/acpi/core.c | 39 +

Re: [Qemu-devel] [PATCH v2 05/10] target-ppc: emulate prtyw and prtyd instructions

2013-04-26 Thread Alexander Graf
On 26.04.2013, at 11:38, Aurelien Jarno wrote: > On Fri, Apr 26, 2013 at 09:50:31AM +0200, Alexander Graf wrote: >> >> On 20.04.2013, at 20:56, Aurelien Jarno wrote: >> >>> Needed for Power ISA version 2.05 compliance. >>> >>> Reviewed-by: Richard Henderson >>> Signed-off-by: Aurelien Jarno

Re: [Qemu-devel] [PATCH] linux-user: fix compile error due to stray colon at end of #ifdef line

2013-04-26 Thread Stefan Hajnoczi
On Sun, Apr 21, 2013 at 01:30:03PM +0100, Peter Maydell wrote: > Remove a stray colon from the end of a #ifdef line. Some versions > of gcc complain about this: > linux-user/syscall.c: In function ‘do_syscall’: > linux-user/syscall.c:7606:28: error: extra tokens at end of #ifdef directive > [-We

Re: [Qemu-devel] [PATCH v2 05/10] target-ppc: emulate prtyw and prtyd instructions

2013-04-26 Thread Aurelien Jarno
On Fri, Apr 26, 2013 at 11:53:49AM +0200, Alexander Graf wrote: > > On 26.04.2013, at 11:38, Aurelien Jarno wrote: > > > On Fri, Apr 26, 2013 at 09:50:31AM +0200, Alexander Graf wrote: > >> > >> On 20.04.2013, at 20:56, Aurelien Jarno wrote: > >> > >>> Needed for Power ISA version 2.05 complian

Re: [Qemu-devel] [Qemu-trivial] [PATCH] s390x: use CONFIG_INT128 to detect __uint128_t

2013-04-26 Thread Stefan Hajnoczi
On Tue, Apr 23, 2013 at 06:15:12PM +0100, Gabriel Kerneis wrote: > Target s390x uses ad-hoc macro magic to guess if the compiler > supports the GCC extension __uint128_t. This patch uses the > the dedicated macro CONFIG_INT128 defined by configure instead. > > This fixes compilation with the CIL

Re: [Qemu-devel] [PATCH v6 00/20] tcg-arm improvments

2013-04-26 Thread Peter Maydell
On 23 April 2013 21:46, Richard Henderson wrote: > Changes v5-v6, feedback from Aurelien: > * Drop tcg_out_goto changes. > * Add Z constraint for deposit. > * Small comment cleanups. > * Rebase vs master. I gave this a quick spin on an A15 and an A8 -- seems to work (though I have by no m

Re: [Qemu-devel] [PATCH v2] qapi: add struct strList and visit_type_strList()

2013-04-26 Thread Amos Kong
Hi Michael, On Wed, Apr 24, 2013 at 11:46:25AM -0500, mdroth wrote: > On Thu, Apr 25, 2013 at 12:08:05AM +0800, Amos Kong wrote: > > Currently we can only use ['String'] to add string to a list, > > it contains some additional JSON structure. > > "multicast": [ > > { > > "s

Re: [Qemu-devel] [PATCH] PPC: Fix dcbz for linux-user on 970

2013-04-26 Thread Riku Voipio
On Fri, Apr 26, 2013 at 09:20:51AM +0200, Alexander Graf wrote: > The default with linux-user for dcbz on 970 is to emulate 32 byte clears. > However, redoing the dcbzl support we added a check to not honor the bit > in HID5 that sets this. > > Remove the #ifdef check on linux user, so that we get

Re: [Qemu-devel] [PATCH] virtio: abort on zero config length

2013-04-26 Thread Eric Blake
On 04/25/2013 11:06 PM, Jason Wang wrote: >>> if (addr > (vdev->config_len - sizeof(val))) >>> >>> ^ quiz: spot a bug above if config_len is 0:) >> Then we need to fix these bugs and allocate a CVE. virtio-rng has >> shipped. This code is also dumb. > > Ok, but since the discussi

Re: [Qemu-devel] [PATCH] virtio: abort on zero config length

2013-04-26 Thread Jason Wang
On 04/26/2013 06:32 PM, Eric Blake wrote: > On 04/25/2013 11:06 PM, Jason Wang wrote: if (addr > (vdev->config_len - sizeof(val))) ^ quiz: spot a bug above if config_len is 0:) >>> Then we need to fix these bugs and allocate a CVE. virtio-rng has >>> shipped. This c

[Qemu-devel] [PATCH] virtio-net: properly check the vhost status during status set

2013-04-26 Thread Jason Wang
Commit 32993698 (vhost: disable on tap link down) tries to disable the vhost also when the peer's link is down. But the check was not done properly, the vhost were only started when: 1) peer's link is not down 2) virtio-net has already been started. Since == have a higher precedence than &&, plac

Re: [Qemu-devel] [PATCH v2] virtio-net: unbreak the minix guest

2013-04-26 Thread Aurelien Jarno
On Thu, Apr 25, 2013 at 03:24:23PM +0800, Jason Wang wrote: > Multiqueue patchset conditionally add control vq only when guest negotiate the > feature. Though the spec is not clear on this but it breaks the minix guest > since it will identify the ctrl vq even if it does not support it. Though this

Re: [Qemu-devel] [PATCH] qemu-doc: Option -ignore-environment removed.

2013-04-26 Thread Stefan Hajnoczi
On Thu, Apr 25, 2013 at 06:41:16PM +0200, Thomas Schwinge wrote: > Has been removed in commit fc9c54124d134dbd76338a92a91804dab2df8166. > > Signed-off-by: Thomas Schwinge > --- > qemu-doc.texi |3 --- > 1 file changed, 3 deletions(-) Thanks, applied to the trivial patches tree: https://gith

Re: [Qemu-devel] [Qemu-trivial] [PATCH] Track change in FreeBSD SYSCTL(9) types

2013-04-26 Thread Stefan Hajnoczi
On Thu, Apr 25, 2013 at 01:59:41PM -0400, Ed Maste wrote: CCing qemu-devel@nongnu.org. > Originally from Garrett Cooper in FreeBSD PR ports/18 > http://www.freebsd.org/cgi/query-pr.cgi?pr=18 > > Signed-off-by: Ed Maste > --- > bsd-user/syscall.c | 4 > 1 file changed, 4 insertions

Re: [Qemu-devel] [Qemu-trivial] [PATCH] Fix compilation without CONFIG_VHOST_SCSI

2013-04-26 Thread Stefan Hajnoczi
On Thu, Apr 25, 2013 at 01:38:28PM -0400, Ed Maste wrote: CCing qemu-devel@nongnu.org. > > Signed-off-by: Ed Maste > --- > hw/s390x/s390-virtio-bus.c | 2 ++ > hw/s390x/virtio-ccw.c | 2 ++ > hw/s390x/virtio-ccw.h | 2 ++ > 3 files changed, 6 insertions(+) > > diff --git a/hw/s390x/

Re: [Qemu-devel] [PATCH v4 7/7] hw/i386: build ACPI MADT (APIC) for fw_cfg clients

2013-04-26 Thread Laszlo Ersek
On 04/25/13 21:03, Anthony Liguori wrote: > Laszlo Ersek writes: > >> This patch reuses some code from SeaBIOS, which was originally under >> LGPLv2 and then relicensed to GPLv3 or LGPLv3, in QEMU under GPLv2+. This >> relicensing has been acked by all contributors that had contributed to the >>

Re: [Qemu-devel] [PATCH 1/4] Fix vmw_pvscsi.c for iommu support changes

2013-04-26 Thread David Gibson
On Fri, Apr 26, 2013 at 10:19:55AM +0200, Paolo Bonzini wrote: > Il 26/04/2013 08:02, David Gibson ha scritto: > > vmw_pvscsi.c directly calls pci_dma_sglist_init() instead of using the > > helper for PCI devices, which means it was broken by Paolo Bonzini's > > recent addition of iommu support to

[Qemu-devel] [PATCH 1/2] gtk: fix cursor unref

2013-04-26 Thread Gerd Hoffmann
Use correct unref function, cursors are not gobjects (at least in gtk2). Fixes crash, reproducer: "qemu -vga qxl -display gtk". Signed-off-by: Gerd Hoffmann --- ui/gtk.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ui/gtk.c b/ui/gtk.c index 42e3c0a..71fda24 100644 --- a

[Qemu-devel] [PATCH 2/2] console: nicer initial screen

2013-04-26 Thread Gerd Hoffmann
Now that we have a function to create a fancy DisplaySurface with a message for the user, to handle non-existing graphics hardware, we can make it more generic and use it for other things too. This patch adds a text line to the in initial DisplaySurface, notifying the user that the display isn't i

[Qemu-devel] [PATCH 0/2] console tweaks

2013-04-26 Thread Gerd Hoffmann
Hi, Two little console tweaks. One gtk bugfix, and one small usability improvement. please review, Gerd The following changes since commit e3351000cd682200835763caca87adf708ed1c65: Makefile: Use QEMU_FLAGS for DTC compilation (2013-04-26 02:48:10 +0200) are available in the git reposito

[Qemu-devel] [PULL 0/5] Trivial patches for 20 to 26 April 2013

2013-04-26 Thread Stefan Hajnoczi
The following changes since commit e3351000cd682200835763caca87adf708ed1c65: Makefile: Use QEMU_FLAGS for DTC compilation (2013-04-26 02:48:10 +0200) are available in the git repository at: git://github.com/stefanha/qemu.git trivial-patches for you to fetch changes up to 6ea8430d98c48eb67e4

Re: [Qemu-devel] [RFC PATCH v5 05/14] net: port socket to GSource

2013-04-26 Thread Stefan Hajnoczi
On Fri, Apr 26, 2013 at 10:47:26AM +0800, Liu Ping Fan wrote: > @@ -141,6 +134,59 @@ static ssize_t net_socket_receive_dgram(NetClientState > *nc, const uint8_t *buf, > return ret; > } > > +static gushort socket_connecting_readable(void *opaque) > +{ > +return G_IO_IN; > +} > + > +stat

Re: [Qemu-devel] [PATCH 3/4] vfio: Move container list to iommu MemoryRegion

2013-04-26 Thread David Gibson
On Fri, Apr 26, 2013 at 10:23:40AM +0200, Paolo Bonzini wrote: > Il 26/04/2013 08:02, David Gibson ha scritto: > > At the moment, vfio maintains a global list of containers that are assumed > > to be more or less interchangeable, since they are all set up with a > > MemoryListener to have all of sy

Re: [Qemu-devel] [PATCH] virtio: abort on zero config length

2013-04-26 Thread Laszlo Ersek
On 04/26/13 12:32, Eric Blake wrote: > On 04/25/2013 11:06 PM, Jason Wang wrote: if (addr > (vdev->config_len - sizeof(val))) ^ quiz: spot a bug above if config_len is 0:) >>> Then we need to fix these bugs and allocate a CVE. virtio-rng has >>> shipped. This code i

Re: [Qemu-devel] [PATCH 2/2] block: move bdrv_dev_change_media_cb() to callers that really need it

2013-04-26 Thread Markus Armbruster
Luiz Capitulino writes: > On Thu, 25 Apr 2013 20:18:35 +0200 > Markus Armbruster wrote: > >> Luiz Capitulino writes: >> >> > Commit 9ca111544c64b5abed2e79cf52e19a8f227b347b moved the call to >> > bdrv_dev_change_media_cb() outside the media check in bdrv_close(), >> > this added a regression w

Re: [Qemu-devel] [sheepdog] [PATCH v4] sheepdog: fix loadvm operation

2013-04-26 Thread Stefan Hajnoczi
On Fri, Apr 26, 2013 at 05:04:06PM +0800, Liu Yuan wrote: > On 04/25/2013 09:06 PM, MORITA Kazutaka wrote: > > At Thu, 25 Apr 2013 20:49:39 +0800, > > Liu Yuan wrote: > >> > >> From: Liu Yuan > >> > >> Currently the 'loadvm' opertaion works as following: > >> 1. switch to the snapshot > >> 2. mark

Re: [Qemu-devel] [PATCH v4] sheepdog: fix loadvm operation

2013-04-26 Thread Stefan Hajnoczi
On Thu, Apr 25, 2013 at 08:49:39PM +0800, Liu Yuan wrote: > From: Liu Yuan > > Currently the 'loadvm' opertaion works as following: > 1. switch to the snapshot > 2. mark current working VDI as a snapshot > 3. rely on sd_create_branch to create a new working VDI based on the snapshot > > This wor

[Qemu-devel] [PULL 00/11] Block patches

2013-04-26 Thread Stefan Hajnoczi
The following changes since commit e3351000cd682200835763caca87adf708ed1c65: Makefile: Use QEMU_FLAGS for DTC compilation (2013-04-26 02:48:10 +0200) are available in the git repository at: git://github.com/stefanha/qemu.git block for you to fetch changes up to 859e5553a428225de6b8ef302cdcf

[Qemu-devel] [PATCH 01/11] block/ssh: Require libssh2 >= 1.2.8.

2013-04-26 Thread Stefan Hajnoczi
From: "Richard W.M. Jones" libssh2 >= 1.2.8 is required to enable this block device (because that version introduced the libssh2_session_handshake call). Change the test to use pkg-config exclusively. If the user requests --enable-libssh2 and the minimum version is not available, then the follo

[Qemu-devel] [PATCH 02/11] savevm: Implement block_writev_buffer()

2013-04-26 Thread Stefan Hajnoczi
From: Kevin Wolf Instead of breaking up RAM state into many small chunks, pass the iovec to the block layer for better performance. Signed-off-by: Kevin Wolf Signed-off-by: Stefan Hajnoczi --- include/migration/qemu-file.h | 2 +- savevm.c | 25 +

[Qemu-devel] [PATCH 01/11] block: Introduce bdrv_writev_vmstate

2013-04-26 Thread Stefan Hajnoczi
From: Kevin Wolf Signed-off-by: Kevin Wolf Signed-off-by: Stefan Hajnoczi --- block.c | 25 - block/qcow2.c | 12 +--- block/sheepdog.c | 13 ++--- include/block/block.h | 1 + include/block/block_int.h | 4 ++

[Qemu-devel] [PATCH 03/11] block: Introduce bdrv_pwritev() for qcow2_save_vmstate

2013-04-26 Thread Stefan Hajnoczi
From: Kevin Wolf Directly pass the QEMUIOVector on instead of linearising it. Signed-off-by: Kevin Wolf Signed-off-by: Stefan Hajnoczi --- block.c | 80 +-- block/qcow2.c | 8 +- include/block/block.h | 2 ++ 3 files

[Qemu-devel] [PATCH 07/11] ide: refuse WIN_READ_NATIVE_MAX on empty device

2013-04-26 Thread Stefan Hajnoczi
What is the highest addressable sector on an empty CD-ROM? Nothing is addressable so produce an error. This patch prevents a divide-by-zero in ide_set_sector() since s->sectors and s->heads would be 0. Not to mention that a sector=-1 argument would be nonsense. Note that WIN_READ_NATIVE_MAX can

[Qemu-devel] [PATCH 03/11] sheepdog: use BDRV_SECTOR_SIZE

2013-04-26 Thread Stefan Hajnoczi
From: Liu Yuan Cc: MORITA Kazutaka Cc: Kevin Wolf Cc: Stefan Hajnoczi Signed-off-by: Liu Yuan Signed-off-by: Stefan Hajnoczi --- block/sheepdog.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/block/sheepdog.c b/block/sheepdog.c index 6a72c1e..2772e8e 100644

[Qemu-devel] [PATCH 02/11] sheepdog: add discard/trim support for sheepdog

2013-04-26 Thread Stefan Hajnoczi
From: Liu Yuan The 'TRIM' command from VM that is to release underlying data storage for better thin-provision is already supported by the Sheepdog. This patch adds the TRIM support at QEMU part. For older Sheepdog that doesn't support it, we return 0(success) to upper layer. Cc: MORITA Kazuta

[Qemu-devel] [PATCH 05/11] qemu-iotests: Add test for -drive options

2013-04-26 Thread Stefan Hajnoczi
From: Kevin Wolf Signed-off-by: Kevin Wolf Signed-off-by: Stefan Hajnoczi --- tests/qemu-iotests/051 | 148 + tests/qemu-iotests/051.out | 162 + tests/qemu-iotests/group | 1 + 3 files changed, 311 ins

[Qemu-devel] [PATCH 04/11] qemu-iotests: A few more bdrv_pread/pwrite tests

2013-04-26 Thread Stefan Hajnoczi
From: Kevin Wolf Signed-off-by: Kevin Wolf Signed-off-by: Stefan Hajnoczi --- tests/qemu-iotests/002 | 13 + tests/qemu-iotests/002.out | 26 ++ 2 files changed, 39 insertions(+) diff --git a/tests/qemu-iotests/002 b/tests/qemu-iotests/002 index bebed84

[Qemu-devel] [PATCH 08/11] block: Add support for Secure Shell (ssh) block device.

2013-04-26 Thread Stefan Hajnoczi
From: "Richard W.M. Jones" qemu-system-x86_64 -drive file=ssh://hostname/some/image QEMU will ssh into 'hostname' and open '/some/image' which is made available as a standard block device. You can specify a username (ssh://user@host/...) and/or a port number (ssh://host:port/...). You can al

[Qemu-devel] [PATCH 07/11] sheepdog: cleanup find_vdi_name

2013-04-26 Thread Stefan Hajnoczi
From: MORITA Kazutaka This makes 'filename' and 'tag' constant variables, and renames 'for_snapshot' to 'lock' to clear how it works. Signed-off-by: MORITA Kazutaka Signed-off-by: Stefan Hajnoczi --- block/sheepdog.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff

[Qemu-devel] [PATCH 06/11] rbd: Fix use after free in rbd_open()

2013-04-26 Thread Stefan Hajnoczi
From: Kevin Wolf Commit a9ccedc3 frees the QemuOpts for the driver-specific options immediately, even though it still needs the filename string that is contained there. This doesn't work. Move the deletion of the QemuOpts to the end of the function where its content isn't needed any more. Signed

[Qemu-devel] [PATCH 11/11] sheepdog: fix loadvm operation

2013-04-26 Thread Stefan Hajnoczi
From: Liu Yuan Currently the 'loadvm' opertaion works as following: 1. switch to the snapshot 2. mark current working VDI as a snapshot 3. rely on sd_create_branch to create a new working VDI based on the snapshot This works not the same as other format as QCOW2. For e.g, qemu > savevm # get a

[Qemu-devel] [PATCH 09/11] sheepdog: add helper function to reload inode

2013-04-26 Thread Stefan Hajnoczi
From: MORITA Kazutaka This adds a helper function to update the current inode state with the specified vdi object. Signed-off-by: MORITA Kazutaka Signed-off-by: Stefan Hajnoczi --- block/sheepdog.c | 67 +--- 1 file changed, 39 insertions(+)

Re: [Qemu-devel] [PATCH v2] microblaze: Add internal base vectors reg

2013-04-26 Thread Andreas Färber
Hi, Am 24.04.2013 13:31, schrieb edgar.igles...@gmail.com: > From: "Edgar E. Iglesias" > > Configurable at CPU synthesis/instantiation. > > Signed-off-by: Edgar E. Iglesias > --- > > Peter Crosthwaite commented off-list that it would be a good idea > to make this a device property. I agree bu

[Qemu-devel] [PATCH 09/11] block: ssh: Use libssh2_sftp_fsync (if supported by libssh2) to flush to disk.

2013-04-26 Thread Stefan Hajnoczi
From: "Richard W.M. Jones" libssh2_sftp_fsync is an extension to libssh2 to support fsync(2) over sftp, which is itself an extension of OpenSSH. If both libssh2 and the ssh daemon support it, this will allow bdrv_flush_to_disk to commit changes through to disk on the remote server. Signed-off-b

Re: [Qemu-devel] [PATCH v21 5/8] pvpanic: pass configurable ioport to seabios

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > This lets seabios patch the corresponding SSDT entry. > > Also add fw_cfg object to /machine/fw_cfg so we can reference > it elsewhere. > > Signed-off-by: Hu Tao Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [sheepdog] [PATCH v4] sheepdog: fix loadvm operation

2013-04-26 Thread Liu Yuan
On 04/26/2013 07:39 PM, Stefan Hajnoczi wrote: > Please send patches as top-level email threads instead of replies so > that they get noticed. Okay, will do for future patches. Thanks for reminding. Yuan

Re: [Qemu-devel] [PATCH v21 3/8] introduce a new qom device to deal with panicked event

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > pvpanic device is used to send guest panic event from guest to qemu. > > When guest panic happens, pvpanic device driver will write a event > number to IO port 0x505(which is the IO port occupied by pvpanic device, > by default). On receiving the event, pvpanic device will pause

[Qemu-devel] [PATCH 10/11] sheepdog: resend write requests when SD_RES_READONLY is received

2013-04-26 Thread Stefan Hajnoczi
From: MORITA Kazutaka When a snapshot is taken from out side of qemu (e.g. qemu-img snapshot), write requests to the current vdi return SD_RES_READONLY. In this case, the sheepdog block driver needs to update the current inode to the latest one and resend the write requests. Signed-off-by: MORIT

Re: [Qemu-devel] [PATCH v21 4/8] fw_cfg: add required header files

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > If fw_cfg.h is included alone, gcc gives error messages like these: > > error: unknown type name ‘uint32_t’ > error: unknown type name ‘size_t’ > error: unknown type name ‘hwaddr’ > ... > > Signed-off-by: Hu Tao Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH v21 8/8] Wire up disabled wait a panicked event on s390

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > From: Christian Borntraeger > > On s390 the disabled wait state indicates a state of attention. > For example Linux uses that state after a panic. Lets > put the system into panicked state. > > An alternative implementation would be to state > disabled-wait instead of pause in

[Qemu-devel] [PATCH 11/11] rbd: add an asynchronous flush

2013-04-26 Thread Stefan Hajnoczi
From: Josh Durgin The existing bdrv_co_flush_to_disk implementation uses rbd_flush(), which is sychronous and causes the main qemu thread to block until it is complete. This results in unresponsiveness and extra latency for the guest. Fix this by using an asynchronous version of flush. This was

Re: [Qemu-devel] [PATCH v21 1/8] add a new runstate: RUN_STATE_GUEST_PANICKED

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > The guest will be in this state when it is panicked. > > Signed-off-by: Wen Congyang > Signed-off-by: Hu Tao Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH v21 2/8] add a new qevent: QEVENT_GUEST_PANICKED

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > This event will be emited when qemu detects guest panic. > > Signed-off-by: Wen Congyang > Signed-off-by: Hu Tao Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [PATCH v21 7/8] pvpanic: create pvpanic by default for machine 1.5

2013-04-26 Thread Markus Armbruster
Hu Tao writes: > Signed-off-by: Paolo Bonzini > Signed-off-by: Hu Tao Reviewed-by: Markus Armbruster

Re: [Qemu-devel] [Qemu-ppc] [PATCH] ppc: initialize GPRs as per epapr

2013-04-26 Thread David Gibson
On Fri, Apr 26, 2013 at 08:21:24AM +0200, Alexander Graf wrote: > > On 26.04.2013, at 08:17, Bharat Bhushan wrote: > > > ePAPR defines the initial values of cpu registers. This patch initialize > > the GPRs as per ePAPR specification. > > > > This resolves the issue of guest reboot/reset (guest

Re: [Qemu-devel] VFIO-VGA Issue

2013-04-26 Thread deniv
Alex Williamson: > On Thu, 2013-04-25 at 11:38 +, deniv wrote: >> Sorry for the long delay. >> >> Alex Williamson: >>> On Thu, 2013-04-11 at 13:59 -0400, de...@lavabit.com wrote: > On Wed, 2013-04-10 at 16:32 -0400, de...@lavabit.com wrote: However, turning gfx_passthru off did >>>

Re: [Qemu-devel] [PATCH v2] microblaze: Add internal base vectors reg

2013-04-26 Thread Edgar E. Iglesias
On Fri, Apr 26, 2013 at 01:48:45PM +0200, Andreas Färber wrote: > Hi, > > Am 24.04.2013 13:31, schrieb edgar.igles...@gmail.com: > > From: "Edgar E. Iglesias" > > > > Configurable at CPU synthesis/instantiation. > > > > Signed-off-by: Edgar E. Iglesias > > --- > > > > Peter Crosthwaite commen

[Qemu-devel] [PATCH 06/10] s390-ccw.img: Enhance drain_irqs().

2013-04-26 Thread Dominik Dingel
From: Cornelia Huck - Use tpi + tsch to get interrupts. - Return an error if the irb indicates problems. Signed-off-by: Cornelia Huck diff --git a/pc-bios/s390-ccw/virtio.c b/pc-bios/s390-ccw/virtio.c index 1968fc6..1bd17d4 100644 --- a/pc-bios/s390-ccw/virtio.c +++ b/pc-bios/s390-ccw/virtio.c

[Qemu-devel] [PATCH 03/10] S390: Check Bootdevice Type

2013-04-26 Thread Dominik Dingel
Check for a kernel IPL entry and load kernel image if one was specified. If no kernel image was supplied and the first boot device is not a virtio-ccw-blk device, print error message and exit. In case it is a virtio-ccw-blk device store the dev_no in reg[7] Signed-off-by: Christian Paro Signed-o

[Qemu-devel] [PATCH 08/10] s390-ccw.img: Get queue config from host.

2013-04-26 Thread Dominik Dingel
From: Cornelia Huck Ask the host about the configuration instead of guessing it. Signed-off-by: Cornelia Huck diff --git a/pc-bios/s390-ccw/virtio.c b/pc-bios/s390-ccw/virtio.c index e0cede5..5b9e1dc 100644 --- a/pc-bios/s390-ccw/virtio.c +++ b/pc-bios/s390-ccw/virtio.c @@ -257,17 +257,21 @@ i

[Qemu-devel] [PATCH 04/10] S390: check if BIOS is available and create links

2013-04-26 Thread Dominik Dingel
Signed-off-by: Dominik Dingel Check if the BIOS is available before loading it into the memory. Create the needed Links for build. Add a if the BIOS is available, also add the needed links for the build process. diff --git a/configure b/configure index 19777de..2bbbd54 100755 --- a/configure ++

[Qemu-devel] [PATCH 09/10] S390: Pass per-device loadparm values for CCW blk and net devs.

2013-04-26 Thread Dominik Dingel
From: Christian Paro Provide a loadparm property which can be used to pass IPL load parameters on a per-device basis for VirtioCcwDevice instances representing block and network devices. Signed-off-by: Christian Paro diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c index 56539d3..23d

  1   2   3   4   >