Re: assigned EHCI USB headset not working

2011-06-08 Thread Jan Kiszka
On 2011-06-09 08:19, André Weidemann wrote: > Hi Gerd, > > On 08.06.2011 14:49, Gerd Hoffmann wrote: >> Hi, >> >>> The sound device shows up under Windows7 and drivers are installed >>> automatically. Unfortunately it does not work. All the players I tried, >>> did not even start playing the sound

Re: [Qemu-devel] [PATCH 00/11] cpu model bug fixes and definition corrections (v2)

2011-06-08 Thread Markus Armbruster
Jan Kiszka writes: [...] >> Something else I may have missed? >> > > Nothing critical, I'm just hoping someone finds the time to fix > sysconfigs loading when starting qemu from a build directory. :) There's nice to have, important, critical, and really annoying. This one's really annoying. -

Re: [PATCH] KVM: Adjust shadow paging to work when SMEP=1 and CR0.WP=0

2011-06-08 Thread Avi Kivity
On 06/09/2011 05:43 AM, Li, Xin wrote: Do we have test cases with guest.wp=0 in KVM test suite? Yes, see x86/access.c, AC_CPU_CR0_WP. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to ma

Re: differencing disks support

2011-06-08 Thread Markus Armbruster
Iordan Iordanov writes: > Does KVM support or plan to support differencing disks (where there is > a read-only source disk, and each person running a virtual machine can > save block-level changes that their virtual machine is making to the > disk in a separate "differencing" image)? > > If so, c

Re: assigned EHCI USB headset not working

2011-06-08 Thread André Weidemann
Hi Gerd, On 08.06.2011 14:49, Gerd Hoffmann wrote: Hi, The sound device shows up under Windows7 and drivers are installed automatically. Unfortunately it does not work. All the players I tried, did not even start playing the sound file, although they detected the DirectSound Device. iso xfer

RE: [PATCH] KVM: Adjust shadow paging to work when SMEP=1 and CR0.WP=0

2011-06-08 Thread Li, Xin
Do we have test cases with guest.wp=0 in KVM test suite? Thanks! -Xin > -Original Message- > From: Avi Kivity [mailto:a...@redhat.com] > Sent: Monday, June 06, 2011 9:19 PM > To: Marcelo Tosatti; kvm@vger.kernel.org; Yang, Wei Y; Shan, Haitao; Li, Xin > Subject: [PATCH] KVM: Adjust shadow

Re: [PATCH 1/1] [virt] virtio-blk: Use ida to allocate disk index

2011-06-08 Thread Greg KH
On Thu, Jun 09, 2011 at 08:51:05AM +0930, Rusty Russell wrote: > On Wed, 08 Jun 2011 09:08:29 -0400, Mark Wu wrote: > > Hi Rusty, > > Yes, I can't figure out an instance of disk probing in parallel either, but > > as > > per the following commit, I think we still need use lock for safety. What's

[PATCH] tun: do not put self in waitq if doing a nonblock read

2011-06-08 Thread Amos Kong
Perf shows a relatively high rate (about 8%) race in spin_lock_irqsave() when doing netperf between external host and guest. It's mainly becuase the lock contention between the tun_do_read() and tun_xmit_skb(), so this patch do not put self into waitqueue to reduce this kind of race. After this pat

Re: virtio scsi host draft specification, v3

2011-06-08 Thread Rusty Russell
On Tue, 07 Jun 2011 15:43:49 +0200, Paolo Bonzini wrote: > Hi all, > > after some preliminary discussion on the QEMU mailing list, I present a > draft specification for a virtio-based SCSI host (controller, HBA, you > name it). OK, I'm impressed. This is very well written and I doesn't make any

Re: [PATCH 1/1] [virt] virtio-blk: Use ida to allocate disk index

2011-06-08 Thread Rusty Russell
On Wed, 08 Jun 2011 09:08:29 -0400, Mark Wu wrote: > Hi Rusty, > Yes, I can't figure out an instance of disk probing in parallel either, but as > per the following commit, I think we still need use lock for safety. What's > your opinion? > > commit 4034cc68157bfa0b6622efe368488d3d3e20f4e6 > Auth

