[PATCH] virtio-9p: fix build on !CONFIG_UTIMENSAT v2

2010-11-04 Thread Hidetoshi Seto
This patch introduce a fallback mechanism for old systems that do not support utimensat. This will fix build failure with following warnings: hw/virtio-9p-local.c: In function 'local_utimensat': hw/virtio-9p-local.c:479: warning: implicit declaration of function 'utimensat' hw/virtio-9p-local.c:4

Re: [Qemu-devel] [PATCHv2 4/8] Store IDE bus id in IDEBus structure for easy access.

2010-11-04 Thread Gleb Natapov
On Wed, Nov 03, 2010 at 06:22:11PM +0100, Markus Armbruster wrote: > Gleb Natapov writes: > > > On Wed, Nov 03, 2010 at 04:18:18PM +0100, Markus Armbruster wrote: > >> Gleb Natapov writes: > >> > >> > On Wed, Nov 03, 2010 at 02:39:52PM +0100, Markus Armbruster wrote: > >> >> Here's a generic an

[PATCH v14 05/17] Add a function to indicate if device use external buffer.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- include/linux/netdevice.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 8dcf6de..f91d9bb 100644 --- a/inclu

[PATCH v14 08/17] Modify netdev_free_page() to release external buffer

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Currently, it can get external buffers from mp device. Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- include/linux/skbuff.h |4 +++- net/core/skbuff.c | 24 2 files changed, 27 insertions(+), 1 deletions(-)

[PATCH v14 09/17] Don't do skb recycle, if device use external buffer.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- net/core/skbuff.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/net/core/skbuff.c b/net/core/skbuff.c index 02439e0..196aa99 100644 --- a/net/core/skbuff.c +++ b/net/

[PATCH v14 10/17] If device is in zero-copy mode first, bonding will fail.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui If device is in this zero-copy mode first, we cannot handle this, so fail it. This patch is for this. If bonding is created first, and one of the device will be in zero-copy mode, this will be handled by mp device. It will first check if all the slaves have the zero-copy capabi

[PATCH v14 13/17] Add mp(mediate passthru) device.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui The patch add mp(mediate passthru) device, which now based on vhost-net backend driver and provides proto_ops to send/receive guest buffers data from/to guest vitio-net driver. It also exports async functions which can be used by other drivers like macvtap to utilize zero-copy t

[PATCH v14 14/17] Add a kconfig entry and make entry for mp device.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Signed-off-by: Xin Xiaohui Reviewed-by: Jeff Dike --- drivers/vhost/Kconfig | 10 ++ drivers/vhost/Makefile |2 ++ 2 files changed, 12 insertions(+), 0 deletions(-) diff --git a/drivers/vhost/Kconfig b/drivers/vhost/Kconfig index e4e2fd1..a6b8cbf 100644 --- a/

[PATCH v14 17/17] An example how to alloc user buffer based on napi_gro_frags() interface.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui This example is made on ixgbe driver which using napi_gro_frags(). It can get buffers from guest side directly using netdev_alloc_page() and release guest buffers using netdev_free_page(). Signed-off-by: Xin Xiaohui --- drivers/net/ixgbe/ixgbe_main.c | 37 ++

[PATCH v14 16/17] An example how to modifiy NIC driver to use napi_gro_frags() interface

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui This example is made on ixgbe driver. It provides API is_rx_buffer_mapped_as_page() to indicate if the driver use napi_gro_frags() interface or not. The example allocates 2 pages for DMA for one ring descriptor using netdev_alloc_page(). When packets is coming, using napi_gro_fr

[PATCH v14 15/17]Provides multiple submits and asynchronous notifications.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui The vhost-net backend now only supports synchronous send/recv operations. The patch provides multiple submits and asynchronous notifications. This is needed for zero-copy case. Signed-off-by: Xin Xiaohui --- drivers/vhost/net.c | 355 +++

[PATCH v14 12/17] Add header file for mp device.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- include/linux/mpassthru.h | 133 + 1 files changed, 133 insertions(+), 0 deletions(-) create mode 100644 include/linux/mpassthru.h diff --git a/include

[PATCH v14 11/17] Add a hook to intercept external buffers from NIC driver.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui The hook is called in __netif_receive_skb(). Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- net/core/dev.c | 40 1 files changed, 40 insertions(+), 0 deletions(-) diff --git a/net/core/dev.c b/net/cor

