From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1J17hm-0004u2-9v for qemu-devel@nongnu.org; Sat, 08 Dec 2007 16:59:26 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1J17hj-0004tq-Pw for qemu-devel@nongnu.org; Sat, 08 Dec 2007 16:59:24 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1J17hj-0004tn-LD for qemu-devel@nongnu.org; Sat, 08 Dec 2007 16:59:23 -0500 Received: from an-out-0708.google.com ([209.85.132.248]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1J17hj-0008L9-BV for qemu-devel@nongnu.org; Sat, 08 Dec 2007 16:59:23 -0500 Received: by an-out-0708.google.com with SMTP id b38so554082ana for ; Sat, 08 Dec 2007 13:59:22 -0800 (PST) Message-ID: <475B13B7.9040807@codemonkey.ws> Date: Sat, 08 Dec 2007 15:59:19 -0600 From: Anthony Liguori MIME-Version: 1.0 Subject: Re: [Qemu-devel] Re: [PATCH 2/3] virtio network device References: <4755CC8C.6000001@us.ibm.com> <200712051744.03916.paul@codesourcery.com> <47570817.9090908@us.ibm.com> <200712081322.39762.paul@codesourcery.com> In-Reply-To: <200712081322.39762.paul@codesourcery.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Anthony Liguori , Rusty Russell Paul Brook wrote: >> virtio makes things a bit trickier though. There's a shared ring queue >> between the host and guest. The ring queue is lock-less and depends on >> the ability to atomically increment ring queue indices to be SMP safe. >> Using a copy-API wouldn't be a problem for QEMU since the host and guest >> are always running in lock-step. A copy API is actually needed to deal >> with differing host/guest alignment and endianness. >> > > That seems a rather poor design choice, as many architectures don't have an > atomic increment instruction. Oh well. > Sorry, I should have been more clear. An atomic increment isn't needed, just an atomic store. Regards, Anthony Liguori > Paul > > > >