Hi Thomas,

> -----Original Message-----
> From: Thomas Monjalon <tho...@monjalon.net>
> Sent: Tuesday, October 13, 2020 5:31 AM
> To: Bing Zhao <bi...@nvidia.com>
> Cc: Ori Kam <or...@nvidia.com>; ferruh.yi...@intel.com;
> arybche...@solarflare.com; m...@ashroe.eu; nhor...@tuxdriver.com;
> bernard.iremon...@intel.com; beilei.x...@intel.com;
> wenzhuo...@intel.com; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v3 6/6] doc: update for two ports
> hairpin mode
> 
> External email: Use caution opening links or attachments
> 
> 
> 08/10/2020 14:05, Bing Zhao:
> > In the release notes, 2 ports hairpin mode feature is added.
> >
> > In rte flow part, one suggestion is added to mention that metadata
> > could be used to connect the hairpin RX and TX flows if the
> hairpin is
> > working in explicit TX flow rule mode.
> >
> > In the testpmd command line, the new parameter to set hairpin
> working
> > mode is described.
> >
> > Signed-off-by: Bing Zhao <bi...@nvidia.com>
> > Acked-by: Ori Kam <or...@nvidia.com>
> > ---
> >  doc/guides/prog_guide/rte_flow.rst     | 3 +++
> >  doc/guides/rel_notes/release_20_11.rst | 8 ++++++++
> > doc/guides/testpmd_app_ug/run_app.rst  | 8 ++++++++
> >  3 files changed, 19 insertions(+)
> 
> Please squash doc changes in related code changes commits, so both
> are atomic. Thanks

OK, I will rebase and squash them into other commits. So there is no
need to care about the check patches warning of the headline prefix,
am I right?

> 
> 

Thanks a lot

BR. Bing

Reply via email to