Message ID | 20210122191925.24308-1-lironh@marvell.com (mailing list archive) |
---|---|
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 87067A0A0A; Fri, 22 Jan 2021 20:19:52 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 44E43141079; Fri, 22 Jan 2021 20:19:52 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id B1703141078 for <dev@dpdk.org>; Fri, 22 Jan 2021 20:19:50 +0100 (CET) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 10MJCPpr019092 for <dev@dpdk.org>; Fri, 22 Jan 2021 11:19:50 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding : content-type; s=pfpt0220; bh=gi9JZ+q3HmrGbRyf6gSiSUq5wBM8RJouK2fMisjpLs4=; b=HaPHxpCLcOWiOs5ylAjMyduUHtoMYpgty80N37IqjmLEVL1h/sUIoBwxJOPUXKz5JkiJ 545KXFF84U7gtizNEOWefa0HQk01wD/0DIZqiDUfdVZxW3yMr6ycXmP16T8DXwW6ljFn O7VkzdFkMzsLeLn+GtWxIuEzY25efJ5wJRaMCTc+5A31rmiwRqaFmEXfJghFcm+6NDKX w8V74EHoay1PE1Cb6FuxbyBpOY67NQvg1NISPwyDY41P0jJxSq1uw3vJjF4ggHm8Sqqq sOSzkXKkRgpv4V0wrBSjv/KC5xugfAbDtKOJRU25KnSpvgt1+XbR9O1FIxb2bDyMQHNs 3g== Received: from dc5-exch02.marvell.com ([199.233.59.182]) by mx0b-0016f401.pphosted.com with ESMTP id 3668p7tpdg-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for <dev@dpdk.org>; Fri, 22 Jan 2021 11:19:50 -0800 Received: from SC-EXCH01.marvell.com (10.93.176.81) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Fri, 22 Jan 2021 11:19:48 -0800 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by SC-EXCH01.marvell.com (10.93.176.81) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Fri, 22 Jan 2021 11:19:47 -0800 Received: from pt-lxl0023.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Fri, 22 Jan 2021 11:19:46 -0800 From: <lironh@marvell.com> To: <jerinj@marvell.com> CC: <dev@dpdk.org>, Liron Himi <lironh@marvell.com> Date: Fri, 22 Jan 2021 21:18:48 +0200 Message-ID: <20210122191925.24308-1-lironh@marvell.com> X-Mailer: git-send-email 2.28.0 In-Reply-To: <20201202101212.4717-1-lironh@marvell.com> References: <20201202101212.4717-1-lironh@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.343, 18.0.737 definitions=2021-01-22_14:2021-01-22, 2021-01-22 signatures=0 Subject: [dpdk-dev] [PATCH v2 00/37] net/mvpp2: misc updates X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
net/mvpp2: misc updates
|
|
Message
Liron Himi
Jan. 22, 2021, 7:18 p.m. UTC
From: Liron Himi <lironh@marvell.com>
This patch series align the mainline driver with all changes since 19.11
some of the patches are fixes which should be pushed to stable
v2:
- fix commit msg styling
- Addressed various review comments
Dana Vardi (5):
net/mvpp2: update start hdr name
net/mvpp2: support custom header before ethernet
net/mvpp2: forward bad packets support
net/mvpp2: update qos defaults parameter name
net/mvpp2: add fill buffs to configuration file
Liron Himi (20):
net/mvpp2: remove debug log on fast-path
net/mvpp2: skip VLAN flush
net/mvpp2: remove CRC len from MRU validation
net/mvpp2: fix frame size checking
net/mvpp2: update Tx checksum
net/mvpp2: add dsa mode support
net/mvpp2: adjust the number of unicast address
net/mvpp2: use generic name for the 'cfg'
net/mvpp2: flow: support generic pattern combinations
net/mvpp2: flow: build table key along with rule
net/mvpp2: move common functions to common location
net/mvpp2: flow: add support for RAW type
net/mvpp2: skip qos init if not requested
net/mvpp2: support udf configuration
net/mvpp2: rearrange functions order
net/mvpp2: introduce fixup for fifo overrun
net/mvpp2: propagate port-id in udata64
net/mvpp2: expose max MTU size
net/mvpp2: apply flow-ctrl after port init
net/mvpp2: consider ptype in cksum info
Meir Levi (1):
net/mvpp2: add 2.5G LINK info
Yuri Chipchev (11):
net/mvpp2: fix stack corruption
net/mvpp2: fix Rx/Tx bytes statistics
net/mvpp2: update RSS tables reservation
net/mvpp2: extend xstats support
net/mvpp2: cosmetic changes to cookie usage
net/mvpp2: align checking order
net/mvpp2: save initial configuration
net/mvpp2: add loopback support
net/mvpp2: add VLAN offload support
net/mvpp2: add Tx flow control
net/mvpp2: autoneg disable handling
doc/guides/nics/mvpp2.rst | 1 +
drivers/net/mvpp2/mrvl_ethdev.c | 811 +++++++++++-----
drivers/net/mvpp2/mrvl_ethdev.h | 84 +-
drivers/net/mvpp2/mrvl_flow.c | 1522 +++++++------------------------
drivers/net/mvpp2/mrvl_qos.c | 400 ++++++--
drivers/net/mvpp2/mrvl_qos.h | 21 +-
6 files changed, 1341 insertions(+), 1498 deletions(-)
Comments
On Sat, Jan 23, 2021 at 12:49 AM <lironh@marvell.com> wrote: > > From: Liron Himi <lironh@marvell.com> > > This patch series align the mainline driver with all changes since 19.11 > some of the patches are fixes which should be pushed to stable > > v2: > - fix commit msg styling > - Addressed various review comments Series applied to dpdk-next-net-mrvl/for-main. Thanks. > > Dana Vardi (5): > net/mvpp2: update start hdr name > net/mvpp2: support custom header before ethernet > net/mvpp2: forward bad packets support > net/mvpp2: update qos defaults parameter name > net/mvpp2: add fill buffs to configuration file > > Liron Himi (20): > net/mvpp2: remove debug log on fast-path > net/mvpp2: skip VLAN flush > net/mvpp2: remove CRC len from MRU validation > net/mvpp2: fix frame size checking > net/mvpp2: update Tx checksum > net/mvpp2: add dsa mode support > net/mvpp2: adjust the number of unicast address > net/mvpp2: use generic name for the 'cfg' > net/mvpp2: flow: support generic pattern combinations > net/mvpp2: flow: build table key along with rule > net/mvpp2: move common functions to common location > net/mvpp2: flow: add support for RAW type > net/mvpp2: skip qos init if not requested > net/mvpp2: support udf configuration > net/mvpp2: rearrange functions order > net/mvpp2: introduce fixup for fifo overrun > net/mvpp2: propagate port-id in udata64 > net/mvpp2: expose max MTU size > net/mvpp2: apply flow-ctrl after port init > net/mvpp2: consider ptype in cksum info > > Meir Levi (1): > net/mvpp2: add 2.5G LINK info > > Yuri Chipchev (11): > net/mvpp2: fix stack corruption > net/mvpp2: fix Rx/Tx bytes statistics > net/mvpp2: update RSS tables reservation > net/mvpp2: extend xstats support > net/mvpp2: cosmetic changes to cookie usage > net/mvpp2: align checking order > net/mvpp2: save initial configuration > net/mvpp2: add loopback support > net/mvpp2: add VLAN offload support > net/mvpp2: add Tx flow control > net/mvpp2: autoneg disable handling > > doc/guides/nics/mvpp2.rst | 1 + > drivers/net/mvpp2/mrvl_ethdev.c | 811 +++++++++++----- > drivers/net/mvpp2/mrvl_ethdev.h | 84 +- > drivers/net/mvpp2/mrvl_flow.c | 1522 +++++++------------------------ > drivers/net/mvpp2/mrvl_qos.c | 400 ++++++-- > drivers/net/mvpp2/mrvl_qos.h | 21 +- > 6 files changed, 1341 insertions(+), 1498 deletions(-) > > -- > 2.28.0 >
On 1/25/2021 6:00 PM, Jerin Jacob wrote: > On Sat, Jan 23, 2021 at 12:49 AM <lironh@marvell.com> wrote: >> >> From: Liron Himi <lironh@marvell.com> >> >> This patch series align the mainline driver with all changes since 19.11 >> some of the patches are fixes which should be pushed to stable >> >> v2: >> - fix commit msg styling >> - Addressed various review comments > > > Series applied to dpdk-next-net-mrvl/for-main. Thanks. > There were multiple review tag from "Michael Shamis <michaelsh@marvell.com>" to the first version, tags seems not carried to v2, should we add them to v2? > >> >> Dana Vardi (5): >> net/mvpp2: update start hdr name >> net/mvpp2: support custom header before ethernet >> net/mvpp2: forward bad packets support >> net/mvpp2: update qos defaults parameter name >> net/mvpp2: add fill buffs to configuration file >> >> Liron Himi (20): >> net/mvpp2: remove debug log on fast-path >> net/mvpp2: skip VLAN flush >> net/mvpp2: remove CRC len from MRU validation >> net/mvpp2: fix frame size checking >> net/mvpp2: update Tx checksum >> net/mvpp2: add dsa mode support >> net/mvpp2: adjust the number of unicast address >> net/mvpp2: use generic name for the 'cfg' >> net/mvpp2: flow: support generic pattern combinations >> net/mvpp2: flow: build table key along with rule >> net/mvpp2: move common functions to common location >> net/mvpp2: flow: add support for RAW type >> net/mvpp2: skip qos init if not requested >> net/mvpp2: support udf configuration >> net/mvpp2: rearrange functions order >> net/mvpp2: introduce fixup for fifo overrun >> net/mvpp2: propagate port-id in udata64 >> net/mvpp2: expose max MTU size >> net/mvpp2: apply flow-ctrl after port init >> net/mvpp2: consider ptype in cksum info >> >> Meir Levi (1): >> net/mvpp2: add 2.5G LINK info >> >> Yuri Chipchev (11): >> net/mvpp2: fix stack corruption >> net/mvpp2: fix Rx/Tx bytes statistics >> net/mvpp2: update RSS tables reservation >> net/mvpp2: extend xstats support >> net/mvpp2: cosmetic changes to cookie usage >> net/mvpp2: align checking order >> net/mvpp2: save initial configuration >> net/mvpp2: add loopback support >> net/mvpp2: add VLAN offload support >> net/mvpp2: add Tx flow control >> net/mvpp2: autoneg disable handling >> >> doc/guides/nics/mvpp2.rst | 1 + >> drivers/net/mvpp2/mrvl_ethdev.c | 811 +++++++++++----- >> drivers/net/mvpp2/mrvl_ethdev.h | 84 +- >> drivers/net/mvpp2/mrvl_flow.c | 1522 +++++++------------------------ >> drivers/net/mvpp2/mrvl_qos.c | 400 ++++++-- >> drivers/net/mvpp2/mrvl_qos.h | 21 +- >> 6 files changed, 1341 insertions(+), 1498 deletions(-) >> >> -- >> 2.28.0 >>
-----Original Message----- From: Ferruh Yigit <ferruh.yigit@intel.com> Sent: Tuesday, 26 January 2021 18:59 To: Jerin Jacob <jerinjacobk@gmail.com>; Liron Himi <lironh@marvell.com> Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; dpdk-dev <dev@dpdk.org>; Michael Shamis <michaelsh@marvell.com> Subject: [EXT] Re: [dpdk-dev] [PATCH v2 00/37] net/mvpp2: misc updates External Email ---------------------------------------------------------------------- On 1/25/2021 6:00 PM, Jerin Jacob wrote: > On Sat, Jan 23, 2021 at 12:49 AM <lironh@marvell.com> wrote: >> >> From: Liron Himi <lironh@marvell.com> >> >> This patch series align the mainline driver with all changes since >> 19.11 some of the patches are fixes which should be pushed to stable >> >> v2: >> - fix commit msg styling >> - Addressed various review comments > > > Series applied to dpdk-next-net-mrvl/for-main. Thanks. > There were multiple review tag from "Michael Shamis <michaelsh@marvell.com>" to the first version, tags seems not carried to v2, should we add them to v2? [L.H.] is this done automatically? If yes, lets add them. > >> >> Dana Vardi (5): >> net/mvpp2: update start hdr name >> net/mvpp2: support custom header before ethernet >> net/mvpp2: forward bad packets support >> net/mvpp2: update qos defaults parameter name >> net/mvpp2: add fill buffs to configuration file >> >> Liron Himi (20): >> net/mvpp2: remove debug log on fast-path >> net/mvpp2: skip VLAN flush >> net/mvpp2: remove CRC len from MRU validation >> net/mvpp2: fix frame size checking >> net/mvpp2: update Tx checksum >> net/mvpp2: add dsa mode support >> net/mvpp2: adjust the number of unicast address >> net/mvpp2: use generic name for the 'cfg' >> net/mvpp2: flow: support generic pattern combinations >> net/mvpp2: flow: build table key along with rule >> net/mvpp2: move common functions to common location >> net/mvpp2: flow: add support for RAW type >> net/mvpp2: skip qos init if not requested >> net/mvpp2: support udf configuration >> net/mvpp2: rearrange functions order >> net/mvpp2: introduce fixup for fifo overrun >> net/mvpp2: propagate port-id in udata64 >> net/mvpp2: expose max MTU size >> net/mvpp2: apply flow-ctrl after port init >> net/mvpp2: consider ptype in cksum info >> >> Meir Levi (1): >> net/mvpp2: add 2.5G LINK info >> >> Yuri Chipchev (11): >> net/mvpp2: fix stack corruption >> net/mvpp2: fix Rx/Tx bytes statistics >> net/mvpp2: update RSS tables reservation >> net/mvpp2: extend xstats support >> net/mvpp2: cosmetic changes to cookie usage >> net/mvpp2: align checking order >> net/mvpp2: save initial configuration >> net/mvpp2: add loopback support >> net/mvpp2: add VLAN offload support >> net/mvpp2: add Tx flow control >> net/mvpp2: autoneg disable handling >> >> doc/guides/nics/mvpp2.rst | 1 + >> drivers/net/mvpp2/mrvl_ethdev.c | 811 +++++++++++----- >> drivers/net/mvpp2/mrvl_ethdev.h | 84 +- >> drivers/net/mvpp2/mrvl_flow.c | 1522 +++++++------------------------ >> drivers/net/mvpp2/mrvl_qos.c | 400 ++++++-- >> drivers/net/mvpp2/mrvl_qos.h | 21 +- >> 6 files changed, 1341 insertions(+), 1498 deletions(-) >> >> -- >> 2.28.0 >>
On 1/26/2021 6:07 PM, Liron Himi wrote: > > -----Original Message----- > From: Ferruh Yigit <ferruh.yigit@intel.com> > Sent: Tuesday, 26 January 2021 18:59 > To: Jerin Jacob <jerinjacobk@gmail.com>; Liron Himi <lironh@marvell.com> > Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; dpdk-dev <dev@dpdk.org>; Michael Shamis <michaelsh@marvell.com> > Subject: [EXT] Re: [dpdk-dev] [PATCH v2 00/37] net/mvpp2: misc updates > > External Email > > ---------------------------------------------------------------------- > On 1/25/2021 6:00 PM, Jerin Jacob wrote: >> On Sat, Jan 23, 2021 at 12:49 AM <lironh@marvell.com> wrote: >>> >>> From: Liron Himi <lironh@marvell.com> >>> >>> This patch series align the mainline driver with all changes since >>> 19.11 some of the patches are fixes which should be pushed to stable >>> >>> v2: >>> - fix commit msg styling >>> - Addressed various review comments >> >> >> Series applied to dpdk-next-net-mrvl/for-main. Thanks. >> > > There were multiple review tag from "Michael Shamis <michaelsh@marvell.com>" to the first version, tags seems not carried to v2, should we add them to v2? > [L.H.] is this done automatically? If yes, lets add them. > No, not manually unfortunately, but I can do it if this is requested. >> >>> >>> Dana Vardi (5): >>> net/mvpp2: update start hdr name >>> net/mvpp2: support custom header before ethernet >>> net/mvpp2: forward bad packets support >>> net/mvpp2: update qos defaults parameter name >>> net/mvpp2: add fill buffs to configuration file >>> >>> Liron Himi (20): >>> net/mvpp2: remove debug log on fast-path >>> net/mvpp2: skip VLAN flush >>> net/mvpp2: remove CRC len from MRU validation >>> net/mvpp2: fix frame size checking >>> net/mvpp2: update Tx checksum >>> net/mvpp2: add dsa mode support >>> net/mvpp2: adjust the number of unicast address >>> net/mvpp2: use generic name for the 'cfg' >>> net/mvpp2: flow: support generic pattern combinations >>> net/mvpp2: flow: build table key along with rule >>> net/mvpp2: move common functions to common location >>> net/mvpp2: flow: add support for RAW type >>> net/mvpp2: skip qos init if not requested >>> net/mvpp2: support udf configuration >>> net/mvpp2: rearrange functions order >>> net/mvpp2: introduce fixup for fifo overrun >>> net/mvpp2: propagate port-id in udata64 >>> net/mvpp2: expose max MTU size >>> net/mvpp2: apply flow-ctrl after port init >>> net/mvpp2: consider ptype in cksum info >>> >>> Meir Levi (1): >>> net/mvpp2: add 2.5G LINK info >>> >>> Yuri Chipchev (11): >>> net/mvpp2: fix stack corruption >>> net/mvpp2: fix Rx/Tx bytes statistics >>> net/mvpp2: update RSS tables reservation >>> net/mvpp2: extend xstats support >>> net/mvpp2: cosmetic changes to cookie usage >>> net/mvpp2: align checking order >>> net/mvpp2: save initial configuration >>> net/mvpp2: add loopback support >>> net/mvpp2: add VLAN offload support >>> net/mvpp2: add Tx flow control >>> net/mvpp2: autoneg disable handling >>> >>> doc/guides/nics/mvpp2.rst | 1 + >>> drivers/net/mvpp2/mrvl_ethdev.c | 811 +++++++++++----- >>> drivers/net/mvpp2/mrvl_ethdev.h | 84 +- >>> drivers/net/mvpp2/mrvl_flow.c | 1522 +++++++------------------------ >>> drivers/net/mvpp2/mrvl_qos.c | 400 ++++++-- >>> drivers/net/mvpp2/mrvl_qos.h | 21 +- >>> 6 files changed, 1341 insertions(+), 1498 deletions(-) >>> >>> -- >>> 2.28.0 >>> >
On 1/25/2021 6:00 PM, Jerin Jacob wrote: > On Sat, Jan 23, 2021 at 12:49 AM <lironh@marvell.com> wrote: >> >> From: Liron Himi <lironh@marvell.com> >> >> This patch series align the mainline driver with all changes since 19.11 >> some of the patches are fixes which should be pushed to stable >> >> v2: >> - fix commit msg styling >> - Addressed various review comments > > > Series applied to dpdk-next-net-mrvl/for-main. Thanks. > > I tried to partially get the set, but the refactoring and the function rearrangements in the middle is not helping. Specially I tried to remove all config file patches because of missing documentation, but those also spread in the set. It would be safer to get a new version of the set. Can you please send a new version of the set with updates? Thanks, ferruh > >> >> Dana Vardi (5): >> net/mvpp2: update start hdr name >> net/mvpp2: support custom header before ethernet >> net/mvpp2: forward bad packets support >> net/mvpp2: update qos defaults parameter name >> net/mvpp2: add fill buffs to configuration file >> >> Liron Himi (20): >> net/mvpp2: remove debug log on fast-path >> net/mvpp2: skip VLAN flush >> net/mvpp2: remove CRC len from MRU validation >> net/mvpp2: fix frame size checking >> net/mvpp2: update Tx checksum >> net/mvpp2: add dsa mode support >> net/mvpp2: adjust the number of unicast address >> net/mvpp2: use generic name for the 'cfg' >> net/mvpp2: flow: support generic pattern combinations >> net/mvpp2: flow: build table key along with rule >> net/mvpp2: move common functions to common location >> net/mvpp2: flow: add support for RAW type >> net/mvpp2: skip qos init if not requested >> net/mvpp2: support udf configuration >> net/mvpp2: rearrange functions order >> net/mvpp2: introduce fixup for fifo overrun >> net/mvpp2: propagate port-id in udata64 >> net/mvpp2: expose max MTU size >> net/mvpp2: apply flow-ctrl after port init >> net/mvpp2: consider ptype in cksum info >> >> Meir Levi (1): >> net/mvpp2: add 2.5G LINK info >> >> Yuri Chipchev (11): >> net/mvpp2: fix stack corruption >> net/mvpp2: fix Rx/Tx bytes statistics >> net/mvpp2: update RSS tables reservation >> net/mvpp2: extend xstats support >> net/mvpp2: cosmetic changes to cookie usage >> net/mvpp2: align checking order >> net/mvpp2: save initial configuration >> net/mvpp2: add loopback support >> net/mvpp2: add VLAN offload support >> net/mvpp2: add Tx flow control >> net/mvpp2: autoneg disable handling >> >> doc/guides/nics/mvpp2.rst | 1 + >> drivers/net/mvpp2/mrvl_ethdev.c | 811 +++++++++++----- >> drivers/net/mvpp2/mrvl_ethdev.h | 84 +- >> drivers/net/mvpp2/mrvl_flow.c | 1522 +++++++------------------------ >> drivers/net/mvpp2/mrvl_qos.c | 400 ++++++-- >> drivers/net/mvpp2/mrvl_qos.h | 21 +- >> 6 files changed, 1341 insertions(+), 1498 deletions(-) >> >> -- >> 2.28.0 >>