[CLOUD-DEV,v3,8/9] linux-gen: pktio: pcap: use generic pktio_ops data storage

Message ID 1507636815-18816-9-git-send-email-odpbot@yandex.ru
State Superseded
Headers show
Series
  • Cloud dev pktio ops data pr
Related show

Commit Message

Github ODP bot Oct. 10, 2017, noon
From: Bogdan Pricope <bogdan.pricope@linaro.org>


Signed-off-by: Bogdan Pricope <bogdan.pricope@linaro.org>

---
/** Email created from pull request 196 (bogdanPricope:cloud_dev_pktio_ops_data_pr)
 ** https://github.com/Linaro/odp/pull/196
 ** Patch: https://github.com/Linaro/odp/pull/196.patch
 ** Base sha: 3d3c8f71f39ff139695d6f4b8e5ea17502c5f7cf
 ** Merge commit sha: 2bae88983ff377d83fcf9c99bf5c380275454327
 **/
 .../linux-generic/include/odp_pktio_ops_subsystem.h   |  4 ----
 platform/linux-generic/pktio/pcap.c                   | 19 +++++++++----------
 2 files changed, 9 insertions(+), 14 deletions(-)

Patch

diff --git a/platform/linux-generic/include/odp_pktio_ops_subsystem.h b/platform/linux-generic/include/odp_pktio_ops_subsystem.h
index ece4d46e0..5449eccbd 100644
--- a/platform/linux-generic/include/odp_pktio_ops_subsystem.h
+++ b/platform/linux-generic/include/odp_pktio_ops_subsystem.h
@@ -78,15 +78,11 @@  typedef ODP_MODULE_CLASS(pktio_ops) {
 	odp_api_proto(pktio_ops, print) print;
 } pktio_ops_module_t;
 
-/* All implementations of this subsystem */
-#include <odp_pktio_ops_pcap.h>
-
 /* Per implementation private data
  * TODO: refactory each implementation to hide it internally
  */
 typedef union {
 	void *dpdk;
-	pktio_ops_pcap_data_t pcap;
 } pktio_ops_data_t;
 
 /* Extract pktio ops data from pktio entry structure */
diff --git a/platform/linux-generic/pktio/pcap.c b/platform/linux-generic/pktio/pcap.c
index 7808242f5..acadbd87d 100644
--- a/platform/linux-generic/pktio/pcap.c
+++ b/platform/linux-generic/pktio/pcap.c
@@ -41,6 +41,7 @@ 
 #include <odp_api.h>
 #include <odp_packet_internal.h>
 #include <odp_packet_io_internal.h>
+#include <odp_pktio_ops_pcap.h>
 
 #include <protocols/eth.h>
 
@@ -139,8 +140,7 @@  static int _pcapif_init_tx(pktio_ops_pcap_data_t *pcap)
 static int pcapif_init(odp_pktio_t id ODP_UNUSED, pktio_entry_t *pktio_entry,
 		       const char *devname, odp_pool_t pool)
 {
-	pktio_ops_pcap_data_t *pcap =
-		&pktio_entry->ops_data(pcap);
+	pktio_ops_pcap_data_t *pcap = odp_ops_data(pktio_entry, pcap);
 	int ret;
 
 	memset(pcap, 0, sizeof(pktio_ops_pcap_data_t));
@@ -167,7 +167,7 @@  static int pcapif_init(odp_pktio_t id ODP_UNUSED, pktio_entry_t *pktio_entry,
 
 static int pcapif_close(pktio_entry_t *pktio_entry)
 {
-	pktio_ops_pcap_data_t *pcap = &pktio_entry->ops_data(pcap);
+	pktio_ops_pcap_data_t *pcap = odp_ops_data(pktio_entry, pcap);
 
 	if (pcap->tx_dump)
 		pcap_dump_close(pcap->tx_dump);
@@ -214,8 +214,7 @@  static int pcapif_recv_pkt(pktio_entry_t *pktio_entry, int index ODP_UNUSED,
 	odp_packet_t pkt;
 	odp_packet_hdr_t *pkt_hdr;
 	uint32_t pkt_len;
-	pktio_ops_pcap_data_t *pcap =
-		&pktio_entry->ops_data(pcap);
+	pktio_ops_pcap_data_t *pcap = odp_ops_data(pktio_entry, pcap);
 	odp_time_t ts_val;
 	odp_time_t *ts = NULL;
 
@@ -298,8 +297,7 @@  static int _pcapif_dump_pkt(pktio_ops_pcap_data_t *pcap, odp_packet_t pkt)
 static int pcapif_send_pkt(pktio_entry_t *pktio_entry, int index ODP_UNUSED,
 			   const odp_packet_t pkts[], int len)
 {
-	pktio_ops_pcap_data_t *pcap =
-		&pktio_entry->ops_data(pcap);
+	pktio_ops_pcap_data_t *pcap = odp_ops_data(pktio_entry, pcap);
 	int i;
 
 	odp_ticketlock_lock(&pktio_entry->s.txl);
@@ -367,8 +365,7 @@  static int pcapif_promisc_mode_set(pktio_entry_t *pktio_entry,
 {
 	char filter_exp[64] = {0};
 	struct bpf_program bpf;
-	pktio_ops_pcap_data_t *pcap =
-		&pktio_entry->ops_data(pcap);
+	pktio_ops_pcap_data_t *pcap = odp_ops_data(pktio_entry, pcap);
 
 	if (!pcap->rx) {
 		pcap->promisc = enable;
@@ -408,7 +405,9 @@  static int pcapif_promisc_mode_set(pktio_entry_t *pktio_entry,
 
 static int pcapif_promisc_mode_get(pktio_entry_t *pktio_entry)
 {
-	return pktio_entry->ops_data(pcap).promisc;
+	pktio_ops_pcap_data_t *pcap = odp_ops_data(pktio_entry, pcap);
+
+	return pcap->promisc;
 }
 
 static int pcapif_stats_reset(pktio_entry_t *pktio_entry)