From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6ED21A053D for ; Fri, 17 Jul 2020 18:33:13 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 67B611BEBF; Fri, 17 Jul 2020 18:33:13 +0200 (CEST) Received: from us-smtp-delivery-1.mimecast.com (us-smtp-1.mimecast.com [205.139.110.61]) by dpdk.org (Postfix) with ESMTP id EE2741BEE7 for ; Fri, 17 Jul 2020 18:33:11 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1595003591; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=AuoQUxmlB5FkRS0GwY61RHSEJpjgBVE92bgHqoZspsk=; b=BVBj/SO9XbAEterDX6mbVYDhDp7+pQ2jKKdqCv24aVLL8yCXnxEtY5mOHZLrY98INYpIcR LfznZKJGJfsyauc1vQ1WseHUK2H2fwC3UtBv47x+KcD2iiVOSLtsG1eHLgEGkpE+o/Tr7b oSvhs21ykPvW2L4pmohMhqka0KxClT8= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-198-2rdBoFDGN0CwU5qLuXbgSw-1; Fri, 17 Jul 2020 12:33:02 -0400 X-MC-Unique: 2rdBoFDGN0CwU5qLuXbgSw-1 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 614A28018A1; Fri, 17 Jul 2020 16:32:59 +0000 (UTC) Received: from rh.redhat.com (unknown [10.33.36.168]) by smtp.corp.redhat.com (Postfix) with ESMTP id 8015772E48; Fri, 17 Jul 2020 16:32:58 +0000 (UTC) From: Kevin Traynor To: Alan Dewar Cc: Jasvinder Singh , dpdk stable Date: Fri, 17 Jul 2020 17:32:09 +0100 Message-Id: <20200717163214.4839-21-ktraynor@redhat.com> In-Reply-To: <20200717163214.4839-1-ktraynor@redhat.com> References: <20200717163214.4839-1-ktraynor@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8bit Subject: [dpdk-stable] patch 'sched: fix port time rounding' has been queued to LTS release 18.11.10 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 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 Sender: "stable" Hi, FYI, your patch has been queued to LTS release 18.11.10 Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet. It will be pushed if I get no objections before 07/23/20. So please shout if anyone has objections. Also note that after the patch there's a diff of the upstream commit vs the patch applied to the branch. This will indicate if there was any rebasing needed to apply to the stable branch. If there were code changes for rebasing (ie: not only metadata diffs), please double check that the rebase was correctly done. Queued patches are on a temporary branch at: https://github.com/kevintraynor/dpdk-stable-queue This queued commit can be viewed at: https://github.com/kevintraynor/dpdk-stable-queue/commit/38f30a1f450b56666f5339ee4e68f795cdf51b06 Thanks. Kevin. --- >From 38f30a1f450b56666f5339ee4e68f795cdf51b06 Mon Sep 17 00:00:00 2001 From: Alan Dewar Date: Thu, 25 Jun 2020 10:59:30 +0100 Subject: [PATCH] sched: fix port time rounding [ upstream commit 83415d4fd88c925002655aa755601998a3cdef2c ] The QoS scheduler works off port time that is computed from the number of CPU cycles that have elapsed since the last time the port was polled. It divides the number of elapsed cycles to calculate how many bytes can be sent, however this division can generate rounding errors, where some fraction of a byte sent may be lost. Lose enough of these fractional bytes and the QoS scheduler underperforms. The problem is worse with low bandwidths. To compensate for this rounding error this fix doesn't advance the port's time_cpu_cycles by the number of cycles that have elapsed, but by multiplying the computed number of bytes that can be sent (which has been rounded down) by number of cycles per byte. This will mean that port's time_cpu_cycles will lag behind the CPU cycles momentarily. At the next poll, the lag will be taken into account. Fixes: de3cfa2c98 ("sched: initial import") Signed-off-by: Alan Dewar Acked-by: Jasvinder Singh --- lib/librte_sched/rte_sched.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/lib/librte_sched/rte_sched.c b/lib/librte_sched/rte_sched.c index 89c3d1e7f7..d8ace20fe0 100644 --- a/lib/librte_sched/rte_sched.c +++ b/lib/librte_sched/rte_sched.c @@ -201,4 +201,5 @@ struct rte_sched_port { uint64_t time; /* Current NIC TX time measured in bytes */ struct rte_reciprocal inv_cycles_per_byte; /* CPU cycles per byte */ + uint64_t cycles_per_byte; /* Scheduling loop detection */ @@ -683,4 +684,5 @@ rte_sched_port_config(struct rte_sched_port_params *params) / params->rate; port->inv_cycles_per_byte = rte_reciprocal_value(cycles_per_byte); + port->cycles_per_byte = cycles_per_byte; /* Scheduling loop detection */ @@ -2191,7 +2193,11 @@ rte_sched_port_time_resync(struct rte_sched_port *port) { uint64_t cycles = rte_get_tsc_cycles(); - uint64_t cycles_diff = cycles - port->time_cpu_cycles; + uint64_t cycles_diff; uint64_t bytes_diff; + if (cycles < port->time_cpu_cycles) + port->time_cpu_cycles = 0; + + cycles_diff = cycles - port->time_cpu_cycles; /* Compute elapsed time in bytes */ bytes_diff = rte_reciprocal_divide(cycles_diff << RTE_SCHED_TIME_SHIFT, @@ -2199,5 +2205,6 @@ rte_sched_port_time_resync(struct rte_sched_port *port) /* Advance port time */ - port->time_cpu_cycles = cycles; + port->time_cpu_cycles += + (bytes_diff * port->cycles_per_byte) >> RTE_SCHED_TIME_SHIFT; port->time_cpu_bytes += bytes_diff; if (port->time < port->time_cpu_bytes) -- 2.21.3 --- Diff of the applied patch vs upstream commit (please double-check if non-empty: --- --- - 2020-07-17 17:17:01.165113282 +0100 +++ 0021-sched-fix-port-time-rounding.patch 2020-07-17 17:16:59.997771060 +0100 @@ -1 +1 @@ -From 83415d4fd88c925002655aa755601998a3cdef2c Mon Sep 17 00:00:00 2001 +From 38f30a1f450b56666f5339ee4e68f795cdf51b06 Mon Sep 17 00:00:00 2001 @@ -5,0 +6,2 @@ +[ upstream commit 83415d4fd88c925002655aa755601998a3cdef2c ] + @@ -24 +25,0 @@ -Cc: stable@dpdk.org @@ -33 +34 @@ -index 68a171b508..0fa0741664 100644 +index 89c3d1e7f7..d8ace20fe0 100644 @@ -36 +37 @@ -@@ -223,4 +223,5 @@ struct rte_sched_port { +@@ -201,4 +201,5 @@ struct rte_sched_port { @@ -41,2 +42,2 @@ - /* Grinders */ -@@ -853,4 +854,5 @@ rte_sched_port_config(struct rte_sched_port_params *params) + /* Scheduling loop detection */ +@@ -683,4 +684,5 @@ rte_sched_port_config(struct rte_sched_port_params *params) @@ -47,2 +48,2 @@ - /* Grinders */ -@@ -2674,8 +2676,12 @@ rte_sched_port_time_resync(struct rte_sched_port *port) + /* Scheduling loop detection */ +@@ -2191,7 +2193,11 @@ rte_sched_port_time_resync(struct rte_sched_port *port) @@ -54 +54,0 @@ - uint32_t i; @@ -62 +62 @@ -@@ -2683,5 +2689,6 @@ rte_sched_port_time_resync(struct rte_sched_port *port) +@@ -2199,5 +2205,6 @@ rte_sched_port_time_resync(struct rte_sched_port *port)