2015-02-19 21:48, Zhou Danny:
> v4 changes:
> - Adjust position of new-added structure fields
>
> v3 changes:
> - Fix review comments
>
> v2 changes:
> - Fix compilation issue for a missed header file
> - Bug fix: free unreleased resources on the exception path before return
> - Consolidate codin
> > v3 changes
> > - made processing of uio devices identical, irrespective of igb_uio or
> > uio_pci_generic
> > - removed storage of kernel driver name from dev node as now unneeded.
> >
> > v2 changes:
> > - Change variable name of kernel driver with precise comment
> > - Fix a union definition
> > v4 changes:
> > - license year modified
> > - aded license to rte_eth_bond_alb files
> > - removed RTE_EXEC_ENV_BAREMETAL in #define
> >
> > v3 changes:
> > - completed description for mode 5 unit tests patch
> > - fixed errors required by checkpatch.pl
> > - moved patch version changes from pa
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ildar Mustafin
> Sent: Friday, February 20, 2015 10:35 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] RTE_PIPELINE_ACTION_PORT_META fix for
> non-default entries
>
> ---
> lib/librte_pipeline/rte_pipe
2015-02-20 20:23, Dumitrescu, Cristian:
> From: Stephen Hemminger [mailto:stephen at networkplumber.org]
> > On Fri, 20 Feb 2015 18:32:03 +
> > "Dumitrescu, Cristian" wrote:
> >
> > > Stephen, I suggest adding a new build-time configuration option for the
> > > librte_sched library in config/
2015-02-20 19:26, Dumitrescu, Cristian:
>
> > -Original Message-
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of ??
> > Sent: Friday, February 20, 2015 4:25 PM
> > To: dev at dpdk.org
> > Subject: [dpdk-dev] RTE_PIPELINE_ACTION_PORT_META doesn't work
> > properly
>
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Friday, February 20, 2015 9:01 PM
> To: Dumitrescu, Cristian
> Cc: dev at dpdk.org; Stephen Hemminger
> Subject: Re: [dpdk-dev] [PATCH v2 4/7] rte_sched: don't clear statistics when
> read
>
> 2015
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Friday, February 20, 2015 8:47 PM
> To: Dumitrescu, Cristian; imustafin at bk.ru
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] RTE_PIPELINE_ACTION_PORT_META doesn't work
> properly
>
> 2015-02-20
> -Original Message-
> From: Stephen Hemminger [mailto:stephen at networkplumber.org]
> Sent: Friday, February 20, 2015 7:53 PM
> To: Dumitrescu, Cristian
> Cc: dev at dpdk.org; Stephen Hemminger
> Subject: Re: [dpdk-dev] [PATCH v2 4/7] rte_sched: don't clear statistics when
> read
>
> O
On 2015/02/20 19:14, Maxime Leroy wrote:
> Hi Tetsuya,
>
> On Fri, Feb 20, 2015 at 7:39 AM, Tetsuya Mukawa wrote:
>> These functions are used for attaching or detaching a port.
> [..]
>> +
>> +static void
>> +get_vdev_name(char *vdevargs)
>> +{
>> + char *sep;
>> +
>> + if (vdevargs ==
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of ??
> Sent: Friday, February 20, 2015 4:25 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] RTE_PIPELINE_ACTION_PORT_META doesn't work
> properly
>
> Hi, list!
>
> RTE_PIPELINE_ACTION_PORT_META opt
Hi, list!
RTE_PIPELINE_ACTION_PORT_META option in rte_pipeline library doesn't work for
non-default table entries. Is this bug or correct behaviour?
This is my patch, that changes this behaviour:
--- a/lib/librte_pipeline/rte_pipeline.c
+++ b/lib/librte_pipeline/rte_pipeline.c
@@ -999,8 +999,1
> On 2/9/2015 9:14 AM, Ouyang, Changchun wrote:
> > This is the patch set for single virtio implementation.
> >
> > Why we need single virtio?
> >
> > As we know currently there are at least 3 virtio PMD driver implementations:
> > A) lib/librte_pmd_virtio(refer as vi
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen
> Hemminger
> Sent: Thursday, February 5, 2015 6:14 AM
> To: dev at dpdk.org
> Cc: Stephen Hemminger
> Subject: [dpdk-dev] [PATCH v2 7/7] rte_sched: rearrange data structures
>
> From: Stephen Hemminge
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen
> Hemminger
> Sent: Thursday, February 5, 2015 6:14 AM
> To: dev at dpdk.org
> Cc: Stephen Hemminger
> Subject: [dpdk-dev] [PATCH v2 5/7] rte_sched: don't put tabs in log
> messages
>
> From: Stephen H
> -Original Message-
> From: Stephen Hemminger [mailto:stephen at networkplumber.org]
> Sent: Monday, February 9, 2015 10:55 PM
> To: Dumitrescu, Cristian
> Cc: dev at dpdk.org; Stephen Hemminger
> Subject: Re: [dpdk-dev] [PATCH v2 4/7] rte_sched: don't clear statistics when
> read
>
> O
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen
> Hemminger
> Sent: Thursday, February 5, 2015 6:05 AM
> To: dev at dpdk.org
> Cc: Stephen Hemminger
> Subject: [dpdk-dev] [PATCH v2 3/7] rte_sched: keep track of RED drops
>
> From: Stephen Hemminger
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen
> Hemminger
> Sent: Thursday, February 5, 2015 6:05 AM
> To: dev at dpdk.org
> Cc: Stephen Hemminger
> Subject: [dpdk-dev] [PATCH v2 2/7] rte_sched: expand scheduler hierarchy
> for more VLAN's
>
> Fro
On Fri, 20 Feb 2015 13:15:38 +0100
Thomas Monjalon wrote:
> 2015-02-12 17:18, Panu Matilainen:
> > Add RTE_DEVEL_BUILD make-variable which can be used to do things
> > differently when doing development vs building a release,
> > autodetected from source root .git presence and overridable via
> >
On Thu, 12 Feb 2015 16:54:44 +0200
Panu Matilainen wrote:
> On 02/12/2015 04:38 PM, Stephen Hemminger wrote:
> > On Thu, 12 Feb 2015 13:13:22 +0200
> > Panu Matilainen wrote:
> >
> >> This adds new CONFIG_RTE_ERROR_ON_WARNING config option to enable
> >> fail-on-warning compile behavior, default
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen
> Hemminger
> Sent: Thursday, February 5, 2015 6:05 AM
> To: dev at dpdk.org
> Cc: Stephen Hemminger
> Subject: [dpdk-dev] [PATCH v2 1/7] rte_sched: make RED optional at runtime
>
> From: Stephen Hemmi
On Fri, 20 Feb 2015 21:28:55 +
"Dumitrescu, Cristian" wrote:
> Agree.
> Stephen, how about a run-time solution (I agree it would be much better, why
> did I not consider this in the first place?) of adding a new bool parameter
> in struct rte_sched_port_params: clear_stats_on_reset?
> Both
On 20/02/15 16:09, Michal Jastrzebski wrote:
> v4 changes:
> - license year modified
> - aded license to rte_eth_bond_alb files
> - removed RTE_EXEC_ENV_BAREMETAL in #define
>
> v3 changes:
> - completed description for mode 5 unit tests patch
> - fixed errors required by checkpatch.pl
> - moved pa
On 20/02/15 16:59, Bruce Richardson wrote:
> v3 changes
> - made processing of uio devices identical, irrespective of igb_uio or
> uio_pci_generic
> - removed storage of kernel driver name from dev node as now unneeded.
>
> v2 changes:
> - Change variable name of kernel driver with precise comment
> -Original Message-
> From: Wu, Jingjing
> Sent: Tuesday, February 10, 2015 4:48 AM
> To: dev at dpdk.org
> Cc: Wu, Jingjing; De Lara Guarch, Pablo; Cao, Min; Xu, HuilongX
> Subject: [PATCH v3 0/6] new ntuple filter replaces 2tuple and 5tuple filters
>
> v2 changes:
> - remove the cod
On 02/20/2015 04:04 PM, Thomas Monjalon wrote:
> Hi Panu,
>
> 2015-02-18 14:11, Panu Matilainen:
>> Separately comparing major and minor versions becomes seriously clumsy
>> when with major version changes, convert the entire version string into
>> a numeric value (ie 4.6.0 becomes 460 and 5.0.0 be
Function match_inst is used to take buffor using sizeof() which is size_t type.
This modification also involved changing '%u' to '%zu' in printf function.
Signed-off-by: Daniel Mrzyglod
---
lib/librte_cmdline/cmdline_parse.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a
From: Maciej Gajdzica
Added 4 unit tests checking link bonding mode 6 behavior.
Also modified virtual_pmd so it is possible to provide packets,
that should be received with rx_burst and to inspect packets
transmitted by tx_burst.
In packet_burst_generator.c function creating eth_header is
modif
From: Daniel Mrzyglod
This patch modify mode older name from
BONDING_MODE_ADAPTIVE_TRANSMIT_LOAD_BALANCING to BONDING_MODE_TLB
This patch also changes order of TEST_ASSERT macro in
test_tlb_verify_slave_link_status_change_failover.
Signed-off-by: Daniel Mrzyglod
---
app/test/test_link_bonding.
This patch contains an example for link bonding mode 6.
It interact with user by a command prompt. Available commands are:
Start - starts ARP_thread which respond to ARP_requests and sends
ARP_updates (this
Is enabled by default after startup),
Stop -stops ARP_thread,
Send count ip - send count AR
This patch add some debug information when using link bonding mode 6.
It prints basic information about ARP packets on RX and TX (MAC, ip,
packet number, arp packet type).
If CONFIG_RTE_LIBRTE_BOND_DEBUG_ALB == y.
If CONFIG_RTE_LIBRTE_BOND_DEBUG_ALB_L1 is enabled instead of previous
one, use show c
From: Maciej Gajdzica
This mode includes adaptive TLB and receive load balancing (RLB). In RLB
the bonding driver intercepts ARP replies send by local system and
overwrites its source MAC address, so that different peers send data to
the server on different slave interfaces. When local system sen
From: Maciej Gajdzica
Changed MAC address type from uint8_t[6] to struct ether_addr and IP
address type from uint8_t[4] to uint32_t to make it consistent with
other DPDK code using MAC and IP addresses. It allows us to use
is_same_ether_addr and ether_addr_copy functions on MAC addresses in ARP
v4 changes:
- license year modified
- aded license to rte_eth_bond_alb files
- removed RTE_EXEC_ENV_BAREMETAL in #define
v3 changes:
- completed description for mode 5 unit tests patch
- fixed errors required by checkpatch.pl
- moved patch version changes from patches to cover-letter
v2 changes:
From: Richardson, Bruce
Example showing how callbacks can be used to insert a timestamp
into each packet on RX. On TX the timestamp is used to calculate
the packet latency through the app, in cycles.
Signed-off-by: Bruce Richardson
Signed-off-by: John McNamara
---
MAINTAINERS
From: Richardson, Bruce
Add optional support for inline processing of packets inside the RX
or TX call. For an RX callback, what happens is that we get a set of
packets from the NIC and then pass them to a callback function, if
configured, to allow additional processing to be done on them, e.g.
f
From: Richardson, Bruce
The 'callbacks' member of the rte_eth_dev structure has been renamed
to 'link_intr_cbs' to make it clear that it refers to callbacks from
NIC interrupts. This allows us to add other types of callbacks to
the structure without ambiguity.
Signed-off-by: Bruce Richardson
Si
This patchset is for a small optional addition to the ethdev library,
to add support for callbacks at the RX and TX stages. This allows
packet processing to be done on packets before they get returned
to applications using rte_eth_rx_burst call.
See the RFC cover letter for the use cases:
htt
From: Jijiang Liu
Enhance csum fwd engine based on current TX checksum framework in order
to test TX Checksum offload for NVGRE packet.
It includes:
- IPv4 and IPv6 packet
- outer L3, inner L3 and L4 checksum offload for Tx side.
Signed-off-by: Declan Doherty
---
app/test-pmd/csumonly.c | 2
From: Jijiang Liu
Extend the "tunnel_filter" command in testpmd to test the RX tunnel filter API
for NVGRE packet.
Signed-off-by: Jijiang Liu
Signed-off-by: Declan Doherty
---
app/test-pmd/cmdline.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/app/test-pmd
From: Jijiang Liu
The filter types supported are listed below for NVGRE packet:
1. Inner MAC and Inner VLAN ID.
2. Inner MAC address, inner VLAN ID and tenant ID.
3. Inner MAC and tenant ID.
4. Inner MAC address.
5. Outer MAC address, tenant ID and inner MAC address.
Signed-of
From: Jijiang Liu
Add an Ethernet type definition for Transparent Ethernet Bridging.
Signed-off-by: Jijiang Liu
Signed-off-by: Declan Doherty
---
lib/librte_ether/rte_ether.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/librte_ether/rte_ether.h b/lib/librte_ether/rte_ether.h
index
The patch set supports NVGRE on i40e.
It includes:
- Support RX filters for NVGRE packet. It uses MAC and VLAN to point
to a queue. The filter types supported are listed below:
1. Inner MAC and Inner VLAN ID
2. Inner MAC address, inner VLAN ID and tenant ID.
3. Inner MAC and tenant
From: Zhou Danny
Add uio_pci_generic to the list of supported kernel drivers.
Signed-off-by: Danny Zhou
Tested-by: Qun Wan
Signed-off-by: Bruce Richardson
---
tools/dpdk_nic_bind.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/dpdk_nic_bind.py b/tools/dpdk_nic_bi
From: Zhou Danny
enable/disable interrupt by manipulating a control bit of command
register on NIC's PCIe configuration space.
Signed-off-by: Danny Zhou
Tested-by: Qun Wan
Signed-off-by: Bruce Richardson
---
lib/librte_eal/linuxapp/eal/eal_interrupts.c | 68 +---
1 fi
From: Zhou Danny
Change the EAL PCI code so that it can work with both the
uio_pci_generic in-tree driver, as well as the igb_uio
DPDK-specific driver.
This involves changes to
1) Modify method of retrieving BAR resource mapping information
2) Mapping using resource files in /sys rather than /de
v3 changes
- made processing of uio devices identical, irrespective of igb_uio or
uio_pci_generic
- removed storage of kernel driver name from dev node as now unneeded.
v2 changes:
- Change variable name of kernel driver with precise comment
- Fix a union definition error in v1 patchset
- Move re
2015-02-20 14:54, Gajdzica, MaciejX T:
> > 2015-02-19 18:26, Michal Jastrzebski:
> > > From: Maciej Gajdzica
> > >
> > > Changed MAC address type from uint8_t[6] to struct ether_addr and IP
> > > address type from uint8_t[4] to uint32_t. Also removed union from
> > > arp_hdr struct. Updated test-p
Hi Tetsuya,
Thanks for your comment.
On Fri, Feb 20, 2015 at 11:32 AM, Tetsuya Mukawa wrote:
> On 2015/02/20 19:14, Maxime Leroy wrote:
>> Hi Tetsuya,
[...]
>>
>
> Hi Maxime,
>
> I appreciate for your comment.
>
> When rte_eal_init() is called, if we have "--vdev" options, these will
> be stored
LPM table overflow may occur if table is full and added rule has the biggest
depth that already have some rules.
Signed-off-by: Igor Ryzhov
---
lib/librte_lpm/rte_lpm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/lib/librte_lpm/rte_lpm.c b/lib/librte_lpm/rte_lpm.c
index 983e04b..cc51
Macro GET_CONFIG passes result of sizeof operation to the function
vnic_dev_spec. This function expects parameter of type unsigned int.
Changed that parameter type to size_t in function declaration to prevent
possible data loss. Issue found with static code analysis tool.
Signed-off-by: Maciej Gaj
In function writeq is written in two 32-bit long registers with writel
function. When trying to write val >> 32, static code analysis tool
reports that 64-bit value is passed to function expecting 32-bit value.
Added cast to clear this warning.
Signed-off-by: Maciej Gajdzica
---
lib/librte_pmd_e
Define ENIC_ALIGN is used only by function rte_memzone_reserve_aligned,
which expects argument of type unsigned. Defined constant is of type
unsigned long long. Changed type to unsigned long to prevent possible
loss of precision. Issue found with static code analysis tool.
Signed-off-by: Maciej Ga
Field ig_vlan_strip_en in struct enic type is int. It is used only by
function enic_set_nic_cfg which expects uint_8 as argument. Changed type
of the field to prevent possible loss of precision. Issue found with
static code analysis tool.
Signed-off-by: Maciej Gajdzica
---
lib/librte_pmd_enic/en
Variable rcv_cnt declared in function virtio_recv_mergeable_pkts was of
type uint32_t. It was used by virtqueue_dequeue_burst_rx function, which
expects argument of type uint16_t. Changed rcv_cnt variable type to
uint16_t to prevent possible data loss. Issue found with static code
analysis tool.
S
Changed vtpci_queue_idx type in function virtio_dev_queue_setup from
uint8_t to uint16_t to prevent possible data loss. Also changed type of
head variable in function virtio_send_command from uint32_t to uint16_t.
Issues found with static code analysis tool.
Signed-off-by: Maciej Gajdzica
---
li
In some places slave_id is declared as uint16_t, which causes data loss
when using functions expecting port_id as uint8_t. Issue found with
static code analysis tool.
Signed-off-by: Maciej Gajdzica
---
lib/librte_pmd_bond/rte_eth_bond_8023ad.c |4 ++--
1 file changed, 2 insertions(+), 2 dele
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Friday, February 20, 2015 3:43 PM
> To: Jastrzebski, MichalX K
> Cc: dev at dpdk.org; Butler, Siobhan A
> Subject: Re: [dpdk-dev] [PATCH v3 4/6] bond: add example application for
> link bonding mode 6
In some places port_id is declared as int, which causes data loss when
using functions expecting port_id as uint8_t. Issue found with static
code analysis tool.
Signed-off-by: Maciej Gajdzica
---
lib/librte_pmd_bond/rte_eth_bond_pmd.c |5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
Previous type of rx and tx queue_id field was int, which caused possible
data loss when calling functions expecting uint16_t as an argument.
Issue found with static code analysis tool.
Signed-off-by: Maciej Gajdzica
---
lib/librte_pmd_bond/rte_eth_bond_pmd.c |3 ++-
lib/librte_pmd_bond/r
Hello,
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of David Marchand
> Sent: Monday, February 9, 2015 10:11 AM
> To: Liu, Jijiang
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 2/2] i40e:enable TSO support
>
> Hello,
>
> This patch does two things
2015-02-13 09:48, Jijiang Liu:
> This patch set enables i40e TSO feature for both non-tunneling packet and
> tunneling packet.
>
> Change logs:
> v2 change: rework based on Olivier's patch set [PATCH v2 00/20] enhance
> tx checksum offload API
> http://dpdk.org/ml/archives/dev/
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Neil Horman
> Sent: Thursday, February 19, 2015 3:34 PM
> To: Wodkowski, PawelX
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v5 0/3] new headroom stats library and
> example application
>
> On Thu, Feb
2015-02-19 18:26, Michal Jastrzebski:
> + * Copyright(c) 2010-2014 Intel Corporation. All rights reserved.
You should check the date.
Is there some reviews of the copyright at Intel? It's quite common to see
these dates starting and finishing too early.
> +#ifdef RTE_EXEC_ENV_BAREMETAL
> +#defi
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
Null PMD is a driver of the virtual device particularly designed to measure
performance of DPDK PMDs. When an application call rx, Null PMD just allocates
mbufs and returns those. Also tx, the PMD just frees mbufs.
The PMD has following options.
- size: specify packe size allocated by RX. Default
The patch introduces following commands.
- port attach [ident]
- port detach [port_id]
- attach: attaching a port
- detach: detaching a port
- ident: pci address of physical device.
Or device name and parameters of virtual device.
(ex. :02:00.0, eth_pcap0,iface=eth0)
- po
This patch adds finalization code to free resources allocated by the
PMD.
v6:
- Fix a paramter of rte_eth_dev_free().
v4:
- Change function name.
Signed-off-by: Tetsuya Mukawa
---
lib/librte_pmd_pcap/rte_eth_pcap.c | 40 ++
1 file changed, 40 insertions(+)
This patch adds a new section for describing port hotplug framework.
Signed-off-by: Tetsuya Mukawa
---
doc/guides/prog_guide/index.rst | 1 +
doc/guides/prog_guide/port_hotplug_framework.rst | 110 +++
2 files changed, 111 insertions(+)
create mode 100644
These functions are used for attaching or detaching a port.
When rte_eal_dev_attach() is called, the function tries to realize the
device name as pci address. If this is done successfully,
rte_eal_dev_attach() will attach physical device port. If not, attaches
virtual devive port.
When rte_eal_dev_
This new parameter is needed to keep device type like PCI or virtual.
Port detaching processes are different between PCI device and virtual
device.
RTE_ETH_DEV_PCI indicates device type is PCI. RTE_ETH_DEV_VIRTUAL
indicates device is virtual.
v10:
- Change order of version.map.
(Thanks to Thomas
- Add pci_close_all_drivers()
The function tries to find a driver for the specified device, and
then close the driver.
- Add rte_eal_pci_probe_one() and rte_eal_pci_close_one()
The functions are used for probe and close a device.
First the function tries to find a device that has the specif
The function removes the specified devargs entry from devargs_list.
Also, the patch adds sanity checking to rte_eal_devargs_add().
v5:
- Change function definition of rte_eal_devargs_remove().
v4:
- Fix sanity check code.
Signed-off-by: Tetsuya Mukawa
---
lib/librte_eal/common/eal_common_devarg
The patch adds functions for unmapping igb_uio resources. The patch is only
for Linux and igb_uio environment. VFIO and BSD are not supported.
v9:
- Remove "rte_dev_hotplug.h".
- Remove needless "#ifdef".
(Thanks to Thomas Monjalon and Neil Horman)
- Remove pci_unmap_device(). It will be impleme
The patch adds following functions.
- rte_eth_dev_save()
The function is used for saving current rte_eth_dev structures.
- rte_eth_dev_get_changed_port()
The function receives the rte_eth_dev structures, then compare
these with current values to know which port is actually
attached or deta
The patch adds function pointer to rte_pci_driver and eth_driver
structure. These function pointers are used when ports are detached.
Also, the patch adds rte_eth_dev_uninit(). So far, it's not called
by anywhere, but it will be called when port hotplug function is
implemented.
v10:
- Add size par
This patch adds rte_eth_dev_release_port(). The function is used for
changing an attached status of the device that has specified name.
v9:
- rte_eth_dev_free() is replaced by rte_eth_dev_release_port().
(Thanks to Thomas Monjalon)
v6:
- Use rte_eth_dev structure as the paramter of rte_eth_dev_f
This patch replaces pci_addr_comparison() and memcmp() of pci addresses by
rte_eal_compare_pci_addr().
To compare PCI addresses, rte_eal_compare_pci_addr() doesn't use memcmp().
This is because sizeof(struct rte_pci_addr) returns 6, but actually
this structure is like below.
struct rte_pci_addr {
To remove assumption, do like followings.
This patch adds "RTE_PCI_DRV_DETACHABLE" to drv_flags of rte_pci_driver
structure. The flags indicate the driver can detach devices at runtime.
Also, remove assumption that port will not be detached.
To remove the assumption.
- Add 'attached' member to rt
From: Michael Qiu
With the driver type flag in struct rte_pci_dev, we do not need
to always map uio devices with vfio related function when
vfio enabled.
Signed-off-by: Michael Qiu
Signed-off-by: Tetsuya Mukawa
---
lib/librte_eal/linuxapp/eal/eal_pci.c | 30 +-
1
From: Michael Qiu
Currently, dpdk has no ability to know which type of driver(
vfio-pci/igb_uio/uio_pci_generic) the device used. It only can
check whether vfio is enabled or not staticly.
It really useful to have the flag, becasue different type need to
handle differently in runtime. For exampl
The patch adds CONFIG_RTE_LIBRTE_EAL_HOTPLUG in Linux and BSD
configuration. So far, Hotplug functions only support linux.
v9:
- Move this patch at the top of this patch series.
(Thanks to Thomas Monjalon)
Signed-off-by: Tetsuya Mukawa
---
config/common_bsdapp | 6 ++
config/common_linu
This patch series adds a dynamic port hotplug framework to DPDK.
With the patches, DPDK apps can attach or detach ports at runtime.
The basic concept of the port hotplug is like followings.
- DPDK apps must have responsibility to manage ports.
DPDK apps only know which ports are attached or deta
2015-02-19 18:26, Michal Jastrzebski:
> From: Maciej Gajdzica
>
> Changed MAC address type from uint8_t[6] to struct ether_addr and IP
> address type from uint8_t[4] to uint32_t. Also removed union from
> arp_hdr struct. Updated test-pmd to match new arp_hdr version.
>
> Signed-off-by: Maciej Ga
Fixed two more places where non-C++11 string concatenation is
used. Spotted by John McNamara.
Signed-off-by: Stefan Puiu
---
lib/librte_mempool/rte_mempool.h |8
lib/librte_vhost/vhost-net-cdev.h |4 ++--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/lib/librte
From: Stefan Puiu
In C++11 concatenated string literals need to have a
space in between. Found with clang++-3.4, IIRC g++-4.8 also complains about
this.
Sample error message:
dpdk/include/rte_pci.h:96:26: error: invalid suffix on literal; C++11 requires
a space between literal and identifie
2015-02-20 11:18, kuldeep.samasi at wipro.com:
> I am using DPDK-1.7.1 and want to enhance the packet processing speed .
> But couldn't get how memory get initialize and lcore - port-queue mapping .
>
> Suggestions are welcome on this .
The description of your problem is too short and vague.
The
In C++11 concatenated string literals need to have a
space in between. Found with clang++-3.4, IIRC g++-4.8 also complains about
this.
Sample error message:
dpdk/include/rte_pci.h:96:26: error: invalid suffix on literal; C++11 requires
a space between literal and identifier [-Wreserved-user-de
> > Eliminate ambiguity in the condition which trips up a "logical not
> > is only applied to the left..." warning from gcc 5, causing build
> > failure with -Werror. Besides non-ambiguous, the condition is
> > far more obvious this way.
> >
> > Signed-off-by: Panu Matilainen
>
> Acked-by: Konst
Hi Panu,
2015-02-18 14:11, Panu Matilainen:
> Separately comparing major and minor versions becomes seriously clumsy
> when with major version changes, convert the entire version string into
> a numeric value (ie 4.6.0 becomes 460 and 5.0.0 becomes 500) and use
> that for comparisons. This simplif
> 2015-02-19 18:26, Michal Jastrzebski:
> > From: Maciej Gajdzica
> >
> > Changed MAC address type from uint8_t[6] to struct ether_addr and IP
> > address type from uint8_t[4] to uint32_t. Also removed union from
> > arp_hdr struct. Updated test-pmd to match new arp_hdr version.
> >
> > Signed-off
On 13/02/2015 12:51, Neil Horman wrote:
> On Fri, Feb 13, 2015 at 11:08:02AM +, Gonzalez Monroy, Sergio wrote:
>> On 13/02/2015 10:14, Panu Matilainen wrote:
>>> On 02/12/2015 05:52 PM, Neil Horman wrote:
On Thu, Feb 12, 2015 at 04:07:50PM +0200, Panu Matilainen wrote:
> On 02/12/2015
Hi and thanks for reviewing,
I seem to have somehow missed the email for this one and thought it
got lost. Sorry.
On Fri, Feb 13, 2015 at 12:27 PM, Mcnamara, John
wrote:
>> -Original Message-
>> From: Richardson, Bruce
>> Sent: Friday, February 13, 2015 10:18 AM
>> To: Mcnamara, John
>>
> -Original Message-
> From: Olivier MATZ [mailto:olivier.matz at 6wind.com]
> Sent: Tuesday, February 17, 2015 5:02 PM
> To: Zhang, Helin; dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH v3 03/16] mbuf: add definitions of unified
> packet
> types
>
> Hi Helin,
>
> On 02/17/2015 07:59
Hi,
On Fri, Feb 20, 2015 at 1:26 PM, Thomas Monjalon
wrote:
> Hi Stefan,
>
> Thanks for improving DPDK for C++ usage.
>
> I'm a bit lost because that's the fourth time you send the same patch
> (without versioning):
> http://dpdk.org/dev/patchwork/patch/3140/
> (this one had some comment
On 2015/02/19 22:29, Tetsuya Mukawa wrote:
> On 2015/02/19 20:17, Thomas Monjalon wrote:
>>> @@ -152,6 +159,7 @@ struct rte_pci_device {
>>> uint16_t max_vfs; /**< sriov enable if not zero */
>>> int numa_node; /**< NUMA node connection */
>>>
On Fri, Feb 20, 2015 at 11:19:59AM +0100, Thomas Monjalon wrote:
> Hi Cunming,
>
> You would have more chance to have a review by CC'ing John.
> I checked your patch and have a comment below.
>
> 2015-02-12 17:08, Cunming Liang:
> > Klocwork report 'req' might be used uninitialized.
> > In some c
2015-02-12 17:18, Panu Matilainen:
> Add RTE_DEVEL_BUILD make-variable which can be used to do things
> differently when doing development vs building a release,
> autodetected from source root .git presence and overridable via
> commandline. Use it to only enable -Werror compiler flag when
> build
Hi Stefan,
Thanks for improving DPDK for C++ usage.
I'm a bit lost because that's the fourth time you send the same patch (without
versioning):
http://dpdk.org/dev/patchwork/patch/3140/
(this one had some comments you didn't address)
http://dpdk.org/dev/patchwork/patch/3517/
(is
2015-02-17 15:52, Pablo de Lara:
> rx_fc_en and tx_fc_en in cmd_link_flow_ctrl_set_parsed
> could be used without being initialized.
>
> Signed-off-by: Pablo de Lara
Applied, thanks
1 - 100 of 134 matches
Mail list logo
