From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH v3 1/1] Shared memory uio_pci driver Date: Thu, 25 Mar 2010 18:32:15 +0200 Message-ID: <4BAB900F.5030104@redhat.com> References: <1269497376-21903-1-git-send-email-cam@cs.ualberta.ca> <20100325091552.GB11153@redhat.com> <8286e4ee1003250918x659e87ai9e5ca7c68c23a7c3@mail.gmail.com> <4BAB8DE9.9080603@codemonkey.ws> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Cam Macdonell , "Michael S. Tsirkin" , kvm@vger.kernel.org, qemu-devel@nongnu.org, virtualization@lists.linux-foundation.org To: Anthony Liguori Return-path: Received: from mx1.redhat.com ([209.132.183.28]:23458 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752950Ab0CYQc0 (ORCPT ); Thu, 25 Mar 2010 12:32:26 -0400 In-Reply-To: <4BAB8DE9.9080603@codemonkey.ws> Sender: kvm-owner@vger.kernel.org List-ID: On 03/25/2010 06:23 PM, Anthony Liguori wrote: >> There has been previous discussion of virtio, however while virtio is >> good for exporting guest memory, it's not ideal for importing memory >> into a guest. > > virtio is a DMA-based API which means that it doesn't assume cache > coherent shared memory. The PCI transport takes advantage of cache > coherent shared memory but it's not strictly required. Aren't we violating this by not using dma_alloc_coherent() for the queues? -- error compiling committee.c: too many arguments to function