Re: [Qemu-devel] Cirrus bugs vs endian: how two bugs cancel each other out

2012-08-06 Thread Anthony Liguori
proving stdvga to support basic 2d accel isn't that much effort. I > think it is worth doing it, even when it is obsoleted by a redesigned / > rewritten qxl2 some day. I think we're pretty much in agreement here. I think if we improve stdvga by making it a virtio device, then it becomes pretty easy to eventually make it qxl2. Regards, Anthony Liguori > > cheers, > Gerd

Re: [Qemu-devel] [PATCH] virtio: fix vhost handling

2012-08-06 Thread Anthony Liguori
for irqfd: add a parameter to > virtio_queue_set_host_notifier_fd_handler and in that case only set > the notifier, not the handler. > > Cc: Stefan Hajnoczi > Signed-off-by: Paolo Bonzini Applied. Thanks. Regards, Anthony Liguori > --- > hw/virtio-pci.c | 14 +++--- > hw/virtio.c | 7 +-

[Qemu-devel] [PATCH] slirp: fix build on mingw32

2012-08-06 Thread Anthony Liguori
in_addr_t isn't available on mingw32. Just use an unsigned long instead. I considered typedef'ing in_addr_t on mingw32 but this would potentially be brittle if mingw32 did introduce the type. Cc: Jan Kiszka Signed-off-by: Anthony Liguori --- slirp/main.h |2 +- slirp/slir

Re: [Qemu-devel] [PULL 00/12] Block patches

2012-08-07 Thread Anthony Liguori
Kevin Wolf writes: > The following changes since commit 26b9b5fe17cc1b6be2e8bf8b9d16094f420bb8ad: > > virtio: fix vhost handling (2012-08-06 14:01:44 -0500) > > are available in the git repository at: > http://repo.or.cz/r/qemu/kevin.git for-anthony Pulled. Thanks.

Re: [Qemu-devel] [PULL 0/2] usb patch queue

2012-08-07 Thread Anthony Liguori
Gerd Hoffmann writes: > Hi, > > This is the usb patch queue, pretty small this time, > carrying a little fix for usb-storage. > > please pull, > Gerd Pulled. Thanks. Regards, Anthony Liguori > > Gerd Hoffmann (2): > usb-storage: improve debu

Re: [Qemu-devel] [PATCH 4/6] migration: remove iohandlers before closing the file

2012-08-07 Thread Anthony Liguori
Paolo Bonzini writes: > This will be needed as soon as process_incoming_migration will set > handlers on the file. The patch may be removed if ...? Regards, Anthony Liguori > > Signed-off-by: Paolo Bonzini > --- > savevm.c | 3 +++ > 1 file modificato, 3 inserzion

Re: [Qemu-devel] For all targets and machine types: "start to monitor" smoke test

2012-08-07 Thread Anthony Liguori
on > > qemu-system-ppcemb mac99 > qemu-system-ppcemb g3beige > qemu-system-ppcemb prep > Unable to find PowerPC CPU definition > > qemu-system-xtensaeb lx60 > qemu-system-xtensaeb lx200 > qemu-system-xtensaeb sim > Unable to find CPU definition

Re: [Qemu-devel] [PATCH 2/2] pseries: Use new hook to correct reset sequence

2012-08-07 Thread Anthony Liguori
to push that! :) > I will be without a proper email client for 24 hours so I'll keep this brief for now. What Ben et al are trying to model is something that exists outside of the model of virtual hardware that QOM is designed for. Since they are trying to model something that exists

Re: [Qemu-devel] [SeaBIOS] [PULL 0/1] update qemu seabios / seabios release?

2012-08-10 Thread Anthony Liguori
esting. > - roll out seabios release by end of august, so it can be included > into qemu 1.2 We really would need a SeaBIOS release by no later than August 21st to do this. The 1.2 release is on September 1st but we're in hard freeze starting on August 15th. I wouldn't wan

Re: [Qemu-devel] [PATCH v3] Support 'help' as a synonym for '?' in command line options

2012-08-10 Thread Anthony Liguori
Eduardo Habkost writes: > On Thu, Aug 09, 2012 at 10:02:22PM +0100, Peter Maydell wrote: >> On 9 August 2012 20:25, Eduardo Habkost wrote: >> > On Fri, Aug 03, 2012 at 03:42:39PM -0500, Anthony Liguori wrote: >> >> Peter Maydell writes: >> >> > For

Re: [Qemu-devel] [PATCH 1/7] qmp: introduce device-list-properties command

2012-08-10 Thread Anthony Liguori
Luiz Capitulino writes: > On Fri, 27 Jul 2012 08:37:13 -0500 > Anthony Liguori wrote: > >> This can be used in conjunction with qom-list-types to determine the >> supported >> set of devices and their parameters. >> >> Signed-off-by: Anthony Ligu

Re: [Qemu-devel] [PATCH 2/7] qapi: mark QOM commands stable

2012-08-10 Thread Anthony Liguori
Luiz Capitulino writes: > On Fri, 27 Jul 2012 08:37:14 -0500 > Anthony Liguori wrote: > >> We've had a cycle to tweak. It is time to commit to supporting them. > > qmp_qom_get() and qpm_qom_set() still use the legacy monitor interface, can't > we convert it

Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-08-10 Thread Anthony Liguori
Luiz Capitulino writes: > On Fri, 27 Jul 2012 08:37:15 -0500 > Anthony Liguori wrote: > >> This provides the same output as -M ? but in a structured way. >> >> Signed-off-by: Anthony Liguori >> --- >> qapi-schema.json | 28 +

