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:tho...@monjalon.net] > Sent: Thursday, January 18, 2018 3:30 PM > To: Zhao1, Wei <wei.zh...@intel.com> > Cc: Zhang, Helin <helin.zh...@intel.com>; dev@dpdk.org; Mcnamara, John > <john.mcnam...@intel.com>; Yigit, Ferruh <ferruh.yi...@intel.com>; Xing, > Beilei <beilei.x...@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:tho...@monjalon.net] > > > Sent: Wednesday, January 17, 2018 5:53 AM > > > To: Zhang, Helin <helin.zh...@intel.com>; Zhao1, Wei > > > <wei.zh...@intel.com> > > > Cc: dev@dpdk.org; Mcnamara, John <john.mcnam...@intel.com>; Yigit, > > > Ferruh <ferruh.yi...@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.zh...@intel.com> > > > > > Acked-by: Helin Zhang <helin.zh...@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. > >