DPDK patches and discussions
 help / color / mirror / Atom feed
From: Jesse Brandeburg <jesse.brandeburg@intel.com>
To: brian.johnson@intel.com, thomas@monjalon.net, david.marchand@redhat.com
Cc: Jesse Brandeburg <jesse.brandeburg@intel.com>,
	dev@dpdk.org, qi.z.zhang@intel.com
Subject: [dpdk-dev] [PATCH v2 2/2] ice: update writeback policy to reduce latency
Date: Fri, 23 Oct 2020 13:22:00 -0700	[thread overview]
Message-ID: <20201023202200.1909832-3-jesse.brandeburg@intel.com> (raw)
In-Reply-To: <20201023202200.1909832-1-jesse.brandeburg@intel.com>

Just like iavf, setting the value to 2us allows for generally good
streaming packet performance while keeping latency down, and
generally keeps the performance of the PF and VF interfaces similar.

The previous value of 0x10 was making latency on a single packet
receive be as much as 16us.

Fixes: 65dfc889d86b ("net/ice: support Rx queue interruption")

Reported-by: Brian Johnson <brian.johnson@intel.com>
Signed-off-by: Jesse Brandeburg <jesse.brandeburg@intel.com>
---
 drivers/net/ice/ice_ethdev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c
index c65125ff3260..0df491f93126 100644
--- a/drivers/net/ice/ice_ethdev.c
+++ b/drivers/net/ice/ice_ethdev.c
@@ -3345,7 +3345,7 @@ __vsi_queues_bind_intr(struct ice_vsi *vsi, uint16_t msix_vect,
 		PMD_DRV_LOG(INFO, "queue %d is binding to vect %d",
 			    base_queue + i, msix_vect);
 		/* set ITR0 value */
-		ICE_WRITE_REG(hw, GLINT_ITR(0, msix_vect), 0x10);
+		ICE_WRITE_REG(hw, GLINT_ITR(0, msix_vect), 0x2);
 		ICE_WRITE_REG(hw, QINT_RQCTL(base_queue + i), val);
 		ICE_WRITE_REG(hw, QINT_TQCTL(base_queue + i), val_tx);
 	}
-- 
2.25.4


  parent reply	other threads:[~2020-10-23 20:22 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-23 20:21 [dpdk-dev] [PATCH v2 0/2] intel: fix writeback Jesse Brandeburg
2020-10-23 20:21 ` [dpdk-dev] [PATCH v2 1/2] iavf: Fix performance with writeback policy Jesse Brandeburg
2020-10-23 20:22 ` Jesse Brandeburg [this message]
2020-10-26  1:11 ` [dpdk-dev] [PATCH v2 0/2] intel: fix writeback Zhang, Qi Z

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201023202200.1909832-3-jesse.brandeburg@intel.com \
    --to=jesse.brandeburg@intel.com \
    --cc=brian.johnson@intel.com \
    --cc=david.marchand@redhat.com \
    --cc=dev@dpdk.org \
    --cc=qi.z.zhang@intel.com \
    --cc=thomas@monjalon.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).