[Qemu-devel] [RFC 3/9] Virtio: Add transport bindings.

2012-04-24 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- Makefile.objs |1 + hw/virtio-transport.c | 46 ++ hw/virtio-transport.h | 39 +++ 3 files changed, 86 insertions(+), 0 deletions(-) create mode 100644 hw/virtio-tra

[Qemu-devel] [RFC 5/9] hw/virtio-serial-bus.c: Add virtio-serial device.

2012-04-24 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- hw/virtio-serial-bus.c | 42 ++ hw/virtio-serial.h |9 + 2 files changed, 51 insertions(+), 0 deletions(-) diff --git a/hw/virtio-serial-bus.c b/hw/virtio-serial-bus.c index e22940e..52a5382 100644 --- a

[Qemu-devel] [RFC 0/9] Virtio-mmio refactoring.

2012-04-24 Thread Evgeny Voevodin
This patchset is derived from patchset provided by Peter Maydell : http://lists.gnu.org/archive/html/qemu-devel/2011-11/msg01870.html Also this patchset includes some fixes for bugs spotted by Ying-Shiuan Pan . Still isue pointed by Peter presents here I think: * extra vring alignment field not

[Qemu-devel] [RFC 2/9] virtio: Support transports which can specify the vring alignment

2012-04-24 Thread Evgeny Voevodin
From: Peter Maydell Support virtio transports which can specify the vring alignment (ie where the guest communicates this to the host) by providing a new virtio_queue_set_align() function. (The default alignment remains as before.) FIXME save/load support for this new field! Signed-off-by: Pete

Re: [Qemu-devel] [PATCH 11/12] Migrate 64bit entries to 64bit pci regions

2012-04-24 Thread Alexey Korolev
On 25/04/12 13:48, Kevin O'Connor wrote: > On Tue, Apr 24, 2012 at 06:25:39PM +1200, Alexey Korolev wrote: >> Migrate 64bit entries to 64bit pci regions if they do >> not fit in 32bit range. > [...] >> +static void pci_region_migrate_64bit_entries(struct pci_region *from, >> +

[Qemu-devel] [RFC 6/9] hw/virtio-balloon.c: Add virtio-balloon device.

2012-04-24 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- hw/virtio-balloon.c | 34 ++ hw/virtio-balloon.h |7 +++ 2 files changed, 41 insertions(+), 0 deletions(-) diff --git a/hw/virtio-balloon.c b/hw/virtio-balloon.c index ce9d2c9..5640e58 100644 --- a/hw/virtio-balloon.c +

[Qemu-devel] [RFC 7/9] hw/virtio-net.c: Add virtio-net device.

2012-04-24 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- hw/virtio-net.c | 46 ++ hw/virtio-net.h | 11 +++ 2 files changed, 57 insertions(+), 0 deletions(-) diff --git a/hw/virtio-net.c b/hw/virtio-net.c index bc5e3a8..febf384 100644 --- a/hw/virtio-net.c +++ b

[Qemu-devel] [RFC 9/9] hw/exynos4210.c: Create two virtio-mmio transport instances.

2012-04-24 Thread Evgeny Voevodin
NB: This is for test purposes only. Signed-off-by: Evgeny Voevodin --- hw/exynos4210.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/hw/exynos4210.c b/hw/exynos4210.c index afc4bdc..e065a86 100644 --- a/hw/exynos4210.c +++ b/hw/exynos4210.c @@ -26,6 +26,7 @

[Qemu-devel] [RFC 4/9] Add MMIO based virtio transport

2012-04-24 Thread Evgeny Voevodin
From: Peter Maydell Add support for the generic MMIO based virtio transport. This patch is a modyfied patch of Peter Maydell . Changes are to have virtio-mmio bridge device which provides virtio-mmio bus. To this bus virtio-mmio-transport device is connect

[Qemu-devel] [RFC 8/9] hw/virtio-blk.c: Add virtio-blk device.

2012-04-24 Thread Evgeny Voevodin
Signed-off-by: Evgeny Voevodin --- hw/virtio-blk.c | 42 ++ hw/virtio-blk.h | 11 +++ 2 files changed, 53 insertions(+), 0 deletions(-) diff --git a/hw/virtio-blk.c b/hw/virtio-blk.c index 49990f8..b6c5ff5 100644 --- a/hw/virtio-blk.c +++ b/hw/

[Qemu-devel] [RFC 1/9] virtio: Add support for guest setting of queue size

2012-04-24 Thread Evgeny Voevodin
From: Peter Maydell The MMIO virtio transport spec allows the guest to tell the host how large the queue size is. Add virtio_queue_set_num() function which implements this in the QEMU common virtio support code. Signed-off-by: Peter Maydell Signed-off-by: Evgeny Voevodin --- hw/virtio.c |

Re: [Qemu-devel] [SeaBIOS] [PATCH 05/12] pciinit: Track region alignment explicitly.

2012-04-24 Thread Alexey Korolev
On 24/04/12 18:56, Gerd Hoffmann wrote: > On 04/24/12 08:17, Alexey Korolev wrote: >> Don't round up bridge regions to the next highest size - instead track >> alignment explicitly. This should improve the memory layout for >> bridge regions. > This one got mangled too: > > Applying: pciinit: Trac

Re: [Qemu-devel] [PATCH v9 2/4] sockets: change inet_connect() to support nonblock socket

