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 EE2A3A0579; Thu, 8 Apr 2021 15:27:58 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 70BBA140FC3; Thu, 8 Apr 2021 15:27:58 +0200 (CEST) Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by mails.dpdk.org (Postfix) with ESMTP id 254FF40138 for ; Thu, 8 Apr 2021 15:27:56 +0200 (CEST) IronPort-SDR: kNBgHuoutq31pZ0B8KHGRKz81oZ9nvy4vXG1SC1bRLf7qMmzYAl1QEEBdzw5Pe6PaxvrRi9xOj V19H66nz2yjA== X-IronPort-AV: E=McAfee;i="6000,8403,9948"; a="213946581" X-IronPort-AV: E=Sophos;i="5.82,206,1613462400"; d="scan'208";a="213946581" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Apr 2021 06:27:55 -0700 IronPort-SDR: 5+mTF9BmM6Z2i9L721HLLdUROO3z8KEFP9od1NVoIkiIib4c86l/yApDtIviXpNi7WuL4mejtU QO/iPiGxrf5Q== X-IronPort-AV: E=Sophos;i="5.82,206,1613462400"; d="scan'208";a="380250794" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.213.203.5]) ([10.213.203.5]) by orsmga003-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Apr 2021 06:27:54 -0700 To: "Min Hu (Connor)" , Ajit Khaparde Cc: dpdk-dev , Xiaoyun Li References: <1614906276-34293-1-git-send-email-oulijun@huawei.com> <1617068905-5364-1-git-send-email-humin29@huawei.com> <1dfa9ed4-052e-22f1-a84a-c49303edbc00@intel.com> <1b992d1c-2af7-8c38-8916-f62b21bd46fe@huawei.com> <6c3381a7-631f-fdfa-db1d-a4ff2d67dcaf@huawei.com> From: Ferruh Yigit X-User: ferruhy Message-ID: <656b7564-8ba5-f036-d15c-7488b08bd549@intel.com> Date: Thu, 8 Apr 2021 14:27:50 +0100 MIME-Version: 1.0 In-Reply-To: <6c3381a7-631f-fdfa-db1d-a4ff2d67dcaf@huawei.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v8] app/testpmd: support multi-process 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 Sender: "dev" On 4/8/2021 11:32 AM, Min Hu (Connor) wrote: > Hi, Ferry and all, >     This patch has been acked: >     Acked-by: Xiaoyun Li >     Acked-by: Ajit Khaparde > >     Are there any other comments about that patch? > Hi Connor, In today's release status meeting I asked if more parties can test this patch, Lets wait a little more for it to be tested. > > 在 2021/3/30 18:43, Min Hu (Connor) 写道: >> >> >> 在 2021/3/30 18:19, Ferruh Yigit 写道: >>> On 3/30/2021 7:41 AM, Min Hu (Connor) wrote: >>>> >>>> >>>> 在 2021/3/30 11:11, Ajit Khaparde 写道: >>>>> On Mon, Mar 29, 2021 at 6:48 PM Min Hu (Connor) wrote: >>>>>> >>>>>> From: Lijun Ou >>>>>> >>>>>> This patch adds multi-process support for testpmd. >>>>>> The test cmd example as follows: >>>>>> the primary cmd: >>>>>> ./dpdk-testpmd -a xxx --proc-type=auto -l 0-1 -- -i \ >>>>>> --rxq=4 --txq=4 --num-procs=2 --proc-id=0 >>>>>> >>>>>> the secondary cmd: >>>>>> ./dpdk-testpmd -a xxx --proc-type=auto -l 2-3 -- -i \ >>>>>> --rxq=4 --txq=4 --num-procs=2 --proc-id=1 >>>>>> >>>>>> Signed-off-by: Min Hu (Connor) >>>>>> Signed-off-by: Lijun Ou >>>>> Some minor nits below. Otherwise looks fine to me. >>>>> Acked-by: Ajit Khaparde >>>>> >>>> Thanks Ajit. >>>> >>>> Hi, Ferruh, >>>>      Should I send v9 to fix the grammar bugs in doc which Ajit point >>>> out or fix it in future? >>>> >>> >>> Hi Connor, if they are only outstanding issues, I can fix them while merging. >>> >> Thanks Ferruh. >>>>>> --- >>>>>> v8: >>>>>> * Added warning info about queue numbers and process numbers. >>>>>> >>>>> :::snip:::: >>>>> >>>>>> +*   ``--rxq=N`` >>>>>> + >>>>>> +    Set the number of Rx queues per port to N. N is the sum of queues >>>>>> used by primary >>>>>> +    and secondary process. Primary process and secondary process should >>>>>> have separate >>>>>> +    queues, and each should occupy at least one queue. Where N should be >>>>>> the multiple >>>>>> +    of number of processes. >>>>> of the number of processes. >>>>> >>>>>> + >>>>>> +*   ``--txq=N`` >>>>>> + >>>>>> +    Set the number of Tx queues per port to N. N is the sum of queues >>>>>> used by primary >>>>>> +    and secondary process. Primary process and secondary process should >>>>>> have separate >>>>>> +    queues, and each should occupy at least one queue. Where N should be >>>>>> the multiple >>>>>> +    of number of processes. >>>>> of the number of processes. >>>>> >>>>>> + >>>>>> +*   ``--num-procs=N`` >>>>>> + >>>>>> +    The number of processes which will be used. >>>>>> + >>>>> :::: snip :::: >>>>>> +The number of rings should be a multiple of the number of processes. If not, >>>>>> +redundant queues will exist after queues are allocated to processes. >>>>>> After RSS is >>>>>> +enabled, packet loss occurs when traffic is sent to all processes at the >>>>>> same time. >>>>>> +Some traffic enters redundant queues and cannot be forwarded. >>>>>> + >>>>>> +Most dev ops is supported in primary and secondary process. While >>>>>> secondary process >>>>> Most dev ops are supported in the primary and secondary process. While.... >>>>> >>>>>> +is not permitted to allocate or release shared memory, so some ops are >>>>>> not supported >>>>>> +as follows: >>>>>> +``dev_configure`` >>>>>> +``dev_start`` >>>>>> +``dev_stop`` >>>>>> +``rx_queue_setup`` >>>>>> +``tx_queue_setup`` >>>>>> +``rx_queue_release`` >>>>>> +``tx_queue_release`` >>>>> :::: snip::: >>>>> >>> >>> . >> .