Re: [lng-odp] [API-NEXT PATCH] api: pktio: added drv_name to pktio info structure

[Maxim Uvarov]

Merged,
Maxim.

On 03/01/16 16:19, Juha-Matti Tilli wrote:

Add drv_name to pktio info structure. This allows applications to see which
pktio driver is used. For example, an application might want to print a
message with netmap or DPDK installation instructions if it seems that
sockets are used. The pktio driver information is also useful information
to be stored to a log file so that debugging problems is easier.

Signed-off-by: Juha-Matti Tilli 
Reviewed-by: Petri Savolainen 
---
  include/odp/api/spec/packet_io.h   | 1 +
  platform/linux-generic/odp_packet_io.c | 1 +
  2 files changed, 2 insertions(+)

diff --git a/include/odp/api/spec/packet_io.h b/include/odp/api/spec/packet_io.h
index ce3308a..302d431 100644
--- a/include/odp/api/spec/packet_io.h
+++ b/include/odp/api/spec/packet_io.h
@@ -716,6 +716,7 @@ int odp_pktio_link_status(odp_pktio_t pktio);
   */
  typedef struct odp_pktio_info_t {
const char   *name;  /**< Packet IO device name */
+   const char   *drv_name; /**< Packet IO driver name (implementation 
specific) */
odp_pool_tpool;  /**< Packet pool */
odp_pktio_param_t param; /**< Packet IO parameters */
  } odp_pktio_info_t;
diff --git a/platform/linux-generic/odp_packet_io.c 
b/platform/linux-generic/odp_packet_io.c
index 513cfcb..4ba9ab9 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -861,6 +861,7 @@ int odp_pktio_info(odp_pktio_t id, odp_pktio_info_t *info)
  
  	memset(info, 0, sizeof(odp_pktio_info_t));

info->name = entry->s.name;
+   info->drv_name = entry->s.ops->name;
info->pool = entry->s.pool;
memcpy(>param, >s.param, sizeof(odp_pktio_param_t));
  


___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

[lng-odp] [API-NEXT PATCH] api: pktio: added drv_name to pktio info structure

[Juha-Matti Tilli]

Add drv_name to pktio info structure. This allows applications to see which
pktio driver is used. For example, an application might want to print a
message with netmap or DPDK installation instructions if it seems that
sockets are used. The pktio driver information is also useful information
to be stored to a log file so that debugging problems is easier.

Signed-off-by: Juha-Matti Tilli 
Reviewed-by: Petri Savolainen 
---
 include/odp/api/spec/packet_io.h   | 1 +
 platform/linux-generic/odp_packet_io.c | 1 +
 2 files changed, 2 insertions(+)

diff --git a/include/odp/api/spec/packet_io.h b/include/odp/api/spec/packet_io.h
index ce3308a..302d431 100644
--- a/include/odp/api/spec/packet_io.h
+++ b/include/odp/api/spec/packet_io.h
@@ -716,6 +716,7 @@ int odp_pktio_link_status(odp_pktio_t pktio);
  */
 typedef struct odp_pktio_info_t {
const char   *name;  /**< Packet IO device name */
+   const char   *drv_name; /**< Packet IO driver name (implementation 
specific) */
odp_pool_tpool;  /**< Packet pool */
odp_pktio_param_t param; /**< Packet IO parameters */
 } odp_pktio_info_t;
diff --git a/platform/linux-generic/odp_packet_io.c 
b/platform/linux-generic/odp_packet_io.c
index 513cfcb..4ba9ab9 100644
--- a/platform/linux-generic/odp_packet_io.c
+++ b/platform/linux-generic/odp_packet_io.c
@@ -861,6 +861,7 @@ int odp_pktio_info(odp_pktio_t id, odp_pktio_info_t *info)
 
memset(info, 0, sizeof(odp_pktio_info_t));
info->name = entry->s.name;
+   info->drv_name = entry->s.ops->name;
info->pool = entry->s.pool;
memcpy(>param, >s.param, sizeof(odp_pktio_param_t));
 
-- 
2.1.4

___
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp