patches for DPDK stable branches
 help / color / mirror / Atom feed
From: "Wei Hu (Xavier)" <huwei013@chinasoftinc.com>
To: Luca Boccassi <bluca@debian.org>
Cc: <stable@dpdk.org>, <xavier.huwei@huawei.com>
Subject: Re: [dpdk-stable] [PATCH 00/24] backport for 19.11.4
Date: Tue, 18 Aug 2020 11:25:04 +0800
Message-ID: <ee556f28-3b6a-12f5-5c5a-6350811e66ae@chinasoftinc.com> (raw)
In-Reply-To: <a7d84dc989ea36eb46bce5f027d3b4b33db4e4e7.camel@debian.org>

Hi, Luca Boccassi

On 2020/8/17 21:42, Luca Boccassi wrote:
> On Mon, 2020-08-17 at 19:54 +0800, Wei Hu (Xavier) wrote:
>> Hi, Luca Boccassi
>>
>> On 2020/8/17 17:51, Luca Boccassi wrote:
>>> On Mon, 2020-08-17 at 17:25 +0800, Wei Hu (Xavier) wrote:
>>>> This series are backport for 19.11.4 about hns3 PMD driver, bonding
>>>> and testpmd.
>>>>
>>>> Chengchang Tang (6):
>>>>     net/hns3: remove restriction on setting VF MTU
>>>>     net/hns3: support promiscuous and allmulticast mode for VF
>>>>     net/hns3: fix adding multicast MAC address
>>>>     net/hns3: decrease non-nearby memory access in Rx
>>>>     net/hns3: support setting VF PVID by PF driver
>>>>     net/hns3: cleanup duplicated code on processing TSO in Tx
>>>>
>>>> Chengwen Feng (1):
>>>>     net/hns3: get Rx/Tx queue fbd in xstats
>>>>
>>>> Hao Chen (1):
>>>>     net/hns3: optimize default RSS algorithm
>>>>
>>>> Hongbo Zheng (3):
>>>>     net/hns3: get link status change through mailbox
>>>>     net/hns3: support TSO
>>>>     net/hns3: check TSO segment size during Tx
>>>>
>>>> Lijun Ou (4):
>>>>     net/hns3: remove unnecessary branch
>>>>     net/hns3: support symmetric RSS
>>>>     net/hns3: report Tx descriptor segment limitations
>>>>     app/testpmd: remove hardcoded descriptors limit
>>>>
>>>> Wei Hu (Xavier) (8):
>>>>     net/hns3: support setting VF MAC address by PF driver
>>>>     net/hns3: get Tx abnormal errors in xstats
>>>>     net/hns3: get PCI revision ID
>>>>     net/hns3: support LRO
>>>>     net/hns3: get device capability in primary process
>>>>     net/hns3: support copper media type
>>>>     net/hns3: fix reassembling multiple segment packets in Tx
>>>>     net/hns3: fix inserted VLAN tag position in Tx
>>>>
>>>> Weifeng Li (1):
>>>>     net/bonding: change state machine to defaulted
>>>>
>>>>    app/test-pmd/cmdline.c                        |  11 +-
>>>>    app/test-pmd/testpmd.c                        | 171 ++++++
>>>>    app/test-pmd/testpmd.h                        |   2 +
>>>>    doc/guides/nics/features/hns3.ini             |   2 +
>>>>    doc/guides/nics/features/hns3_vf.ini          |   4 +
>>>>    doc/guides/nics/hns3.rst                      |   2 +
>>>>    drivers/net/bonding/eth_bond_8023ad_private.h |   3 +
>>>>    drivers/net/bonding/rte_eth_bond_8023ad.c     |  21 +-
>>>>    drivers/net/hns3/hns3_ethdev.c                | 269 ++++++--
>>>>    drivers/net/hns3/hns3_ethdev.h                |  34 +-
>>>>    drivers/net/hns3/hns3_ethdev_vf.c             | 578 +++++++++++++++---
>>>>    drivers/net/hns3/hns3_flow.c                  |  12 +-
>>>>    drivers/net/hns3/hns3_mbx.c                   |  93 +++
>>>>    drivers/net/hns3/hns3_mbx.h                   |  13 +
>>>>    drivers/net/hns3/hns3_rss.c                   |  35 +-
>>>>    drivers/net/hns3/hns3_rss.h                   |   6 +-
>>>>    drivers/net/hns3/hns3_rxtx.c                  | 525 ++++++++++++++--
>>>>    drivers/net/hns3/hns3_rxtx.h                  |  65 +-
>>>>    drivers/net/hns3/hns3_stats.c                 | 243 +++++---
>>>>    drivers/net/hns3/hns3_stats.h                 |   3 +-
>>>>    20 files changed, 1779 insertions(+), 313 deletions(-)
>>> Hi,
>>>
>>> If I understand correctly, this series adds support for 6 new features:
>>>
>>> --- a/doc/guides/nics/features/hns3.ini
>>> +++ b/doc/guides/nics/features/hns3.ini
>>> @@ -8,6 +8,8 @@ Link status          = Y
>>>    Rx interrupt         = Y
>>>    MTU update           = Y
>>>    Jumbo frame          = Y
>>> +TSO                  = Y
>>> +LRO                  = Y
>>>    Promiscuous mode     = Y
>>>    Allmulticast mode    = Y
>>>    Unicast MAC filter   = Y
>>>
>>> --- a/doc/guides/nics/features/hns3_vf.ini
>>> +++ b/doc/guides/nics/features/hns3_vf.ini
>>> @@ -8,6 +8,10 @@ Link status          = Y
>>>    Rx interrupt         = Y
>>>    MTU update           = Y
>>>    Jumbo frame          = Y
>>> +TSO                  = Y
>>> +LRO                  = Y
>>> +Promiscuous mode     = Y
>>> +Allmulticast mode    = Y
>>>    Unicast MAC filter   = Y
>>>    Multicast MAC filter = Y
>>>    RSS hash             = Y
>>>
>>> None of those are available in 20.02, and LRO is not available in
>>> 20.05.
>>>
>>> What happens if a user upgrades from 19.11.4 with any of these features
>>> enabled to 20.02 or 20.05?
>> If a user upgrades from 19.11.4 to 20.02 or 20.05 based on hns3 network
>> engine,
>>
>> some features of hns3 will not be supported.  But in many cases
>> customers prefer
>>
>> to use the LTS version. We will recommend them to upgrade from 19.11.0
>> to 19.11.4,
>>
>> and will recommend that customers upgrade from 19.11.4 to 20.11.0 after the
>>
>> release of 20.11.0.
>>
>>
>>       Regards
>>
>> Xavier
> Sorry, but I don't feel too comfortable adding multiple new features to
> the LTS branch, especially as they are not available in the following
> release. Stable is valuable because it is stable. Since we got ABI
> stability now, users that want new features can upgrade to
> 20.02/20.05/20.08 without any application breakages.
>
> Would it be possible to break this series into a smaller bugfix-only
> patchset?

