d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

pcap: fix build error introduced by kvargs

Browse Source 
Due to a merge conflict between commits 4c745617a1 and 9d5752d80,
rte_eth_pcap.c was not compiling with the following error:

rte_eth_pcap.c: In function 'rte_pmd_init_internals':
rte_eth_pcap.c:559:30: error: dereferencing pointer to incomplete type
rte_eth_pcap.c:560:15: error: dereferencing pointer to incomplete type
rte_eth_pcap.c:561:18: error: dereferencing pointer to incomplete type
rte_eth_pcap.c:603:47: error: dereferencing pointer to incomplete type
rte_eth_pcap.c: In function 'rte_pmd_pcap_init':
rte_eth_pcap.c:732:73: error: 'dict' undeclared (first use in this
  function)
rte_eth_pcap.c:732:73: note: each undeclared identifier is reported
  only once for each function it appears in

This commit replaces "struct args_dict" by "struct rte_kvargs" to fix
the compilation issue.

By the way, it also removes the declaration of these functions from
the header file as no other file in DPDK references one of them. It
avoids to include <rte_kvargs.h> in rte_eth_pcap.h.

Reported-by: Meir Tseitlin <mirots@gmail.com>
Signed-off-by: Olivier Matz <olivier.matz@6wind.com>
Acked-by: Thomas Monjalon <thomas.monjalon@6wind.com>
This commit is contained in:
Olivier Matz 2014-03-02 22:51:28 +01:00 committed by Thomas Monjalon
parent f95b372558
commit 266ffe3494
2 changed files with 14 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
lib/librte_pmd_pcap/rte_eth_pcap.c
View File

@ -549,15 +549,15 @@ rte_pmd_init_internals(const unsigned nb_rx_queues,
const unsigned numa_node,
struct pmd_internals **internals,
struct rte_eth_dev **eth_dev,
struct args_dict *dict)
struct rte_kvargs *kvlist)
{
struct rte_eth_dev_data *data = NULL;
struct rte_pci_device *pci_dev = NULL;
unsigned k_idx;
struct key_value *pair = NULL;
struct rte_kvargs_pair *pair = NULL;
for (k_idx = 0; k_idx < dict->index; k_idx++) {
pair = &dict->pairs[k_idx];
for (k_idx = 0; k_idx < kvlist->count; k_idx++) {
pair = &kvlist->pairs[k_idx];
if (strstr(pair->key, ETH_PCAP_IFACE_ARG) != NULL)
break;
}
@ -626,13 +626,13 @@ rte_pmd_init_internals(const unsigned nb_rx_queues,
return -1;
}
int
static int
rte_eth_from_pcaps_n_dumpers(pcap_t * const rx_queues[],
const unsigned nb_rx_queues,
pcap_dumper_t * const tx_queues[],
const unsigned nb_tx_queues,
const unsigned numa_node,
struct args_dict *dict)
struct rte_kvargs *kvlist)
{
struct pmd_internals *internals = NULL;
struct rte_eth_dev *eth_dev = NULL;
@ -645,7 +645,7 @@ rte_eth_from_pcaps_n_dumpers(pcap_t * const rx_queues[],
return -1;
if (rte_pmd_init_internals(nb_rx_queues, nb_tx_queues, numa_node,
&internals, &eth_dev, dict) < 0)
&internals, &eth_dev, kvlist) < 0)
return -1;
for (i = 0; i < nb_rx_queues; i++) {
@ -661,13 +661,13 @@ rte_eth_from_pcaps_n_dumpers(pcap_t * const rx_queues[],
return 0;
}
int
static int
rte_eth_from_pcaps(pcap_t * const rx_queues[],
const unsigned nb_rx_queues,
pcap_t * const tx_queues[],
const unsigned nb_tx_queues,
const unsigned numa_node,
struct args_dict *dict)
struct rte_kvargs *kvlist)
{
struct pmd_internals *internals = NULL;
struct rte_eth_dev *eth_dev = NULL;
@ -680,7 +680,7 @@ rte_eth_from_pcaps(pcap_t * const rx_queues[],
return -1;
if (rte_pmd_init_internals(nb_rx_queues, nb_tx_queues, numa_node,
&internals, &eth_dev, dict) < 0)
&internals, &eth_dev, kvlist) < 0)
return -1;
for (i = 0; i < nb_rx_queues; i++) {
@ -729,7 +729,8 @@ rte_pmd_pcap_init(const char *name, const char *params)
if (ret < 0)
return -1;
return rte_eth_from_pcaps(pcaps.pcaps, 1, pcaps.pcaps, 1, numa_node, &dict);
return rte_eth_from_pcaps(pcaps.pcaps, 1, pcaps.pcaps, 1,
numa_node, kvlist);
}
/*
@ -770,10 +771,10 @@ rte_pmd_pcap_init(const char *name, const char *params)
if (using_dumpers)
return rte_eth_from_pcaps_n_dumpers(pcaps.pcaps, pcaps.num_of_rx,
dumpers.dumpers, dumpers.num_of_tx, numa_node, &dict);
dumpers.dumpers, dumpers.num_of_tx, numa_node, kvlist);
return rte_eth_from_pcaps(pcaps.pcaps, pcaps.num_of_rx, dumpers.pcaps,
dumpers.num_of_tx, numa_node, &dict);
dumpers.num_of_tx, numa_node, kvlist);
}

17
lib/librte_pmd_pcap/rte_eth_pcap.h
View File

@ -47,23 +47,6 @@ extern "C" {
#define RTE_ETH_PCAP_PARAM_NAME "eth_pcap"
/* struct args_dict is declared in rte_eth_pcap_args_parser.h */
struct args_dict;
int rte_eth_from_pcaps(pcap_t * const rx_queues[],
const unsigned nb_rx_queues,
pcap_t * const tx_queues[],
const unsigned nb_tx_queues,
const unsigned numa_node,
struct args_dict *dict);
int rte_eth_from_pcaps_n_dumpers(pcap_t * const rx_queues[],
const unsigned nb_rx_queues,
pcap_dumper_t * const tx_queues[],
const unsigned nb_tx_queues,
const unsigned numa_node,
struct args_dict *dict);
/**
* For use by the EAL only. Called as part of EAL init to set up any dummy NICs
* configured on command line.