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.
> 
> 

Reply via email to