From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <dev-bounces@dpdk.org> Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6D7E7A0540; Mon, 20 Jul 2020 06:50:59 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 487AB2B9A; Mon, 20 Jul 2020 06:50:59 +0200 (CEST) Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70084.outbound.protection.outlook.com [40.107.7.84]) by dpdk.org (Postfix) with ESMTP id 47D401023 for <dev@dpdk.org>; Mon, 20 Jul 2020 06:50:58 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SuOAKYjUOsaFkjy6sDZHg/NJyryi4DRKXGsKYOZLWTsEGeAaZiIcI+f/4sfDanT5HJk/m/Zfw+a6pXw4Otj+QJGsacogkfI1rFGQzKXZ6EjtBd/k8IoZiJKrLocOlRi+wPQv8v+YeL/hivTB7NJlr5rHT5o/fHvuRs6KIyg5vl5JXPTZAbwofSNLyikFRAX4j/10XfBewewyOdBeMCdNmgyHjtIv3PH2UtxeMTnCSCHrrSmg0u5Ahnu3JwqLKbRgzyLtslQKaWx7f07qi7MWBP5iog8HiXAl93XhFeuc1p36fXlQK35M6b3VOUqjuoQE5icmWDX9OZ4EMBCgFhmJgw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=R1VByeLv3aADq5CmsRVKmrj8/WwjDpvLSE62lRvW/Qg=; b=KCjv0a+3VnaCa8g4ovU6UxH4R3QD4essZ5qIbui/8ly2xUKQ/bNDuLxXZAMO8Vl7TmU0BWZdiAGnFkJHntMpFB/iEUhdRaAwALsLfC+8mPZfnugNCdc8v1mm6GcQA9iEpdRKqgBgONJNlEwJ2rGHDxDaNZwa7cN1Y55Ex9mqyHSPlbI4HnyqaKUw1M9EhIb0n2t7mzAqaInFVXB6O1IgGF2K4JolqC/cuH9052qMq2Y18qHAzNs3sjI/Sl5SISb11cQMQ+l3buopBE1yUSYGBiHbvoSbFg2ElbLZrXE4DEHOC20sf2RqDwA92l5XcqIdoSHePdQ9TeTTBLazmArY5Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=mellanox.com; dmarc=pass action=none header.from=mellanox.com; dkim=pass header.d=mellanox.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Mellanox.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=R1VByeLv3aADq5CmsRVKmrj8/WwjDpvLSE62lRvW/Qg=; b=oPLiHgtRpQuqsUmMm4as5NIw917MRsFnBs+F6yifX16fJEBs3XmHQ1t27dC2dYsPqumtacyP2SoNit5bO1mr7ifIk9henAZsyfNZjhynubIinYqLRRPYhb0ULfl/8827Ic2Un0wwxJqKwwPRC3Xylhm/sFPtMslhn4+26Y5ULK4= Received: from AM6PR05MB5176.eurprd05.prod.outlook.com (2603:10a6:20b:63::30) by AM7PR05MB6946.eurprd05.prod.outlook.com (2603:10a6:20b:1a3::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3195.23; Mon, 20 Jul 2020 04:50:56 +0000 Received: from AM6PR05MB5176.eurprd05.prod.outlook.com ([fe80::9024:accb:4d6b:805f]) by AM6PR05MB5176.eurprd05.prod.outlook.com ([fe80::9024:accb:4d6b:805f%3]) with mapi id 15.20.3195.025; Mon, 20 Jul 2020 04:50:56 +0000 From: Ori Kam <orika@mellanox.com> To: Thomas Monjalon <thomas@monjalon.net> CC: "jerinj@marvell.com" <jerinj@marvell.com>, "xiang.w.wang@intel.com" <xiang.w.wang@intel.com>, Matan Azrad <matan@mellanox.com>, Slava Ovsiienko <viacheslavo@mellanox.com>, John McNamara <john.mcnamara@intel.com>, Marko Kovacevic <marko.kovacevic@intel.com>, Parav Pandit <parav@mellaox.com>, Shahaf Shuler <shahafs@mellanox.com>, Ray Kinsella <mdr@ashroe.eu>, Neil Horman <nhorman@tuxdriver.com>, "guyk@marvell.com" <guyk@marvell.com>, "dev@dpdk.org" <dev@dpdk.org>, "pbhagavatula@marvell.com" <pbhagavatula@marvell.com>, "hemant.agrawal@nxp.com" <hemant.agrawal@nxp.com>, Opher Reviv <opher@mellanox.com>, Alex Rosenbaum <alexr@mellanox.com>, "dovrat@marvell.com" <dovrat@marvell.com>, "pkapoor@marvell.com" <pkapoor@marvell.com>, "nipun.gupta@nxp.com" <nipun.gupta@nxp.com>, "bruce.richardson@intel.com" <bruce.richardson@intel.com>, "yang.a.hong@intel.com" <yang.a.hong@intel.com>, "harry.chang@intel.com" <harry.chang@intel.com>, "gu.jian1@zte.com.cn" <gu.jian1@zte.com.cn>, "shanjiangh@chinatelecom.cn" <shanjiangh@chinatelecom.cn>, "zhangy.yun@chinatelecom.cn" <zhangy.yun@chinatelecom.cn>, "lixingfu@huachentel.com" <lixingfu@huachentel.com>, "wushuai@inspur.com" <wushuai@inspur.com>, "yuyingxia@yxlink.com" <yuyingxia@yxlink.com>, "fanchenggang@sunyainfo.com" <fanchenggang@sunyainfo.com>, "davidfgao@tencent.com" <davidfgao@tencent.com>, "liuzhong1@chinaunicom.cn" <liuzhong1@chinaunicom.cn>, "zhaoyong11@huawei.com" <zhaoyong11@huawei.com>, "oc@yunify.com" <oc@yunify.com>, "jim@netgate.com" <jim@netgate.com>, "hongjun.ni@intel.com" <hongjun.ni@intel.com>, "deri@ntop.org" <deri@ntop.org>, "fc@napatech.com" <fc@napatech.com>, "arthur.su@lionic.com" <arthur.su@lionic.com>, Raslan Darawsheh <rasland@mellanox.com>, Yuval Avnery <yuvalav@mellanox.com> Thread-Topic: [PATCH v5 01/13] regex/mlx5: add RegEx PMD layer and mlx5 driver Thread-Index: AQHWXffEpkLK/Dr6WEajWEM6t4jo0akPeGeAgABo4yA= Date: Mon, 20 Jul 2020 04:50:56 +0000 Message-ID: <AM6PR05MB517624467C341092A04F2EDFDB7B0@AM6PR05MB5176.eurprd05.prod.outlook.com> References: <1593941027-86651-1-git-send-email-orika@mellanox.com> <1595182154-36508-1-git-send-email-orika@mellanox.com> <1595182154-36508-2-git-send-email-orika@mellanox.com> <3058241.yiR4NS12MS@thomas> In-Reply-To: <3058241.yiR4NS12MS@thomas> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: monjalon.net; dkim=none (message not signed) header.d=none;monjalon.net; dmarc=none action=none header.from=mellanox.com; x-originating-ip: [147.236.152.129] x-ms-publictraffictype: Email x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: 2aeb9b9f-e01d-4ab2-e792-08d82c687d8a x-ms-traffictypediagnostic: AM7PR05MB6946: x-ld-processed: a652971c-7d2e-4d9b-a6a4-d149256f461b,ExtFwd,ExtAddr x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: <AM7PR05MB694601A050286E39ECC8C4FCDB7B0@AM7PR05MB6946.eurprd05.prod.outlook.com> x-ms-oob-tlc-oobclassifiers: OLM:10000; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: BWxZyPEyqcelzs3tSpzTuh2r7vsYkldr76cdIrgp5yocbfwhmd0BHdlRBL+hR7aY94zKoQ/foiWQ6/i1ZrAJOIUZLx+eW1wvh3QuK+wsD8uPJenf/bQ1RKolwNAetla1SpLH4EqdXEfUtz+mQbDABZDNfP4y2t2xoPHVAdX3jihVjFO5r1jZxwdGfhCqSSQg8uzSIohAIL+RUUZeNLlJ/mxe2HoEDF+B+cSeWUk4PI8V4dJwjW2NQMZcsINLVurXbLE+tjKXN+uDp/7bmCdlM9HMA5TDQQWLULFUiuJP+SR1SSQsZ5sQrhZKeo5HZU4+7xPUZMIaLeYnyKfMAvSpzg== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM6PR05MB5176.eurprd05.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(39860400002)(346002)(136003)(376002)(396003)(366004)(107886003)(8676002)(6916009)(7696005)(83380400001)(71200400001)(66946007)(66476007)(478600001)(2906002)(7406005)(7416002)(8936002)(316002)(55016002)(6506007)(186003)(86362001)(5660300002)(52536014)(66446008)(64756008)(66556008)(4326008)(26005)(9686003)(33656002)(76116006)(54906003); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata: sYQ3VDL8Es+xPOvwkrBo+cwOhddLMKlgvjWRs9JI95s2iSRpHhy1qHm9nkRDsncq/o8pFMEqJ9j8CQf6uZAEs+yeoe4AElyXX7IOiFtkX4NT77KJyeQ2Ntks2TkW4pHsgNK5Ne4sbXoVFa6hU8/L0LvsXwLfHED4xuIrRITxoFuU2g79WvPlUy8QIbUZCkpRzGKO7295zUFQl6vRvALhrgwdDleOQIyW7OjjYaRZHDTiJtCbfW1rJK0oPULPeXVd0vgrJ0gXoyLOBUv9z79wR6WE5CSmElBJjA0oh4nVaQ3OBkT8O0VhpMjLVyyBi6Qcqk4t9hFktqP5/pkb8jnT7OeQMTbd1ZKNmeBZ9vudlZu8WjdORlkovyU8o9+YE8Cp0VaOuIi+rymGjTXwib0Ng06DmFpvIpTvI6Kp9l+IP4o2Ok0MmC9hMqCXyUtEQv02HME9nYw9pcCQ5/+h/bgIXKxu2rbdtIB1Kcr0848w6dlc2oBjLbzF8ie9TFRMYA5p Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: AM6PR05MB5176.eurprd05.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2aeb9b9f-e01d-4ab2-e792-08d82c687d8a X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Jul 2020 04:50:56.4483 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: g9RBXy3LofzxhqDiCA2mfWx7VdddDXnZD1M6CT7hZULlIc2dhOR/MeT51vrERJIMfTilnHOIa/K5fuOfRcZDOQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR05MB6946 Subject: Re: [dpdk-dev] [PATCH v5 01/13] regex/mlx5: add RegEx PMD layer and mlx5 driver X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 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> Hi Thomas, Thanks for the review. > -----Original Message----- > From: Thomas Monjalon <thomas@monjalon.net> >=20 > 19/07/2020 20:09, Ori Kam: > > From: Yuval Avnery <yuvalav@mellanox.com> > > > > This commit introduce the RegEx pull mode drivers class, and > > adds Mellanox RegEx PMD. > [...] > > --- /dev/null > > +++ b/doc/guides/regexdevs/features_overview.rst > > @@ -0,0 +1,100 @@ > > +.. SPDX-License-Identifier: BSD-3-Clause > > + Copyright 2020 Mellanox Technologies, Ltd >=20 > SPDX is not aligned with the Copyright. > You should not have a double space before SPDX. >=20 O.K. > [...] > > --- /dev/null > > +++ b/doc/guides/regexdevs/index.rst > > @@ -0,0 +1,15 @@ > > +.. SPDX-License-Identifier: BSD-3-Clause > > + Copyright 2020 Mellanox Technologies, Ltd >=20 > Same here. RST annotations start at the third column. >=20 O.K > > + > > +REGEX Device Drivers > > +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > + > > +The following are a list of RegEx (Regular Expression) device drivers, > > +which can be used from an application through RegEx API. > > + > > +.. toctree:: > > + :maxdepth: 2 > > + :numbered: >=20 > same here >=20 O.K. > [...] > > + Due to external dependencies, this driver is disabled in default > > + configuration of the "make" build. It can be enabled with > > + ``CONFIG_RTE_LIBRTE_MLX5_REGEX_PMD=3Dy`` or by using "meson" build > system which > > + will detect dependencies. >=20 > I would drop this part. I don't think it is an useful info, > especially because make is going to be removed. >=20 I think it is a good comment at least for now, since the RegEx is going to = be used by costumers that are not used to Mellanox devices. But if you wish I will remove. > > +Mellanox mlx5 PCI device can be probed by number of different pci devi= ces, > for example >=20 > Inconsistency on the same line: "pci" vs "PCI". > Will fix. =20 > Better to break line after the comma and after a dot. >=20 Will fix > > +net / vDPA / RegEx. To select the RegEx PMD ``class=3Dregex`` should b= e > specified > > +as device parameter. >=20 > This does not explain if RegEx is exclusive with other classes. >=20 Will clarify. > [...] > > +- BlueField 2 running Mellonx supported kernel. >=20 > typo: Mellonx >=20 Will fix. > > +- Enable the RegEx caps using system call from the BlueField 2. >=20 > This is a doc. Please write full words: "capabilities". > Will fix. =20 > [...] > > +These options can be modified in the ``.config`` file. > > + > > +- ``CONFIG_RTE_LIBRTE_MLX5_REGEX_PMD`` (default **n**) > > + > > + Toggle compilation of librte_pmd_mlx5 itself. >=20 > Better to drop this "make" part. > [...] O.K. but Make is still a build system, that is used, but I will remove. > > enum mlx5_class { > > MLX5_CLASS_INVALID, > > MLX5_CLASS_NET =3D RTE_BIT64(0), > > + MLX5_CLASS_REGEX =3D RTE_BIT64(2), > > MLX5_CLASS_VDPA =3D RTE_BIT64(1), >=20 > I think regex would be better sorted after vDPA. >=20 I did this to have alphabetic ordering. > [...] > > +++ b/drivers/regex/meson.build > > @@ -0,0 +1,9 @@ > > +# SPDX-License-Identifier: BSD-3-Clause > > +# Copyright 2020 Mellanox Technologies, Ltd > > + > > +drivers =3D ['mlx5'] > > +std_deps =3D ['ethdev', 'kvargs'] # 'ethdev' also pulls in mbuf, net, = eal etc > > +std_deps +=3D ['bus_pci'] # very many PMDs depend on PCI, so m= ake std > > +std_deps +=3D ['bus_vdev'] # same with vdev bus >=20 > I disagree about making bus some standard deps. O.K will remove >=20 >=20