RE: please help backporting some patches to stable release 19.11.13

["Richardson, Bruce" <bruce.richardson@intel.com>]

Hi,

I don't believe patch f940a2e333e3 ("test: validate test names in non interactive mode") can be backported as it depends on new functionality added to cmdline lib [Patch 758d92795b1c ("cmdline: add function to verify valid commands")]. 
While you could look to take the cmdline lib patch also - which should be very low risk - I think for a unit test app it's probably ok to skip the fix. I'll leave the decision to you as maintainer, though.

/Bruce

> -----Original Message-----
> From: christian.ehrhardt@canonical.com <christian.ehrhardt@canonical.com>
> Sent: Thursday, July 7, 2022 9:08 AM
> Cc: Aaron Conole <aconole@redhat.com>; Adrien Mazarguil
> <adrien.mazarguil@6wind.com>; Ajit Khaparde <ajit.khaparde@broadcom.com>;
> Akhil Goyal <akhil.goyal@nxp.com>; Akhil Goyal <gakhil@marvell.com>;
> Alejandro Lucero <alejandro.lucero@netronome.com>; Singh, Aman Deep
> <aman.deep.singh@intel.com>; Burakov, Anatoly <anatoly.burakov@intel.com>;
> Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>; Andy Gospodarek
> <andrew.gospodarek@broadcom.com>; Pei, Andy <andy.pei@intel.com>; Apeksha
> Gupta <apeksha.gupta@nxp.com>; Ashish Jain <ashish.jain@nxp.com>; Xing,
> Beilei <beilei.xing@intel.com>; Iremonger, Bernard
> <bernard.iremonger@intel.com>; Richardson, Bruce
> <bruce.richardson@intel.com>; Changchun Ouyang
> <changchun.ouyang@intel.com>; Chaoyong He <chaoyong.he@corigine.com>; Chas
> Williams <chas3@att.com>; Xia, Chenbo <chenbo.xia@intel.com>; Chengwen
> Feng <fengchengwen@huawei.com>; Chen Jing D(Mark) <jing.d.chen@intel.com>;
> Chunsong Feng <fengchunsong@huawei.com>; Power, Ciara
> <ciara.power@intel.com>; Damodharam Ammepalli
> <damodharam.ammepalli@broadcom.com>; David Christensen
> <david.christensen@broadcom.com>; David Christensen
> <drc@linux.vnet.ibm.com>; David Marchand <david.marchand@redhat.com>; Gao,
> DaxueX <daxuex.gao@intel.com>; Doherty, Declan <declan.doherty@intel.com>;
> Jain, Deepak K <deepak.k.jain@intel.com>; Dmitry Kozlyuk
> <dkozlyuk@nvidia.com>; Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>; Dmitry
> Malloy <dmitrym@microsoft.com>; Dongdong Liu <liudongdong3@huawei.com>;
> Duncan Bellamy <dunk@denkimushi.com>; Evgeny Schemeilin
> <evgenys@amazon.com>; Zhang, Roy Fan <roy.fan.zhang@intel.com>; Ferruh
> Yigit <ferruh.yigit@intel.com>; Ferruh Yigit <ferruh.yigit@xilinx.com>;
> Trahe, Fiona <fiona.trahe@intel.com>; Gagandeep Singh <g.singh@nxp.com>;
> Gregory Etelson <getelson@nvidia.com>; Haiyang Zhang
> <haiyangz@microsoft.com>; Hao Chen <chenhao164@huawei.com>; Van Haaren,
> Harry <harry.van.haaren@intel.com>; Zhang, Helin <helin.zhang@intel.com>;
> Hemant Agrawal <hemant.agrawal@nxp.com>; Lipiec, Herakliusz
> <herakliusz.lipiec@intel.com>; Huawei Xie <huawei.xie@intel.com>; Huisong
> Li <lihuisong@huawei.com>; Igor Chauskin <igorch@amazon.com>; Ivan Boule
> <ivan.boule@6wind.com>; Poczatek, Jakub <jakub.poczatek@intel.com>; Jan
> Viktorin <viktorin@rehivetech.com>; Jerin Jacob
> <jerin.jacob@caviumnetworks.com>; Jerin Jacob <jerinj@marvell.com>;
> Jianfeng Tan <jianfeng.tan@intel.com>; Hu, Jiayu <jiayu.hu@intel.com>; Wu,
> Jingjing <jingjing.wu@intel.com>; Mcnamara, John
> <john.mcnamara@intel.com>; Kalesh AP <kalesh-
> anakkur.purayil@broadcom.com>; Laatz, Kevin <kevin.laatz@intel.com>;
> Zhang, Ke1X <ke1x.zhang@intel.com>; Kiran Kumar K
> <kirankumark@marvell.com>; Konstantin Ananyev
> <konstantin.ananyev@intel.com>; Konstantin Ananyev
> <konstantin.v.ananyev@yandex.ru>; Kumar Sanghvi <kumaras@chelsio.com>;
> Lijun Ou <oulijun@huawei.com>; Long Li <longli@microsoft.com>; Louis Peens
> <louis.peens@corigine.com>; Marcin Baran <marcinx.baran@intel.com>
> Subject: please help backporting some patches to stable release 19.11.13
> 
> Hi commit authors (and maintainers),
> 
> Despite being selected by the DPDK maintenance tool ./devtools/git-log-
> fixes.sh
> I didn't apply following commits from DPDK main to 19.11
> stable branch, as conflicts or build errors occur.
> 
> Can authors check your patches in the following list and either:
>     - Backport your patches to the 19.11 branch, or
>     - Indicate that the patch should not be backported
> 
> Please do either of the above by 07/17/22.
> 
> You can find the a temporary work-in-progress branch of the coming
> 19.11.13
> release at:
>     https://github.com/cpaelzer/dpdk-stable-queue
> It is recommended to backport on top of that to minimize further conflicts
> or
> misunderstandings.
> 
> Some notes on stable backports:
> 
> A backport should contain a reference to the DPDK main branch commit
> in it's commit message in the following fashion:
>     [ upstream commit <commit's dpdk main 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 19.11] foo/bar: fix baz
> 
> With git format-patch, this can be achieved by appending the parameter:
>     --subject-prefix='PATCH 19.11'
> 
> Send the backported patch to "stable@dpdk.org" but not "dev@dpdk.org".
> 
> FYI, branch 19.11 is located at tree:
>    https://git.dpdk.org/dpdk-stable
> 
> Thanks.
> 
> Christian Ehrhardt <christian.ehrhardt@canonical.com>
> 
> ---
> f940a2e333  Bruce Richardson test: validate test names in non interactive
> mode
> 7d3cb76fba  Chengwen Feng    examples/dma: fix Tx drop statistics
> a42ab1eb33  Damodharam Ammepalli net/bnxt: disallow MTU change when device
> is started
> 5fc2eece8d  David Marchand   app/flow-perf: fix build with GCC 12
> 468f31eb71  David Marchand   crypto/ipsec_mb: fix build with GCC 12
> 2f51bc9c27  David Marchand   eal/freebsd: fix use of newer cpuset macros
> 1e230b9be8  David Marchand   eal/windows: add missing C++ include guards
> 2449949584  David Marchand   net/ena: fix build with GCC 12
> 7c3c0d0f29  David Marchand   net/enetfec: fix build with GCC 12
> 3c4426db54  Dmitry Kozlyuk   app/testpmd: do not poll stopped queues
> e96d3d02d6  Dmitry Kozlyuk   common/mlx5: fix non-expandable global MR
> cache
> 979bb5d493  Dmitry Kozlyuk   doc: add more instructions for running as
> non-root
> 19ec0f421e  Dmitry Kozlyuk   doc: update running as non-root in mlx5 guide
> 0615dd2aa1  Duncan Bellamy   eal/ppc: fix compilation for musl
> e3d264383b  Gagandeep Singh  crypto/dpaa2_sec: fix crypto operation
> pointer
> 6a2b01a410  Gagandeep Singh  crypto/dpaa2_sec: fix fle buffer leak
> d05828ac58  Gagandeep Singh  doc: add missing auth algo for IPsec example
> 85bd316d08  Harry van Haaren doc: fix formatting and link in BPF library
> guide
> c8a3ee49c9  Herakliusz Lipiec doc: fix readability in vhost guide
> 3889a3220c  Huisong Li       app/testpmd: fix bonding slave devices not
> released
> e46372d7b0  Huisong Li       app/testpmd: fix port status of bonding slave
> device
> 3c23ee6cdd  Huisong Li       app/testpmd: fix supported RSS offload
> display
> f5e72e8e8d  Huisong Li       net/bonding: fix stopping non-active slaves
> ea1ef0b937  Huisong Li       net/hns3: fix an unreasonable memset
> 8ba42ce98d  Huisong Li       net/hns3: fix crash from secondary process
> a65342d9d5  Huisong Li       net/hns3: fix MAC and queues HW statistics
> overflow
> 07f64b5f57  Huisong Li       net/hns3: fix rollback on RSS hash update
> 75ccc3f3d7  Huisong Li       net/hns3: fix RSS disable
> 7b29666261  Huisong Li       net/hns3: fix statistics locking
> bfd0b54dc8  Huisong Li       net/hns3: remove redundant RSS tuple field
> ec16914942  Huisong Li       net/hns3: remove unnecessary RSS switch
> 7fe84b0de2  Jakub Poczatek   doc: fix grammar and parameters in l2fwd-
> crypto guide
> 47a956a8fe  Kalesh AP        net/bnxt: fix device capability reporting
> 7b6eba8856  Kalesh AP        net/bnxt: fix link status when port is
> stopped
> 0a90c56eac  Kalesh AP        net/bnxt: fix Rx configuration
> c0278f6e52  Kalesh AP        net/bnxt: fix tunnel stateless offloads
> 9c1410bed3  Kalesh AP        net/bnxt: force PHY update on certain
> configurations
> 68629be3a6  Ke Zhang         app/testpmd: fix multicast address pool leak
> fced83c122  Ke Zhang         net/iavf: fix mbuf release in multi-process
> a08f9cb698  Ke Zhang         net/iavf: fix Rx queue interrupt setting
> 559a1f2eea  Long Li          net/netvsc: fix calculation of checksums
> based on mbuf flag
> 7b1a614dcb  Long Li          net/netvsc: fix vmbus device reference in
> multi-process
> a213b86821  Michael Baum     net/mlx5: fix LRO validation in Rx setup
> 87063aaff3  Mike Pattrick    net/i40e: populate error in flow director
> parser
> ea6101a8e4  Min Hu (Connor)  ethdev: fix port close in secondary process
> 7988cdca98  Peng Zhang       net/nfp: fix disabling VLAN stripping
> 417be15e5f  Peng Zhang       net/nfp: make sure MTU is never larger than
> mbuf size
> 151e828f64  Rahul Lakkireddy net/cxgbe: fix Tx queue stuck with mbuf chain
> coalescing
> 773a7de21a  Raja Zidane      net/mlx5: fix Rx/Tx stats concurrency
> 5ddb903824  Raja Zidane      net/mlx5: reject negative integrity item
> configuration
> 1b27f8245e  Somnath Kotur    net/bnxt: fix ring group on Rx restart
> f75dd6d3b1  Stanislaw Kardach config: fix C++ cross compiler for Arm and
> PPC
> 0b5da8cc52  Tianli Lai       net/dpaa2: fix dpdmux default interface
> 2ddf4b110c  Tomasz Duszynski common/cnxk: allow changing PTP mode on CN10K
> ae835aba40  Wei Huang        raw/ifpga: remove virtual devices on close
> 2545683564  Wei Huang        raw/ifpga: unregister interrupt on close
> 1945c64674  Wenwu Ma         app/testpmd: perform SW IP checksum for
> GRO/GSO packets
> 3e87e12dc8  Wenxuan Wu       net/ice/base: fix build with GCC 12
> 1907ce4bae  Yuan Wang        examples/vhost: fix retry logic on Rx path
> 41f9a1757f  Yuan Wang        net/virtio-user: fix socket non-blocking mode