Re: [Qemu-devel] [PATCHv2 4/8] Store IDE bus id in IDEBus structure for easy access.

2010-11-04 Thread Markus Armbruster
Gleb Natapov writes: > On Wed, Nov 03, 2010 at 06:22:11PM +0100, Markus Armbruster wrote: >> Gleb Natapov writes: >> >> > On Wed, Nov 03, 2010 at 04:18:18PM +0100, Markus Armbruster wrote: >> >> Gleb Natapov writes: >> >> >> >> > On Wed, Nov 03, 2010 at 02:39:52PM +0100, Markus Armbruster wro

[PATCH v14 07/17]Modify netdev_alloc_page() to get external buffer

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Currently, it can get external buffers from mp device. Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- net/core/skbuff.c | 27 +++ 1 files changed, 27 insertions(+), 0 deletions(-) diff --git a/net/core

[PATCH v14 06/17] Use callback to deal with skb_release_data() specially.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui If buffer is external, then use the callback to destruct buffers. Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- net/core/skbuff.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/net/core/skbuff.c b/net/core/skbu

[PATCH v14 00/17] Provide a zero-copy method on KVM virtio-net.

2010-11-04 Thread xiaohui . xin
We provide an zero-copy method which driver side may get external buffers to DMA. Here external means driver don't use kernel space to allocate skb buffers. Currently the external buffer can be from guest virtio-net driver. The idea is simple, just to pin the guest VM user space and then let host

[PATCH v14 02/17] Add a new struct for device to manipulate external buffer.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Add a structure in structure net_device, the new field is named as mp_port. It's for mediate passthru (zero-copy). It contains the capability for the net device driver, a socket, and an external buffer creator, external means skb buffer belongs to the device

[PATCH v14 03/17] Add a ndo_mp_port_prep pointer to net_device_ops.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui If the driver want to allocate external buffers, then it can export it's capability, as the skb buffer header length, the page length can be DMA, etc. The external buffers owner may utilize this. Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Re

[PATCH v14 04/17] Add a function make external buffer owner to query capability.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui The external buffer owner can use the functions to get the capability of the underlying NIC driver. Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- include/linux/netdevice.h |2 ++ net/core/dev.c| 41 +++

[PATCH v14 01/17] Add a new structure for skb buffer from external.

2010-11-04 Thread xiaohui . xin
From: Xin Xiaohui Signed-off-by: Xin Xiaohui Signed-off-by: Zhao Yu Reviewed-by: Jeff Dike --- include/linux/skbuff.h |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 77eb60d..696e690 100644 --- a/include/li

Re: [PATCH v14 06/17] Use callback to deal with skb_release_data() specially.

2010-11-04 Thread Eric Dumazet
Le jeudi 04 novembre 2010 à 17:05 +0800, xiaohui@intel.com a écrit : > From: Xin Xiaohui > > If buffer is external, then use the callback to destruct > buffers. > > Signed-off-by: Xin Xiaohui > Signed-off-by: Zhao Yu > Reviewed-by: Jeff Dike > --- > net/core/skbuff.c |8 > 1

Re: [PATCH v14 06/17] Use callback to deal with skb_release_data() specially.

2010-11-04 Thread Eric Dumazet
Le jeudi 04 novembre 2010 à 10:04 +0100, Eric Dumazet a écrit : > Hmm, I suggest you read the comment two lines above. > > If destructor_arg is now cleared each time we allocate a new skb, then, > please move it before dataref in shinfo structure, so that the following > memset() does the job eff

Re: [Qemu-devel] [PATCHv2 1/8] Introduce deriver_name field to DeviceInfo structure.

2010-11-04 Thread Markus Armbruster
Gleb Natapov writes: > Add "deriver_name" to DeviceInfo to use in device path building. In Typo "deriver". Same in subject. > contrast to "name" "driver_name" should refer to functionality device > provides instead of particular device model like "name" does. Why is that useful in a device pa

Re: [Qemu-devel] [PATCHv2 4/8] Store IDE bus id in IDEBus structure for easy access.

2010-11-04 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 09:46:57AM +0100, Markus Armbruster wrote: > > But why order of device creation is important? It shouldn't be if we > > want to move HW description into config file. We even may allow creating > > piix3-ide with only second IDE bus, but not first. > > That's not how buses w

