15/10/2017 14:51, Aviad Yehezkel: > > On 10/15/2017 1:17 AM, Akhil Goyal wrote: > > From: Radu Nicolau <radu.nico...@intel.com> > > > > Signed-off-by: Radu Nicolau <radu.nico...@intel.com> > > Signed-off-by: Declan Doherty <declan.dohe...@intel.com> > > --- > > drivers/net/Makefile | 2 +- > > drivers/net/ixgbe/Makefile | 2 +- > > drivers/net/ixgbe/base/ixgbe_osdep.h | 8 + > > drivers/net/ixgbe/ixgbe_ethdev.c | 19 + > > drivers/net/ixgbe/ixgbe_ethdev.h | 6 +- > > drivers/net/ixgbe/ixgbe_flow.c | 47 +++ > > drivers/net/ixgbe/ixgbe_ipsec.c | 744 > > +++++++++++++++++++++++++++++++++ > > drivers/net/ixgbe/ixgbe_ipsec.h | 147 +++++++ > > drivers/net/ixgbe/ixgbe_rxtx.c | 53 ++- > > drivers/net/ixgbe/ixgbe_rxtx.h | 11 +- > > drivers/net/ixgbe/ixgbe_rxtx_vec_sse.c | 50 ++- > > 11 files changed, 1079 insertions(+), 10 deletions(-) > > create mode 100644 drivers/net/ixgbe/ixgbe_ipsec.c > > create mode 100644 drivers/net/ixgbe/ixgbe_ipsec.h > > [all code lines cut]
Please Aviad, remove the useless lines when replying. It is really annoying to scroll the whole patch to find where you replied. > Tested-by: Aviad Yehezkel <avia...@mellanox.com> Really? You have tested the ixgbe driver? When providing a test acknowledgement, it is more valuable to provide a brief test report: - which hardware - which use case - results