[Qemu-devel] [Bug 1013467] [NEW] network in bridge mode doesn't work in SMP Linux guest

2012-06-14 Thread Yongjie Ren
Public bug reported: Environment: Host OS (ia32/ia32e/IA64):ia32e Guest OS (ia32/ia32e/IA64):ia32e Guest OS Type (Linux/Windows):Linux kvm.git Commit:25e531a988ea5a64bd97a72dc9d3c65ad5850120 qemu-kvm Commit:0a948cbb1835e5f36990e173966d30bc4c8cc038 Host Kernel Version:3.5.0-rc1 Hardwar

[Qemu-devel] [Bug 1013467] Re: network in bridge mode doesn't work in SMP Linux guest

2012-06-14 Thread Yongjie Ren
** Attachment added: "lspci -vvv in guest" https://bugs.launchpad.net/qemu/+bug/1013467/+attachment/3190180/+files/lspci-guest.log -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1013467 Title: n

Re: [Qemu-devel] [libseccomp-discuss] [RFC] [PATCHv2 0/2] Sandboxing Qemu guests with Libseccomp

2012-06-14 Thread Kees Cook
On Wed, Jun 13, 2012 at 1:31 PM, Paul Moore wrote: > On Wednesday, June 13, 2012 04:20:20 PM Eduardo Otubo wrote: >> Hello all, >> >> This is the second effort to sandbox Qemu guests using Libseccomp[0]. > > ... > >> [0] - http://sourceforge.net/projects/libseccomp/ [1] - >> http://git.kernel.org/

Re: [Qemu-devel] IO performance test on the tcm-vhost scsi

2012-06-14 Thread Asias He
On 06/14/2012 08:07 PM, Stefan Hajnoczi wrote: On Thu, Jun 14, 2012 at 05:45:22PM +0800, Cong Meng wrote: On Thu, 2012-06-14 at 09:30 +0100, Stefan Hajnoczi wrote: On Wed, Jun 13, 2012 at 11:13 AM, mengcong wrote: seq-readseq-write rand-read rand-write

[Qemu-devel] [Bug 1013467] Re: network in bridge mode doesn't work in SMP Linux guest

2012-06-14 Thread Yongjie Ren
** Attachment added: "guest dmesg log" https://bugs.launchpad.net/qemu/+bug/1013467/+attachment/3190179/+files/dmesg-guest.log -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1013467 Title: netwo

Re: [Qemu-devel] [PATCH 1/1] Add usb option in machine options.

2012-06-14 Thread Li Zhang
On Fri, Jun 15, 2012 at 1:15 PM, Peter Crosthwaite < peter.crosthwa...@petalogix.com> wrote: > On Fri, Jun 15, 2012 at 1:06 PM, Li Zhang wrote: > > > > > > On Thu, Jun 14, 2012 at 10:27 PM, Andreas Färber > wrote: > >> > >> Am 14.06.2012 07:17, schrieb zhlci...@gmail.com: > >> > From: Li Zhang

Re: [Qemu-devel] [PATCH 1/1] Add usb option in machine options.

2012-06-14 Thread Benjamin Herrenschmidt
On Fri, 2012-06-15 at 15:15 +1000, Peter Crosthwaite wrote: > > What should really happen here, is if you do change over to a machine > opt, then all clients of the -usb should use the machine opt > infrastructure, then the need for global state is eliminated > completely. > > But does this new -

Re: [Qemu-devel] [PATCH v6 2/3] Simpletrace v2: Support multiple arguments, strings.