Re: [Qemu-devel] [PATCH 6/7] target-i386: add implementation of query-cpudefs

2012-08-10 Thread Anthony Liguori
Eduardo Habkost writes: > On Fri, Jul 27, 2012 at 08:37:18AM -0500, Anthony Liguori wrote: >> Signed-off-by: Anthony Liguori >> --- >> target-i386/cpu.c | 22 ++ >> 1 files changed, 22 insertions(+), 0 deletions(-) >> >> diff --git

Re: [Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-08-10 Thread Anthony Liguori
Luiz Capitulino writes: > On Fri, 10 Aug 2012 09:41:20 -0500 > Anthony Liguori wrote: > >> Luiz Capitulino writes: >> >> > On Fri, 27 Jul 2012 08:37:15 -0500 >> > Anthony Liguori wrote: >> > >> >> This provides the same output

[Qemu-devel] [PATCH 4/7] compiler: add macro for GCC weak symbols

2012-08-10 Thread Anthony Liguori
This lets us provide a default implementation of a symbol which targets can override. Signed-off-by: Anthony Liguori --- compiler.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/compiler.h b/compiler.h index 736e770..f76921e 100644 --- a/compiler.h +++ b/compiler.h

[Qemu-devel] [PATCH 1/7] qmp: introduce device-list-properties command

2012-08-10 Thread Anthony Liguori
This can be used in conjunction with qom-list-types to determine the supported set of devices and their parameters. Signed-off-by: Anthony Liguori --- qapi-schema.json | 28 qmp-commands.hx |7 +++ qmp.c| 50

[Qemu-devel] [PATCH 5/7] qapi: add query-cpu-definitions command (v2)

2012-08-10 Thread Anthony Liguori
implement this command if it makes sense for them. Signed-off-by: Anthony Liguori --- v1 -> v2 - rename query-cpudefs -> query-cpu-definitions --- qapi-schema.json | 23 +++ qmp-commands.hx |6 ++ qmp.c|6 ++ 3 files changed, 35 inse

[Qemu-devel] [PATCH 7/7] target-ppc: add implementation of query-cpu-definitions (v2)

2012-08-10 Thread Anthony Liguori
Signed-off-by: Anthony Liguori --- v1 -> v2 - rename query-cpudefs -> query-cpu-definitions --- target-ppc/translate_init.c | 26 ++ 1 files changed, 26 insertions(+), 0 deletions(-) diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c index 5

Re: [Qemu-devel] [PATCH 6/7] target-i386: add implementation of query-cpudefs

2012-08-10 Thread Anthony Liguori
Eduardo Habkost writes: > On Fri, Aug 10, 2012 at 09:43:21AM -0500, Anthony Liguori wrote: >> Eduardo Habkost writes: >> >> > On Fri, Jul 27, 2012 at 08:37:18AM -0500, Anthony Liguori wrote: >> >> Signed-off-by: Anthony Liguori >&

[Qemu-devel] [PATCH 3/7] qapi: add query-machines command

2012-08-10 Thread Anthony Liguori
This provides the same output as -M ? but in a structured way. Signed-off-by: Anthony Liguori --- qapi-schema.json | 28 qmp-commands.hx |6 ++ vl.c | 31 +++ 3 files changed, 65 insertions(+), 0 deletions

[Qemu-devel] [PATCH 2/7] qapi: mark QOM commands stable

2012-08-10 Thread Anthony Liguori
We've had a cycle to tweak. It is time to commit to supporting them. Signed-off-by: Anthony Liguori --- qapi-schema.json | 19 --- 1 files changed, 4 insertions(+), 15 deletions(-) diff --git a/qapi-schema.json b/qapi-schema.json index 191a889..a938c8d 100644 --- a

[Qemu-devel] [PATCH 0/7] qapi: add commands to remove the need (v2)

2012-08-10 Thread Anthony Liguori
This series implements the necessary commands to implements danpb's idea to remove -help parsing in libvirt. We would introduce all of these commands in 1.2 and then change the -help output starting in 1.3. Here is Dan's plan from a previous thread: Basically I'd sum up my new idea as "just u

[Qemu-devel] [PATCH 0/2] cpu: make a child of DeviceState

2012-08-10 Thread Anthony Liguori
This is just a rebase of this series that I posted back in June. Andreas had a different approach involving a #define but I think doing a full split ends up being nicer. For instance, the recently added thread information is only relevant to cpu-softmmu so we can limit that entirely to cpu-softmm

[Qemu-devel] [PATCH 1/2] qdev: split up header so it can be used in cpu.h

2012-08-10 Thread Anthony Liguori
y splitting out qdev.h along the same lines that we previously split the C file. Then cpu.h just needs to include qdev-core.h Signed-off-by: Anthony Liguori Signed-off-by: Anthony Liguori --- hw/irq.h |2 + hw/mc146818rtc.c |1 + hw/qdev-addr.c |1 + hw/qdev-core

[Qemu-devel] [PATCH 2/2] cpu: for cpu-user and cpu-softmmu and make cpu-softmmu a child of DeviceState

2012-08-10 Thread Anthony Liguori
volve independently. We also need to push a qdev_init_nofail() into all callers of cpu_init(). This is needed eventually anyway so might as well do this now. Signed-off-by: Anthony Liguori --- Makefile.objs |6 --- hw/armv7m.c |1 + hw/axis_de

Re: [Qemu-devel] [PATCH 6/7] target-i386: add implementation of query-cpudefs

2012-08-10 Thread Anthony Liguori
Eduardo Habkost writes: > On Fri, Aug 10, 2012 at 11:37:30AM -0500, Anthony Liguori wrote: >> Eduardo Habkost writes: >> >> > - add machine-type-specific cpudef compatibility changes? >> >> >> >> I think we've discussed this in IRC. I don&#

[Qemu-devel] [PATCH 6/7] target-i386: add implementation of query-cpu-definitions (v2)

2012-08-10 Thread Anthony Liguori
Signed-off-by: Anthony Liguori --- v1 -> v2 - rename query-cpudefs -> query-cpu-definitions --- target-i386/cpu.c | 22 ++ 1 files changed, 22 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.c b/target-i386/cpu.c index 880cfea..6d5d0d6 100644 --- a/targe

Re: [Qemu-devel] [PATCH 2/7] qapi: mark QOM commands stable

2012-08-10 Thread Anthony Liguori
Eric Blake writes: > On 08/10/2012 10:04 AM, Anthony Liguori wrote: >> We've had a cycle to tweak. It is time to commit to supporting them. >> >> Signed-off-by: Anthony Liguori >> --- >> qapi-schema.json | 19 --- >> 1

Re: [Qemu-devel] [PATCH 2/2] cpu: for cpu-user and cpu-softmmu and make cpu-softmmu a child of DeviceState

2012-08-10 Thread Anthony Liguori
Eduardo Habkost writes: > On Fri, Aug 10, 2012 at 12:00:44PM -0500, Anthony Liguori wrote: >> The line between linux-user and softmmu is not very well defined right now. >> linux-user really don't want to include devices and making CpuState a child >> of >> D

Re: [Qemu-devel] github mirror still stale

2012-08-11 Thread Anthony Liguori
ithub and which let me setup a mirror from qemu.org that pushes every 30 minutes to github. It should be active now. I can also add other people to the github organization. Regards, Anthony Liguori > > -- PMM

Re: [Qemu-devel] [PULL 0/7] last SCSI changes for 1.2

2012-08-11 Thread Anthony Liguori
i-next > > for you to fetch changes up to 5222aaf223e52961cabeb7cabc579892ccd8bc59: > > scsi-disk: add support for the UNMAP command (2012-08-09 15:04:09 +0200) Pulled. Thanks. Regards, Anthony Liguori > > &g

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

2012-08-12 Thread Anthony Liguori
ny Pulled. Thanks. Regards, Anthony Liguori > > Avi Kivity (1): > virtio-blk: fix use-after-free while handling scsi commands > > Jason Baron (2): > ahci: Fix ahci cdrom read corruptions for reads > 128k > ahci: Fix sglist memleak in ahci_dma_rw_buf() > >

Re: [Qemu-devel] [PULL 0/3] Trivial patches for 3 to 10 August 2012

2012-08-12 Thread Anthony Liguori
l-patches > > for you to fetch changes up to b90372ad2a69a9cdad2a40766eb46f0a89d98535: > > target-arm: Fix typos in comments (2012-08-10 14:37:28 +0100) Pulled. Thanks. Regards, Anthony Liguori > > > Dunrong H

Re: [Qemu-devel] [PATCH 05/11] virtio-blk: support VIRTIO_BLK_F_CONFIG_WCE

2012-08-12 Thread Anthony Liguori
1/01/2007 /sys/class/dmi/id/bios_version=Bochs Guest fingerprint changed for pc-1.0! Need to make this a qdev-visible feature and then add compatibility properties for all of the old machine types. Regards, Anthony Liguori > --- > hw/virtio-blk.c | 16 ++-- > hw/virti

Re: [Qemu-devel] [PATCH for-1.2 v5 14/14] pci: Tidy up PCI host bridges

2012-08-13 Thread Anthony Liguori
r_pci.c > @@ -36,16 +36,18 @@ static PCIDevice *find_dev(sPAPREnvironment *spapr, > uint64_t buid, uint32_t config_addr) > { > int devfn = (config_addr >> 8) & 0xFF; > -sPAPRPHBState *phb; > +sPAPRPHBState *sphb; > > - QLIST

Re: [Qemu-devel] [PATCH 0/3] VFIO-based PCI device assignment for QEMU 1.2

2012-08-13 Thread Anthony Liguori
a few Reviewed-by's too for this series before it goes in. I expect they won't be hard to get but I also expect it will take a few more revisions of this series to get there. Regards, Anthony Liguori > > This series is also available at: > > git://github.com/awill

Re: [Qemu-devel] [PATCH 4/7] compiler: add macro for GCC weak symbols

2012-08-13 Thread Anthony Liguori
Peter Maydell writes: > On 10 August 2012 17:04, Anthony Liguori wrote: >> This lets us provide a default implementation of a symbol which targets can >> override. >> >> Signed-off-by: Anthony Liguori > > I'm sure you'll be thrilled to hear tha

Re: [Qemu-devel] [PATCH for-1.2 v5 14/14] pci: Tidy up PCI host bridges

2012-08-13 Thread Anthony Liguori
at up this afternoon and send it out today. We can debate the merits of this sort of thing in the style guide. Regards, Anthony Liguori > > --> > > From: Andreas Färber > > piix: minor code simplification > > There's no need to deal with qdev internals in piix

Re: [Qemu-devel] Funny -m arguments can crash

2012-08-13 Thread Anthony Liguori
to ask for < 1MB unless they know what they're doing. If it's truly a mistake, then asking for 2MB is just as much of a mistake because no real guest will run with 2MB of memory anyway (you can't even load a kernel). So if we're just going for theoritical correctness, we ought to do it the Right Way which is fixing the CMOS values and putting the check in SeaBIOS. Regards, Anthony Liguori

Re: [Qemu-devel] [PATCH] update-linux-headers.sh: Pull in asm-generic/kvm_para.h

2012-08-13 Thread Anthony Liguori
guys pick this up through uq/master? Regards, Anthony Liguori > > On 8 August 2012 13:34, Peter Maydell wrote: >> Ping? >> >> patchwork url: http://patchwork.ozlabs.org/patch/173202/ >> >> -- PMM >> >> On 25 July 2012 16:29, Peter Maydell wr

[Qemu-devel] [PATCH] qom: add style guide

2012-08-13 Thread Anthony Liguori
Signed-off-by: Anthony Liguori --- docs/qom-style-guide.md | 489 +++ 1 files changed, 489 insertions(+), 0 deletions(-) create mode 100644 docs/qom-style-guide.md diff --git a/docs/qom-style-guide.md b/docs/qom-style-guide.md new file mode 100644

Re: [Qemu-devel] [PATCH v6 0/3] Sandboxing Qemu guests with Libseccomp

2012-08-13 Thread Anthony Liguori
lt for sandbox.enable=off but for 1.3, will switch it to 'on' for the default. By making it QemuOpts, we can actually disable/enable through the global configuration file which is a really nice touch. If you can make those changes and resubmit tomorrow, we can include it for the 1.2 release

Re: [Qemu-devel] [PATCH 0/3] VFIO-based PCI device assignment for QEMU 1.2

2012-08-13 Thread Anthony Liguori
Jan Kiszka writes: > On 2012-08-13 15:58, Avi Kivity wrote: >> On 08/13/2012 04:27 PM, Anthony Liguori wrote: >> >>> Thanks for pushing this forward! Hopefully this will finally kill off >>> qemu-kvm.git for good. >> >> No, it won't. vfio re

Re: [Qemu-devel] [PATCH 0/3] VFIO-based PCI device assignment for QEMU 1.2

2012-08-13 Thread Anthony Liguori
Alex Williamson writes: > On Mon, 2012-08-13 at 08:27 -0500, Anthony Liguori wrote: >> Alex Williamson writes: >> >> > VFIO kernel support was just merged into Linux, so I'd like to >> > formally propose inclusion of the QEMU vfio-pci driver for >> &

Re: [Qemu-devel] [PATCH 0/7] qapi: add commands to remove the need (v2)

2012-08-13 Thread Anthony Liguori
Luiz Capitulino writes: > On Mon, 13 Aug 2012 14:16:58 -0300 > Luiz Capitulino wrote: > >> On Fri, 10 Aug 2012 11:04:08 -0500 >> Anthony Liguori wrote: >> >> > This series implements the necessary commands to implements danpb's idea to >> > re

Re: [Qemu-devel] [PATCH] qom: add style guide

2012-08-13 Thread Anthony Liguori
"Michael S. Tsirkin" writes: > On Mon, Aug 13, 2012 at 01:46:46PM -0500, Anthony Liguori wrote: >> +typedef struct MyType MyType; >> + >> +struct MyType >> +{ > > This seems to violate our style:should be > >> +struct MyType

Re: [Qemu-devel] [PULL] kvm updates

2012-08-13 Thread Anthony Liguori
Avi Kivity writes: > Please pull from: > > git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master Pulled. Thanks. Regards, Anthony Liguori > > This is mostly Peter's kvm_irqchip_in_kern

Re: [Qemu-devel] [PATCH 3/3] vfio: Enable vfio-pci and mark supported

2012-08-13 Thread Anthony Liguori
t;> @@ -824,6 +826,10 @@ for opt do >>;; >>--disable-guest-agent) guest_agent="no" >>;; >> + --disable-vfio-pci) vfio_pci="no" >> + ;; >> + --enable-vfio-pci) vfio_pci="yes" >> + ;; > > Do we need thi

Re: [Qemu-devel] [PULL 00/48]: QMP queue

2012-08-13 Thread Anthony Liguori
ry: > > git://repo.or.cz/qemu/qmp-unstable.git queue/qmp > Pulled. Thanks. Regards, Anthony Liguori > Anthony Liguori (7): > qmp: introduce device-list-properties command > qapi: mark QOM commands stable > qapi: add query-machines command > c

Re: [Qemu-devel] [PULL] Migration next 20120808

2012-08-13 Thread Anthony Liguori
Juan Quintela writes: > [Resent with PULL request instead of PATCH, sorry for the inconvenience] > > Hi > > This includes xbzrle series (v12) That were reviewed by Erik and Luiz. > > Anthony, please apply. Pulled. Thanks. Regards, Anthony Liguori > > Later, Juan

Re: [Qemu-devel] [PULL for-1.2 00/10] arm-devs queue

2012-08-13 Thread Anthony Liguori
t; thanks > -- PMM > Pulled. Thanks. Regards, Anthony Liguori > The following changes since commit 33e95c6328a3149a52615176617997c4f8f7088b: > > qom: Reimplement Interfaces (2012-08-13 11:20:41 +0200) > > are available in the git repository at: > > git://git.linaro

Re: [Qemu-devel] [RFC v0] HACK: qom: object_property_set: abort on failure

2012-08-14 Thread Anthony Liguori
ly is) or should it fail > gracefully? Are you calling set via object_property_set_[str,int,bool,...]? Why don't you add a terminal error to those functions if setting fails and errp is NULL. Regards, Anthony Liguori > > Whats the best meachinism for creating a no_fail version of

[Qemu-devel] [PATCH] block-migration: deprecate block migration for the 1.2 release

2012-08-14 Thread Anthony Liguori
To be replaced with live block copy. Signed-off-by: Anthony Liguori --- migration.c |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/migration.c b/migration.c index 653a3c1..babccf4 100644 --- a/migration.c +++ b/migration.c @@ -482,10 +482,19 @@ void qmp_migrate

Re: [Qemu-devel] [PULL 0/6 1.2] Tracing patches for QEMU 1.2

2012-08-14 Thread Anthony Liguori
Stefan Hajnoczi writes: > The last pull request from 19 July was not merged. Here it is rebased on > qemu.git/master with two additional fixes from Stefan Weil. I'm not sure what you mean: commit 903f650b0c77827f8d92b35f61419401d648df1e Merge: 61dc008 90a147a Author: Anthony Li

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-14 Thread Anthony Liguori
a "panic-device" should cover other OSes is also > something to consider. Even for Linux, is "panic" the only case which > should be reported via the mechanism? What about oopses without panic? > > Is the mechanism general enough for supporting new events, etc. Hi, I

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-14 Thread Anthony Liguori
Marcelo Tosatti writes: > On Tue, Aug 14, 2012 at 01:53:01PM -0500, Anthony Liguori wrote: >> Marcelo Tosatti writes: >> >> > On Tue, Aug 14, 2012 at 05:55:54PM +0300, Yan Vugenfirer wrote: >> >> >> >> On Aug 14, 2012, at 1:42 PM, Jan Kiszka wro

Re: [Qemu-devel] [PATCH] block-migration: deprecate block migration for the 1.2 release

2012-08-14 Thread Anthony Liguori
Luiz Capitulino writes: > On Tue, 14 Aug 2012 08:32:31 -0500 > Anthony Liguori wrote: > >> To be replaced with live block copy. >> >> Signed-off-by: Anthony Liguori >> --- >> migration.c |9 + >> 1 files changed, 9 insertions(+), 0 d

Re: [Qemu-devel] [PATCH 3/4] s390/kvm: Add a channel I/O based virtio transport driver.

2012-08-14 Thread Anthony Liguori
Huck Hi, Have you written an appendix for the virtio specification for virtio-ccw? I think it would be good to include in this series for the purposes of review. Regards, Anthony Liguori > --- > arch/s390/include/asm/irq.h | 1 + > arch/s390/kernel/irq.c| 1 + > dr

Re: [Qemu-devel] [PATCH v8] kvm: notify host when the guest is panicked

2012-08-14 Thread Anthony Liguori
Marcelo Tosatti writes: > On Tue, Aug 14, 2012 at 02:35:34PM -0500, Anthony Liguori wrote: >> Marcelo Tosatti writes: >> >> > On Tue, Aug 14, 2012 at 01:53:01PM -0500, Anthony Liguori wrote: >> >> Marcelo Tosatti writes: >> >> >> >>

[Qemu-devel] [PATCH] win32: provide separate macros for weak decls and definitions

2012-08-14 Thread Anthony Liguori
mingw32 seems to want the declaration to also carry the weak attribute. Strangely, gcc on Linux absolutely does not want the declaration to be marked as weak. This may not be the right fix, but it seems to do the trick. Signed-off-by: Anthony Liguori --- arch_init.h |4

Re: [Qemu-devel] [PATCH 2/2] cpu: for cpu-user and cpu-softmmu and make cpu-softmmu a child of DeviceState

2012-08-15 Thread Anthony Liguori
Andreas Färber writes: > Am 10.08.2012 19:00, schrieb Anthony Liguori: >> The line between linux-user and softmmu is not very well defined right now. >> linux-user really don't want to include devices and making CpuState a child >> of >> DeviceState would re

Re: [Qemu-devel] [PATCH 0/4] [PULL] qemu-kvm.git uq/master queue

2012-08-15 Thread Anthony Liguori
t; > git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git uq/master Pulled. Thanks. Regards, Anthony Liguori > > Jan Kiszka (3): > kvm: i8254: Cache kernel clock offset in KVMPITState > kvm: i8254: Finish time conversion fix > kvmvapic: Disable if

Re: [Qemu-devel] [PULL 00/10] Block patches

2012-08-15 Thread Anthony Liguori
or.cz/r/qemu/kevin.git for-anthony > Pulled. Thanks. Regards, Anthony Liguori > Corey Bryant (7): > qemu-char: Add MSG_CMSG_CLOEXEC flag to recvmsg > qapi: Introduce add-fd, remove-fd, query-fdsets > block: Prevent detection of /dev/fdset/ as floppy > block: Co

Re: [Qemu-devel] [PULL 0/9 for-1.2] Trivial patches for 11 to 15 August 2012

2012-08-15 Thread Anthony Liguori
Stefan Hajnoczi writes: > The last pull request for QEMU 1.2! > > The following changes since commit 03834e22abafbc8dc4052d46a5ccd6dd135a54a3: > > Merge remote-tracking branch 'origin/master' into staging (2012-08-14 > 15:19:50 -0500) > Pulled. Thanks. R

Re: [Qemu-devel] [PATCH v2 for-1.2 00/27] Suppress unused default drives

2012-08-15 Thread Anthony Liguori
drive and the controller, right? Not sure about cdrom... This is a valid critique and suggests that more review is needed. Given that, I won't pick this up today. But let's not throw around Nacks without justification. Regards, Anthony Liguori > > -- PMM

Re: [Qemu-devel] Hard freeze for 1.2 today

2012-08-15 Thread Anthony Liguori
information from. I think this probably needs to wait until 1.3 (assuming it's even the right interface). Regards, Anthony Liguori > > -- > Eric Blake ebl...@redhat.com+1-919-301-3266 > Libvirt virtualization library http://libvirt.org

Re: [Qemu-devel] [PATCH v2 for-1.2 00/27] Suppress unused default drives

2012-08-15 Thread Anthony Liguori
Markus Armbruster writes: > Anthony Liguori writes: > >> Peter Maydell writes: >> >>> On 15 August 2012 20:25, Alexander Graf wrote: >>>> On 15.08.2012, at 21:17, Markus Armbruster wrote: >>>> >>>>> We create a number of

Re: [Qemu-devel] [PATCH v2 for-1.2 00/27] Suppress unused default drives

2012-08-15 Thread Anthony Liguori
Peter Maydell writes: > On 15 August 2012 20:58, Anthony Liguori wrote: >> Peter Maydell writes: >>> On 15 August 2012 20:25, Alexander Graf wrote: >>>> Nack from my POV. Too late for 1.2. Better get this in early for 1.3. >> >> No, it's not

Re: [Qemu-devel] [PULL 0/2] s390 patch queue 2012-08-15

2012-08-15 Thread Anthony Liguori
Alexander Graf writes: > Hi Blue / Aurelien, > > This is my current patch queue for s390. Please pull. > Pulled. Thanks. Regards, Anthony Liguori > Alex > > > The following changes since commit 03834e22abafbc8dc4052d46a5ccd6dd135a54a3: > Anthony Liguori

Re: [Qemu-devel] [PULL 00/24] ppc patch queue 2012-08-15

2012-08-15 Thread Anthony Liguori
Alexander Graf writes: > Hi Blue / Aurelien, > > This is my current patch queue for ppc. Please pull. > Pulled. Thanks. Regards, Anthony Liguori > Alex > > > The following changes since commit 03834e22abafbc8dc4052d46a5ccd6dd135a54a3: > Anthony Liguori (1): >

[Qemu-devel] [PATCH 2/4] Adding qemu-seccomp.[ch] (v8)

2012-08-15 Thread Anthony Liguori
From: Eduardo Otubo Signed-off-by: Eduardo Otubo Signed-off-by: Anthony Liguori --- v1: - I added a syscall struct using priority levels as described in the libseccomp man page. The priority numbers are based to the frequency they appear in a sample strace from a regular qemu guest run

[Qemu-devel] [PATCH 1/4] Adding support for libseccomp in configure and Makefile (v8)

2012-08-15 Thread Anthony Liguori
From: Eduardo Otubo Adding basic options to the configure script to use libseccomp or not. The default is set to 'no'. If the flag --enable-libseccomp is used, the script will check for its existence using pkg-config. Signed-off-by: Eduardo Otubo Signed-off-by: Anthony Liguori --

[Qemu-devel] [PATCH 3/4] Adding seccomp calls to vl.c (v8)

2012-08-15 Thread Anthony Liguori
From: Eduardo Otubo Signed-off-by: Eduardo Otubo Signed-off-by: Anthony Liguori --- v1: - Full seccomp calls and data included in vl.c v1 -> v2: - Full seccomp calls and data removed from vl.c and put into separate qemu-seccomp.[ch] file. --- vl.c | 13 + 1 files chan

[Qemu-devel] [PATCH 0/4] Add -sandbox option to enable seccomp mode 2

2012-08-15 Thread Anthony Liguori
Hi, I attempted to apply Eduardo's seccomp patches and ran into a number of problems. In the interest of time, I thought it would be easier for me to fix them and just respin the series myself. I've tested this as best I can--I don't have a seccomp capable kernel. But since the option is availa

[Qemu-devel] [PATCH 4/4] Command line support for seccomp with -sandbox (v8)

2012-08-15 Thread Anthony Liguori
From: Eduardo Otubo Signed-off-by: Eduardo Otubo Signed-off-by: Anthony Liguori --- v7 -> v8 - Parse options correctly (aliguori) --- qemu-config.c | 14 ++ qemu-config.h |1 + qemu-options.hx | 10 ++ vl.c|

Re: [Qemu-devel] x86, nops settings result in kernel crash

2012-08-16 Thread Anthony Liguori
ding on what feature bits are exposed to the guest. So my guess is that the odd combination of CPUID bits that are exposed to the guest is confusing the kernel. Can you post dmesg from the host kernel? Perhaps there's instruction emulation failing in the host KVM? That would manifest in strange behavior in the guest. Regards, Anthony Liguori > > Alan

Re: [Qemu-devel] commit da57febfed "qdev: give all devices a canonical path" broke usb_del

2012-08-20 Thread Anthony Liguori
object_finalize(void *data) > @@ -385,8 +383,9 @@ Object *object_new(const char *typename) > > void object_delete(Object *obj) > { > +object_unparent(obj); > +g_assert(obj->ref == 1); > object_unref(obj); > -g_assert(obj->ref == 0); > g_free(obj); > } This won't work with composition. object_delete() is never called for child<> objects. Regards, Anthony Liguori >

Re: [Qemu-devel] [PATCH 1/5] move qemu_irq typedef out of cpu-common.h

2012-08-20 Thread Anthony Liguori
>>> > @@ -8,6 +8,7 @@ >>> >*/ >>> > #include >>> > >>> > +#include "hw/irq.h" >>> > #include "qemu-common.h" >>> > >>> > /* xen-machine.c */ >>> > diff --git a

Re: [Qemu-devel] [PATCH] qapi: add 'query-target' command to return target arch/bit size

2012-08-20 Thread Anthony Liguori
you expect it to mean. We're going to most likely fix TARGET_PHYS_ADDR_BITS to 64 real soon. Why did you include this field? What information are you looking to get from QEMU and what decisions do you plan to make with it? Regards, Anthony Liguori > > Signed-off-by: Daniel P. Berrange

Re: [Qemu-devel] [PATCH] qapi: add 'query-target' command to return target arch/bit size

2012-08-21 Thread Anthony Liguori
"Daniel P. Berrange" writes: > On Mon, Aug 20, 2012 at 04:48:24PM -0500, Anthony Liguori wrote: >> "Daniel P. Berrange" writes: >> >> > From: "Daniel P. Berrange" >> > >> > Add a 'query-target' QAPI command

Re: [Qemu-devel] Ping 1.2 PATCH "eventfd: making it thread safe" (was Re: [PATCH v2 1/3] net: asynchronous send/receive infrastructure for net/socket.c)

2012-08-21 Thread Anthony Liguori
before the change takes effect. > > Why haven't we applied yet the patch to do that unconditionally? > > http://permalink.gmane.org/gmane.comp.emulators.qemu/162828 Can you ping the actual patch... Gmane seems to be down right now. Regards, Anthony Liguori > > Paolo

Re: [Qemu-devel] commit da57febfed "qdev: give all devices a canonical path" broke usb_del

2012-08-21 Thread Anthony Liguori
Paolo Bonzini writes: > Il 20/08/2012 19:58, Anthony Liguori ha scritto: >> Michael Tokarev writes: >> >>> On 08.08.2012 17:09, Michael Tokarev wrote: >>> [] >>>> Something similar should be applied to 1.1-stable. FWIW, some >>>> change

Re: [Qemu-devel] [PATCH] qapi: add 'query-target' command to return target arch/bit size

2012-08-22 Thread Anthony Liguori
gt;> > > > + >> > > > +## >> > > > +# @TargetInfo: >> > > > +# >> > > > +# Information describing the QEMU target. >> > > > +# >> > > > +# @arch: the name of the target architecture (eg "x86_64&q

Re: [Qemu-devel] For 1.2: Re: [PATCH 3/8] migration: move total_time from ram stats to migration info

2012-08-22 Thread Anthony Liguori
#x27;, '*ram': 'MigrationStats', >> '*disk': 'MigrationStats', >> - '*xbzrle-cache': 'XBZRLECacheStats'} } >> + '*xbzrle-cache': 'XBZRLECacheStats', >> + 'total-time': 'int'} } > > Anthony - are you planning on taking this series for 1.2? No. This is a new feature and we're past freeze. > If we don't > get this patch in on time, then taking this for 1.3 would result in > changing released QMP interface (right now, there has been no release > with the field in the wrong type). Ack. We need to preserve compat with the 1.2 interface. Regards, Anthony Liguori > > -- > Eric Blake ebl...@redhat.com+1-919-301-3266 > Libvirt virtualization library http://libvirt.org

[Qemu-devel] [PATCH] qapi: add 'query-target' command to return target arch/bit size (v2)

2012-08-22 Thread Anthony Liguori
'return': { u'arch': u'x86_64' }} Signed-off-by: Daniel P. Berrange Signed-off-by: Anthony Liguori --- v1 -> v2 (aliguori) - remove 'bits' field - switch command to return an enum --- arch_init.c | 11 ++

