DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH 0/3] eventdev fixes to set positive rte_errno
@ 2019-07-04 10:03 Andrew Rybchenko
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 1/3] eventdev: fix " Andrew Rybchenko
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Andrew Rybchenko @ 2019-07-04 10:03 UTC (permalink / raw)
  To: Jerin Jacob; +Cc: dev

rte_errno values should be positive, but there are many places where
negative value is set or simply reused from function return value
which has negative errno for error condition indication.

Dilshod Urazov (3):
  eventdev: fix to set positive rte_errno
  event/sw: fix to set positive rte_errno
  event/opdl: fix to set positive rte_errno

 drivers/event/opdl/opdl_evdev.c                | 10 +++++-----
 drivers/event/opdl/opdl_evdev_init.c           | 12 ++++++------
 drivers/event/opdl/opdl_ring.c                 |  2 +-
 drivers/event/sw/sw_evdev.c                    |  8 ++++----
 lib/librte_eventdev/rte_event_eth_tx_adapter.h |  8 ++++----
 lib/librte_eventdev/rte_event_timer_adapter.c  | 18 +++++++++---------
 lib/librte_eventdev/rte_eventdev.c             | 16 ++++++++--------
 lib/librte_eventdev/rte_eventdev.h             | 26 +++++++++++++-------------
 8 files changed, 50 insertions(+), 50 deletions(-)

-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [dpdk-dev] [PATCH 1/3] eventdev: fix to set positive rte_errno
  2019-07-04 10:03 [dpdk-dev] [PATCH 0/3] eventdev fixes to set positive rte_errno Andrew Rybchenko
@ 2019-07-04 10:03 ` Andrew Rybchenko
  2019-07-04 11:34   ` [dpdk-dev] [dpdk-stable] " David Marchand
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 2/3] event/sw: " Andrew Rybchenko
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 3/3] event/opdl: " Andrew Rybchenko
  2 siblings, 1 reply; 10+ messages in thread
From: Andrew Rybchenko @ 2019-07-04 10:03 UTC (permalink / raw)
  To: Jerin Jacob, Nikhil Rao, Erik Gabriel Carrillo
  Cc: dev, Dilshod Urazov, stable

From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>

Fixes: c9bf83947e2e ("eventdev: add eth Tx adapter APIs")
Fixes: 47d05b292820 ("eventdev: add timer adapter common code")
Fixes: 6750b21bd6af ("eventdev: add default software timer adapter")
Fixes: c75f7897ea35 ("eventdev: set error code in port link/unlink functions")
Fixes: 7d1acc9dde93 ("eventdev: introduce helper function for enqueue burst")
Fixes: 406aed4e0dd9 ("eventdev: add errno-style return values")
Fixes: c64e1b7b20d2 ("eventdev: add new software event timer adapter")
Cc: stable@dpdk.org

Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
---
 lib/librte_eventdev/rte_event_eth_tx_adapter.h |  8 ++++----
 lib/librte_eventdev/rte_event_timer_adapter.c  | 18 +++++++++---------
 lib/librte_eventdev/rte_eventdev.c             | 16 ++++++++--------
 lib/librte_eventdev/rte_eventdev.h             | 26 +++++++++++++-------------
 4 files changed, 34 insertions(+), 34 deletions(-)