Re: [Qemu-devel] Re: [PATCHv2 0/8 RFC] boot order specification

2010-11-04 Thread Markus Armbruster
Gleb Natapov writes: > On Sun, Oct 31, 2010 at 06:25:53PM -0400, Kevin O'Connor wrote: >> On Sun, Oct 31, 2010 at 01:40:01PM +0200, Gleb Natapov wrote: >> > This is current sate of the patch series for people to comment on. >> > I tried to use open firmware naming scheme to specify device path na

Re: [RFC PATCH 1/1] vhost: TX used buffer guest signal accumulation

2010-11-04 Thread Michael S. Tsirkin
On Wed, Nov 03, 2010 at 10:38:46PM -0700, Shirley Ma wrote: > On Wed, 2010-11-03 at 12:48 +0200, Michael S. Tsirkin wrote: > > I mean in practice, you see a benefit from this patch? > > Yes, I tested it. It does benefit the performance. > > > > My concern here is whether checking only in set up w

Re: [Qemu-devel] [PATCHv2 1/8] Introduce deriver_name field to DeviceInfo structure.

2010-11-04 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 10:20:18AM +0100, Markus Armbruster wrote: > Gleb Natapov writes: > > > Add "deriver_name" to DeviceInfo to use in device path building. In > > Typo "deriver". Same in subject. > Heh. > > contrast to "name" "driver_name" should refer to functionality device > > provide

Re: [PATCH 3/3] qemu-kvm: device assignment: emulate MSI-X mask bits

2010-11-04 Thread Michael S. Tsirkin
On Thu, Nov 04, 2010 at 02:18:21PM +0800, Sheng Yang wrote: > This patch emulated MSI-X per vector mask bit on assigned device. > > Signed-off-by: Sheng Yang > --- > hw/device-assignment.c | 161 > ++-- > 1 files changed, 155 insertions(+), 6 deletio

Re: [Qemu-devel] Re: [PATCHv2 0/8 RFC] boot order specification

2010-11-04 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 10:24:45AM +0100, Markus Armbruster wrote: > Gleb Natapov writes: > > > On Sun, Oct 31, 2010 at 06:25:53PM -0400, Kevin O'Connor wrote: > >> On Sun, Oct 31, 2010 at 01:40:01PM +0200, Gleb Natapov wrote: > >> > This is current sate of the patch series for people to comment

Re: [PATCH 2/3] qemu-kvm: device assignment: Some clean up about MSI-X code

2010-11-04 Thread Michael S. Tsirkin
On Thu, Nov 04, 2010 at 02:18:20PM +0800, Sheng Yang wrote: > > Signed-off-by: Sheng Yang If you have the energy for cleanups, how about removing most of msi/msix code in device-assignment and using msi.c and msix.c instead? > --- > hw/device-assignment.c | 77 +++

Re: [PATCH 1/3] qemu-kvm: Ioctl for in-kernel mask support

2010-11-04 Thread Michael S. Tsirkin
On Thu, Nov 04, 2010 at 02:18:19PM +0800, Sheng Yang wrote: > > Signed-off-by: Sheng Yang > --- > qemu-kvm.c | 15 +++ > qemu-kvm.h |6 ++ > 2 files changed, 21 insertions(+), 0 deletions(-) > > diff --git a/qemu-kvm.c b/qemu-kvm.c > index 733d0a9..ba6db51 100644 > --- a/q

Re: [PATCH 2/5] PCI: Add mask bit definition for MSI-X table

2010-11-04 Thread Michael S. Tsirkin
On Thu, Nov 04, 2010 at 02:15:18PM +0800, Sheng Yang wrote: > Then we can use it instead of magic number 1. > > Cc: Matthew Wilcox > Cc: Jesse Barnes > Cc: linux-...@vger.kernel.org > Signed-off-by: Sheng Yang Looks good to me > --- > drivers/pci/msi.c|4 ++-- > include/linux/pci

Re: [PATCH 3/3] qemu-kvm: device assignment: emulate MSI-X mask bits

2010-11-04 Thread Michael S. Tsirkin
On Thu, Nov 04, 2010 at 02:18:21PM +0800, Sheng Yang wrote: > This patch emulated MSI-X per vector mask bit on assigned device. > > Signed-off-by: Sheng Yang Also pls update the in-tree header for the new ioctls. > --- > hw/device-assignment.c | 161 >

[PATCH] KVM: remove unused function declaration