2012-04-24 Thread Amos Kong
On 25/04/12 13:00, Orit Wasserman wrote: On 04/25/2012 06:32 AM, Amos Kong wrote: Add a bool argument to inet_connect() to assign if set socket to block/nonblock, and delete original argument 'socktype' that is unused. Add a new argument to inet_connect()/inet_connect_opts(), to pass back connec

[Qemu-devel] [PATCH v10 2/4] sockets: change inet_connect() to support nonblock socket

2012-04-24 Thread Amos Kong
Add a bool argument to inet_connect() to assign if set socket to block/nonblock, and delete original argument 'socktype' that is unused. Add a new argument to inet_connect()/inet_connect_opts(), to pass back connect error by error class. Retry to connect when -EINTR is got. Connect's successful fo

Re: [Qemu-devel] [PATCH 00/16] QEMU vhost-scsi support

2012-04-24 Thread ronnie sahlberg
On Tue, Apr 24, 2012 at 7:13 PM, Stefan Hajnoczi wrote: > On Tue, Apr 24, 2012 at 8:05 AM, Paolo Bonzini wrote: >> Il 24/04/2012 06:21, ronnie sahlberg ha scritto: >>> Hi Stefan, >>> >>> A little bit off-topic but >>> >>> When you design the proper place and API to plug virt-scsi into an >>> exte

Re: [Qemu-devel] [PATCH v9 2/4] sockets: change inet_connect() to support nonblock socket

2012-04-24 Thread Orit Wasserman
On 04/25/2012 06:32 AM, Amos Kong wrote: > Add a bool argument to inet_connect() to assign if set socket > to block/nonblock, and delete original argument 'socktype' > that is unused. > Add a new argument to inet_connect()/inet_connect_opts(), > to pass back connect error by error class. > > Retry

[Qemu-devel] [Bug 988128] [NEW] smbd crashes when called with "smb ports = 0"

2012-04-24 Thread Nikolaus Rath
Public bug reported: The smb.conf generated by qemu-kvm contains a "smb ports = 0" directive. This causes at least version 3.6.4 of Samba to crash with [0] vostro:/tmp/qemu-smb.6836-0# smbd -i -s smb.conf Unable to setup corepath for smbd: Operation not permitted smbd version 3.6.4 started. Copyr

[Qemu-devel] [Bug 988125] Re: Generated smb.conf needs to declare state directory

2012-04-24 Thread Nikolaus Rath
** Patch added: "Previous patch was broken" https://bugs.launchpad.net/qemu/+bug/988125/+attachment/3102896/+files/smb-patch.diff ** Patch removed: "smb-patch.diff" https://bugs.launchpad.net/qemu/+bug/988125/+attachment/3102872/+files/smb-patch.diff -- You received this bug notification

[Qemu-devel] [Bug 988125] [NEW] Generated smb.conf needs to declare state directory

2012-04-24 Thread Nikolaus Rath
Public bug reported: The smb.conf generated by the userspace networking does not include a state directory directive. Samba therefore falls back to the default value. Since the user generally does not have write access to this path, smbd immediately crashes. I have attached a patch that adds th

[Qemu-devel] [Bug 988125] Re: Generated smb.conf needs to declare state directory

2012-04-24 Thread Nikolaus Rath
** Patch added: "smb-patch.diff" https://bugs.launchpad.net/bugs/988125/+attachment/3102872/+files/smb-patch.diff -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/988125 Title: Generated smb.conf

[Qemu-devel] [PATCH v9 2/4] sockets: change inet_connect() to support nonblock socket

2012-04-24 Thread Amos Kong
Add a bool argument to inet_connect() to assign if set socket to block/nonblock, and delete original argument 'socktype' that is unused. Add a new argument to inet_connect()/inet_connect_opts(), to pass back connect error by error class. Retry to connect when -EINTR is got. Connect's successful fo

Re: [Qemu-devel] [PATCH v4] ARM: Exynos4210 IRQ: Introduce new IRQ gate functionality.

2012-04-24 Thread Evgeny Voevodin
On 24.04.2012 20:17, Peter Maydell wrote: On 24 April 2012 05:19, Evgeny Voevodin wrote: New IRQ gate consists of n_in input qdev gpio lines and one output sysbus IRQ line. The output IRQ level is formed as OR between all gpio inputs. Signed-off-by: Evgeny Voevodin Reviewed-by: Peter Maydell

Re: [Qemu-devel] [PATCH 11/12] Migrate 64bit entries to 64bit pci regions

2012-04-24 Thread Kevin O'Connor
On Tue, Apr 24, 2012 at 06:25:39PM +1200, Alexey Korolev wrote: > Migrate 64bit entries to 64bit pci regions if they do > not fit in 32bit range. [...] > +static void pci_region_migrate_64bit_entries(struct pci_region *from, > + struct pci_region *to) > +

Re: [Qemu-devel] [PATCH 10/12] Calculate pci region stats on demand

2012-04-24 Thread Kevin O'Connor
On Tue, Apr 24, 2012 at 06:24:27PM +1200, Alexey Korolev wrote: > Do not store pci region stats - instead calulate the > sum and alignment on demand. [...] > @@ -446,9 +459,9 @@ static int pci_bios_check_devices(struct pci_bus *busses) > for (type = 0; type < PCI_REGION_TYPE_COUNT; type++)

[Qemu-devel] How can I see the data for IDE DMA?

2012-04-24 Thread Sungchan Kim
Dear all, I'm trying to integrate our custom SSD simulator by modifying the ide part of qemu. To do this, beside a boot disk, I'm using an additional disk image in raw format that is mounted to /dev/sdb. I'm working with the ubuntu 11.04 image that is mounted as /dev/sda. I need to monitor

