[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function
Mcnamara, John
john.mcnamara at intel.com
Thu Feb 26 16:36:16 CET 2015
> -----Original Message-----
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mcnamara, John
> Sent: Thursday, February 26, 2015 12:21 PM
> To: Tetsuya Mukawa; Thomas Monjalon
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port
> hotplug function
>
> The following commit introduced this issue:
>
> $ git bisect good
> c743e50c475f73edf78e5ba26445d7c6ea217f40 is the first bad commit
> commit c743e50c475f73edf78e5ba26445d7c6ea217f40
Hi,
The above commit also has throws a warning with ICC:
lib/librte_pmd_null/rte_eth_null.c(47): error #83:
type qualifier specified more than once
static const char const *valid_arguments[] = {
^
An earlier commit also throws a warning with ICC:
lib/librte_pmd_null/rte_eth_null.c(47): error #83:
type qualifier specified more than once
static const char const *valid_arguments[] = {
^
Commit:
92d94d3744d7760d8d5e490be810612cf4a9cfb0 is the first bad commit
commit 92d94d3744d7760d8d5e490be810612cf4a9cfb0
Author: Tetsuya Mukawa <mukawa at igel.co.jp>
Date: Thu Feb 26 04:32:26 2015 +0900
ethdev: attach or detach port
I applied the following patch to fix these issues, (in order to test with the HEAD). You can review them to see if they are valid and apply something similar:
diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c
index bb94ccb..6ea7a17 100644
--- a/lib/librte_ether/rte_ethdev.c
+++ b/lib/librte_ether/rte_ethdev.c
@@ -433,7 +433,7 @@ static enum rte_eth_dev_type
rte_eth_dev_get_device_type(uint8_t port_id)
{
if (!rte_eth_dev_is_valid_port(port_id))
- return -1;
+ return RTE_ETH_DEV_UNKNOWN;
return rte_eth_devices[port_id].dev_type;
}
diff --git a/lib/librte_pmd_null/rte_eth_null.c b/lib/librte_pmd_null/rte_eth_null.c
index bb10276..3ef5842 100644
--- a/lib/librte_pmd_null/rte_eth_null.c
+++ b/lib/librte_pmd_null/rte_eth_null.c
@@ -44,7 +44,7 @@
static unsigned default_packet_size = 64;
static unsigned default_packet_copy;
-static const char const *valid_arguments[] = {
+static const char *valid_arguments[] = {
ETH_NULL_PACKET_SIZE_ARG,
ETH_NULL_PACKET_COPY_ARG,
NULL
John.
--
More information about the dev
mailing list