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 E9D20A0540 for ; Mon, 4 Jul 2022 07:28:34 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D92A042826; Mon, 4 Jul 2022 07:28:34 +0200 (CEST) Received: from mail-oa1-f53.google.com (mail-oa1-f53.google.com [209.85.160.53]) by mails.dpdk.org (Postfix) with ESMTP id 9426D40E09; Mon, 4 Jul 2022 07:28:32 +0200 (CEST) Received: by mail-oa1-f53.google.com with SMTP id 586e51a60fabf-10be1122de0so4944375fac.7; Sun, 03 Jul 2022 22:28:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=kJMaqRue6qvAmIBwDrwe4T8LGtfhZwlBk55RdgY+N4I=; b=Hux/u5sDKZiBnujs9eXzfCjMOINXWcLHqEivPrku9JpjOuzB6HLKyYZ8NyqBcLukn+ uUEGq1zO0ej2Q3URSV+OQ7b9HjQZWptv2d+J/DQbmVQvy24GDIi4e0jU7Bhp7U00RTlL fsk2kg1tB61Zjm7HMg6GyH+6d65H3XxdVB5kay4hjrNCQ9dCE8zivALHMXmo4UhtXsZX u5o2125AH8A25H3jQ4fERITGkbmbH2l0V34lPjGR1cxufq9yhGGG+cHEbKlUfw21Kpmm amOV6EMy5mQVDIm3k2joF9SoOJyNh1KrnZ5+8jo4QKoy+U06fkNusydfz62T0/OI3Lrc dHeg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=kJMaqRue6qvAmIBwDrwe4T8LGtfhZwlBk55RdgY+N4I=; b=nEUeTy2Ox2Ct2L8vXltIsHOeBXowcNd4T5velyPknLoIoNYFWglbxXwdugPemVQii9 pvM7H+cixrr6+ubttjoaJnz3V1EVUnejpKDIqtL2XILL1UOC5EeUkAoyGiDjgegvtFNH WfMwnN2st+fgVJZNJrcW24PKojcSMBrcant9FDBbr8B8n98B4Xg67ydb61wDcPDI6Hrk xW4cs6IdKTpOv2682NocPeq8jf9ZK8u3yqDOvglSG96mToCEBLL+Zxg2kqkvxdoE5O9d BEXHfKtoIgo+P5u6LjU9m+R7uz0+O7alkGUwB4jX3prgP4RhMFKQRxzyr86SdJ+WhpZ2 i1yg== X-Gm-Message-State: AJIora8Q/AfQhcmYLL0TRsQcqJj1VF4kD5yFqt1M9buvL/6aPsrEajfm oNMGlrJ+xDYIJWNGLaZWgpiqT+kJHcBz+GvcsHI= X-Google-Smtp-Source: AGRyM1viaZ+/YcSXN2J/QLgFFaLo7L4nGA/ZY+tb20/UpM0rfOEKzWpmPwl1JqB3lnR4xVi4qfsgd8SeD4LSQgIgh7g= X-Received: by 2002:a05:6870:430d:b0:102:572e:ffb with SMTP id w13-20020a056870430d00b00102572e0ffbmr15798332oah.232.1656912511939; Sun, 03 Jul 2022 22:28:31 -0700 (PDT) MIME-Version: 1.0 References: <20220623181502.181567-1-peng1x.zhang@intel.com> <5bd19b16-f878-98fe-e1ea-d992c1ffcaa7@oktetlabs.ru> <9a5ab8bb-ccdd-25d2-65d6-370ef95ab712@huawei.com> In-Reply-To: <9a5ab8bb-ccdd-25d2-65d6-370ef95ab712@huawei.com> From: Dmitry Kozlyuk Date: Mon, 4 Jul 2022 08:28:21 +0300 Message-ID: Subject: Re: [PATCH] app/testpmd: fix secondary process cannot dump packet To: "lihuisong (C)" Cc: "Zhang, Peng1X" , Andrew Rybchenko , "dev@dpdk.org" , "Singh, Aman Deep" , "Zhang, Yuying" , "stable@dpdk.org" Content-Type: multipart/alternative; boundary="000000000000b2b9b505e2f4018e" X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org --000000000000b2b9b505e2f4018e Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Mon, Jul 4, 2022 at 5:37 AM lihuisong (C) wrote: > [...] > =E5=9C=A8 2022/7/1 19:36, Zhang, Peng1X =E5=86=99=E9=81=93: > [...] > > The reason why not use 'dev->data->rx_queue_state' is whether queue > state is start or stop in primary > > process depend on rx_conf->rx_deferred_start after start testpmd. And > after having started testpmd, > > queue state can be controlled by command for example 'port x rxq x > start'. > > Should we align with the same behavior of queues state for primary and > secondary process after start testpmd? > If primary process stops a queue, but secondary doesn't know. > we have to simplify this queue state problem like you momentioned > if we don't have a good idea. > AFAIU, dev->data->rx_queue_state should be aligned with rx_conf->rx_deferred_start. The reason why testpmd manages the queue state itself in the offending patc= h is that not all PMDs implement `rte_eth_rx/tx_queue_info_get()` and ethdev won't return `dev->data->rx_queue_state` in this case. --000000000000b2b9b505e2f4018e Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
On Mon, Jul 4, 2022 at 5:37 AM lihuisong (C) <lihuisong@huawei.com> wrote:
[...]
=E5=9C=A8 2022/7/1 19:36, Zhang, Peng1X =E5=86=99=E9=81=93:
[...]
> The reason why not use 'dev->data->rx_queue_state' is wh= ether queue state is start or stop in primary
> process depend on rx_conf->rx_deferred_start after start testpmd. A= nd after having started testpmd,
> queue state can be controlled by command for example 'port x rxq x= start'.
> Should we align with the same behavior of queues state for primary and= secondary process after start testpmd?
If primary process stops a queue, but secondary doesn't know.
we have to simplify this queue state problem like you momentioned
if we don't have a good idea.

AFAIU= , dev->data->rx_queue_state should be aligned with rx_conf->rx_def= erred_start.
The reason why testpmd manages the q= ueue state itself in the offending patch
is that not all PMDs= implement `rte_eth_rx/tx_queue_info_get()`
and ethdev won't = return `dev->data->rx_queue_state` in this case.
--000000000000b2b9b505e2f4018e--