* [dpdk-dev] [PATCH 1/2] eal: remove zombie symbols
@ 2015-12-15 10:47 Thomas Monjalon
2015-12-15 10:47 ` [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols Thomas Monjalon
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2015-12-15 10:47 UTC (permalink / raw)
To: dev
test_mp_secondary was initially added by mistake.
rte_snprintf has been removed.
Fixes: 9d41beed24b0 ("lib: provide initial versioning")
Fixes: 3185322809c1 ("eal: remove rte_snprintf")
Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
doc/guides/sample_app_ug/exception_path.rst | 2 +-
doc/guides/sample_app_ug/ip_reassembly.rst | 2 +-
doc/guides/sample_app_ug/kernel_nic_interface.rst | 6 +++---
doc/guides/sample_app_ug/l3_forward.rst | 2 +-
examples/l3fwd-power/main.c | 4 ++--
lib/librte_eal/bsdapp/eal/rte_eal_version.map | 2 --
lib/librte_eal/linuxapp/eal/rte_eal_version.map | 2 --
7 files changed, 8 insertions(+), 12 deletions(-)
diff --git a/doc/guides/sample_app_ug/exception_path.rst b/doc/guides/sample_app_ug/exception_path.rst
index 3cc7cbe..c3f8f88 100644
--- a/doc/guides/sample_app_ug/exception_path.rst
+++ b/doc/guides/sample_app_ug/exception_path.rst
@@ -173,7 +173,7 @@ The code for creating the TAP interface is as follows:
}
if (name)
- rte_snprintf(name, IFNAMSIZ, ifr.ifr_name);
+ snprintf(name, IFNAMSIZ, ifr.ifr_name);
return fd;
}
diff --git a/doc/guides/sample_app_ug/ip_reassembly.rst b/doc/guides/sample_app_ug/ip_reassembly.rst
index 050802a..6bf5938 100644
--- a/doc/guides/sample_app_ug/ip_reassembly.rst
+++ b/doc/guides/sample_app_ug/ip_reassembly.rst
@@ -223,7 +223,7 @@ each RX queue uses its own mempool.
nb_mbuf += RTE_TEST_RX_DESC_DEFAULT + RTE_TEST_TX_DESC_DEFAULT;
nb_mbuf = RTE_MAX(nb_mbuf, (uint32_t)NB_MBUF);
- rte_snprintf(buf, sizeof(buf), "mbuf_pool_%u_%u", lcore, queue);
+ snprintf(buf, sizeof(buf), "mbuf_pool_%u_%u", lcore, queue);
if ((rxq->pool = rte_mempool_create(buf, nb_mbuf, MBUF_SIZE, 0, sizeof(struct rte_pktmbuf_pool_private), rte_pktmbuf_pool_init, NULL,
rte_pktmbuf_init, NULL, socket, MEMPOOL_F_SP_PUT | MEMPOOL_F_SC_GET)) == NULL) {
diff --git a/doc/guides/sample_app_ug/kernel_nic_interface.rst b/doc/guides/sample_app_ug/kernel_nic_interface.rst
index c8fc10a..985c664 100644
--- a/doc/guides/sample_app_ug/kernel_nic_interface.rst
+++ b/doc/guides/sample_app_ug/kernel_nic_interface.rst
@@ -265,11 +265,11 @@ The code for allocating the kernel NIC interfaces for a specific port is as foll
memset(&conf, 0, sizeof(conf));
if (params[port_id]->nb_lcore_k) {
- rte_snprintf(conf.name, RTE_KNI_NAMESIZE, "vEth%u_%u", port_id, i);
+ snprintf(conf.name, RTE_KNI_NAMESIZE, "vEth%u_%u", port_id, i);
conf.core_id = params[port_id]->lcore_k[i];
conf.force_bind = 1;
} else
- rte_snprintf(conf.name, RTE_KNI_NAMESIZE, "vEth%u", port_id);
+ snprintf(conf.name, RTE_KNI_NAMESIZE, "vEth%u", port_id);
conf.group_id = (uint16_t)port_id;
conf.mbuf_size = MAX_PACKET_SZ;
@@ -352,7 +352,7 @@ The code is as follows:
goto fail;
}
- rte_snprintf(s, sizeof(s), "%.*s", size, p);
+ snprintf(s, sizeof(s), "%.*s", size, p);
nb_token = rte_strsplit(s, sizeof(s), str_fld, _NUM_FLD, ',');
if (nb_token <= FLD_LCORE_TX) {
diff --git a/doc/guides/sample_app_ug/l3_forward.rst b/doc/guides/sample_app_ug/l3_forward.rst
index 34a84f1..4ce734b 100644
--- a/doc/guides/sample_app_ug/l3_forward.rst
+++ b/doc/guides/sample_app_ug/l3_forward.rst
@@ -233,7 +233,7 @@ The LPM object is created and loaded with the pre-configured entries read from a
/* create the LPM table */
- rte_snprintf(s, sizeof(s), "IPV4_L3FWD_LPM_%d", socketid);
+ snprintf(s, sizeof(s), "IPV4_L3FWD_LPM_%d", socketid);
ipv4_l3fwd_lookup_struct[socketid] = rte_lpm_create(s, socketid, IPV4_L3FWD_LPM_MAX_RULES, 0);
diff --git a/examples/l3fwd-power/main.c b/examples/l3fwd-power/main.c
index 9175989..828c18a 100644
--- a/examples/l3fwd-power/main.c
+++ b/examples/l3fwd-power/main.c
@@ -1373,7 +1373,7 @@ setup_hash(int socketid)
char s[64];
/* create ipv4 hash */
- rte_snprintf(s, sizeof(s), "ipv4_l3fwd_hash_%d", socketid);
+ snprintf(s, sizeof(s), "ipv4_l3fwd_hash_%d", socketid);
ipv4_l3fwd_hash_params.name = s;
ipv4_l3fwd_hash_params.socket_id = socketid;
ipv4_l3fwd_lookup_struct[socketid] =
@@ -1383,7 +1383,7 @@ setup_hash(int socketid)
"socket %d\n", socketid);
/* create ipv6 hash */
- rte_snprintf(s, sizeof(s), "ipv6_l3fwd_hash_%d", socketid);
+ snprintf(s, sizeof(s), "ipv6_l3fwd_hash_%d", socketid);
ipv6_l3fwd_hash_params.name = s;
ipv6_l3fwd_hash_params.socket_id = socketid;
ipv6_l3fwd_lookup_struct[socketid] =
diff --git a/lib/librte_eal/bsdapp/eal/rte_eal_version.map b/lib/librte_eal/bsdapp/eal/rte_eal_version.map
index 7a88387..9d7adf1 100644
--- a/lib/librte_eal/bsdapp/eal/rte_eal_version.map
+++ b/lib/librte_eal/bsdapp/eal/rte_eal_version.map
@@ -94,7 +94,6 @@ DPDK_2.0 {
rte_set_application_usage_hook;
rte_set_log_level;
rte_set_log_type;
- rte_snprintf;
rte_socket_id;
rte_strerror;
rte_strsplit;
@@ -106,7 +105,6 @@ DPDK_2.0 {
rte_xen_dom0_memory_init;
rte_zmalloc;
rte_zmalloc_socket;
- test_mp_secondary;
local: *;
};
diff --git a/lib/librte_eal/linuxapp/eal/rte_eal_version.map b/lib/librte_eal/linuxapp/eal/rte_eal_version.map
index 828496f..cbe175f 100644
--- a/lib/librte_eal/linuxapp/eal/rte_eal_version.map
+++ b/lib/librte_eal/linuxapp/eal/rte_eal_version.map
@@ -94,7 +94,6 @@ DPDK_2.0 {
rte_set_application_usage_hook;
rte_set_log_level;
rte_set_log_type;
- rte_snprintf;
rte_socket_id;
rte_strerror;
rte_strsplit;
@@ -106,7 +105,6 @@ DPDK_2.0 {
rte_xen_dom0_memory_init;
rte_zmalloc;
rte_zmalloc_socket;
- test_mp_secondary;
local: *;
};
--
2.5.2
^ permalink raw reply [flat|nested] 6+ messages in thread* [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols
2015-12-15 10:47 [dpdk-dev] [PATCH 1/2] eal: remove zombie symbols Thomas Monjalon
@ 2015-12-15 10:47 ` Thomas Monjalon
2015-12-15 11:41 ` Panu Matilainen
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Monjalon @ 2015-12-15 10:47 UTC (permalink / raw)
To: dev
The API has been removed but the symbols were still declared in the map.
Fixes: a421b86a4a02 ("ethdev: remove old flow director API")
Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
---
lib/librte_ether/rte_ether_version.map | 8 --------
1 file changed, 8 deletions(-)
diff --git a/lib/librte_ether/rte_ether_version.map b/lib/librte_ether/rte_ether_version.map
index 17a11c7..d8db24d 100644
--- a/lib/librte_ether/rte_ether_version.map
+++ b/lib/librte_ether/rte_ether_version.map
@@ -27,14 +27,6 @@ DPDK_2.2 {
rte_eth_dev_count;
rte_eth_dev_default_mac_addr_set;
rte_eth_dev_detach;
- rte_eth_dev_fdir_add_perfect_filter;
- rte_eth_dev_fdir_add_signature_filter;
- rte_eth_dev_fdir_get_infos;
- rte_eth_dev_fdir_remove_perfect_filter;
- rte_eth_dev_fdir_remove_signature_filter;
- rte_eth_dev_fdir_set_masks;
- rte_eth_dev_fdir_update_perfect_filter;
- rte_eth_dev_fdir_update_signature_filter;
rte_eth_dev_filter_ctrl;
rte_eth_dev_filter_supported;
rte_eth_dev_flow_ctrl_get;
--
2.5.2
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols
2015-12-15 10:47 ` [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols Thomas Monjalon
@ 2015-12-15 11:41 ` Panu Matilainen
2015-12-15 12:15 ` Thomas Monjalon
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Panu Matilainen @ 2015-12-15 11:41 UTC (permalink / raw)
To: Thomas Monjalon, dev
On 12/15/2015 12:47 PM, Thomas Monjalon wrote:
> The API has been removed but the symbols were still declared in the map.
>
> Fixes: a421b86a4a02 ("ethdev: remove old flow director API")
>
> Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> ---
> lib/librte_ether/rte_ether_version.map | 8 --------
> 1 file changed, 8 deletions(-)
>
> diff --git a/lib/librte_ether/rte_ether_version.map b/lib/librte_ether/rte_ether_version.map
> index 17a11c7..d8db24d 100644
> --- a/lib/librte_ether/rte_ether_version.map
> +++ b/lib/librte_ether/rte_ether_version.map
> @@ -27,14 +27,6 @@ DPDK_2.2 {
> rte_eth_dev_count;
> rte_eth_dev_default_mac_addr_set;
> rte_eth_dev_detach;
> - rte_eth_dev_fdir_add_perfect_filter;
> - rte_eth_dev_fdir_add_signature_filter;
> - rte_eth_dev_fdir_get_infos;
> - rte_eth_dev_fdir_remove_perfect_filter;
> - rte_eth_dev_fdir_remove_signature_filter;
> - rte_eth_dev_fdir_set_masks;
> - rte_eth_dev_fdir_update_perfect_filter;
> - rte_eth_dev_fdir_update_signature_filter;
> rte_eth_dev_filter_ctrl;
> rte_eth_dev_filter_supported;
> rte_eth_dev_flow_ctrl_get;
>
Good spotting. What did you use find these and the ones in eal? Just
thinking this seems like something that could and should be automated.
- Panu -
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols
2015-12-15 11:41 ` Panu Matilainen
@ 2015-12-15 12:15 ` Thomas Monjalon
2015-12-15 12:53 ` Thomas Monjalon
2015-12-15 14:28 ` Neil Horman
2 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2015-12-15 12:15 UTC (permalink / raw)
To: Panu Matilainen; +Cc: dev
2015-12-15 13:41, Panu Matilainen:
> On 12/15/2015 12:47 PM, Thomas Monjalon wrote:
> > The API has been removed but the symbols were still declared in the map.
> >
> > Fixes: a421b86a4a02 ("ethdev: remove old flow director API")
> >
> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> > ---
> > lib/librte_ether/rte_ether_version.map | 8 --------
> > 1 file changed, 8 deletions(-)
> >
> > diff --git a/lib/librte_ether/rte_ether_version.map b/lib/librte_ether/rte_ether_version.map
> > index 17a11c7..d8db24d 100644
> > --- a/lib/librte_ether/rte_ether_version.map
> > +++ b/lib/librte_ether/rte_ether_version.map
> > @@ -27,14 +27,6 @@ DPDK_2.2 {
> > rte_eth_dev_count;
> > rte_eth_dev_default_mac_addr_set;
> > rte_eth_dev_detach;
> > - rte_eth_dev_fdir_add_perfect_filter;
> > - rte_eth_dev_fdir_add_signature_filter;
> > - rte_eth_dev_fdir_get_infos;
> > - rte_eth_dev_fdir_remove_perfect_filter;
> > - rte_eth_dev_fdir_remove_signature_filter;
> > - rte_eth_dev_fdir_set_masks;
> > - rte_eth_dev_fdir_update_perfect_filter;
> > - rte_eth_dev_fdir_update_signature_filter;
> > rte_eth_dev_filter_ctrl;
> > rte_eth_dev_filter_supported;
> > rte_eth_dev_flow_ctrl_get;
> >
>
> Good spotting. What did you use find these and the ones in eal? Just
> thinking this seems like something that could and should be automated.
Yes, it must be automated.
There are also some symbols which are defined in headers and do not need
to be in the .map. I'll send more cleanup and the script in the coming days
(for 2.3).
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols
2015-12-15 11:41 ` Panu Matilainen
2015-12-15 12:15 ` Thomas Monjalon
@ 2015-12-15 12:53 ` Thomas Monjalon
2015-12-15 14:28 ` Neil Horman
2 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2015-12-15 12:53 UTC (permalink / raw)
To: Panu Matilainen; +Cc: dev
2015-12-15 13:41, Panu Matilainen:
> On 12/15/2015 12:47 PM, Thomas Monjalon wrote:
> > The API has been removed but the symbols were still declared in the map.
> >
> > Fixes: a421b86a4a02 ("ethdev: remove old flow director API")
> >
> > Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
[...]
>
> Good spotting. What did you use find these and the ones in eal? Just
> thinking this seems like something that could and should be automated.
Series applied
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols
2015-12-15 11:41 ` Panu Matilainen
2015-12-15 12:15 ` Thomas Monjalon
2015-12-15 12:53 ` Thomas Monjalon
@ 2015-12-15 14:28 ` Neil Horman
2 siblings, 0 replies; 6+ messages in thread
From: Neil Horman @ 2015-12-15 14:28 UTC (permalink / raw)
To: Panu Matilainen; +Cc: dev
On Tue, Dec 15, 2015 at 01:41:53PM +0200, Panu Matilainen wrote:
> On 12/15/2015 12:47 PM, Thomas Monjalon wrote:
> >The API has been removed but the symbols were still declared in the map.
> >
> >Fixes: a421b86a4a02 ("ethdev: remove old flow director API")
> >
> >Signed-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>
> >---
> > lib/librte_ether/rte_ether_version.map | 8 --------
> > 1 file changed, 8 deletions(-)
> >
> >diff --git a/lib/librte_ether/rte_ether_version.map b/lib/librte_ether/rte_ether_version.map
> >index 17a11c7..d8db24d 100644
> >--- a/lib/librte_ether/rte_ether_version.map
> >+++ b/lib/librte_ether/rte_ether_version.map
> >@@ -27,14 +27,6 @@ DPDK_2.2 {
> > rte_eth_dev_count;
> > rte_eth_dev_default_mac_addr_set;
> > rte_eth_dev_detach;
> >- rte_eth_dev_fdir_add_perfect_filter;
> >- rte_eth_dev_fdir_add_signature_filter;
> >- rte_eth_dev_fdir_get_infos;
> >- rte_eth_dev_fdir_remove_perfect_filter;
> >- rte_eth_dev_fdir_remove_signature_filter;
> >- rte_eth_dev_fdir_set_masks;
> >- rte_eth_dev_fdir_update_perfect_filter;
> >- rte_eth_dev_fdir_update_signature_filter;
> > rte_eth_dev_filter_ctrl;
> > rte_eth_dev_filter_supported;
> > rte_eth_dev_flow_ctrl_get;
> >
>
> Good spotting. What did you use find these and the ones in eal? Just
> thinking this seems like something that could and should be automated.
>
> - Panu -
>
You can likely do it with this command:
nm -A ./*.o | grep <sym> | wc -l
or something simmilar. nm -A dysplays all the symbols in an object file. if
you grep for your sym and wc -l returns more than 1 line, the symbols has a
reference, and can't be removed. Note it needs to be more than 1 line, as you
have to account for the object defining the symbol
Neil
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-12-15 14:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-15 10:47 [dpdk-dev] [PATCH 1/2] eal: remove zombie symbols Thomas Monjalon
2015-12-15 10:47 ` [dpdk-dev] [PATCH 2/2] ethdev: remove old flow director symbols Thomas Monjalon
2015-12-15 11:41 ` Panu Matilainen
2015-12-15 12:15 ` Thomas Monjalon
2015-12-15 12:53 ` Thomas Monjalon
2015-12-15 14:28 ` Neil Horman
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).