Re: KVM induced panic on 2.6.38[2367] & 2.6.39

2011-06-08 Thread Eric Dumazet
Le jeudi 09 juin 2011 à 01:02 +0800, Brad Campbell a écrit : > On 08/06/11 11:59, Eric Dumazet wrote: > > > Well, a bisection definitely should help, but needs a lot of time in > > your case. > > Yes. compile, test, crash, walk out to the other building to press > reset, lather, rinse, repeat. >

[PATCH 2/2] kvm tools: Use double buffering with SDL

2011-06-08 Thread Sasha Levin
Page flip every time we copy the buffer over instead of invalidating rects. This should improve performance by letting hardware do the page flipping. Signed-off-by: Sasha Levin --- tools/kvm/ui/sdl.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/kvm/ui/sdl.c b

[PATCH 1/2] kvm tools: Fix some SDL keyboard translations

2011-06-08 Thread Sasha Levin
This patch adds unmapped '<', '>', '|', '-', '+' and '=' which are quite useful in linux. Signed-off-by: Sasha Levin --- tools/kvm/ui/sdl.c | 10 +++--- 1 files changed, 7 insertions(+), 3 deletions(-) diff --git a/tools/kvm/ui/sdl.c b/tools/kvm/ui/sdl.c index 2e7c395..30fd511 100644 ---

Re: differencing disks support

