[Qemu-devel] [PATCH v2 4/6] qemu-options.hx: mention retrace= VGA option

2012-08-23 Thread Matthew Ogilvie
The feature was added in commit cb5a7aa8c32141bb Sep 2008. My description is based on "Better VGA retrace emulation (needed for some DOS games/demos)" from http://www.boblycat.org/~malc/code/patches/qemu/index.html Signed-off-by: Matthew Ogilvie --- This is the first version of this patch. I no

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 03/17] hvm-pci: Handle PCI config space in Xen

2012-08-23 Thread Jan Beulich
>>> Julien Grall 08/22/12 8:56 PM >>> >+int hvm_register_pcidev(domid_t domid, ioservid_t id, >+uint8_t domain, uint8_t bus, >+uint8_t device, uint8_t function) >+{ "domain" needs to be "uint16_t". Also, just to double check: we don't currently exp

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 05/17] hvm: Modify hvm_op

2012-08-23 Thread Jan Beulich
>>> Julien Grall 08/22/12 8:56 PM >>> >@@ -4069,20 +4053,12 @@ long do_hvm_op(unsigned long op, >XEN_GUEST_HANDLE(void) arg) > >switch ( a.index ) >{ >-case HVM_PARAM_IOREQ_PFN: Removing sub-ops which a domain can issue for itself (which for this and another one below appears to be

Re: [Qemu-devel] [PATCH 4/6] QAPI: Introduce memchar_read QMP command

2012-08-23 Thread Lei Li
On 08/23/2012 01:46 PM, Eric Blake wrote: On 08/22/2012 11:14 PM, Lei Li wrote: Signed-off-by: Lei Li Again, subject line should use '-' not '_' for QMP. --- hmp-commands.hx | 16 hmp.c| 15 +++ hmp.h|1 + qapi-schema.json |

Re: [Qemu-devel] [PATCH V4 1/2] qapi: Add SnapshotInfo and ImageInfo.

2012-08-23 Thread Benoît Canet
> That seems backwards - with a raw file, don't you know the actual size, > but have no idea what it can further grow to? In raw it commes from fstat() with the field st.st_blocks. from man: blkcnt_t st_blocks; /* number of 512B blocks allocated */ Benoît

Re: [Qemu-devel] [PATCH V4 1/2] qapi: Add SnapshotInfo and ImageInfo.

2012-08-23 Thread Benoît Canet
> > +## > > +# @ImageInfo: > > +# > > +# Information about a QEMU image file > > +# > > +# @filename: name of the image file > > +# > > +# @format: format of the image file > > +# > > +# @virtual-size: maximum capacity in bytes of the image > > +# > > +# @actual-size: #optional actual size on disk

Re: [Qemu-devel] QEMU version for ARM7 adaptation

2012-08-23 Thread Peter Maydell
On 23 August 2012 04:03, Vinicius Sanches wrote: > What is the best QEMU version available to enable ARM7 core? If you want to add a new feature to QEMU you should always work on git master; we don't accept patches against anything else. -- PMM

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

2012-08-23 Thread liu ping fan
On Thu, Aug 23, 2012 at 12:36 AM, Anthony Liguori wrote: > Paolo Bonzini writes: > >> Il 22/08/2012 05:02, Liu Ping Fan ha scritto: >>> From: Liu Ping Fan >>> >>> Currently, link property's target is only managed by >>> object_set_link_property(). This will raise such issue that when >>> the pro

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

2012-08-23 Thread Paolo Bonzini
Il 23/08/2012 00:40, Anthony Liguori ha scritto: > I don't really like the notion of a "forced eject" where we delete a > device when the guest is using it and not cooperative. I don't see the > benefit at all. > > Forcing detachment of a BlockDriverState from a device followed by EIO > being rep

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

2012-08-23 Thread Daniel P. Berrange
On Wed, Aug 22, 2012 at 03:25:38PM -0500, Anthony Liguori wrote: > "Daniel P. Berrange" writes: > > > I've been adapting libvirt to use to the various new QMP commands to > > query QEMU's capabilities, instead of the hated -help parsing. Obviously > > the critical part of this is being able to re

Re: [Qemu-devel] [PATCH v12 1/2] qxl: add QXL_IO_MONITORS_CONFIG_ASYNC

2012-08-23 Thread Gerd Hoffmann
On 08/22/12 10:16, Alon Levy wrote: > v11->v12: > fix build with older spice server (Gerd). Replaced patches in the spice queue. thanks, Gerd

Re: [Qemu-devel] [PATCH 2/5] trace-cmd: Use tracing directory to count CPUs

2012-08-23 Thread Steven Rostedt
On Thu, 2012-08-23 at 12:00 +0900, Masami Hiramatsu wrote: > (2012/08/23 11:01), Masami Hiramatsu wrote: > > (2012/08/22 22:41), Steven Rostedt wrote: > >> On Wed, 2012-08-22 at 17:43 +0900, Yoshihiro YUNOMAE wrote: > >>> From: Masami Hiramatsu > >>> > >>> Count debugfs/tracing/per_cpu/cpu* to det

Re: [Qemu-devel] [PATCH 1/2 v1] blkdrv: Add queue limits parameters for sg block drive

2012-08-23 Thread Cong Meng
On Wed 22 Aug 2012 10:13:44 PM CST, Paolo Bonzini wrote: Il 22/08/2012 15:13, Stefan Hajnoczi ha scritto: http://lists.gnu.org/archive/html/qemu-devel/2010-12/msg01741.html "This is a real problem in practice. IE. the USB CD-ROM on this POWER7 blade limits transfers to 0x1e000 bytes for examp

Re: [Qemu-devel] [PATCH v2] register reset handler to write image into memory

2012-08-23 Thread Yin Olivia-R63875
Dear All, I can't find MAINTAINER of hw/loader.c. Who can help review and apply this patch? Best Regards, Olivia Yin > -Original Message- > From: Yin Olivia-R63875 > Sent: Friday, August 17, 2012 5:08 PM > To: qemu-...@nongnu.org; qemu-devel@nongnu.org > Cc: Yin Olivia-R63875 > Subject:

Re: [Qemu-devel] [PATCH 1/2 v1] blkdrv: Add queue limits parameters for sg block drive

2012-08-23 Thread Paolo Bonzini
Il 23/08/2012 11:31, Cong Meng ha scritto: >> For disks, this should be fixed simply by using scsi-block instead of >> scsi-generic. >> >> CD-ROMs are indeed more complicated because burning CDs cannot be done >> with syscalls. :/ > > So, as the problem exist to CD-ROM, I will continue to get thes

Re: [Qemu-devel] passing translated address out in QEMU

2012-08-23 Thread Wei-Ren Chen
On Wed, Aug 22, 2012 at 09:38:17PM -0700, Xin Tong wrote: > On Wed, Aug 22, 2012 at 8:14 PM, 陳韋任 (Wei-Ren Chen) > wrote: > >> In tcg_gen_qemu_ld8s(tmp, addr, index); a TCGv addr is passed to the > >> INDEX_op_qemu_ld8s as param and the loaded value is passed back in > >> tmp. i want to get the tra

Re: [Qemu-devel] [PATCH 1/2 v1] blkdrv: Add queue limits parameters for sg block drive

2012-08-23 Thread Stefan Hajnoczi
On Thu, Aug 23, 2012 at 11:03 AM, Paolo Bonzini wrote: > Il 23/08/2012 11:31, Cong Meng ha scritto: >>> For disks, this should be fixed simply by using scsi-block instead of >>> scsi-generic. >>> >>> CD-ROMs are indeed more complicated because burning CDs cannot be done >>> with syscalls. :/ >> >>

Re: [Qemu-devel] passing translated address out in QEMU

2012-08-23 Thread Wei-Ren Chen
> subtracting host virtual with a fixed offset for ram. but the thing i > do not know how to do is how to pass the guest physical back, can i > store the guest physical in the TCGv from the guest virtual. Maybe you can store it on a host unused register, perhaps xmm0? ;) Regards, chenwj -- We

Re: [Qemu-devel] [PATCH v2] register reset handler to write image into memory

2012-08-23 Thread Dunrong Huang
2012/8/23 Yin Olivia-R63875 : > Dear All, > > I can't find MAINTAINER of hw/loader.c. > Who can help review and apply this patch? > Please use the script scripts/get_maintainer.pl, like: $ scripts/get_maintainer.pl your_patch_file.patch or $ scripts/get_maintainer.pl -f hw/loader.c > Best Regards,

Re: [Qemu-devel] [PATCH v9 5/6] introduce a new qom device to deal with panicked event

2012-08-23 Thread Jan Kiszka
On 2012-08-23 04:32, Wen Congyang wrote: > If the target is x86/x86_64, the guest's kernel will write 0x01 to the > port KVM_PV_EVENT_PORT when it is panciked. This patch introduces a new > qom device kvm_pv_ioport to listen this I/O port, and deal with panicked > event according to panicked_action

Re: [Qemu-devel] [PATCH 1/2 v1] blkdrv: Add queue limits parameters for sg block drive

2012-08-23 Thread Paolo Bonzini
Il 23/08/2012 12:08, Stefan Hajnoczi ha scritto: >> I'm still trying to understand the extent of the problem. >> >> The problem occurs for _USB_ CD-ROMs according to Ben. Passthrough of >> USB storage devices should be done via USB passthrough, not virtio-scsi. >> If we do USB passthrough via the

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

2012-08-23 Thread Daniel P. Berrange
When testing with the new "-M none" arg, I've noticed that ~70% of the time libvirt starts QEMU will result in a SEGV from QEMU with the following stack trace: (gdb) bt #0 0x in ?? () #1 0x5567a37f in json_lexer_feed_char (lexer=0x5658fb20, ch=123 '{', flush=false) a

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

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 12:01:05PM +0100, Daniel P. Berrange wrote: > When testing with the new "-M none" arg, I've noticed that ~70% > of the time libvirt starts QEMU will result in a SEGV from QEMU > with the following stack trace: > > (gdb) bt > #0 0x in ?? () > #1 0x5

[Qemu-devel] [PATCH v3 uq/master 4/6] memory: Flush coalesced MMIO on mapping and state changes

2012-08-23 Thread Jan Kiszka
Flush pending coalesced MMIO before performing mapping or state changes that could affect the event orderings or route the buffered requests to a wrong region. Signed-off-by: Jan Kiszka --- memory.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/memory.c b/memory.c inde

[Qemu-devel] [PATCH v3 uq/master 0/6] kvm: Get coalesced MMIO flushing out of the hot-path

2012-08-23 Thread Jan Kiszka
This is just a repost, now targeting uq/master as agreed. No changes compared to v2 except that "i82378: Remove bogus MMIO coalescing" was dropped as it is already in QEMU upstream by now. Original description: We currently flush the coalesced MMIO buffer on every vmexit to userspace. KVM only pr

[Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Jan Kiszka
No need for this indirection via qemu_notify_event. On Unix, we already catch SIGALRM via signalfd (or its emulation) and run the handler synchronously. Under Win32, handlers run in separate threads. So we just need to grab the global lock around the handler execution. Signed-off-by: Jan Kiszka -

[Qemu-devel] [PATCH v3 uq/master 2/6] memory: Use transaction_begin/commit also for single-step operations

2012-08-23 Thread Jan Kiszka
Wrap also simple operations consisting only of a single step with memory_region_transaction_begin/commit. This allows to perform additional steps like coalesced MMIO flushing from a single place. This requires dropping some micro-optimizations: The skipping of topology updates after updating disab

[Qemu-devel] [PATCH v3 uq/master 6/6] kvm: Stop flushing coalesced MMIO on vmexit

2012-08-23 Thread Jan Kiszka
The memory subsystem will now take care of flushing whenever affected regions are accessed or the memory mapping changes. Signed-off-by: Jan Kiszka --- kvm-all.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/kvm-all.c b/kvm-all.c index e0244b6..432b84f 100644 --- a/kv

[Qemu-devel] [PATCH v3 uq/master 1/6] memory: Flush coalesced MMIO on selected region access

2012-08-23 Thread Jan Kiszka
Instead of flushing pending coalesced MMIO requests on every vmexit, this provides a mechanism to selectively flush when memory regions related to the coalesced one are accessed. This first of all includes the coalesced region itself but can also applied to other regions, e.g. of the same device, b

[Qemu-devel] [PATCH v3 uq/master 5/6] VGA: Flush coalesced MMIO on related MMIO/PIO accesses

2012-08-23 Thread Jan Kiszka
In preparation of stopping to flush coalesced MMIO unconditionally on vmexits, mark VGA MMIO and PIO regions as synchronous /wrt coalesced MMIO and flush the buffer explicitly on PIO accesses that do not use generic memory regions yet. Signed-off-by: Jan Kiszka --- hw/cirrus_vga.c |7 +++

Re: [Qemu-devel] [PATCH v2] register reset handler to write image into memory

2012-08-23 Thread Andreas Färber
Hi, Am 23.08.2012 11:45, schrieb Yin Olivia-R63875: > Dear All, > > I can't find MAINTAINER of hw/loader.c. > Who can help review and apply this patch? This patch is not a small bugfix so it won't be applied during the v1.2 Hard Freeze. You based it onto ppc-next so the obvious answer is, Alex n

Re: [Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Paolo Bonzini
Il 23/08/2012 13:23, Jan Kiszka ha scritto: > No need for this indirection via qemu_notify_event. On Unix, we already > catch SIGALRM via signalfd (or its emulation) and run the handler > synchronously. Under Win32, handlers run in separate threads. So we just > need to grab the global lock around

[Qemu-devel] [PATCH v3 uq/master 3/6] memory: Fold memory_region_update_topology into memory_region_transaction_commit

2012-08-23 Thread Jan Kiszka
Simplify the code as we are using now only a subset of the original features of memory_region_update_topology. Signed-off-by: Jan Kiszka --- memory.c | 39 +++ 1 files changed, 11 insertions(+), 28 deletions(-) diff --git a/memory.c b/memory.c index 58583cf

Re: [Qemu-devel] [PATCH v2 1/6] fix some debug printf format strings

2012-08-23 Thread Andreas Färber
Hi, Am 23.08.2012 08:24, schrieb Matthew Ogilvie: > These are normally ifdefed out and don't matter. But if you enable > them, they ought to be correct. > > Signed-off-by: Matthew Ogilvie > --- > > This version of the patch adds i8259.c. > > An alternative approach might be to eliminate these

Re: [Qemu-devel] [PATCH v2] kvm-all.c: Move init of irqchip_inject_ioctl out of kvm_irqchip_create()

2012-08-23 Thread Marcelo Tosatti
On Tue, Aug 21, 2012 at 12:27:51PM +0200, Jan Kiszka wrote: > On 2012-08-21 10:25, Peter Maydell wrote: > > On 21 August 2012 09:19, Jan Kiszka wrote: > >> On 2012-08-15 13:08, Peter Maydell wrote: > >>> Move the init of the irqchip_inject_ioctl field of KVMState out of > >>> kvm_irqchip_create()

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

2012-08-23 Thread Andreas Färber
Am 23.08.2012 00:42, schrieb 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 missi

Re: [Qemu-devel] [PATCH 1/2 v1] blkdrv: Add queue limits parameters for sg block drive

2012-08-23 Thread Stefan Hajnoczi
On Thu, Aug 23, 2012 at 11:52 AM, Paolo Bonzini wrote: > Il 23/08/2012 12:08, Stefan Hajnoczi ha scritto: >>> I'm still trying to understand the extent of the problem. >>> >>> The problem occurs for _USB_ CD-ROMs according to Ben. Passthrough of >>> USB storage devices should be done via USB pass

Re: [Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Jan Kiszka
On 2012-08-23 13:39, Paolo Bonzini wrote: > Il 23/08/2012 13:23, Jan Kiszka ha scritto: >> No need for this indirection via qemu_notify_event. On Unix, we already >> catch SIGALRM via signalfd (or its emulation) and run the handler >> synchronously. Under Win32, handlers run in separate threads. So

Re: [Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Paolo Bonzini
Il 23/08/2012 14:10, Jan Kiszka ha scritto: >> Can you expand on this? > > Well, this patch removes an indirection from timer event deliveries. So > it reduces overhead, though only noticeable if you have high-rate timers. Actually, timers (and bottom halves) are always run after iohandlers. So t

[Qemu-devel] [PATCH 0/2] Add file url support to migration

2012-08-23 Thread Benoît Canet
This patchset add support of file: urls allowing a user to save to and restore from disk the state of a vm in an easy way. An fflush was added before closing the fd for this use case in order to be sure that all data have reached stable storage. One new usage would be migrating between two virtua

[Qemu-devel] [PATCH 1/2] migration: Allow the migrate command to work on file: urls

2012-08-23 Thread Benoît Canet
Usage: (qemu) migrate file:/path/to/vm_statefile Signed-off-by: Benoit Canet --- migration-fd.c |4 ++-- migration.c| 20 +++- migration.h|2 +- 3 files changed, 22 insertions(+), 4 deletions(-) diff --git a/migration-fd.c b/migration-fd.c index 50138ed..d39e44

[Qemu-devel] [PATCH 2/2] migration: Allow -incoming to work on file: urls

2012-08-23 Thread Benoît Canet
Usage: -incoming file:/path/to/vm_statefile Signed-off-by: Benoit Canet --- migration-fd.c |5 ++--- migration.c| 14 +- migration.h|2 +- 3 files changed, 16 insertions(+), 5 deletions(-) diff --git a/migration-fd.c b/migration-fd.c index d39e44a..bf500a0 100644 -

Re: [Qemu-devel] [PATCH 2/5] trace-cmd: Use tracing directory to count CPUs

2012-08-23 Thread Masami Hiramatsu
(2012/08/23 18:08), Steven Rostedt wrote: > On Thu, 2012-08-23 at 12:00 +0900, Masami Hiramatsu wrote: >> (2012/08/23 11:01), Masami Hiramatsu wrote: >>> (2012/08/22 22:41), Steven Rostedt wrote: On Wed, 2012-08-22 at 17:43 +0900, Yoshihiro YUNOMAE wrote: > From: Masami Hiramatsu > >>

Re: [Qemu-devel] [PATCH 1/2] migration: Allow the migrate command to work on file: urls

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 02:28:07PM +0200, Benoît Canet wrote: > Usage: > (qemu) migrate file:/path/to/vm_statefile > > Signed-off-by: Benoit Canet > --- > migration-fd.c |4 ++-- > migration.c| 20 +++- > migration.h|2 +- > 3 files changed, 22 insertions(+), 4

[Qemu-devel] [PATCH V5 1/2] qapi: Add SnapshotInfo and ImageInfo.

2012-08-23 Thread Benoît Canet
Signed-off-by: Benoit Canet --- qapi-schema.json | 64 ++ 1 file changed, 64 insertions(+) diff --git a/qapi-schema.json b/qapi-schema.json index a92adb1..ffe3a0a 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -126,6 +126,70 @@

[Qemu-devel] [PATCH V5 0/2] Add JSON output to qemu-img info

2012-08-23 Thread Benoît Canet
This patchset add a JSON output mode to the qemu-img info command. It's a rewrite from scratch of the original patchset by Wenchao Xia following Anthony Liguori advices on JSON formating. the --output=(json|human) option is now mandatory on the command line. Benoît Canet (3): qapi: Add Snapshot

[Qemu-devel] [PATCH V5 2/2] qemu-img: Add json output option to the info command.

2012-08-23 Thread Benoît Canet
This option --output=[human|json] make qemu-img info output on human or JSON representation at the choice of the user. example: { "snapshots": [ { "vm-clock-nsec": 637102488, "name": "vm-20120821145509", "date-sec": 1345553709, "date-nsec

Re: [Qemu-devel] [PATCH] Add guest-get-hostname to retrieve the guests current hostname

2012-08-23 Thread Luiz Capitulino
On Tue, 21 Aug 2012 13:57:54 +0200 Guido Günther wrote: > This allows to retrieve the guest's hostname via gethostname(2). > > This can be useful to identify a VM e.g. one without network. > > Signed-off-by: Guido Günther > --- > We have an API in libvirt for that (virDomainGetHostname). > Che

Re: [Qemu-devel] [PATCH] i2c: factor out VMSD to parent class

2012-08-23 Thread Juan Quintela
Peter Crosthwaite wrote: > > - I guess there is same reasy why you want to split the device state, > it could be on the other series where I haven't read it though. >>> >>> So this is exactly what I have done in the SSI. Correct me if I am >>> wrong but it is the same setup as PCI wh

Re: [Qemu-devel] [PATCH 1/2] migration: Allow the migrate command to work on file: urls

2012-08-23 Thread Benoît Canet
Le Thursday 23 Aug 2012 à 13:34:01 (+0100), Daniel P. Berrange a écrit : > On Thu, Aug 23, 2012 at 02:28:07PM +0200, Benoît Canet wrote: > > Usage: > > (qemu) migrate file:/path/to/vm_statefile > > > > Signed-off-by: Benoit Canet > > --- > > migration-fd.c |4 ++-- > > migration.c| 20

[Qemu-devel] [PATCH v4] ivshmem: add 64bit option

2012-08-23 Thread Gerd Hoffmann
This patch adds a "use64" property which will make the ivshmem driver register a 64bit memory bar when set, so you have something to play with when testing 64bit pci bits. It also allows to have quite big shared memory regions, like this: [root@fedora ~]# lspci -vs1:1 01:01.0 RAM memory: Red Hat,

Re: [Qemu-devel] [PATCH 1/2] migration: Allow the migrate command to work on file: urls

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 02:48:19PM +0200, Benoît Canet wrote: > Le Thursday 23 Aug 2012 à 13:34:01 (+0100), Daniel P. Berrange a écrit : > > On Thu, Aug 23, 2012 at 02:28:07PM +0200, Benoît Canet wrote: > > > Usage: > > > (qemu) migrate file:/path/to/vm_statefile > > > > > > Signed-off-by: Benoit

Re: [Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Jan Kiszka
On 2012-08-23 14:24, Paolo Bonzini wrote: > Il 23/08/2012 14:10, Jan Kiszka ha scritto: >>> Can you expand on this? >> >> Well, this patch removes an indirection from timer event deliveries. So >> it reduces overhead, though only noticeable if you have high-rate timers. > > Actually, timers (and b

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 trailing semicolon (it doesn't need one >>> but it does

Re: [Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Paolo Bonzini
Il 23/08/2012 15:01, Jan Kiszka ha scritto: > On 2012-08-23 14:24, Paolo Bonzini wrote: >> Il 23/08/2012 14:10, Jan Kiszka ha scritto: Can you expand on this? >>> >>> Well, this patch removes an indirection from timer event deliveries. So >>> it reduces overhead, though only noticeable if you

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 01/17] hvm: Modify interface to support multiple ioreq server

2012-08-23 Thread Ian Campbell
> diff --git a/xen/include/asm-x86/hvm/domain.h > b/xen/include/asm-x86/hvm/domain.h > index 27b3de5..49d1ca0 100644 > --- a/xen/include/asm-x86/hvm/domain.h > +++ b/xen/include/asm-x86/hvm/domain.h [...] > struct hvm_domain { > +/* Use for the IO handles by Xen */ > struct hvm_ioreq_pag

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 09/17] xc: Add the hypercall for multiple servers

2012-08-23 Thread Ian Campbell
On Wed, 2012-08-22 at 13:31 +0100, Julien Grall wrote: > This patch add 5 hypercalls to register server, io range and PCI. > > Signed-off-by: Julien Grall Looks correct to me at least so far as the use of the hypercall buffers goes, thanks. Acked-by: Ian Campbell

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

2012-08-23 Thread Anthony Liguori
At some point in the past, the OPEN event was changed to be issued from a bottom half. This creates a small window whereas a data callback registered in init may be invoked before the OPEN event has been issued. This is reproducible with: echo "{'execute': 'qmp_capabilities'}" | qemu-system-x86

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 11/17] xc: modify save/restore to support multiple device models

2012-08-23 Thread Ian Campbell
On Wed, 2012-08-22 at 13:31 +0100, Julien Grall wrote: > - add save/restore new special pages and remove unused > - modify save file structure to allow multiple qemu states > > Signed-off-by: Julien Grall > --- > tools/libxc/xc_domain_restore.c | 150 > +

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 12/17] xl: Add interface to handle qemu disaggregation

2012-08-23 Thread Ian Campbell
On Wed, 2012-08-22 at 13:31 +0100, Julien Grall wrote: > This patch modifies libxl interface for qemu disaggregation. I'd rather see the interfaces changes in the same patch as the implementation of the new interfaces. > For the moment, due to some dependencies between devices, we > can't let the

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

2012-08-23 Thread Anthony Liguori
"Daniel P. Berrange" writes: > When testing with the new "-M none" arg, I've noticed that ~70% > of the time libvirt starts QEMU will result in a SEGV from QEMU > with the following stack trace: > > (gdb) bt > #0 0x in ?? () > #1 0x5567a37f in json_lexer_feed_char (lexer

Re: [Qemu-devel] [PATCH v6 01/13] target-mips-ase-dsp: Add internal functions

2012-08-23 Thread Aurelien Jarno
On Tue, Aug 21, 2012 at 02:53:07PM +0800, Jia Liu wrote: > Add internal functions using by MIPS ASE DSP instructions. > > Signed-off-by: Jia Liu > --- > target-mips/Makefile.objs |2 +- > target-mips/dsp_helper.c | 1277 > + > 2 files changed, 12

Re: [Qemu-devel] [PATCH v6 02/13] target-mips-ase-dsp: Use correct acc value to index cpu_HI/cpu_LO rather than using a fix number

2012-08-23 Thread Aurelien Jarno
On Tue, Aug 21, 2012 at 02:53:08PM +0800, Jia Liu wrote: > Use correct acc value to index cpu_HI/cpu_LO rather than using a fix number. > > Signed-off-by: Jia Liu > --- > target-mips/translate.c | 107 > +++ > 1 file changed, 81 insertions(+), 26 del

Re: [Qemu-devel] [PATCH 1/2] migration: Allow the migrate command to work on file: urls

2012-08-23 Thread Benoît Canet
Le Thursday 23 Aug 2012 à 14:48:19 (+0200), Benoît Canet a écrit : > Le Thursday 23 Aug 2012 à 13:34:01 (+0100), Daniel P. Berrange a écrit : > > On Thu, Aug 23, 2012 at 02:28:07PM +0200, Benoît Canet wrote: > > > Usage: > > > (qemu) migrate file:/path/to/vm_statefile > > > > > > Signed-off-by: Be

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 14/17] xl-parsing: Parse new device_models option

2012-08-23 Thread Ian Campbell
On Wed, 2012-08-22 at 13:32 +0100, Julien Grall wrote: > Add new option "device_models". The user can specify the capability of the > QEMU (ui, vifs, ...). This option only works with QEMU upstream (qemu-xen). > > For instance: > device_models= [ 'name=all,vifs=nic1', 'name=qvga,ui', 'name=qide,id

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

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 08:22:35AM -0500, Anthony Liguori wrote: > At some point in the past, the OPEN event was changed to be issued from a > bottom half. This creates a small window whereas a data callback registered > in > init may be invoked before the OPEN event has been issued. > > This is

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 15/17] xl: support spawn/destroy on multiple device model

2012-08-23 Thread Ian Campbell
On Wed, 2012-08-22 at 13:32 +0100, Julien Grall wrote: > Old configuration file is still working with qemu disaggregation. > Before to spawn any QEMU, the toolstack will fill correctly, if needed, > configuration structure. > > For the moment, the toolstack spawns device models one by one. > > Si

[Qemu-devel] [PATCH 0/2] Add TPCI200 and IP-Octal 232 IndustryPack emulation

2012-08-23 Thread Alberto Garcia
Hello, I have been working on the emulation of the IP-Octal 232 IndustryPack module, a device that implements eight RS-232 serial ports. IndustryPack modules are small boards that are attached to a carrier board, so in order to have a complete and working system I also wrote an emulation of the T

[Qemu-devel] [PATCH 1/2] Add TEWS TPCI200 IndustryPack emulation

2012-08-23 Thread Alberto Garcia
The TPCI200 is a PCI board that supports up to 4 IndustryPack modules. A new bus type called 'IndustryPack' has been created so any compatible module can be attached to this board. Signed-off-by: Alberto Garcia --- default-configs/pci.mak |1 + hw/Makefile.objs|3 + hw/ipack.c

Re: [Qemu-devel] [PATCH v3 uq/master 0/6] kvm: Get coalesced MMIO flushing out of the hot-path

2012-08-23 Thread Marcelo Tosatti
On Thu, Aug 23, 2012 at 01:02:28PM +0200, Jan Kiszka wrote: > This is just a repost, now targeting uq/master as agreed. No changes > compared to v2 except that "i82378: Remove bogus MMIO coalescing" was > dropped as it is already in QEMU upstream by now. > > Original description: > > We currently

Re: [Qemu-devel] [PATCH] Add guest-get-hostname to retrieve the guests current hostname

2012-08-23 Thread Michael Roth
On Thu, Aug 23, 2012 at 09:48:54AM -0300, Luiz Capitulino wrote: > On Tue, 21 Aug 2012 13:57:54 +0200 > Guido Günther wrote: > > > This allows to retrieve the guest's hostname via gethostname(2). > > > > This can be useful to identify a VM e.g. one without network. > > > > Signed-off-by: Guido

Re: [Qemu-devel] [PATCH v2] kvm-all.c: Move init of irqchip_inject_ioctl out of kvm_irqchip_create()

2012-08-23 Thread Peter Maydell
On 23 August 2012 10:13, Marcelo Tosatti wrote: > On Tue, Aug 21, 2012 at 12:27:51PM +0200, Jan Kiszka wrote: >> On 2012-08-21 10:25, Peter Maydell wrote: >> > On 21 August 2012 09:19, Jan Kiszka wrote: >> >> As it's not yet merged, some late comment: irqchip_inject_ioctl should >> >> be renamed

Re: [Qemu-devel] [PATCH v6 03/13] target-mips-ase-dsp: Add branch instructions

2012-08-23 Thread Aurelien Jarno
On Tue, Aug 21, 2012 at 02:53:09PM +0800, Jia Liu wrote: > Add MIPS ASE DSP Branch instructions. > > Signed-off-by: Jia Liu > --- > target-mips/translate.c | 52 > +++ > 1 file changed, 48 insertions(+), 4 deletions(-) > > diff --git a/target-mips/

[Qemu-devel] [PATCH 2/2] Add IP-Octal 232 IndustryPack emulation

2012-08-23 Thread Alberto Garcia
The IP-Octal 232 is an IndustryPack module that implements eight RS-232 serial ports, each one of which can be redirected to a character device in the host. Signed-off-by: Alberto Garcia --- hw/Makefile.objs |2 +- hw/ipoctal.c | 662 +

Re: [Qemu-devel] [PATCH v6 04/13] target-mips-ase-dsp: Add load instructions

2012-08-23 Thread Aurelien Jarno
On Tue, Aug 21, 2012 at 02:53:10PM +0800, Jia Liu wrote: > Add MIPS ASE DSP Load instructions. > > Signed-off-by: Jia Liu > --- > target-mips/translate.c | 69 > +++ > 1 file changed, 69 insertions(+) > > diff --git a/target-mips/translate.c b/targ

Re: [Qemu-devel] [QEMU][RFC V2 01/10] xen: add new machine options to support QEMU disaggregation in Xen environment

2012-08-23 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Julien Grall wrote: > - xen_dmid: specify the id of QEMU. It will be used to > retrieve/store information inside XenStore. > - xen_default_dev (on/off): as default devices need to be create in > each QEMU (due to code dependency), this option specifies if it wil

Re: [Qemu-devel] [QEMU][RFC V2 05/10] xen-memory: register memory/IO range in Xen

2012-08-23 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Julien Grall wrote: > Add Memory listener on IO and modify the one on memory. > Becareful, the first listener is not called is the range is still register > with > register_ioport*. So Xen will never know that this QEMU is handle the range. I don't understand what you mean he

Re: [Qemu-devel] [QEMU][RFC V2 06/10] xen-pci: register PCI device in Xen and handle IOREQ_TYPE_PCI_CONFIG

2012-08-23 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Julien Grall wrote: > With QEMU disaggregation QEMU needs to specify which PCI device it's able to > handle. It will use the device place in the topology (domain, bus, device, > function). > When Xen will trap an access for the config space, it will forge a new > ioreq and forw

Re: [Qemu-devel] [QEMU][RFC V2 07/10] xen: specify which device is part of default devices

2012-08-23 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Julien Grall wrote: > One major problem of QEMU disaggregation is that some devices needs to > be "emulate" in each QEMU, but only one need to register it in Xen. > > This patch introduces helpers that can be used in QEMU code (for > instance hw/pc_piix.c) to specify if the de

Re: [Qemu-devel] [QEMU][RFC V2 08/10] xen: audio is not a part of default devices

2012-08-23 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Julien Grall wrote: > Signed-off-by: Julien Grall > --- > arch_init.c |6 ++ > 1 files changed, 6 insertions(+), 0 deletions(-) > > diff --git a/arch_init.c b/arch_init.c > index 9b46bfc..1077b16 100644 > --- a/arch_init.c > +++ b/arch_init.c > @@ -44,6 +44,7 @@ > #

Re: [Qemu-devel] [QEMU][RFC V2 10/10] xen: emulate IDE outside default device set

2012-08-23 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Julien Grall wrote: > IDE can be emulate in a different QEMU that the default. > > This patch also fixes ide_get_geometry. When QEMU didn't emulate IDE, ^doesn't > it try to derefence a NULL bus. ^tries to dereference

Re: [Qemu-devel] [PATCH v6 08/13] target-mips-ase-dsp: Add bit/manipulation instructions

2012-08-23 Thread Aurelien Jarno
On Tue, Aug 21, 2012 at 02:53:14PM +0800, Jia Liu wrote: > Add MIPS ASE DSP Bit/Manipulation instructions. > > Signed-off-by: Jia Liu > --- > target-mips/dsp_helper.c | 79 + > target-mips/helper.h |7 ++ > target-mips/translate.c | 289 >

Re: [Qemu-devel] [PATCH for-1.2 v2] target-mips: Enable access to required RDHWR hardware registers

2012-08-23 Thread Aurelien Jarno
On Tue, Aug 21, 2012 at 12:31:37PM -0500, Meador Inge wrote: > While running in the usermode emulator all of the required* > MIPS32r2 RDHWR hardware registers should be accessible (the > Linux kernel enables access to these same registers). Note > that these registers are still enabled when the MI

Re: [Qemu-devel] passing translated address out in QEMU

2012-08-23 Thread Xin Tong
I am emulating arm on x86. i want to track the virt and physical address of last memory operation. so i put 2 fields in the CPUState and make tcg_global_mem_new_i32 on them Therefore, before every translation i generate code to save the virtual address as follow: static inline void gen_st32(TCGv v

[Qemu-devel] [PATCH ] lan9118: fix multicast filtering

2012-08-23 Thread Aurelien Jarno
The lan9118 emulation tries to compute the multicast index by calling directly the crc32() function from zlib, but fails to get the correct result. Use the common compute_mcast_idx() function instead, which gives the correct result. This fixes IPv6 support. Signed-off-by: Aurelien Jarno --- hw/

Re: [Qemu-devel] passing translated address out in QEMU

2012-08-23 Thread Peter Maydell
On 23 August 2012 16:34, Xin Tong wrote: > I am emulating arm on x86. i want to track the virt and physical > address of last memory operation. so i put 2 fields in the CPUState > and make tcg_global_mem_new_i32 on them Therefore, before every > translation i generate code to save the virtual addr

[Qemu-devel] [ANNOUNCE] QEMU 1.2.0-rc3 release

2012-08-23 Thread anthony
Hi, On behalf of the QEMU Team, I'd like to announce the availability of the first release candidate for the QEMU 1.2 release. This release is meant for testing purposes and should not be used in a production environment. http://wiki.qemu.org/download/qemu-1.2.0-rc1.tar.bz2 You can help improve

Re: [Qemu-devel] [PATCH for-1.2] linux-user: Clarify "Unable to reserve guest address space" error

2012-08-23 Thread Peter Maydell
Ping! This didn't get into rc1. thanks -- PMM On 20 August 2012 11:36, Peter Maydell wrote: > Now that we default to reserving nearly 4GB of RAM for the guest > address space when running a 32 bit linux-user guest on 64 bit > hosts, users are much more likely to run into it. Reword the > message

Re: [Qemu-devel] passing translated address out in QEMU

2012-08-23 Thread Xin Tong
On Thu, Aug 23, 2012 at 8:52 AM, Peter Maydell wrote: > On 23 August 2012 16:34, Xin Tong wrote: >> I am emulating arm on x86. i want to track the virt and physical >> address of last memory operation. so i put 2 fields in the CPUState >> and make tcg_global_mem_new_i32 on them Therefore, before

Re: [Qemu-devel] qemu-kvm 1.1.1 hangs using 100% CPU when using ES1370 emulation

2012-08-23 Thread Jan Kiszka
[likely not kvm related, CC'ing the appropriate community] On 2012-08-23 18:16, Mike Gerber wrote: > Hi, > > I'm using a KVM guest to stream audio using darkice to an icecast2 server on > the > same guest. The guest uses an emulated ES1370 sound card to capture the host's > audio input (ASUS Xon

[Qemu-devel] [PATCH] Better name usb braille device

2012-08-23 Thread Samuel Thibault
Windows users need to know that they have to use the Baum driver to make the qemu braille device work. Signed-off-by: Samuel Thibault diff --git a/hw/usb/dev-serial.c b/hw/usb/dev-serial.c index 56743ee..20d60ba 100644 --- a/hw/usb/dev-serial.c +++ b/hw/usb/dev-serial.c @@ -113,7 +113,7 @@ enum

Re: [Qemu-devel] [QEMU][RFC V2 05/10] xen-memory: register memory/IO range in Xen

2012-08-23 Thread Julien Grall
On 08/23/2012 03:41 PM, Stefano Stabellini wrote: On Wed, 22 Aug 2012, Julien Grall wrote: Add Memory listener on IO and modify the one on memory. Becareful, the first listener is not called is the range is still register with register_ioport*. So Xen will never know that this QEMU is handle

[Qemu-devel] [Bug 1037606] Re: vmwgfx does not work with kvm vmware vga

2012-08-23 Thread Joseph Salisbury
Would it be possible for you to test the latest upstream kernel? Refer to https://wiki.ubuntu.com/KernelMainlineBuilds . Please test the latest v3.6 kernel[0] (Not a kernel in the daily directory) and install both the linux-image and linux-image-extra .deb packages. Once you've tested the upstrea

Re: [Qemu-devel] [PATCH V5 2/8] hw/acpi_piix4.c: replace register_ioport*

2012-08-23 Thread Jan Kiszka
On 2012-08-22 14:27, Julien Grall wrote: > This patch replaces all register_ioport* with the new memory API. It permits > to use the new Memory stuff like listener. > > Signed-off-by: Julien Grall > --- > hw/acpi_piix4.c | 160 ++ > 1 files ch

Re: [Qemu-devel] [PATCH V5 0/8] memory: unifiy ioport registration

2012-08-23 Thread Jan Kiszka
On 2012-08-22 14:27, Julien Grall wrote: > This is the fifth version of patch series about ioport registration. > The fourth version was sent few months ago > (https://lists.gnu.org/archive/html/qemu-devel/2012-04/msg01396.html). > > Some part of QEMU still use register_ioport* functions to regist

Re: [Qemu-devel] [RFC][PATCH] qemu-timer: Run timers in alarm timer handler

2012-08-23 Thread Jan Kiszka
On 2012-08-23 15:11, Paolo Bonzini wrote: >>> So, in order to remove the qemu_notify_event completely, what about not >>> using signals anymore for timers? You could just tweak the select >>> timeout and drop all the -clock madness. Zero syscalls, practically no >>> overhead. If this is not prec

[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, i

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 11/17] xc: modify save/restore to support multiple device models

2012-08-23 Thread Julien Grall
On 08/23/2012 02:27 PM, Ian Campbell wrote: @@ -103,6 +103,9 @@ static ssize_t rdexact(xc_interface *xch, struct restore_ctx *ctx, #else #define RDEXACT read_exact #endif + +#define QEMUSIG_SIZE 21 + /* ** In the state file (or during transfer), all page-table pages are ** converte

Re: [Qemu-devel] [Xen-devel] [XEN][RFC PATCH V2 11/17] xc: modify save/restore to support multiple device models

2012-08-23 Thread Ian Campbell
On Thu, 2012-08-23 at 20:13 +0100, Julien Grall wrote: > On 08/23/2012 02:27 PM, Ian Campbell wrote: > > > >> @@ -103,6 +103,9 @@ static ssize_t rdexact(xc_interface *xch, struct > >> restore_ctx *ctx, > >> #else > >> #define RDEXACT read_exact > >> #endif > >> + > >> +#define QEMUSIG_SIZE 2

Re: [Qemu-devel] Is is possible to virtualise or share the TPM?

2012-08-23 Thread Corey Bryant
On 08/21/2012 06:31 AM, Jordi Cucurull Juan wrote: Dear all, After applying the TPM patches to QEMU, I was wondering if it is possible to simultaneously use the TPM in more than one virtual machine, i.e. virtualisation of the TPM. According to the paper "Stefan Berger, Ramón Cáceres, Kenneth

  1   2   >