Message ID | 20231103062606.2632012-1-chaoyong.he@corigine.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 46F5C43275; Fri, 3 Nov 2023 07:26:32 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 10742402D7; Fri, 3 Nov 2023 07:26:32 +0100 (CET) Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2136.outbound.protection.outlook.com [40.107.237.136]) by mails.dpdk.org (Postfix) with ESMTP id 4D5B040265 for <dev@dpdk.org>; Fri, 3 Nov 2023 07:26:31 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cPQKzj7/jkvUGXmx9XDrgEuSaZ2cq0itO1QbG4T8td7J0YSEwk/915Rc+ej0I7kpTCA3wzZ/t58olkeYVyGo9YFHz5aMRx6SHp5HgP0lP/FqnHQSydy943NXq/0FbdtygwDTud8XhXjsMtJCYGbERQCSyW7iZqx8GGjk2s20HUemgJHKb42iV9ZWFFAXscPjWTxeDtoV/f3l1RmrZrHPUunZIQKswVNn186j9jKvdrOzPjDmehTU5KsfGX8rWB1RXL260uco8lyBtoSoDG5fs2qzeSOyto2QRF9qXFqPeqnD39KMeCsEMxsbCwE15ChEIRjWyCZ6f8ylMpzxCryoOQ== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=dGPfxLL3fb5oc7kJJVgJG80Psptncp8TvF44zLQJlBw=; b=dmV86nnKHiETn8UsxgxGIHzXJ3ARh/Og+M9e44L5hQLsz3jNlf+mGVYt0xVg0Cba6ILS2HpCLqcSIk0lG+ka1gg12Af5XaQa9y/XtTTx/6h8JX+6W2fgiN0x4FIRWc/6k+9V6Ws2aod+y87TGFZqOTXY8Erzr0hec3KmL3tBoltJF/JQeE5+kEuAHKO+QIh2pkfNGs6Lue1zHS4RE40H8ZqpaoRgcAAfLOGegbz3VkLfEyC4fsMSNe+Q/bMPD1wQuKhoEwZtYY/8St1LGYPEhDNmGXPcw8Vbr1aMO2Co+FgS/KaCgTBeDtrge9zmHYULtzlFyeSRGH7JQ75ZuXJ6DQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=dGPfxLL3fb5oc7kJJVgJG80Psptncp8TvF44zLQJlBw=; b=q+5sR7EZv1W4kJuBmYMRlt0bwsTMaSuEYPQJxlDYeS6MXqUA/XsA6qaXbL99mdRlUbGMVpZXkBwE+pLn7/e7xOghEncRKe/0DHBUGwVFQnEpYI3YjLAeOccSLqb/axxZdNiGY3gWkwMqb39OuggVgpsBIaplRndjrMgBDrbJsPM= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by DM6PR13MB4416.namprd13.prod.outlook.com (2603:10b6:5:1b5::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6954.21; Fri, 3 Nov 2023 06:26:28 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::45b3:d83c:cff2:4a1b]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::45b3:d83c:cff2:4a1b%6]) with mapi id 15.20.6954.021; Fri, 3 Nov 2023 06:26:28 +0000 From: Chaoyong He <chaoyong.he@corigine.com> To: dev@dpdk.org Cc: oss-drivers@corigine.com, Chaoyong He <chaoyong.he@corigine.com> Subject: [PATCH 00/11] Add basic flow support for corenic firmware Date: Fri, 3 Nov 2023 14:25:55 +0800 Message-Id: <20231103062606.2632012-1-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: BYAPR02CA0044.namprd02.prod.outlook.com (2603:10b6:a03:54::21) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|DM6PR13MB4416:EE_ X-MS-Office365-Filtering-Correlation-Id: 89bcd374-4441-43d4-97d9-08dbdc35cff9 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: EbozGgWMYm5r6YFZy3V1wCfjnlGArmzeQRUBV0SWj0iSJnY5zSfb43+9XiFwaekOu1ZFIRwe+GK12/sTEzpF2+XcYT7mrdKYz+iXZjBgKnTbT1yeI8KyL7gwsl5DI+aQuHDsm2HwNCI9zY3MvzCMSS6wAU+aCfAOf6Y1iqQ5RZEMHpTCO/yz1xnH/27Ayl1s5CVIx28J+kgEEYnM/Nfpvo99gGu/6sCIuHE2gsUmZ+qHPrBz0mkTWBytG3VR9wjPrqA/ywzY/Uh7YtI1Yl6oXbvGTdY7IjjoILXVGBcIg+FETW7YG9mdpvz/QE2SoTWSo8et8ESVVPYryOgDU/R6YpzLG9wJeUU0hxhrVV89bo6zVepD2DBUsBejFxynK/4bwx5texG/KfxGCKzhx1qCwmPTiom02w4EklFPTHSQsK/lkjkCAoAMLLwyGfIeOaRpbioCc+NJqHh/IdtAjqXO0bY5AHlJ+c7AUInAVRyWBsR0K4dYGJJip577ssEFgvCUSDYYbd53Pkvhh1iZn0TnNmiHDcKmIaUQs2hhOq47395vX+OVh/xA/j+KuPXNr7ba5EMF9sM8LdWvqA1Z21FADF/lmyjLlS4tdewYeL+0Z0Bq6PdO9z1n0Q+gvUWn6pqxjU7k5QNk7d3DKyPTjoti19e1DYF54sDHjHlOQlpy4ik= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230031)(39830400003)(366004)(376002)(136003)(396003)(346002)(230922051799003)(64100799003)(186009)(1800799009)(451199024)(107886003)(2616005)(6512007)(1076003)(38350700005)(86362001)(36756003)(38100700002)(83380400001)(2906002)(5660300002)(52116002)(478600001)(6666004)(26005)(6506007)(4326008)(8936002)(8676002)(6916009)(66946007)(66476007)(316002)(66556008)(41300700001)(44832011)(6486002); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: VX4XelVSXoROF1oAuI0Kx+Zy4mR+aRpbEzfWq0eqpD3zpIKB2U7/uoETJi4FCDzdGoyZVwctMZ/9F0/mup1UMjxlfRdps72qywc6dTmU3v+6WYyRim8DHE3dcQGRSwUTmVVT1HCdJsJk0OVrhp53MGjnRjtVZGKS27Tc595CJ8XI7j3oWJfmg7GQqpDULqdZV9P/fgluLI9urJsMwu8zGkMLARdRb40KYSeaSCvMsaH4l4mXa5PjqR1S+97RwU7OLt6Z8knHXTvbrNgl6q7jkkEomuJIgm080vmT308sh7blxUvP6vuKH0XDKLPxgykDQdhPGzVMqmEY9M4nGVOIXsdbrNLnRVo4r8d1PqZlJgcXrdBXrs7p9La0V18euVr3gUnMZhe6GMDadfQOkVYIfsPKxOIg76brxeLsR6Gp4ccMW3RATOrQX4Aay8iE6+jMEYYP2RtmPDFZx3FTGnxoHlcjwvojmyITN+q/i9UccO5kaTjqkmgOCegpmkZazntOmwLwogIBj7z6IoUVJ1w0LnjpXLgwutlG66sBr9mMWs+QpR5eCzkp3q143KsGH9EXZshLUgNlRQUbvy9n2uva8g66jMQFMQEKKQoYjVaViX+sgMrioAGbwOqXDVaTcOwyBacF+5hP1pGOJ1XjXhPFo9ebaiExMSLB0DQ9905uhxUBLRzUoiAJGjzOYE1XfKkQV47FhZnWC+vvQo9blklAYm76FdhjCiOWJI5efDUDTRdNN3UKuw3VCrdUIBDPWWwjL9Z8t6ACgkxECVVtgRm2KCTWMom08R+evgMjZR8dv7XFcHSnHFj1c34tVa1cPPmRZihFq6ddAuuB6ucDKzFn90Ts+IJ79EBpeB7NtQWrQ0rhmzplOluJB8D02LZxdxnMJiFKsCFmdP//hTTpTxFuBikZInJ8OSBuJYCAkn0gofmCPod7xmvimaK2OxucXnZKdEhPhxWhghDc+UAz48BkkfSX8CmK+seyo6+DQF9dziSMt812FgvFWHt61Aew0eOpjqI0T6jcf693xUVOl7Eiw2FVkxffuJAWz9dTi7fYHkzhHHgozXRDCF4O+JvFDjQPgAbSlEJZM34cRpdbNZbqR6TrBH+f4S5/9KK99ug9KayIgrHVjSS+yFlzCDzuxxJXOZ62VlRDy0vsyjfCmY9SOxzJceVTs+jM2zaReu5L+8CN7uLNDoqVSn01vChGJjH7Nwsi8H1PR6v0jKHS2KEz6CmvZXTaZEEIHAUAuZsK+ho7M+QFU5fAe44c+Q0c//P38VhFaGJTSfo6UuDdy1kx52h7splwjVtDeyMzfe39qlC5M3q8wYqXV/u23xyxSic0C1wKUTtv7lxMsMsRa4isCdRj59VKJBUCOBa90xrF8N+79k56GpV3WvnnjY039WLNXUihBEFtBJPCuLB1NI7fIDteXUkD0aV3Ig4b9lsX3vS30hDJCA2idjgCvOXwV/VDlLX4omO+mxaNTBTw4UiP83VLPZ/kq5f3W8VQkSPO3kmjYpg0jqhG3SSdQfGrjZ2tQmZb7hUv/DElJm/DcRrv7hF9p1NHk7MakNFNCuzyAzU0TrHxw4IgwhzmhRa41YCEKstvJ2CGHv476x8xpYUxPw== X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 89bcd374-4441-43d4-97d9-08dbdc35cff9 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Nov 2023 06:26:28.6363 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: P6f1VoJFwAAgmNi2kKPcfCpzB/j28elR7s3BY8ryUepbbGGviUDRQ1jY/0vGUUPP3GqPQrTr1GkdouiQR7T7GvyLDexli/0U7asNEK7mig4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR13MB4416 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 |
Series |
Add basic flow support for corenic firmware
|
|
Message
Chaoyong He
Nov. 3, 2023, 6:25 a.m. UTC
Add the very basic rte_flow support for corenic firmware. Chaoyong He (11): net/nfp: move some source files drivers: add the structures and functions for flow offload net/nfp: add the control message channel net/nfp: support flow API for CoreNIC firmware net/nfp: support Ethernet flow item net/nfp: support drop flow action net/nfp: support IPv4 flow item net/nfp: support IPv6 flow item net/nfp: support TCP/UDP/SCTP flow items drivers: support MARK flow action net/nfp: support QUEUE flow action drivers/common/nfp/nfp_common_ctrl.h | 2 + drivers/net/nfp/flower/nfp_conntrack.h | 2 +- drivers/net/nfp/flower/nfp_flower_cmsg.h | 2 +- .../{nfp_flow.c => flower/nfp_flower_flow.c} | 4 +- .../{nfp_flow.h => flower/nfp_flower_flow.h} | 10 +- .../net/nfp/flower/nfp_flower_representor.c | 2 +- drivers/net/nfp/meson.build | 4 +- drivers/net/nfp/nfp_ethdev.c | 27 +- drivers/net/nfp/nfp_net_cmsg.c | 66 ++ drivers/net/nfp/nfp_net_cmsg.h | 176 +++ drivers/net/nfp/nfp_net_common.h | 12 + drivers/net/nfp/nfp_net_ctrl.h | 1 + drivers/net/nfp/nfp_net_flow.c | 1003 +++++++++++++++++ drivers/net/nfp/nfp_net_flow.h | 30 + drivers/net/nfp/nfp_rxtx.c | 18 + 15 files changed, 1341 insertions(+), 18 deletions(-) rename drivers/net/nfp/{nfp_flow.c => flower/nfp_flower_flow.c} (99%) rename drivers/net/nfp/{nfp_flow.h => flower/nfp_flower_flow.h} (96%) create mode 100644 drivers/net/nfp/nfp_net_cmsg.c create mode 100644 drivers/net/nfp/nfp_net_cmsg.h create mode 100644 drivers/net/nfp/nfp_net_flow.c create mode 100644 drivers/net/nfp/nfp_net_flow.h
Comments
On 11/3/2023 6:25 AM, Chaoyong He wrote: > Add the very basic rte_flow support for corenic firmware. > > Chaoyong He (11): > net/nfp: move some source files > drivers: add the structures and functions for flow offload > net/nfp: add the control message channel > net/nfp: support flow API for CoreNIC firmware > net/nfp: support Ethernet flow item > net/nfp: support drop flow action > net/nfp: support IPv4 flow item > net/nfp: support IPv6 flow item > net/nfp: support TCP/UDP/SCTP flow items > drivers: support MARK flow action > net/nfp: support QUEUE flow action > Recheck-request: iol-compile-amd64-testing, iol-unit-amd64-testing
On 11/3/2023 6:25 AM, Chaoyong He wrote: > Add the very basic rte_flow support for corenic firmware. > > Chaoyong He (11): > net/nfp: move some source files > drivers: add the structures and functions for flow offload > net/nfp: add the control message channel > net/nfp: support flow API for CoreNIC firmware > net/nfp: support Ethernet flow item > net/nfp: support drop flow action > net/nfp: support IPv4 flow item > net/nfp: support IPv6 flow item > net/nfp: support TCP/UDP/SCTP flow items > drivers: support MARK flow action > net/nfp: support QUEUE flow action > `./devtools/check-doc-vs-code.sh` gives some warnings [1], can you please check them? Please update the .ini file in the relevant patch where feature is added, don't update it in a separate patch. [1] $ ./devtools/check-doc-vs-code.sh rte_flow doc out of sync for nfp action mark action queue
> On 11/3/2023 6:25 AM, Chaoyong He wrote: > > Add the very basic rte_flow support for corenic firmware. > > > > Chaoyong He (11): > > net/nfp: move some source files > > drivers: add the structures and functions for flow offload > > net/nfp: add the control message channel > > net/nfp: support flow API for CoreNIC firmware > > net/nfp: support Ethernet flow item > > net/nfp: support drop flow action > > net/nfp: support IPv4 flow item > > net/nfp: support IPv6 flow item > > net/nfp: support TCP/UDP/SCTP flow items > > drivers: support MARK flow action > > net/nfp: support QUEUE flow action > > > > Recheck-request: iol-compile-amd64-testing, iol-unit-amd64-testing Seems both compile problems was introduced by this commit: https://github.com/Corigine/dpdk-next-net-private/commit/34ff088cc24159c9fa6e61242efb76d0289b4e37 ``` ethdev: set and query RSS hash algorithm Currently, rte_eth_rss_conf supports configuring and querying RSS hash functions, rss key and it's length, but not RSS hash algorithm. The structure ``rte_eth_dev_info`` is extended by adding a new field "rss_algo_capa". Drivers are responsible for reporting this capa and configurations of RSS hash algorithm can be verified based on the capability. The default value of "rss_algo_capa" is RTE_ETH_HASH_ALGO_CAPA_MASK(DEFAULT) if drivers do not report it. The structure ``rte_eth_rss_conf`` is extended by adding a new field "algorithm". This represents the RSS algorithms to apply. If the value of "algorithm" used for configuration is a gibberish value, drivers should report the error. To check whether the drivers report valid "algorithm", it is set to default value before querying in rte_eth_dev_rss_hash_conf_get(). Signed-off-by: Jie Hai <haijie1@huawei.com> Signed-off-by: Dongdong Liu <liudongdong3@huawei.com> Acked-by: Huisong Li <lihuisong@huawei.com> Acked-by: Chengwen Feng <fengchengwen@huawei.com> Reviewed-by: Ferruh Yigit <ferruh.yigit@amd.com> ``` Which was not related with this patch series, I think?
On 11/7/2023 1:42 AM, Chaoyong He wrote: >> On 11/3/2023 6:25 AM, Chaoyong He wrote: >>> Add the very basic rte_flow support for corenic firmware. >>> >>> Chaoyong He (11): >>> net/nfp: move some source files >>> drivers: add the structures and functions for flow offload >>> net/nfp: add the control message channel >>> net/nfp: support flow API for CoreNIC firmware >>> net/nfp: support Ethernet flow item >>> net/nfp: support drop flow action >>> net/nfp: support IPv4 flow item >>> net/nfp: support IPv6 flow item >>> net/nfp: support TCP/UDP/SCTP flow items >>> drivers: support MARK flow action >>> net/nfp: support QUEUE flow action >>> >> >> Recheck-request: iol-compile-amd64-testing, iol-unit-amd64-testing > > Seems both compile problems was introduced by this commit: > https://github.com/Corigine/dpdk-next-net-private/commit/34ff088cc24159c9fa6e61242efb76d0289b4e37 > > ``` > ethdev: set and query RSS hash algorithm > Currently, rte_eth_rss_conf supports configuring and querying > RSS hash functions, rss key and it's length, but not RSS hash > algorithm. > > The structure ``rte_eth_dev_info`` is extended by adding a new > field "rss_algo_capa". Drivers are responsible for reporting this > capa and configurations of RSS hash algorithm can be verified based > on the capability. The default value of "rss_algo_capa" is > RTE_ETH_HASH_ALGO_CAPA_MASK(DEFAULT) if drivers do not report it. > > The structure ``rte_eth_rss_conf`` is extended by adding a new > field "algorithm". This represents the RSS algorithms to apply. > If the value of "algorithm" used for configuration is a gibberish > value, drivers should report the error. > > To check whether the drivers report valid "algorithm", it is set > to default value before querying in rte_eth_dev_rss_hash_conf_get(). > > Signed-off-by: Jie Hai <haijie1@huawei.com> > Signed-off-by: Dongdong Liu <liudongdong3@huawei.com> > Acked-by: Huisong Li <lihuisong@huawei.com> > Acked-by: Chengwen Feng <fengchengwen@huawei.com> > Reviewed-by: Ferruh Yigit <ferruh.yigit@amd.com> > ``` > > Which was not related with this patch series, I think? > Yes, build error is not related with this patch, and it is fixed in next-net, that is why I tried to re-trigger the test, but it didn't work somehow [1]. Btw, this set is pending because of other change request, not because of the reported build failure. [1] https://patchwork.dpdk.org/project/dpdk/patch/20231103062606.2632012-12-chaoyong.he@corigine.com/
> On 11/7/2023 1:42 AM, Chaoyong He wrote: > >> On 11/3/2023 6:25 AM, Chaoyong He wrote: > >>> Add the very basic rte_flow support for corenic firmware. > >>> > >>> Chaoyong He (11): > >>> net/nfp: move some source files > >>> drivers: add the structures and functions for flow offload > >>> net/nfp: add the control message channel > >>> net/nfp: support flow API for CoreNIC firmware > >>> net/nfp: support Ethernet flow item > >>> net/nfp: support drop flow action > >>> net/nfp: support IPv4 flow item > >>> net/nfp: support IPv6 flow item > >>> net/nfp: support TCP/UDP/SCTP flow items > >>> drivers: support MARK flow action > >>> net/nfp: support QUEUE flow action > >>> > >> > >> Recheck-request: iol-compile-amd64-testing, iol-unit-amd64-testing > > > > Seems both compile problems was introduced by this commit: > > https://github.com/Corigine/dpdk-next-net- > private/commit/34ff088cc2415 > > 9c9fa6e61242efb76d0289b4e37 > > > > ``` > > ethdev: set and query RSS hash algorithm Currently, rte_eth_rss_conf > > supports configuring and querying RSS hash functions, rss key and it's > > length, but not RSS hash algorithm. > > > > The structure ``rte_eth_dev_info`` is extended by adding a new field > > "rss_algo_capa". Drivers are responsible for reporting this capa and > > configurations of RSS hash algorithm can be verified based on the > > capability. The default value of "rss_algo_capa" is > > RTE_ETH_HASH_ALGO_CAPA_MASK(DEFAULT) if drivers do not report it. > > > > The structure ``rte_eth_rss_conf`` is extended by adding a new field > > "algorithm". This represents the RSS algorithms to apply. > > If the value of "algorithm" used for configuration is a gibberish > > value, drivers should report the error. > > > > To check whether the drivers report valid "algorithm", it is set to > > default value before querying in rte_eth_dev_rss_hash_conf_get(). > > > > Signed-off-by: Jie Hai <haijie1@huawei.com> > > Signed-off-by: Dongdong Liu <liudongdong3@huawei.com> > > Acked-by: Huisong Li <lihuisong@huawei.com> > > Acked-by: Chengwen Feng <fengchengwen@huawei.com> > > Reviewed-by: Ferruh Yigit <ferruh.yigit@amd.com> ``` > > > > Which was not related with this patch series, I think? > > > > Yes, build error is not related with this patch, and it is fixed in next-net, that is > why I tried to re-trigger the test, but it didn't work somehow [1]. > > Btw, this set is pending because of other change request, not because of the > reported build failure. Oh, yes, I understand, the reason is I need to also update the 'nfp.ini' document. But we decide delay this patch set to the next version of DPDK as we found still need some modification to the logic of flower firmware. Thanks. > > > [1] > https://patchwork.dpdk.org/project/dpdk/patch/20231103062606.263201 > 2-12-chaoyong.he@corigine.com/ >
Recheck-request: iol-compile-amd64-testing, iol-unit-amd64-testing Weird. I will dig up the original job which polled for the previous retest request, and see whether this new one runs through properly.
Sorry, now I understand what is causing confusion. When a retest is requested, that retest is done on the original dpdk artifact we created for the patchseries. We don't pull again, re-apply, and then rerun the testing. I think that is the behavior you were expecting, Ferruh. We have discussed whether this is an important option to support in future iterations of the retest-request feature. One idea is to add another option in the retest request schema which allows for the requester to indicate they want the patch re-applied onto the newer branch, and testing rerun with that new dpdk. It's good to see that this is a use case which would see application. Will be aiming to add this support in the future.
On 11/7/2023 5:04 PM, Patrick Robb wrote: > Sorry, now I understand what is causing confusion. When a retest is > requested, that retest is done on the original dpdk artifact we created > for the patchseries. We don't pull again, re-apply, and then rerun the > testing. I think that is the behavior you were expecting, Ferruh. > Hi Patrick, Thanks for clarification, yes that was my expectation. > We have discussed whether this is an important option to support in > future iterations of the retest-request feature. One idea is to add > another option in the retest request schema which allows for the > requester to indicate they want the patch re-applied onto the newer > branch, and testing rerun with that new dpdk. It's good to see that this > is a use case which would see application. Will be aiming to add this > support in the future. > I am also not sure if this is the common usecase, lets record the need, we can decide what to support based on it. And as a generic approach, what about following syntax: Recheck-request,attribute=value: ... And for above case, attribute can be "pull=true" ...
On Tue, Nov 7, 2023 at 3:19 PM Ferruh Yigit <ferruh.yigit@amd.com> wrote: > > I am also not sure if this is the common usecase, lets record the need, > we can decide what to support based on it. > > > And as a generic approach, what about following syntax: > Recheck-request,attribute=value: ... > > And for above case, attribute can be "pull=true" ... > > That syntax looks reasonable, and if it's omitted, default is pull=false. Thanks!