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

["Wang, Haiyue" <haiyue.wang@intel.com>]

Hi Kevin,

> -----Original Message-----
> From: Kevin Traynor <ktraynor@redhat.com>
> Sent: Friday, November 27, 2020 18:34
> To: Ruifeng Wang <Ruifeng.Wang@arm.com>; dpdk stable <stable@dpdk.org>
> Cc: Zhu, TaoX <taox.zhu@intel.com>; Guo, Jia <jia.guo@intel.com>; Wang, Haiyue <haiyue.wang@intel.com>;
> Thomas Monjalon <thomas@monjalon.net>
> Subject: Re: [dpdk-stable] please help backporting some patches to LTS release 18.11.11
> 
> On 25/11/2020 02:44, Ruifeng Wang wrote:
> > 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.
> >
> > 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.
> >
> 
> Thanks for reply Ruifeng. Dropped these ^ from the list.
> 
> > d70a869db  Phil Yang        net/ixgbe: fix link state timing on fiber ports
> > //This patch depends on 819d0d1d5 and d68ab7a9f that on the same list.
> >
> 
> ok, let's see the reply of those patches author and ixgbe maintainer.
> 
> Tao/Jeff/Haiyue - are these patches needed for 18.11 branch and will you
> be backporting them?
> 
> 819d0d1d5  Tao Zhu net/ixgbe: fix blocking system events
> efa8c72f1  Tao Zhu net/ixgbe: fix link status inconsistencies
> d68ab7a9f  Tao Zhu net/ixgbe: fix resource leak after thread exits normally
> d70a869db2  Phil Yang  net/ixgbe: fix link state timing on fiber ports
> 

Since it is the last 18.11 release, and the above patches introduce new
design for handling corner case, not a real small fix, so we can drop them.

Thanks!

BR,
Haiyue

