On Tue, Apr 29, 2008 at 01:16:05AM +1000, Rusty Russell wrote: > > + /* If we has a buffer left over from last time, send it now. */ > + if (vi->last_xmit_skb) { > + if (xmit_skb(vi, vi->last_xmit_skb) != 0) { > + /* Drop this skb: we only queue one. */ > + kfree_skb(skb); > + goto stop_queue; We should increment the drop counter here. Otherwise these patches all look good to me. Acked-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Thanks, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/virtualization