Message ID | 1446108423-14231-1-git-send-email-david.marchand@6wind.com (mailing list archive) |
---|---|
State | Accepted, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 0EAE35A69; Thu, 29 Oct 2015 09:47:11 +0100 (CET) Received: from mail-wi0-f180.google.com (mail-wi0-f180.google.com [209.85.212.180]) by dpdk.org (Postfix) with ESMTP id 19F69567C for <dev@dpdk.org>; Thu, 29 Oct 2015 09:47:10 +0100 (CET) Received: by wicfv8 with SMTP id fv8so36014912wic.0 for <dev@dpdk.org>; Thu, 29 Oct 2015 01:47:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind_com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id; bh=DmpZTpsBY70t7HP1POJxKxdJfC5/E6NLyZIFOoMXS18=; b=ycRgx2iiVcAJi56h/qJ6MyBvXjj0H2M9ovEr8/LXmIYLOOjdeGH6kRji/osvOB0Kot HED3RCESRx8fTkhb1FujaRqmo63yH9GqOH7gqbLWlcH4evlU2UVfG8SyyQJPdpGNTvg8 Rp2iCJpixg3w7hnADriZ+ajnoEUDt+b8RaF+R+ZqmqQRlPi4AtfvLn5ryUbNOY9hIyg/ RIzb7J3EQjiq6DxLtN+1LfDobSf124uY+DrOlSelt5vsI9n5OljG3a26ZlyszS4HlbEx YNWM7ruJZqwz8XL8z+ueLONtYwOA83jLKjH7F2xVq7fXJCL1/cKqH2TF9aqYjhmYNqaM SfXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=DmpZTpsBY70t7HP1POJxKxdJfC5/E6NLyZIFOoMXS18=; b=PyEjWypWJupRxzyW99LAIrf2L/OOIZd+3JSoJL/ykZYmJYb9HVPNlgjzDzB8Mchfpp PqcnPvSGYUcuLRYSy24+a75fOnmbjoUTafM+bcJh14pAa264XyznQ7EdXzZrfpU1deK1 qD6ViDCTvMi3arB3yKWWAEvKYBnNaImRa+RatEzNRIaElikhitKWg+lU1yQyp9mxKEmf te7T7OaewQkdQBUu0JTU18Dh0AjD5TbgLG5XyoNNFoYeXD/1CFQ0oHjmrEPmM2lKKRCD M6mxwSWi/bunFItxnnL7TwNN2AQ+jO47VoGVz2zHqJTiA+eoPP5s8rj+rvDM6hel6asV ZoNg== X-Gm-Message-State: ALoCoQnOt7UP7YTMGn20y+J+N+86hBpTUNnrLafdAEUMVJwDO51BhoKhXNjjmXhUCxgILFplYcw/ X-Received: by 10.194.85.75 with SMTP id f11mr687293wjz.9.1446108429876; Thu, 29 Oct 2015 01:47:09 -0700 (PDT) Received: from gloops.dev.6wind.com (89-158-215-180.rev.numericable.fr. [89.158.215.180]) by smtp.gmail.com with ESMTPSA id h198sm2326712wmd.9.2015.10.29.01.47.08 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Thu, 29 Oct 2015 01:47:09 -0700 (PDT) From: David Marchand <david.marchand@6wind.com> To: dev@dpdk.org Date: Thu, 29 Oct 2015 09:47:03 +0100 Message-Id: <1446108423-14231-1-git-send-email-david.marchand@6wind.com> X-Mailer: git-send-email 1.9.1 Subject: [dpdk-dev] [PATCH] app/testpmd: fix icmp echo tx queues X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
David Marchand
Oct. 29, 2015, 8:47 a.m. UTC
icmp echo forward engine relies on the core rank rather than the number of
queues available.
This fix is still wrong, because if we set a nb-rxq > nb-txq, then we will use
an invalid txq, but with this, we are aligned on other fwd engines.
Signed-off-by: David Marchand <david.marchand@6wind.com>
---
app/test-pmd/config.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
2015-10-29 09:47, David Marchand: > icmp echo forward engine relies on the core rank rather than the number of > queues available. > This fix is still wrong, because if we set a nb-rxq > nb-txq, then we will use > an invalid txq, but with this, we are aligned on other fwd engines. > > Signed-off-by: David Marchand <david.marchand@6wind.com> [...] > - fs->tx_queue = lc_id; > + fs->tx_queue = rxq; Pablo, any opinion?
Hi, > -----Original Message----- > From: David Marchand [mailto:david.marchand@6wind.com] > Sent: Thursday, October 29, 2015 8:47 AM > To: dev@dpdk.org > Cc: De Lara Guarch, Pablo > Subject: [PATCH] app/testpmd: fix icmp echo tx queues > > icmp echo forward engine relies on the core rank rather than the number of > queues available. > This fix is still wrong, because if we set a nb-rxq > nb-txq, then we will use > an invalid txq, but with this, we are aligned on other fwd engines. > > Signed-off-by: David Marchand <david.marchand@6wind.com> > --- > app/test-pmd/config.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c > index cf2aa6e..314215c 100644 > --- a/app/test-pmd/config.c > +++ b/app/test-pmd/config.c > @@ -1278,7 +1278,7 @@ icmp_echo_config_setup(void) > fs->rx_port = fwd_ports_ids[rxp]; > fs->rx_queue = rxq; > fs->tx_port = fs->rx_port; > - fs->tx_queue = lc_id; > + fs->tx_queue = rxq; > fs->peer_addr = fs->tx_port; > if (verbose_level > 0) > printf(" stream=%d port=%d rxq=%d > txq=%d\n", > -- > 1.9.1 Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com>
> > icmp echo forward engine relies on the core rank rather than the number of > > queues available. > > This fix is still wrong, because if we set a nb-rxq > nb-txq, then we will use > > an invalid txq, but with this, we are aligned on other fwd engines. > > > > Signed-off-by: David Marchand <david.marchand@6wind.com> > > Acked-by: Pablo de Lara <pablo.de.lara.guarch@intel.com> Applied, thanks
diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index cf2aa6e..314215c 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -1278,7 +1278,7 @@ icmp_echo_config_setup(void) fs->rx_port = fwd_ports_ids[rxp]; fs->rx_queue = rxq; fs->tx_port = fs->rx_port; - fs->tx_queue = lc_id; + fs->tx_queue = rxq; fs->peer_addr = fs->tx_port; if (verbose_level > 0) printf(" stream=%d port=%d rxq=%d txq=%d\n",