Re: [kvm-devel] Chaning disks from IDE to SCSI on a Windows VM

2008-04-15 Thread Felix Leimbach
On 14.04.2008 Alberto Treviño wrote: If anyone can attest to this procedures and report, it would be greatly appreciated. Hello Alberto, I can confirm that this method works on Windows 2003 Server. Along those lines, if anyone knows how to grab an existing Windows VM that was set up in

Re: [kvm-devel] Chaning disks from IDE to SCSI on a Windows VM

2008-04-15 Thread Jun Koi
On Tue, Apr 15, 2008 at 3:41 PM, Felix Leimbach [EMAIL PROTECTED] wrote: On 14.04.2008 Alberto Treviño wrote: If anyone can attest to this procedures and report, it would be greatly appreciated. Hello Alberto, I can confirm that this method works on Windows 2003 Server. Looks like a

[kvm-devel] [PATCH 0/2]kvmtrace: add event mask to limit tracing data.

2008-04-15 Thread Liu, Eric E
Hi, The following two patches add event mask support in kvmtrace and the corresponding kernel code, so that we can specify filter masks to limit the events being captured. --Eric (Liu, Feng) - This SF.net email is sponsored

[kvm-devel] [PATCH 1/2]kvmtrace: add event mask support (kernel part)

2008-04-15 Thread Liu, Eric E
From a1b062cfd4d1a91c447b680ac9a2250fe55119ec Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu [EMAIL PROTECTED] Date: Wed, 16 Apr 2008 05:29:37 -0400 Subject: [PATCH] KVM: trace: Add event mask support. Allow user space application to specify one or more filter masks to limit the events being

[kvm-devel] [PATCH 2/2]kvmtrace: add event mask support(userspace part)

2008-04-15 Thread Liu, Eric E
From 9314f8b249fe6c32f006e6a1cec5d3868b66e6ff Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu [EMAIL PROTECTED] Date: Wed, 16 Apr 2008 05:45:40 -0400 Subject: [PATCH] kvm: user: Add event mask support in kvmtrace. Signed-off-by: Feng (Eric) Liu [EMAIL PROTECTED] --- user/kvmtrace.c | 56

[kvm-devel] [PATCH]kvmtrace:fix a redundant assignment.

2008-04-15 Thread Liu, Eric E
From 193536b1cd96f646b66bea52bc46fdebcdcfc1cf Mon Sep 17 00:00:00 2001 From: Feng (Eric) Liu [EMAIL PROTECTED] Date: Wed, 16 Apr 2008 07:05:22 -0400 Subject: [PATCH] KVM: trace: fix a redundant assignment. Romove the redundant assignment, since the variable extra will not be used again.

Re: [kvm-devel] [PATCH] [v2] Move kvm_get_pit to libkvm.c common code

2008-04-15 Thread Avi Kivity
Jerone Young wrote: I just took the old description that was there before. A much better one would be: Remove declarations of kvm_*_pit() on architectures who do not support not have a PIT. That is what I was really intending. It removes a lot of compile warnings, when compiling anything

[kvm-devel] KVM Test result, kernel 8d3a833.., userspace e665b72.. --- One Issue Fixed

2008-04-15 Thread Yunfeng Zhao
Hi All, This is today's KVM test result against kvm.git 8d3a833dc9d42f0967e57717f89c518375d6a417 and kvm-userspace.git e665b723bd12c01068879fe8106a8108b070c653. One Issue Fixed: 1. Fails to save/restore ia32e guests

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Anders
Marcelo Tosatti wrote: On Mon, Apr 14, 2008 at 07:24:06PM +0300, Avi Kivity wrote: Why not enable SIGIO on stdio input, like the rest of the fd handling in qemu? Thats a possibility, but I think we've now agreed that doing select() with a timeout is cleaner and possibly half a cent faster.

Re: [kvm-devel] [PATCH 1 of 4] Add make sync to sync need kernel headers to kvm-userspace

2008-04-15 Thread Avi Kivity
Jerone Young wrote: 2 files changed, 25 insertions(+), 4 deletions(-) Makefile | 21 - configure |8 +--- This patch adds ability for kvm-userspace build system to sync needed kernel headers locally without the need of compiled kernel source. Signed-off-by:

Re: [kvm-devel] [PATCH 4 of 4] Remove kvm kernel-path option from qemu

