[dpdk-stable] patch 'event/dlb2: remove references to deferred scheduling' has been queued to stable release 20.11.2

[Xueming Li]

Hi,

FYI, your patch has been queued to stable release 20.11.2

Note it hasn't been pushed to http://dpdk.org/browse/dpdk-stable yet.
It will be pushed if I get no objections before 06/14/21. 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/steevenlee/dpdk

This queued commit can be viewed at:
https://github.com/steevenlee/dpdk/commit/8ba82ed84d54592b07a202ab5c945866b17cdc64

Thanks.

Xueming Li <xuemingl at nvidia.com>

---
>From 8ba82ed84d54592b07a202ab5c945866b17cdc64 Mon Sep 17 00:00:00 2001
From: Timothy McDaniel <timothy.mcdaniel at intel.com>
Date: Fri, 21 May 2021 11:11:34 +0200
Subject: [PATCH] event/dlb2: remove references to deferred scheduling
Cc: Luca Boccassi <bluca at debian.org>

[ upstream commit 0e94408f0028f888c638810973a7f2ddbfdbb795 ]

Deferred scheduling is a DLB v1.0 feature, and is not valid for
DLB v2.0 or v2.5.

Fixes: bc62748bd7d4 ("event/dlb2: add private data structures and constants")
Fixes: a2e4f1f5e79f ("event/dlb2: add dequeue and its burst variants")

Signed-off-by: Timothy McDaniel <timothy.mcdaniel at intel.com>
---
 doc/guides/eventdevs/dlb2.rst  | 21 ---------------------
 drivers/event/dlb2/dlb2_priv.h |  3 ---
 2 files changed, 24 deletions(-)

diff --git a/doc/guides/eventdevs/dlb2.rst b/doc/guides/eventdevs/dlb2.rst
index 94d2c77ff4..6ba67f278c 100644
--- a/doc/guides/eventdevs/dlb2.rst
+++ b/doc/guides/eventdevs/dlb2.rst
@@ -314,27 +314,6 @@ The PMD does not support the following configuration sequences:
 This sequence is not supported because the event device must be reconfigured
 before its ports or queues can be.
 
-Deferred Scheduling
-~~~~~~~~~~~~~~~~~~~
-
-The DLB2 PMD's default behavior for managing a CQ is to "pop" the CQ once per
-dequeued event before returning from rte_event_dequeue_burst(). This frees the
-corresponding entries in the CQ, which enables the DLB2 to schedule more events
-to it.
-
-To support applications seeking finer-grained scheduling control -- for example
-deferring scheduling to get the best possible priority scheduling and
-load-balancing -- the PMD supports a deferred scheduling mode. In this mode,
-the CQ entry is not popped until the *subsequent* rte_event_dequeue_burst()
-call. This mode only applies to load-balanced event ports with dequeue depth of
-1.
-
-To enable deferred scheduling, use the defer_sched vdev argument like so:
-
-    .. code-block:: console
-
-       --vdev=dlb1_event,defer_sched=on
-
 Atomic Inflights Allocation
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
diff --git a/drivers/event/dlb2/dlb2_priv.h b/drivers/event/dlb2/dlb2_priv.h
index b73cf3ff14..8a01938785 100644
--- a/drivers/event/dlb2/dlb2_priv.h
+++ b/drivers/event/dlb2/dlb2_priv.h
@@ -27,7 +27,6 @@
 #define DLB2_MAX_NUM_EVENTS "max_num_events"
 #define DLB2_NUM_DIR_CREDITS "num_dir_credits"
 #define DEV_ID_ARG "dev_id"
-#define DLB2_DEFER_SCHED_ARG "defer_sched"
 #define DLB2_QID_DEPTH_THRESH_ARG "qid_depth_thresh"
 #define DLB2_COS_ARG "cos"
 
@@ -498,7 +497,6 @@ struct dlb2_eventdev {
 	uint16_t num_dir_ports; /* total num of dir ports requested */
 	bool umwait_allowed;
 	bool global_dequeue_wait; /* Not using per dequeue wait if true */
-	bool defer_sched;
 	enum dlb2_cq_poll_modes poll_mode;
 	uint8_t revision;
 	bool configured;
@@ -523,7 +521,6 @@ struct dlb2_devargs {
 	int max_num_events;
 	int num_dir_credits_override;
 	int dev_id;
-	int defer_sched;
 	struct dlb2_qid_depth_thresholds qid_depth_thresholds;
 	enum dlb2_cos cos_id;
 };
-- 
2.25.1

---
  Diff of the applied patch vs upstream commit (please double-check if non-empty:
---
--- -	2021-06-12 06:54:01.041480000 +0800
+++ 0178-event-dlb2-remove-references-to-deferred-scheduling.patch	2021-06-12 06:53:56.710000000 +0800
@@ -1 +1 @@
-From 0e94408f0028f888c638810973a7f2ddbfdbb795 Mon Sep 17 00:00:00 2001
+From 8ba82ed84d54592b07a202ab5c945866b17cdc64 Mon Sep 17 00:00:00 2001
@@ -4,0 +5,3 @@
+Cc: Luca Boccassi <bluca at debian.org>
+
+[ upstream commit 0e94408f0028f888c638810973a7f2ddbfdbb795 ]
@@ -11 +13,0 @@
-Cc: stable at dpdk.org
@@ -20 +22 @@
-index 31de6bc470..c60c454d6b 100644
+index 94d2c77ff4..6ba67f278c 100644
@@ -23 +25 @@
-@@ -293,27 +293,6 @@ The PMD does not support the following configuration sequences:
+@@ -314,27 +314,6 @@ The PMD does not support the following configuration sequences:
@@ -30 +32 @@
--The DLB PMD's default behavior for managing a CQ is to "pop" the CQ once per
+-The DLB2 PMD's default behavior for managing a CQ is to "pop" the CQ once per
@@ -32 +34 @@
--corresponding entries in the CQ, which enables the DLB to schedule more events
+-corresponding entries in the CQ, which enables the DLB2 to schedule more events
@@ -46 +48 @@
--       --vdev=dlb2_event,defer_sched=on
+-       --vdev=dlb1_event,defer_sched=on
@@ -52 +54 @@
-index 3140764a59..b1225af37e 100644
+index b73cf3ff14..8a01938785 100644
@@ -55 +57 @@
-@@ -32,7 +32,6 @@
+@@ -27,7 +27,6 @@
@@ -62,2 +64,2 @@
- #define DLB2_POLL_INTERVAL_ARG "poll_interval"
-@@ -585,7 +584,6 @@ struct dlb2_eventdev {
+ 
+@@ -498,7 +497,6 @@ struct dlb2_eventdev {
@@ -69,3 +71,3 @@
- 	int poll_interval;
- 	int sw_credit_quanta;
-@@ -620,7 +618,6 @@ struct dlb2_devargs {
+ 	uint8_t revision;
+ 	bool configured;
+@@ -523,7 +521,6 @@ struct dlb2_devargs {
@@ -78 +80 @@
- 	int poll_interval;
+ };