[dpdk-dev] eventdev/eth_rx_adapter: update the note field
Checks
Commit Message
rte_event_eth_rx_adapter_create allocates eth_devices for currently
available eth devices. For dynamically created eth devices a new instance
for rx adapter has to be created.
Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
---
lib/librte_eventdev/rte_event_eth_rx_adapter.h | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
Comments
Hi Vipin,
Thanks for the patch, I have a suggestion below.
On 5/17/2018 1:49 PM, Vipin Varghese wrote:
> rte_event_eth_rx_adapter_create allocates eth_devices for currently
> available eth devices. For dynamically created eth devices a new instance
> for rx adapter has to be created.
>
> Signed-off-by: Vipin Varghese <vipin.varghese@intel.com>
> ---
> lib/librte_eventdev/rte_event_eth_rx_adapter.h | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/lib/librte_eventdev/rte_event_eth_rx_adapter.h b/lib/librte_eventdev/rte_event_eth_rx_adapter.h
> index 834eb53..0ef18ce 100644
> --- a/lib/librte_eventdev/rte_event_eth_rx_adapter.h
> +++ b/lib/librte_eventdev/rte_event_eth_rx_adapter.h
> @@ -63,7 +63,10 @@
> * rte_event_eth_rx_adapter_service_id_get() function can be used to retrieve
> * the service function ID of the adapter in this case.
> *
> - * Note: Interrupt driven receive queues are currently unimplemented.
> + * Note:
> + * 1) Interrupt driven receive queues are currently unimplemented.
> + * 2) rte_event_eth_rx_adapter_create() is not accounted for dynamic
> + * eth devices.
> */
Reword:
2) Devices created after calling rte_event_eth_rx_adapter_create()
should be added to a new instance of the rx adapter.
>
> #ifdef __cplusplus
With the change above
Acked-by: Nikhil Rao <nikhil.rao@intel.com>
Thanks Nikhil, I have made the changes and send v2
From: Rao, Nikhil
Sent: Friday, May 18, 2018 11:01 AM
To: Varghese, Vipin <vipin.varghese@intel.com>; dev@dpdk.org; Jain, Deepak K <deepak.k.jain@intel.com>
Cc: Rao, Nikhil <nikhil.rao@intel.com>
Subject: Re: [PATCH] eventdev/eth_rx_adapter: update the note field
Hi Vipin,
Thanks for the patch, I have a suggestion below.
On 5/17/2018 1:49 PM, Vipin Varghese wrote:
rte_event_eth_rx_adapter_create allocates eth_devices for currently
available eth devices. For dynamically created eth devices a new instance
for rx adapter has to be created.
Signed-off-by: Vipin Varghese <vipin.varghese@intel.com><mailto:vipin.varghese@intel.com>
---
lib/librte_eventdev/rte_event_eth_rx_adapter.h | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/lib/librte_eventdev/rte_event_eth_rx_adapter.h b/lib/librte_eventdev/rte_event_eth_rx_adapter.h
index 834eb53..0ef18ce 100644
--- a/lib/librte_eventdev/rte_event_eth_rx_adapter.h
+++ b/lib/librte_eventdev/rte_event_eth_rx_adapter.h
@@ -63,7 +63,10 @@
* rte_event_eth_rx_adapter_service_id_get() function can be used to retrieve
* the service function ID of the adapter in this case.
*
- * Note: Interrupt driven receive queues are currently unimplemented.
+ * Note:
+ * 1) Interrupt driven receive queues are currently unimplemented.
+ * 2) rte_event_eth_rx_adapter_create() is not accounted for dynamic
+ * eth devices.
*/
Reword:
2) Devices created after calling rte_event_eth_rx_adapter_create() should be added to a new instance of the rx adapter.
#ifdef __cplusplus
With the change above
Acked-by: Nikhil Rao <nikhil.rao@intel.com><mailto:nikhil.rao@intel.com>
@@ -63,7 +63,10 @@
* rte_event_eth_rx_adapter_service_id_get() function can be used to retrieve
* the service function ID of the adapter in this case.
*
- * Note: Interrupt driven receive queues are currently unimplemented.
+ * Note:
+ * 1) Interrupt driven receive queues are currently unimplemented.
+ * 2) rte_event_eth_rx_adapter_create() is not accounted for dynamic
+ * eth devices.
*/
#ifdef __cplusplus