[dpdk-dev,v3] doc: add queue region feature info to release notes

Message ID 20180108033953.142410-1-wei.zhao1@intel.com (mailing list archive)
State Accepted, archived
Delegated to: Helin Zhang
Headers

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK

Commit Message

Zhao1, Wei Jan. 8, 2018, 3:39 a.m. UTC
  This patch add inforation about i40e queue region
realted to release notes, it has been missed before
in v17.11 release notes. This feature has been
implemented in v17.11.

Signed-off-by: Wei Zhao <wei.zhao1@intel.com>

---

v2:
-change this information to v18.02 release notes.

v3:
-rework it on dpdk-next-net-intel sub tree.
---
 doc/guides/rel_notes/release_18_02.rst | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
  

Comments

Zhang, Helin Jan. 9, 2018, 6:15 a.m. UTC | #1
> -----Original Message-----
> From: Zhao1, Wei
> Sent: Monday, January 8, 2018 11:40 AM
> To: dev@dpdk.org
> Cc: Zhang, Helin; Mcnamara, John; Zhao1, Wei
> Subject: [PATCH v3] doc: add queue region feature info to release notes
> 
> This patch add inforation about i40e queue region realted to release notes, it
> has been missed before in v17.11 release notes. This feature has been
> implemented in v17.11.
> 
> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> 
> ---
> 
> v2:
> -change this information to v18.02 release notes.
> 
> v3:
> -rework it on dpdk-next-net-intel sub tree.
> ---
>  doc/guides/rel_notes/release_18_02.rst | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/release_18_02.rst
> b/doc/guides/rel_notes/release_18_02.rst
> index 621b8f5..d7c8b68 100644
> --- a/doc/guides/rel_notes/release_18_02.rst
> +++ b/doc/guides/rel_notes/release_18_02.rst
> @@ -47,6 +47,24 @@ New Features
>    users. According to the device configuration, it will setup TX->RX loopback
>    link or not.
> 
> +* **Added the i40e ethernet driver to support queue region feature.**
Please follow others about the format. I guess blank space might be needed.
Thanks,
Helin

> +
> +  This feature enable queue regions configuration for RSS in PF,  so
> + that different traffic classes or different packet  classification
> + types can be separated to different queues in  different queue
> + regions. This feature has been implemented in v17.11.
> +
> +* **Added the testpmd app to support queue region configuration with
> +CLI.**
> +
> +  This patch add a API configuration of queue region in rss with command line.
> +  It can parse the parameters of region index, queue number,  queue
> + start index, user priority, traffic classes and so on.
> +  According to commands from command line, it will call i40e  private
> + API and start the process of set or flush queue region  configure. As
> + this feature is specific for i40e, so private API  will be used. Aslo
> + add a document for these new commands.
> +  This feature has been implemented in v17.11.
> +
>  API Changes
>  -----------
> 
> --
> 2.9.3
  
Zhang, Helin Jan. 9, 2018, 6:17 a.m. UTC | #2
> -----Original Message-----
> From: Zhang, Helin
> Sent: Tuesday, January 9, 2018 2:15 PM
> To: Zhao1, Wei; dev@dpdk.org
> Cc: Mcnamara, John
> Subject: RE: [PATCH v3] doc: add queue region feature info to release notes
> 
> 
> 
> > -----Original Message-----
> > From: Zhao1, Wei
> > Sent: Monday, January 8, 2018 11:40 AM
> > To: dev@dpdk.org
> > Cc: Zhang, Helin; Mcnamara, John; Zhao1, Wei
> > Subject: [PATCH v3] doc: add queue region feature info to release
> > notes
> >
> > This patch add inforation about i40e queue region realted to release
> > notes, it has been missed before in v17.11 release notes. This feature
> > has been implemented in v17.11.
> >
> > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> >
> > ---
> >
> > v2:
> > -change this information to v18.02 release notes.
> >
> > v3:
> > -rework it on dpdk-next-net-intel sub tree.
> > ---
> >  doc/guides/rel_notes/release_18_02.rst | 18 ++++++++++++++++++
> >  1 file changed, 18 insertions(+)
> >
> > diff --git a/doc/guides/rel_notes/release_18_02.rst
> > b/doc/guides/rel_notes/release_18_02.rst
> > index 621b8f5..d7c8b68 100644
> > --- a/doc/guides/rel_notes/release_18_02.rst
> > +++ b/doc/guides/rel_notes/release_18_02.rst
> > @@ -47,6 +47,24 @@ New Features
> >    users. According to the device configuration, it will setup TX->RX loopback
> >    link or not.
> >
> > +* **Added the i40e ethernet driver to support queue region feature.**
> Please follow others about the format. I guess blank space might be needed.
> Thanks,
> Helin
Sorry, it seems that different formats are all there. Please just ignore my comments here. Thanks!

