Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Alexander Graf
On 22.08.2012, at 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote: cpu_physical_memory_write_rom(), despite the name, can also be used to write images into RAM - and will often be used that way if the

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread David Gibson
On Wed, Aug 22, 2012 at 08:02:11AM +0200, Alexander Graf wrote: On 22.08.2012, at 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote: cpu_physical_memory_write_rom(), despite the name, can also be

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Alexander Graf
On 22.08.2012, at 08:10, David Gibson wrote: On Wed, Aug 22, 2012 at 08:02:11AM +0200, Alexander Graf wrote: On 22.08.2012, at 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote:

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

2012-08-22 Thread Wen Congyang
At 08/15/2012 04:53 AM, Marcelo Tosatti Wrote: On Tue, Aug 14, 2012 at 02:35:34PM -0500, Anthony Liguori wrote: Marcelo Tosatti mtosa...@redhat.com writes: On Tue, Aug 14, 2012 at 01:53:01PM -0500, Anthony Liguori wrote: Marcelo Tosatti mtosa...@redhat.com writes: On Tue, Aug 14, 2012 at

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Alexander Graf
On 22.08.2012, at 08:10, David Gibson wrote: On Wed, Aug 22, 2012 at 08:02:11AM +0200, Alexander Graf wrote: On 22.08.2012, at 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote:

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Jan Kiszka
On 2012-08-22 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote: cpu_physical_memory_write_rom(), despite the name, can also be used to write images into RAM - and will often be used that way if the machine

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Jan Kiszka
On 2012-08-22 08:47, Jan Kiszka wrote: On 2012-08-22 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote: cpu_physical_memory_write_rom(), despite the name, can also be used to write images into RAM - and

[Qemu-devel] [PATCH] xen-all.c: fix multiply issue for int and uint types

2012-08-22 Thread Dongxiao Xu
If the two multiply operands are int and uint types separately, the int type will be transformed to uint firstly, which is not the intent in our code piece. The fix is to add (int64_t) transform for the uint type before the multiply. Signed-off-by: Dongxiao Xu dongxiao...@intel.com --- xen-all.c

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

2012-08-22 Thread Wen Congyang
At 08/09/2012 03:01 AM, Blue Swirl Wrote: On Wed, Aug 8, 2012 at 2:47 AM, Wen Congyang we...@cn.fujitsu.com 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

Re: [Qemu-devel] [PATCH] Correct computation of bytes per pixel from bits per pixel

2012-08-22 Thread Jan Kiszka
Can you use this free space to provide a scenario where the missing round-up caused a problem? On 2012-08-21 23:32, BALATON Zoltan wrote: Signed-off-by: BALATON Zoltan bala...@eik.bme.hu --- console.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/console.c

Re: [Qemu-devel] [PATCH] vmware_vga: Cleanup and allow simple drivers to work without the fifo