diff --git a/lib/librte_eventdev/rte_event_eth_tx_adapter.h b/lib/librte_eventdev/rte_event_eth_tx_adapter.h
index 9bed12b..c848261 100644
--- a/lib/librte_eventdev/rte_event_eth_tx_adapter.h
+++ b/lib/librte_eventdev/rte_event_eth_tx_adapter.h
@@ -332,10 +332,10 @@ struct rte_event_eth_tx_adapter_stats {
  *   *rte_event*. If the return value is less than *nb_events*, the remaining
  *   events at the end of ev[] are not consumed and the caller has to take care
  *   of them, and rte_errno is set accordingly. Possible errno values include:
- *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's queue
+ *   - EINVAL   The port ID is invalid, device ID is invalid, an event's queue
  *              ID is invalid, or an event's sched type doesn't match the
  *              capabilities of the destination queue.
- *   - -ENOSPC  The event port was backpressured and unable to enqueue
+ *   - ENOSPC   The event port was backpressured and unable to enqueue
  *              one or more events. This error code is only applicable to
  *              closed systems.
  */
@@ -350,12 +350,12 @@ struct rte_event_eth_tx_adapter_stats {
 #ifdef RTE_LIBRTE_EVENTDEV_DEBUG
 	if (dev_id >= RTE_EVENT_MAX_DEVS ||
 		!rte_eventdevs[dev_id].attached) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
 	if (port_id >= dev->data->nb_ports) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 #endif
diff --git a/lib/librte_eventdev/rte_event_timer_adapter.c b/lib/librte_eventdev/rte_event_timer_adapter.c
index 459bc47..5ce399e 100644
--- a/lib/librte_eventdev/rte_event_timer_adapter.c
+++ b/lib/librte_eventdev/rte_event_timer_adapter.c
@@ -192,17 +192,17 @@ struct rte_event_timer_adapter *
 						   &adapter->data->caps,
 						   &adapter->ops);
 	if (ret < 0) {
-		rte_errno = ret;
+		rte_errno = -ret;
 		goto free_memzone;
 	}
 
 	if (!(adapter->data->caps &
 	      RTE_EVENT_TIMER_ADAPTER_CAP_INTERNAL_PORT)) {
-		FUNC_PTR_OR_NULL_RET_WITH_ERRNO(conf_cb, -EINVAL);
+		FUNC_PTR_OR_NULL_RET_WITH_ERRNO(conf_cb, EINVAL);
 		ret = conf_cb(adapter->data->id, adapter->data->event_dev_id,
 			      &adapter->data->event_port_id, conf_arg);
 		if (ret < 0) {
-			rte_errno = ret;
+			rte_errno = -ret;
 			goto free_memzone;
 		}
 	}
@@ -214,10 +214,10 @@ struct rte_event_timer_adapter *
 		adapter->ops = &swtim_ops;
 
 	/* Allow driver to do some setup */
-	FUNC_PTR_OR_NULL_RET_WITH_ERRNO(adapter->ops->init, -ENOTSUP);
+	FUNC_PTR_OR_NULL_RET_WITH_ERRNO(adapter->ops->init, ENOTSUP);
 	ret = adapter->ops->init(adapter);
 	if (ret < 0) {
-		rte_errno = ret;
+		rte_errno = -ret;
 		goto free_memzone;
 	}
 
@@ -509,11 +509,11 @@ struct event_buffer {
 	*nb_events_flushed = rte_event_enqueue_burst(dev_id, port_id,
 						     &events[tail_idx], n);
 	if (*nb_events_flushed != n) {
-		if (rte_errno == -EINVAL) {
+		if (rte_errno == EINVAL) {
 			EVTIM_LOG_ERR("failed to enqueue invalid event - "
 				      "dropping it");
 			(*nb_events_inv)++;
-		} else if (rte_errno == -ENOSPC)
+		} else if (rte_errno == ENOSPC)
 			rte_pause();
 	}
 
@@ -832,7 +832,7 @@ struct swtim {
 	if (ret < 0) {
 		if (ret != -EALREADY) {
 			EVTIM_LOG_ERR("failed to initialize timer subsystem");
-			rte_errno = ret;
+			rte_errno = -ret;
 			goto free_mempool;
 		}
 	}
@@ -840,7 +840,7 @@ struct swtim {
 	ret = rte_timer_data_alloc(&sw->timer_data_id);
 	if (ret < 0) {
 		EVTIM_LOG_ERR("failed to allocate timer data instance");
-		rte_errno = ret;
+		rte_errno = -ret;
 		goto free_mempool;
 	}
 
diff --git a/lib/librte_eventdev/rte_eventdev.c b/lib/librte_eventdev/rte_eventdev.c
index cc3199f..f44c869 100644
--- a/lib/librte_eventdev/rte_eventdev.c
+++ b/lib/librte_eventdev/rte_eventdev.c
@@ -889,18 +889,18 @@
 	uint16_t *links_map;
 	int i, diag;
 
-	RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, -EINVAL, 0);
+	RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, EINVAL, 0);
 	dev = &rte_eventdevs[dev_id];
 
 	if (*dev->dev_ops->port_link == NULL) {
 		RTE_EDEV_LOG_ERR("Function not supported\n");
-		rte_errno = -ENOTSUP;
+		rte_errno = ENOTSUP;
 		return 0;
 	}
 
 	if (!is_valid_port(dev, port_id)) {
 		RTE_EDEV_LOG_ERR("Invalid port_id=%" PRIu8, port_id);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
@@ -921,7 +921,7 @@
 
 	for (i = 0; i < nb_links; i++)
 		if (queues[i] >= dev->data->nb_queues) {
-			rte_errno = -EINVAL;
+			rte_errno = EINVAL;
 			return 0;
 		}
 
@@ -948,18 +948,18 @@
 	int i, diag, j;
 	uint16_t *links_map;
 
-	RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, -EINVAL, 0);
+	RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, EINVAL, 0);
 	dev = &rte_eventdevs[dev_id];
 
 	if (*dev->dev_ops->port_unlink == NULL) {
 		RTE_EDEV_LOG_ERR("Function not supported");
-		rte_errno = -ENOTSUP;
+		rte_errno = ENOTSUP;
 		return 0;
 	}
 
 	if (!is_valid_port(dev, port_id)) {
 		RTE_EDEV_LOG_ERR("Invalid port_id=%" PRIu8, port_id);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
@@ -988,7 +988,7 @@
 	nb_unlinks = j;
 	for (i = 0; i < nb_unlinks; i++)
 		if (queues[i] >= dev->data->nb_queues) {
-			rte_errno = -EINVAL;
+			rte_errno = EINVAL;
 			return 0;
 		}
 
diff --git a/lib/librte_eventdev/rte_eventdev.h b/lib/librte_eventdev/rte_eventdev.h
index 927f43c..5044a13 100644
--- a/lib/librte_eventdev/rte_eventdev.h
+++ b/lib/librte_eventdev/rte_eventdev.h
@@ -1318,12 +1318,12 @@ struct rte_eventdev {
 
 #ifdef RTE_LIBRTE_EVENTDEV_DEBUG
 	if (dev_id >= RTE_EVENT_MAX_DEVS || !rte_eventdevs[dev_id].attached) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
 	if (port_id >= dev->data->nb_ports) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 #endif
@@ -1372,10 +1372,10 @@ struct rte_eventdev {
  *   *rte_event*. If the return value is less than *nb_events*, the remaining
  *   events at the end of ev[] are not consumed and the caller has to take care
  *   of them, and rte_errno is set accordingly. Possible errno values include:
- *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's queue
+ *   - EINVAL   The port ID is invalid, device ID is invalid, an event's queue
  *              ID is invalid, or an event's sched type doesn't match the
  *              capabilities of the destination queue.
- *   - -ENOSPC  The event port was backpressured and unable to enqueue
+ *   - ENOSPC   The event port was backpressured and unable to enqueue
  *              one or more events. This error code is only applicable to
  *              closed systems.
  * @see rte_event_port_attr_get(), RTE_EVENT_PORT_ATTR_ENQ_DEPTH
@@ -1422,10 +1422,10 @@ struct rte_eventdev {
  *   *rte_event*. If the return value is less than *nb_events*, the remaining
  *   events at the end of ev[] are not consumed and the caller has to take care
  *   of them, and rte_errno is set accordingly. Possible errno values include:
- *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's queue
+ *   - EINVAL   The port ID is invalid, device ID is invalid, an event's queue
  *              ID is invalid, or an event's sched type doesn't match the
  *              capabilities of the destination queue.
- *   - -ENOSPC  The event port was backpressured and unable to enqueue
+ *   - ENOSPC   The event port was backpressured and unable to enqueue
  *              one or more events. This error code is only applicable to
  *              closed systems.
  * @see rte_event_port_attr_get(), RTE_EVENT_PORT_ATTR_ENQ_DEPTH
@@ -1473,10 +1473,10 @@ struct rte_eventdev {
  *   *rte_event*. If the return value is less than *nb_events*, the remaining
  *   events at the end of ev[] are not consumed and the caller has to take care
  *   of them, and rte_errno is set accordingly. Possible errno values include:
- *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's queue
+ *   - EINVAL   The port ID is invalid, device ID is invalid, an event's queue
  *              ID is invalid, or an event's sched type doesn't match the
  *              capabilities of the destination queue.
- *   - -ENOSPC  The event port was backpressured and unable to enqueue
+ *   - ENOSPC   The event port was backpressured and unable to enqueue
  *              one or more events. This error code is only applicable to
  *              closed systems.
  * @see rte_event_port_attr_get(), RTE_EVENT_PORT_ATTR_ENQ_DEPTH
@@ -1595,12 +1595,12 @@ struct rte_eventdev {
 
 #ifdef RTE_LIBRTE_EVENTDEV_DEBUG
 	if (dev_id >= RTE_EVENT_MAX_DEVS || !rte_eventdevs[dev_id].attached) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
 	if (port_id >= dev->data->nb_ports) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 #endif
@@ -1673,9 +1673,9 @@ struct rte_eventdev {
  * of link[] are not established, and the caller has to take care of them.
  * If return value is less than *nb_links* then implementation shall update the
  * rte_errno accordingly, Possible rte_errno values are
- * (-EDQUOT) Quota exceeded(Application tried to link the queue configured with
+ * (EDQUOT) Quota exceeded(Application tried to link the queue configured with
  *  RTE_EVENT_QUEUE_CFG_SINGLE_LINK to more than one event ports)
- * (-EINVAL) Invalid parameter
+ * (EINVAL) Invalid parameter
  *
  */
 int
@@ -1720,7 +1720,7 @@ struct rte_eventdev {
  * end of queues[] are not unlinked, and the caller has to take care of them.
  * If return value is less than *nb_unlinks* then implementation shall update
  * the rte_errno accordingly, Possible rte_errno values are
- * (-EINVAL) Invalid parameter
+ * (EINVAL) Invalid parameter
  */
 int
 rte_event_port_unlink(uint8_t dev_id, uint8_t port_id,
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [dpdk-dev] [PATCH 2/3] event/sw: fix to set positive rte_errno
  2019-07-04 10:03 [dpdk-dev] [PATCH 0/3] eventdev fixes to set positive rte_errno Andrew Rybchenko
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 1/3] eventdev: fix " Andrew Rybchenko
@ 2019-07-04 10:03 ` Andrew Rybchenko
  2019-07-04 10:43   ` Van Haaren, Harry
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 3/3] event/opdl: " Andrew Rybchenko
  2 siblings, 1 reply; 10+ messages in thread
From: Andrew Rybchenko @ 2019-07-04 10:03 UTC (permalink / raw)
  To: Jerin Jacob, Harry van Haaren; +Cc: dev, Dilshod Urazov, stable

From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>

Fixes: 371a688fc159 ("event/sw: support linking queues to ports")
Cc: stable@dpdk.org

Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
---
 drivers/event/sw/sw_evdev.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/event/sw/sw_evdev.c b/drivers/event/sw/sw_evdev.c
index 1175d6c..fb8e8be 100644
--- a/drivers/event/sw/sw_evdev.c
+++ b/drivers/event/sw/sw_evdev.c
@@ -38,12 +38,12 @@
 
 		/* check for qid map overflow */
 		if (q->cq_num_mapped_cqs >= RTE_DIM(q->cq_map)) {
-			rte_errno = -EDQUOT;
+			rte_errno = EDQUOT;
 			break;
 		}
 
 		if (p->is_directed && p->num_qids_mapped > 0) {
-			rte_errno = -EDQUOT;
+			rte_errno = EDQUOT;
 			break;
 		}
 
@@ -59,12 +59,12 @@
 		if (q->type == SW_SCHED_TYPE_DIRECT) {
 			/* check directed qids only map to one port */
 			if (p->num_qids_mapped > 0) {
-				rte_errno = -EDQUOT;
+				rte_errno = EDQUOT;
 				break;
 			}
 			/* check port only takes a directed flow */
 			if (num > 1) {
-				rte_errno = -EDQUOT;
+				rte_errno = EDQUOT;
 				break;
 			}
 
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [dpdk-dev] [PATCH 3/3] event/opdl: fix to set positive rte_errno
  2019-07-04 10:03 [dpdk-dev] [PATCH 0/3] eventdev fixes to set positive rte_errno Andrew Rybchenko
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 1/3] eventdev: fix " Andrew Rybchenko
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 2/3] event/sw: " Andrew Rybchenko
@ 2019-07-04 10:03 ` Andrew Rybchenko
  2019-07-04 11:41   ` David Marchand
  2 siblings, 1 reply; 10+ messages in thread
From: Andrew Rybchenko @ 2019-07-04 10:03 UTC (permalink / raw)
  To: Jerin Jacob, Liang Ma, Peter Mccarthy; +Cc: dev, Dilshod Urazov, stable

From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>

Fixes: 0bf298e39286 ("event/opdl: add event port config get/set")
Fixes: 3c7f3dcfb099 ("event/opdl: add PMD main body and helper function")
Fixes: 4236ce9bf5bf ("event/opdl: add OPDL ring infrastructure library")
Cc: stable@dpdk.org

Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
---
 drivers/event/opdl/opdl_evdev.c      | 10 +++++-----
 drivers/event/opdl/opdl_evdev_init.c | 12 ++++++------
 drivers/event/opdl/opdl_ring.c       |  2 +-
 3 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/drivers/event/opdl/opdl_evdev.c b/drivers/event/opdl/opdl_evdev.c
index d2d2be4..3beca89 100644
--- a/drivers/event/opdl/opdl_evdev.c
+++ b/drivers/event/opdl/opdl_evdev.c
@@ -102,7 +102,7 @@
 			     dev->data->dev_id,
 				queues[0],
 				p->id);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
@@ -113,7 +113,7 @@
 			     dev->data->dev_id,
 				num,
 				p->id);
-		rte_errno = -EDQUOT;
+		rte_errno = EDQUOT;
 		return 0;
 	}
 
@@ -123,7 +123,7 @@
 			     dev->data->dev_id,
 				p->id,
 				queues[0]);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
@@ -134,7 +134,7 @@
 				p->id,
 				p->external_qid,
 				queues[0]);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
@@ -160,7 +160,7 @@
 			     dev->data->dev_id,
 			     queues[0],
 			     p->id);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 	RTE_SET_USED(nb_unlinks);
diff --git a/drivers/event/opdl/opdl_evdev_init.c b/drivers/event/opdl/opdl_evdev_init.c
index 582ad69..15aae47 100644
--- a/drivers/event/opdl/opdl_evdev_init.c
+++ b/drivers/event/opdl/opdl_evdev_init.c
@@ -35,7 +35,7 @@
 					     p->id,
 					     ev[i].queue_id,
 					     p->next_external_qid);
-				rte_errno = -EINVAL;
+				rte_errno = EINVAL;
 				return 0;
 			}
 		}