2008-04-15 Thread Avi Kivity
Jerone Young wrote: 2 files changed, 2 insertions(+), 8 deletions(-) qemu/Makefile.target |3 +-- qemu/configure |7 +-- Now that kvm headers are synced locally, qemu does not need a specific option to find the kernel headers as they can now be specified in the

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Avi Kivity
Anders wrote: Why not enable SIGIO on stdio input, like the rest of the fd handling in qemu? Thats a possibility, but I think we've now agreed that doing select() with a timeout is cleaner and possibly half a cent faster. Since I can only follow this list as a hobby, I managed

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-15 Thread Guillaume Thouvenin
On Mon, 07 Apr 2008 11:05:06 -0500 Anthony Liguori [EMAIL PROTECTED] wrote: Perhaps a viable way to fix this upstream would be to catch the vmentry failure, look to see if SS.CPL != CS.CPL, and if so, invoke x86_emulate() in a loop until SS.CPL == CS.CPL. There are very few instructions

Re: [kvm-devel] Extboot Option Rom rewritten in C

2008-04-15 Thread Jun Koi
On Tue, Apr 15, 2008 at 12:34 PM, Anthony Liguori [EMAIL PROTECTED] wrote: Nguyen Anh Quynh wrote: On 4/15/08, Anthony Liguori [EMAIL PROTECTED] wrote: Hi Nguyen, Nguyen Anh Quynh wrote: Hi Anthony, I spot a bug and few dead code in the extboot option rom. Perhaps

Re: [kvm-devel] [PATCH] gfxboot VMX workaround v2

2008-04-15 Thread Avi Kivity
Guillaume Thouvenin wrote: On Mon, 07 Apr 2008 11:05:06 -0500 Anthony Liguori [EMAIL PROTECTED] wrote: Perhaps a viable way to fix this upstream would be to catch the vmentry failure, look to see if SS.CPL != CS.CPL, and if so, invoke x86_emulate() in a loop until SS.CPL == CS.CPL.

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Avi Kivity
Anthony Liguori wrote: Avi Kivity wrote: Anthony Liguori wrote: BTW, when we set O_ASYNC on the tap fd, we're eliminating O_NONBLOCK. This means that we have to poll loop select() when readv()'ing packets instead of just reading until hitting AGAIN. This means at least an extra syscall

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Anthony Liguori
Avi Kivity wrote: Anthony Liguori wrote: BTW, when we set O_ASYNC on the tap fd, we're eliminating O_NONBLOCK. This means that we have to poll loop select() when readv()'ing packets instead of just reading until hitting AGAIN. This means at least an extra syscall per packet. I didn't

Re: [kvm-devel] [PATCH] [kvm-userspace] Make make sync in kernel dir work for multiple archs

2008-04-15 Thread Hollis Blanchard
On Monday 14 April 2008 21:46:43 Jerone Young wrote: 1 file changed, 13 insertions(+), 5 deletions(-) kernel/Makefile | 18 +- This patch add the ability for make sync in the kernel directory to work for mulitiple architectures and not just x86. Signed-off-by: Jerone Young

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Anthony Liguori
Avi Kivity wrote: Anders wrote: Why not enable SIGIO on stdio input, like the rest of the fd handling in qemu? Thats a possibility, but I think we've now agreed that doing select() with a timeout is cleaner and possibly half a cent faster. Since I can only

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Marcelo Tosatti
On Tue, Apr 15, 2008 at 05:45:28PM +0300, Avi Kivity wrote: Anthony Liguori wrote: Why did we ever need sigtimedwait() anyway? Even if we were select()ing within the VCPU context, we should break out of the select() on signal delivery. select() is no good since if the signal is

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Marcelo Tosatti
On Tue, Apr 15, 2008 at 08:40:09AM -0500, Anthony Liguori wrote: Avi Kivity wrote: Anthony Liguori wrote: BTW, when we set O_ASYNC on the tap fd, we're eliminating O_NONBLOCK. This means that we have to poll loop select() when readv()'ing packets instead of just reading until hitting

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Anthony Liguori
Marcelo Tosatti wrote: On Tue, Apr 15, 2008 at 05:45:28PM +0300, Avi Kivity wrote: Anthony Liguori wrote: Why did we ever need sigtimedwait() anyway? Even if we were select()ing within the VCPU context, we should break out of the select() on signal delivery. select() is

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Avi Kivity
Anthony Liguori wrote: With the IO thread, we don't have to worry about lost signals like we do in a VCPU thread so it's fine to just use select() and install signal handlers IIUC. What about aio completions? The only race-free way to handle both posix aio completion and fd readiness is

