[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-03-04 Thread Tetsuya Mukawa
2015-02-26 19:57 GMT+09:00 Thomas Monjalon : > 2015-02-26 18:06, Tetsuya Mukawa: >> 2015-02-26 16:03 GMT+09:00 Thomas Monjalon : >> > 2015-02-25 16:49, Stephen Hemminger: >> >> Build fails if HOTPLUG is disabled >> >> Hi Stephen, >> >> I appreciate for you reporting. >> >> > >> > OK thanks for repo

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-28 Thread Thomas Monjalon
2015-02-26 12:21, Mcnamara, John: > Hi, > > The HEAD doesn't compile with gcc 4.7.2: > > $ git clone http://dpdk.org/git/dpdk > $ cd dpdk > $ make T=x86_64-native-linuxapp-gcc -j install > > ... > == Build lib/librte_pipeline > SYMLINK-FILE include/rte_pipeline.h >

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-27 Thread Tetsuya Mukawa
On 2015/02/27 0:36, Mcnamara, John wrote: >> -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 >> Su

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Tetsuya Mukawa
2015-02-26 19:57 GMT+09:00 Thomas Monjalon : > 2015-02-26 18:06, Tetsuya Mukawa: >> 2015-02-26 16:03 GMT+09:00 Thomas Monjalon : >> > 2015-02-25 16:49, Stephen Hemminger: >> >> Build fails if HOTPLUG is disabled >> >> Hi Stephen, >> >> I appreciate for you reporting. >> >> > >> > OK thanks for repo

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Tetsuya Mukawa
2015-02-26 16:03 GMT+09:00 Thomas Monjalon : > 2015-02-25 16:49, Stephen Hemminger: >> Build fails if HOTPLUG is disabled Hi Stephen, I appreciate for you reporting. > > OK thanks for reporting. > Actually there is no good reason to disable hotplug on Linux. > Though it means that it's impossibl

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Mcnamara, John
> -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

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Mcnamara, John
Hi, The HEAD doesn't compile with gcc 4.7.2: $ git clone http://dpdk.org/git/dpdk $ cd dpdk $ make T=x86_64-native-linuxapp-gcc -j install ... == Build lib/librte_pipeline SYMLINK-FILE include/rte_pipeline.h CC rte_pipeline.o /tmp/dpdk/lib/librte_pmd_null/rte

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Thomas Monjalon
2015-02-26 18:06, Tetsuya Mukawa: > 2015-02-26 16:03 GMT+09:00 Thomas Monjalon : > > 2015-02-25 16:49, Stephen Hemminger: > >> Build fails if HOTPLUG is disabled > > Hi Stephen, > > I appreciate for you reporting. > > > > > OK thanks for reporting. > > Actually there is no good reason to disable

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Thomas Monjalon
2015-02-25 16:49, Stephen Hemminger: > Build fails if HOTPLUG is disabled OK thanks for reporting. Actually there is no good reason to disable hotplug on Linux. Though it means that it's impossible to build on FreeBSD. Tetsuya, the right fix is to remove this option. You should manage to graceful

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-26 Thread Thomas Monjalon
> This patch adds port hotplug support to Null PMD. > > v9: > - Use rte_eth_dev_release_port() instead of rte_eth_dev_free(). > v7: > - Add parameter checkings. >(Thanks to Iremonger, Bernard) > v6: > - Fix a parameter of rte_eth_dev_free(). > v4: > - Fix commit title. > > Signed-off-by:

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-25 Thread Stephen Hemminger
Build fails if HOTPLUG is disabled == Build lib/librte_ether CC rte_ethdev.o /var/src/dpdk/lib/librte_ether/rte_ethdev.c:430:1: error: ?rte_eth_dev_get_device_type? defined but not used [-Werror=unused-function] rte_eth_dev_get_device_type(uint8_t port_id) ^ /var/src/dpdk/lib/librte_ether/rte

[dpdk-dev] [PATCH v11 2/2] librte_pmd_null: Support port hotplug function

2015-02-23 Thread Tetsuya Mukawa
This patch adds port hotplug support to Null PMD. v9: - Use rte_eth_dev_release_port() instead of rte_eth_dev_free(). v7: - Add parameter checkings. (Thanks to Iremonger, Bernard) v6: - Fix a parameter of rte_eth_dev_free(). v4: - Fix commit title. Signed-off-by: Tetsuya Mukawa --- lib/l