[dpdk-dev,v4,1/5] doc: remove RSS configuration change announcement

Message ID 20180413110218.105894-2-xuemingl@mellanox.com (mailing list archive)
State Superseded, archived
Delegated to: Ferruh Yigit
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation fail apply patch file failure

Commit Message

Xueming Li April 13, 2018, 11:02 a.m. UTC
  Remove deprecation as implementation of RSS level provided in Adrien's
patch set: http://www.dpdk.org/dev/patchwork/patch/37399/

Signed-off-by: Xueming Li <xuemingl@mellanox.com>
Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
---
 doc/guides/rel_notes/deprecation.rst | 4 ----
 1 file changed, 4 deletions(-)
  

Comments

Iremonger, Bernard April 17, 2018, 3:16 p.m. UTC | #1
Hi Li,

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Xueming Li
> Sent: Friday, April 13, 2018 12:02 PM
> To: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Wu, Jingjing
> <jingjing.wu@intel.com>; Thomas Monjalon <thomas@monjalon.net>; Adrien
> Mazarguil <adrien.mazarguil@6wind.com>
> Cc: Xueming Li <xuemingl@mellanox.com>; Nelio Laranjeiro
> <nelio.laranjeiro@6wind.com>; Shahaf Shuler <shahafs@mellanox.com>;
> dev@dpdk.org; Olivier Matz <olivier.matz@6wind.com>
> Subject: [dpdk-dev] [PATCH v4 1/5] doc: remove RSS configuration change
> announcement
> 
> Remove deprecation as implementation of RSS level provided in Adrien's patch
> set: http://www.dpdk.org/dev/patchwork/patch/37399/
> 
> Signed-off-by: Xueming Li <xuemingl@mellanox.com>
> Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
> ---
>  doc/guides/rel_notes/deprecation.rst | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index ec70b5fa9..8b8af47e3 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -108,10 +108,6 @@ Deprecation Notices
>    Target release for removal of the legacy API will be defined once most
>    PMDs have switched to rte_flow.
> 
> -* ethdev: A new rss level field planned in 18.05.
> -  The new API add rss_level field to ``rte_eth_rss_conf`` to enable a choice
> -  of RSS hash calculation on outer or inner header of tunneled packet.
> -
>  * ethdev:  Currently, if the  rte_eth_rx_burst() function returns a value less
>    than *nb_pkts*, the application will assume that no more packets are present.
>    Some of the hw queue based hardware can only support smaller burst for RX
> --
> 2.13.3

This patch fails to apply to the current master branch.

Regards,

Bernard.
  
Xueming Li April 17, 2018, 3:37 p.m. UTC | #2
Hi Bernard,

> -----Original Message-----

> From: Iremonger, Bernard <bernard.iremonger@intel.com>

> Sent: Tuesday, April 17, 2018 11:16 PM

> To: Xueming(Steven) Li <xuemingl@mellanox.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Wu, Jingjing

> <jingjing.wu@intel.com>; Thomas Monjalon <thomas@monjalon.net>; Adrien Mazarguil

> <adrien.mazarguil@6wind.com>

> Cc: Nélio Laranjeiro <nelio.laranjeiro@6wind.com>; Shahaf Shuler <shahafs@mellanox.com>; dev@dpdk.org;

> Olivier Matz <olivier.matz@6wind.com>

> Subject: RE: [dpdk-dev] [PATCH v4 1/5] doc: remove RSS configuration change announcement

> 

> Hi Li,

> 

> > -----Original Message-----

> > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Xueming Li

> > Sent: Friday, April 13, 2018 12:02 PM

> > To: Lu, Wenzhuo <wenzhuo.lu@intel.com>; Wu, Jingjing

> > <jingjing.wu@intel.com>; Thomas Monjalon <thomas@monjalon.net>; Adrien

> > Mazarguil <adrien.mazarguil@6wind.com>

> > Cc: Xueming Li <xuemingl@mellanox.com>; Nelio Laranjeiro

> > <nelio.laranjeiro@6wind.com>; Shahaf Shuler <shahafs@mellanox.com>;

> > dev@dpdk.org; Olivier Matz <olivier.matz@6wind.com>

> > Subject: [dpdk-dev] [PATCH v4 1/5] doc: remove RSS configuration

> > change announcement

> >

> > Remove deprecation as implementation of RSS level provided in Adrien's

> > patch

> > set:

> > https://emea01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.

> > dpdk.org%2Fdev%2Fpatchwork%2Fpatch%2F37399%2F&data=02%7C01%7Cxuemingl%

> > 40mellanox.com%7Cd264ab2220ae4151b20208d5a4762fe0%7Ca652971c7d2e4d9ba6

> > a4d149256f461b%7C0%7C0%7C636595749918860284&sdata=r5OtZ18aGoTXy%2Byuv9

> > kTDsRDD0fKF3FcPWJ%2FMRZ6VL8%3D&reserved=0

> >

> > Signed-off-by: Xueming Li <xuemingl@mellanox.com>

> > Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

> > ---

> >  doc/guides/rel_notes/deprecation.rst | 4 ----

> >  1 file changed, 4 deletions(-)

> >

> > diff --git a/doc/guides/rel_notes/deprecation.rst

> > b/doc/guides/rel_notes/deprecation.rst

> > index ec70b5fa9..8b8af47e3 100644

> > --- a/doc/guides/rel_notes/deprecation.rst

> > +++ b/doc/guides/rel_notes/deprecation.rst

> > @@ -108,10 +108,6 @@ Deprecation Notices

> >    Target release for removal of the legacy API will be defined once most

> >    PMDs have switched to rte_flow.

> >

> > -* ethdev: A new rss level field planned in 18.05.

> > -  The new API add rss_level field to ``rte_eth_rss_conf`` to enable a

> > choice

> > -  of RSS hash calculation on outer or inner header of tunneled packet.

> > -

> >  * ethdev:  Currently, if the  rte_eth_rx_burst() function returns a value less

> >    than *nb_pkts*, the application will assume that no more packets are present.

> >    Some of the hw queue based hardware can only support smaller burst

> > for RX

> > --

> > 2.13.3

> 

> This patch fails to apply to the current master branch.


Could you please try v5 version which rebased on latest code:
	http://www.dpdk.org/ml/archives/dev/2018-April/097669.html

Thanks,
Xueming Li

> 

> Regards,

> 

> Bernard.
  

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index ec70b5fa9..8b8af47e3 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -108,10 +108,6 @@  Deprecation Notices
   Target release for removal of the legacy API will be defined once most
   PMDs have switched to rte_flow.
 
-* ethdev: A new rss level field planned in 18.05.
-  The new API add rss_level field to ``rte_eth_rss_conf`` to enable a choice
-  of RSS hash calculation on outer or inner header of tunneled packet.
-
 * ethdev:  Currently, if the  rte_eth_rx_burst() function returns a value less
   than *nb_pkts*, the application will assume that no more packets are present.
   Some of the hw queue based hardware can only support smaller burst for RX