[dpdk-stable] please help backporting some patches to LTS release 18.11.11

Ruifeng Wang Ruifeng.Wang at arm.com
Wed Nov 25 10:19:09 CET 2020


> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Wednesday, November 25, 2020 4:24 PM
> To: Ruifeng Wang <Ruifeng.Wang at arm.com>
> Cc: dpdk stable <stable at dpdk.org>; Ajit Khaparde
> (ajit.khaparde at broadcom.com) <ajit.khaparde at broadcom.com>;
> Akhil.goyal at nxp.com; Alexander Kozyrev <akozyrev at mellanox.com>;
> Anatoly Burakov <anatoly.burakov at intel.com>; Andrew Rybchenko
> <arybchenko at solarflare.com>; Ankur Dwivedi <adwivedi at marvell.com>;
> Apeksha Gupta <apeksha.gupta at nxp.com>; Archana Muniganti
> <marchana at marvell.com>; Beilei Xing <beilei.xing at intel.com>; Bernard
> Iremonger <bernard.iremonger at intel.com>; Brian Johnson
> <brian.johnson at intel.com>; Bruce Richardson
> <bruce.richardson at intel.com>; Chenbo Xia <chenbo.xia at intel.com>;
> Chengchang Tang <tangchengchang at huawei.com>; Chenxu Di
> <chenxux.di at intel.com>; Chunsong Feng <fengchunsong at huawei.com>;
> David Marchand <david.marchand at redhat.com>; Dekel Peled
> <dekelp at mellanox.com>; Dharmik Thakkar <Dharmik.Thakkar at arm.com>;
> Didier Pallard <didier.pallard at 6wind.com>; Erik Gabriel Carrillo
> <erik.g.carrillo at intel.com>; Fady Bader <fady at mellanox.com>; Fan Zhang
> <roy.fan.zhang at intel.com>; Ferruh Yigit <ferruh.yigit at intel.com>;
> Gagandeep Singh <g.singh at nxp.com>; Gregory Etelson
> <getelson at mellanox.com>; Guinan Sun <guinanx.sun at intel.com>; Guy
> Tzalik <gtzalik at amazon.com>; Hailin Xu <hailinx.xu at intel.com>; Harman
> Kalra <hkalra at marvell.com>; hemant.agrawal at nxp.com; Honnappa
> Nagarahalli <Honnappa.Nagarahalli at arm.com>; Huisong Li
> <lihuisong at huawei.com>; Igor Chauskin <igorch at amazon.com>; Igor
> Romanov <igor.romanov at oktetlabs.ru>; Jack Min <jackmin at mellanox.com>;
> Jeff Guo <jia.guo at intel.com>; Jesse Brandeburg
> <jesse.brandeburg at intel.com>; Jiaqi Min <jiaqix.min at intel.com>; Jun W
> Zhou <junx.w.zhou at intel.com>; Kalesh AP <kalesh-
> anakkur.purayil at broadcom.com>; Konstantin Ananyev
> <konstantin.ananyev at intel.com>; Lance Richardson
> <lance.richardson at broadcom.com>; Lijian Zhang <Lijian.Zhang at arm.com>;
> Long Li <longli at microsoft.com>; Luca Boccassi <bluca at debian.org>; Lunyuan
> Cui <lunyuanx.cui at intel.com>; Matan Azrad <matan at mellanox.com>;
> Matan Azrad <matan at nvidia.com>; Maxime Coquelin
> <maxime.coquelin at redhat.com>; Michael Baum <michaelba at nvidia.com>;
> Michal Krawczyk <mk at semihalf.com>; Min Hu <humin29 at huawei.com>;
> Min Wang <wangmin3 at huawei.com>; Mohsin Shaikh
> <mohsinshaikh at niometrics.com>; Morten Brørup
> <mb at smartsharesystems.com>; Nan Chen <whutchennan at gmail.com>;
> Nikhil Rao <nikhil.rao at intel.com>; Nipun.gupta at nxp.com; Olivier Matz
> <olivier.matz at 6wind.com>; Ophir Munk <ophirmu at mellanox.com>; Ori
> Kam <orika at mellanox.com>; Pablo de Lara
> <pablo.de.lara.guarch at intel.com>; Pavan Nikhilesh
> <pbhagavatula at marvell.com>; Phil Yang <Phil.Yang at arm.com>; Qiming Yang
> <qiming.yang at intel.com>; Qingmin Liu <qingmin.liu at broadcom.com>; Qi
> Zhang <qi.z.zhang at intel.com>; Radu Bulie <radu-andrei.bulie at nxp.com>;
> Rahul Lakkireddy <rahul.lakkireddy at chelsio.com>; Ralf Hoffmann
> <ralf.hoffmann at allegro-packets.com>; Randy Schacher
> <stuart.schacher at broadcom.com>; Remy Horton <remy.horton at intel.com>;
> Reshma Pattan <reshma.pattan at intel.com>; Shiri Kuzin <shirik at nvidia.com>;
> Shougang Wang <shougangx.wang at intel.com>; Shy Shyman
> <shys at mellanox.com>; Somnath Kotur <somnath.kotur at broadcom.com>;
> Stephen Hemminger <stephen at networkplumber.org>; Suanming Mou
> <suanmingm at mellanox.com>; Tao Zhu <taox.zhu at intel.com>; Tiwei Bie
> <tiwei.bie at intel.com>; Venkat Duvvuru
> <venkatkumar.duvvuru at broadcom.com>; Viacheslav Ovsiienko
> <viacheslavo at mellanox.com>; Viacheslav Ovsiienko
> <viacheslavo at nvidia.com>; Wei Huang <wei.huang at intel.com>; Wei Hu
> <xavier.huwei at huawei.com>; Wei Ling <weix.ling at intel.com>; Wei Zhao
> <wei.zhao1 at intel.com>; Xiaolong Ye <xiaolong.ye at intel.com>; Xueming Li
> <xuemingl at nvidia.com>; Yingya Han <yingyax.han at intel.com>; Yu Jiang
> <yux.jiang at intel.com>; Yunjian Wang <wangyunjian at huawei.com>; nd
> <nd at arm.com>
> Subject: Re: please help backporting some patches to LTS release 18.11.11
> 
> 25/11/2020 03:44, Ruifeng Wang:
> > Hi Kevin,
> >
> > 76a4ed5b3  Ruifeng Wang     crypto/armv8: fix mempool object returning
> > //This patch should be dropped from missed list and not backported. The
> issue it fixes doesn't exist in 18.11 branch.
> 
> Why it is marked as fixing a commit from 2017?

This was a mistake on fix tag. 
The patch changed two lines of code. Fix tag was generated for both of the lines which involves two commits.
However, issue was introduced by only the later patch.
> 
> > Reply for some patches from Phil Yang:
> >
> > 059a23108  Phil Yang        app/testpmd: enable burst stats for noisy VNF
> mode
> > 030c21641  Phil Yang        eventdev: relax SMP barriers with C11 atomics
> > e84d9c62c  Phil Yang        eventdev: remove redundant reset on timer
> cancel
> > 1028d63eb  Phil Yang        eventdev: use C11 atomics for lcore timer armed
> flag
> > //These patches should be dropped from missed list and not backported.
> They are optimization or not critical bug fix.
> 
> Why they are marked with Cc: stable?

Discussion in the thread:
http://patches.dpdk.org/patch/71351/
Because one of bug fix patch in the patch needed to be backported, and it was
hard to use rte_atomic* API to do the fix, so C11 atomic built-in was used. To keep code in stable
release clean, decision was made to backport all other patches in the patch set that involves
C11 atomic built-in.
> 
> 



More information about the stable mailing list