On Tue, Mar 14, 2017 at 01:59:49PM +0100, Maxime Coquelin wrote: > > > On 03/03/2017 10:51 AM, Yuanhan Liu wrote: > >Rename "rte_virtio_net.h" to "rte_vhost.h", to not let it be virtio > >net specific. > > > >Signed-off-by: Yuanhan Liu <yuanhan....@linux.intel.com> > >--- > > doc/guides/rel_notes/deprecation.rst | 9 -- > > drivers/net/vhost/rte_eth_vhost.c | 2 +- > > drivers/net/vhost/rte_eth_vhost.h | 2 +- > > examples/tep_termination/main.c | 2 +- > > examples/tep_termination/vxlan_setup.c | 2 +- > > examples/vhost/main.c | 2 +- > > lib/librte_vhost/Makefile | 2 +- > > lib/librte_vhost/rte_vhost.h | 259 > > +++++++++++++++++++++++++++++++++ > > lib/librte_vhost/rte_virtio_net.h | 259 > > --------------------------------- > Did you perform the rename using git mv?
Yes, and I forgot to add '-M' option while run "git format-patch". --yliu > > > lib/librte_vhost/vhost.c | 2 +- > > lib/librte_vhost/vhost.h | 2 +- > > lib/librte_vhost/vhost_user.h | 2 +- > > lib/librte_vhost/virtio_net.c | 2 +- > > 13 files changed, 269 insertions(+), 278 deletions(-) > > create mode 100644 lib/librte_vhost/rte_vhost.h > > delete mode 100644 lib/librte_vhost/rte_virtio_net.h > > Other than that, the change looks good to me: > Reviewed-by: Maxime Coquelin <maxime.coque...@redhat.com> > > Thanks, > Maxime