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:[email protected]]
> Sent: Thursday, January 18, 2018 3:30 PM
> To: Zhao1, Wei <[email protected]>
> Cc: Zhang, Helin <[email protected]>; [email protected]; Mcnamara, John
> <[email protected]>; Yigit, Ferruh <[email protected]>; Xing,
> Beilei <[email protected]>
> 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:[email protected]]
> > > Sent: Wednesday, January 17, 2018 5:53 AM
> > > To: Zhang, Helin <[email protected]>; Zhao1, Wei
> > > <[email protected]>
> > > Cc: [email protected]; Mcnamara, John <[email protected]>; Yigit,
> > > Ferruh <[email protected]>
> > > 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 <[email protected]>
> > > > > Acked-by: Helin Zhang <[email protected]>
> > > > 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