2011-06-08 Thread Dan VerWeire
On Wed, Jun 8, 2011 at 4:09 PM, Iordan Iordanov wrote: > Does KVM support or plan to support differencing disks (where there is a > read-only source disk, and each person running a virtual machine can save > block-level changes that their virtual machine is making to the disk in a > separate "diff

differencing disks support

2011-06-08 Thread Iordan Iordanov
Does KVM support or plan to support differencing disks (where there is a read-only source disk, and each person running a virtual machine can save block-level changes that their virtual machine is making to the disk in a separate "differencing" image)? If so, can somebody suggest how I may mak

restricting users to only power control of VMs

2011-06-08 Thread Iordan Iordanov
Hi, As the subject suggests, we are wondering whether there is any way to restrict certain classes of users from performing any action other than powering a VM up and down, and resetting it? If this can't be done with KVM, does anybody have suggestions on how this can be accomplished? The on

Re: KVM induced panic on 2.6.38[2367] & 2.6.39

2011-06-08 Thread Brad Campbell
On 08/06/11 11:59, Eric Dumazet wrote: Well, a bisection definitely should help, but needs a lot of time in your case. Yes. compile, test, crash, walk out to the other building to press reset, lather, rinse, repeat. I need a reset button on the end of a 50M wire, or a hardware watchdog! Ac

Re: [PATCH v3 01/12] Add kernel header update script

2011-06-08 Thread Peter Maydell
On 8 June 2011 17:22, Jan Kiszka wrote: > This helper pulls the required kernel headers for KVM and vhost into a > specified directory. The update is triggered via > >    scripts/update-linux-headers.sh LINUX_PATH > Signed-off-by: Jan Kiszka > --- > > Changes in v3: >  - remove bashism Thanks;

[PATCH v3 01/12] Add kernel header update script

2011-06-08 Thread Jan Kiszka
This helper pulls the required kernel headers for KVM and vhost into a specified directory. The update is triggered via scripts/update-linux-headers.sh LINUX_PATH and will place the output under linux-headers/linux and linux-headers/asm-*. It also imports the COPYING to care for headers witho

Re: [PATCH v2 01/12] Add kernel header update script

2011-06-08 Thread Peter Maydell
On 8 June 2011 16:06, Jan Kiszka wrote: > +    if [ $arch == x86 ]; then This should be a single '=' -- '==' is a bashism. The 'checkbashisms' script (available in 'devscripts' package on debian and ubuntu) catches this: cam-vm-266:maverick:testing$ checkbashisms scripts/update-linux-headers.sh

[PATCH v2 01/12] Add kernel header update script

2011-06-08 Thread Jan Kiszka
This helper pulls the required kernel headers for KVM and vhost into a specified directory. The update is triggered via scripts/update-linux-headers.sh LINUX_PATH and will place the output under linux-headers/linux and linux-headers/asm-*. It also imports the COPYING to care for headers witho

Re: [PATCH 01/12] Add kernel header update script

2011-06-08 Thread Peter Maydell
2011/6/8 Jan Kiszka : > On 2011-06-08 16:33, Peter Maydell wrote: >> On 8 June 2011 15:10, Jan Kiszka wrote: >>> +tmpdir=$TMPDIR/.tmp-hdrs-$$ >> >> Better (safer) to use mktemp, I think. > > Is that portable? I don't think so. We don't expect every random end user to run this script, though, righ

Re: [PATCH 01/12] Add kernel header update script

2011-06-08 Thread Jan Kiszka
On 2011-06-08 16:33, Peter Maydell wrote: > On 8 June 2011 15:10, Jan Kiszka wrote: >> --- /dev/null >> +++ b/scripts/update-linux-headers.sh >> @@ -0,0 +1,55 @@ >> +#!/bin/sh -e >> +# > >> +if [ -z "$output" ]; then >> +output=$PWD >> +fi > >> +mkdir -p $output/linux-headers/asm-$arch >

Re: [PATCH 01/12] Add kernel header update script

2011-06-08 Thread Peter Maydell
On 8 June 2011 15:10, Jan Kiszka wrote: > --- /dev/null > +++ b/scripts/update-linux-headers.sh > @@ -0,0 +1,55 @@ > +#!/bin/sh -e > +# > +if [ -z "$output" ]; then > +    output=$PWD > +fi > +    mkdir -p $output/linux-headers/asm-$arch This script is rather lacking in quoting throughout. As a

[PATCH 07/12] kvm: Drop KVM_CAP build dependencies

2011-06-08 Thread Jan Kiszka
No longer needed with accompanied kernel headers. We are only left with build dependencies that are controlled by kvm arch headers. CC: Alexander Graf Signed-off-by: Jan Kiszka --- kvm-all.c |8 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/kvm-all.c b/kvm-all.c in

[PATCH 05/12] kvm: ppc: Drop CONFIG_KVM_PPC_PVR

2011-06-08 Thread Jan Kiszka
Required header support is now unconditionally available. CC: Alexander Graf Signed-off-by: Jan Kiszka --- configure|1 - target-ppc/kvm.c |9 - 2 files changed, 0 insertions(+), 10 deletions(-) diff --git a/configure b/configure index ed54db9..0947f98 100755 --- a/conf

[PATCH 09/12] kvm: ppc: Drop KVM_CAP build dependencies

2011-06-08 Thread Jan Kiszka
No longer needed with accompanied kernel headers. CC: Alexander Graf Signed-off-by: Jan Kiszka --- target-ppc/kvm.c | 14 -- 1 files changed, 0 insertions(+), 14 deletions(-) diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c index 0500e3f..21f35af 100644 --- a/target-ppc/kvm.c +++

[PATCH 03/12] Switch build system to accompanied kernel headers

2011-06-08 Thread Jan Kiszka
This helps reducing our build-time checks for feature support in the available Linux kernel headers. And it helps users that do not have sufficiently recent headers installed on their build machine. Consequently, the patch removes and build-time checks for kvm and vhost in configure, the --kerneld

[PATCH 11/12] kvm: x86: Pass KVMState to kvm_arch_get_supported_cpuid

2011-06-08 Thread Jan Kiszka
kvm_arch_get_supported_cpuid checks for global cpuid restrictions, it does not require any CPUState reference. Changing its interface allows to call it before any VCPU is initialized. CC: Eduardo Habkost Signed-off-by: Jan Kiszka --- kvm.h |2 +- target-i386/cpuid.c | 20 +++

[PATCH 08/12] kvm: x86: Drop KVM_CAP build dependencies

2011-06-08 Thread Jan Kiszka
No longer needed with accompanied kernel headers. Signed-off-by: Jan Kiszka --- target-i386/kvm.c | 67 ++-- 1 files changed, 3 insertions(+), 64 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 0efcf97..1c2d32c 100644 --- a

[PATCH 04/12] kvm: Drop CONFIG_KVM_PARA

2011-06-08 Thread Jan Kiszka
The kvm_para.h header is now always available. Signed-off-by: Jan Kiszka --- configure |1 - hw/kvmclock.c |9 - target-i386/kvm.c | 26 +- 3 files changed, 1 insertions(+), 35 deletions(-) diff --git a/configure b/configure index 0e1dc46..e

[PATCH 00/12] [uq/master] Import linux headers and some cleanups

2011-06-08 Thread Jan Kiszka
Licensing of the virtio headers is no clarified. So we can finally resolve the clumbsy and constantly buggy #ifdef'ery around old KVM and virtio headers. Recent example: current qemu-kvm does not build against 2.6.32 headers. This series introduces an import mechanism for all required Linux header

[PATCH 01/12] Add kernel header update script

2011-06-08 Thread Jan Kiszka
This helper pulls the required kernel headers for KVM and vhost into a specified directory. The update is triggered via scripts/update-linux-headers.sh LINUX_PATH and will place the output under linux-headers/linux and linux-headers/asm-*. It also imports the COPYING to care for headers witho

[PATCH 06/12] kvm: Drop useless zero-initializations

2011-06-08 Thread Jan Kiszka
Backing KVMState is alreay zero-initialized. Signed-off-by: Jan Kiszka --- kvm-all.c |5 - 1 files changed, 0 insertions(+), 5 deletions(-) diff --git a/kvm-all.c b/kvm-all.c index 106eb3a..4a9910a 100644 --- a/kvm-all.c +++ b/kvm-all.c @@ -764,28 +764,23 @@ int kvm_init(void) } #

