[dpdk-dev] [PATCH 3/3] vhost: update comments
yuanhan.liu at linux.intel.com
Fri Nov 4 15:15:18 CET 2016
On Fri, Nov 04, 2016 at 01:28:45PM +0000, Mcnamara, John wrote:
> > -----Original Message-----
> > From: Yuanhan Liu [mailto:yuanhan.liu at linux.intel.com]
> > Sent: Wednesday, November 2, 2016 3:15 AM
> > To: dev at dpdk.org
> > Cc: Mcnamara, John <john.mcnamara at intel.com>; Yuanhan Liu
> > <yuanhan.liu at linux.intel.com>
> > Subject: [PATCH 3/3] vhost: update comments
> > vhost-cuse is removed, update corresponding comments that are still
> > referencing it.
> > Signed-off-by: Yuanhan Liu <yuanhan.liu at linux.intel.com>
> > ...
> > /*
> > - * Backend-specific cleanup. Defined by vhost-cuse and vhost-user.
> > + * Backend-specific cleanup.
> > + *
> > + * TODO: fix it; we have one backend now
> > */
> > void vhost_backend_cleanup(struct virtio_net *dev);
> In general it is not a good idea to leave TODOs/Fixmes around unless they
> that a better explanation of why they are there.
In this case, if you look at the delete line, you will find we have such
function implemented both in vhost-user and vhost-cuse. Now we get one
only. So, we may could just rename it to vhost_user_cleanup(). Or even,
remove it completely by doing some small refactors.
It's obvious that we need do that in another patch. Thus, the TODO is
> Apart from that:
> Acked-by: John McNamara <john.mcnamara at intel.com>
More information about the dev