"Michael S. Tsirkin" <[email protected]> wrote on 04/22/2010 05:02:25 AM:

> On Mon, Apr 19, 2010 at 03:12:19PM -0700, David L Stevens wrote:
> > This patch adds the mergeable RX buffers feature to vhost.
> > 
> > Signed-off-by: David L Stevens <[email protected]>
> 
> Looks pretty clean to me.
> Could you send a checkpatch-clean version please?

The original passes checkpatch already, but I guess I must
still be getting whitespace mangling if it didn't for you. (sigh)
Here it is as an attachment:



> We should also check performance implications.
> Do you have any data?

        I'm getting on the order of 10-20% improvement in
throughput over stock vhost guest to host, but I do see
a lot of variability in the results, even with no KVM
and just over loopback. I don't know where that's coming
from, but I'll do some runs and post.

Thanks for all the reviews!

                                                +-DLS

Attachment: MRXBv4.patch
Description: Binary data

Reply via email to