[PATCH 12/12] Remove unneeded kvm.h from cpu-exec.c

2011-06-08 Thread Jan Kiszka
This was obsoleted by 6792a57bf1. Signed-off-by: Jan Kiszka --- cpu-exec.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index 6ddd8dd..9bb6405 100644 --- a/cpu-exec.c +++ b/cpu-exec.c @@ -20,7 +20,6 @@ #include "exec.h" #include "disas.h" #i

[PATCH 10/12] kvm: Clean up stubs

2011-06-08 Thread Jan Kiszka
No one references kvm_check_extension, kvm_has_vcpu_events, and kvm_has_robust_singlestep outside KVM code. kvm_update_guest_debug is never called, thus has no job besides returning an error. Signed-off-by: Jan Kiszka --- kvm-stub.c | 18 +- 1 files changed, 1 insertions(+), 1

Implementing Virtio Net driver for Solaris

2011-06-08 Thread Conor Murphy
Hi, I'm in the middle of writing a network driver for Solaris 10 to use a VirtIO backend. I've gotten the basics working and throughput between two VMs on the same host is ~ 4x faster then when using the rtls interface. When I'm looking for is some guidance as to which of the features (CSUM,MRG_

Re: [PATCH 1/1] [virt] virtio-blk: Use ida to allocate disk index

