From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id CAA9F436A0; Fri, 8 Dec 2023 03:25:21 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BB447402CD; Fri, 8 Dec 2023 03:25:21 +0100 (CET) Received: from szxga03-in.huawei.com (szxga03-in.huawei.com [45.249.212.189]) by mails.dpdk.org (Postfix) with ESMTP id 4CD0C402AC for ; Fri, 8 Dec 2023 03:25:20 +0100 (CET) Received: from kwepemm000004.china.huawei.com (unknown [172.30.72.54]) by szxga03-in.huawei.com (SkyGuard) with ESMTP id 4SmZjm3C0rz14L4V; Fri, 8 Dec 2023 10:25:16 +0800 (CST) Received: from [10.67.121.59] (10.67.121.59) by kwepemm000004.china.huawei.com (7.193.23.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Fri, 8 Dec 2023 10:25:17 +0800 Message-ID: Date: Fri, 8 Dec 2023 10:25:17 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 Subject: Re: [PATCH RESEND v6 0/5] app/testpmd: support multiple process attach and detach port From: "lihuisong (C)" To: , CC: , , , , References: <20220825024425.10534-1-lihuisong@huawei.com> <20230802031600.23631-1-lihuisong@huawei.com> <021c14be-5216-b7f3-05b3-fd29bfc42b2d@huawei.com> <7303e1be-bec1-ab21-d24a-4af96010df39@huawei.com> In-Reply-To: <7303e1be-bec1-ab21-d24a-4af96010df39@huawei.com> Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.121.59] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To kwepemm000004.china.huawei.com (7.193.23.18) X-CFilter-Loop: Reflected X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Hi Ferruh and Thomas, This series have been discussing for over a year and a half. Looking back on previous discussions, we have also made some progress and consensus. I am sticking to track it. Because they resolve a real exist issue. Can you take a look at it again? BR, /Huisong 在 2023/10/30 20:17, lihuisong (C) 写道: > Hi Ferruh and Thomas, > > This series have been discussing more than one year. > Kindly ping for reivew. > > > 在 2023/10/9 18:34, lihuisong (C) 写道: >> Hi Ferruh and Thomas, >> >> Can you take a look at this series? They've been over a year on >> disscussion. >> >> >> 在 2023/8/2 11:15, Huisong Li 写道: >>> This patchset fix some bugs and support attaching and detaching port >>> in primary and secondary. >>> >>> --- >>>   -v6: adjust rte_eth_dev_is_used position based on alphabetical order >>>        in version.map >>>   -v5: move 'ALLOCATED' state to the back of 'REMOVED' to avoid abi >>> break. >>>   -v4: fix a misspelling. >>>   -v3: >>>     #1 merge patch 1/6 and patch 2/6 into patch 1/5, and add >>> modification >>>        for other bus type. >>>     #2 add a RTE_ETH_DEV_ALLOCATED state in rte_eth_dev_state to >>> resolve >>>        the probelm in patch 2/5. >>>   -v2: resend due to CI unexplained failure. >>> >>> Huisong Li (5): >>>    drivers/bus: restore driver assignment at front of probing >>>    ethdev: fix skip valid port in probing callback >>>    app/testpmd: check the validity of the port >>>    app/testpmd: add attach and detach port for multiple process >>>    app/testpmd: stop forwarding in new or destroy event >>> >>>   app/test-pmd/testpmd.c                   | 47 >>> +++++++++++++++--------- >>>   app/test-pmd/testpmd.h                   |  1 - >>>   drivers/bus/auxiliary/auxiliary_common.c |  9 ++++- >>>   drivers/bus/dpaa/dpaa_bus.c              |  9 ++++- >>>   drivers/bus/fslmc/fslmc_bus.c            |  8 +++- >>>   drivers/bus/ifpga/ifpga_bus.c            | 12 ++++-- >>>   drivers/bus/pci/pci_common.c             |  9 ++++- >>>   drivers/bus/vdev/vdev.c                  | 10 ++++- >>>   drivers/bus/vmbus/vmbus_common.c         |  9 ++++- >>>   drivers/net/bnxt/bnxt_ethdev.c           |  3 +- >>>   drivers/net/bonding/bonding_testpmd.c    |  1 - >>>   drivers/net/mlx5/mlx5.c                  |  2 +- >>>   lib/ethdev/ethdev_driver.c               | 13 +++++-- >>>   lib/ethdev/ethdev_driver.h               | 12 ++++++ >>>   lib/ethdev/ethdev_pci.h                  |  2 +- >>>   lib/ethdev/rte_class_eth.c               |  2 +- >>>   lib/ethdev/rte_ethdev.c                  |  4 +- >>>   lib/ethdev/rte_ethdev.h                  |  4 +- >>>   lib/ethdev/version.map                   |  1 + >>>   19 files changed, 114 insertions(+), 44 deletions(-) >>> >> >> . > > .