> Kevin.
> 
> > Thanks.
> > /Ruifeng
> >
> >> -----Original Message-----
> >> From: Kevin Traynor <ktraynor@redhat.com>
> >> Sent: Wednesday, November 25, 2020 12:13 AM
> >> To: dpdk stable <stable@dpdk.org>
> >> Cc: Ajit Khaparde (ajit.khaparde@broadcom.com)
> >> <ajit.khaparde@broadcom.com>; Akhil.goyal@nxp.com; Alexander Kozyrev
> >> <akozyrev@mellanox.com>; Anatoly Burakov <anatoly.burakov@intel.com>;
> >> Andrew Rybchenko <arybchenko@solarflare.com>; Ankur Dwivedi
> >> <adwivedi@marvell.com>; Apeksha Gupta <apeksha.gupta@nxp.com>;
> >> Archana Muniganti <marchana@marvell.com>; Beilei Xing
> >> <beilei.xing@intel.com>; Bernard Iremonger
> >> <bernard.iremonger@intel.com>; Brian Johnson <brian.johnson@intel.com>;
> >> Bruce Richardson <bruce.richardson@intel.com>; Chenbo Xia
> >> <chenbo.xia@intel.com>; Chengchang Tang
> >> <tangchengchang@huawei.com>; Chenxu Di <chenxux.di@intel.com>;
> >> Chunsong Feng <fengchunsong@huawei.com>; David Marchand
> >> <david.marchand@redhat.com>; Dekel Peled <dekelp@mellanox.com>;
> >> Dharmik Thakkar <Dharmik.Thakkar@arm.com>; Didier Pallard
> >> <didier.pallard@6wind.com>; Erik Gabriel Carrillo <erik.g.carrillo@intel.com>;
> >> Fady Bader <fady@mellanox.com>; Fan Zhang <roy.fan.zhang@intel.com>;
> >> Ferruh Yigit <ferruh.yigit@intel.com>; Gagandeep Singh <g.singh@nxp.com>;
> >> Gregory Etelson <getelson@mellanox.com>; Guinan Sun
> >> <guinanx.sun@intel.com>; Guy Tzalik <gtzalik@amazon.com>; Hailin Xu
> >> <hailinx.xu@intel.com>; Harman Kalra <hkalra@marvell.com>;
> >> hemant.agrawal@nxp.com; Honnappa Nagarahalli
> >> <Honnappa.Nagarahalli@arm.com>; Huisong Li <lihuisong@huawei.com>;
> >> Igor Chauskin <igorch@amazon.com>; Igor Romanov
> >> <igor.romanov@oktetlabs.ru>; Jack Min <jackmin@mellanox.com>; Jeff Guo
> >> <jia.guo@intel.com>; Jesse Brandeburg <jesse.brandeburg@intel.com>;
> >> Jiaqi Min <jiaqix.min@intel.com>; Jun W Zhou <junx.w.zhou@intel.com>;
> >> Kalesh AP <kalesh-anakkur.purayil@broadcom.com>; Konstantin Ananyev
> >> <konstantin.ananyev@intel.com>; Lance Richardson
> >> <lance.richardson@broadcom.com>; Lijian Zhang <Lijian.Zhang@arm.com>;
> >> Long Li <longli@microsoft.com>; Luca Boccassi <bluca@debian.org>; Lunyuan
> >> Cui <lunyuanx.cui@intel.com>; Matan Azrad <matan@mellanox.com>;
> >> Matan Azrad <matan@nvidia.com>; Maxime Coquelin
> >> <maxime.coquelin@redhat.com>; Michael Baum <michaelba@nvidia.com>;
> >> Michal Krawczyk <mk@semihalf.com>; Min Hu <humin29@huawei.com>;
> >> Min Wang <wangmin3@huawei.com>; Mohsin Shaikh
> >> <mohsinshaikh@niometrics.com>; Morten Brørup
> >> <mb@smartsharesystems.com>; Nan Chen <whutchennan@gmail.com>;
> >> Nikhil Rao <nikhil.rao@intel.com>; Nipun.gupta@nxp.com; Olivier Matz
> >> <olivier.matz@6wind.com>; Ophir Munk <ophirmu@mellanox.com>; Ori
> >> Kam <orika@mellanox.com>; Pablo de Lara
> >> <pablo.de.lara.guarch@intel.com>; Pavan Nikhilesh
> >> <pbhagavatula@marvell.com>; Phil Yang <Phil.Yang@arm.com>; Qiming Yang
> >> <qiming.yang@intel.com>; Qingmin Liu <qingmin.liu@broadcom.com>; Qi
> >> Zhang <qi.z.zhang@intel.com>; Radu Bulie <radu-andrei.bulie@nxp.com>;
> >> Rahul Lakkireddy <rahul.lakkireddy@chelsio.com>; Ralf Hoffmann
> >> <ralf.hoffmann@allegro-packets.com>; Randy Schacher
> >> <stuart.schacher@broadcom.com>; Remy Horton <remy.horton@intel.com>;
> >> Reshma Pattan <reshma.pattan@intel.com>; Ruifeng Wang
> >> <Ruifeng.Wang@arm.com>; Shiri Kuzin <shirik@nvidia.com>; Shougang
> >> Wang <shougangx.wang@intel.com>; Shy Shyman <shys@mellanox.com>;
> >> Somnath Kotur <somnath.kotur@broadcom.com>; Stephen Hemminger
> >> <stephen@networkplumber.org>; Suanming Mou
> >> <suanmingm@mellanox.com>; Tao Zhu <taox.zhu@intel.com>;
> >> thomas@monjalon.net; Tiwei Bie <tiwei.bie@intel.com>; Venkat Duvvuru
> >> <venkatkumar.duvvuru@broadcom.com>; Viacheslav Ovsiienko
> >> <viacheslavo@mellanox.com>; Viacheslav Ovsiienko
> >> <viacheslavo@nvidia.com>; Wei Huang <wei.huang@intel.com>; Wei Hu
> >> <xavier.huwei@huawei.com>; Wei Ling <weix.ling@intel.com>; Wei Zhao
> >> <wei.zhao1@intel.com>; Xiaolong Ye <xiaolong.ye@intel.com>; Xueming Li
> >> <xuemingl@nvidia.com>; Yingya Han <yingyax.han@intel.com>; Yu Jiang
> >> <yux.jiang@intel.com>; Yunjian Wang <wangyunjian@huawei.com>
> >> Subject: please help backporting some patches to LTS release 18.11.11
> >>
> >> Hi commit authors (and maintainers),
> >>
> >> I didn't apply following commits from DPDK master to 18.11 LTS branch, as
> >> conflicts or build errors occur. Note, it may be dependent on another patch
> >> below that has not been applied.
> >>
> >> As it is the last 18.11 release and there will not be an opportunity to fix any
> >> regressions, I would like to only take backports from the below list at this
> >> stage *if* they are deemed important for the 18.11 branch.
> >>
> >> Can authors check your patches in the following list and either:
> >>     - Backport your important patches to the 18.11 branch, or
> >>     - Indicate patch should be dropped from missed list and not backported, or
> >>     - Take no action => patch will remain on missed list and not be backported
> >>
> >> Please do the above by 12/02/20.
> >>
> >> Some notes on LTS backports:
> >>
> >> A backport should contain a reference to the DPDK master branch commit in
> >> it's commit message in the following fashion:
> >>     [ upstream commit <commit's dpdk master branch SHA-1 checksum> ]
> >>
> >> For example:
> >>     https://git.dpdk.org/dpdk-
> >> stable/commit/?h=18.11&id=d90e6ae6f936ecdc2fd3811ff9f26aec7f3c06eb
> >>
> >> When sending the backported patch, please indicate the target branch in the
> >> subject line, as we have multiple branches, for example:
> >>     [PATCH 18.11] foo/bar: fix baz
> >>
> >> With git format-patch, this can be achieved by appending the parameter:
> >>     --subject-prefix='PATCH 18.11'
> >>
> >> Send the backported patch to "stable@dpdk.org" but not "dev@dpdk.org".
> >>
> >> FYI, branch 18.11 is located at tree:
> >>    https://git.dpdk.org/dpdk-stable
> >>
> >> Thanks.
> >>
> >> Kevin.
> >>
> >> ---
> >> 45c6e1c66  Ajit Khaparde    net/bnxt: increase size of Rx CQ
> >> 91d581dc1  Akhil Goyal      crypto/dpaa2_sec: fix HFN override
> >> 6d3f9917f  Anatoly Burakov  eal: fix memory config allocation for multi-
> >> process
> >> b149a7064  Anatoly Burakov  eal/freebsd: add config reattach in secondary
> >> process
> >> 6080796f6  Anatoly Burakov  mem: make base address hint OS specific
> >> 4236694f0  Anatoly Burakov  mem: preallocate VA space in no-huge mode
> >> be048a1aa  Andrew Rybchenko net/virtio: fix descriptor addressed in Tx
> >> 1526dd053  Andrew Rybchenko net/virtio: fix Tx checksum offloads
> >> 29fdc5bf4  Apeksha Gupta    test/crypto: fix statistics case
> >> 793650184  Archana Muniganti examples/fips_validation: fix version
> >> compatibility
> >> ae08c73e6  Bernard Iremonger net/i40e: fix flow director initialisation
> >> b0b672aea  Bruce Richardson build: add defines for compatibility with make
> >> build e8a83681f  Bruce Richardson eal/x86: fix memcpy AVX-512 enablement
> >> b253a6bbf  Chengchang Tang  app/testpmd: fix packet header in txonly
> >> mode
> >> fe5d0e85b  Chenxu Di        net/i40e: fix flow director flex configuration
> >> dcd05da0a  Dekel Peled      app/testpmd: fix GENEVE flow item
> >> 92818d839  Dekel Peled      net/mlx5: fix match on empty VLAN item in DV
> >> mode
> >> ea81c1b81  Dekel Peled      net/mlx5: fix NVGRE matching
> >> ff55182ce  Dekel Peled      net/mlx5: fix VLAN flow action with wildcard VLAN
> >> item
> >> 9c4971e52  Dekel Peled      net/mlx5: update VLAN and encap actions
> >> validation
> >> 16c011472  Didier Pallard   crypto/octeontx: fix out-of-place support
> >> 036d82365  Fady Bader       mempool: remove inline functions from export list
> >> ea1b835a0  Fan Zhang        vhost/crypto: fix feature negotiation
> >> f6e63e59e  Ferruh Yigit     app/testpmd: fix global variable multiple definitions
> >> 0b42b92ae  Ferruh Yigit     net/bnxt: fix xstats by id
> >> c6887eca5  Gagandeep Singh  crypto/caam_jr: fix device tree parsing for
> >> SEC_ERA
> >> ec8615607  Gagandeep Singh  crypto/dpaa_sec: fix IOVA conversions
> >> 750ff30a8  Gregory Etelson  net/mlx5: fix tunnel flow priority
> >> 3db12449e  Guinan Sun       net/i40e: fix link status
> >> 0acf70d47  Guinan Sun       net/i40e: fix recreating flexible flow director rule
> >> 9d10f53e4  Harman Kalra     test/metrics: fix second run
> >> f6752f660  Igor Romanov     net/sfc: set priority of created filters to manual
> >> 1eab95fe2  Jeff Guo         net/iavf: fix command after PF reset
> >> c02ea7410  Jeff Guo         net/iavf: fix flow flush after PF reset
> >> 851b22ff6  Jeff Guo         net/iavf: fix vector Rx
> >> ead06572b  Jesse Brandeburg net/iavf: fix performance with writeback policy
> >> c92f053d4  Kalesh AP        net/bnxt: fix L2 filter allocation
> >> 99ad2abc0  Kalesh AP        net/bnxt: fix speed setting on certain adapters
> >> ce41561d1  Kalesh AP        net/bnxt: fix structure variable initialization
> >> c4ae7adaf  Kalesh AP        net/bnxt: fix UDP tunnel port removal
> >> a135e050a  Konstantin Ananyev examples/ipsec-secgw: fix packet length
> >> 47b707616  Lance Richardson net/bnxt: fix boolean operator usage
> >> bd881e8d2  Lance Richardson net/bnxt: fix drop enable in get Rx queue info
> >> 06387be8e  Matan Azrad      net/mlx5: fix encap/decap validation
> >> 721c95301  Matan Azrad      net/mlx5: fix Rx scatter mode validation
> >> 050bfe033  Matan Azrad      net/mlx5: fix tunnel flow priority
> >> c59898131  Maxime Coquelin  vhost: validate index in async API
> >> 8b3799531  Michael Baum     net/mlx5: remove unused calculation in RSS
> >> expansion
> >> fa2dd3d4d  Michael Baum     net/mlx5: remove unused variable in Tx queue
> >> creation
> >> 856edce2b  Michal Krawczyk  net/ena: fix setting Rx checksum flags in mbuf
> >> 00437823c  Mohsin Shaikh    net/mlx5: use open/read/close for ib stats query
> >> 9275af3bd  Olivier Matz     examples/fips_validation: fix buffer overflow
> >> f2e8b4556  Ophir Munk       net/mlx5: remove unused includes
> >> 9e0d81c1a  Ori Kam          net/mlx5: fix selection between encap and decap
> >> 07bfb9047  Pablo de Lara    crypto/aesni_mb: fix CCM digest size check
> >> 7392ad06f  Pavan Nikhilesh  app/testpmd: use better randomness for Tx split
> >> cb4261e0b  Pavan Nikhilesh  event/octeontx2: improve datapath memory
> >> locality
> >> 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
> >> d70a869db  Phil Yang        net/ixgbe: fix link state timing on fiber ports
> >> f0617163b  Radu Bulie       mempool/dpaa2: report error on endless loop in
> >> mbuf release
> >> 6b78a6299  Rahul Lakkireddy net/cxgbe: fix queue DMA ring leaks during port
> >> close
> >> 207b1c813  Reshma Pattan    test: fix build without ring PMD
> >> 76a4ed5b3  Ruifeng Wang     crypto/armv8: fix mempool object returning
> >> 42dcd453d  Shiri Kuzin      net/mlx5: fix xstats reset reinitialization
> >> 11b58ac70  Shougang Wang    net/i40e: fix filter pctype
> >> da7018ec2  Shougang Wang    net/i40e: fix queue region in RSS flow
> >> 3b931ddb5  Shy Shyman       app/testpmd: fix error detection in MTU
> >> command
> >> 4b029f02d  Somnath Kotur    net/bnxt: fix checking VNIC in shutdown path
> >> 97c327178  Somnath Kotur    net/bnxt: fix queue release
> >> 1ef8c6290  Stephen Hemminger net/bnxt: remove useless prefetches
> >> 7838d3a6a  Stephen Hemminger net/netvsc: check for overflow on packet
> >> info from host
> >> a4f53bec7  Stephen Hemminger net/netvsc: do not query VF link state
> >> c6854a412  Stephen Hemminger net/netvsc: fix warning when VF is removed
> >> 2786b7bf9  Suanming Mou     net/mlx5: fix secondary process resources
> >> release
> >> 819d0d1d5  Tao Zhu          net/ixgbe: fix blocking system events
> >> efa8c72f1  Tao Zhu          net/ixgbe: fix link status inconsistencies
> >> d68ab7a9f  Tao Zhu          net/ixgbe: fix resource leak after thread exits
> >> normally
> >> e21492a51  Thomas Monjalon  net/mlx: fix overlinking with meson and glue
> >> dlopen
> >> ef10add37  Venkat Duvvuru   net/bnxt: fix PF support in SR-IOV mode
> >> 482a1d34b  Viacheslav Ovsiienko common/mlx5: fix PCI address lookup
> >> ceccbcd73  Wei Huang        raw/ifpga: use trusted buffer to free
> >> 150c9ac2d  Wei Hu (Xavier)  app/testpmd: update Rx offload after setting
> >> MTU 2e02a2aff  Wei Hu (Xavier)  ethdev: fix VLAN offloads set if no driver
> >> callback
> >> c52ff3668  Wei Zhao         net/i40e: enable QinQ stripping
> >> be4ef2e0d  Wei Zhao         net/i40e: fix flow director enabling
> >> dcfbc594f  Wei Zhao         net/iavf: fix queue interrupt for ice
> >> c21e5facf  Xueming Li       net/mlx5: use bond index for netdev operations
> >> 2477be96e  Yunjian Wang     net/bnxt: add memory allocation check in VF info
> >> init
> >> 9fb9a94ca  Yunjian Wang     net/bnxt: fix memory leak when freeing VF info
> >> 24cb500c1  Yunjian Wang     net/tap: fix mbuf double free when writev fails
> >