2011-06-08 Thread Mark Wu
On 06/02/2011 06:34 AM, Michael S. Tsirkin wrote: > On Wed, Jun 01, 2011 at 04:25:48AM -0400, Mark Wu wrote: >> On 06/01/2011 03:24 AM, Mark Wu wrote: >>> - if (index_to_minor(index)>= 1<< MINORBITS) >>> - return -ENOSPC; >>> + do { >>> + if (!ida_pre_get(&vd_index_ida, GFP

Re: [PATCH 1/1] [virt] virtio-blk: Use ida to allocate disk index

2011-06-08 Thread Mark Wu
On 06/01/2011 07:57 PM, Rusty Russell wrote: > On Wed, 1 Jun 2011 03:24:29 -0400, Mark Wu wrote: >> Current index allocation in virtio-blk is based on a monotonically >> increasing variable "index". It could cause some confusion about >> disk name in the case of hot-plugging disks. And it's imp

Re: assigned EHCI USB headset not working

2011-06-08 Thread Gerd Hoffmann
Hi, The sound device shows up under Windows7 and drivers are installed automatically. Unfortunately it does not work. All the players I tried, did not even start playing the sound file, although they detected the DirectSound Device. iso xfer's from usb-linux via ehci are flaky for reasons no

Re: [PATCH 04/15] KVM: MMU: cache mmio info on page fault path

2011-06-08 Thread Alexander Graf
On 08.06.2011, at 11:33, Xiao Guangrong wrote: > On 06/08/2011 05:18 PM, Alexander Graf wrote: > >>> >>> Also, when context switched, CR3 is reloaded, mmio cache info can be >>> cleared too. right? :-) >> >> Only when using SPT. In the NPT case, you will never see cr3 getting >> reloaded or

Re: [PATCH 04/15] KVM: MMU: cache mmio info on page fault path

2011-06-08 Thread Xiao Guangrong
On 06/08/2011 05:18 PM, Alexander Graf wrote: >> >> Also, when context switched, CR3 is reloaded, mmio cache info can be cleared >> too. right? :-) > > Only when using SPT. In the NPT case, you will never see cr3 getting reloaded > or INVLPG :). > In the NPT case, we only cache the GPA, GVA i

Re: [PATCH 04/15] KVM: MMU: cache mmio info on page fault path

2011-06-08 Thread Alexander Graf
On 08.06.2011, at 10:58, Xiao Guangrong wrote: > On 06/08/2011 04:22 PM, Alexander Graf wrote: > >>> +static int vcpu_gva_to_gpa(struct kvm_vcpu *vcpu, unsigned long gva, >>> + gpa_t *gpa, struct x86_exception *exception, >>> + bool write) >>> +{ >>> +

Re: [PATCH 04/15] KVM: MMU: cache mmio info on page fault path

2011-06-08 Thread Xiao Guangrong
On 06/08/2011 04:22 PM, Alexander Graf wrote: >> +static int vcpu_gva_to_gpa(struct kvm_vcpu *vcpu, unsigned long gva, >> + gpa_t *gpa, struct x86_exception *exception, >> + bool write) >> +{ >> +u32 access = (kvm_x86_ops->get_cpl(vcpu) == 3) ? PFERR

Re: [PATCH 0/15] KVM: optimize for MMIO handled

2011-06-08 Thread Takuya Yoshikawa
On Wed, 08 Jun 2011 14:22:36 +0800 Xiao Guangrong wrote: > On 06/08/2011 11:47 AM, Takuya Yoshikawa wrote: > > >>> Sure, KVM guest is the client, and it uses e1000 NIC, and uses NAT > >>> network connect to the netperf server, the bandwidth of our network > >>> is 100M. > >>> > > > > I see the

Re: [PATCH 04/15] KVM: MMU: cache mmio info on page fault path

2011-06-08 Thread Alexander Graf
On 07.06.2011, at 15:00, Xiao Guangrong wrote: > If the page fault is caused by mmio, we can cache the mmio info, later, we do > not need to walk guest page table and quickly know it is a mmio fault while we > emulate the mmio instruction > > Signed-off-by: Xiao Guangrong > --- > arch/x86/inclu

Re: [PATCH] pci-assign: Do not reset the device unless the kernel supports it

2011-06-08 Thread Jan Kiszka
On 2011-06-07 20:46, Alex Williamson wrote: > On Tue, 2011-06-07 at 10:14 +0200, Jan Kiszka wrote: >> On 2011-06-07 10:06, Avi Kivity wrote: >>> On 06/07/2011 01:04 AM, Jan Kiszka wrote: On 2011-06-06 23:48, Alex Williamson wrote: > On Mon, 2011-06-06 at 23:30 +0200, Jan Kiszka wrote: >>>