/Helin
> 
> > +
> > +  This feature enable queue regions configuration for RSS in PF,  so
> > + that different traffic classes or different packet  classification
> > + types can be separated to different queues in  different queue
> > + regions. This feature has been implemented in v17.11.
> > +
> > +* **Added the testpmd app to support queue region configuration with
> > +CLI.**
> > +
> > +  This patch add a API configuration of queue region in rss with command
> line.
> > +  It can parse the parameters of region index, queue number,  queue
> > + start index, user priority, traffic classes and so on.
> > +  According to commands from command line, it will call i40e  private
> > + API and start the process of set or flush queue region  configure.
> > + As this feature is specific for i40e, so private API  will be used.
> > + Aslo add a document for these new commands.
> > +  This feature has been implemented in v17.11.
> > +
> >  API Changes
> >  -----------
> >
> > --
> > 2.9.3
  
Zhao1, Wei Jan. 9, 2018, 6:18 a.m. UTC | #3
> -----Original Message-----
> From: Zhang, Helin
> Sent: Tuesday, January 9, 2018 2:17 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> Cc: Mcnamara, John <john.mcnamara@intel.com>
> Subject: RE: [PATCH v3] doc: add queue region feature info to release notes
> 
> 
> 
> > -----Original Message-----
> > From: Zhang, Helin
> > Sent: Tuesday, January 9, 2018 2:15 PM
> > To: Zhao1, Wei; dev@dpdk.org
> > Cc: Mcnamara, John
> > Subject: RE: [PATCH v3] doc: add queue region feature info to release
> > notes
> >
> >
> >
> > > -----Original Message-----
> > > From: Zhao1, Wei
> > > Sent: Monday, January 8, 2018 11:40 AM
> > > To: dev@dpdk.org
> > > Cc: Zhang, Helin; Mcnamara, John; Zhao1, Wei
> > > Subject: [PATCH v3] doc: add queue region feature info to release
> > > notes
> > >
> > > This patch add inforation about i40e queue region realted to release
> > > notes, it has been missed before in v17.11 release notes. This
> > > feature has been implemented in v17.11.
> > >
> > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > >
> > > ---
> > >
> > > v2:
> > > -change this information to v18.02 release notes.
> > >
> > > v3:
> > > -rework it on dpdk-next-net-intel sub tree.
> > > ---
> > >  doc/guides/rel_notes/release_18_02.rst | 18 ++++++++++++++++++
> > >  1 file changed, 18 insertions(+)
> > >
> > > diff --git a/doc/guides/rel_notes/release_18_02.rst
> > > b/doc/guides/rel_notes/release_18_02.rst
> > > index 621b8f5..d7c8b68 100644
> > > --- a/doc/guides/rel_notes/release_18_02.rst
> > > +++ b/doc/guides/rel_notes/release_18_02.rst
> > > @@ -47,6 +47,24 @@ New Features
> > >    users. According to the device configuration, it will setup TX->RX
> loopback
> > >    link or not.
> > >
> > > +* **Added the i40e ethernet driver to support queue region
> > > +feature.**
> > Please follow others about the format. I guess blank space might be
> needed.
> > Thanks,
> > Helin
> Sorry, it seems that different formats are all there. Please just ignore my
> comments here. Thanks!

Ok.