2012-08-22 Thread Jan Kiszka
On 2012-08-21 23:33, BALATON Zoltan wrote: Detailed changes: Removing info available elsewhere from vmsvga_state. Fix mixup between depth and bits per pixel. Return a value for FB_SIZE even before enabled (according to the documentation, drivers should read this value before enabling the

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

2012-08-22 Thread Guido Günther
On Tue, Aug 21, 2012 at 07:31:17PM +0100, Daniel P. Berrange wrote: On Tue, Aug 21, 2012 at 01:57:54PM +0200, Guido Günther wrote: [..snip..] Why no impl ? Winsock has the gethostname() API too $ grep gethostname /usr/i686-w64-mingw32/sys-root/mingw/include/*.h

Re: [Qemu-devel] ivshmem assertion failure with EventNotifier

2012-08-22 Thread Jan Kiszka
On 2012-08-22 06:29, Cam Macdonell wrote: Hi Paolo, I've noticed an assertion error when sending interrupts via ivshmem. I bisected to this patch. commit 563027cc0c94aa4846c18f9d665a4c90f8c42ba8 Author: Paolo Bonzini pbonz...@redhat.com Date: Thu Jul 5 17:16:25 2012 +0200

[Qemu-devel] setting up breakpoints - kernel debugging help wit gdb in qemu-kvm

2012-08-22 Thread Onkar
I am running a linux guest like this - which I wish to debug - but the breakpoint gets set but I am not able to stop the guest execution at the set breakpoint - it just hangs after I do continue. Can you please provide me some clues - # ./qemu-system-x86_64 -m 2048 -smp 4 -vga std -vnc :5

[Qemu-devel] [PATCH v12 2/2] configure: print spice-protocol and spice-server versions

2012-08-22 Thread Alon Levy
Signed-off-by: Alon Levy al...@redhat.com --- configure | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 84fa2c6..02c06aa 100755 --- a/configure +++ b/configure @@ -2657,6 +2657,8 @@ EOF spice=yes libs_softmmu=$libs_softmmu $spice_libs

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

2012-08-22 Thread Alon Levy
Revision bumped to 4 for new IO support, enabled for spice-server = 0.11.1. New io enabled iff revision is 4. Revision can be set to 4, and defaults to 4, iff spice-server = 0.11.1 spice-protocol = 0.12.0. This io calls the corresponding new spice api spice_qxl_monitors_config_async to let

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

2012-08-22 Thread Stefan Hajnoczi
On Tue, Aug 21, 2012 at 7:31 PM, Blue Swirl blauwir...@gmail.com wrote: On Tue, Aug 21, 2012 at 8:23 AM, Cong Meng m...@linux.vnet.ibm.com wrote: diff --git a/block_int.h b/block_int.h index d72317f..a9d07a2 100644 --- a/block_int.h +++ b/block_int.h @@ -333,6 +333,10 @@ struct

Re: [Qemu-devel] [PATCH] Correct computation of bytes per pixel from bits per pixel

2012-08-22 Thread Peter Maydell
On 21 August 2012 22:32, BALATON Zoltan bala...@eik.bme.hu wrote: If you make the Subject console: Correct computation of bytes per pixel from bits per pixel it's easier to tell what bit of qemu the patch is dealing with. Signed-off-by: BALATON Zoltan bala...@eik.bme.hu --- console.c |4

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

2012-08-22 Thread Daniel P. Berrange
On Wed, Aug 22, 2012 at 10:04:33AM +0200, Guido Günther wrote: On Tue, Aug 21, 2012 at 07:31:17PM +0100, Daniel P. Berrange wrote: On Tue, Aug 21, 2012 at 01:57:54PM +0200, Guido Günther wrote: [..snip..] Why no impl ? Winsock has the gethostname() API too $ grep gethostname

[Qemu-devel] [PATCH 0/5] trace-cmd: Add a recorder readable feature for virtio-trace

2012-08-22 Thread Yoshihiro YUNOMAE
Hi Steven, The following patch set provides a feature which can read trace data of a guest using virtio-trace (https://lkml.org/lkml/2012/8/9/210) for a recorder function of trace-cmd. This patch set depends on the trace-agent running on a guest in the virtio-trace system. To translate raw data

[Qemu-devel] [PATCH 1/5] trace-cmd: Use TRACE_DIR envrionment variable if defined

2012-08-22 Thread Yoshihiro YUNOMAE
From: Masami Hiramatsu masami.hiramatsu...@hitachi.com Use TRACE_DIR environment variable for setting debugfs/tracing directory if defined. This is for controlling guest(or remote) ftrace. Signed-off-by: Masami Hiramatsu masami.hiramatsu...@hitachi.com Signed-off-by: Yoshihiro YUNOMAE

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

2012-08-22 Thread Yoshihiro YUNOMAE
From: Masami Hiramatsu masami.hiramatsu...@hitachi.com Count debugfs/tracing/per_cpu/cpu* to determine the number of CPUs. Signed-off-by: Masami Hiramatsu masami.hiramatsu...@hitachi.com Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- trace-record.c | 41

[Qemu-devel] [PATCH 5/5] trace-cmd: Use polling function

2012-08-22 Thread Yoshihiro YUNOMAE
Use poll() for avoiding a busy loop to read trace data of a guest from FIFO. Signed-off-by: Yoshihiro YUNOMAE yoshihiro.yunomae...@hitachi.com --- trace-recorder.c | 42 -- 1 files changed, 36 insertions(+), 6 deletions(-) diff --git a/trace-recorder.c

[Qemu-devel] [PATCH 4/5] trace-cmd: Add non-blocking option for open() and splice_read()

2012-08-22 Thread Yoshihiro YUNOMAE
Add non-blocking option for open() and splice_read() for avoiding block to read trace data of a guest from FIFO. If SIGINT comes to read/write processes from the parent process in the case where FIFO as a read I/F is assigned, then reading is normally blocked for splice_read(). So, we added

[Qemu-devel] [PATCH 3/5] trace-cmd: Support trace-agent of virtio-trace

2012-08-22 Thread Yoshihiro YUNOMAE
Add read path and control path to use trace-agent of virtio-trace. When we use trace-agent, trace-cmd will be used as follows: # AGENT_READ_DIR=/tmp/virtio-trace/tracing \ AGENT_CTL=/tmp/virtio-trace/agent-ctl-path.in \ TRACING_DIR=/tmp/virtio-trace/debugfs/tracing \

Re: [Qemu-devel] [RFC 0/8] include qdev core in *-user, make CPU child of DeviceState

2012-08-22 Thread Avi Kivity
On 08/21/2012 06:42 PM, Eduardo Habkost wrote: So, here's a third suggestion to the CPU/DeviceState problem. Basically I split the qdev code into a core (that can be easily compiled into *-user), and a part specific to qemu-system-*. I'm barging in late here, so sorry if this has been

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-22 Thread Avi Kivity
On 08/21/2012 06:21 PM, Richard Davies wrote: Avi Kivity wrote: Richard Davies wrote: We're running host kernel 3.5.1 and qemu-kvm 1.1.1. I hadn't though about it, but I agree this is related to cpu overcommit. The slow boots are intermittent (and infrequent) with cpu overcommit

Re: [Qemu-devel] [PATCH v2 1/3] net: asynchronous send/receive infrastructure for net/socket.c

2012-08-22 Thread Stefan Hajnoczi
On Tue, Aug 21, 2012 at 5:39 PM, ronnie sahlberg ronniesahlb...@gmail.com wrote: In net_socket_update_fd_handler() shouldnt you call qemu_notify_event() if any of the handlers have changed from NULL to non-NULL ? or else it might take a while before the change takes effect. In this case

[Qemu-devel] [Bug 657006] Re: arm v7M - svc insn doesn't trigger PendSV handler

2012-08-22 Thread Mark Phillips
OK, I'll re-read the documentation maybe I am wrong! It does say synchronous in the description and I don't understand how it can work if it is asynchronous because for Cortex the SVC argument is not transfered to a register and the only way the exception code can access it is by reading it

[Qemu-devel] [Bug 657006] Re: arm v7M - svc insn doesn't trigger PendSV handler

2012-08-22 Thread Mark Phillips
In particular table 2-16 of DUI0552A_Cortex_m3_dgug.pdf states that the Activation of the SVC exception is Synchronous. And after the table it states For an asynchronous exception, other than reset, the processor can execute another instruction between when the exception is triggered and when

[Qemu-devel] [Bug 657006] Re: arm v7M - svc insn doesn't trigger PendSV handler

2012-08-22 Thread Peter Maydell
Yes, I'm not disputing that the SVC exception should be handled synchronously, I'm asking you to provide a test case demonstrating the wrong behaviour. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU.

Re: [Qemu-devel] setting up breakpoints - kernel debugging help wit gdb in qemu-kvm

2012-08-22 Thread Mulyadi Santosa
Hi... On Wed, Aug 22, 2012 at 6:53 AM, Onkar kern...@gmail.com wrote: # ./qemu-system-x86_64 -m 2048 -smp 4 -vga std -vnc :5 -drive file=/sda4/bin/disk/disk0.img,cache=writeback -S -s (gdb) symbol-file /sda4/kvm/vmlinux Reading symbols from /sda4/kvm/vmlinux...done. (gdb) target remote

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

2012-08-22 Thread Avi Kivity
On 08/21/2012 12:28 PM, Tomas Racek wrote: http://fi.muni.cz/~xracek/debian2.img.bz2 Other things are the same. The runtest.sh sets environment for xfstests and runs test 285 which I wrote and and which should test if FS sends discard requests only on free sectors: 285: 1. Create loop

[Qemu-devel] [PATCH] x86, alternative: fix p6 nops on non-modular kernels

2012-08-22 Thread Avi Kivity
On 08/22/2012 12:54 PM, Avi Kivity wrote: On 08/21/2012 12:28 PM, Tomas Racek wrote: http://fi.muni.cz/~xracek/debian2.img.bz2 Other things are the same. The runtest.sh sets environment for xfstests and runs test 285 which I wrote and and which should test if FS sends discard requests

Re: [Qemu-devel] [PATCH] xen-all.c: fix multiply issue for int and uint types

2012-08-22 Thread Stefano Stabellini
On Wed, 22 Aug 2012, Dongxiao Xu wrote: If the two multiply operands are int and uint types separately, the int type will be transformed to uint firstly, which is not the intent in our code piece. The fix is to add (int64_t) transform for the uint type before the multiply. Signed-off-by:

Re: [Qemu-devel] [PATCH] vmware_vga: Cleanup and allow simple drivers to work without the fifo

2012-08-22 Thread BALATON Zoltan
On Wed, 22 Aug 2012, Jan Kiszka wrote: This is a rather big patch. I strongly suspect you can break it up into smaller pieces that address separate aspects one-by-one. Also, it is definitely to heavy for qemu-trivial. Despite its size the changes included are fairly simple but I can try to

[Qemu-devel] [PULL] Xen fixes 2012-08-22

2012-08-22 Thread Stefano Stabellini
Hi Anthony, please pull two fixes to xen-all.c: git://xenbits.xen.org/people/sstabellini/qemu-dm.git xen-fixes-20120822 Dongxiao Xu (1): xen-all.c: fix multiply issue for int and uint types Frediano Ziglio (1): Fix invalidate if memory requested was not bucket aligned xen-all.c

Re: [Qemu-devel] setting up breakpoints - kernel debugging help wit gdb in qemu-kvm

2012-08-22 Thread Jan Kiszka
On 2012-08-22 01:53, Onkar wrote: I am running a linux guest like this - which I wish to debug - but the breakpoint gets set but I am not able to stop the guest execution at the set breakpoint - it just hangs after I do continue. Can you please provide me some clues - #

[Qemu-devel] [Qemu-ppc][PATCH v9 1/1] Add USB option in machine options

2012-08-22 Thread Li Zhang
When -usb option is used, global varible usb_enabled is set. And all the plafrom will create one USB controller according to this variable. In fact, global varibles make code hard to read. So this patch is to remove global variable usb_enabled and add USB option in machine options. All the

Re: [Qemu-devel] [PATCH] x86, alternative: fix p6 nops on non-modular kernels

2012-08-22 Thread Tomas Racek
- Original Message - On 08/22/2012 12:54 PM, Avi Kivity wrote: On 08/21/2012 12:28 PM, Tomas Racek wrote: http://fi.muni.cz/~xracek/debian2.img.bz2 Other things are the same. The runtest.sh sets environment for xfstests and runs test 285 which I wrote and and which

[Qemu-devel] [Bug 988291] Re: virsh -c qemu+ssh://root@source/system migrate --domain client --desturi qemu+ssh://root@destination/system fails

2012-08-22 Thread Aaron Toponce
*** This bug is a duplicate of bug 989452 *** https://bugs.launchpad.net/bugs/989452 He means disabling kerberos authentication in your SSH client config, whether it be globally in /etc/ssh/ssh_config or ~/.ssh/config, assuming you're using qemu+ssh:// to migrate your VMs. Either way, this

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

2012-08-22 Thread Cong Meng
On 08/21/2012 06:14 PM, Paolo Bonzini wrote: Il 21/08/2012 11:52, Stefan Hajnoczi ha scritto: Using /sys/dev/block or /sys/dev/char seems easier, and lets you retrieve the parameters for block devices too. what do you mean with block devices? Using /dev/sda instead of /dev/sg0? Yes.

[Qemu-devel] [PATCH v4] configure: properly check if -lrt and -lm is needed

2012-08-22 Thread Natanael Copa
Fixes build against uClibc. uClibc provides 2 versions of clock_gettime(), one with realtime support and one without (this is so you can avoid linking in -lrt unless actually needed). This means that the clock_gettime() don't need -lrt. We still need it for timer_create() so we check for this

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread David Gibson
On Wed, Aug 22, 2012 at 09:05:52AM +0200, Jan Kiszka wrote: On 2012-08-22 08:47, Jan Kiszka wrote: On 2012-08-22 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson wrote: cpu_physical_memory_write_rom(),

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Alexander Graf
On 22.08.2012, at 13:38, David Gibson wrote: On Wed, Aug 22, 2012 at 09:05:52AM +0200, Jan Kiszka wrote: On 2012-08-22 08:47, Jan Kiszka wrote: On 2012-08-22 07:57, David Gibson wrote: On Wed, Aug 22, 2012 at 07:55:31AM +0200, Alexander Graf wrote: On 22.08.2012, at 06:59, David Gibson

[Qemu-devel] [PATCH] s390: use sync regs for register transfer

2012-08-22 Thread Jens Freimann
From: Christian Borntraeger borntrae...@de.ibm.com Newer kernels provide the guest registers in kvm_run. Lets use those if available. This avoids ioctls on cpu_synchronize_state making intercepts faster. In addition, we have now the prefix register, the access registers the control registers up

[Qemu-devel] [PATCHv3 1/5] qemu-ga: don't leak a file descriptor upon failed lockf

2012-08-22 Thread Jim Meyering
From: Jim Meyering meyer...@redhat.com Signed-off-by: Jim Meyering meyer...@redhat.com --- qemu-ga.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/qemu-ga.c b/qemu-ga.c index 8f87621..26671fe 100644 --- a/qemu-ga.c +++ b/qemu-ga.c @@ -247,6 +247,9 @@ static bool ga_open_pidfile(const

[Qemu-devel] [PATCHv3 2/5] linux-user: do_msgrcv: don't leak host_mb upon TARGET_EFAULT failure

2012-08-22 Thread Jim Meyering
From: Jim Meyering meyer...@redhat.com Also, use g_malloc to avoid NULL-deref upon OOM. Signed-off-by: Jim Meyering meyer...@redhat.com --- linux-user/syscall.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index

[Qemu-devel] [PATCHv3 0/5] plug memory and file-descriptor leaks

2012-08-22 Thread Jim Meyering
From: Jim Meyering meyer...@redhat.com Hi Anthony, I posted this series back in May, got some good feedback leading to a pair of v2 patches. Since then one of the 6 patches was applied. I'm calling this v3, but it is merely a trivial rebase of the v1 and v2 patches. Hoping it's not too late

[Qemu-devel] [PATCHv3 4/5] arm-semi: don't leak 1KB user string lock buffer upon TARGET_SYS_OPEN

2012-08-22 Thread Jim Meyering
From: Jim Meyering meyer...@redhat.com Always call unlock_user before returning. Signed-off-by: Jim Meyering meyer...@redhat.com --- target-arm/arm-semi.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/target-arm/arm-semi.c b/target-arm/arm-semi.c index

[Qemu-devel] [PATCHv3 3/5] sheepdog: don't leak socket file descriptor upon connection failure

2012-08-22 Thread Jim Meyering
From: Jim Meyering meyer...@redhat.com Signed-off-by: Jim Meyering meyer...@redhat.com --- block/sheepdog.c | 1 + 1 file changed, 1 insertion(+) diff --git a/block/sheepdog.c b/block/sheepdog.c index a04ad99..df4f441 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -485,6 +485,7 @@

[Qemu-devel] [PATCHv3 5/5] softmmu-semi: fix lock_user* functions not to deref NULL upon OOM

2012-08-22 Thread Jim Meyering
From: Jim Meyering meyer...@redhat.com Return NULL upon malloc failure. Signed-off-by: Jim Meyering meyer...@redhat.com --- softmmu-semi.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/softmmu-semi.h b/softmmu-semi.h index 648cb95..bcb979a 100644 --- a/softmmu-semi.h

Re: [Qemu-devel] ivshmem assertion failure with EventNotifier

2012-08-22 Thread Paolo Bonzini
Il 22/08/2012 06:29, Cam Macdonell ha scritto: Hi Paolo, I've noticed an assertion error when sending interrupts via ivshmem. I bisected to this patch. Does this help? diff --git a/hw/ivshmem.c b/hw/ivshmem.c index b4d65a6..47f2a16 100644 --- a/hw/ivshmem.c +++ b/hw/ivshmem.c @@ -366,6

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

2012-08-22 Thread Paolo Bonzini
Il 22/08/2012 05:02, Liu Ping Fan ha scritto: From: Liu Ping Fan pingf...@linux.vnet.ibm.com Currently, link property's target is only managed by object_set_link_property(). This will raise such issue that when the property is finalized, its target has no opportunity to release. Fix this

Re: [Qemu-devel] Dump guest page table inside QEMU makes system hang

2012-08-22 Thread Max Filippov
On Tue, Aug 21, 2012 at 10:19 PM, Blue Swirl blauwir...@gmail.com wrote: On Tue, Aug 21, 2012 at 7:21 AM, 陳韋任 (Wei-Ren Chen) che...@iis.sinica.edu.tw wrote: Hi all, I want to dump guest page table when guest writes to cr3, the code snipt below, --- uint32_t pgd[1024][1024]; // guest

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

2012-08-22 Thread Benoît Canet
Le Thursday 16 Aug 2012 à 11:13:05 (+0800), Wenchao Xia a écrit : I was busy in coding libqblock so forgot to move forward on qemu json info patches, thanks for your advance. one suggestion: how about folder all human printf into function: dump_human_image_info(image_info), and at

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

2012-08-22 Thread Paolo Bonzini
Il 22/08/2012 13:04, Cong Meng ha scritto: Cong, what is the limit that the host HBA enforces (and what is the HBA)? What commands see a problem? Is it fixed by using scsi-block instead of scsi-generic (if you can use scsi-block at all, i.e. it's not a tape or similar device)? I don't see

[Qemu-devel] [Bug 657006] Re: arm v7M - svc insn doesn't trigger PendSV handler

2012-08-22 Thread Mark Phillips
ok, I'll double check that backing out the local patches doesn't make a difference. If it still happens I will try and come up with a reduced test case. What do you expect to happen? Should the SVC exception 11 run immediately? What should happen if a clock tick interrupt is also pending at level

[Qemu-devel] FYI: Delaying 1.2 release to Sept. 5th.

2012-08-22 Thread anthony
I cannot provide additional details so please don't ask. There's a issue in the 1.2 release being investigated. We need to delay the release by a few days to ensure the fix can be in the release. I've updated the wiki appropriately. Regards, Anthony Liguori

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-22 Thread Avi Kivity
On 08/22/2012 03:40 PM, Richard Davies wrote: I can trigger the slow boots without KSM and they have the same profile, with _raw_spin_lock_irqsave and isolate_freepages_block at the top. I reduced to 3x 20GB 8-core VMs on a 128GB host (rather than 3x 40GB 8-core VMs), and haven't managed

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

2012-08-22 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

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

2012-08-22 Thread Benoît Canet
Signed-off-by: Benoit Canet ben...@irqsave.net --- qapi-schema.json | 62 ++ 1 file changed, 62 insertions(+) diff --git a/qapi-schema.json b/qapi-schema.json index a92adb1..4c4b9b9 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@

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

2012-08-22 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: 20637102488, name: vm-20120821145509, date-sec: 1345553709, date-nsec:

Re: [Qemu-devel] qemu-kvm-1.0.1 - unable to exit if vcpu is in infinite loop

2012-08-22 Thread Peter Lieven
On 08/21/12 10:23, Stefan Hajnoczi wrote: On Tue, Aug 21, 2012 at 8:21 AM, Jan Kiszkajan.kis...@siemens.com wrote: On 2012-08-19 11:42, Avi Kivity wrote: On 08/17/2012 06:04 PM, Jan Kiszka wrote: Can anyone imagine that such a barrier may actually be required? If it is currently possible

Re: [Qemu-devel] [RFC 0/8] include qdev core in *-user, make CPU child of DeviceState

2012-08-22 Thread Eduardo Habkost
On Wed, Aug 22, 2012 at 12:05:44PM +0300, Avi Kivity wrote: On 08/21/2012 06:42 PM, Eduardo Habkost wrote: So, here's a third suggestion to the CPU/DeviceState problem. Basically I split the qdev code into a core (that can be easily compiled into *-user), and a part specific to

Re: [Qemu-devel] [PATCH] cpu_physical_memory_write_rom() needs to do TB invalidates

2012-08-22 Thread Avi Kivity
On 08/22/2012 02:47 PM, Alexander Graf wrote: Ok, so IIUC, that means we do need the cpu_physical_memory_write_rom() version for load_image_targphys(), and so my original patch is basically the right fix. Sure it is, I don't think anyone argued about that :). But it's duplicating code in

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

2012-08-22 Thread Stefan Hajnoczi
On Wed, Aug 22, 2012 at 02:09:28PM +0200, Paolo Bonzini wrote: Il 22/08/2012 13:04, Cong Meng ha scritto: Cong, what is the limit that the host HBA enforces (and what is the HBA)? What commands see a problem? Is it fixed by using scsi-block instead of scsi-generic (if you can use

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

2012-08-22 Thread Anthony Liguori
Daniel P. Berrange berra...@redhat.com writes: On Tue, Aug 21, 2012 at 09:53:55AM -0300, Luiz Capitulino wrote: On Tue, 21 Aug 2012 11:07:50 +0100 Daniel P. Berrange berra...@redhat.com wrote: On Mon, Aug 20, 2012 at 04:02:39PM -0300, Luiz Capitulino wrote: On Mon, 20 Aug 2012 15:31:38

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
Eric Blake ebl...@redhat.com writes: On 08/18/2012 05:17 AM, Juan Quintela wrote: Signed-off-by: Juan Quintela quint...@redhat.com --- hmp.c| 4 ++-- migration.c | 6 +++--- qapi-schema.json | 14 +++--- qmp-commands.hx | 6 +++--- 4 files changed, 15

Re: [Qemu-devel] [PATCH 1/5] trace-cmd: Use TRACE_DIR envrionment variable if defined

2012-08-22 Thread Steven Rostedt
On Wed, 2012-08-22 at 17:43 +0900, Yoshihiro YUNOMAE wrote: From: Masami Hiramatsu masami.hiramatsu...@hitachi.com Use TRACE_DIR environment variable for setting TRACING_DIR would be better, as we are searching for /debug/tracing and not /debug/trace. Perhaps DEBUG_TRACING_DIR would be even

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

2012-08-22 Thread Steven Rostedt
On Wed, 2012-08-22 at 17:43 +0900, Yoshihiro YUNOMAE wrote: From: Masami Hiramatsu masami.hiramatsu...@hitachi.com Count debugfs/tracing/per_cpu/cpu* to determine the number of CPUs. I'm curious, do you find that sysconf doesn't return the # of CPUs the system has? I've had boxes where the

Re: [Qemu-devel] [PATCH 3/5] trace-cmd: Support trace-agent of virtio-trace

2012-08-22 Thread Steven Rostedt
On Wed, 2012-08-22 at 17:43 +0900, Yoshihiro YUNOMAE wrote: Add read path and control path to use trace-agent of virtio-trace. When we use trace-agent, trace-cmd will be used as follows: # AGENT_READ_DIR=/tmp/virtio-trace/tracing \ AGENT_CTL=/tmp/virtio-trace/agent-ctl-path.in \

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

2012-08-22 Thread Eric Blake
On 08/22/2012 06:45 AM, Benoît Canet wrote: Signed-off-by: Benoit Canet ben...@irqsave.net --- qapi-schema.json | 62 ++ 1 file changed, 62 insertions(+) diff --git a/qapi-schema.json b/qapi-schema.json index a92adb1..4c4b9b9 100644

[Qemu-devel] [PATCH 1/3] qapi: Fix memory leak

2012-08-22 Thread Luiz Capitulino
From: Stefan Weil s...@weilnetz.de valgrind report: ==24534== 232 bytes in 2 blocks are definitely lost in loss record 1,245 of 1,601 ==24534==at 0x4824F20: malloc (vg_replace_malloc.c:236) ==24534==by 0x293C88: malloc_and_trace (vl.c:2281) ==24534==by 0x489AD99: ??? (in

[Qemu-devel] [PULL for-1.2 0/3]: QMP queue

2012-08-22 Thread Luiz Capitulino
Three little fixes. Two for QMP and one for HMP. The changes (since 682527c00409d676c0d3b9fac99ca3b2fdfd6d2c) are available in the following repository: git://repo.or.cz/qemu/qmp-unstable.git queue/qmp Blue Swirl (1): monitor: avoid declaring unused variables Juan Quintela (1):

[Qemu-devel] [PATCH 2/3] monitor: avoid declaring unused variables

2012-08-22 Thread Luiz Capitulino
From: Blue Swirl blauwir...@gmail.com Some variables are only used on !win32, declare them only when used. This avoids a warning in mingw32 build: CCi386-softmmu/monitor.o /src/qemu/monitor.c: In function 'monitor_fdset_get_fd': /src/qemu/monitor.c:2575: warning: unused variable

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

2012-08-22 Thread Paolo Bonzini
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 example and the Linux sr driver on the guest is going to try

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

2012-08-22 Thread Eric Blake
On 08/22/2012 06:45 AM, Benoît Canet wrote: This option --output=[human|json] make qemu-img info output on human or JSON representation at the choice of the user. @@ -1083,7 +1088,6 @@ out: return 0; } - static void dump_snapshots(BlockDriverState *bs) { Spurious whitespace

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

2012-08-22 Thread Luiz Capitulino
From: Juan Quintela quint...@redhat.com Signed-off-by: Juan Quintela quint...@redhat.com Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- hmp.c| 4 ++-- migration.c | 7 --- qapi-schema.json | 14 +++--- qmp-commands.hx | 6 +++--- 4 files changed, 16

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

2012-08-22 Thread Benoît Canet
Le Wednesday 22 Aug 2012 à 08:03:14 (-0600), Eric Blake a écrit : On 08/22/2012 06:45 AM, Benoît Canet wrote: Signed-off-by: Benoit Canet ben...@irqsave.net --- qapi-schema.json | 62 ++ 1 file changed, 62 insertions(+) diff

[Qemu-devel] [Bug 657006] Re: arm v7M - svc insn doesn't trigger PendSV handler

2012-08-22 Thread Mark Phillips
I have put together a test program and tried against a vanila copy of qemu 1.1.1 The SVC wil be completely masked unless I apply patch 0002-target-arm- Disable-priority_mask-feature.patch, which hacks arm_gic.c to initialise the gic priority_mask to 0x100 instead of 0xf0. There doesn't appear to

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

2012-08-22 Thread Eric Blake
On 08/22/2012 07:22 AM, Anthony Liguori wrote: Just restating things to make sure I'm clear... { 'type': 'MigrationInfo', 'data': {'*status': 'str', '*ram': 'MigrationStats', '*disk': 'MigrationStats', - '*xbzrle-cache': 'XBZRLECacheStats'} } +

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-22 Thread Avi Kivity
On 08/22/2012 05:41 PM, Richard Davies wrote: Avi Kivity wrote: Richard Davies wrote: I can trigger the slow boots without KSM and they have the same profile, with _raw_spin_lock_irqsave and isolate_freepages_block at the top. I reduced to 3x 20GB 8-core VMs on a 128GB host (rather than

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

2012-08-22 Thread Eric Blake
On 08/22/2012 08:32 AM, Benoît Canet wrote: Since we have two fields named *-nsec, it might be worth clarifying that date-nsec is merely the fractional portion to be combined with date-sec (always less than 10), while vm-clock-nsec includes seconds if the drift is that large. For

[Qemu-devel] memtest 4.20+ does not work with -cpu host

2012-08-22 Thread Peter Lieven
Hi, has anyone ever tested to run memtest with -cpu host flag passed to qemu-kvm? For me it resets when probing the chipset. With -cpu qemu64 it works just fine. Maybe this is specific to memtest, but it might be sth that can happen in other applications to. Any thoughts? Thanks, Peter

[Qemu-devel] Bug in option parsing

2012-08-22 Thread Martin Kletzander
Hi everybody, while coding the support for Jason's dump-guest-core option I realized there is (probably) a problem with the way QEMU parses additional machine options ('dump-guest-core', 'kvm_shadow_mem' etc.). Running QEMU with option to -machine works ok, but using '-M' (as libvirt does) works

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

2012-08-22 Thread Anthony Liguori
From: Daniel P. Berrange berra...@redhat.com Add a 'query-target' QAPI command to allow management applications to determine what target architecture a QEMU binary is emulating without having to parse the binary name or -help output $ qmp-shell -p /tmp/qemu (QEMU) query-target {

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

2012-08-22 Thread Daniel P. Berrange
You can remove '/bit size' in the commit message 1st line. On Wed, Aug 22, 2012 at 10:09:38AM -0500, Anthony Liguori wrote: From: Daniel P. Berrange berra...@redhat.com Add a 'query-target' QAPI command to allow management applications to determine what target architecture a QEMU binary is

Re: [Qemu-devel] Bug in option parsing

2012-08-22 Thread Jan Kiszka
On 2012-08-22 17:05, Martin Kletzander wrote: Hi everybody, while coding the support for Jason's dump-guest-core option I realized there is (probably) a problem with the way QEMU parses additional machine options ('dump-guest-core', 'kvm_shadow_mem' etc.). Running QEMU with option to

[Qemu-devel] [PATCH v2] Fix copypaste typos in documentation comments

2012-08-22 Thread BALATON Zoltan
Signed-off-by: BALATON Zoltan bala...@eik.bme.hu --- memory.h | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) v2: indented memory_region_init_ram_ptr diff --git a/memory.h b/memory.h index bd1bbae..f6c8e32 100644 --- a/memory.h +++ b/memory.h @@ -252,9 +252,9 @@ void

[Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel

2012-08-22 Thread BALATON Zoltan
Division with round up is the correct way to compute this even if the only case where division with round down gives incorrect result is probably 15 bpp. This case was explicitely patched up in one of these functions but was unhandled in the other. (I'm not sure about setting 16 bpp for the 15bpp

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-22 Thread Rik van Riel
On 08/22/2012 10:41 AM, Richard Davies wrote: Avi Kivity wrote: Richard Davies wrote: I can trigger the slow boots without KSM and they have the same profile, with _raw_spin_lock_irqsave and isolate_freepages_block at the top. I reduced to 3x 20GB 8-core VMs on a 128GB host (rather than 3x

Re: [Qemu-devel] [PATCH] vmware_vga: Cleanup and allow simple drivers to work without the fifo

2012-08-22 Thread Jan Kiszka
On 2012-08-22 12:19, BALATON Zoltan wrote: On Wed, 22 Aug 2012, Jan Kiszka wrote: This is a rather big patch. I strongly suspect you can break it up into smaller pieces that address separate aspects one-by-one. Also, it is definitely to heavy for qemu-trivial. Despite its size the changes

Re: [Qemu-devel] [PATCH v2] console: Correct computation of bytes per pixel from bits per pixel

2012-08-22 Thread Jan Kiszka
On 2012-08-22 17:19, BALATON Zoltan wrote: Division with round up is the correct way to compute this even if the only case where division with round down gives incorrect result is probably 15 bpp. This case was explicitely patched up in one of these functions but was unhandled in the other.

Re: [Qemu-devel] Windows slow boot: contractor wanted

2012-08-22 Thread Richard Davies
Rik van Riel wrote: Richard Davies wrote: I've now triggered a very slow boot at 3x 36GB 8-core VMs on a 128GB host (i.e. 108GB on a 128GB host). It has the same profile with _raw_spin_lock_irqsave and isolate_freepages_block at the top. That's the page compaction code. Mel Gorman

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

2012-08-22 Thread Eric Blake
On 08/22/2012 09:09 AM, Anthony Liguori wrote: From: Daniel P. Berrange berra...@redhat.com Add a 'query-target' QAPI command to allow management applications to determine what target architecture a QEMU binary is emulating without having to parse the binary name or -help output $

[Qemu-devel] [PATCH for-1.2 v2 4/5] MAINTAINERS: Document Bamboo machine and ppc4xx devices

2012-08-22 Thread Andreas Färber
Place it in alphabetical order and add new Devices section ppc4xx to share file rules with 405 and virtex_ml507. Signed-off-by: Andreas Färber afaer...@suse.de Cc: Alexander Graf ag...@suse.de --- MAINTAINERS | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git

[Qemu-devel] [PATCH for-1.2 v2 0/5] target-ppc: MAINTAINERS additions

2012-08-22 Thread Andreas Färber
Hi Alex, Here's an updated and squashed series adding some MAINTAINERS sections for ppc machines and their devices. Based on Peter's suggestion the machines are no longer grouped by ppc4xx chipset but ordered alphabetically, and the shared devices both for ppc4xx and e500 get their own Devices

[Qemu-devel] [PATCH for-1.2 v2 3/5] MAINTAINERS: Downgrade ppc405 to Odd Fixes

2012-08-22 Thread Andreas Färber
As requested by Alex. Signed-off-by: Andreas Färber afaer...@suse.de --- MAINTAINERS |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 618045b..e6b2409 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -349,7 +349,7 @@ PowerPC Machines 405 M:

[Qemu-devel] [PATCH for-1.2 v2 2/5] MAINTAINERS: Document e500 machines and devices

2012-08-22 Thread Andreas Färber
Signed-off-by: Andreas Färber afaer...@suse.de Cc: Alexander Graf ag...@suse.de Cc: Scott Wood scottw...@freescale.com --- MAINTAINERS | 23 +++ roms/SLOF |2 +- 2 files changed, 24 insertions(+), 1 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index

  1   2   3   >