@@ -63,7 +63,7 @@
 	} else {
 		if (num > 0 &&
 				ev[0].queue_id != p->next_external_qid) {
-			rte_errno = -EINVAL;
+			rte_errno = EINVAL;
 			return 0;
 		}
 	}
@@ -116,7 +116,7 @@
 	RTE_SET_USED(ev);
 	RTE_SET_USED(num);
 
-	rte_errno = -ENOSPC;
+	rte_errno = ENOSPC;
 
 	return 0;
 }
@@ -145,7 +145,7 @@
 
 
 	if (enqueued < num)
-		rte_errno = -ENOSPC;
+		rte_errno = ENOSPC;
 
 	return enqueued;
 }
@@ -164,7 +164,7 @@
 	RTE_SET_USED(ev);
 	RTE_SET_USED(num);
 
-	rte_errno = -ENOSPC;
+	rte_errno = ENOSPC;
 
 	return 0;
 }
@@ -240,7 +240,7 @@
 			     "Attempt to dequeue num of events larger than port (%d) max",
 			     opdl_pmd_dev_id(p->opdl),
 			     p->id);
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 
diff --git a/drivers/event/opdl/opdl_ring.c b/drivers/event/opdl/opdl_ring.c
index e988f1c..e8b29e2 100644
--- a/drivers/event/opdl/opdl_ring.c
+++ b/drivers/event/opdl/opdl_ring.c
@@ -756,7 +756,7 @@ struct opdl_ring {
 opdl_stage_disclaim(struct opdl_stage *s, uint32_t num_entries, bool block)
 {
 	if (num_entries != s->num_event) {
-		rte_errno = -EINVAL;
+		rte_errno = EINVAL;
 		return 0;
 	}
 	if (s->threadsafe == false) {
-- 
1.8.3.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 2/3] event/sw: fix to set positive rte_errno
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 2/3] event/sw: " Andrew Rybchenko
@ 2019-07-04 10:43   ` Van Haaren, Harry
  0 siblings, 0 replies; 10+ messages in thread
From: Van Haaren, Harry @ 2019-07-04 10:43 UTC (permalink / raw)
  To: Andrew Rybchenko, Jerin Jacob; +Cc: dev, Dilshod Urazov, stable

> -----Original Message-----
> From: Andrew Rybchenko [mailto:arybchenko@solarflare.com]
> Sent: Thursday, July 4, 2019 11:04 AM
> To: Jerin Jacob <jerinj@marvell.com>; Van Haaren, Harry
> <harry.van.haaren@intel.com>
> Cc: dev@dpdk.org; Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>;
> stable@dpdk.org
> Subject: [PATCH 2/3] event/sw: fix to set positive rte_errno
> 
> From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
> 
> Fixes: 371a688fc159 ("event/sw: support linking queues to ports")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
> ---
>  drivers/event/sw/sw_evdev.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)

Code changes are fine, I see you're updating eventdev header in 1/3, all good for git master.

Not sure if it should be backported to stable? (and stable is on CC at the moment..)
I would suggest dropping this from stable - given stable should try to minimize change.

Acked-by: Harry van Haaren <harry.van.haaren@intel.com>

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [dpdk-stable] [PATCH 1/3] eventdev: fix to set positive rte_errno
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 1/3] eventdev: fix " Andrew Rybchenko
@ 2019-07-04 11:34   ` David Marchand
  2019-07-04 11:56     ` Andrew Rybchenko
  0 siblings, 1 reply; 10+ messages in thread
From: David Marchand @ 2019-07-04 11:34 UTC (permalink / raw)
  To: Andrew Rybchenko
  Cc: Jerin Jacob, Nikhil Rao, Erik Gabriel Carrillo, dev,
	Dilshod Urazov, dpdk stable

We have fixes for different releases, expect the stable maintainers to ask
for help :-)


On Thu, Jul 4, 2019 at 12:04 PM Andrew Rybchenko <arybchenko@solarflare.com>
wrote:

> From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
>
> Fixes: c9bf83947e2e ("eventdev: add eth Tx adapter APIs")
>

v18.11-rc1~724

Fixes: 47d05b292820 ("eventdev: add timer adapter common code")
>

v18.05-rc1~462


> Fixes: 6750b21bd6af ("eventdev: add default software timer adapter")
>

v18.05-rc1~460


> Fixes: c75f7897ea35 ("eventdev: set error code in port link/unlink
> functions")
>

v18.02-rc1~202


> Fixes: 7d1acc9dde93 ("eventdev: introduce helper function for enqueue
> burst")
>

v17.08-rc1~90

Fixes: 406aed4e0dd9 ("eventdev: add errno-style return values")
>

v17.05-rc1~166


> Fixes: c64e1b7b20d2 ("eventdev: add new software event timer adapter")
>

This last sha1 c64e1b7b20d2 is unknown to my git.
Please, can you double check?


Cc: stable@dpdk.org
>
> Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
> ---
>  lib/librte_eventdev/rte_event_eth_tx_adapter.h |  8 ++++----
>  lib/librte_eventdev/rte_event_timer_adapter.c  | 18 +++++++++---------
>  lib/librte_eventdev/rte_eventdev.c             | 16 ++++++++--------
>  lib/librte_eventdev/rte_eventdev.h             | 26
> +++++++++++++-------------
>  4 files changed, 34 insertions(+), 34 deletions(-)
>
> diff --git a/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> b/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> index 9bed12b..c848261 100644
> --- a/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> +++ b/lib/librte_eventdev/rte_event_eth_tx_adapter.h
> @@ -332,10 +332,10 @@ struct rte_event_eth_tx_adapter_stats {
>   *   *rte_event*. If the return value is less than *nb_events*, the
> remaining
>   *   events at the end of ev[] are not consumed and the caller has to
> take care
>   *   of them, and rte_errno is set accordingly. Possible errno values
> include:
> - *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's
> queue
> + *   - EINVAL   The port ID is invalid, device ID is invalid, an event's
> queue
>   *              ID is invalid, or an event's sched type doesn't match the
>   *              capabilities of the destination queue.
> - *   - -ENOSPC  The event port was backpressured and unable to enqueue
> + *   - ENOSPC   The event port was backpressured and unable to enqueue
>   *              one or more events. This error code is only applicable to
>   *              closed systems.
>   */
> @@ -350,12 +350,12 @@ struct rte_event_eth_tx_adapter_stats {
>  #ifdef RTE_LIBRTE_EVENTDEV_DEBUG
>         if (dev_id >= RTE_EVENT_MAX_DEVS ||
>                 !rte_eventdevs[dev_id].attached) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
>         if (port_id >= dev->data->nb_ports) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>  #endif
> diff --git a/lib/librte_eventdev/rte_event_timer_adapter.c
> b/lib/librte_eventdev/rte_event_timer_adapter.c
> index 459bc47..5ce399e 100644
> --- a/lib/librte_eventdev/rte_event_timer_adapter.c
> +++ b/lib/librte_eventdev/rte_event_timer_adapter.c
> @@ -192,17 +192,17 @@ struct rte_event_timer_adapter *
>                                                    &adapter->data->caps,
>                                                    &adapter->ops);
>         if (ret < 0) {
> -               rte_errno = ret;
> +               rte_errno = -ret;
>                 goto free_memzone;
>         }
>
>         if (!(adapter->data->caps &
>               RTE_EVENT_TIMER_ADAPTER_CAP_INTERNAL_PORT)) {
> -               FUNC_PTR_OR_NULL_RET_WITH_ERRNO(conf_cb, -EINVAL);
> +               FUNC_PTR_OR_NULL_RET_WITH_ERRNO(conf_cb, EINVAL);
>                 ret = conf_cb(adapter->data->id,
> adapter->data->event_dev_id,
>                               &adapter->data->event_port_id, conf_arg);
>                 if (ret < 0) {
> -                       rte_errno = ret;
> +                       rte_errno = -ret;
>                         goto free_memzone;
>                 }
>         }
> @@ -214,10 +214,10 @@ struct rte_event_timer_adapter *
>                 adapter->ops = &swtim_ops;
>
>         /* Allow driver to do some setup */
> -       FUNC_PTR_OR_NULL_RET_WITH_ERRNO(adapter->ops->init, -ENOTSUP);
> +       FUNC_PTR_OR_NULL_RET_WITH_ERRNO(adapter->ops->init, ENOTSUP);
>         ret = adapter->ops->init(adapter);
>         if (ret < 0) {
> -               rte_errno = ret;
> +               rte_errno = -ret;
>                 goto free_memzone;
>         }
>
> @@ -509,11 +509,11 @@ struct event_buffer {
>         *nb_events_flushed = rte_event_enqueue_burst(dev_id, port_id,
>                                                      &events[tail_idx], n);
>         if (*nb_events_flushed != n) {
> -               if (rte_errno == -EINVAL) {
> +               if (rte_errno == EINVAL) {
>                         EVTIM_LOG_ERR("failed to enqueue invalid event - "
>                                       "dropping it");
>                         (*nb_events_inv)++;
> -               } else if (rte_errno == -ENOSPC)
> +               } else if (rte_errno == ENOSPC)
>                         rte_pause();
>         }
>
> @@ -832,7 +832,7 @@ struct swtim {
>         if (ret < 0) {
>                 if (ret != -EALREADY) {
>                         EVTIM_LOG_ERR("failed to initialize timer
> subsystem");
> -                       rte_errno = ret;
> +                       rte_errno = -ret;
>                         goto free_mempool;
>                 }
>         }
> @@ -840,7 +840,7 @@ struct swtim {
>         ret = rte_timer_data_alloc(&sw->timer_data_id);
>         if (ret < 0) {
>                 EVTIM_LOG_ERR("failed to allocate timer data instance");
> -               rte_errno = ret;
> +               rte_errno = -ret;
>                 goto free_mempool;
>         }
>
> diff --git a/lib/librte_eventdev/rte_eventdev.c
> b/lib/librte_eventdev/rte_eventdev.c
> index cc3199f..f44c869 100644
> --- a/lib/librte_eventdev/rte_eventdev.c
> +++ b/lib/librte_eventdev/rte_eventdev.c
> @@ -889,18 +889,18 @@
>         uint16_t *links_map;
>         int i, diag;
>
> -       RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, -EINVAL, 0);
> +       RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, EINVAL, 0);
>         dev = &rte_eventdevs[dev_id];
>
>         if (*dev->dev_ops->port_link == NULL) {
>                 RTE_EDEV_LOG_ERR("Function not supported\n");
> -               rte_errno = -ENOTSUP;
> +               rte_errno = ENOTSUP;
>                 return 0;
>         }
>
>         if (!is_valid_port(dev, port_id)) {
>                 RTE_EDEV_LOG_ERR("Invalid port_id=%" PRIu8, port_id);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
> @@ -921,7 +921,7 @@
>
>         for (i = 0; i < nb_links; i++)
>                 if (queues[i] >= dev->data->nb_queues) {
> -                       rte_errno = -EINVAL;
> +                       rte_errno = EINVAL;
>                         return 0;
>                 }
>
> @@ -948,18 +948,18 @@
>         int i, diag, j;
>         uint16_t *links_map;
>
> -       RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, -EINVAL, 0);
> +       RTE_EVENTDEV_VALID_DEVID_OR_ERRNO_RET(dev_id, EINVAL, 0);
>         dev = &rte_eventdevs[dev_id];
>
>         if (*dev->dev_ops->port_unlink == NULL) {
>                 RTE_EDEV_LOG_ERR("Function not supported");
> -               rte_errno = -ENOTSUP;
> +               rte_errno = ENOTSUP;
>                 return 0;
>         }
>
>         if (!is_valid_port(dev, port_id)) {
>                 RTE_EDEV_LOG_ERR("Invalid port_id=%" PRIu8, port_id);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
> @@ -988,7 +988,7 @@
>         nb_unlinks = j;
>         for (i = 0; i < nb_unlinks; i++)
>                 if (queues[i] >= dev->data->nb_queues) {
> -                       rte_errno = -EINVAL;
> +                       rte_errno = EINVAL;
>                         return 0;
>                 }
>
> diff --git a/lib/librte_eventdev/rte_eventdev.h
> b/lib/librte_eventdev/rte_eventdev.h
> index 927f43c..5044a13 100644
> --- a/lib/librte_eventdev/rte_eventdev.h
> +++ b/lib/librte_eventdev/rte_eventdev.h
> @@ -1318,12 +1318,12 @@ struct rte_eventdev {
>
>  #ifdef RTE_LIBRTE_EVENTDEV_DEBUG
>         if (dev_id >= RTE_EVENT_MAX_DEVS ||
> !rte_eventdevs[dev_id].attached) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
>         if (port_id >= dev->data->nb_ports) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>  #endif
> @@ -1372,10 +1372,10 @@ struct rte_eventdev {
>   *   *rte_event*. If the return value is less than *nb_events*, the
> remaining
>   *   events at the end of ev[] are not consumed and the caller has to
> take care
>   *   of them, and rte_errno is set accordingly. Possible errno values
> include:
> - *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's
> queue
> + *   - EINVAL   The port ID is invalid, device ID is invalid, an event's
> queue
>   *              ID is invalid, or an event's sched type doesn't match the
>   *              capabilities of the destination queue.
> - *   - -ENOSPC  The event port was backpressured and unable to enqueue
> + *   - ENOSPC   The event port was backpressured and unable to enqueue
>   *              one or more events. This error code is only applicable to
>   *              closed systems.
>   * @see rte_event_port_attr_get(), RTE_EVENT_PORT_ATTR_ENQ_DEPTH
> @@ -1422,10 +1422,10 @@ struct rte_eventdev {
>   *   *rte_event*. If the return value is less than *nb_events*, the
> remaining
>   *   events at the end of ev[] are not consumed and the caller has to
> take care
>   *   of them, and rte_errno is set accordingly. Possible errno values
> include:
> - *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's
> queue
> + *   - EINVAL   The port ID is invalid, device ID is invalid, an event's
> queue
>   *              ID is invalid, or an event's sched type doesn't match the
>   *              capabilities of the destination queue.
> - *   - -ENOSPC  The event port was backpressured and unable to enqueue
> + *   - ENOSPC   The event port was backpressured and unable to enqueue
>   *              one or more events. This error code is only applicable to
>   *              closed systems.
>   * @see rte_event_port_attr_get(), RTE_EVENT_PORT_ATTR_ENQ_DEPTH
> @@ -1473,10 +1473,10 @@ struct rte_eventdev {
>   *   *rte_event*. If the return value is less than *nb_events*, the
> remaining
>   *   events at the end of ev[] are not consumed and the caller has to
> take care
>   *   of them, and rte_errno is set accordingly. Possible errno values
> include:
> - *   - -EINVAL  The port ID is invalid, device ID is invalid, an event's
> queue
> + *   - EINVAL   The port ID is invalid, device ID is invalid, an event's
> queue
>   *              ID is invalid, or an event's sched type doesn't match the
>   *              capabilities of the destination queue.
> - *   - -ENOSPC  The event port was backpressured and unable to enqueue
> + *   - ENOSPC   The event port was backpressured and unable to enqueue
>   *              one or more events. This error code is only applicable to
>   *              closed systems.
>   * @see rte_event_port_attr_get(), RTE_EVENT_PORT_ATTR_ENQ_DEPTH
> @@ -1595,12 +1595,12 @@ struct rte_eventdev {
>
>  #ifdef RTE_LIBRTE_EVENTDEV_DEBUG
>         if (dev_id >= RTE_EVENT_MAX_DEVS ||
> !rte_eventdevs[dev_id].attached) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
>         if (port_id >= dev->data->nb_ports) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>  #endif
> @@ -1673,9 +1673,9 @@ struct rte_eventdev {
>   * of link[] are not established, and the caller has to take care of them.
>   * If return value is less than *nb_links* then implementation shall
> update the
>   * rte_errno accordingly, Possible rte_errno values are
> - * (-EDQUOT) Quota exceeded(Application tried to link the queue
> configured with
> + * (EDQUOT) Quota exceeded(Application tried to link the queue configured
> with
>   *  RTE_EVENT_QUEUE_CFG_SINGLE_LINK to more than one event ports)
> - * (-EINVAL) Invalid parameter
> + * (EINVAL) Invalid parameter
>   *
>   */
>  int
> @@ -1720,7 +1720,7 @@ struct rte_eventdev {
>   * end of queues[] are not unlinked, and the caller has to take care of
> them.
>   * If return value is less than *nb_unlinks* then implementation shall
> update
>   * the rte_errno accordingly, Possible rte_errno values are
> - * (-EINVAL) Invalid parameter
> + * (EINVAL) Invalid parameter
>   */
>  int
>  rte_event_port_unlink(uint8_t dev_id, uint8_t port_id,
> --
> 1.8.3.1
>
>
Reviewed-by: David Marchand <david.marchand@redhat.com>


-- 
David Marchand

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [PATCH 3/3] event/opdl: fix to set positive rte_errno
  2019-07-04 10:03 ` [dpdk-dev] [PATCH 3/3] event/opdl: " Andrew Rybchenko
@ 2019-07-04 11:41   ` David Marchand
  0 siblings, 0 replies; 10+ messages in thread
From: David Marchand @ 2019-07-04 11:41 UTC (permalink / raw)
  To: Andrew Rybchenko
  Cc: Jerin Jacob, Liang Ma, Peter Mccarthy, dev, Dilshod Urazov, dpdk stable

On Thu, Jul 4, 2019 at 12:04 PM Andrew Rybchenko <arybchenko@solarflare.com>
wrote:

> From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
>
> Fixes: 0bf298e39286 ("event/opdl: add event port config get/set")
> Fixes: 3c7f3dcfb099 ("event/opdl: add PMD main body and helper function")
> Fixes: 4236ce9bf5bf ("event/opdl: add OPDL ring infrastructure library")
> Cc: stable@dpdk.org
>
> Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>
> Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com>
> ---
>  drivers/event/opdl/opdl_evdev.c      | 10 +++++-----
>  drivers/event/opdl/opdl_evdev_init.c | 12 ++++++------
>  drivers/event/opdl/opdl_ring.c       |  2 +-
>  3 files changed, 12 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/event/opdl/opdl_evdev.c
> b/drivers/event/opdl/opdl_evdev.c
> index d2d2be4..3beca89 100644
> --- a/drivers/event/opdl/opdl_evdev.c
> +++ b/drivers/event/opdl/opdl_evdev.c
> @@ -102,7 +102,7 @@
>                              dev->data->dev_id,
>                                 queues[0],
>                                 p->id);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
> @@ -113,7 +113,7 @@
>                              dev->data->dev_id,
>                                 num,
>                                 p->id);
> -               rte_errno = -EDQUOT;
> +               rte_errno = EDQUOT;
>                 return 0;
>         }
>
> @@ -123,7 +123,7 @@
>                              dev->data->dev_id,
>                                 p->id,
>                                 queues[0]);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
> @@ -134,7 +134,7 @@
>                                 p->id,
>                                 p->external_qid,
>                                 queues[0]);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
> @@ -160,7 +160,7 @@
>                              dev->data->dev_id,
>                              queues[0],
>                              p->id);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>         RTE_SET_USED(nb_unlinks);
> diff --git a/drivers/event/opdl/opdl_evdev_init.c
> b/drivers/event/opdl/opdl_evdev_init.c
> index 582ad69..15aae47 100644
> --- a/drivers/event/opdl/opdl_evdev_init.c
> +++ b/drivers/event/opdl/opdl_evdev_init.c
> @@ -35,7 +35,7 @@
>                                              p->id,
>                                              ev[i].queue_id,
>                                              p->next_external_qid);
> -                               rte_errno = -EINVAL;
> +                               rte_errno = EINVAL;
>                                 return 0;
>                         }
>                 }
> @@ -63,7 +63,7 @@
>         } else {
>                 if (num > 0 &&
>                                 ev[0].queue_id != p->next_external_qid) {
> -                       rte_errno = -EINVAL;
> +                       rte_errno = EINVAL;
>                         return 0;
>                 }
>         }
> @@ -116,7 +116,7 @@
>         RTE_SET_USED(ev);
>         RTE_SET_USED(num);
>
> -       rte_errno = -ENOSPC;
> +       rte_errno = ENOSPC;
>
>         return 0;
>  }
> @@ -145,7 +145,7 @@
>
>
>         if (enqueued < num)
> -               rte_errno = -ENOSPC;
> +               rte_errno = ENOSPC;
>
>         return enqueued;
>  }
> @@ -164,7 +164,7 @@
>         RTE_SET_USED(ev);
>         RTE_SET_USED(num);
>
> -       rte_errno = -ENOSPC;
> +       rte_errno = ENOSPC;
>
>         return 0;
>  }
> @@ -240,7 +240,7 @@
>                              "Attempt to dequeue num of events larger than
> port (%d) max",
>                              opdl_pmd_dev_id(p->opdl),
>                              p->id);
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>
> diff --git a/drivers/event/opdl/opdl_ring.c
> b/drivers/event/opdl/opdl_ring.c
> index e988f1c..e8b29e2 100644
> --- a/drivers/event/opdl/opdl_ring.c
> +++ b/drivers/event/opdl/opdl_ring.c
> @@ -756,7 +756,7 @@ struct opdl_ring {
>  opdl_stage_disclaim(struct opdl_stage *s, uint32_t num_entries, bool
> block)
>  {
>         if (num_entries != s->num_event) {
> -               rte_errno = -EINVAL;
> +               rte_errno = EINVAL;
>                 return 0;
>         }
>         if (s->threadsafe == false) {
> --
> 1.8.3.1
>


Reviewed-by: David Marchand <david.marchand@redhat.com>

-- 
David Marchand

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [dpdk-stable] [PATCH 1/3] eventdev: fix to set positive rte_errno
  2019-07-04 11:34   ` [dpdk-dev] [dpdk-stable] " David Marchand
@ 2019-07-04 11:56     ` Andrew Rybchenko
  2019-07-04 11:58       ` David Marchand
  0 siblings, 1 reply; 10+ messages in thread
From: Andrew Rybchenko @ 2019-07-04 11:56 UTC (permalink / raw)
  To: David Marchand
  Cc: Jerin Jacob, Nikhil Rao, Erik Gabriel Carrillo, dev,
	Dilshod Urazov, dpdk stable

On 04.07.2019 14:34, David Marchand wrote:
>
> We have fixes for different releases, expect the stable maintainers to 
> ask for help :-)
>
>
> On Thu, Jul 4, 2019 at 12:04 PM Andrew Rybchenko 
> <arybchenko@solarflare.com <mailto:arybchenko@solarflare.com>> wrote:
>
>     From: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru
>     <mailto:Dilshod.Urazov@oktetlabs.ru>>
>
>     Fixes: c9bf83947e2e ("eventdev: add eth Tx adapter APIs")
>
>
> v18.11-rc1~724
>
>     Fixes: 47d05b292820 ("eventdev: add timer adapter common code")
>
>
> v18.05-rc1~462
>
>     Fixes: 6750b21bd6af ("eventdev: add default software timer adapter")
>
>
> v18.05-rc1~460
>
>     Fixes: c75f7897ea35 ("eventdev: set error code in port link/unlink
>     functions")
>
>
> v18.02-rc1~202
>
>     Fixes: 7d1acc9dde93 ("eventdev: introduce helper function for
>     enqueue burst")
>
>
> v17.08-rc1~90
>
>     Fixes: 406aed4e0dd9 ("eventdev: add errno-style return values")
>
>
> v17.05-rc1~166
>
>     Fixes: c64e1b7b20d2 ("eventdev: add new software event timer adapter")
>
>
> This last sha1 c64e1b7b20d2 is unknown to my git.
> Please, can you double check?

The patch is for dpdk-next-eventdev tree and this changeset was ther
(but not in main dpdk repo yet).


>
>     Cc: stable@dpdk.org <mailto:stable@dpdk.org>
>
>     Signed-off-by: Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru
>     <mailto:Dilshod.Urazov@oktetlabs.ru>>
>     Signed-off-by: Andrew Rybchenko <arybchenko@solarflare.com
>     <mailto:arybchenko@solarflare.com>>
>

[...]

>
> Reviewed-by: David Marchand <david.marchand@redhat.com 
> <mailto:david.marchand@redhat.com>>

Many thanks for review.



^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [dpdk-stable] [PATCH 1/3] eventdev: fix to set positive rte_errno
  2019-07-04 11:56     ` Andrew Rybchenko
@ 2019-07-04 11:58       ` David Marchand
  2019-07-04 12:13         ` [dpdk-dev] [EXT] " Jerin Jacob Kollanukkaran
  0 siblings, 1 reply; 10+ messages in thread
From: David Marchand @ 2019-07-04 11:58 UTC (permalink / raw)
  To: Andrew Rybchenko
  Cc: Jerin Jacob, Nikhil Rao, Erik Gabriel Carrillo, dev,
	Dilshod Urazov, dpdk stable

On Thu, Jul 4, 2019 at 1:56 PM Andrew Rybchenko <arybchenko@solarflare.com>
wrote:

> On 04.07.2019 14:34, David Marchand wrote:
> >     Fixes: c64e1b7b20d2 ("eventdev: add new software event timer
> adapter")
> >
> >
> > This last sha1 c64e1b7b20d2 is unknown to my git.
> > Please, can you double check?
>
> The patch is for dpdk-next-eventdev tree and this changeset was ther
> (but not in main dpdk repo yet).
>

If it's still in the eventdev tree, how about fixing it now before the pull?


-- 
David Marchand

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH 1/3] eventdev: fix to set positive rte_errno
  2019-07-04 11:58       ` David Marchand
@ 2019-07-04 12:13         ` Jerin Jacob Kollanukkaran
  0 siblings, 0 replies; 10+ messages in thread
From: Jerin Jacob Kollanukkaran @ 2019-07-04 12:13 UTC (permalink / raw)
  To: David Marchand, Andrew Rybchenko
  Cc: Nikhil Rao, Erik Gabriel Carrillo, dev, Dilshod Urazov, dpdk stable


From: David Marchand <david.marchand@redhat.com>
Sent: Thursday, July 4, 2019 5:29 PM
To: Andrew Rybchenko <arybchenko@solarflare.com>
Cc: Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Nikhil Rao <nikhil.rao@intel.com>; Erik Gabriel Carrillo <erik.g.carrillo@intel.com>; dev <dev@dpdk.org>; Dilshod Urazov <Dilshod.Urazov@oktetlabs.ru>; dpdk stable <stable@dpdk.org>
Subject: [EXT] Re: [dpdk-stable] [PATCH 1/3] eventdev: fix to set positive rte_errno


On Thu, Jul 4, 2019 at 1:56 PM Andrew Rybchenko <arybchenko@solarflare.com<mailto:arybchenko@solarflare.com>> wrote:
On 04.07.2019 14:34, David Marchand wrote:
>     Fixes: c64e1b7b20d2 ("eventdev: add new software event timer adapter")
>
>
> This last sha1 c64e1b7b20d2 is unknown to my git.
> Please, can you double check?

The patch is for dpdk-next-eventdev tree and this changeset was ther
(but not in main dpdk repo yet).

If it's still in the eventdev tree, how about fixing it now before the pull?

[Jerin] it is not in eventdev tree(the patches got merged). I will fix sha id on apply. No need to send separate patch for it


--
David Marchand

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2019-07-04 12:14 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-07-04 10:03 [dpdk-dev] [PATCH 0/3] eventdev fixes to set positive rte_errno Andrew Rybchenko
2019-07-04 10:03 ` [dpdk-dev] [PATCH 1/3] eventdev: fix " Andrew Rybchenko
2019-07-04 11:34   ` [dpdk-dev] [dpdk-stable] " David Marchand
2019-07-04 11:56     ` Andrew Rybchenko
2019-07-04 11:58       ` David Marchand
2019-07-04 12:13         ` [dpdk-dev] [EXT] " Jerin Jacob Kollanukkaran
2019-07-04 10:03 ` [dpdk-dev] [PATCH 2/3] event/sw: " Andrew Rybchenko
2019-07-04 10:43   ` Van Haaren, Harry
2019-07-04 10:03 ` [dpdk-dev] [PATCH 3/3] event/opdl: " Andrew Rybchenko
2019-07-04 11:41   ` David Marchand

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).