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

[Somnath Kotur <somnath.kotur@broadcom.com>]

OK Goti it! Thanks for the update Kevin

-Som

On Wed, Dec 18, 2019 at 4:40 PM Kevin Traynor <ktraynor@redhat.com> wrote:
>
> On 18/12/2019 06:26, Somnath Kotur wrote:
> > Kevin,
> >            Taken care of both things (rebased with latest
> > dpdk-stable:18.11) and added the master commit ids for patches that
> > were squashed and sent the patch set just now to stable@dpdk.org
> >
>
> Thanks Som. Just to note that the master commit ids should go on all
> rebased patches that are being sent directly for stable branches.
> (Stable maintainer will add for patches that they can backport directly
> from master). No need to resend, I've taken care of it for the patches
> in this series.
>
> fyi - you've covered the majority of bnxt patches but there are still a
> handful that weren't in the list not to backport, or didn't seem to be
> in the series. We can discuss them in January, as I want to release
> 18.11.6-rc1 today.
>
> thanks,
> Kevin.
>
>
> > Thanks
> > Som
> >
> > On Tue, Dec 17, 2019 at 7:40 PM Somnath Kotur
> > <somnath.kotur@broadcom.com> wrote:
> >>
> >> Oh ok sure will do
> >>
> >> Thanks
> >> Som
> >>
> >> On Tue, 17 Dec 2019, 19:34 Kevin Traynor, <ktraynor@redhat.com> wrote:
> >>>
> >>> On 17/12/2019 11:06, Kevin Traynor wrote:
> >>>> On 17/12/2019 10:06, Somnath Kotur wrote:
> >>>>> Hi Kevin,
> >>>>>                Out of the 30 bnxt patches that you have identified for
> >>>>> 18.11,  we have managed to rebase 25 of them to a total of 19
> >>>>> patches(some of them squashed)
> >>>>> which i will be sending as a patch-set to dpdk-stable soon (before
> >>>>> your EOD Dec 18th)
> >>>>>
> >>>
> >>> One other thing, no problem to squash patches if it makes sense, but
> >>> please have a reference to each of the master commit ids in the commit
> >>> log. That way it is easy to cross reference now and in future which
> >>> commits are backported.
> >>>
> >>> thanks,
> >>> Kevin.
> >>>
> >>> e.g.
> >>> commit 0c97c2c11cb9408f35cebbb6aaf94ed45671ccbc
> >>> Author: Xiao Zhang <xiao.zhang@intel.com>
> >>> Date:   Mon Jul 22 23:11:52 2019 +0800
> >>>
> >>>     net/e1000: fix i219 hang on reset/close
> >>>
> >>>     [ upstream commit 1fc9701238edcf0541289b9ae15565b6d9d7ab30]
> >>>     [ upstream commit 675f65dc660805eff1953e3dfb6242ec076a9444]
> >>>
> >>>
> >>>>
> >>>> Thanks Somnath, this is great to hear. Just one request - there have
> >>>> been a lot of patches queued last week and pushed this week, so can you
> >>>> check that your rebases are still ok with the current dpdk-stable:18.11
> >>>> branch.
> >>>>
> >>>>> The commits that won't be backported for 18.11 are
> >>>>>
> >>>>> e9ccabda0  Rahul Gupta      net/bnxt: reduce cleanup time during reset recovery
> >>>>> d6c7475b9  Santoshkumar Karanappa Rastapur net/bnxt: free default
> >>>>> completion ring before VF config
> >>>>> 5526c8025  Somnath Kotur    net/bnxt: fix race between interrupt
> >>>>> handler and dev config
> >>>>> 037cc747d  Venkat Duvvuru   net/bnxt: fix default MAC check
> >>>>> 6118503d8  Venkat Duvvuru   net/bnxt: fix VLAN filtering
> >>>>>
> >>>>
> >>>> Sure, I will remove them from list
> >>>>
> >>>> thanks,
> >>>> Kevin.
> >>>>
> >>>>>
> >>>>>
> >>>>> Thanks
> >>>>> Som
> >>>>>
> >>>>>
> >>>>> On Thu, Dec 12, 2019 at 5:29 PM Kevin Traynor <ktraynor@redhat.com> wrote:
> >>>>>>
> >>>>>> 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 also not been applied.
> >>>>>>
> >>>>>> Can authors check your patches in the following list and either:
> >>>>>>     - Backport your patches to the 18.11 branch, or
> >>>>>>     - Indicate that the patch should not be backported
> >>>>>>
> >>>>>> Please do either of the above by 12/18/19.
> >>>>>>
> >>>>>> 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.
> >>>>>>
> >>>>>> ---
> >>>>>> 1fc3b3f06  Adrian Moreno    vhost: convert buffer addresses to GPA for logging
> >>>>>> c023cd5b2  Ajit Khaparde    net/bnxt: fix async link handling and update
> >>>>>> 21b1f1abe  Ajit Khaparde    net/bnxt: fix COS queue mapping
> >>>>>> fe7848521  Ajit Khaparde    net/bnxt: fix doorbell register offset for Tx ring
> >>>>>> 89278c59d  Ajit Khaparde    net/bnxt: fix flow flush handling
> >>>>>> 458f0360e  Ajit Khaparde    net/bnxt: get default HWRM command timeout from FW
> >>>>>> 305d1fd39  Ajit Khaparde    net/bnxt: update trusted VF status only when it changes
> >>>>>> 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
> >>>>>> be048a1aa  Andrew Rybchenko net/virtio: fix descriptor addressed in Tx
> >>>>>> 1526dd053  Andrew Rybchenko net/virtio: fix Tx checksum offloads
> >>>>>> 7bbc7dc43  Andrzej Ostruszka net/dpaa2: fix possible use of uninitialized vars
> >>>>>> ea81c1b81  Dekel Peled      net/mlx5: fix NVGRE matching
> >>>>>> 9d10f53e4  Harman Kalra     test/metrics: fix second run
> >>>>>> 0964a9512  Hemant Agrawal   bus/dpaa: fix dpaa_sec blacklist
> >>>>>> 2967612f4  Hemant Agrawal   test/crypto: fix session init failure for wireless case
> >>>>>> b0437f8b0  Honnappa Nagarahalli hash: load value after full key compare
> >>>>>> 6f43682e0  Honnappa Nagarahalli test/hash: init parameters in the correct function
> >>>>>> 05817057f  Igor Chauskin    net/ena: fix indication of bad L4 Rx checksums
> >>>>>> e0d865357  Kalesh AP        net/bnxt: enable interrupts according interface status
> >>>>>> 1f3cea004  Kalesh AP        net/bnxt: fix check of address mapping
> >>>>>> 996355970  Kalesh AP        net/bnxt: fix coding style
> >>>>>> 2171e66f9  Kalesh AP        net/bnxt: fix deferred start of Tx queues
> >>>>>> 324c56551  Kalesh AP        net/bnxt: fix error checking of FW commands
> >>>>>> 9bc556e3e  Kalesh AP        net/bnxt: fix error handling in xstats
> >>>>>> 96b0931d5  Kalesh AP        net/bnxt: fix extended port counter statistics
> >>>>>> dc177579d  Kalesh AP        net/bnxt: fix L2 context calculation for Thor
> >>>>>> f5019a53d  Kalesh AP        net/bnxt: fix memory leak
> >>>>>> 3955e2687  Kalesh AP        net/bnxt: fix Rx queue start/stop
> >>>>>> 988562f0b  Kalesh AP        net/bnxt: fix setting default MAC address
> >>>>>> 9f55e6ac7  Kalesh AP        net/bnxt: fix stats errors handling
> >>>>>> aa2c00702  Kalesh AP        net/bnxt: fix traffic stall on Rx queue stop/start
> >>>>>> 72aaa312e  Kalesh AP        net/bnxt: fix VF probe when MAC address is zero
> >>>>>> 9924dfd60  Kalesh AP        net/bnxt: move macro definitions to header file
> >>>>>> a135e050a  Konstantin Ananyev examples/ipsec-secgw: fix packet length
> >>>>>> e76097f84  Marvin Liu       net/virtio: cleanup on demand when in-order Tx
> >>>>>> 1ae55ad38  Marvin Liu       net/virtio: fix mbuf data and packet length mismatch
> >>>>>> 721c95301  Matan Azrad      net/mlx5: fix Rx scatter mode validation
> >>>>>> ce4fd609b  Nipun Gupta      net/dpaa2: add retry and timeout in packet enqueue API
> >>>>>> 005d943e5  Nipun Gupta      net/dpaa2: set port in mbuf
> >>>>>> 0da362b8f  Nipun Gupta      raw/dpaa2_cmdif: add retry and timeout in packet enqueue API
> >>>>>> 9e0d81c1a  Ori Kam          net/mlx5: fix selection between encap and decap
> >>>>>> 7392ad06f  Pavan Nikhilesh  app/testpmd: use better randomness for Tx split
> >>>>>> e9924638f  Pavel Belous     net/atlantic: add FW mailbox guard mutex
> >>>>>> f0617163b  Radu Bulie       mempool/dpaa2: report error on endless loop in mbuf release
> >>>>>> 34c0ba839  Rahul Gupta      net/bnxt: fix Rx queue count
> >>>>>> e9ccabda0  Rahul Gupta      net/bnxt: reduce cleanup time during reset recovery
> >>>>>> 8937597cf  Santoshkumar Karanappa Rastapur net/bnxt: fix crash after removing and adding slaves
> >>>>>> 09aac391a  Santoshkumar Karanappa Rastapur net/bnxt: fix forwarding with higher mbuf size
> >>>>>> b4e190d55  Santoshkumar Karanappa Rastapur net/bnxt: fix MAC/VLAN filter allocation
> >>>>>> d6c7475b9  Santoshkumar Karanappa Rastapur net/bnxt: free default completion ring before VF config
> >>>>>> 5526c8025  Somnath Kotur    net/bnxt: fix race between interrupt handler and dev config
> >>>>>> f35eaaca5  Stephen Hemminger net/bnxt: fix crash in secondary process
> >>>>>> 037cc747d  Venkat Duvvuru   net/bnxt: fix default MAC check
> >>>>>> 6118503d8  Venkat Duvvuru   net/bnxt: fix VLAN filtering
> >>>>>> dcfbc594f  Wei Zhao         net/iavf: fix queue interrupt for ice
> >>>>>> 6977f1451  Wisam Jaddo      app/testpmd: fix missing GENEVE item in raw encap
> >>>>>> 8e2f25cf3  Xiaoyu Min       net/mlx5: fix crash on hash Rx queue handling for drop
> >>>>>> fba321301  Xiaoyu Min       net/mlx5: improve flow item IP validation
> >>>>>>
> >>>>>
> >>>>
> >>>
> >
>