> 
> /Helin
> >
> > > +
> > > +  This feature enable queue regions configuration for RSS in PF,
> > > + so that different traffic classes or different packet
> > > + classification types can be separated to different queues in
> > > + different queue regions. This feature has been implemented in v17.11.
> > > +
> > > +* **Added the testpmd app to support queue region configuration
> > > +with
> > > +CLI.**
> > > +
> > > +  This patch add a API configuration of queue region in rss with
> > > + command
> > line.
> > > +  It can parse the parameters of region index, queue number,  queue
> > > + start index, user priority, traffic classes and so on.
> > > +  According to commands from command line, it will call i40e
> > > + private API and start the process of set or flush queue region  configure.
> > > + As this feature is specific for i40e, so private API  will be used.
> > > + Aslo add a document for these new commands.
> > > +  This feature has been implemented in v17.11.
> > > +
> > >  API Changes
> > >  -----------
> > >
> > > --
> > > 2.9.3
  
Zhang, Helin Jan. 9, 2018, 6:26 a.m. UTC | #4
> -----Original Message-----
> From: Zhao1, Wei
> Sent: Monday, January 8, 2018 11:40 AM
> To: dev@dpdk.org
> Cc: Zhang, Helin; Mcnamara, John; Zhao1, Wei
> Subject: [PATCH v3] doc: add queue region feature info to release notes
> 
> This patch add inforation about i40e queue region realted to release notes, it
> has been missed before in v17.11 release notes. This feature has been
> implemented in v17.11.
> 
> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
Acked-by: Helin Zhang <helin.zhang@intel.com>
  
Zhang, Helin Jan. 9, 2018, 6:29 a.m. UTC | #5
> -----Original Message-----
> From: Zhang, Helin
> Sent: Tuesday, January 9, 2018 2:26 PM
> To: Zhao1, Wei; dev@dpdk.org
> Cc: Mcnamara, John
> Subject: RE: [PATCH v3] doc: add queue region feature info to release notes
> 
> 
> 
> > -----Original Message-----
> > From: Zhao1, Wei
> > Sent: Monday, January 8, 2018 11:40 AM
> > To: dev@dpdk.org
> > Cc: Zhang, Helin; Mcnamara, John; Zhao1, Wei
> > Subject: [PATCH v3] doc: add queue region feature info to release
> > notes
> >
> > This patch add inforation about i40e queue region realted to release
> > notes, it has been missed before in v17.11 release notes. This feature
> > has been implemented in v17.11.
> >
> > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> Acked-by: Helin Zhang <helin.zhang@intel.com>
Applied to dpdk-next-net-intel, with minor changes. Thanks!

/Helin
  
Thomas Monjalon Jan. 16, 2018, 9:53 p.m. UTC | #6
> > > This patch add inforation about i40e queue region realted to release
> > > notes, it has been missed before in v17.11 release notes. This feature
> > > has been implemented in v17.11.
> > >
> > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > Acked-by: Helin Zhang <helin.zhang@intel.com>
> Applied to dpdk-next-net-intel, with minor changes. Thanks!

This patch has not been reviewed by John, doc maintainer.
There are some english errors.
It documents extensively a testpmd addition, which is usually not
documented in release notes.
And it is adding 17.11 features in 18.02 release notes.

It is too much, so it is just removed when pulling next-net.

If you want to explain what is possible with i40e, you should
document it in i40e.rst.
The release notes is just a short list of what is added.
  
Zhao1, Wei Jan. 18, 2018, 2 a.m. UTC | #7
Hi,  Thomas
   I will update it in 17.11 release nots later as your comment. 
And it has been CC to John,  but not get any feedback.

Thanks.

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Wednesday, January 17, 2018 5:53 AM
> To: Zhang, Helin <helin.zhang@intel.com>; Zhao1, Wei
> <wei.zhao1@intel.com>
> Cc: dev@dpdk.org; Mcnamara, John <john.mcnamara@intel.com>; Yigit,
> Ferruh <ferruh.yigit@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature info to
> release notes
> 
> > > > This patch add inforation about i40e queue region realted to
> > > > release notes, it has been missed before in v17.11 release notes.
> > > > This feature has been implemented in v17.11.
> > > >
> > > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > > Acked-by: Helin Zhang <helin.zhang@intel.com>
> > Applied to dpdk-next-net-intel, with minor changes. Thanks!
> 
> This patch has not been reviewed by John, doc maintainer.
> There are some english errors.
> It documents extensively a testpmd addition, which is usually not
> documented in release notes.
> And it is adding 17.11 features in 18.02 release notes.
> 
> It is too much, so it is just removed when pulling next-net.
> 
> If you want to explain what is possible with i40e, you should document it in
> i40e.rst.
> The release notes is just a short list of what is added.
  
