Re: [Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-27 Thread Jan Kiszka
Frank Mehnert wrote: > On Wednesday 26 May 2010, Stefano Stabellini wrote: >> I think the patch is correct. > > serial: fixed bug which prevented the use of the xmit fifo > > Signed-off-by: Frank Mehnert > --- > hw/serial.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff

[Qemu-devel] [Bug 586221] [NEW] Linux on ARM/Mainstone machine fails at bootstrap

2010-05-27 Thread Daniel Rossier
Public bug reported: When QEMU (0.12.4) starts a ARM machine which boots a Linux, it immediately fails because the initial PC is wrong (is equal to 0 instead of 0xa000). First investigations indicate that the reset handlers queue does not return the correct opaque structure in main_cpu_reset()

[Qemu-devel] [PATCH] qemu: virtio-9p: Recognize 9P2000.L protocol

2010-05-27 Thread M. Mohan Kumar
Make 9P server recognize 9P2000.L protocol version Signed-off-by: M. Mohan Kumar --- hw/virtio-9p.c |6 +- hw/virtio-9p.h |6 ++ 2 files changed, 11 insertions(+), 1 deletions(-) diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c index e5d0112..25bd723 100644 --- a/hw/virtio-9p.c +++

[Qemu-devel] [Bug 586221] Re: Linux on ARM/Mainstone machine fails at bootstrap

2010-05-27 Thread Daniel Rossier
Workaround: just comment the last line "qemu_register_reset(main_cpu_reset, env);" of the function "arm_load_kernel()" in hw/arm_boot.c -- Linux on ARM/Mainstone machine fails at bootstrap https://bugs.launchpad.net/bugs/586221 You received this bug notification because you are a member of qemu-

[Qemu-devel] [PATCH] qemu: virtio-9p: Implement statfs support in qemu server

2010-05-27 Thread M. Mohan Kumar
Implement statfs support in qemu 9p server. statfs gets file system statistics such as type of file system, optimal transfer block size etc. statfs - get file system statistics size[4] Tstatfs tag[2] fid[4] size[4] Rstatfs tag[2] type[4] bsize[4] blocks[8] bfree[8] bavail[8]

[Qemu-devel] [PATCH] virtio-9p: readdir implementation for 9p2000.L

2010-05-27 Thread Sripathi Kodi
This patch implements the server part of readdir() implementation for 9p2000.L SYNOPSIS size[4] Treaddir tag[2] fid[4] offset[8] count[4] size[4] Rreaddir tag[2] count[4] data[count] DESCRIPTION The readdir request asks the server to read the directory specified by 'fid' at an offset specified

[Qemu-devel] [PATCH] qemu: Enable XSAVE related CPUID

2010-05-27 Thread Sheng Yang
We can support it in KVM now. The initial values are the minimal requirement of XSAVE capable processor. Signed-off-by: Sheng Yang --- target-i386/cpuid.c | 32 1 files changed, 32 insertions(+), 0 deletions(-) diff --git a/target-i386/cpuid.c b/target-i386/cp

[Qemu-devel] [PATCH] qemu: kvm: Enable XSAVE live migration support

2010-05-27 Thread Sheng Yang
Signed-off-by: Sheng Yang --- qemu-kvm-x86.c| 77 + qemu-kvm.c| 12 +++ qemu-kvm.h| 14 + target-i386/cpu.h |5 +++ target-i386/machine.c | 20 + 5 files changed, 109 insertion

[Qemu-devel] [PATCH] vhost_net.c: Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 The format statement expects unsigned long on x86_64, but receives unsigned long long, so gcc exits with an error. Signed-off-by: Jes Sorensen --- hw/vhost_net.c |2 +- 1 files changed, 1 insertion

Re: [Qemu-devel] [PATCH] vhost_net.c: Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes Sorensen
On 05/27/10 12:08, jes.soren...@redhat.com wrote: > From: Jes Sorensen > > Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 > > The format statement expects unsigned long on x86_64, but receives > unsigned long long, so gcc exits with an error. > > Signed-off-by: Jes Sor

[Qemu-devel] [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 The format statement expects unsigned long on x86_64, but receives unsigned long long, so gcc exits with an error. Signed-off-by: Jes Sorensen --- hw/vhost_net.c |2 +- 1 files changed, 1 insertion

[Qemu-devel] Re: [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 12:22:29PM +0200, jes.soren...@redhat.com wrote: > From: Jes Sorensen > > Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 > > The format statement expects unsigned long on x86_64, but receives > unsigned long long, so gcc exits with an error. > >

[Qemu-devel] Re: [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes Sorensen
On 05/27/10 12:44, Michael S. Tsirkin wrote: > On Thu, May 27, 2010 at 12:22:29PM +0200, jes.soren...@redhat.com wrote: >> From: Jes Sorensen >> >> Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 >> >> The format statement expects unsigned long on x86_64, but receives >> u

[Qemu-devel] Re: [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 12:55:49PM +0200, Jes Sorensen wrote: > On 05/27/10 12:44, Michael S. Tsirkin wrote: > > On Thu, May 27, 2010 at 12:22:29PM +0200, jes.soren...@redhat.com wrote: > >> From: Jes Sorensen > >> > >> Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 > >>

[Qemu-devel] Re: [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 12:55:49PM +0200, Jes Sorensen wrote: > On 05/27/10 12:44, Michael S. Tsirkin wrote: > > On Thu, May 27, 2010 at 12:22:29PM +0200, jes.soren...@redhat.com wrote: > >> From: Jes Sorensen > >> > >> Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 > >>

[Qemu-devel] Re: [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes Sorensen
On 05/27/10 12:53, Michael S. Tsirkin wrote: > On Thu, May 27, 2010 at 12:55:49PM +0200, Jes Sorensen wrote: >> On 05/27/10 12:44, Michael S. Tsirkin wrote: >>> I think this part of 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 >>> should just be reverted. We have unsigned long, it should be printed >>>

[Qemu-devel] Re: [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes Sorensen
On 05/27/10 12:54, Michael S. Tsirkin wrote: > On Thu, May 27, 2010 at 12:55:49PM +0200, Jes Sorensen wrote: >> On 05/27/10 12:44, Michael S. Tsirkin wrote: >>> On Thu, May 27, 2010 at 12:22:29PM +0200, jes.soren...@redhat.com wrote: From: Jes Sorensen Fix build failure introduced b

[Qemu-devel] [PATCH] posix-aio-compat: Expand tabs that have crept in

2010-05-27 Thread Stefan Hajnoczi
This patch expands tabs on a few lines so the code formats nicely and follows the QEMU coding style. Signed-off-by: Stefan Hajnoczi --- posix-aio-compat.c | 58 ++-- 1 files changed, 29 insertions(+), 29 deletions(-) diff --git a/posix-aio-compa

[Qemu-devel] Re: [PATCHv2-RFC 0/2] virtio: put last seen used index into ring itself

2010-05-27 Thread Avi Kivity
On 05/26/2010 10:50 PM, Michael S. Tsirkin wrote: Here's a rewrite of the original patch with a new layout. I haven't tested it yet so no idea how this performs, but I think this addresses the cache bounce issue raised by Avi. Posting for early flames/comments. Generally, the Host end of the vir

[Qemu-devel] [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 The format statement expects unsigned long on x86_64, but receives unsigned long long, so gcc exits with an error. Signed-off-by: Jes Sorensen --- hw/vhost_net.c |2 +- 1 files changed, 1 insertion

Re: [Qemu-devel] [PATCH] posix-aio-compat: Expand tabs that have crept in

2010-05-27 Thread Kevin Wolf
Am 27.05.2010 13:52, schrieb Stefan Hajnoczi: > This patch expands tabs on a few lines so the code formats nicely and > follows the QEMU coding style. > > Signed-off-by: Stefan Hajnoczi Thanks, applied to the block branch. I'm usually not a big supporter of patches that only change style, but t

[Qemu-devel] Re: [PATCH] qemu: Enable XSAVE related CPUID

2010-05-27 Thread Avi Kivity
On 05/27/2010 12:50 PM, Sheng Yang wrote: We can support it in KVM now. The initial values are the minimal requirement of XSAVE capable processor. Signed-off-by: Sheng Yang --- target-i386/cpuid.c | 32 1 files changed, 32 insertions(+), 0 deletions(-) diff

Re: [Qemu-devel] [PATCH] posix-aio-compat: Expand tabs that have crept in

2010-05-27 Thread Stefan Hajnoczi
On Thu, May 27, 2010 at 02:50:54PM +0200, Kevin Wolf wrote: > I'm usually not a big supporter of patches that only change style, but > tabs are nasty and you can ignore these whitespace changes in git blame > at least, so for all I care let's take it for once. Interesting, I tend to prefer separat

Re: [Qemu-devel] [PATCH] posix-aio-compat: Expand tabs that have crept in

2010-05-27 Thread Kevin Wolf
Am 27.05.2010 15:12, schrieb Stefan Hajnoczi: > On Thu, May 27, 2010 at 02:50:54PM +0200, Kevin Wolf wrote: >> I'm usually not a big supporter of patches that only change style, but >> tabs are nasty and you can ignore these whitespace changes in git blame >> at least, so for all I care let's take

[Qemu-devel] [PATCH] block.h: Make BDRV_SECTOR_SIZE 64 bit safe

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen C defaults to int, so make definition of BDRV_SECTOR_SIZE 64 bit safe as it and BDRV_SECTOR_MASK may be used against 64 bit addresses. Signed-off-by: Jes Sorensen --- block.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/block.h b/block.h index 24

[Qemu-devel] Re: [PATCH 3/5] QMP: Introduce MIGRATION events

2010-05-27 Thread Luiz Capitulino
On Wed, 26 May 2010 11:55:31 -0500 Anthony Liguori wrote: > On 05/26/2010 10:15 AM, Daniel P. Berrange wrote: > > On Wed, May 26, 2010 at 09:54:22AM -0500, Anthony Liguori wrote: > > > >> On 05/26/2010 05:33 AM, Daniel P. Berrange wrote: > >> > > I'm not sure why you would need a no

[Qemu-devel] Re: [PATCH] qemu: Enable XSAVE related CPUID

2010-05-27 Thread Sheng Yang
On Thursday 27 May 2010 20:56:17 Avi Kivity wrote: > On 05/27/2010 12:50 PM, Sheng Yang wrote: > > We can support it in KVM now. The initial values are the minimal > > requirement of XSAVE capable processor. > > > > Signed-off-by: Sheng Yang > > --- > > > > target-i386/cpuid.c | 32 ++

[Qemu-devel] Re: [PATCH] pci: fix pci_default_read_config().

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 02:44:42PM +0900, Isaku Yamahata wrote: > address and config_size are both unsigned. > So check which is bigger before minus operation. > Otherwise the result of minus can be unexpected > big value. > > Signed-off-by: Isaku Yamahata An this happen in practice? If yes, how

[Qemu-devel] Re: [PATCH] pci: clean up of pci_set_default_subsystem_id().

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 02:42:06PM +0900, Isaku Yamahata wrote: > Use pci accessor function. > don't return value because it always return 0 and > the caller doesn't check the return value. > > Signed-off-by: Isaku Yamahata Applied, thanks! > --- > hw/pci.c | 12 +--- > 1 files chang

[Qemu-devel] Re: [PATCHv2-RFC 0/2] virtio: put last seen used index into ring itself

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 03:07:52PM +0300, Avi Kivity wrote: > I missed the spec patch, can you repost it? Still work in progress, but here it is. Note I am still debating with myself whether we should split avail idx and flags into separate cache lines. diff --git a/virtio-spec.lyx b/virtio-spec.

[Qemu-devel] [Bug 586175] Re: Windows XP/2003 doesn't boot

2010-05-27 Thread Cole Robinson
There's a fedora rawhide bug about this as well: https://bugzilla.redhat.com/show_bug.cgi?id=579348 Which points to a qemu-devel posting talking about disk geometry confusion: http://article.gmane.org/gmane.comp.emulators.qemu/66135 ** Bug watch added: Red Hat Bugzilla #579348 https://bugzil

Re: [Qemu-devel] [PATCH] posix-aio-compat: Expand tabs that have crept in

2010-05-27 Thread Stefan Hajnoczi
On Thu, May 27, 2010 at 2:46 PM, Kevin Wolf wrote: > I don't think it's written down explicitly, but I remember some > discussion about style fixes which basically resulted in "do new code > right, leave old code alone". Okay, fair enough. Stefan

[Qemu-devel] Re: [PATCH] msix: remove duplicated defines.

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 02:40:37PM +0900, Isaku Yamahata wrote: > remove defines which are already defined in pci_regs.h > > Signed-off-by: Isaku Yamahata Applied, thanks! > --- > hw/msix.c |8 > 1 files changed, 0 insertions(+), 8 deletions(-) > > diff --git a/hw/msix.c b/hw/msi

[Qemu-devel] [PATCH 0/4] Cleanups for block code

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Hi, Reading through some of the blk code, I noticed a lot of cases where we mix and match between hard-coded values for the block size of 512 and using BDRV_SECTOR_SIZE. Trying to clean it up a bit and change the 512 constants to BDRV_SECTOR_SIZE as it is more explaning when r

[Qemu-devel] Re: [PATCH] pci-hotplug: make them aware of pci domain.

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 02:47:42PM +0900, Isaku Yamahata wrote: > add helper function which converts root bus to pci domain. > make functions which ignored domain aware of pci domain. > > Signed-off-by: Isaku Yamahata > --- > hw/pci-hotplug.c |7 --- > hw/pci.c | 24 +++

[Qemu-devel] [PATCH 2/4] Cleanup: Be consistent and use BDRV_SECTOR_SIZE instead of 512

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Clean up block.c and use BDRV_SECTOR_SIZE rather than hard coded numbers (512) when referring to sector size throughout the code. Signed-off-by: Jes Sorensen --- block.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/block.c b/block.c in

[Qemu-devel] Re: [PATCH] pci: add const to pci_is_express(), pci_config_size().

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 02:42:37PM +0900, Isaku Yamahata wrote: > add const to pci_is_express(), pci_config_size(). > > Signed-off-by: Isaku Yamahata Applied, thanks. > --- > hw/pci.h |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/hw/pci.h b/hw/pci.h > index

[Qemu-devel] [PATCH 1/4] Cleanup: bdrv_open() no need to shift total_size just to shift back.

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen In bdrv_open() there is no need to shift total_size >> 9 just to multiply it by 512 again just a few lines later, since this is the only place the variable is used. Mask with BDRV_SECTOR_MASK to protect against case where we are passed a corrupted image. Signed-off-by: Jes So

[Qemu-devel] Re: [PATCH 0/8] Make event_notifier more useful and more used

2010-05-27 Thread Michael S. Tsirkin
On Wed, May 26, 2010 at 04:09:30PM +0200, Paolo Bonzini wrote: > Hi, > > this patch adds all the eventfd bells and whistles from vl.c/cpus.c > to event_notifier, including pipe emulation and Win32 support. > It then modifies the iothread code to use it instead. I only seem to have got patches 4 a

[Qemu-devel] Re: [PATCH] pci.h: remove unused constants.

2010-05-27 Thread Michael S. Tsirkin
On Thu, May 27, 2010 at 02:40:02PM +0900, Isaku Yamahata wrote: > So remove unused constants, > PCI_STATUS_RESERVED_MASK_LO, PCI_STATUS_RESERVED_MASK_HI, > PCI_COMMAND_RESERVED, PCI_COMMAND_RESERVED_MASK_HI. > They were used once, but they aren't used anymore. > > Signed-off-by: Isaku Yamahata A

[Qemu-devel] [Bug 586175] Re: Windows XP/2003 doesn't boot

2010-05-27 Thread Anthony Liguori
Can someone try to reproduce this without ,boot=on? It's possible that extboot is screwing up the disk geometry. ** Also affects: fedora via https://bugzilla.redhat.com/show_bug.cgi?id=579348 Importance: Unknown Status: Unknown -- Windows XP/2003 doesn't boot https://bugs.launchpad

Re: [Qemu-devel] [PATCH] block.h: Make BDRV_SECTOR_SIZE 64 bit safe

2010-05-27 Thread Kevin Wolf
Am 27.05.2010 15:46, schrieb jes.soren...@redhat.com: > From: Jes Sorensen > > C defaults to int, so make definition of BDRV_SECTOR_SIZE 64 bit > safe as it and BDRV_SECTOR_MASK may be used against 64 bit addresses. > > Signed-off-by: Jes Sorensen Thanks, applied to the block branch. Kevin

[Qemu-devel] [PATCH 4/4] Cleanup: virtio-blk.c: Be more consistent using BDRV_SECTOR_SIZE instead

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Clean up virtio-blk.c to be more consistent using BDRV_SECTOR_SIZE instead of hard coded 512 values. Signed-off-by: Jes Sorensen --- hw/virtio-blk.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/virtio-blk.c b/hw/virtio-blk.c index 5d7f1a2

[Qemu-devel] [Bug 586175] Re: Windows XP/2003 doesn't boot

2010-05-27 Thread Natalia Portillo
I don't have any problem using TCG. Tested with Windows XP Home Update in 0.12.4 and Windows 2003 Enterprise Server in 0.12.3. -- Windows XP/2003 doesn't boot https://bugs.launchpad.net/bugs/586175 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed

[Qemu-devel] Re: [PATCH, RFC 1/4] pci: add I/O registration functions

2010-05-27 Thread Michael S. Tsirkin
On Sun, May 23, 2010 at 08:34:30PM +, Blue Swirl wrote: > Convert also APB to use the registration so that > we can remove mem_base. > > Signed-off-by: Blue Swirl > --- > hw/apb_pci.c | 23 - > hw/pci.c | 64 ++--

[Qemu-devel] [PATCH 3/4] Cleanup: raw-posix.c: Be more consistent using BDRV_SECTOR_SIZE instead of 512

2010-05-27 Thread Jes . Sorensen
From: Jes Sorensen Clean up raw-posix.c to be more consistent using BDRV_SECTOR_SIZE instead of hard coded 512 values. Signed-off-by: Jes Sorensen --- block/raw-posix.c | 20 +++- 1 files changed, 11 insertions(+), 9 deletions(-) diff --git a/block/raw-posix.c b/block/raw-po

[Qemu-devel] Re: [PATCH, RFC 0/4] Byte swapping I/O memory, preview

2010-05-27 Thread Michael S. Tsirkin
On Sun, May 23, 2010 at 08:34:17PM +, Blue Swirl wrote: > Hi, > > The overall plan is to add a new flag, IO_MEM_BSWAP, which triggers > byte swapping inside CPU load/store functions. Convert all PCI devices > to register their memory areas via the PCI host. Then the big endian > PCI hosts enab

Re: [Qemu-devel] [PATCH 1/3] vnc: tight: don't forget last pixel in tight_encode_indexed_rect

2010-05-27 Thread Richard Henderson
On 05/26/2010 11:21 PM, Corentin Chary wrote: > -int rep = 0;\ > +int i = 0, rep = 0; \ Dead initialization. r~

[Qemu-devel] [Bug 586175] Re: Windows XP/2003 doesn't boot

2010-05-27 Thread tekditt
It's a very strange bug. Starting qemu without boot=on results in the same dilemma. But: I've used a .vdi image (because qcow2 is terribly slow). Just now I tried a raw image. Now I can boot neither directly from the image nor with the install cd. Both ways boot finally NTLDR, but now the loader

[Qemu-devel] [Bug 586420] [NEW] WinXP install cd hangs at boot time if machine started with floppy

2010-05-27 Thread tekditt
Public bug reported: I have a second problem: I wanted to install Windows Server 2003 on a virtio drive, so I tried to start the machine with the install cd as the boot drive and a floppy image with the viostor drivers. The problem is, the install cd hangs at boot time. If I start VNC I just see

[Qemu-devel] Re: [PATCH 0/8] Make event_notifier more useful and more used

2010-05-27 Thread Paolo Bonzini
On 05/27/2010 04:21 PM, Michael S. Tsirkin wrote: On Wed, May 26, 2010 at 04:09:30PM +0200, Paolo Bonzini wrote: Hi, this patch adds all the eventfd bells and whistles from vl.c/cpus.c to event_notifier, including pipe emulation and Win32 support. It then modifies the iothread code to use it in

[Qemu-devel] Re: [PATCH] block.h: Make BDRV_SECTOR_SIZE 64 bit safe

2010-05-27 Thread Paolo Bonzini
On 05/27/2010 04:27 PM, Kevin Wolf wrote: Am 27.05.2010 15:46, schrieb jes.soren...@redhat.com: From: Jes Sorensen C defaults to int, so make definition of BDRV_SECTOR_SIZE 64 bit safe as it and BDRV_SECTOR_MASK may be used against 64 bit addresses. Signed-off-by: Jes Sorensen Thanks, applie

[Qemu-devel] Re: [PATCH] block.h: Make BDRV_SECTOR_SIZE 64 bit safe

2010-05-27 Thread Jes Sorensen
On 05/27/10 17:38, Paolo Bonzini wrote: > On 05/27/2010 04:27 PM, Kevin Wolf wrote: >> Am 27.05.2010 15:46, schrieb jes.soren...@redhat.com: >>> From: Jes Sorensen >>> >>> C defaults to int, so make definition of BDRV_SECTOR_SIZE 64 bit >>> safe as it and BDRV_SECTOR_MASK may be used against 64 bit

[Qemu-devel] [Bug 586424] [NEW] SMC91C111 failed when booting Linux/ARM(Mainstone) since 0.10.0

2010-05-27 Thread Daniel Rossier
Public bug reported: Since QEMU 0.10.0, the SMC91C111 emulation on a ARM machine like Mainstone, fails when performing some read/write. ** Affects: qemu Importance: Undecided Assignee: Daniel Rossier (daniel-rossier) Status: In Progress ** Changed in: qemu Assignee: (unas

Re: [Qemu-devel] [PATCH 0/2] Fix scsi-generic breakage in upstream qemu-kvm.git

2010-05-27 Thread Nicholas A. Bellinger
On Thu, 2010-05-20 at 15:18 +0200, Kevin Wolf wrote: > Am 17.05.2010 18:45, schrieb Nicholas A. Bellinger: > > From: Nicholas Bellinger > > > > Greetings, > > > > Attached are the updated patches following hch's comments to fix > > scsi-generic > > device breakage with find_image_format() and r

[Qemu-devel] SVM emulation: EVENTINJ marked valid when a pagefault happens while issuing a software interrupt

2010-05-27 Thread Erik van der Kouwe
Dear all, I have been experiencing problems with duplicate delivery of software interrupts when running a VMM inside QEMU with SVM emulation. I believe QEMU's behaviour deviates from the SVM specification in "AMD64 Architecture Programmer’s Manual Volume 2 System Programming" but I am not ent

[Qemu-devel] Re: [PATCH 3/5] QMP: Introduce MIGRATION events

2010-05-27 Thread Juan Quintela
Luiz Capitulino wrote: > On Wed, 26 May 2010 11:55:31 -0500 > Anthony Liguori wrote: >> That's exactly how the protocol is designed. That was one of the major >> improvements of QMP over the human monior. > > Yes and it already has 'id' support: > > { "execute": "cont", "id": "luiz" } > {"tim

[Qemu-devel] Re: [PATCH 3/5] QMP: Introduce MIGRATION events

2010-05-27 Thread Anthony Liguori
On 05/27/2010 10:58 AM, Juan Quintela wrote: Luiz Capitulino wrote: On Wed, 26 May 2010 11:55:31 -0500 Anthony Liguori wrote: That's exactly how the protocol is designed. That was one of the major improvements of QMP over the human monior. Yes and it already has 'id'

[Qemu-devel] Re: [PATCH 3/5] QMP: Introduce MIGRATION events

2010-05-27 Thread Luiz Capitulino
On Thu, 27 May 2010 17:58:03 +0200 Juan Quintela wrote: > Luiz Capitulino wrote: > > On Wed, 26 May 2010 11:55:31 -0500 > > Anthony Liguori wrote: > > >> That's exactly how the protocol is designed. That was one of the major > >> improvements of QMP over the human monior. > > > > Yes and it

[Qemu-devel] [Bug 586420] Re: WinXP install cd hangs at boot time if machine started with floppy

2010-05-27 Thread Anthony Liguori
Can you please include information about what version of QEMU you are using including the full command line you are using to launch it? ** Changed in: qemu Status: New => Incomplete -- WinXP install cd hangs at boot time if machine started with floppy https://bugs.launchpad.net/bugs/58642

Re: [Qemu-devel] Re: [PATCH 1/2] Pad iommu with an empty slot (necessary for SunOS 4.1.4)

2010-05-27 Thread Bob Breuer
Artyom Tarasenko wrote: > Was going to put some more empty slots into SS-10/20 (VSIMMs, SX) > after we are done with SS-5 (due to technical limitations I can switch > access from one real SS model to another one once a few days only). > I have a partial implementation of the SS-20 VSIMM (cg14) t

[Qemu-devel] [Bug 586420] Re: WinXP install cd hangs at boot time if machine started with floppy

2010-05-27 Thread tekditt
Oh, I forgot, I'm the one who also reported this bug: https://bugs.launchpad.net/qemu/+bug/586175 qemu-kvm-0.12.4 with SeaBIOS 0.5.1 on Gentoo (No-Multilib and AMD64) The install cd hangs at every combination of -fda/-cdrom/-hdc and -drive, just like > qemu-kvm -cdrom /srv/vm/win2k3srv.iso -hda

[Qemu-devel] Re: [OpenBIOS] [PATCH 0/3] sparc64 cleanups v1

2010-05-27 Thread Mark Cave-Ayland
Blue Swirl wrote: On Tue, May 25, 2010 at 12:12 PM, Igor V. Kovalenko wrote: One code cleanup and another pci host bridge remap change, the latter requires qemu update with patch already posted to qemu list. v0->v1: added missing patch moving asi.h to arch includes Thanks, applied all. Wh

[Qemu-devel] Re: [PATCH v4 0/10] Introduce 'info netdevices' with QMP support

2010-05-27 Thread Luiz Capitulino
On Tue, 18 May 2010 14:07:39 -0300 Miguel Di Ciurcio Filho wrote: > The VLANClientState structure has the member info_str, a simple string that > is filled with information about NIC devices and used on monitor calls. > > There is no coherent formatting of this string by all the NIC devices, > m

[Qemu-devel] Re: [PATCH v4 01/10] QObject API: introduce qdict_to_qstring() function

2010-05-27 Thread Luiz Capitulino
On Tue, 18 May 2010 14:07:40 -0300 Miguel Di Ciurcio Filho wrote: > This is a helper function that converts a QDict to a QString, using > the format: > > key1=value1 SEP key2=value2 SEP key3=value3 > > Handy for debugging and formating the Monitor output. > > Signed-off-by: Miguel Di Ciurcio F

[Qemu-devel] Re: [PATCH-V4 0/7] virtio-9p:Introducing security model for VirtFS

2010-05-27 Thread Venkateswararao Jujjuri (JV)
Andy Lutomirski wrote: > Venkateswararao Jujjuri (JV) wrote: >> This patch series introduces the security model for VirtFS. >> >> Brief description of this patch series: >> >> It introduces two type of security models for VirtFS. >> They are: mapped and passthrough. >> >> The following is common to

[Qemu-devel] Re: [PATCH v4 05/10] net: tap/tap-win32: introduce info_dict

2010-05-27 Thread Luiz Capitulino
On Tue, 18 May 2010 14:07:44 -0300 Miguel Di Ciurcio Filho wrote: > Signed-off-by: Miguel Di Ciurcio Filho > --- > net/tap-win32.c |9 - > net/tap.c | 18 +- > 2 files changed, 25 insertions(+), 2 deletions(-) > > diff --git a/net/tap-win32.c b/net/tap-win32

[Qemu-devel] Re: [PATCH v4 06/10] net: vde: introduce info_dict

2010-05-27 Thread Luiz Capitulino
On Tue, 18 May 2010 14:07:45 -0300 Miguel Di Ciurcio Filho wrote: > Signed-off-by: Miguel Di Ciurcio Filho > --- > net/vde.c | 11 ++- > 1 files changed, 10 insertions(+), 1 deletions(-) > > diff --git a/net/vde.c b/net/vde.c > index 0b46fa6..6a3d0ba 100644 > --- a/net/vde.c > +++ b/

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-27 Thread Blue Swirl
On Wed, May 26, 2010 at 11:26 PM, Paul Brook wrote: >> At the other extreme, would it be possible to make the educated guests >> aware of the virtualization also in clock aspect: virtio-clock? > > The guest doesn't even need to be aware of virtualization. It just needs to be > able to accommodate

[Qemu-devel] Re: [PATCH] Add dependency of JSON unit tests on config-host.h

2010-05-27 Thread Luiz Capitulino
On Thu, 20 May 2010 09:18:52 +0200 Jan Kiszka wrote: > From: Jan Kiszka > > Signed-off-by: Jan Kiszka Looks good. > --- > Makefile |2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/Makefile b/Makefile > index 110698e..aa81d9b 100644 > --- a/Makefile > +++ b/Ma

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-27 Thread Blue Swirl
2010/5/27 Gleb Natapov : > On Wed, May 26, 2010 at 08:35:00PM +, Blue Swirl wrote: >> On Wed, May 26, 2010 at 8:09 PM, Jan Kiszka wrote: >> > Blue Swirl wrote: >> >> On Tue, May 25, 2010 at 9:44 PM, Jan Kiszka wrote: >> >>> Anthony Liguori wrote: >> On 05/25/2010 02:09 PM, Blue Swirl wro

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-27 Thread Jan Kiszka
Blue Swirl wrote: > On Wed, May 26, 2010 at 11:26 PM, Paul Brook wrote: >>> At the other extreme, would it be possible to make the educated guests >>> aware of the virtualization also in clock aspect: virtio-clock? >> The guest doesn't even need to be aware of virtualization. It just needs to >>

[Qemu-devel] Matheus Teles cantor sertanejo

2010-05-27 Thread Matheus Teles
Matheus Teles, 15 anos de idade, cantor sertanejo. Acesse: www.MatheusTeles.com.br

[Qemu-devel] Re: SVM emulation: EVENTINJ marked valid when a pagefault happens while issuing a software interrupt

2010-05-27 Thread Jan Kiszka
Erik van der Kouwe wrote: > Dear all, > > I have been experiencing problems with duplicate delivery of software > interrupts when running a VMM inside QEMU with SVM emulation. I believe Be warned: Though my experience is already more than a year old, the SVM emulation in QEMU is most probably not

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-27 Thread Blue Swirl
On Thu, May 27, 2010 at 6:31 PM, Jan Kiszka wrote: > Blue Swirl wrote: >> On Wed, May 26, 2010 at 11:26 PM, Paul Brook wrote: At the other extreme, would it be possible to make the educated guests aware of the virtualization also in clock aspect: virtio-clock? >>> The guest doesn't even

[Qemu-devel] [Bug 586424] Re: SMC91C111 failed when booting Linux/ARM(Mainstone) since 0.10.0

2010-05-27 Thread Lars Munch
This was fixed some time ago in commit 3b4b86aace17ef07fc4f85a9662c991efbc83e15 -- SMC91C111 failed when booting Linux/ARM(Mainstone) since 0.10.0 https://bugs.launchpad.net/bugs/586424 You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. St

[Qemu-devel] [Bug 586221] Re: Linux on ARM/Mainstone machine fails at bootstrap

2010-05-27 Thread Lars Munch
There are already patches pending to solve these issues: http://article.gmane.org/gmane.comp.emulators.qemu/69598 and http://article.gmane.org/gmane.comp.emulators.qemu/69597 Hopefully they will be reviewed/applied soon. -- Linux on ARM/Mainstone machine fails at bootstrap https://bugs.launchpa

Re: [Qemu-devel] [PATCH] vhost_net.c: v2 Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2

2010-05-27 Thread Blue Swirl
Thanks, applied. On Thu, May 27, 2010 at 12:26 PM, wrote: > From: Jes Sorensen > > Fix build failure introduced by 0bfcd599e3f5c5679cc7d0165a0a1822e2f60de2 > > The format statement expects unsigned long on x86_64, but receives > unsigned long long, so gcc exits with an error. > > Signed-off-by:

[Qemu-devel] Re: [PATCH, RFC 1/4] pci: add I/O registration functions

2010-05-27 Thread Blue Swirl
On Thu, May 27, 2010 at 2:39 PM, Michael S. Tsirkin wrote: > On Sun, May 23, 2010 at 08:34:30PM +, Blue Swirl wrote: >> Convert also APB to use the registration so that >> we can remove mem_base. >> >> Signed-off-by: Blue Swirl >> --- >>  hw/apb_pci.c |   23 - >>  hw/pci.c

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-27 Thread Jan Kiszka
Blue Swirl wrote: > On Thu, May 27, 2010 at 6:31 PM, Jan Kiszka wrote: >> Blue Swirl wrote: >>> On Wed, May 26, 2010 at 11:26 PM, Paul Brook wrote: > At the other extreme, would it be possible to make the educated guests > aware of the virtualization also in clock aspect: virtio-clock? >>

[Qemu-devel] [PATCH 0/1] ceph/rbd block driver for qemu-kvm (v2)

2010-05-27 Thread Christian Brunner
Hi, Based on the review notes Blue Swirl sent us after my last mail, Yehuda cleaned up the header files. The patch is much smaller now and I hope that you accept it for inclusion. To build it, you will need the testing (or unstable) git head of ceph now. The required header files will be part of

[Qemu-devel] [PATCH 1/1] ceph/rbd block driver for qemu-kvm (v2)

2010-05-27 Thread Christian Brunner
This is a block driver for the distributed file system Ceph (http://ceph.newdream.net/). This driver uses librados (which is part of the Ceph server) for direct access to the Ceph object store and is running entirely in userspace. Therefore it is called "rbd" - rados block device. To compile t

Re: [Qemu-devel] Re: [RFT][PATCH 07/15] qemu_irq: Add IRQ handlers with delivery feedback

2010-05-27 Thread Blue Swirl
On Thu, May 27, 2010 at 7:08 PM, Jan Kiszka wrote: > Blue Swirl wrote: >> On Thu, May 27, 2010 at 6:31 PM, Jan Kiszka wrote: >>> Blue Swirl wrote: On Wed, May 26, 2010 at 11:26 PM, Paul Brook wrote: >> At the other extreme, would it be possible to make the educated guests >> aware o

[Qemu-devel] Re: [PATCH v3 06/17] qdev: Allow device specification by qtree path for device_del

2010-05-27 Thread Luiz Capitulino
On Sun, 23 May 2010 12:59:19 +0200 Jan Kiszka wrote: > From: Jan Kiszka > > Allow to specify the device to be removed via device_del not only by ID > but also by its full or abbreviated qtree path. For this purpose, > qdev_find is introduced which combines walking the qtree with searching > for

[Qemu-devel] Re: SVM emulation: EVENTINJ marked valid when a pagefault happens while issuing a software interrupt

2010-05-27 Thread Erik van der Kouwe
Hi, Be warned: Though my experience is already more than a year old, the SVM emulation in QEMU is most probably not yet rock-stable. Always check suspicious behavior against real hardware and/or the spec. [ As real hardware is everywhere, nesting works with KVM+SVM and is much faster, motivation

[Qemu-devel] Re: [PATCH v3 10/17] QMP: Reserve namespace for complex object classes

2010-05-27 Thread Luiz Capitulino
On Sun, 23 May 2010 12:59:23 +0200 Jan Kiszka wrote: > From: Jan Kiszka > > This reserves JSON objects that contain the key '__class__' for QMP-specific > complex objects. First user will be the buffer class. > > Signed-off-by: Jan Kiszka > --- > QMP/qmp-spec.txt | 16 +--- > 1

[Qemu-devel] [PATCH] Extra scan codes for missing keys

2010-05-27 Thread Brendan Sleight
Hi All, First - Qemu is fantastic and allows lots of wonderful things. Second, when using qemu-system-ppc, I wanted to use sendkey to emulate a colon. This patch enables shift-semicolon to emulate a ':' Whilst I was adding semicolon, I used the following link to look up some other missing keys :

[Qemu-devel] Re: [PATCH v3 13/17] monitor: Allow to exclude commands from QMP

2010-05-27 Thread Luiz Capitulino
On Sun, 23 May 2010 12:59:26 +0200 Jan Kiszka wrote: > From: Jan Kiszka > > Ported commands that are marked 'user_only' will not be considered for > QMP monitor sessions. This allows to implement new commands that do not > (yet) provide a sufficiently stable interface for QMP use (e.g. > device

[Qemu-devel] Re: [PATCH v3 16/17] QMP: Fix python helper /wrt long return strings

2010-05-27 Thread Luiz Capitulino
On Sun, 23 May 2010 12:59:29 +0200 Jan Kiszka wrote: > From: Jan Kiszka > > Remove the arbitrary limitation of 1024 characters per return string and > read complete lines instead. Required for device_show. Thanks for both fixes, I have started working on a better version of this script that m

[Qemu-devel] Re: [OpenBIOS] [PATCH 0/3] sparc64 cleanups v1

2010-05-27 Thread Blue Swirl
On Thu, May 27, 2010 at 4:57 PM, Mark Cave-Ayland wrote: > Blue Swirl wrote: > >> On Tue, May 25, 2010 at 12:12 PM, Igor V. Kovalenko >> wrote: >>> >>> One code cleanup and another pci host bridge remap change, >>> the latter requires qemu update with patch already posted to qemu list. >>> >>> v0

[Qemu-devel] [PATCH 00/62] s390x tcg target

2010-05-27 Thread Richard Henderson
The following patch series is available at git://repo.or.cz/qemu/rth.git tcg-s390-2 It begins with Uli Hecht's original patch, posted by Alexander sometime last year. I then make incremental changes to (1) Make it compile -- first patch that compiles is tagged as tcg-s390-2-first-comp

[Qemu-devel] [PATCH 03/62] tcg-s390: Only validate CPUTLBEntry for system mode.

2010-05-27 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/s390/tcg-target.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tcg/s390/tcg-target.c b/tcg/s390/tcg-target.c index 45c1bf7..9ab1d96 100644 --- a/tcg/s390/tcg-target.c +++ b/tcg/s390/tcg-target.c @@ -1198,10 +1198,12 @@ do_logic

[Qemu-devel] [PATCH 02/62] add lost chunks from the original patch

2010-05-27 Thread Richard Henderson
From: Alexander Graf --- tcg/s390/tcg-target.c |3 ++ tcg/s390/tcg-target.h | 86 +++-- 2 files changed, 86 insertions(+), 3 deletions(-) diff --git a/tcg/s390/tcg-target.c b/tcg/s390/tcg-target.c index d2a93c2..45c1bf7 100644 --- a/tcg/s390/tcg

[Qemu-devel] [PATCH 13/62] tcg-s390: Add -m64 and -march to s390x compilation.

2010-05-27 Thread Richard Henderson
Signed-off-by: Richard Henderson --- configure |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/configure b/configure index e2b389d..72d3df8 100755 --- a/configure +++ b/configure @@ -697,7 +697,11 @@ case "$cpu" in fi ;; s390) -

[Qemu-devel] [PATCH 04/62] tcg-s390: Fix tcg_prepare_qemu_ldst for user mode.

2010-05-27 Thread Richard Henderson
This isn't the most efficient way to implement user memory accesses, but it's the minimal change to fix the compilation error. Signed-off-by: Richard Henderson --- tcg/s390/tcg-target.c |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/tcg/s390/tcg-target.c b/tcg/s3

[Qemu-devel] [PATCH 06/62] s390x: Avoid _llseek.

2010-05-27 Thread Richard Henderson
There's no _llseek on s390x either. Replace the existing test for __x86_64__ with a functional test for __NR_llseek. Signed-off-by: Richard Henderson --- linux-user/syscall.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c

[Qemu-devel] [PATCH 16/62] tcg-s390: Compute is_write in cpu_signal_handler.

2010-05-27 Thread Richard Henderson
Signed-off-by: Richard Henderson --- cpu-exec.c | 42 +++--- 1 files changed, 39 insertions(+), 3 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index c776605..026980a 100644 --- a/cpu-exec.c +++ b/cpu-exec.c @@ -1156,11 +1156,47 @@ int cpu_signal_handler

[Qemu-devel] [PATCH 01/62] S390 TCG target

2010-05-27 Thread Richard Henderson
From: Alexander Graf We already have stubs for a TCG target on S390, but were missing code that would actually generate instructions. So I took Uli's patch, cleaned it up and present it to you again :-). I hope I found all odd coding style and unprettiness issues, but if you still spot one feel

[Qemu-devel] [PATCH 10/62] tcg-s390: R6 is a function argument register

2010-05-27 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/s390/tcg-target.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/tcg/s390/tcg-target.c b/tcg/s390/tcg-target.c index 6988937..25c80e6 100644 --- a/tcg/s390/tcg-target.c +++ b/tcg/s390/tcg-target.c @@ -137,6 +137,7 @@ static const

[Qemu-devel] [PATCH 21/62] tcg-s390: Generalize the direct load/store emission.

2010-05-27 Thread Richard Henderson
Define tcg_out_ldst which can properly choose between RX and RXY format instructions based on the offset used, and also handles large offsets. Use it to implement all the INDEX_op_ld/st operations. Signed-off-by: Richard Henderson --- tcg/s390/tcg-target.c | 152 +++

  1   2   >