2010-11-04 Thread Xiao Guangrong
Remove the declaration of kvm_mmu_set_base_ptes() Signed-off-by: Xiao Guangrong --- arch/x86/include/asm/kvm_host.h |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/arch/x86/include/asm/kvm_host.h b/arch/x86/include/asm/kvm_host.h index 1be0058..7f20f2c 100644 --- a/arch/

[PATCH 1/3] KVM: MMU: fix missing post sync audit

2010-11-04 Thread Xiao Guangrong
Add AUDIT_POST_SYNC audit for 'root_level == PT64_ROOT_LEVEL' case Signed-off-by: Xiao Guangrong --- arch/x86/kvm/mmu.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index 5275c50..f3fad4f 100644 --- a/arch/x86/kvm/mmu.c +++ b/ar

[PATCH 2/3] KVM: MMU: don not retry #PF for nonpaging guest

2010-11-04 Thread Xiao Guangrong
nonpaing guest's 'direct_map' is also true, retry #PF for those guests is useless, so use 'tdp_enabled' instead Signed-off-by: Xiao Guangrong --- arch/x86/kvm/x86.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 2044302..aa

[PATCH 3/3] KVM: MMU: retry #PF for softmmu

2010-11-04 Thread Xiao Guangrong
Retry #PF for softmmu only when the current vcpu has the same root shadow page as the time when #PF occurs. it means they have same paging environment. Signed-off-by: Xiao Guangrong --- arch/x86/include/asm/kvm_host.h |6 ++ arch/x86/kvm/mmu.c | 33

Re: [PATCH 2/3] KVM: MMU: don not retry #PF for nonpaging guest

2010-11-04 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 06:32:42PM +0800, Xiao Guangrong wrote: > nonpaing guest's 'direct_map' is also true, retry #PF for those > guests is useless, so use 'tdp_enabled' instead > nonpaging guest will not attempt async pf. And by checking tdp_enabled here instead of direct_map we will screw nest

Re: [PATCH 5/5] KVM: assigned dev: MSI-X mask support

2010-11-04 Thread Michael S. Tsirkin
On Thu, Nov 04, 2010 at 02:15:21PM +0800, Sheng Yang wrote: > This patch enable per-vector mask for assigned devices using MSI-X. > > This patch provided two new APIs: one is for guest to specific device's MSI-X > table address in MMIO, the other is for userspace to get information about > mask >

vhost-net-next updates

2010-11-04 Thread Michael S. Tsirkin
I pushed out some optimization patches on vhost-net-next branch on my vhost tree (intended for 2.6.38). It would be helpful if people working on vhost-net optimizations base their work on that tree just to make sure comparisons are apples to apples. I might rebase this as I didn't send a pull requ

Problem booting Microsoft Windows KVM virtual machine

2010-11-04 Thread RaSca
Hi all, I'm having problems with a vm's startup, I cloned the entire disk of a Windows 2000 with dd on a drbd device, that disk was configured with two partitions. I'm able to see all the partitions contents by using kpartx and mount them: # kpartx -l /dev/drbd0 drbd0p1 : 0 202751488 /dev/drb

RE: Problem booting Microsoft Windows KVM virtual machine

2010-11-04 Thread Martin Maurer
Hi, Before you begin, prepare the running w2k to use ide disk (and boot on KVM with ide disks) For w2k I followed this link (solution 2 worked for me): http://www.motherboard.windowsreinstall.com/problems.htm And I am using clonezilla live cd´s to do cloning jobs, for me this is a fast and con

Re: [Qemu-devel] [PATCHv2 4/8] Store IDE bus id in IDEBus structure for easy access.

2010-11-04 Thread Markus Armbruster
Gleb Natapov writes: > On Thu, Nov 04, 2010 at 09:46:57AM +0100, Markus Armbruster wrote: >> > But why order of device creation is important? It shouldn't be if we >> > want to move HW description into config file. We even may allow creating >> > piix3-ide with only second IDE bus, but not first.

Re: [Qemu-devel] [PATCHv2 1/8] Introduce deriver_name field to DeviceInfo structure.

2010-11-04 Thread Markus Armbruster
Gleb Natapov writes: > On Thu, Nov 04, 2010 at 10:20:18AM +0100, Markus Armbruster wrote: >> Gleb Natapov writes: >> >> > Add "deriver_name" to DeviceInfo to use in device path building. In >> >> Typo "deriver". Same in subject. >> > Heh. > >> > contrast to "name" "driver_name" should refer