Thomas Monjalon Jan. 18, 2018, 7:30 a.m. UTC | #8
You can probably work with Beilei who is trying to do the same thing
for other i40e features:
	https://dpdk.org/dev/patchwork/patch/33953/


18/01/2018 03:00, Zhao1, Wei:
> Hi,  Thomas
>    I will update it in 17.11 release nots later as your comment. 
> And it has been CC to John,  but not get any feedback.
> 
> Thanks.
> 
> > -----Original Message-----
> > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > Sent: Wednesday, January 17, 2018 5:53 AM
> > To: Zhang, Helin <helin.zhang@intel.com>; Zhao1, Wei
> > <wei.zhao1@intel.com>
> > Cc: dev@dpdk.org; Mcnamara, John <john.mcnamara@intel.com>; Yigit,
> > Ferruh <ferruh.yigit@intel.com>
> > Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature info to
> > release notes
> > 
> > > > > This patch add inforation about i40e queue region realted to
> > > > > release notes, it has been missed before in v17.11 release notes.
> > > > > This feature has been implemented in v17.11.
> > > > >
> > > > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > > > Acked-by: Helin Zhang <helin.zhang@intel.com>
> > > Applied to dpdk-next-net-intel, with minor changes. Thanks!
> > 
> > This patch has not been reviewed by John, doc maintainer.
> > There are some english errors.
> > It documents extensively a testpmd addition, which is usually not
> > documented in release notes.
> > And it is adding 17.11 features in 18.02 release notes.
> > 
> > It is too much, so it is just removed when pulling next-net.
> > 
> > If you want to explain what is possible with i40e, you should document it in
> > i40e.rst.
> > The release notes is just a short list of what is added.
  
Zhao1, Wei Jan. 19, 2018, 3:15 a.m. UTC | #9
Hi,  Thomas
    Thank you! After discussion with Beilei, I have commit a new patch v4.
 https://dpdk.org/dev/patchwork/patch/33956/
Beilei also commit another new patch, we are waiting for review by Mcnamara, John.
Do you think this is ok?


> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Thursday, January 18, 2018 3:30 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>
> Cc: Zhang, Helin <helin.zhang@intel.com>; dev@dpdk.org; Mcnamara, John
> <john.mcnamara@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>; Xing,
> Beilei <beilei.xing@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature info to
> release notes
> 
> You can probably work with Beilei who is trying to do the same thing for
> other i40e features:
> 	https://dpdk.org/dev/patchwork/patch/33953/
> 
> 
> 18/01/2018 03:00, Zhao1, Wei:
> > Hi,  Thomas
> >    I will update it in 17.11 release nots later as your comment.
> > And it has been CC to John,  but not get any feedback.
> >
> > Thanks.
> >
> > > -----Original Message-----
> > > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > > Sent: Wednesday, January 17, 2018 5:53 AM
> > > To: Zhang, Helin <helin.zhang@intel.com>; Zhao1, Wei
> > > <wei.zhao1@intel.com>
> > > Cc: dev@dpdk.org; Mcnamara, John <john.mcnamara@intel.com>; Yigit,
> > > Ferruh <ferruh.yigit@intel.com>
> > > Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature
> > > info to release notes
> > >
> > > > > > This patch add inforation about i40e queue region realted to
> > > > > > release notes, it has been missed before in v17.11 release notes.
> > > > > > This feature has been implemented in v17.11.
> > > > > >
> > > > > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > > > > Acked-by: Helin Zhang <helin.zhang@intel.com>
> > > > Applied to dpdk-next-net-intel, with minor changes. Thanks!
> > >
> > > This patch has not been reviewed by John, doc maintainer.
> > > There are some english errors.
> > > It documents extensively a testpmd addition, which is usually not
> > > documented in release notes.
> > > And it is adding 17.11 features in 18.02 release notes.
> > >
> > > It is too much, so it is just removed when pulling next-net.
> > >
> > > If you want to explain what is possible with i40e, you should
> > > document it in i40e.rst.
> > > The release notes is just a short list of what is added.
> 
>
  
Zhao1, Wei Jan. 19, 2018, 3:38 a.m. UTC | #10
Commit second time to cc  stable@dpdk.org
https://dpdk.org/dev/patchwork/patch/34065/