OK, I will send a smaller V2.

Thanks, Xavier


  reply	other threads:[~2020-08-18  3:25 UTC|newest]

Thread overview: 49+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-17  9:25 Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 01/24] net/hns3: get link status change through mailbox Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 02/24] net/hns3: optimize default RSS algorithm Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 03/24] net/hns3: support setting VF MAC address by PF driver Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 04/24] net/hns3: remove unnecessary branch Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 05/24] net/hns3: support TSO Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 06/24] net/hns3: remove restriction on setting VF MTU Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 07/24] net/hns3: support promiscuous and allmulticast mode for VF Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 08/24] net/hns3: fix adding multicast MAC address Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 09/24] net/hns3: get Rx/Tx queue fbd in xstats Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 10/24] net/hns3: get Tx abnormal errors " Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 11/24] net/hns3: get PCI revision ID Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 12/24] net/hns3: check TSO segment size during Tx Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 13/24] net/hns3: support symmetric RSS Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 14/24] net/hns3: support LRO Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 15/24] net/hns3: decrease non-nearby memory access in Rx Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 16/24] net/hns3: support setting VF PVID by PF driver Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 17/24] net/hns3: get device capability in primary process Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 18/24] net/hns3: report Tx descriptor segment limitations Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 19/24] net/hns3: cleanup duplicated code on processing TSO in Tx Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 20/24] net/hns3: support copper media type Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 21/24] net/hns3: fix reassembling multiple segment packets in Tx Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 22/24] net/hns3: fix inserted VLAN tag position " Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 23/24] app/testpmd: remove hardcoded descriptors limit Wei Hu (Xavier)
2020-08-17  9:25 ` [dpdk-stable] [PATCH 24/24] net/bonding: change state machine to defaulted Wei Hu (Xavier)
2020-08-17  9:51 ` [dpdk-stable] [PATCH 00/24] backport for 19.11.4 Luca Boccassi
2020-08-17 11:54   ` Wei Hu (Xavier)
2020-08-17 13:42     ` Luca Boccassi
2020-08-18  3:25       ` Wei Hu (Xavier) [this message]
2020-08-18  6:49 ` [dpdk-stable] [PATCH v2 00/10] " Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 01/10] net/hns3: get link status change through mailbox Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 02/10] net/hns3: optimize default RSS algorithm Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 03/10] net/hns3: remove unnecessary branch Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 04/10] net/hns3: remove restriction on setting VF MTU Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 05/10] net/hns3: fix adding multicast MAC address Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 06/10] net/hns3: get device capability in primary process Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 07/10] net/hns3: report Tx descriptor segment limitations Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 08/10] net/hns3: fix reassembling multiple segment packets in Tx Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 09/10] app/testpmd: remove hardcoded descriptors limit Wei Hu (Xavier)
2020-08-18  6:49   ` [dpdk-stable] [PATCH v2 10/10] net/bonding: change state machine to defaulted Wei Hu (Xavier)
2020-08-18  7:15 ` [dpdk-stable] [PATCH v3 0/7] backport for 19.11.4 Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 1/7] net/hns3: get link status change through mailbox Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 2/7] net/hns3: optimize default RSS algorithm Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 3/7] net/hns3: remove unnecessary branch Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 4/7] net/hns3: remove restriction on setting VF MTU Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 5/7] net/hns3: fix adding multicast MAC address Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 6/7] app/testpmd: remove hardcoded descriptors limit Wei Hu (Xavier)
2020-08-18  7:15   ` [dpdk-stable] [PATCH v3 7/7] net/bonding: change state machine to defaulted Wei Hu (Xavier)
2020-08-18 18:00   ` [dpdk-stable] [PATCH v3 0/7] backport for 19.11.4 Luca Boccassi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ee556f28-3b6a-12f5-5c5a-6350811e66ae@chinasoftinc.com \
    --to=huwei013@chinasoftinc.com \
    --cc=bluca@debian.org \
    --cc=stable@dpdk.org \
    --cc=xavier.huwei@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

patches for DPDK stable branches

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/stable/0 stable/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 stable stable/ https://inbox.dpdk.org/stable \
		stable@dpdk.org
	public-inbox-index stable

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.stable


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git