[dpdk-dev] [PATCH v11 0/5] link bonding library
2014-06-30 09:28, Doherty, Declan: > > -Original Message- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > > > Hi Declan, > > > > 2014-06-29 18:49, Declan Doherty: > > > This patchset contains the name change fix in the link bonding library > > > makefile. I have also put a change into the unit test application > > > makefile > > > so that the link bonding tests are not built if the library is also not > > > being built. It doesn't make sense to split the bonding libraries APIs > > > into > > > a seperate library, as they are directly coupled to with the bonding > > > library implmentation > > > > You fixed optional building but there is still an issue when building > > shared library. > > As you don't want to split the library to separate the PMD plugin from the > > specific API, I suggest to not consider it as a plugin (unlike other > > PMDs). > > > > I've sent some patches about these 2 topics. > > If you agree, I'll apply them, merged with your serie. > > I'm happy for you to apply the makefile changes you have suggested and not > consider the bonding PMD as a plugin, I think this makes the most sense > considering it is a purely software library. Applied with Robert's changes for version 1.7.0. Thanks for all these revisions (v11 is currently the best score :) -- Thomas
[dpdk-dev] [PATCH] kni: compatibility with RHEL 7
> Compilation in RHEL7 is failed. This fixes the build issue. > > RHEL7 has skb_set_hash, the kernel version is 3.10 though. > Don't define skb_set_hash for RHEL7. > > Signed-off-by: Hiroshi Shimamoto > Reviewed-by: Hayato Momma Applied for version 1.7.0. Thanks -- Thomas
[dpdk-dev] [PATCH 1/2] kni: fix build with kernel 3.16
2014-06-17 17:32, Aaro Koskinen: > SET_ETHTOOL_OPS is gone in 3.16, so modify drivers accordingly. > > Signed-off-by: Aaro Koskinen Applied for version 1.7.0. Thanks -- Thomas
[dpdk-dev] [PATCH 2/2] kni: igb: modify rate configation to support min/max rate fields
2014-06-17 17:32, Aaro Koskinen: > This follows the mainline Linux kernel commit > ed616689a3d95eb6c9bdbb1ef74b0f50cbdf276a (Add support to configure SR-IOV > VF minimum and maximum Tx rate) by Sucheta Chakraborty, and enables to > build the driver against 3.16. > > Signed-off-by: Aaro Koskinen Acked-by: Thomas Monjalon Applied for version 1.7.0. Thanks -- Thomas
[dpdk-dev] [PATCH] app/test: fix build switches to enable cmdline tests
> > There were 2 typos since these commits (in 1.6.0 releases): > > 21a7f4e264 fix build without librte_cmdline > > cac6d08c8b replace --use-device option by --pci-whitelist and --vdev > > > > In makefiles, the build options are prefixed with CONFIG_RTE_ > > but in .c file, it is only RTE_. > > > > These typos were disabling cmdline unit tests and test of "--vdev > > eth_ring" > > option. > > > > Signed-off-by: Thomas Monjalon > > Acked-by: Pablo de Lara Applied for version 1.7.0. -- Thomas
[dpdk-dev] [PATCH] test: Remove NULL snprintf tests in cmdline unit test
> > Cmdline unit test calls several functions that use snprintf, > > with NULL pointer as the destination buffer. This patch > > eliminates these NULL pointer tests. > > > > Signed-off-by: Pablo de Lara > > Acked-by: Olivier Matz Applied for version 1.7.0. Thanks -- Thomas
[dpdk-dev] [PATCH] app/test: fix and separate vdev test in test_eal_flags
> In EAL flags unit test, vdev test was being included > (and broken) in whitelist test. It has been separated > in different test and fixed, by calling another > primary process, instead of a secondary one. > > Signed-off-by: Pablo de Lara Acked-by: Thomas Monjalon Applied for version 1.7.0. Thanks -- Thomas
[dpdk-dev] Compilation errors in FreeBSD and OR Linux (Kerne 2.6.39l
Hi Thomas, We defected two compilation errors in RC3: - One from FreeBSD 10: target: x86_64-native-bsdapp-gcc == Build lib/librte_eal/bsdapp/eal CC eal.o /jenkins/workspace/DPDK_AUTO_IDT_VM_FreeBSD10.0_64_BUILD/DPDK/lib/librte_eal/bsdapp/eal/eal.c: In function 'rte_eal_init': /jenkins/workspace/DPDK_AUTO_IDT_VM_FreeBSD10.0_64_BUILD/DPDK/lib/librte_eal/bsdapp/eal/eal.c:877:2: error: implicit declaration of function 'rte_eal_dev_init' [-Werror=implicit-function-declaration] if (rte_eal_dev_init(PMD_INIT_PRE_PCI_PROBE) < 0) ^ /jenkins/workspace/DPDK_AUTO_IDT_VM_FreeBSD10.0_64_BUILD/DPDK/lib/librte_eal/bsdapp/eal/eal.c:877:2: error: nested extern declaration of 'rte_eal_dev_init' [-Werror=nested-externs] /jenkins/workspace/DPDK_AUTO_IDT_VM_FreeBSD10.0_64_BUILD/DPDK/lib/librte_eal/bsdapp/eal/eal.c:877:23: error: 'PMD_INIT_PRE_PCI_PROBE' undeclared (first use in this function) if (rte_eal_dev_init(PMD_INIT_PRE_PCI_PROBE) < 0) ^ /jenkins/workspace/DPDK_AUTO_IDT_VM_FreeBSD10.0_64_BUILD/DPDK/lib/librte_eal/bsdapp/eal/eal.c:877:23: note: each undeclared identifier is reported only once for each function it appears in /jenkins/workspace/DPDK_AUTO_IDT_VM_FreeBSD10.0_64_BUILD/DPDK/lib/librte_eal/bsdapp/eal/eal.c:914:23: error: 'PMD_INIT_POST_PCI_PROBE' undeclared (first use in this function) if (rte_eal_dev_init(PMD_INIT_POST_PCI_PROBE) < 0) ^ cc1: all warnings being treated as errors gmake[7]: *** [eal.o] Error 1 gmake[6]: *** [eal] Error 2 gmake[5]: *** [bsdapp] Error 2 gmake[4]: *** [librte_eal] Error 2 gmake[3]: *** [lib] Error 2 gmake[2]: *** [all] Error 2 gmake[1]: *** [x86_64-native-bsdapp-gcc_install] Error 2 gmake: *** [install] Error 2 --- The other from ORACLELINUX64_32 (Linux Kernel 2.6.39) >>>i686-native-linuxapp-gcc%CONFIG_RTE_LIBRTE_PMD_PCAP=y at >>>CONFIG_RTE_NIC_BYPASS=y<<< == Installing i686-native-linuxapp-gcc gmake[5]: Nothing to be done for `depdirs'. Configuration done == Build scripts == Build scripts/testhost == Build lib == Build lib/librte_eal == Build lib/librte_eal/common == Build lib/librte_eal/linuxapp == Build lib/librte_eal/linuxapp/igb_uio Building modules, stage 2. MODPOST 1 modules == Build lib/librte_eal/linuxapp/eal == Build lib/librte_malloc == Build lib/librte_ring == Build lib/librte_mempool == Build lib/librte_mbuf == Build lib/librte_timer == Build lib/librte_cfgfile == Build lib/librte_cmdline == Build lib/librte_ether == Build lib/librte_net == Build lib/librte_pmd_e1000 == Build lib/librte_pmd_ixgbe == Build lib/librte_pmd_i40e == Build lib/librte_kvargs == Build lib/librte_pmd_bond CC rte_eth_bond_api.o In file included from /usr/include/stdlib.h:320, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/i686-native-linuxapp-gcc/include/rte_mempool.h:63, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/i686-native-linuxapp-gcc/include/rte_mbuf.h:61, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/lib/librte_pmd_bond/rte_eth_bond_api.c:37: /usr/include/sys/types.h:61: error: conflicting types for 'dev_t' /usr/include/linux/types.h:22: note: previous declaration of 'dev_t' was here /usr/include/sys/types.h:66: error: conflicting types for 'gid_t' /usr/include/linux/types.h:52: note: previous declaration of 'gid_t' was here /usr/include/sys/types.h:71: error: conflicting types for 'mode_t' /usr/include/linux/types.h:24: note: previous declaration of 'mode_t' was here /usr/include/sys/types.h:76: error: conflicting types for 'nlink_t' /usr/include/linux/types.h:25: note: previous declaration of 'nlink_t' was here /usr/include/sys/types.h:81: error: conflicting types for 'uid_t' /usr/include/linux/types.h:51: note: previous declaration of 'uid_t' was here In file included from /usr/include/sys/types.h:133, from /usr/include/stdlib.h:320, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/i686-native-linuxapp-gcc/include/rte_mempool.h:63, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/i686-native-linuxapp-gcc/include/rte_mbuf.h:61, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/lib/librte_pmd_bond/rte_eth_bond_api.c:37: /usr/include/time.h:104: error: conflicting types for 'timer_t' /usr/include/linux/types.h:31: note: previous declaration of 'timer_t' was here In file included from /usr/include/sys/types.h:220, from /usr/include/stdlib.h:320, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/i686-native-linuxapp-gcc/include/rte_mempool.h:63, from /jenkins/workspace/DPDK_AUTO_IDT_VM_ORACLELINUX64_32_BUILD/DPDK/i686-native-linuxapp-gcc/include/rte_mbuf.h:61,
[dpdk-dev] [PATCH 0/2] i40e improvements
These patches include: * use i40e_clear_hw() to clear hardware before PF reset. It is quite straightforward, use new perfect function to repalce old ones for clearing hardware before doing PF reset. * add required steps in TX queue enable, disable. Hardware specification changed recently which requries additional steps. Without the steps, DPDK might be not able to transmit packets after standard Linux i40e driver loaded. Helin Zhang (2): i40e: use i40e_clear_hw() to clear hardware before PF reset i40e: add required steps in TX queue enable, disable lib/librte_pmd_i40e/i40e_ethdev.c | 116 +- 1 file changed, 15 insertions(+), 101 deletions(-) -- 1.8.1.4
[dpdk-dev] [PATCH 1/2] i40e: use i40e_clear_hw() to clear hardware before PF reset
i40e_clear_hw() was provided recently in shared code to clear
hardware, which can cover disabling all queues. The code
changes are to remove i40e_pf_disable_all_queues() and use
i40e_clear_hw() instead.
Signed-off-by: Helin Zhang
Acked-by: Jijiang Liu
Acked-by: Jing Chen
---
lib/librte_pmd_i40e/i40e_ethdev.c | 103 +-
1 file changed, 2 insertions(+), 101 deletions(-)
diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c
b/lib/librte_pmd_i40e/i40e_ethdev.c
index 068b847..6624586 100644
--- a/lib/librte_pmd_i40e/i40e_ethdev.c
+++ b/lib/librte_pmd_i40e/i40e_ethdev.c
@@ -188,7 +188,6 @@ static struct i40e_veb *i40e_veb_setup(struct i40e_pf *pf,
struct i40e_vsi *vsi);
static int i40e_pf_config_mq_rx(struct i40e_pf *pf);
static int i40e_vsi_config_double_vlan(struct i40e_vsi *vsi, int on);
-static int i40e_pf_disable_all_queues(struct i40e_hw *hw);
static inline int i40e_find_all_vlan_for_mac(struct i40e_vsi *vsi,
struct i40e_macvlan_filter *mv_f,
int num,
@@ -373,12 +372,8 @@ eth_i40e_dev_init(__rte_unused struct eth_driver *eth_drv,
hw->bus.device = pci_dev->addr.devid;
hw->bus.func = pci_dev->addr.function;
- /* Disable all queues before PF reset, as required */
- ret = i40e_pf_disable_all_queues(hw);
- if (ret != I40E_SUCCESS) {
- PMD_INIT_LOG(ERR, "Failed to disable queues %u\n", ret);
- return ret;
- }
+ /* Make sure all is clean before doing PF reset */
+ i40e_clear_hw(hw);
/* Reset here to make sure all is clean for each PF */
ret = i40e_pf_reset(hw);
@@ -3948,97 +3943,3 @@ i40e_pf_config_mq_rx(struct i40e_pf *pf)
return 0;
}
-
-static int
-i40e_disable_queue(struct i40e_hw *hw, uint16_t q_idx)
-{
- uint16_t i;
- uint32_t reg;
-
- /* Disable TX queue */
- for (i = 0; i < I40E_CHK_Q_ENA_COUNT; i++) {
- reg = I40E_READ_REG(hw, I40E_QTX_ENA(q_idx));
- if (!(((reg >> I40E_QTX_ENA_QENA_REQ_SHIFT) & 0x1) ^
- ((reg >> I40E_QTX_ENA_QENA_STAT_SHIFT) & 0x1)))
- break;
- rte_delay_us(I40E_CHK_Q_ENA_INTERVAL_US);
- }
- if (i >= I40E_CHK_Q_ENA_COUNT) {
- PMD_DRV_LOG(ERR, "Failed to disable "
- "tx queue[%u]\n", q_idx);
- return I40E_ERR_TIMEOUT;
- }
-
- if (reg & I40E_QTX_ENA_QENA_STAT_MASK) {
- reg &= ~I40E_QTX_ENA_QENA_REQ_MASK;
- I40E_WRITE_REG(hw, I40E_QTX_ENA(q_idx), reg);
- for (i = 0; i < I40E_CHK_Q_ENA_COUNT; i++) {
- rte_delay_us(I40E_CHK_Q_ENA_INTERVAL_US);
- reg = I40E_READ_REG(hw, I40E_QTX_ENA(q_idx));
- if (!(reg & I40E_QTX_ENA_QENA_REQ_MASK) &&
- !(reg & I40E_QTX_ENA_QENA_STAT_MASK))
- break;
- }
- if (i >= I40E_CHK_Q_ENA_COUNT) {
- PMD_DRV_LOG(ERR, "Failed to disable "
- "tx queue[%u]\n", q_idx);
- return I40E_ERR_TIMEOUT;
- }
- }
-
- /* Disable RX queue */
- for (i = 0; i < I40E_CHK_Q_ENA_COUNT; i++) {
- reg = I40E_READ_REG(hw, I40E_QRX_ENA(q_idx));
- if (!((reg >> I40E_QRX_ENA_QENA_REQ_SHIFT) & 0x1) ^
- ((reg >> I40E_QRX_ENA_QENA_STAT_SHIFT) & 0x1))
- break;
- rte_delay_us(I40E_CHK_Q_ENA_INTERVAL_US);
- }
- if (i >= I40E_CHK_Q_ENA_COUNT) {
- PMD_DRV_LOG(ERR, "Failed to disable "
- "rx queue[%u]\n", q_idx);
- return I40E_ERR_TIMEOUT;
- }
-
- if (reg & I40E_QRX_ENA_QENA_STAT_MASK) {
- reg &= ~I40E_QRX_ENA_QENA_REQ_MASK;
- I40E_WRITE_REG(hw, I40E_QRX_ENA(q_idx), reg);
- for (i = 0; i < I40E_CHK_Q_ENA_COUNT; i++) {
- rte_delay_us(I40E_CHK_Q_ENA_INTERVAL_US);
- reg = I40E_READ_REG(hw, I40E_QRX_ENA(q_idx));
- if (!(reg & I40E_QRX_ENA_QENA_REQ_MASK) &&
- !(reg & I40E_QRX_ENA_QENA_STAT_MASK))
- break;
- }
- if (i >= I40E_CHK_Q_ENA_COUNT) {
- PMD_DRV_LOG(ERR, "Failed to disable "
- "rx queue[%u]\n", q_idx);
- return I40E_ERR_TIMEOUT;
- }
- }
-
- return I40E_SUCCESS;
-}
-
-static int
-i40e_pf_disable_all_queues(struct i40e_hw *hw)
-{
- uint32_t reg;
- uint16_t firstq, lastq, maxq, i;
- int ret;
- reg = I40E_READ_REG(hw, I40E_PFLAN_QALLOC);
- if (!(reg & I40E_PFLAN_QALLOC_VALID_
[dpdk-dev] [PATCH 2/2] i40e: add required steps in TX queue enable, disable
Hardware specification changed recently which requires to set
or clear TX queue disable flags before actually enabling or
disabling a specific TX queue. 'QTX_HEAD' register needs to
be cleared before setting the QENA_REQ flag.
Signed-off-by: Helin Zhang
Acked-by: Jijiang Liu
Acked-by: Jing Chen
---
lib/librte_pmd_i40e/i40e_ethdev.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c
b/lib/librte_pmd_i40e/i40e_ethdev.c
index 6624586..9ed31b5 100644
--- a/lib/librte_pmd_i40e/i40e_ethdev.c
+++ b/lib/librte_pmd_i40e/i40e_ethdev.c
@@ -85,6 +85,8 @@
#define I40E_QUEUE_ITR_INTERVAL_DEFAULT 32 /* 32 us */
#define I40E_QUEUE_ITR_INTERVAL_MAX 8160 /* 8160 us */
+#define I40E_PRE_TX_Q_CFG_WAIT_US 10 /* 10 us */
+
#define I40E_RSS_OFFLOAD_ALL ( \
ETH_RSS_NONF_IPV4_UDP | \
ETH_RSS_NONF_IPV4_TCP | \
@@ -2780,6 +2782,13 @@ i40e_switch_tx_queue(struct i40e_hw *hw, uint16_t q_idx,
bool on)
uint32_t reg;
uint16_t j;
+ /**
+* Set or clear TX Queue Disable flags,
+* which is required by hardware.
+*/
+ i40e_pre_tx_queue_cfg(hw, q_idx, on);
+ rte_delay_us(I40E_PRE_TX_Q_CFG_WAIT_US);
+
/* Wait until the request is finished */
for (j = 0; j < I40E_CHK_Q_ENA_COUNT; j++) {
rte_delay_us(I40E_CHK_Q_ENA_INTERVAL_US);
@@ -2793,6 +2802,8 @@ i40e_switch_tx_queue(struct i40e_hw *hw, uint16_t q_idx,
bool on)
if (on) {
if (reg & I40E_QTX_ENA_QENA_STAT_MASK)
return I40E_SUCCESS; /* already on, skip next steps */
+
+ I40E_WRITE_REG(hw, I40E_QTX_HEAD(q_idx), 0);
reg |= I40E_QTX_ENA_QENA_REQ_MASK;
} else {
if (!(reg & I40E_QTX_ENA_QENA_STAT_MASK))
@@ -2821,8 +2832,10 @@ i40e_switch_tx_queue(struct i40e_hw *hw, uint16_t q_idx,
bool on)
(on ? "enable" : "disable"), q_idx);
return I40E_ERR_TIMEOUT;
}
+
return I40E_SUCCESS;
}
+
/* Swith on or off the tx queues */
static int
i40e_vsi_switch_tx_queues(struct i40e_vsi *vsi, bool on)
--
1.8.1.4
[dpdk-dev] [PATCH 0/2] i40e improvements
Test-by: Min Cao This patch includes 3 files, and has been tested by Intel. Please see test environment information as the following: Fedora 20 x86_64, Linux Kernel 3.11.10-301, GCC 4.8.2 Intel Xeon CPU E5-2680 v2 @ 2.80GHz NIC: Intel Fortville 4*10G. We verified testpmd.
[dpdk-dev] [PATCH] bond: fix for compile issue on 32 bits Oracle Linux 6.4
The compile error on 32 bits Oracle Linux 6.4 is as below. ** Kernel: 2.6.39 GCC: 4.4.7 20120313 In file included from /usr/include/stdlib.h:320, from /home/dpdk/i686-native-linuxapp-gcc/include/rte_mempool.h:63, from /home/dpdk/i686-native-linuxapp-gcc/include/rte_mbuf.h:61, from /home/dpdk/lib/librte_pmd_bond/rte_eth_bond_api.c:37: /usr/include/sys/types.h:61: error: conflicting types for dev_t /usr/include/linux/types.h:22: note: previous declaration of dev_t was here /usr/include/sys/types.h:66: error: conflicting types for gid_t /usr/include/linux/types.h:52: note: previous declaration of gid_t was here /usr/include/sys/types.h:71: error: conflicting types for mode_t /usr/include/linux/types.h:24: note: previous declaration of mode_t was here /usr/include/sys/types.h:76: error: conflicting types for nlink_t /usr/include/linux/types.h:25: note: previous declaration of nlink_t was here /usr/include/sys/types.h:81: error: conflicting types for uid_t /usr/include/linux/types.h:51: note: previous declaration of uid_t was here *** Signed-off-by: Helin Zhang Acked-by: Cunming Liang --- lib/librte_pmd_bond/rte_eth_bond_api.c | 14 ++ lib/librte_pmd_bond/rte_eth_bond_pmd.c | 15 +-- 2 files changed, 7 insertions(+), 22 deletions(-) diff --git a/lib/librte_pmd_bond/rte_eth_bond_api.c b/lib/librte_pmd_bond/rte_eth_bond_api.c index 9be5f72..75f5694 100644 --- a/lib/librte_pmd_bond/rte_eth_bond_api.c +++ b/lib/librte_pmd_bond/rte_eth_bond_api.c @@ -31,20 +31,10 @@ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include -#include - #include -#include -#include -#include -#include -#include -#include #include -#include -#include -#include +#include +#include #include "rte_eth_bond.h" #include "rte_eth_bond_private.h" diff --git a/lib/librte_pmd_bond/rte_eth_bond_pmd.c b/lib/librte_pmd_bond/rte_eth_bond_pmd.c index aa910ba..048de7f 100644 --- a/lib/librte_pmd_bond/rte_eth_bond_pmd.c +++ b/lib/librte_pmd_bond/rte_eth_bond_pmd.c @@ -31,20 +31,15 @@ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ -#include -#include - #include -#include -#include -#include +#include #include +#include +#include #include +#include #include -#include -#include -#include -#include +#include #include "rte_eth_bond.h" #include "rte_eth_bond_private.h" -- 1.8.1.4
[dpdk-dev] [PATCH 0/2] i40e improvements
> These patches include: > * use i40e_clear_hw() to clear hardware before PF reset. > It is quite straightforward, use new perfect function to repalce > old ones for clearing hardware before doing PF reset. > * add required steps in TX queue enable, disable. > Hardware specification changed recently which requries additional > steps. Without the steps, DPDK might be not able to transmit > packets after standard Linux i40e driver loaded. > > Helin Zhang (2): > i40e: use i40e_clear_hw() to clear hardware before PF reset > i40e: add required steps in TX queue enable, disable Tested-by: Min Cao Applied as last minute fixes for version 1.7.0. Thanks -- Thomas
[dpdk-dev] [PATCH] bond: fix for compile issue on 32 bits Oracle Linux 6.4
2014-07-02 15:29, Helin Zhang: > The compile error on 32 bits Oracle Linux 6.4 is as below. > ** > Kernel: 2.6.39 > GCC: 4.4.7 20120313 > > In file included from /usr/include/stdlib.h:320, > from /home/dpdk/i686-native-linuxapp-gcc/include/rte_mempool.h:63, > from /home/dpdk/i686-native-linuxapp-gcc/include/rte_mbuf.h:61, > from /home/dpdk/lib/librte_pmd_bond/rte_eth_bond_api.c:37: > /usr/include/sys/types.h:61: error: conflicting types for dev_t > /usr/include/linux/types.h:22: note: previous declaration of dev_t was here > /usr/include/sys/types.h:66: error: conflicting types for gid_t > /usr/include/linux/types.h:52: note: previous declaration of gid_t was here > /usr/include/sys/types.h:71: error: conflicting types for mode_t > /usr/include/linux/types.h:24: note: previous declaration of mode_t was here > /usr/include/sys/types.h:76: error: conflicting types for nlink_t > /usr/include/linux/types.h:25: note: previous declaration of nlink_t was > here /usr/include/sys/types.h:81: error: conflicting types for uid_t > /usr/include/linux/types.h:51: note: previous declaration of uid_t was here > *** > > Signed-off-by: Helin Zhang > Acked-by: Cunming Liang Applied for version 1.7.0. Thanks -- Thomas
[dpdk-dev] [PATCH] eal: fix build for bsd
When adding link bonding to EAL initialization (a155d430119), an include was missing for BSD. Signed-off-by: Thomas Monjalon --- lib/librte_eal/bsdapp/eal/eal.c | 1 + 1 file changed, 1 insertion(+) Please, a quick test and acknowledgement of this fix would be helpful. diff --git a/lib/librte_eal/bsdapp/eal/eal.c b/lib/librte_eal/bsdapp/eal/eal.c index ec57b26..69ead31 100644 --- a/lib/librte_eal/bsdapp/eal/eal.c +++ b/lib/librte_eal/bsdapp/eal/eal.c @@ -66,6 +66,7 @@ #include #include #include +#include #include #include #include -- 2.0.0
[dpdk-dev] Intel DPDK: exception_path:RTE_ARCH
Hello! I started playing Intel DPDK example. I used to compile exception_path code. When I do make command "make", I got an error, RTE_ARCH is not set. So I set the variable via terminal bash using export RTE_ARCH=x86_64 Stilll it is not working. Any wrong anywhere? Thanks for your reponse. Best regards Sothy
[dpdk-dev] [PATCH] net: get rid of SET_ETHTOOL_OPS
Hi Flavio, 2014-07-01 15:19, Flavio Leitner: > The SET_ETHTOOL_OPS has been removed from upstream, so it > breaks the dpdk build with recent kernels. > > Signed-off-by: Flavio Leitner You are removing SET_ETHTOOL_OPS calls. In a previous patch, Aaro Koskinen made the choice to redefine the macro in kcompat files. I don't know what will be the choice of Intel in the sourceforge base driver. So I've applied Aaro's patch as there are less modifications of the base driver: http://dpdk.org/browse/dpdk/commit/?id=e0b7ca0c0383411 It would be better to have patches from people working on the base driver. So upgrades from sourceforge driver to KNI would be smoother. Thanks -- Thomas
[dpdk-dev] Intel DPDK: exception_path:RTE_ARCH
You need, to define the following variables before compiling, please refer to the DPDK documentation. export RTE_TARGET=x86_64-native-linuxapp-gcc export RTE_SDK=/home/user/dpdk On Wed, Jul 2, 2014 at 12:48 PM, sothy shan wrote: > Hello! > > I started playing Intel DPDK example. I used to compile exception_path > code. > When I do make command "make", I got an error, > > RTE_ARCH is not set. So I set the variable via terminal bash using > > export RTE_ARCH=x86_64 > > Stilll it is not working. Any wrong anywhere? > > Thanks for your reponse. > > Best regards > Sothy >
[dpdk-dev] [PATCH] eal: fix build for bsd
Hi I have tested the patch, it still has a build error although it is different from the previous one. OS: FreeBSD10 == Build lib/librte_pmd_bond CC rte_eth_bond_api.o /root/zzc/dpdk-1.7.0-rc3/lib/librte_pmd_bond/rte_eth_bond_api.c:35:27: fatal error: linux/binfmts.h: No such file or directory #include ^ compilation terminated. gmake[5]: *** [rte_eth_bond_api.o] Error 1 gmake[4]: *** [librte_pmd_bond] Error 2 gmake[3]: *** [lib] Error 2 gmake[2]: *** [all] Error 2 gmake[1]: *** [x86_64-native-bsdapp-gcc_install] Error 2 gmake: *** [install] Error 2 Thanks, Zhaochen > -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Wednesday, July 02, 2014 5:22 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] eal: fix build for bsd > > When adding link bonding to EAL initialization (a155d430119), > an include was missing for BSD. > > Signed-off-by: Thomas Monjalon > --- > lib/librte_eal/bsdapp/eal/eal.c | 1 + > 1 file changed, 1 insertion(+) > > > Please, a quick test and acknowledgement of this fix would be helpful. > > > diff --git a/lib/librte_eal/bsdapp/eal/eal.c b/lib/librte_eal/bsdapp/eal/eal.c > index ec57b26..69ead31 100644 > --- a/lib/librte_eal/bsdapp/eal/eal.c > +++ b/lib/librte_eal/bsdapp/eal/eal.c > @@ -66,6 +66,7 @@ > #include > #include > #include > +#include > #include > #include > #include > -- > 2.0.0
[dpdk-dev] Intel DPDK: exception_path:RTE_ARCH
2014-07-02 12:56, Alex Markuze: > On Wed, Jul 2, 2014 at 12:48 PM, sothy shan wrote: > > I started playing Intel DPDK example. I used to compile exception_path > > code. > > When I do make command "make", I got an error, > > > > RTE_ARCH is not set. So I set the variable via terminal bash using > > > > export RTE_ARCH=x86_64 > > > > Stilll it is not working. Any wrong anywhere? > > You need, to define the following variables before compiling, please refer > to the DPDK documentation. > > export RTE_TARGET=x86_64-native-linuxapp-gcc > > export RTE_SDK=/home/user/dpdk You can also try to build all examples with these commands: RTE_TARGET=x86_64-native-linuxapp-gcc make T=$RTE_TARGET install make T=$RTE_TARGET examples O=$RTE_TARGET/examples (with DPDK >= 1.7) -- Thomas
[dpdk-dev] Intel DPDK: exception_path:RTE_ARCH
On Wed, Jul 2, 2014 at 12:00 PM, Thomas Monjalon wrote: > 2014-07-02 12:56, Alex Markuze: > > On Wed, Jul 2, 2014 at 12:48 PM, sothy shan wrote: > > > I started playing Intel DPDK example. I used to compile exception_path > > > code. > > > When I do make command "make", I got an error, > > > > > > RTE_ARCH is not set. So I set the variable via terminal bash using > > > > > > export RTE_ARCH=x86_64 > > > > > > Stilll it is not working. Any wrong anywhere? > > > > You need, to define the following variables before compiling, please > refer > > to the DPDK documentation. > > > > export RTE_TARGET=x86_64-native-linuxapp-gcc > > > > export RTE_SDK=/home/user/dpdk > > You can also try to build all examples with these commands: > RTE_TARGET=x86_64-native-linuxapp-gcc > make T=$RTE_TARGET install > make T=$RTE_TARGET examples O=$RTE_TARGET/examples > > (with DPDK >= 1.7) > >
[dpdk-dev] [PATCH] eal: fix build for bsd
Hi Zhaochen, Please read answer below: 2014-07-02 09:58, Zhan, Zhaochen: > I have tested the patch, it still has a build error although it is different > from the previous one. > > OS: FreeBSD10 > > == Build lib/librte_pmd_bond > CC rte_eth_bond_api.o > /root/zzc/dpdk-1.7.0-rc3/lib/librte_pmd_bond/rte_eth_bond_api.c:35:27: fatal > error: linux/binfmts.h: No such file or directory #include > Please check with the latest version in master branch (after rc3). It should be fixed by Helin's patch of today. Thanks -- Thomas > > -Original Message- > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > Sent: Wednesday, July 02, 2014 5:22 PM > > To: dev at dpdk.org > > Subject: [dpdk-dev] [PATCH] eal: fix build for bsd > > > > When adding link bonding to EAL initialization (a155d430119), > > an include was missing for BSD. > > > > Signed-off-by: Thomas Monjalon > > --- > > > > lib/librte_eal/bsdapp/eal/eal.c | 1 + > > 1 file changed, 1 insertion(+) > > > > Please, a quick test and acknowledgement of this fix would be helpful. > > > > > > diff --git a/lib/librte_eal/bsdapp/eal/eal.c > > b/lib/librte_eal/bsdapp/eal/eal.c index ec57b26..69ead31 100644 > > --- a/lib/librte_eal/bsdapp/eal/eal.c > > +++ b/lib/librte_eal/bsdapp/eal/eal.c > > @@ -66,6 +66,7 @@ > > > > #include > > #include > > #include > > > > +#include > > > > #include > > #include > > #include > > > > -- > > 2.0.0
[dpdk-dev] Intel DPDK: exception_path:RTE_ARCH
I followed and error is there again.
See more description here: make didnot worked. I still use 1.6. :-)
cubiq at WKS-LNX-006:~$ ${RTE_SDK}
cubiq at WKS-LNX-006:~$ ${RTE_TARGET}
cubiq at WKS-LNX-006:~$ export RTE_SDK=/home/cubiq/sothy/dpdk-1.6.0r2
cubiq at WKS-LNX-006:~$ export RTE_TARGET=x86_64-native-linuxapp-gcc
cubiq at WKS-LNX-006:~$ cd ${RTE_SDK}
cubiq at WKS-LNX-006:~/sothy/dpdk-1.6.0r2$ cd examples/exception_path/
cubiq at WKS-LNX-006:~/sothy/dpdk-1.6.0r2/examples/exception_path$ make
/home/cubiq/sothy/dpdk-1.6.0r2/mk/rte.extvars.mk:71:
/home/cubiq/sothy/dpdk-1.6.0r2/x86_64-native-linuxapp-gcc/.config: Aucun
fichier ou dossier de ce type
/home/cubiq/sothy/dpdk-1.6.0r2/mk/rte.vars.mk:99: *** RTE_ARCH is not
defined. Arr?t.
cubiq at WKS-LNX-006:~/sothy/dpdk-1.6.0r2/examples/exception_path$
On Wed, Jul 2, 2014 at 11:56 AM, Alex Markuze wrote:
> You need, to define the following variables before compiling, please refer
> to the DPDK documentation.
>
>
> export RTE_TARGET=x86_64-native-linuxapp-gcc
>
> export RTE_SDK=/home/user/dpdk
>
>
> On Wed, Jul 2, 2014 at 12:48 PM, sothy shan wrote:
>
>> Hello!
>>
>> I started playing Intel DPDK example. I used to compile exception_path
>> code.
>> When I do make command "make", I got an error,
>>
>> RTE_ARCH is not set. So I set the variable via terminal bash using
>>
>> export RTE_ARCH=x86_64
>>
>> Stilll it is not working. Any wrong anywhere?
>>
>> Thanks for your reponse.
>>
>> Best regards
>> Sothy
>>
>
>
[dpdk-dev] [PATCH] net: get rid of SET_ETHTOOL_OPS
On Wed, Jul 02, 2014 at 11:48:51AM +0200, Thomas Monjalon wrote: > Hi Flavio, > > 2014-07-01 15:19, Flavio Leitner: > > The SET_ETHTOOL_OPS has been removed from upstream, so it > > breaks the dpdk build with recent kernels. > > > > Signed-off-by: Flavio Leitner > > You are removing SET_ETHTOOL_OPS calls. Yes. > In a previous patch, Aaro Koskinen made the choice to redefine the macro > in kcompat files. I missed that one. > I don't know what will be the choice of Intel in the sourceforge base driver. > So I've applied Aaro's patch as there are less modifications of the base > driver: > http://dpdk.org/browse/dpdk/commit/?id=e0b7ca0c0383411 Since the macro is a simple pointer assignment, I didn't see any reason for keeping it around. But maybe I am missing something. fbl
[dpdk-dev] Intel DPDK: exception_path:RTE_ARCH
On Wed, Jul 2, 2014 at 12:04 PM, sothy shan wrote: > > > > On Wed, Jul 2, 2014 at 12:00 PM, Thomas Monjalon < > thomas.monjalon at 6wind.com> wrote: > >> 2014-07-02 12:56, Alex Markuze: >> > On Wed, Jul 2, 2014 at 12:48 PM, sothy shan >> wrote: >> > > I started playing Intel DPDK example. I used to compile exception_path >> > > code. >> > > When I do make command "make", I got an error, >> > > >> > > RTE_ARCH is not set. So I set the variable via terminal bash using >> > > >> > > export RTE_ARCH=x86_64 >> > > >> > > Stilll it is not working. Any wrong anywhere? >> > >> > You need, to define the following variables before compiling, please >> refer >> > to the DPDK documentation. >> > >> > export RTE_TARGET=x86_64-native-linuxapp-gcc >> > >> > export RTE_SDK=/home/user/dpdk >> >> You can also try to build all examples with these commands: >> RTE_TARGET=x86_64-native-linuxapp-gcc >> make T=$RTE_TARGET install >> make T=$RTE_TARGET examples O=$RTE_TARGET/examples >> >> (with DPDK >= 1.7) >> >> > > When using DPDK 1.7, example/exception_path works. now playing with it. thx
[dpdk-dev] [PATCH 0/7] build fixes
When enabling pcap PMD or debug options, there are many compiler errors. There is also another error on BSD. These errors are due to many recent developments (link bonding, rate limitation, ixgbe upgrade, vmxnet3, FILE argument for debug). So my comments are: - I don't test every build configuration when accepting patches - Many developers don't test debug configurations We should try to improve this process in the future. Please give quick feedbacks on this serie. It requires a new release candidate. Thanks for reading Thomas Monjalon (7): eal: fix build for bsd ethdev: fix build of Tx rate limitation debug ethdev: fix build of named allocation debug pcap: fix build virtio: fix build of debug dump vmxnet3: fix build with debug ixgbe/base: fix build with debug lib/librte_eal/bsdapp/eal/eal.c | 1 + lib/librte_ether/rte_ethdev.c| 6 +++--- lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h | 4 ++-- lib/librte_pmd_pcap/rte_eth_pcap.c | 2 +- lib/librte_pmd_virtio/virtio_rxtx.c | 2 +- lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c| 4 ++-- 6 files changed, 10 insertions(+), 9 deletions(-) -- 2.0.0
[dpdk-dev] [PATCH 3/7] ethdev: fix build of named allocation debug
The commit 83b41136934 (add unique name to devices) didn't compile if
CONFIG_RTE_LIBRTE_ETHDEV_DEBUG is enabled.
Signed-off-by: Thomas Monjalon
---
lib/librte_ether/rte_ethdev.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c
index 8dccc6f..fd1010a 100644
--- a/lib/librte_ether/rte_ethdev.c
+++ b/lib/librte_ether/rte_ethdev.c
@@ -180,7 +180,7 @@ rte_eth_dev_allocate(const char *name)
rte_eth_dev_data_alloc();
if (rte_eth_dev_allocated(name) != NULL) {
- PMD_DEBUG_TRACE("Ethernet Device with name %s already
allocated!\n");
+ PMD_DEBUG_TRACE("Ethernet Device with name %s already
allocated!\n", name);
return NULL;
}
--
2.0.0
[dpdk-dev] [PATCH 2/7] ethdev: fix build of Tx rate limitation debug
The commit 8dbe82b0733 (Tx rate limitation) didn't compile if
CONFIG_RTE_LIBRTE_ETHDEV_DEBUG is enabled.
Signed-off-by: Thomas Monjalon
---
lib/librte_ether/rte_ethdev.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c
index 0ebb8fb..8dccc6f 100644
--- a/lib/librte_ether/rte_ethdev.c
+++ b/lib/librte_ether/rte_ethdev.c
@@ -2261,7 +2261,7 @@ int rte_eth_set_queue_rate_limit(uint8_t port_id,
uint16_t queue_idx,
if (tx_rate > link.link_speed) {
PMD_DEBUG_TRACE("set queue rate limit:invalid tx_rate=%d, "
"bigger than link speed= %d\n",
- tx_rate, link_speed);
+ tx_rate, link.link_speed);
return -EINVAL;
}
@@ -2298,7 +2298,7 @@ int rte_eth_set_vf_rate_limit(uint8_t port_id, uint16_t
vf, uint16_t tx_rate,
if (tx_rate > link.link_speed) {
PMD_DEBUG_TRACE("set VF rate limit:invalid tx_rate=%d, "
"bigger than link speed= %d\n",
- tx_rate, link_speed);
+ tx_rate, link.link_speed);
return -EINVAL;
}
--
2.0.0
[dpdk-dev] [PATCH 4/7] pcap: fix build
The commit 83b41136934 (add unique name to devices) didn't compile if CONFIG_RTE_LIBRTE_PMD_PCAP is enabled. Signed-off-by: Thomas Monjalon --- lib/librte_pmd_pcap/rte_eth_pcap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/librte_pmd_pcap/rte_eth_pcap.c b/lib/librte_pmd_pcap/rte_eth_pcap.c index 12b7e0c..c77ee25 100644 --- a/lib/librte_pmd_pcap/rte_eth_pcap.c +++ b/lib/librte_pmd_pcap/rte_eth_pcap.c @@ -719,7 +719,7 @@ rte_pmd_pcap_devinit(const char *name, const char *params) if (ret < 0) return -1; - return rte_eth_from_pcaps(pcaps.pcaps, 1, pcaps.pcaps, 1, + return rte_eth_from_pcaps(name, pcaps.pcaps, 1, pcaps.pcaps, 1, numa_node, kvlist); } -- 2.0.0
[dpdk-dev] [PATCH 5/7] virtio: fix build of debug dump
The commit 591a9d7985c1230 (add FILE argument to debug functions) didn't
compile if CONFIG_RTE_LIBRTE_VIRTIO_DEBUG_DUMP is enabled.
Signed-off-by: Thomas Monjalon
---
lib/librte_pmd_virtio/virtio_rxtx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/librte_pmd_virtio/virtio_rxtx.c
b/lib/librte_pmd_virtio/virtio_rxtx.c
index 6d20331..6f02a61 100644
--- a/lib/librte_pmd_virtio/virtio_rxtx.c
+++ b/lib/librte_pmd_virtio/virtio_rxtx.c
@@ -55,7 +55,7 @@
#include "virtqueue.h"
#ifdef RTE_LIBRTE_VIRTIO_DEBUG_DUMP
-#define VIRTIO_DUMP_PACKET(m, len) rte_pktmbuf_dump(m, len)
+#define VIRTIO_DUMP_PACKET(m, len) rte_pktmbuf_dump(stdout, m, len)
#else
#define VIRTIO_DUMP_PACKET(m, len) do { } while (0)
#endif
--
2.0.0
[dpdk-dev] [PATCH 1/7] eal: fix build for bsd
When adding link bonding to EAL initialization (a155d430119), an include was missing for BSD. Signed-off-by: Thomas Monjalon Tested-by: Zhaochen Zhan --- lib/librte_eal/bsdapp/eal/eal.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/librte_eal/bsdapp/eal/eal.c b/lib/librte_eal/bsdapp/eal/eal.c index c53f63e..38c6cfc 100644 --- a/lib/librte_eal/bsdapp/eal/eal.c +++ b/lib/librte_eal/bsdapp/eal/eal.c @@ -66,6 +66,7 @@ #include #include #include +#include #include #include #include -- 2.0.0
[dpdk-dev] [PATCH 6/7] vmxnet3: fix build with debug
Functions for queue dump are not used and cause compilation error if
CONFIG_RTE_LIBRTE_VMXNET3_DEBUG_DRIVER is enabled.
Fixed by disabling them.
Signed-off-by: Thomas Monjalon
---
lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c
b/lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c
index c6aa963..13960fa 100644
--- a/lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c
+++ b/lib/librte_pmd_vmxnet3/vmxnet3_rxtx.c
@@ -90,7 +90,7 @@ static uint32_t rxprod_reg[2] = {VMXNET3_REG_RXPROD,
VMXNET3_REG_RXPROD2};
static inline int vmxnet3_post_rx_bufs(vmxnet3_rx_queue_t* , uint8_t);
static inline void vmxnet3_tq_tx_complete(vmxnet3_tx_queue_t *);
-#ifdef RTE_LIBRTE_VMXNET3_DEBUG_DRIVER
+#ifdef RTE_LIBRTE_VMXNET3_DEBUG_DRIVER_NOT_USED
static void vmxnet3_rxq_dump(struct vmxnet3_rx_queue *);
static void vmxnet3_txq_dump(struct vmxnet3_tx_queue *);
#endif
@@ -105,7 +105,7 @@ rte_rxmbuf_alloc(struct rte_mempool *mp)
return (m);
}
-#ifdef RTE_LIBRTE_VMXNET3_DEBUG_DRIVER
+#ifdef RTE_LIBRTE_VMXNET3_DEBUG_DRIVER_NOT_USED
static void
vmxnet3_rxq_dump(struct vmxnet3_rx_queue *rxq)
{
--
2.0.0
[dpdk-dev] [PATCH 7/7] ixgbe/base: fix build with debug
The upgraded base driver, especially commit 9ba80bde4c, didn't compile if CONFIG_RTE_LIBRTE_IXGBE_DEBUG_DRIVER is enabled. Signed-off-by: Thomas Monjalon --- lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h b/lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h index 1636ee1..2bf1a6d 100644 --- a/lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h +++ b/lib/librte_pmd_ixgbe/ixgbe/ixgbe_osdep.h @@ -63,8 +63,8 @@ #define DEBUGOUT7(S, args...) DEBUGOUT(S, ##args) #define ERROR_REPORT1(e, S, args...) DEBUGOUT(S, ##args) -#define ERROR_REPORT2 DEBUGOUT2 -#define ERROR_REPORT3 DEBUGOUT3 +#define ERROR_REPORT2(e, S, args...) DEBUGOUT(S, ##args) +#define ERROR_REPORT3(e, S, args...) DEBUGOUT(S, ##args) #define FALSE 0 #define TRUE1 -- 2.0.0
[dpdk-dev] [PATCH] net: get rid of SET_ETHTOOL_OPS
2014-07-02 10:07, Flavio Leitner: > On Wed, Jul 02, 2014 at 11:48:51AM +0200, Thomas Monjalon wrote: > > You are removing SET_ETHTOOL_OPS calls. > > Yes. > > > In a previous patch, Aaro Koskinen made the choice to redefine the macro > > in kcompat files. > > I missed that one. > > > I don't know what will be the choice of Intel in the sourceforge base > > driver. So I've applied Aaro's patch as there are less modifications of > > the base> > > driver: > > http://dpdk.org/browse/dpdk/commit/?id=e0b7ca0c0383411 > > Since the macro is a simple pointer assignment, I didn't see any reason for > keeping it around. But maybe I am missing something. Me too. Except that this base driver is synchronized (sometimes) with the sourceforge one. So we should try to minimize differences with it. That's why I'd prefer that people managing the sourceforge codebase push their patches here, or pull ours. But today, there is no rule. Bonus game: who can find the commits of those projects? http://sourceforge.net/projects/e1000 -- Thomas
[dpdk-dev] KNI hw Address.
Hi, I'm playing with KNI on a VM (kvm), the Interface that is created has no MAC address until the IP is set via ifconfig - then a random mac is created. The VF has a mac address that is easily retrieved with rte_eth_macaddr_get. What I did not find is a way to create the KNI with that specific mac address enabled. What are the ways to set the KNI mac address? Thanks Alex
[dpdk-dev] Testing memnic for VM to VM transfer
2014-06-18 12:06, Hiroshi Shimamoto: > 2014-06-18 13:49, Thomas Monjalon: > > 2014-06-18 11:42, Hiroshi Shimamoto: > > > 2014-06-18 19:26, GongJinrong: > > > > Do you have any idea that how to write a host application > > > > to put the data to guest memnic PMD? > > > > > > Yes, basically I made the MEMNIC interface work with DPDK vSwitch. > > > > > > By the way, you can mmap() the shm which specified as the ivshmem and > > > put > > > the proper data to send a packet to guest PMD. > > > I don't have time to make proper code, but can advise you; > > > please see common/memnic.h and the memory layout. > > > 1) Set magic and version in header on host. > > > 2) Initialize PMD on guest. > > > 3) Check the reset is 1 and set valid to 1, reset to 0 on host. > > > 4) Use uplink area the default block size 4K. > > > > > >Set len and fill ether frame data, then set the status to 2 on host. > > >Guest PMD may receive the packet. > > >Proceed to the next packet block. > > > > Such application should be integrated in memnic repository. > > I know Olivier wrote one which could be sent on next week. > > yeah, I just begin to feel to need such a software in the repository. I've pushed the test application (host-sim) that Olivier wrote. Feel free to improve it. I think that we could also use the PMD driver in host side with some modifications. -- Thomas
[dpdk-dev] [PATCH 0/7] build fixes
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Wednesday, July 02, 2014 8:03 AM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH 0/7] build fixes > > When enabling pcap PMD or debug options, there are many compiler errors. > There is also another error on BSD. > > These errors are due to many recent developments (link bonding, > rate limitation, ixgbe upgrade, vmxnet3, FILE argument for debug). > So my comments are: > - I don't test every build configuration when accepting patches > - Many developers don't test debug configurations > We should try to improve this process in the future. > > Please give quick feedbacks on this serie. > It requires a new release candidate. > > Thanks for reading > > > Thomas Monjalon (7): Acked-by: Bruce Richardson Tested compilation on Fedora Linux and FreeBSD 10 both in default configuration and with all debug options set for PMDs. No compile errors encountered with main compilation. However, two further minor issues spotted: * Running "gmake" in the examples directory gives errors for applications that only work in Linux. * The common_bsdapp file has all debug options turned on for the i40e driver by default.