> -----Original Message-----
> From: Zhao1, Wei
> Sent: Friday, January 19, 2018 11:15 AM
> To: 'Thomas Monjalon' <thomas@monjalon.net>
> Cc: Zhang, Helin <helin.zhang@intel.com>; dev@dpdk.org; Mcnamara, John
> <john.mcnamara@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>; Xing,
> Beilei <beilei.xing@intel.com>
> Subject: RE: [dpdk-dev] [PATCH v3] doc: add queue region feature info to
> release notes
> 
> Hi,  Thomas
>     Thank you! After discussion with Beilei, I have commit a new patch v4.
>  https://dpdk.org/dev/patchwork/patch/33956/
> Beilei also commit another new patch, we are waiting for review by
> Mcnamara, John.
> Do you think this is ok?
> 
> 
> > -----Original Message-----
> > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > Sent: Thursday, January 18, 2018 3:30 PM
> > To: Zhao1, Wei <wei.zhao1@intel.com>
> > Cc: Zhang, Helin <helin.zhang@intel.com>; dev@dpdk.org; Mcnamara,
> John
> > <john.mcnamara@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>;
> > Xing, Beilei <beilei.xing@intel.com>
> > Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature info
> > to release notes
> >
> > You can probably work with Beilei who is trying to do the same thing
> > for other i40e features:
> > 	https://dpdk.org/dev/patchwork/patch/33953/
> >
> >
> > 18/01/2018 03:00, Zhao1, Wei:
> > > Hi,  Thomas
> > >    I will update it in 17.11 release nots later as your comment.
> > > And it has been CC to John,  but not get any feedback.
> > >
> > > Thanks.
> > >
> > > > -----Original Message-----
> > > > From: Thomas Monjalon [mailto:thomas@monjalon.net]
> > > > Sent: Wednesday, January 17, 2018 5:53 AM
> > > > To: Zhang, Helin <helin.zhang@intel.com>; Zhao1, Wei
> > > > <wei.zhao1@intel.com>
> > > > Cc: dev@dpdk.org; Mcnamara, John <john.mcnamara@intel.com>; Yigit,
> > > > Ferruh <ferruh.yigit@intel.com>
> > > > Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature
> > > > info to release notes
> > > >
> > > > > > > This patch add inforation about i40e queue region realted to
> > > > > > > release notes, it has been missed before in v17.11 release notes.
> > > > > > > This feature has been implemented in v17.11.
> > > > > > >
> > > > > > > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> > > > > > Acked-by: Helin Zhang <helin.zhang@intel.com>
> > > > > Applied to dpdk-next-net-intel, with minor changes. Thanks!
> > > >
> > > > This patch has not been reviewed by John, doc maintainer.
> > > > There are some english errors.
> > > > It documents extensively a testpmd addition, which is usually not
> > > > documented in release notes.
> > > > And it is adding 17.11 features in 18.02 release notes.
> > > >
> > > > It is too much, so it is just removed when pulling next-net.
> > > >
> > > > If you want to explain what is possible with i40e, you should
> > > > document it in i40e.rst.
> > > > The release notes is just a short list of what is added.
> >
> >
  
Thomas Monjalon Jan. 19, 2018, 9:13 a.m. UTC | #11
19/01/2018 04:15, Zhao1, Wei:
> Hi,  Thomas
>     Thank you! After discussion with Beilei, I have commit a new patch v4.
>  https://dpdk.org/dev/patchwork/patch/33956/
> Beilei also commit another new patch, we are waiting for review by Mcnamara, John.
> Do you think this is ok?

Yes it is fine to wait for John's approval.

Note: in release notes, you should group all i40e features in the same place.
In v4, you just added it at the end.
I think v5 is needed.
  
Thomas Monjalon Jan. 19, 2018, 9:17 a.m. UTC | #12
19/01/2018 10:13, Thomas Monjalon:
> 19/01/2018 04:15, Zhao1, Wei:
> > Hi,  Thomas
> >     Thank you! After discussion with Beilei, I have commit a new patch v4.
> >  https://dpdk.org/dev/patchwork/patch/33956/
> > Beilei also commit another new patch, we are waiting for review by Mcnamara, John.
> > Do you think this is ok?
> 
> Yes it is fine to wait for John's approval.
> 
> Note: in release notes, you should group all i40e features in the same place.
> In v4, you just added it at the end.
> I think v5 is needed.