Re: [Qemu-devel] [PATCHv2 4/8] Store IDE bus id in IDEBus structure for easy access.

2010-11-04 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 03:22:50PM +0100, Markus Armbruster wrote: > Gleb Natapov writes: > > > On Thu, Nov 04, 2010 at 09:46:57AM +0100, Markus Armbruster wrote: > >> > But why order of device creation is important? It shouldn't be if we > >> > want to move HW description into config file. We ev

Re: [Qemu-devel] [PATCHv2 1/8] Introduce deriver_name field to DeviceInfo structure.

2010-11-04 Thread Gleb Natapov
On Thu, Nov 04, 2010 at 03:58:03PM +0100, Markus Armbruster wrote: > Gleb Natapov writes: > > > On Thu, Nov 04, 2010 at 10:20:18AM +0100, Markus Armbruster wrote: > >> Gleb Natapov writes: > >> > >> > Add "deriver_name" to DeviceInfo to use in device path building. In > >> > >> Typo "deriver".

Re: qemu-kvm-0.13.0 compile error

2010-11-04 Thread Teck Choon Giam
On Wed, Nov 3, 2010 at 6:00 PM, Ramon Dustet wrote: > Teck Choon Giam gmail.com> writes: > > >> # grep kvm_handle_internal_error ./* >> ./kvm-all.c:static void kvm_handle_internal_error(CPUState *env, >> struct kvm_run *run) >> ./kvm-all.c:            kvm_handle_internal_error(env, run); >> ./qem

Re: [PATCH -v2] Monitor command: pfa2hva, translate guest physical address to host virtual address

2010-11-04 Thread Luiz Capitulino
On Mon, 01 Nov 2010 14:22:21 -0500 Anthony Liguori wrote: > On 11/01/2010 02:20 PM, Huang Ying wrote: > > Yes. As general interface, it may not work so well, but as test > > interface, it works quite well and useful. > > > > Do we have any mechanism to add a test only interface? No. We're likel

Use of anon_inode_getfd in KVM

2010-11-04 Thread Prasad Joshi
Hello All, I trying to understand the KVM code. In userland code kvm_init() creates a VM by calling kvm_ioctl(KVM_CREATE_VM). The character device ioctl operation (kvm_dev_ioctl). Creates a new VM using kvm_create_vm(). The ioctl call also creates an anonymous file with name 'kvm-vm'. This ne

Re: [Qemu-devel] qemu-kvm build issue on RHEL5.1

2010-11-04 Thread Chris Wright
* Hidetoshi Seto (seto.hideto...@jp.fujitsu.com) wrote: > (2010/10/14 4:11), Blue Swirl wrote: > > On Wed, Oct 13, 2010 at 8:00 AM, Hidetoshi Seto > > wrote: > >> (Add CC to k...@vger) > >> > >> (2010/10/12 10:52), Hao, Xudong wrote: > >>> Hi, > >>> Currently qemu-kvm build fail on RHEL5 with gcc

Re: [RFC PATCH 1/1] vhost: TX used buffer guest signal accumulation

2010-11-04 Thread Shirley Ma
On Thu, 2010-11-04 at 11:30 +0200, Michael S. Tsirkin wrote: > One thing to note is that deferred signalling needs to be > benchmarked with old guests which don't orphan skbs on xmit > (or disable orphaning in both networking stack and virtio-net). Yes, we need run more test. > > OK, so I guess

Re: [PATCH 1/5] PCI: MSI: Move MSI-X entry definition to pci_regs.h

2010-11-04 Thread Hidetoshi Seto
Yeah, I think there are many virtualization stuff awaiting this change in these days. Reviewed-by: Hidetoshi Seto (2010/11/04 15:15), Sheng Yang wrote: > Then it can be used by others. > > Reviewed-by: Matthew Wilcox > Cc: Jesse Barnes > Cc: linux-...@vger.kernel.org > Signed-off-by: Sheng Ya

Re: [PATCH 2/5] PCI: Add mask bit definition for MSI-X table