Re: [kvm-devel] [PATCH] [kvm-userspace] Make make sync in kernel dir work for multiple archs

2008-04-15 Thread Jerone Young
On Tue, 2008-04-15 at 09:08 -0500, Hollis Blanchard wrote: On Monday 14 April 2008 21:46:43 Jerone Young wrote: 1 file changed, 13 insertions(+), 5 deletions(-) kernel/Makefile | 18 +- This patch add the ability for make sync in the kernel directory to work for

Re: [kvm-devel] Chaning disks from IDE to SCSI on a Windows VM

2008-04-15 Thread Alberto Treviño
On Tuesday 15 April 2008 12:57:45 am Jun Koi wrote: Looks like a problem, however. In his instruction, part 3: 3. Shut down the VM. This time, don't include the new temporary image from step 1 and define your disk(s) as SCSI disks: qemu-system-x86_64 -m 256 \ -drive

Re: [kvm-devel] [patch 2/2] QEMU: decrease console refresh rate with -nographic

2008-04-15 Thread Anthony Liguori
Avi Kivity wrote: Anthony Liguori wrote: With the IO thread, we don't have to worry about lost signals like we do in a VCPU thread so it's fine to just use select() and install signal handlers IIUC. What about aio completions? The only race-free way to handle both posix aio completion

Re: [kvm-devel] [PATCH] [kvm-userspace] Make make sync in kernel dir work for multiple archs

2008-04-15 Thread Hollis Blanchard
On Tuesday 15 April 2008 11:20:58 Jerone Young wrote: What happened to my suggestion of creating a per-arch HACK_FILES and UNIFDEF_FILES variables, and looping over those? These macros are only for x86. We don't want them or need them. So I just left them be as not to accidentally miss or

[kvm-devel] [PATCH 0 of 2] [RESEND] Fix PowerPC cpu initiliaztion

2008-04-15 Thread Jerone Young
This patch apparently fell through the cracks or I didn't send the rised version to the list. These patches fix cpu initilization for PowerPC. Without them guest cannot be launched. Signed-off-by: Jerone Young [EMAIL PROTECTED] 2 files changed, 6 insertions(+), 3 deletions(-)

[kvm-devel] [PATCH] [v2][kvm-userspace] Make make sync in kernel dir work for multiple archs

2008-04-15 Thread Jerone Young
1 file changed, 31 insertions(+), 17 deletions(-) kernel/Makefile | 48 +++- This patch add the ability for make sync in the kernel directory to work for mulitiple architectures and not just x86. Signed-off-by: Jerone Young [EMAIL PROTECTED] diff

Re: [kvm-devel] [kvm-ppc-devel] [PATCH] [v2 ][kvm-userspace] Make make sync in kern el dir work for multiple archs

2008-04-15 Thread Hollis Blanchard
On Tuesday 15 April 2008 14:43:12 Jerone Young wrote: 1 file changed, 31 insertions(+), 17 deletions(-) kernel/Makefile | 48 +++- This patch add the ability for make sync in the kernel directory to work for mulitiple architectures and not just

[kvm-devel] [PATCH] [v2][kvm-userspace] Make make sync in kernel dir work for multiple archs

2008-04-15 Thread Jerone Young
1 file changed, 31 insertions(+), 17 deletions(-) kernel/Makefile | 48 +++- This patch add the ability for make sync in the kernel directory to work for mulitiple architectures and not just x86. Signed-off-by: Jerone Young [EMAIL PROTECTED] diff

[kvm-devel] [PATCH] [v3][kvm-userspace] Make make sync in kernel dir work for multiple archs

2008-04-15 Thread Jerone Young
1 file changed, 31 insertions(+), 17 deletions(-) kernel/Makefile | 48 +++- - fix where $(ARCH_DIR) is lower case and for -DCONFIG_? in _unifdef macro it needs to be upper case. This patch add the ability for make sync in the kernel directory to

Re: [kvm-devel] [PATCH 1/5]Add some trace markers and expose interfaces in kernel for tracing