[Qemu-devel] [PATCH qemu 1/6] move code to read default config files to a separate function (v2)

2012-04-24 Thread Eduardo Habkost
Function added to arch_init.c because it depends on arch-specific settings. Changes v1 -> v2: - Move qemu_read_default_config_file() prototype to qemu-config.h Signed-off-by: Eduardo Habkost --- arch_init.c | 18 ++ qemu-config.h |4 vl.c | 10 ++

Re: [Qemu-devel] [PATCH v2] configure: Fix creation of symbolic links for MinGW toolchain

2012-04-24 Thread Stefan Weil
Am 19.03.2012 13:20, schrieb Stefan Weil: The MinGW toolchain on w32/w64 hosts does not create symbolic links, but implements 'ln -s' similar to 'cp -r'. In incremental out of tree builds, this resulted in files which were not updated when their counterparts in the QEMU source tree changed. Espe

[Qemu-devel] [PATCH qemu 6/6] move CPU definitions to /usr/share/qemu/cpus-x86_64.conf (v2)

2012-04-24 Thread Eduardo Habkost
Changes v1 -> v2: - userconfig variable is now bool, not int Signed-off-by: Eduardo Habkost --- Makefile | 12 +++- arch_init.c |1 + sysconfigs/target/cpus-x86_64.conf | 128 ++ sysconfigs/target/targ

[Qemu-devel] [PATCH qemu 5/6] implement -no-user-config command-line option (v2)

2012-04-24 Thread Eduardo Habkost
Changes v1 -> v2: - Change 'userconfig' field/variables to bool instead of int - Coding style change Signed-off-by: Eduardo Habkost --- arch_init.c | 11 --- qemu-config.h |2 +- qemu-options.hx | 16 +--- vl.c|6 +- 4 files changed, 27 ins

[Qemu-devel] [PATCH qemu 3/6] move list of default config files to an array

2012-04-24 Thread Eduardo Habkost
More files will be added to the list, with additional attributes, later. Signed-off-by: Eduardo Habkost --- arch_init.c | 25 - 1 files changed, 16 insertions(+), 9 deletions(-) diff --git a/arch_init.c b/arch_init.c index 152cbbb..62332e9 100644 --- a/arch_init.c +++

[Qemu-devel] [PATCH qemu 2/6] eliminate arch_config_name variable

2012-04-24 Thread Eduardo Habkost
Not needed anymore, as the code that uses the variable is already inside arch_init.c. Signed-off-by: Eduardo Habkost --- arch_init.c |3 +-- arch_init.h |2 -- 2 files changed, 1 insertions(+), 4 deletions(-) diff --git a/arch_init.c b/arch_init.c index 4008115..152cbbb 100644 --- a/arc

[Qemu-devel] [PATCH qemu v2 0/6] -no-user-config option, move CPU models to /usr/share

2012-04-24 Thread Eduardo Habkost
Changes v1 -> v2: - Move qemu_read_default_config_files() prototype to qemu-config.h - Make defconfig and userconfig variable bool - Coding style change Patches 1 to 4 just move some code around, patch 5 just adds the new option without adding any new config file. Patch 6 finally creates a /usr

[Qemu-devel] [PATCH qemu 4/6] vl.c: change 'defconfig' variable to bool

2012-04-24 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- vl.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/vl.c b/vl.c index 1e5e593..a4f4676 100644 --- a/vl.c +++ b/vl.c @@ -2279,7 +2279,7 @@ int main(int argc, char **argv, char **envp) #ifdef CONFIG_VNC int show_vnc_port = 0; #e

Re: [Qemu-devel] [PATCH] fdc: simplify media change handling

2012-04-24 Thread Pavel Hrdina
On 04/24/2012 09:15 PM, Hervé Poussineau wrote: This also (partly) fixes IBM OS/2 Warp 4.0 floppy installation, where not all floppies have the same format (2x80x18 for the first ones, 2x80x23 for the next ones). Signed-off-by: Hervé Poussineau Hi, it isn't fix the media missing issue. I booted

Re: [Qemu-devel] [PATCH v2 0/6] block: add optional 'speed' parameter to block-stream

2012-04-24 Thread Luiz Capitulino
On Tue, 24 Apr 2012 14:53:54 +0100 Stefan Hajnoczi wrote: > Eric Blake raised concerns about the inability to start block jobs with a > speed > limit. Current the user needs to follow up the block-stream command with > block-job-set-speed. There is a window of time while the new block job is >

Re: [Qemu-devel] [PATCH 3/4] block: add 'speed' optional parameter to block-stream

2012-04-24 Thread Luiz Capitulino
On Mon, 23 Apr 2012 16:39:48 +0100 Stefan Hajnoczi wrote: > Allow streaming operations to be started with an initial speed limit. > This eliminates the window of time between starting streaming and > issuing block-job-set-speed. Users should use the new optional 'speed' > parameter instead so th

Re: [Qemu-devel] [PATCH] Fix IDE FDC emulation for no media

2012-04-24 Thread Hervé Poussineau
Hi, Pavel Hrdina a écrit : On 04/24/2012 09:48 AM, Andreas Färber wrote: Am 23.04.2012 18:06, schrieb Pavel Hrdina: Hi, this is the patch to fix incorrect handling of IDE floppy drive controller emulation when no media is present. If the guest is booted without a media then the drive was not

[Qemu-devel] [PATCH] fdc: simplify media change handling

