From mboxrd@z Thu Jan 1 00:00:00 1970 From: Amit Shah Subject: Re: [PATCH 1/2] virtio: Add a can_add_buf helper Date: Tue, 25 Aug 2009 20:05:17 +0530 Message-ID: <20090825143517.GA27028@amit-x200.redhat.com> References: <1250611290-2410-1-git-send-email-amit.shah@redhat.com> <200908221731.26834.rusty@rustcorp.com.au> <20090824073944.GA7653@amit-x200.redhat.com> <200908252357.54455.rusty@rustcorp.com.au> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <200908252357.54455.rusty@rustcorp.com.au> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Rusty Russell Cc: virtualization@lists.linux-foundation.org List-Id: virtualization@lists.linuxfoundation.org On (Tue) Aug 25 2009 [23:57:54], Rusty Russell wrote: > On Mon, 24 Aug 2009 05:09:44 pm Amit Shah wrote: > > sg_init_table(sg, 2+MAX_SKB_FRAGS); > > - for (;;) { > > + ret = 1; > > + while(ret > 0) { /* Is there space to add another buffer to the vq */ > > This is called a "do" loop :) > > Please test, and I'll apply. :-) drat! shouldn't have done that on a Sunday anyway I'll test it soon. Amit