Hi Maxime, > -----Original Message----- > From: Maxime Coquelin [mailto:maxime.coque...@redhat.com] > Sent: Tuesday, December 19, 2017 7:15 PM > To: Yang, Zhiyong <zhiyong.y...@intel.com>; dev@dpdk.org; > y...@fridaylinux.org > Cc: Wang, Wei W <wei.w.w...@intel.com>; Tan, Jianfeng > <jianfeng....@intel.com> > Subject: Re: [PATCH 00/11] net/vhostpci: A new vhostpci PMD supporting > VM2VM scenario >
<snip> > Thanks for the RFC. > It seems there is a lot of code duplication between this series and > libvhost-user. > > Does the non-RFC would make reuse of libvhost-user? I'm thinking of all the > code copied from virtio-net.c for example. > > If not, I think this is problematic as it will double the maintenance cost. Thank you for paying attention to the patchset . TX/RX logic basically comes from vhost user code. but some function interfaces and data structures are different, So can not reuse them directly, code duplicate should be removed, I'm aware of this point too, when I was writing the vhostpci PMD. But We should modify the libvhost-user firstly, let libvhostuser become more generic. It looks like that more and more PMDs are becoming the member of virtio family or are on the road. Thanks Zhiyong > > Cheers, > Maxime