2012-06-14 Thread Harsh Bora
On 06/15/2012 12:09 AM, Harsh Prateek Bora wrote: static gpointer writeout_thread(gpointer opaque) { -TraceRecord record; -unsigned int writeout_idx = 0; -unsigned int num_available, idx; +TraceRecord *recordptr, *dropped_ptr; +union { +TraceRecord rec; +ui

Re: [Qemu-devel] [PATCH 1/1] Add usb option in machine options.

2012-06-14 Thread Peter Crosthwaite
On Fri, Jun 15, 2012 at 1:06 PM, Li Zhang wrote: > > > On Thu, Jun 14, 2012 at 10:27 PM, Andreas Färber wrote: >> >> Am 14.06.2012 07:17, schrieb zhlci...@gmail.com: >> > From: Li Zhang >> > >> > For pseries machine, it needs to enable usb >> > to add kbd or usb mouse. -usb option won't >> > be

Re: [Qemu-devel] Broken Microblaze timer

2012-06-14 Thread Peter Chubb
> "Peter" == Peter Crosthwaite writes: Peter> So ptimer has safeguards against misuse in periodic mode but Peter> not one-shot mode? This strikes me as inconsistent. I.E. for Peter> one use case the safeguard is in ptimer, and the other in the Peter> client device. I think if we are in the bu

[Qemu-devel] use after free in usb code

2012-06-14 Thread Bruce Rogers
Hi, A bug was reported against qemu v1.1 in openSUSE 12.2. See: https://bugzilla.novell.com/show_bug.cgi?id=766310 I've discovered that uhci_queue_free is called with a queue that is still active. Bisecting shows that this bug was introduced in git commit id d9a528db7f2d71d92e869e20bda37774f11fbb

Re: [Qemu-devel] [PATCH 0/3] qom: refactor Interfaces

2012-06-14 Thread Peter Crosthwaite
On Thu, Jun 14, 2012 at 6:54 AM, Anthony Liguori wrote: > The interface implementation was pretty busted.  The way it created Objects > for > each interface was extremely clumbsy and brittle. > > This is a new implementation that does something quite a bit more natural.  It > simply modifies clas

Re: [Qemu-devel] wiki account

2012-06-14 Thread Peter Crosthwaite
Thanks Stefan, Regards, Peter On Thu, Jun 14, 2012 at 11:22 PM, Stefan Hajnoczi wrote: > On Thu, Jun 14, 2012 at 3:31 AM, Peter Crosthwaite > wrote: >> I wish to upload my test images to the wiki, and I see the process for >> getting an account is to contact via email. Can someone create me an

Re: [Qemu-devel] Broken Microblaze timer

2012-06-14 Thread Peter Crosthwaite
On Fri, Jun 15, 2012 at 8:39 AM, Peter Chubb wrote: >> "Paolo" == Paolo Bonzini writes: > > Paolo> Il 14/06/2012 04:29, Peter Crosthwaite ha scritto: >>> Obviously this sucks as a patch, but without this hack, the system >>> freezes on boot. I managed to ascertain that its coming from the >>>

Re: [Qemu-devel] [PATCH] trace: added ability to comment out events in the list

2012-06-14 Thread Alexey Kardashevskiy
On 14/06/12 23:18, Stefan Hajnoczi wrote: > On Thu, Jun 14, 2012 at 02:41:40PM +1000, Alexey Kardashevskiy wrote: >> It is convenient for debug to be able to switch on/off some events easily. >> The only possibility now is to remove event name from the file completely >> and type it again when we w

Re: [Qemu-devel] [PATCH 1/1] Add usb option in machine options.

2012-06-14 Thread Li Zhang
On Thu, Jun 14, 2012 at 10:27 PM, Andreas Färber wrote: > Am 14.06.2012 07:17, schrieb zhlci...@gmail.com: > > From: Li Zhang > > > > For pseries machine, it needs to enable usb > > to add kbd or usb mouse. -usb option won't > > be used in the future, and machine options > > is a better way to e

Re: [Qemu-devel] [PATCH 1/1] Add usb option in machine options.

2012-06-14 Thread Li Zhang
OK,sorry for my fault. best regards Li Zhang 在 2012-6-15 上午6:05,"Benjamin Herrenschmidt" 写道: > > On Thu, 2012-06-14 at 16:27 +0200, Andreas Färber wrote: > > Am 14.06.2012 07:17, schrieb zhlci...@gmail.com: > > > From: Li Zhang > > > > > > For pseries machine, it needs to enable usb > > > to add

Re: [Qemu-devel] vio-net driver

2012-06-14 Thread Charles . Tsai-蔡清海-研究發展部
Brain, Thank you for information. With your confirmation, I think our engineer must have done something wrong during the software upgrade process. I will keep an eye on this when we re-do the test. -Original Message- From: Brian Jackson [mailto:i...@theiggy.com] Sent: Friday

Re: [Qemu-devel] How to measure guest memory access (qemu_ld/qemu_st) time?

2012-06-14 Thread YeongKyoon Lee
Title: Samsung Enterprise Portal mySingle I don't know much of instrumentation approach of Lluis, however, I had roughly estimated guest memory access (qemu_ld/st performance) overhead in legacy way last year. My idea was indirect performance estimation by measuring the generated host instruction

[Qemu-devel] [Bug 1002121] Re: disk error when guest boot up via qcow2 image

2012-06-14 Thread Yongjie Ren
Re-tested against latest qemu-kvm master branch with commit 0a948cbb. And found this issue has gone. Jan Kiszka's following patch fixed this bug. commit 0cdd3d14447da1a04e778c219c77db8b96f9cf33 Author: Jan Kiszka Date: Wed Jun 6 16:28:42 2012 +0200 kvm: i8254: Fix conversion of in-kernel

Re: [Qemu-devel] How to measure guest memory access (qemu_ld/qemu_st) time?

2012-06-14 Thread Lluís Vilanova
陳韋任 (Wei-Ren Chen) writes: >> Unfortunately, I had the bad idea of rebasing all my series on top of the >> latest >> makefile changes, and I'll have to go through each patch to check it's still >> working (I'm sure some of them broke). > Need some help? :) Well, it's just a matter of going th

Re: [Qemu-devel] Broken Microblaze timer

2012-06-14 Thread Peter Chubb
> "Paolo" == Paolo Bonzini writes: Paolo> Il 14/06/2012 04:29, Peter Crosthwaite ha scritto: >> Obviously this sucks as a patch, but without this hack, the system >> freezes on boot. I managed to ascertain that its coming from the >> ptimer used by the system timer (hw/xilinx_timer.c). The CP

Re: [Qemu-devel] How to measure guest memory access (qemu_ld/qemu_st) time?

2012-06-14 Thread Lluís Vilanova
陳韋任 (Wei-Ren Chen) writes: > On Wed, Jun 13, 2012 at 12:43:28PM +0200, Laurent Desnogues wrote: >> On Wed, Jun 13, 2012 at 5:14 AM, 陳韋任 (Wei-Ren Chen) >> wrote: >> > Hi all, >> > >> >  I suspect that guest memory access (qemu_ld/qemu_st) account for the >> > major of >> > time spent in system mo

Re: [Qemu-devel] vio-net driver

2012-06-14 Thread Brian Jackson
On Sunday, June 10, 2012 08:50:30 PM Charles.Tsai-蔡清海-研究發展部 wrote: > Hi Brian, > > Do you have any finding for VirtIO Net driver problem? Nothing jumped out at me after looking over your response. It would still be nice to know the actual kvm command line options that were used (since I don't

Re: [Qemu-devel] [PATCH v2 00/41] postcopy live migration

2012-06-14 Thread Juan Quintela
Isaku Yamahata wrote: > After the long time, we have v2. This is qemu part. > The linux kernel part is sent separatedly. > > Changes v1 -> v2: > - split up patches for review > - buffered file refactored > - many bug fixes > Espcially PV drivers can work with postcopy > - optimization/heuristic

Re: [Qemu-devel] [PATCH v2 37/41] postcopy: implement outgoing part of postcopy live migration

2012-06-14 Thread Juan Quintela
Isaku Yamahata wrote: > This patch implements postcopy live migration for outgoing part > > Signed-off-by: Isaku Yamahata > --- > Changes v1 -> v2: > - fix parameter to qemu_fdopen() > - handle QEMU_UMEM_REQ_EOC properly > when PO_STATE_ALL_PAGES_SENT, QEMU_UMEM_REQ_EOC request was ignored. >

Re: [Qemu-devel] [PATCH 1/1] Add usb option in machine options.

2012-06-14 Thread Benjamin Herrenschmidt
On Thu, 2012-06-14 at 16:27 +0200, Andreas Färber wrote: > Am 14.06.2012 07:17, schrieb zhlci...@gmail.com: > > From: Li Zhang > > > > For pseries machine, it needs to enable usb > > to add kbd or usb mouse. -usb option won't > > be used in the future, and machine options > > is a better way to e

Re: [Qemu-devel] [PATCH v2 35/41] postcopy: introduce helper functions for postcopy

2012-06-14 Thread Juan Quintela
Isaku Yamahata wrote: > +//#define DEBUG_UMEM > +#ifdef DEBUG_UMEM > +#include > +#define DPRINTF(format, ...)\ > +do {\ > +printf("%d:%ld %s:%d "format, getpid(), syscall(SYS_g

Re: [Qemu-devel] qemu-system-ppc64 hanging occasionally in disk writes

2012-06-14 Thread Richard W.M. Jones
On Thu, Jun 14, 2012 at 11:34:37PM +0200, Alexander Graf wrote: > > /home/rjones/d/qemu/ppc64-softmmu/qemu-system-ppc64 \ > >-global virtio-blk-pci.scsi=off \ > >-nodefconfig \ > >-nodefaults \ > >-nographic \ > >-device virtio-scsi-pci,id=scsi \ > >-drive file=test1.img,cac

Re: [Qemu-devel] [PATCH v2 36/41] postcopy: implement incoming part of postcopy live migration

2012-06-14 Thread Juan Quintela
Isaku Yamahata wrote: > This patch implements postcopy live migration for incoming part > > Signed-off-by: Isaku Yamahata > +void ram_save_set_params(const MigrationParams *params, void *opaque); > -register_savevm_live(NULL, "ram", 0, RAM_SAVE_VERSION_ID, NULL, > -

Re: [Qemu-devel] [PATCH v2 36/41] postcopy: implement incoming part of postcopy live migration

2012-06-14 Thread Juan Quintela
Isaku Yamahata wrote: > This patch implements postcopy live migration for incoming part > > vl.c |8 +- > 13 files changed, 1409 insertions(+), 21 deletions(-) > copy linux-headers/linux/umem.h => migration-postcopy-stub.c (55%) Ouch, git got re

Re: [Qemu-devel] qemu-system-ppc64 hanging occasionally in disk writes

2012-06-14 Thread Alexander Graf
On 14.06.2012, at 19:13, "Richard W.M. Jones" wrote: > On Thu, Jun 14, 2012 at 05:58:04PM +0200, Alexander Graf wrote: >> [CC'ing qemu-ppc] >> >> On 06/14/2012 05:52 PM, Richard W.M. Jones wrote: >>> I found last week that qemu-system-ppc64 (from git) hangs occasionally >>> under load, and I h

Re: [Qemu-devel] [PATCH v3 0/8] msix: Support specifying offsets, BARs, and capability location

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 12:15:42PM -0600, Alex Williamson wrote: > v3: > - more patches, smaller diff, must be headed in the right direction > - macros for all hardcoded values in msix_init_exclusive_bar > - fold msix_add_config into msix_init allowing less churn to moving >around msix_unini

Re: [Qemu-devel] [PATCH v2 00/41] postcopy live migration

2012-06-14 Thread Juan Quintela
Avi Kivity wrote: > On 06/08/2012 01:16 PM, Juan Quintela wrote: >> Anthony Liguori wrote: >> >> Once told that, we need to measure what is the time of an async page >> fault over the network. If it is too high, post copy just don't work. >> >> And no, I haven't seen any measurement that told us

Re: [Qemu-devel] IO performance test on the tcm-vhost scsi

2012-06-14 Thread Nicholas A. Bellinger
On Thu, 2012-06-14 at 14:27 +0200, Paolo Bonzini wrote: > Il 14/06/2012 14:07, Stefan Hajnoczi ha scritto: > > Perfect, thanks. virtio-scsi userspace is much better than virtio-blk > > here. That's unexpected since they both use the QEMU block layer. If > > anything, I would have expected virtio

Re: [Qemu-devel] IO performance test on the tcm-vhost scsi

2012-06-14 Thread Nicholas A. Bellinger
On Thu, 2012-06-14 at 17:57 +0800, Cong Meng wrote: > On Wed, 2012-06-13 at 12:08 -0700, Nicholas A. Bellinger wrote: > > On Wed, 2012-06-13 at 18:13 +0800, mengcong wrote: > > > Hi folks, I did an IO performance test on the tcm-vhost scsi. I want to > > > share > > > the test result data here. >

Re: [Qemu-devel] q35 chipset support

2012-06-14 Thread Anthony Liguori
On 06/14/2012 02:54 PM, Jason Baron wrote: Hi, I recently updated Isaku Yamahata's q35 patches to work on the latest qemu and seabios trees. On the qemu side, most of the changes revolved around updating to use QOM and updates to the memory API. I was also able to drop quite a few patches that h

Re: [Qemu-devel] KVM internal error with some amounts of guest memory

2012-06-14 Thread Michael Tokarev
On 14.06.2012 23:45, Michael Tokarev wrote: > On 14.06.2012 23:22, Michael Tokarev wrote: >> Now that's something else. Reported by a debian user, but >> trivially reproducible. >> >> $ kvm -m 1.4g >> KVM internal error. Suberror: 1 >> emulation failure >> EAX=000e3c54 EBX= ECX= ED

Re: [Qemu-devel] [Xen-devel] [PATCH V13 4/9] pci.c: Add opaque argument to pci_for_each_device.

2012-06-14 Thread Konrad Rzeszutek Wilk
On Thu, Jun 14, 2012 at 06:01:44PM +0100, Anthony PERARD wrote: > The purpose is to have a more generic pci_for_each_device by passing an extra > argument to the function called on every device. > > This patch will be used in a next patch. > > Signed-off-by: Anthony PERARD > Acked-by: Stefano St

Re: [Qemu-devel] [Xen-devel] [PATCH V13 3/9] Introduce XenHostPCIDevice to access a pci device on the host.

2012-06-14 Thread Konrad Rzeszutek Wilk
On Thu, Jun 14, 2012 at 06:01:43PM +0100, Anthony PERARD wrote: > Signed-off-by: Anthony PERARD Reviewed-by: Konrad Rzeszutek Wilk > --- > hw/i386/Makefile.objs|1 + > hw/xen-host-pci-device.c | 396 > ++ > hw/xen-host-pci-device.h | 55 ++

Re: [Qemu-devel] [Xen-devel] [PATCH V13 1/9] pci_ids: Add INTEL_82599_SFP_VF id.

2012-06-14 Thread Konrad Rzeszutek Wilk
On Thu, Jun 14, 2012 at 06:01:41PM +0100, Anthony PERARD wrote: > We are using this in our quirk lookup provided by patch > titled: Introduce Xen PCI Passthrough, PCI config space helpers. > > Signed-off-by: Anthony PERARD > Acked-by: Stefano Stabellini Reviewed-by: Konrad Rzeszutek Wilk > ---

[Qemu-devel] q35 chipset support

2012-06-14 Thread Jason Baron
Hi, I recently updated Isaku Yamahata's q35 patches to work on the latest qemu and seabios trees. On the qemu side, most of the changes revolved around updating to use QOM and updates to the memory API. I was also able to drop quite a few patches that had already been resolved by the current qemu

Re: [Qemu-devel] KVM internal error with some amounts of guest memory

2012-06-14 Thread Michael Tokarev
On 14.06.2012 23:22, Michael Tokarev wrote: > Now that's something else. Reported by a debian user, but > trivially reproducible. > > $ kvm -m 1.4g > KVM internal error. Suberror: 1 > emulation failure > EAX=000e3c54 EBX= ECX= EDX=0cfd > ESI= EDI= EBP=

Re: [Qemu-devel] [PATCH V13 0/9] Xen PCI Passthrough

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 06:01:40PM +0100, Anthony PERARD wrote: > Hi all, > > This patch series introduces the PCI passthrough for Xen. > > Changes since the last version: > - New patch that introduce a new qdev-property pci-host-devaddr. > => the "export pci_parse_devaddr" patch is not anymo

Re: [Qemu-devel] [PATCH V13 8/9] Introduce apic-msidef.h

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 06:01:48PM +0100, Anthony PERARD wrote: > This patch move the msi definition from apic.c to apic-msidef.h. So it can be > used also by other .c files. > > Signed-off-by: Anthony PERARD > Acked-by: Stefano Stabellini Acked-by: Michael S. Tsirkin > --- > hw/apic-msidef.

Re: [Qemu-devel] [PATCH V13 5/9] qdev-properties: Introduce pci-host-devaddr.

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 06:01:45PM +0100, Anthony PERARD wrote: > This new property will be used to specify a host pci device address. > > Signed-off-by: Anthony PERARD Acked-by: Michael S. Tsirkin > --- > hw/qdev-properties.c | 107 > ++ > hw

Re: [Qemu-devel] [PATCH V13 4/9] pci.c: Add opaque argument to pci_for_each_device.

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 06:01:44PM +0100, Anthony PERARD wrote: > The purpose is to have a more generic pci_for_each_device by passing an extra > argument to the function called on every device. > > This patch will be used in a next patch. > > Signed-off-by: Anthony PERARD > Acked-by: Stefano St

Re: [Qemu-devel] [PATCH V13 1/9] pci_ids: Add INTEL_82599_SFP_VF id.

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 06:01:41PM +0100, Anthony PERARD wrote: > We are using this in our quirk lookup provided by patch > titled: Introduce Xen PCI Passthrough, PCI config space helpers. > > Signed-off-by: Anthony PERARD > Acked-by: Stefano Stabellini Acked-by: Michael S. Tsirkin > --- > h

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

2012-06-14 Thread Liu, Jinsong
Eduardo Habkost wrote: > On Thu, Jun 14, 2012 at 07:02:03PM +, Liu, Jinsong wrote: >> Eduardo, Jan >> >> I will update tsc deadline timer patch (at qemu-kvm side) recently. >> Have you made a final agreement of the issue >> 'KVM_CAP_TSC_DEADLINE_TIMER' vs. 'GET_SUPPORTED_CPUID'? > > I don't

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

2012-06-14 Thread Eduardo Habkost
On Thu, Jun 14, 2012 at 07:02:03PM +, Liu, Jinsong wrote: > Eduardo, Jan > > I will update tsc deadline timer patch (at qemu-kvm side) recently. > Have you made a final agreement of the issue 'KVM_CAP_TSC_DEADLINE_TIMER' vs. > 'GET_SUPPORTED_CPUID'? I don't think there's a final agreement, b

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

2012-06-14 Thread Liu, Jinsong
Eduardo, Jan I will update tsc deadline timer patch (at qemu-kvm side) recently. Have you made a final agreement of the issue 'KVM_CAP_TSC_DEADLINE_TIMER' vs. 'GET_SUPPORTED_CPUID'? Thanks, Jinsong Eduardo Habkost wrote: > (CCing Andre Przywara, in case he can help to clarify what's the > expe

[Qemu-devel] [PATCH v3 3/8] virtio: Convert to msix_init_exclusive_bar() interface

2012-06-14 Thread Alex Williamson
Simple conversion. Signed-off-by: Alex Williamson --- hw/virtio-pci.c | 15 +-- hw/virtio-pci.h |1 - 2 files changed, 5 insertions(+), 11 deletions(-) diff --git a/hw/virtio-pci.c b/hw/virtio-pci.c index 9342eed..3dca37f 100644 --- a/hw/virtio-pci.c +++ b/hw/virtio-pci.c @@

[Qemu-devel] [PATCH v3 7/8] msix: Allow full specification of MSIX layout

2012-06-14 Thread Alex Williamson
Finally, complete the fully specified interface. msix_add_config() gets folded into msix_init() because we now have quite a few parameters to pass and rolling it in let's us error earlier, avoiding the ugly unwind exit path. msix_mmio_setup() also gets rolled in, just because it's redundant to re

[Qemu-devel] [PATCH v3 6/8] msix: Split PBA into it's own MemoryRegion

2012-06-14 Thread Alex Williamson
These don't have to be contiguous. Size them to only what they need and use separate MemoryRegions for the vector table and PBA. Signed-off-by: Alex Williamson --- hw/msix.c | 106 +++-- hw/pci.h | 10 +++--- 2 files changed, 73 inser

[Qemu-devel] [PATCH v3 5/8] msix: Note endian TODO item

2012-06-14 Thread Alex Williamson
MSIX, like PCI, is little endian. Specifying native is wrong here, but we need to check the rest of the file to determine if it's as simple as flipping this macro. Signed-off-by: Alex Williamson --- hw/msix.c |1 + 1 file changed, 1 insertion(+) diff --git a/hw/msix.c b/hw/msix.c index 50

[Qemu-devel] [PATCH v6 2/3] Simpletrace v2: Support multiple arguments, strings.

2012-06-14 Thread Harsh Prateek Bora
Existing simpletrace backend allows to trace at max 6 args and does not support strings. This newer tracelog format gets rid of fixed size records and therefore allows to trace variable number of args including strings. Sample trace with strings: v9fs_version 0.000 tag=0x id=0x64 msize=0x2000

[Qemu-devel] [PATCH v6 0/3] Simpletrace v2: Support multiple args, strings.

2012-06-14 Thread Harsh Prateek Bora
Existing simpletrace backend allows to trace at max 6 args and does not support strings. This newer tracelog format gets rid of fixed size records and therefore allows to trace variable number of args including strings. Sample trace: v9fs_version 0.000 tag=0x id=0x64 msize=0x2000 version=9P200

[Qemu-devel] [PATCH v6 1/3] monitor: remove unused do_info_trace

2012-06-14 Thread Harsh Prateek Bora
Going forward with simpletrace v2 variable size trace records, we cannot have a generic function to print trace event info and therefore this interface becomes invalid. As per Stefan Hajnoczi: "This command is only available from the human monitor. It's not very useful because it historically ha

[Qemu-devel] [PATCH v6 3/3] Update simpletrace.py for new log format

2012-06-14 Thread Harsh Prateek Bora
Support new tracelog format for multiple arguments and strings. Signed-off-by: Harsh Prateek Bora --- scripts/simpletrace.py | 116 +++- 1 file changed, 75 insertions(+), 41 deletions(-) diff --git a/scripts/simpletrace.py b/scripts/simpletrace.py in

Re: [Qemu-devel] [PATCH 1/3] msi/msix: added functions to API to set up message address and data

2012-06-14 Thread Alex Williamson
On Thu, 2012-06-14 at 15:44 +1000, Alexey Kardashevskiy wrote: > On 14/06/12 15:38, Alex Williamson wrote: > > On Thu, 2012-06-14 at 15:17 +1000, Alexey Kardashevskiy wrote: > >> On 14/06/12 14:56, Alex Williamson wrote: > >>> On Thu, 2012-06-14 at 14:31 +1000, Alexey Kardashevskiy wrote: > No

Re: [Qemu-devel] CoW image commit+shrink(= make_empty) support

2012-06-14 Thread Supriya Kannery
On 06/14/2012 07:59 PM, Jeff Cody wrote: On 06/14/2012 10:23 AM, Zhi Hui Li wrote: On 2012年06月11日 23:37, Jeff Cody wrote: On 06/11/2012 10:24 AM, Stefan Hajnoczi wrote: On Mon, Jun 11, 2012 at 1:50 PM, Kevin Wolf wrote: Am 11.06.2012 14:09, schrieb Stefan Hajnoczi: On Fri, Jun 8, 2012 at 6

[Qemu-devel] [PATCH v3 8/8] msix: Switch msix_uninit to return void

2012-06-14 Thread Alex Williamson
It can't fail. Signed-off-by: Alex Williamson --- hw/msix.c |6 +++--- hw/msix.h |4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/hw/msix.c b/hw/msix.c index 15f8d7d..fd9ea95 100644 --- a/hw/msix.c +++ b/hw/msix.c @@ -328,10 +328,10 @@ static void msix_free_irq_en

[Qemu-devel] [PATCH v3 4/8] msix: Move msix_mmio_read

2012-06-14 Thread Alex Williamson
What's this doing so far from msix_mmio_ops? Signed-off-by: Alex Williamson --- hw/msix.c | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/hw/msix.c b/hw/msix.c index bafea94..50885ac 100644 --- a/hw/msix.c +++ b/hw/msix.c @@ -86,16 +86,6 @@ static in

[Qemu-devel] [PATCH v3 2/8] ivshmem: Convert to msix_init_exclusive_bar() interface

2012-06-14 Thread Alex Williamson
Trivial conversion, failed to have an uninit before and after. Signed-off-by: Alex Williamson --- hw/ivshmem.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/hw/ivshmem.c b/hw/ivshmem.c index 05559b6..8b49eee 100644 --- a/hw/ivshmem.c +++ b/hw/ivshmem.c @@ -70,

[Qemu-devel] [PATCH v3 1/8] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Alex Williamson
msi_init() takes over a BAR without really specifying or allowing specification of how it does so. Instead, let's split it into two interfaces, one fully specified, and one trivially easy. This implements the latter. msix_init_exclusive_bar() takes over allocating and filling a PCI BAR _exclusiv

[Qemu-devel] [PATCH v3 0/8] msix: Support specifying offsets, BARs, and capability location

2012-06-14 Thread Alex Williamson
v3: - more patches, smaller diff, must be headed in the right direction - macros for all hardcoded values in msix_init_exclusive_bar - fold msix_add_config into msix_init allowing less churn to moving around msix_uninit - note native endian bug - split msix_mmio_read move to separate patch

Re: [Qemu-devel] qemu-system-ppc64 hanging occasionally in disk writes

2012-06-14 Thread Richard W.M. Jones
On Thu, Jun 14, 2012 at 06:13:11PM +0100, Richard W.M. Jones wrote: > I will try this, but as discussed on IRC last week there's some > problem with the Fedora host kernel where /dev/kvm doesn't show up, > even though the kernel is supposedly compiled with KVM PR enabled. So > I need to fix that f

[Qemu-devel] [PATCH V13 0/9] Xen PCI Passthrough

2012-06-14 Thread Anthony PERARD
Hi all, This patch series introduces the PCI passthrough for Xen. Changes since the last version: - New patch that introduce a new qdev-property pci-host-devaddr. => the "export pci_parse_devaddr" patch is not anymore usefull. Thanks, Allen Kay (2): Introduce Xen PCI Passthrough, qdevice

[Qemu-devel] [PATCH V13 9/9] Introduce Xen PCI Passthrough, MSI (3/3)

2012-06-14 Thread Anthony PERARD
From: Jiang Yunhong A more complete history can be found here: git://xenbits.xensource.com/qemu-xen-unstable.git Signed-off-by: Jiang Yunhong Signed-off-by: Shan Haitao Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/i386/Makefile.objs |2 +- hw/xen_pt.c

Re: [Qemu-devel] [PATCH v5 2/3] Simpletrace v2: Support multiple arguments, strings.

2012-06-14 Thread Harsh Bora
[...] Okay I think, I got it, let me dig a little further ... +void trace_record_finish(TraceBufferRecord *rec) { - trace(event, x1, x2, x3, x4, x5, x6); + uint8_t temp_rec[sizeof(TraceRecord)]; + TraceRecord *record = (TraceRecord *) temp_rec; + read_from_buffer(rec->tbuf_idx, temp_rec, sizeof

[Qemu-devel] [PATCH V13 1/9] pci_ids: Add INTEL_82599_SFP_VF id.

2012-06-14 Thread Anthony PERARD
We are using this in our quirk lookup provided by patch titled: Introduce Xen PCI Passthrough, PCI config space helpers. Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/pci_ids.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/hw/pci_ids.h b/hw/pci_ids

[Qemu-devel] [PATCH V13 3/9] Introduce XenHostPCIDevice to access a pci device on the host.

2012-06-14 Thread Anthony PERARD
Signed-off-by: Anthony PERARD --- hw/i386/Makefile.objs|1 + hw/xen-host-pci-device.c | 396 ++ hw/xen-host-pci-device.h | 55 +++ 3 files changed, 452 insertions(+), 0 deletions(-) create mode 100644 hw/xen-host-pci-device.c create mod

[Qemu-devel] [PATCH V13 6/9] Introduce Xen PCI Passthrough, qdevice (1/3)

2012-06-14 Thread Anthony PERARD
From: Allen Kay A more complete history can be found here: git://xenbits.xensource.com/qemu-xen-unstable.git Signed-off-by: Allen Kay Signed-off-by: Guy Zana Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/i386/Makefile.objs |1 + hw/xen_common.h |3 + hw

[Qemu-devel] [PATCH V13 7/9] Introduce Xen PCI Passthrough, PCI config space helpers (2/3)

2012-06-14 Thread Anthony PERARD
From: Allen Kay A more complete history can be found here: git://xenbits.xensource.com/qemu-xen-unstable.git Signed-off-by: Allen Kay Signed-off-by: Guy Zana Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/xen_pt.c | 10 + hw/xen_pt.h |2 + hw

[Qemu-devel] [PATCH V13 4/9] pci.c: Add opaque argument to pci_for_each_device.

2012-06-14 Thread Anthony PERARD
The purpose is to have a more generic pci_for_each_device by passing an extra argument to the function called on every device. This patch will be used in a next patch. Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/pci.c | 11 +++ hw/pci.h |4

[Qemu-devel] [PATCH V13 8/9] Introduce apic-msidef.h

2012-06-14 Thread Anthony PERARD
This patch move the msi definition from apic.c to apic-msidef.h. So it can be used also by other .c files. Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini --- hw/apic-msidef.h | 30 ++ hw/apic.c| 11 +-- 2 files changed, 31 insertions(+

[Qemu-devel] [PATCH V13 2/9] configure: Introduce --enable-xen-pci-passthrough.

2012-06-14 Thread Anthony PERARD
Signed-off-by: Anthony PERARD Acked-by: Stefano Stabellini Reviewed-by: Konrad Rzeszutek Wilk --- configure | 29 + 1 files changed, 29 insertions(+), 0 deletions(-) diff --git a/configure b/configure index c2366ee..d734a03 100755 --- a/configure +++ b/configure @

[Qemu-devel] [PATCH V13 5/9] qdev-properties: Introduce pci-host-devaddr.

2012-06-14 Thread Anthony PERARD
This new property will be used to specify a host pci device address. Signed-off-by: Anthony PERARD --- hw/qdev-properties.c | 107 ++ hw/qdev.h|3 + qemu-common.h|7 +++ 3 files changed, 117 insertions(+), 0 deletions(-

[Qemu-devel] [PATCH v3 2/2] Add rate limiting of RTC_CHANGE, BALLOON_CHANGE & WATCHDOG events

2012-06-14 Thread Daniel P. Berrange
From: "Daniel P. Berrange" Allow certain event types to be rate limited to avoid flooding monitor clients. The monitor_protocol_event() method is changed such that instead of immediately emitting the event to Monitor instances, it will call a new monitor_protocol_event_queue() method. This will

Re: [Qemu-devel] qemu-system-ppc64 hanging occasionally in disk writes

2012-06-14 Thread Richard W.M. Jones
On Thu, Jun 14, 2012 at 05:58:04PM +0200, Alexander Graf wrote: > [CC'ing qemu-ppc] > > On 06/14/2012 05:52 PM, Richard W.M. Jones wrote: > >I found last week that qemu-system-ppc64 (from git) hangs occasionally > >under load, and I have a reproducer for it now. Unfortunately the > >reproducer re

[Qemu-devel] [PATCH v3 1/2] Add event notification for guest balloon changes

2012-06-14 Thread Daniel P. Berrange
From: "Daniel P. Berrange" After setting a balloon target value, applications have to continually poll 'query-balloon' to determine whether the guest has reacted to this request. The virtio-balloon backend knows exactly when the guest has reacted though, and thus it is possible to emit a JSON eve

[Qemu-devel] [PATCH v3 0/2] Balloon event change notifications

2012-06-14 Thread Daniel P. Berrange
This is a followup to: https://lists.gnu.org/archive/html/qemu-devel/2012-05/msg02833.html Changes since v2: - Drop 'query-events' patch which is already merged - Rename balloon_change() function to balloon_changed per Amit's request - Misc typos fixed - Remove monitor_global_init

Re: [Qemu-devel] How to measure guest memory access (qemu_ld/qemu_st) time?

2012-06-14 Thread YeongKyoon Lee
Hi, I proposed QEMU ld/st optimizations two times, late last year and early this year, however, there was no replies and even more 0/3 patch mail in this year has been missed so I cannot find the mail in QEMU mail archives. :( The link of 0/3 (http://www.mail-archive.com/qemu-devel@nongnu.org/m

Re: [Qemu-devel] [PATCH v2 1/6] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 06:11:18PM +0200, Jan Kiszka wrote: > On 2012-06-14 18:05, Michael S. Tsirkin wrote: > > On Thu, Jun 14, 2012 at 05:55:18PM +0200, Jan Kiszka wrote: > >> On 2012-06-14 17:49, Michael S. Tsirkin wrote: > >>> On Wed, Jun 13, 2012 at 10:51:19PM -0600, Alex Williamson wrote: > >

Re: [Qemu-devel] [PATCH v2 1/6] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 10:10:40AM -0600, Alex Williamson wrote: > On Thu, 2012-06-14 at 19:05 +0300, Michael S. Tsirkin wrote: > > On Thu, Jun 14, 2012 at 05:55:18PM +0200, Jan Kiszka wrote: > > > On 2012-06-14 17:49, Michael S. Tsirkin wrote: > > > > On Wed, Jun 13, 2012 at 10:51:19PM -0600, Alex

Re: [Qemu-devel] [PATCH v2 4/6] msix: Split PBA into it's own MemoryRegion

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 10:02:58AM -0600, Alex Williamson wrote: > On Thu, 2012-06-14 at 18:45 +0300, Michael S. Tsirkin wrote: > > On Thu, Jun 14, 2012 at 09:09:47AM -0600, Alex Williamson wrote: > > > On Thu, 2012-06-14 at 17:50 +0300, Michael S. Tsirkin wrote: > > > > On Thu, Jun 14, 2012 at 08:

[Qemu-devel] [Bug 1013241] [NEW] qemu-system-ppc64 hanging occasionally in disk writes

2012-06-14 Thread Richard W.M. Jones
Public bug reported: I found last week that qemu-system-ppc64 (from git) hangs occasionally under load, and I have a reproducer for it now. Unfortunately the reproducer really takes a long time to run -- usually I can get a hang in under 12 hours.

Re: [Qemu-devel] [PATCH v2 1/6] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Jan Kiszka
On 2012-06-14 18:05, Michael S. Tsirkin wrote: > On Thu, Jun 14, 2012 at 05:55:18PM +0200, Jan Kiszka wrote: >> On 2012-06-14 17:49, Michael S. Tsirkin wrote: >>> On Wed, Jun 13, 2012 at 10:51:19PM -0600, Alex Williamson wrote: msi_init() takes over a BAR without really specifying or allowing

Re: [Qemu-devel] [PATCH v2 1/6] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Alex Williamson
On Thu, 2012-06-14 at 19:05 +0300, Michael S. Tsirkin wrote: > On Thu, Jun 14, 2012 at 05:55:18PM +0200, Jan Kiszka wrote: > > On 2012-06-14 17:49, Michael S. Tsirkin wrote: > > > On Wed, Jun 13, 2012 at 10:51:19PM -0600, Alex Williamson wrote: > > >> msi_init() takes over a BAR without really spec

Re: [Qemu-devel] [PATCH v2 1/6] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Michael S. Tsirkin
On Thu, Jun 14, 2012 at 05:55:18PM +0200, Jan Kiszka wrote: > On 2012-06-14 17:49, Michael S. Tsirkin wrote: > > On Wed, Jun 13, 2012 at 10:51:19PM -0600, Alex Williamson wrote: > >> msi_init() takes over a BAR without really specifying or allowing > >> specification of how it does so. Instead, le

Re: [Qemu-devel] [PATCH v2 4/6] msix: Split PBA into it's own MemoryRegion

2012-06-14 Thread Alex Williamson
On Thu, 2012-06-14 at 18:45 +0300, Michael S. Tsirkin wrote: > On Thu, Jun 14, 2012 at 09:09:47AM -0600, Alex Williamson wrote: > > On Thu, 2012-06-14 at 17:50 +0300, Michael S. Tsirkin wrote: > > > On Thu, Jun 14, 2012 at 08:21:39AM -0600, Alex Williamson wrote: > > > > On Thu, 2012-06-14 at 13:24

Re: [Qemu-devel] [PATCH V12 5/9] Revert "pci: don't export an internal function"

2012-06-14 Thread Anthony PERARD
On 14/06/12 16:21, Jan Kiszka wrote: Is this patch would be a good common property? It's probably close to the patch you send earlier Jan. If it's good, I'll resend my passthrough patch series with this patch. Looks generally OK to me, let's try to move forward with this. Please fix the coding

Re: [Qemu-devel] [PATCH V12 5/9] Revert "pci: don't export an internal function"

2012-06-14 Thread Anthony PERARD
On 14/06/12 16:30, Michael S. Tsirkin wrote: > Is this patch would be a good common property? > > It's probably close to the patch you send earlier Jan. > > If it's good, I'll resend my passthrough patch series with this patch. > > > diff --git a/hw/qdev-properties.c b/hw/qdev-properties.c >

Re: [Qemu-devel] qemu-system-ppc64 hanging occasionally in disk writes

2012-06-14 Thread Alexander Graf
[CC'ing qemu-ppc] On 06/14/2012 05:52 PM, Richard W.M. Jones wrote: I found last week that qemu-system-ppc64 (from git) hangs occasionally under load, and I have a reproducer for it now. Unfortunately the reproducer really takes a long time to run -- usually I can get a hang in under 12 hours.

Re: [Qemu-devel] [PATCH buildfix] configure: Assure existence of linux-headers/ build directory

2012-06-14 Thread Andreas Färber
Am 14.06.2012 17:14, schrieb Andreas Färber: > Commit ec5b06d (configure: ensure directory exists when creating symlinks) > moved the creation of directories into the symlink() function but forgot > the case where no symlink is created. > > This leads to build errors on arm Linux due to -I../linux

[Qemu-devel] [PATCH v3 3/5] osdep: Enable qemu_open to dup pre-opened fd

2012-06-14 Thread Corey Bryant
This patch adds support to qemu_open to dup(fd) a pre-opened file descriptor if the filename is of the format /dev/fd/X. This can be used when QEMU is restricted from opening files, and the management application opens files on QEMU's behalf. If the fd was passed to the monitor with the pass-fd c

Re: [Qemu-devel] [PATCH v2 1/6] msix: Add simple BAR allocation MSIX setup functions

2012-06-14 Thread Jan Kiszka
On 2012-06-14 17:49, Michael S. Tsirkin wrote: > On Wed, Jun 13, 2012 at 10:51:19PM -0600, Alex Williamson wrote: >> msi_init() takes over a BAR without really specifying or allowing >> specification of how it does so. Instead, let's split it into >> two interfaces, one fully specified, and one tr

[Qemu-devel] [PATCH v3 5/5] block: Prevent /dev/fd/X filename from being detected as floppy

2012-06-14 Thread Corey Bryant
Reported-by: Kevin Wolf Reviewed-by: Paolo Bonzini Signed-off-by: Corey Bryant --- v3: -This patch is new in v3. It was previously submitted on its own, and is now being included in this series. block/raw-posix.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/blo

[Qemu-devel] [PATCH v3 4/5] block: Convert open calls to qemu_open

2012-06-14 Thread Corey Bryant
This patch converts all block layer open calls to qemu_open. This enables all block layer open paths to dup(X) a pre-opened file descriptor if the filename is of the format /dev/fd/X. This is useful if QEMU is restricted from opening certain files. Note that this adds the O_CLOEXEC flag to the c

[Qemu-devel] [PATCH v3 2/5] qapi: Add pass-fd QMP command

2012-06-14 Thread Corey Bryant
This patch adds the pass-fd QMP command using the QAPI framework. Like the getfd command, it is used to pass a file descriptor via SCM_RIGHTS. However, the pass-fd command also returns the received file descriptor, which is a difference in behavior from the getfd command, which returns nothing. T

  1   2   3   >