Re: [Qemu-devel] [PATCH] qom: removal of link property need to release its target

2012-08-22 Thread Anthony Liguori
del() to add a notifier to the object that calls object_unparent() and object_unref. 6) Rename DeviceState::unplug to DeviceState::request_unplug 7) Take Ping Fan's patch + another patch to add a reference count in object_property_add_link Regards, Anthony Liguori > > Paolo

Re: [Qemu-devel] How to reliably start a bare QEMU target to query capabilities via QMP

2012-08-22 Thread Anthony Liguori
7;t care about any of the VM setup parts ? This would > be more like what my old patches did which mapped monitors commands > onto '-query-X' command line args, bypassing VM setup, but still > using normal QMP monitor interaction. > > Any other suggestions on how to reliably ge

[Qemu-devel] [PATCH] boards: add a 'none' machine type to all platforms

2012-08-22 Thread Anthony Liguori
Armbruster Signed-off-by: Anthony Liguori --- hw/Makefile.objs |2 ++ hw/null-machine.c | 40 2 files changed, 42 insertions(+), 0 deletions(-) create mode 100644 hw/null-machine.c diff --git a/hw/Makefile.objs b/hw/Makefile.objs index 7f57ed5..6dfebd2

Re: [Qemu-devel] Ping 1.2 PATCH "eventfd: making it thread safe" (was Re: [PATCH v2 1/3] net: asynchronous send/receive infrastructure for net/socket.c)