2012-04-24 Thread Hervé Poussineau
This also (partly) fixes IBM OS/2 Warp 4.0 floppy installation, where not all floppies have the same format (2x80x18 for the first ones, 2x80x23 for the next ones). Signed-off-by: Hervé Poussineau --- hw/fdc.c | 29 +++-- 1 file changed, 11 insertions(+), 18 deletions(-

Re: [Qemu-devel] [PATCH v2 4/6] block: add 'speed' optional parameter to block-stream

2012-04-24 Thread Luiz Capitulino
On Tue, 24 Apr 2012 14:53:58 +0100 Stefan Hajnoczi wrote: > Allow streaming operations to be started with an initial speed limit. > This eliminates the window of time between starting streaming and > issuing block-job-set-speed. Users should use the new optional 'speed' > parameter instead so th

[Qemu-devel] [PATCH V3 4/4] hw: add Atmel maxtouch touchscreen implementation

2012-04-24 Thread Igor Mitsyanko
And use it for exynos4210 NURI board emulation Signed-off-by: Igor Mitsyanko --- Makefile.objs |1 + default-configs/arm-softmmu.mak |1 + hw/exynos4_boards.c | 11 +- hw/maxtouch.c | 1114 +++ 4 files

Re: [Qemu-devel] [PATCH v2 3/6] block: change block-job-set-speed argument from 'value' to 'speed'

2012-04-24 Thread Luiz Capitulino
On Tue, 24 Apr 2012 09:03:06 -0600 Eric Blake wrote: > > +++ b/hmp-commands.hx > > @@ -85,8 +85,8 @@ ETEXI > > > > { > > .name = "block_job_set_speed", > > -.args_type = "device:B,value:o", > > -.params = "device value", > > +.args_type = "devic

[Qemu-devel] [PATCH V3 1/4] exynos4210: add Exynos4210 i2c implementation

2012-04-24 Thread Igor Mitsyanko
Create 9 exynos4210 i2c interfaces. Signed-off-by: Igor Mitsyanko Reviewed-by: Andreas Färber --- Makefile.target |1 + hw/exynos4210.c | 27 hw/exynos4210.h |3 + hw/exynos4210_i2c.c | 334 +++ 4 files changed, 365 ins

Re: [Qemu-devel] [PATCH v2 1/6] block: use Error mechanism instead of -errno for block_job_create()

2012-04-24 Thread Luiz Capitulino
On Tue, 24 Apr 2012 14:53:55 +0100 Stefan Hajnoczi wrote: > The block job API uses -errno return values internally and we convert > these to Error in the QMP functions. This is ugly because the Error > should be created at the point where we still have all the relevant > information. More impor

[Qemu-devel] [PATCH V3 2/4] vmstate.h: introduce VMSTATE_STRUCT_ARRAY_POINTER_TEST_V

2012-04-24 Thread Igor Mitsyanko
It could be used to save/load array of structs with statically known length pointed by device state member. Signed-off-by: Igor Mitsyanko --- vmstate.h | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/vmstate.h b/vmstate.h index 82d97ae..69e8ac1 100644 --- a

[Qemu-devel] [PATCH V3 0/4] Exynos: i2c, gpio and touchscreen support for NURI board

2012-04-24 Thread Igor Mitsyanko
This patchset adds Exynos4210 i2c and GPIO implementation along with Atmel MXT touchscreen which is used for NURI board emulation. v2->v3 - static TypeInfos are made const. - added spaces after "do {". - "All rights reserved" sentence is droped from license. - names are fixed acco

Re: [Qemu-devel] [PATCH v8 4/4] use inet_listen()/inet_connect() to support ipv6 migration

2012-04-24 Thread Michael Roth
On Fri, Apr 20, 2012 at 12:43:57PM +0800, Amos Kong wrote: > Use help functions in qemu-socket.c for tcp migration, > which already support ipv6 addresses. > > Currently errp will be set to UNDEFINED_ERROR when migration fails, > qemu would output "migration failed: ...", and current user can > se

[Qemu-devel] [RFC PATCH V2 2/2] exynos4210: introduce Exynos4210 SD host controller model

2012-04-24 Thread Igor Mitsyanko
Exynos4210 SD/MMC host controller is based on SD association standart host controller ver. 2.00 Signed-off-by: Igor Mitsyanko --- Makefile.target |1 + hw/exynos4210.c | 20 +++ hw/exynos4210_sdhci.c | 438 + 3 files changed, 459

[Qemu-devel] [RFC PATCH V2 0/2] Standard SD host controller model

2012-04-24 Thread Igor Mitsyanko
First patch introduces standard SD host controller model. This is accumulated version of my previous patch I sent a while ago and a recent SDHCI patch by Peter A. G. Crosthwaite. Second patch introduces Exynos4210-specific SDHCI built on top of standard SDHCI model. v1->v2 PATCH1: - add suppo

Re: [Qemu-devel] [PATCH v8 2/4] sockets: change inet_connect() to support nonblock socket

2012-04-24 Thread Michael Roth
On Fri, Apr 20, 2012 at 12:40:24PM +0800, Amos Kong wrote: > Add a bool argument to inet_connect() to assign if set socket > to block/nonblock, and delete original argument 'socktype' > that is unused. > Add a new argument to inet_connect()/inet_connect_opts(), > to pass back connect error by error

Re: [Qemu-devel] [PATCH] qtest: Fix tv_usec != long

2012-04-24 Thread Anthony Liguori
On 04/19/2012 07:09 AM, Richard Henderson wrote: Sparc Debian 5.0.8 does not define __suseconds_t as long, but FMT_timeval expects %ld. Signed-off-by: Richard Henderson Cc: Anthony Liguori Cc: Paolo Bonzini Applied. Thanks. Regards, Anthony Liguori --- qtest.c |8 1 files

Re: [Qemu-devel] [PATCH] configure: Virtfs doesn't require libcap.

2012-04-24 Thread Anthony Liguori
On 04/22/2012 05:16 AM, Kusanagi Kouichi wrote: Only proxy helper does. Signed-off-by: Kusanagi Kouichi Applied. Thanks. Regards, Anthony Liguori --- configure |6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 2d62d12..c8e6fe4 1007

Re: [Qemu-devel] [PATCH] qemu-char: Fix crash when switching consoles

2012-04-24 Thread Anthony Liguori
On 04/19/2012 03:27 PM, Stefan Weil wrote: qemu-system-arm (and other system emulations) crashes with SDL when the user switches consoles (Alt-Ctrl-F4). We already check for NULL pointers in qemu_chr_fe_ioctl, qemu_chr_be_can_write and other functions, so do this also for s->chr_read in qemu_chr

Re: [Qemu-devel] [PATCH] Limit ptimer rate to something achievable

2012-04-24 Thread Anthony Liguori
On 04/20/2012 12:32 AM, Peter Chubb wrote: If a guest sets very short timeouts, and asks for a timer to be reloaded on timeout, QEMU can go to 100%CPU utilisation and become unresponsive, as it is spending all its time generating timeout interrupts. On real hardware this doesn't matter, as the

Re: [Qemu-devel] [PATCH] qom: Refine container_get() to allow using a custom root

2012-04-24 Thread Anthony Liguori
On 04/05/2012 06:21 AM, Andreas Färber wrote: Specify the root to search from as argument. This avoids hardcoding "/machine" in some places and makes it more flexible. Signed-off-by: Andreas Färber Cc: Paolo Bonzini Cc: Anthony Liguori Applied. Thanks. Regards, Anthony Liguori --- hw/qd

Re: [Qemu-devel] [PATCH] Remove extra pthread switch

2012-04-24 Thread Anthony Liguori
On 04/20/2012 09:36 AM, Peter Portante wrote: From the Department of the Redundancy Department: remove the extra pthread switch which might be there from the package config check for gthreads. Signed-off-by: Peter Portante Applied. Thanks. Regards, Anthony Liguori --- configure |

Re: [Qemu-devel] [PATCH v5 00/14] configure: --with-confsuffix option

2012-04-24 Thread Anthony Liguori
On 04/18/2012 02:55 PM, Eduardo Habkost wrote: This is just a rebase of v4, because I noticed that v4 doesn't apply cleanly against current git master anymore (and I will soon submit a series that will depend on this one). Changes v4 -> v5: - Rebase against latest qemu.git Changes v3 -> v4:

[Qemu-devel] qemu-ga: conclusions on shutdown & suspend behavior

2012-04-24 Thread Luiz Capitulino
Hi there, With the risk of becoming repetitive, I'm going to summarize the problems and solutions we've discussed in the last few days for the problems found in qemu-ga's shutdown and suspend commands. Gleb and Igor, you may be interested in items 2 and 4. Basically, we have four issues: 1. T

[Qemu-devel] [PATCH] cocoa: Fix build

2012-04-24 Thread Bernhard Walle
This patch is from MacPorts [1]. I don't know the origin, but as it's quite trivial I hope it's okay to post it without that information. At least it fixes building QEMU on Mac OS 10.7. The compiler error without that patch: -- 8< --- /Users

[Qemu-devel] [Bug 818673] Re: virtio: trying to map MMIO memory

2012-04-24 Thread Vitalis
Hello with bad news! I have: virtio_ioport_write: unexpected address 0x13 value 0x1 on config: LC_ALL=C PATH=/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/sbin:/sbin:/bin QEMU_AUDIO_DRV=none /usr/bin/kvm -S -M pc-0.12 -cpu qemu32 -enable-kvm -m 3072 -smp 1 -name nata_xp -uuid da607499-1d8f-e7ef

[Qemu-devel] [PATCH] target-arm: Make SETEND respect bswap_code (BE8) setting

2012-04-24 Thread Peter Maydell
Make the SETEND instruction respect the setting of bswap_code, so that in BE8 mode we UNDEF for attempts to switch into little-endian mode and nop for attempts to stay in big-endian mode. (This is the inverse of the existing handling of SETEND in the more common little-endian setup, which we use si

Re: [Qemu-devel] [PULL 0/9] Tracing patches

2012-04-24 Thread Andreas Färber
Am 18.04.2012 16:07, schrieb Stefan Hajnoczi: > The last tracing pull for 1.1 before we merge only fixes. > > The following changes since commit 6e7a7f3d9bc2031b4c93c05400b18775ba1b1f55: > > Allow controlling volume with PulseAudio backend (2012-04-17 16:57:58 +0400) > > are available in the g

Re: [Qemu-devel] Spice vdagent on SLED 11

2012-04-24 Thread Andrew Cathrow
- Original Message - > From: "Raj Rajasekaran" > To: "Andrew Cathrow" > Cc: qemu-devel@nongnu.org > Sent: Tuesday, April 24, 2012 1:49:33 PM > Subject: Re: [Qemu-devel] Spice vdagent on SLED 11 > > > SLED Version 11. >From what I learned on #kvm you need a later version of SLED (or u

Re: [Qemu-devel] Spice vdagent on SLED 11

2012-04-24 Thread Raj Rajasekaran
SLED Version 11. On Tue, Apr 24, 2012 at 12:46 PM, Andrew Cathrow wrote: > > > - Original Message - > > From: "Raj Rajasekaran" > > To: "Andrew Cathrow" > > Cc: qemu-devel@nongnu.org > > Sent: Tuesday, April 24, 2012 1:38:59 PM > > Subject: Re: [Qemu-devel] Spice vdagent on SLED 11 > >

Re: [Qemu-devel] Spice vdagent on SLED 11

2012-04-24 Thread Andrew Cathrow
- Original Message - > From: "Raj Rajasekaran" > To: "Andrew Cathrow" > Cc: qemu-devel@nongnu.org > Sent: Tuesday, April 24, 2012 1:38:59 PM > Subject: Re: [Qemu-devel] Spice vdagent on SLED 11 > > > lspci listed 'Communication Controller Virtio console' as one of the > devices expose

Re: [Qemu-devel] [PATCH v2 09/15] target-i386: Add property getter for CPU model

2012-04-24 Thread Michael Roth
On Tue, Apr 24, 2012 at 06:50:28PM +0200, Andreas Färber wrote: > Am 24.04.2012 18:36, schrieb Michael Roth: > > On Tue, Apr 24, 2012 at 11:33:35AM +0200, Andreas Färber wrote: > >> Signed-off-by: Andreas Färber > >> --- > >> target-i386/cpu.c | 14 +- > >> 1 files changed, 13 inser

Re: [Qemu-devel] Spice vdagent on SLED 11

2012-04-24 Thread Raj Rajasekaran
lspci listed 'Communication Controller Virtio console' as one of the devices exposed. But I did not see any /dev/vport*. On Tue, Apr 24, 2012 at 12:17 PM, Andrew Cathrow wrote: > > > - Original Message - > > From: "Raj Rajasekaran" > > To: "Andrew Cathrow" > > Cc: qemu-devel@nongnu.org

Re: [Qemu-devel] Handling of setend instruction for ARM

2012-04-24 Thread Peter Maydell
On 24 April 2012 18:04, Rajat Goyal wrote: > The TCG translation for the ARM setend instruction for user level binaries > is empty. A binary which just prints "hello world" but inverts the > endianness to big endian at the beginning of main gives a segmentation fault > on native ARM but prints out

Re: [Qemu-devel] [PATCH 2/2] Expose tsc deadline timer cpuid to guest

2012-04-24 Thread Eduardo Habkost
(CCing Andre Przywara, in case he can help to clarify what's the expected meaning of "-cpu host") On Tue, Apr 24, 2012 at 06:06:55PM +0200, Jan Kiszka wrote: > On 2012-04-23 22:02, Eduardo Habkost wrote: > > On Mon, Apr 23, 2012 at 06:31:25PM +0200, Jan Kiszka wrote: > >> However, that was how I i

Re: [Qemu-devel] Spice vdagent on SLED 11

2012-04-24 Thread Andrew Cathrow
- Original Message - > From: "Raj Rajasekaran" > To: "Andrew Cathrow" > Cc: qemu-devel@nongnu.org > Sent: Tuesday, April 24, 2012 12:09:32 PM > Subject: Re: [Qemu-devel] Spice vdagent on SLED 11 > > > Here it is > > > > qemu -vga qxl -device qxl -global qxl-vga.vram_size=33554432 -

Re: [Qemu-devel] [PATCH v3 3/4] xilinx_spi: initial version

2012-04-24 Thread Peter Maydell
On 20 April 2012 03:12, Peter A. G. Crosthwaite wrote: > device model for xilinx XPS SPI controller (v2.0) > > Signed-off-by: Peter A. G. Crosthwaite > --- > changed from v2: > converted spi api to ssi api > changed from v1: > converted spi api to modified txrx style > >  Makefile.target |    1 +

Re: [Qemu-devel] [PATCH v3 2/4] m25p80: initial verion

2012-04-24 Thread Peter Maydell
On 20 April 2012 03:12, Peter A. G. Crosthwaite wrote: > Subject: [PATCH v3 2/4] m25p80: initial verion > Added device model for m25p80 SPI flash This commit message could be improved; I'd suggest a summary line of "m25p80: Initial implementation of SPI flash device" > > Signed-off-by: Peter A

[Qemu-devel] Handling of setend instruction for ARM

2012-04-24 Thread Rajat Goyal
The TCG translation for the ARM setend instruction for user level binaries is empty. A binary which just prints "hello world" but inverts the endianness to big endian at the beginning of main gives a segmentation fault on native ARM but prints out hello world on qemu-arm. So does QEMU continue und

[Qemu-devel] [PATCH 13/14] usb-uhci: update irq line on reset

2012-04-24 Thread Gerd Hoffmann
uhci_reset() clears irq mask and irq status registers, but doesn't update the irq line. Which may result in suspious IRQs after uhci reset. Fix it. Signed-off-by: Gerd Hoffmann --- hw/usb/hcd-uhci.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/hw/usb/hcd-uhci.c b/hw

Re: [Qemu-devel] [PATCH 1/3] qemu-iotests: Many parallel allocating I/O requests

2012-04-24 Thread Kevin Wolf
Am 24.04.2012 17:19, schrieb Eric Blake: > On 04/24/2012 09:16 AM, Kevin Wolf wrote: >> Am 24.04.2012 16:38, schrieb Kevin Wolf: >>> This test case manages to let qcow2 abort because its cache is used up >>> and it can't find free cache entries for new requests any more. >>> >>> Signed-off-by: Kevi

[Qemu-devel] [PATCHv3 3/3] virtio: order index/descriptor reads

2012-04-24 Thread Michael S. Tsirkin
virtio has the equivalent of: if (vq->last_avail_index != vring_avail_idx(vq)) { read descriptor head at vq->last_avail_index; } In theory, processor can reorder descriptor head read to happen speculatively before the index read. this would trigger the following ra

Re: [Qemu-devel] [PATCH v2 09/15] target-i386: Add property getter for CPU model

2012-04-24 Thread Andreas Färber
Am 24.04.2012 18:36, schrieb Michael Roth: > On Tue, Apr 24, 2012 at 11:33:35AM +0200, Andreas Färber wrote: >> Signed-off-by: Andreas Färber >> --- >> target-i386/cpu.c | 14 +- >> 1 files changed, 13 insertions(+), 1 deletions(-) >> >> diff --git a/target-i386/cpu.c b/target-i386/

[Qemu-devel] [PATCH] target-ppc: Some support for dumping TLB_EMB TLBs

2012-04-24 Thread François Revol
Add mmubooke_dump_mmu(). TODO: Add printing of individual flags. Signed-off-by: François Revol --- target-ppc/helper.c | 49 + 1 file changed, 49 insertions(+) diff --git a/target-ppc/helper.c b/target-ppc/helper.c index c610ce3..c998efc 100644

[Qemu-devel] [PATCH 12/14] usb: add serial number generator

2012-04-24 Thread Gerd Hoffmann
This patch adds a function which creates unique serial numbers for usb devices and puts it into use. Windows guests tend to become unhappy if they find two identical usb devices in the system. Effects range from non-functional devices (with yellow exclamation mark in device manager) to BSODs. Ha

[Qemu-devel] [PATCH 05/14] usb-host: don't dereference invalid iovecs

2012-04-24 Thread Gerd Hoffmann
usb-host assumes the first iovec element is always valid. In case of a zero-length packet this isn't true though. Signed-off-by: Gerd Hoffmann --- hw/usb/host-linux.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/usb/host-linux.c b/hw/usb/host-linux.c index 061

Re: [Qemu-devel] [PATCH v3 1/4] SSI: Built in multiple device support

2012-04-24 Thread Peter Maydell
On 20 April 2012 03:12, Peter A. G. Crosthwaite wrote: > Added support for multiple devices attached to a single SSI bus (Previously > SSI masters with multiple slaves were emulated as multiple point to point SSI > busses) >  static struct BusInfo ssi_bus_info = { >     .name = "SSI", >     .size

Re: [Qemu-devel] [PATCH v2 09/15] target-i386: Add property getter for CPU model

2012-04-24 Thread Michael Roth
On Tue, Apr 24, 2012 at 11:33:35AM +0200, Andreas Färber wrote: > Signed-off-by: Andreas Färber > --- > target-i386/cpu.c | 14 +- > 1 files changed, 13 insertions(+), 1 deletions(-) > > diff --git a/target-i386/cpu.c b/target-i386/cpu.c > index 9479717..643289f 100644 > --- a/targ

Re: [Qemu-devel] [PATCH] Add mmubooke_dump_mmu

2012-04-24 Thread François Revol
On 24/04/2012 18:22, François Revol wrote: > The following patch adds some support for dumping the TLBs of type > TLB_EMB, at least enough to see the mappings. > I wasn't sure how to deal with the flags anyway, it seems to me the > struct lacks some stuff needed for system emulation, so it will pro

[Qemu-devel] [PATCH 08/14] usb-xhci: fix bit test

2012-04-24 Thread Gerd Hoffmann
From: Lai Jiangshan use & instead of the wrong && Signed-off-by: Lai Jiangshan Signed-off-by: Gerd Hoffmann --- hw/usb/hcd-xhci.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xhci.c index fd13bba..5cf1a64 100644 --- a/hw/usb/hcd-xhci

[Qemu-devel] [PATCH 10/14] usb-redir: Reset device address and speed on disconnect

2012-04-24 Thread Gerd Hoffmann
From: Hans de Goede Without this disconnected devices look like the last redirected device in the monitor in "info usb". Signed-off-by: Hans de Goede Signed-off-by: Gerd Hoffmann --- hw/usb/redirect.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/hw/usb/redirect.c

Re: [Qemu-devel] [PATCHv4 3/3] virtio: order index/descriptor reads

2012-04-24 Thread Paolo Bonzini
Il 24/04/2012 18:21, Michael S. Tsirkin ha scritto: > > +#define smp_rmb() smp_mb() > + > #elif defined(__x86_64__) > This #define seems spurious, but I may be reading the patch wrong. Paolo

[Qemu-devel] [PATCH 11/14] usb-redir: Not finding an async urb id is not an error

2012-04-24 Thread Gerd Hoffmann
From: Hans de Goede We clear our pending async urb list on device disconnect and we may still receive "packet complete" packets from our peer after this, which will then refer to packet ids no longer in our list. Signed-off-by: Hans de Goede Signed-off-by: Gerd Hoffmann --- hw/usb/redirect.c

[Qemu-devel] [PATCH] Add mmubooke_dump_mmu

2012-04-24 Thread François Revol
The following patch adds some support for dumping the TLBs of type TLB_EMB, at least enough to see the mappings. I wasn't sure how to deal with the flags anyway, it seems to me the struct lacks some stuff needed for system emulation, so it will probably need some revamping for the new target I'm ad

[Qemu-devel] [PATCHv4 3/3] virtio: order index/descriptor reads

2012-04-24 Thread Michael S. Tsirkin
virtio has the equivalent of: if (vq->last_avail_index != vring_avail_idx(vq)) { read descriptor head at vq->last_avail_index; } In theory, processor can reorder descriptor head read to happen speculatively before the index read. this would trigger the following ra

[Qemu-devel] [PATCHv4 2/3] virtio: add missing mb() on enable notification

2012-04-24 Thread Michael S. Tsirkin
This fixes an issue dual to the one fixed by patch 'virtio: add missing mb() on notification' and applies on top. In this case, to enable vq kick to exit to host, qemu writes out used flag then reads the avail index. if these are reordered we get a race: host avail index read: ring is empty

[Qemu-devel] [PATCHv4 1/3] virtio: add missing mb() on notification

2012-04-24 Thread Michael S. Tsirkin
During normal operation, virtio first writes a used index and then checks whether it should interrupt the guest by reading guest avail index/flag values. Guest does the reverse: writes the index/flag, then checks the used ring. The ordering is important: if host avail flag read bypasses the used

[Qemu-devel] [PATCHv4 0/3] virtio: fix memory access races

2012-04-24 Thread Michael S. Tsirkin
This is a follow-up to my previous patch: it turns out that a single mb() isn't sufficient as network loss could still be triggered under stress. Changes from v3: Make smp_rmb() with a compiler barrier on x86: reported by paolo. Changes from v2: Fix up GCC detection: reported by malc. Fix barrier

[Qemu-devel] [PATCH 14/14] usb-host: add timeout handler

2012-04-24 Thread Gerd Hoffmann
Add a timeout handler. In case bulk transfers take too long to finish the request will be canceled. The timeout is tunable via property, by default it is 5 seconds. Signed-off-by: Gerd Hoffmann --- hw/usb/host-linux.c | 27 +++ trace-events|1 + 2 files ch

Re: [Qemu-devel] [PATCH v4] ARM: Exynos4210 IRQ: Introduce new IRQ gate functionality.

2012-04-24 Thread Peter Maydell
On 24 April 2012 05:19, Evgeny Voevodin wrote: > New IRQ gate consists of n_in input qdev gpio lines and one > output sysbus IRQ line. The output IRQ level is formed as OR > between all gpio inputs. > > Signed-off-by: Evgeny Voevodin Reviewed-by: Peter Maydell Not convinced it's worth putting

Re: [Qemu-devel] [PATCHv2 3/3] virtio: order index/descriptor reads

2012-04-24 Thread Paolo Bonzini
Il 24/04/2012 18:08, Michael S. Tsirkin ha scritto: > On Tue, Apr 24, 2012 at 05:40:07PM +0200, Paolo Bonzini wrote: >> I would be grateful if, instead of fixing the qemu-barrier.h parts of >> the patches, you picked up the (sole) patch in the atomics branch of >> git://github.com/bonzi

[Qemu-devel] [PATCH 02/14] usb-ehci: add missing usb_packet_init() call

2012-04-24 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- hw/usb/core.c |1 + hw/usb/hcd-ehci.c |1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/hw/usb/core.c b/hw/usb/core.c index 9a14a53..0e02da7 100644 --- a/hw/usb/core.c +++ b/hw/usb/core.c @@ -501,6 +501,7 @@ void usb_packet_set_state

[Qemu-devel] [PATCH 06/14] usb-host: fix zero-length packets

2012-04-24 Thread Gerd Hoffmann
usb-host optimizes away zero-length packets by not entering the processing loop at all. Which isn't correct, we should submit a zero-length urb to the host devicein that case. This patch makes sure we run the processing loop at least once. Signed-off-by: Gerd Hoffmann --- hw/usb/host-linux.c |

[Qemu-devel] [PATCH 03/14] usb-ehci: Ensure frindex writes leave a valid frindex value

2012-04-24 Thread Gerd Hoffmann
From: Hans de Goede frindex is a 14 bits counter, so bits 31-14 should always be 0, and after the commit titled "usb-ehci: frindex always is a 14 bits counter" we rely on frindex always being a multiple of 8. I've not seen this in practice, but theoretically a guest can write a value >= 0x4000 or

Re: [Qemu-devel] Spice vdagent on SLED 11

2012-04-24 Thread Raj Rajasekaran
Here it is qemu -vga qxl -device qxl -global qxl-vga.vram_size=33554432 -device virtio-serial -chardev spicevmc,id=vdagent,debug=0,name=vdagent -device virtserialport,chardev=vdagent,name=com.redhat.spice.0 -spice port=$SPICE_PORT,image-compression=off,disable-ticketing -enable-kvm -m 1024 -net ni

Re: [Qemu-devel] [PATCHv2 3/3] virtio: order index/descriptor reads

2012-04-24 Thread Michael S. Tsirkin
On Tue, Apr 24, 2012 at 05:40:07PM +0200, Paolo Bonzini wrote: > I would be grateful if, instead of fixing the qemu-barrier.h parts of > the patches, you picked up the (sole) patch in the atomics branch of > git://github.com/bonzini/qemu.git. The constructs there are more > com

Re: [Qemu-devel] [PATCH 0/2] QOM'ify ARM CPU init

2012-04-24 Thread Peter Maydell
On 20 April 2012 18:39, Andreas Färber wrote: > This mini-series redoes your drop-reset-model-init patch 14/14 v2. > I've investigated doing it this way for sh4 and plan to adjust all targets. Both patches Reviewed-by: Peter Maydell and I've put them into target-arm.next; I'll do a pullreq at t

  1   2   3   >