"Michael S. Tsirkin" <mst@xxxxxxxxxx> 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 <dlstevens@xxxxxxxxxx>
>
> 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