2012-08-22 Thread Anthony Liguori
Paolo Bonzini writes: > Il 21/08/2012 21:06, Anthony Liguori ha scritto: >> Paolo Bonzini writes: >> >>> Il 21/08/2012 18:39, ronnie sahlberg ha scritto: >>>> In >>>> net_socket_update_fd_handler() >>>> >>>> shouldnt yo

Re: [Qemu-devel] [PATCH] boards: add a 'none' machine type to all platforms

2012-08-22 Thread Anthony Liguori
Eric Blake writes: > On 08/22/2012 02:24 PM, Anthony Liguori wrote: >> This allows any QEMU binary to be executed with: >> >> $QEMU_BINARY -qmp stdio > > Don't you mean: > > $QEMU_BINARY -M none -qmp stdio I did, thanks. > > I agree with includ

Re: [Qemu-devel] [PATCH] qom: removal of link property need to release its target

2012-08-22 Thread Anthony Liguori
Hi Paolo, Paolo Bonzini writes: > Il 22/08/2012 18:36, Anthony Liguori ha scritto: >> We can fix this by adding an extra reference in add_link but this >> creates yet another problem with hotplug. Specificially, qdev_free() >> asserts that ref > 0 because there is now a

Re: [Qemu-devel] [PATCH] qom: removal of link property need to release its target

2012-08-22 Thread Anthony Liguori
Paolo Bonzini writes: > Il 22/08/2012 23:41, Anthony Liguori ha scritto: >> >> (1) should drop the floating reference and the reference held by the >> container. That's what I meant by calling object_unparent in >> qmp_device_del. >> >> (2)

Re: [Qemu-devel] [PATCH] boards: add a 'none' machine type to all platforms

2012-08-22 Thread Anthony Liguori
Peter Maydell writes: > On 22 August 2012 21:24, Anthony Liguori wrote: >> This allows any QEMU binary to be executed with: >> >> $QEMU_BINARY -qmp stdio > > ...presumably you mean -qmp stdio -M none ? > >> >> Without errors from missing options that

Re: [Qemu-devel] [PATCH] boards: add a 'none' machine type to all platforms

2012-08-22 Thread Anthony Liguori
Peter Maydell writes: > On 22 August 2012 23:42, Anthony Liguori wrote: >> Peter Maydell writes: >>> How about documenting this machine (and its purpose) somewhere? >> >> Okay, but where? I don't know an obvious place. > > If we hadn't got

Re: [Qemu-devel] [PATCH] boards: add a 'none' machine type to all platforms

2012-08-23 Thread Anthony Liguori
Andreas Färber writes: > Am 23.08.2012 00:42, schrieb Anthony Liguori: >> Peter Maydell writes: >> >>> On 22 August 2012 21:24, Anthony Liguori wrote: >>> >>> We seem to be about evenly split about whether machine_init() >>> should have a

[Qemu-devel] [PATCH] monitor: move json init from OPEN event to init

2012-08-23 Thread Anthony Liguori
ies'}" | qemu-system-x86_64 -M none -qmp stdio We can fix this for the monitor by moving the parser initialization to init. The remaining state that is set in OPEN appears harmless. Reported-by: Daniel Berrange Signed-off-by: Anthony Liguori --- monitor.c |4 +++- 1 files changed

Re: [Qemu-devel] Race condition in char device setup causing SEGV

2012-08-23 Thread Anthony Liguori
nt > - json_message_parser_init > - json_lexer_init > > > The problem arises if you try to feed data to QEMU before the bottom > half has run. There is a race where qemu_chr_be_write can be called > to process input, before the qemu_chr_generic_open_bh has been > invoked. The char layer reall

[Qemu-devel] [PATCH] monitor: don't try to initialize json parser when monitor is HMP

2012-08-23 Thread Anthony Liguori
Reported-by: Michael Roth Signed-off-by: Anthony Liguori --- monitor.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/monitor.c b/monitor.c index b188582..b17b1bb 100644 --- a/monitor.c +++ b/monitor.c @@ -4946,13 +4946,13 @@ void monitor_init(CharDriverState *chr

Re: [Qemu-devel] [PATCH] Support for loading devices as dynamic libraries

2012-08-25 Thread Anthony Liguori
It's also pretty useful when doing security certifications. Regards, Anthony Liguori > > Example Makefile for out of tree modules: > #v+ > DEVICENAME=pcnet2 > > hw-obj-y=pcnet-pci.o > hw-obj-y+=pcnet.o > > include rules.mak > > .PHONY: all > > QEMU_CF

  1   2   3   4   5   6   7   8   9   10   >