2010-11-04 Thread Hidetoshi Seto
(2010/11/04 15:15), Sheng Yang wrote: > Then we can use it instead of magic number 1. > > Cc: Matthew Wilcox > Cc: Jesse Barnes > Cc: linux-...@vger.kernel.org > Signed-off-by: Sheng Yang > --- > drivers/pci/msi.c|4 ++-- > include/linux/pci_regs.h |1 + > 2 files changed, 3 in

[PATCH 49/49] virt/kvm/kvm_main.c: Use vzalloc

2010-11-04 Thread Joe Perches
Signed-off-by: Joe Perches --- virt/kvm/kvm_main.c | 13 +++-- 1 files changed, 3 insertions(+), 10 deletions(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index 5225052..c2a08e6e 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -604,13 +604,10 @@ int __kvm_set

Re: [PATCH 3/3] qemu-kvm: device assignment: emulate MSI-X mask bits

2010-11-04 Thread Sheng Yang
On Thursday 04 November 2010 18:04:08 Michael S. Tsirkin wrote: > On Thu, Nov 04, 2010 at 02:18:21PM +0800, Sheng Yang wrote: > > This patch emulated MSI-X per vector mask bit on assigned device. > > > > Signed-off-by: Sheng Yang > > Also pls update the in-tree header for the new ioctls. I thou

Re: [PATCH 2/5] PCI: Add mask bit definition for MSI-X table

2010-11-04 Thread Sheng Yang
On Thursday 04 November 2010 17:50:29 Michael S. Tsirkin wrote: > On Thu, Nov 04, 2010 at 02:15:18PM +0800, Sheng Yang wrote: > > Then we can use it instead of magic number 1. > > > > Cc: Matthew Wilcox > > Cc: Jesse Barnes > > Cc: linux-...@vger.kernel.org > > Signed-off-by: Sheng Yang > > Lo

Re: [PATCH 2/5] PCI: Add mask bit definition for MSI-X table

2010-11-04 Thread Sheng Yang
On Friday 05 November 2010 08:48:43 Hidetoshi Seto wrote: > (2010/11/04 15:15), Sheng Yang wrote: > > Then we can use it instead of magic number 1. > > > > Cc: Matthew Wilcox > > Cc: Jesse Barnes > > Cc: linux-...@vger.kernel.org > > Signed-off-by: Sheng Yang > > --- > > > > drivers/pci/msi.c

Re: [PATCH 1/3] qemu-kvm: Ioctl for in-kernel mask support

2010-11-04 Thread Sheng Yang
On Thursday 04 November 2010 17:47:58 Michael S. Tsirkin wrote: > On Thu, Nov 04, 2010 at 02:18:19PM +0800, Sheng Yang wrote: > > Signed-off-by: Sheng Yang > > --- > > > > qemu-kvm.c | 15 +++ > > qemu-kvm.h |6 ++ > > 2 files changed, 21 insertions(+), 0 deletions(-) > >

Re: [PATCH 5/5] KVM: assigned dev: MSI-X mask support

2010-11-04 Thread Sheng Yang
On Thursday 04 November 2010 18:43:22 Michael S. Tsirkin wrote: > On Thu, Nov 04, 2010 at 02:15:21PM +0800, Sheng Yang wrote: > > This patch enable per-vector mask for assigned devices using MSI-X. > > > > This patch provided two new APIs: one is for guest to specific device's > > MSI-X table addr

Re: [PATCH 2/3] qemu-kvm: device assignment: Some clean up about MSI-X code

2010-11-04 Thread Sheng Yang
On Thursday 04 November 2010 17:47:16 Michael S. Tsirkin wrote: > On Thu, Nov 04, 2010 at 02:18:20PM +0800, Sheng Yang wrote: > > Signed-off-by: Sheng Yang > > If you have the energy for cleanups, how about > removing most of msi/msix code in device-assignment and using msi.c and > msix.c instead

[V2 PATCH] virtio-net: init link state correctly

2010-11-04 Thread Jason Wang
For device that supports VIRTIO_NET_F_STATUS, there's no need to assume the link is up and we need to call nerif_carrier_off() before querying device status, otherwise we may get wrong operstate after diver was loaded because the link watch event was not fired as expected. For device that does not

[V2 PATCH] virtio-net: init link state correctly

2010-11-04 Thread Jason Wang
Jason Wang writes: CC netdev missed in the original patch. > For device that supports VIRTIO_NET_F_STATUS, there's no need to > assume the link is up and we need to call nerif_carrier_off() before > querying device status, otherwise we may get wrong operstate after > diver was loaded because t

[PATCH 05/49] arch/x86: Use vzalloc

2010-11-04 Thread Joe Perches
Signed-off-by: Joe Perches --- arch/x86/kernel/microcode_amd.c |3 +-- arch/x86/kvm/x86.c |3 +-- arch/x86/mm/pageattr-test.c |3 +-- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/arch/x86/kernel/microcode_amd.c b/arch/x86/kernel/microcode_amd.c index

[PATCH 03/49] arch/powerpc: Use vzalloc

2010-11-04 Thread Joe Perches
Signed-off-by: Joe Perches --- arch/powerpc/kvm/book3s.c |6 ++ arch/powerpc/platforms/cell/spufs/lscsa_alloc.c |3 +-- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/arch/powerpc/kvm/book3s.c b/arch/powerpc/kvm/book3s.c index e316847..4275463 100

Re: [PATCH 3/3] qemu-kvm: device assignment: emulate MSI-X mask bits

2010-11-04 Thread Sheng Yang
On Thursday 04 November 2010 17:44:27 Michael S. Tsirkin wrote: Thanks very much for reviewing this! Seems nobody cares about userspace one before... > On Thu, Nov 04, 2010 at 02:18:21PM +0800, Sheng Yang wrote: > > This patch emulated MSI-X per vector mask bit on assigned device. > > > > Signe

Re: [PATCH 5/5] KVM: assigned dev: MSI-X mask support

2010-11-04 Thread Sheng Yang
On Friday 05 November 2010 10:44:19 Sheng Yang wrote: > On Thursday 04 November 2010 18:43:22 Michael S. Tsirkin wrote: > > On Thu, Nov 04, 2010 at 02:15:21PM +0800, Sheng Yang wrote: > > > This patch enable per-vector mask for assigned devices using MSI-X. > > > > > > This patch provided two new

Re: [PATCH 05/49] arch/x86: Use vzalloc

2010-11-04 Thread Takuya Yoshikawa
(2010/11/05 12:07), Joe Perches wrote: > diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c > index 2288ad8..624d4da 100644 > --- a/arch/x86/kvm/x86.c > +++ b/arch/x86/kvm/x86.c > @@ -3174,10 +3174,9 @@ int kvm_vm_ioctl_get_dirty_log(struct kvm *kvm, > spin_unlock(&kvm->mmu_lock);

Re: [PATCH 49/49] virt/kvm/kvm_main.c: Use vzalloc

2010-11-04 Thread Takuya Yoshikawa
(2010/11/05 12:08), Joe Perches wrote: > Signed-off-by: Joe Perches > --- > virt/kvm/kvm_main.c | 13 +++-- > 1 files changed, 3 insertions(+), 10 deletions(-) > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c Already changed to vzalloc(), see kvm.git. Takuya > index 5225

Re: [PATCH 49/49] virt/kvm/kvm_main.c: Use vzalloc

2010-11-04 Thread Joe Perches
On Fri, 2010-11-05 at 13:25 +0900, Takuya Yoshikawa wrote: > (2010/11/05 12:08), Joe Perches wrote: > > Signed-off-by: Joe Perches > > --- > > virt/kvm/kvm_main.c | 13 +++-- > > 1 files changed, 3 insertions(+), 10 deletions(-) > > > > diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_

Re: [PATCH 2/3] KVM: MMU: don not retry #PF for nonpaging guest

2010-11-04 Thread Xiao Guangrong
On 11/04/2010 06:35 PM, Gleb Natapov wrote: > On Thu, Nov 04, 2010 at 06:32:42PM +0800, Xiao Guangrong wrote: >> nonpaing guest's 'direct_map' is also true, retry #PF for those >> guests is useless, so use 'tdp_enabled' instead >> > nonpaging guest will not attempt async pf. Ah, my mistake, but w

Re: [Qemu-devel] qemu-kvm build issue on RHEL5.1

2010-11-04 Thread Hidetoshi Seto
(2010/11/05 2:03), Chris Wright wrote: > * Hidetoshi Seto (seto.hideto...@jp.fujitsu.com) wrote: >> (2010/10/14 4:11), Blue Swirl wrote: >>> On Wed, Oct 13, 2010 at 8:00 AM, Hidetoshi Seto >>> wrote: (Add CC to k...@vger) (2010/10/12 10:52), Hao, Xudong wrote: > Hi, > Curren