More precisions:
There is no i40e block in 17.11 release notes.
So you should insert one close to other net devices,
for instance, after the bnxt one.
  
Zhao1, Wei Jan. 22, 2018, 1:58 a.m. UTC | #13
Ok, good idea, v5 will come later.

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Friday, January 19, 2018 5:17 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>
> Cc: Zhang, Helin <helin.zhang@intel.com>; dev@dpdk.org; Mcnamara, John
> <john.mcnamara@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>; Xing,
> Beilei <beilei.xing@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature info to
> release notes
> 
> 19/01/2018 10:13, Thomas Monjalon:
> > 19/01/2018 04:15, Zhao1, Wei:
> > > Hi,  Thomas
> > >     Thank you! After discussion with Beilei, I have commit a new patch v4.
> > >  https://dpdk.org/dev/patchwork/patch/33956/
> > > Beilei also commit another new patch, we are waiting for review by
> Mcnamara, John.
> > > Do you think this is ok?
> >
> > Yes it is fine to wait for John's approval.
> >
> > Note: in release notes, you should group all i40e features in the same place.
> > In v4, you just added it at the end.
> > I think v5 is needed.
> 
> More precisions:
> There is no i40e block in 17.11 release notes.
> So you should insert one close to other net devices, for instance, after the
> bnxt one.
  
Zhao1, Wei Jan. 22, 2018, 5:30 a.m. UTC | #14
HI,  Thomas

> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas@monjalon.net]
> Sent: Friday, January 19, 2018 5:17 PM
> To: Zhao1, Wei <wei.zhao1@intel.com>
> Cc: Zhang, Helin <helin.zhang@intel.com>; dev@dpdk.org; Mcnamara, John
> <john.mcnamara@intel.com>; Yigit, Ferruh <ferruh.yigit@intel.com>; Xing,
> Beilei <beilei.xing@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v3] doc: add queue region feature info to
> release notes
> 
> 19/01/2018 10:13, Thomas Monjalon:
> > 19/01/2018 04:15, Zhao1, Wei:
> > > Hi,  Thomas
> > >     Thank you! After discussion with Beilei, I have commit a new patch v4.
> > >  https://dpdk.org/dev/patchwork/patch/33956/
> > > Beilei also commit another new patch, we are waiting for review by
> Mcnamara, John.
> > > Do you think this is ok?
> >
> > Yes it is fine to wait for John's approval.
> >
> > Note: in release notes, you should group all i40e features in the same place.
> > In v4, you just added it at the end.
> > I think v5 is needed.
> 
> More precisions:
> There is no i40e block in 17.11 release notes.
> So you should insert one close to other net devices, for instance, after the
> bnxt one.

Thank you for your help!
I have commit a v5 patch  https://dpdk.org/dev/patchwork/patch/34211/ 
It is based on beilei xing patch https://dpdk.org/dev/patchwork/patch/33953/
I have add i40e comment after hers.
  

Patch

diff --git a/doc/guides/rel_notes/release_18_02.rst b/doc/guides/rel_notes/release_18_02.rst
index 621b8f5..d7c8b68 100644
--- a/doc/guides/rel_notes/release_18_02.rst
+++ b/doc/guides/rel_notes/release_18_02.rst
@@ -47,6 +47,24 @@  New Features
   users. According to the device configuration, it will setup TX->RX loopback
   link or not.
 
+* **Added the i40e ethernet driver to support queue region feature.**
+
+  This feature enable queue regions configuration for RSS in PF,
+  so that different traffic classes or different packet
+  classification types can be separated to different queues in
+  different queue regions. This feature has been implemented in v17.11.
+
+* **Added the testpmd app to support queue region configuration with CLI.**
+
+  This patch add a API configuration of queue region in rss with command line.
+  It can parse the parameters of region index, queue number,
+  queue start index, user priority, traffic classes and so on.
+  According to commands from command line, it will call i40e
+  private API and start the process of set or flush queue region
+  configure. As this feature is specific for i40e, so private API
+  will be used. Aslo add a document for these new commands.
+  This feature has been implemented in v17.11.
+
 API Changes
 -----------