2008-04-15 Thread Hollis Blanchard
On Wednesday 09 April 2008 05:01:36 Liu, Eric E wrote: +/* This structure represents a single trace buffer record. */ +struct kvm_trace_rec { +   __u32 event:28; +   __u32 extra_u32:3; +   __u32 cycle_in:1; +   __u32 pid; +   __u32 vcpu_id; +   union { +

[kvm-devel] [PATCH] [KVM] Rename debugfs_dir to kvm_debugfs_dir

2008-04-15 Thread Hollis Blanchard
3 files changed, 7 insertions(+), 7 deletions(-) include/linux/kvm_host.h |2 +- virt/kvm/kvm_main.c |8 virt/kvm/kvm_trace.c |4 ++-- # HG changeset patch # User Hollis Blanchard [EMAIL PROTECTED] # Date 1208293411 18000 # Node ID

[kvm-devel] [PATCH 2/5] virtio for QEMU (v3)

2008-04-15 Thread Anthony Liguori
This patch introduces virtio support over PCI. virtio is a generic virtual IO framework for Linux first introduced in 2.6.23. Since 2.6.25, virtio has supported a PCI transport which this patch implements. Since the last time these patches were posted to qemu-devel, I've reworked it to use the

[kvm-devel] [PATCH 3/5] virtio network driver (v3)

2008-04-15 Thread Anthony Liguori
This patch implements the virtio network driver backend. In KVM, this driver can achieve 1gbit tx/rx performance. More patches are required to improve the network IO infrastructure to achieve better performance in QEMU. Since v1, I've updated the patch based on the IOVector refactoring.

[kvm-devel] [PATCH 4/5] virtio block driver (v3)

2008-04-15 Thread Anthony Liguori
This patch implements the virtio block driver backend. Signed-off-by: Anthony Liguori [EMAIL PROTECTED] diff --git a/Makefile.target b/Makefile.target index ea632fa..4d695c7 100644 --- a/Makefile.target +++ b/Makefile.target @@ -535,7 +535,7 @@ OBJS += rtl8139.o OBJS += e1000.o # virtio

[kvm-devel] [PATCH 5/5] virtio balloon driver (v3)

2008-04-15 Thread Anthony Liguori
This patch implements the virtio balloon driver backend. A user can interact with the balloon driver using a newly introduce monitor command 'balloon'. Ballooning is used to request the guest to stop using a certain portion of its memory. The guest notifies the host of this memory so the host

[kvm-devel] [PATCH 1/5] PCI DMA API (v3)

2008-04-15 Thread Anthony Liguori
This patch introduces a DMA API and plumbs support through the DMA layer. We use a mostly opaque structure, IOVector to represent a scatter/gather list of physical memory. Associated with each IOVector is a read/write function and an opaque pointer. This allows arbitrary transformation/mapping

[kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-04-15 Thread David S. Ahern
I have been looking at RHEL3 based guests lately, and to say the least the performance is horrible. Rather than write a long tome on what I've done and observed, I'd like to find out if anyone has some insights or known problem areas running 2.4 guests. The short of it is that % system time

Re: [kvm-devel] Chaning disks from IDE to SCSI on a Windows VM

2008-04-15 Thread Jun Koi
On Wed, Apr 16, 2008 at 1:29 AM, Alberto Treviño [EMAIL PROTECTED] wrote: On Tuesday 15 April 2008 12:57:45 am Jun Koi wrote: Looks like a problem, however. In his instruction, part 3: 3. Shut down the VM. This time, don't include the new temporary image from step 1 and define your

Re: [kvm-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-15 Thread Liu, Eric E
Hollis Blanchard wrote: On Wednesday 09 April 2008 05:01:36 Liu, Eric E wrote: +/* This structure represents a single trace buffer record. */ +struct kvm_trace_rec { +   __u32 event:28; +   __u32 extra_u32:3; +   __u32 cycle_in:1; +   __u32 pid; +   __u32 vcpu_id; + 

Re: [kvm-devel] [PATCH 1/5]Add some trace markers and exposeinterfaces in kernel for tracing

2008-04-15 Thread Hollis Blanchard
On Tuesday 15 April 2008 22:13:28 Liu, Eric E wrote: Hollis Blanchard wrote: On Wednesday 09 April 2008 05:01:36 Liu, Eric E wrote: +/* This structure represents a single trace buffer record. */ +struct kvm_trace_rec { +   __u32 event:28; +   __u32 extra_u32:3; +   __u32