> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jijiang Liu
> Sent: Wednesday, November 4, 2015 6:54 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH v3 0/8] add vhost TX offload support
>
> Adds vhost TX offload support.
>
> The patch set add the neg
2015-11-04 18:54, Jijiang Liu:
> + /* if vhost TX checksum offload is required */
> + if (m->ol_flags & PKT_TX_IP_CKSUM) {
> + hdr->csum_start = m->l2_len;
> + hdr->csum_offset = offsetof(struct ipv4_hdr, hdr_checksum);
> + } else if (m->ol_flags & PKT_
On 2015-11-04 11:35, Thomas Monjalon wrote:
> 2015-08-20 08:51, Simon Kagstrom:
>> -ifeq ($(shell test -f /proc/version_signature && lsb_release -si
>> 2>/dev/null),Ubuntu)
>> +ifeq ($(shell lsb_release -si 2>/dev/null),Ubuntu)
>> MODULE_CFLAGS += -DUBUNTU_RELEASE_CODE=$(shell lsb_release -sr | t
> > The old statistics on i40e only counted the packets on ports.
> > So the discarding packets on VSI were not counted.
> > This patch is to make statistics for packets both on ports and VSI.
> > Also update release notes.
> >
> > Signed-off-by: Xutao Sun
>
> Acked-by: Harry van Haaren
Applie
2015-11-04 11:29, Jingjing Wu:
> This patch set extends flow director to VF filtering in i40e driver.
>
> v2 change:
> - rework the doc, including release notes and testpmd guide
>
> v3 change:
> - rebase doc update to the same commit with code change
>
> v4 change:
> - remove the fix of disa
Hi Thomas,
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Wednesday, November 4, 2015 7:18 PM
> To: Liu, Jijiang
> Cc: dev at dpdk.org; Michael S. Tsirkin
> Subject: Re: [dpdk-dev] [PATCH v3 6/8] driver/virtio:enqueue vhost
The following code
On Wed, Nov 4, 2015 at 9:45 AM, Cunming Liang
wrote:
> During VFIO_DEVICE_SET_IRQS, the previous order is {Q0_fd, ... Qn_fd,
> misc_fd}.
> The vector number of misc is indeterminable which is ugly to some NIC(e.g.
> i40e, fm10k).
> The patch adjusts the order in {misc_fd, Q0_fd, ... Qn_fd}, alway
On Wed, Nov 4, 2015 at 9:45 AM, Cunming Liang
wrote:
> The patch adds condition check to avoid enable nothing.
> In disable state, both max_intr and nb_efd are zero.
>
> Signed-off-by: Cunming Liang
>
Acked-by: David Marchand
--
David Marchand
Hi Thomas,
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Wednesday, November 4, 2015 7:18 PM
> To: Liu, Jijiang
> Cc: dev at dpdk.org; Michael S. Tsirkin
> Subject: Re: [dpdk-dev] [PATCH v3 6/8] driver/virtio:enqueue vhost TX
> offload
>
> 2015
On Wed, Nov 4, 2015 at 9:45 AM, Cunming Liang
wrote:
> VFIO allows multiple MSI-X vector, others doesn't, but maybe will allow it
> in the future.
> Device drivers need to be aware of the capability.
> It's better to avoid condition check on interrupt type(VFIO) everywhere,
> instead
> a capabili
> -Original Message-
> From: Liu, Jijiang
> Sent: Wednesday, November 4, 2015 8:52 PM
> To: 'Thomas Monjalon'
> Cc: dev at dpdk.org; Michael S. Tsirkin
> Subject: RE: [dpdk-dev] [PATCH v3 6/8] driver/virtio:enqueue vhost TX
> offload
>
> Hi Thomas,
>
>
> > -Original Message-
>
On Tue, Nov 3, 2015 at 6:28 PM, Stephen Hemminger
wrote:
>
> No. configuration is the enemy of usability.
> Why does DPDK have to behave differently than BSD and Linux, what possible
> value could this be to the end user?
I honestly can't think of any good reasons for why a user would need
this.
The following structure is defined in virtio standard,
struct virtio_net_hdr {
#define VIRTIO_NET_HDR_F_NEEDS_CSUM 1
u8 flags;
#define VIRTIO_NET_HDR_GSO_NONE 0
#define VIRTIO_NET_HDR_GSO_TCPV4 1
#define VIRTIO_NET_HDR_GSO_UDP 3
#define VIRTIO_NET_HDR_GSO_TCPV6 4
#define VIRTIO_NET_HDR_GSO_ECN 0x
2015-11-04 12:52, Liu, Jijiang:
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> > Please could you check that your virtio implementation works with a vanilla
> > Linux with or without vhost?
> > Thanks
[...]
> Xu Qian can send the test report out.
Yes please, I'd like to see a te
Its a RFC patch and I am not sure about the ordering behavior in
tile and ppc_64 architecture so kept as rte_compiler_barrier()
as existing code.
If it need changes in tile and ppc_64 then let me know, I will send the
updated patch.
Jerin Jacob (1):
Introduce rte_smp_*mb() for memory barrier
This commit introduce rte_smp_mb(), rte_smp_wmb() and rte_smp_rmb(), in
order to enable memory barriers between lcores.
The patch does not provide any functional change for IA, the goal is to
have infrastructure for weakly ordered machines like ARM to work on DPDK.
Signed-off-by: Jerin Jacob
---
On Wed, Nov 04, 2015 at 11:24:18AM +0100, Adrien Mazarguil wrote:
> On Wed, Nov 04, 2015 at 02:19:36AM +0100, Thomas Monjalon wrote:
> > 2015-11-03 12:00, Bruce Richardson:
> > > Move the function ptr and port id checking macros to the header file, so
> > > that they can be used in the static inlin
On Tue, Nov 03, 2015 at 02:00:29PM -0800, Stephen Hemminger wrote:
> On Tue, 3 Nov 2015 12:00:59 +
> Bruce Richardson wrote:
>
> > * @return
> > - * The number of used descriptors in the specific queue.
> > + * The number of used descriptors in the specific queue, or:
> > + * (-EINVAL
The change in tsc value from rte_rdtsc() needs to be multiplied by the scale to
convert from clocks to get change in seconds.
For example from below:
elapse_us = (rte_rdtsc() - entry->tsc_first_packet) * flow_time_scale_us;
The bit rate requires the number of bytes passed in the time period then
2015-11-04 16:45, Cunming Liang:
> v4 change:
> - remove redundancy condition check on PF
>
> v3 changes:
> - rename MISC_VEC_ID and RX_VEC_START
> - add bsdapp dummy
> - split cleanup and fix patches
> - merge doc update along with code change
>
> v2 changes:
> - rework to depend on
> From: Jerin Jacob [mailto:jerin.jacob at caviumnetworks.com]
> Sent: Wednesday, November 04, 2015 1:29 PM
> To: dev at dpdk.org
> Cc: thomas.monjalon at 6wind.com; Hunt, David; viktorin at rehivetech.com;
> Ananyev, Konstantin; Richardson, Bruce; Xie, Huawei;
> changchun.ouyang at intel.com; zlu
Hi Martin
Thank you very much for reporting the issue!
Could you help to add more detailed steps and other information?
We will try to reproduce what have seen in our lab, and then debug.
Regards,
Helin
> -Original Message-
> From: Martin Weiser [mailto:martin.weiser at allegro-packets.c
2015-10-31 03:06, He, Shaopeng:
> > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> > The problem is that i40e interrupt patches are not merge-able.
> > They are not enough reviewed especially for igb/ixgbe and vfio changes.
> Thanks for your time, understood.
Now that i40e interrupt
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Helin Zhang
> Sent: Tuesday, November 3, 2015 3:40 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH v3 0/2] i40e: Enlarge the number of supported
> queues
>
> It enlarges the number of supported queues to h
When stop device, should disable interrupt of FDIR but not
enable it. This patch corrected it.
Signed-off-by: Jingjing Wu
---
drivers/net/i40e/i40e_ethdev.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
ind
2015-11-02 12:25, Alejandro.Lucero:
> +Before using the Netronome's DPDK PMD some NFP-6xxx configuration,
> +which is not related to DPDK, is required. The system requires
> +installation of **Netronome's BSP (Board Support Package)** which includes
> +Linux drivers, programs and libraries.
Do you
This patch updates the release notes with the features
added to ip_pipeline application.
Signed-off-by: Jasvinder Singh
---
doc/guides/rel_notes/release_2_2.rst | 49
1 file changed, 49 insertions(+)
diff --git a/doc/guides/rel_notes/release_2_2.rst
b/doc/g
2015-11-02 12:25, Alejandro.Lucero:
> +static struct eth_dev_ops nfp_net_eth_dev_ops = {
> + .dev_configure = nfp_net_configure,
> + .dev_start = nfp_net_start,
> + .dev_stop = nfp_net_stop,
> + .dev_close = nfp_net_close,
> +
Hi Jasvinder,
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jasvinder Singh
> Sent: Wednesday, November 04, 2015 3:05 PM
> To: dev at dpdk.org
> Subject: [dpdk-dev] [PATCH] doc: update release notes for ip_pipeline app
>
> This patch updates the release no
On Wed, Nov 04, 2015 at 02:10:50PM +, Bruce Richardson wrote:
> On Wed, Nov 04, 2015 at 11:24:18AM +0100, Adrien Mazarguil wrote:
> > On Wed, Nov 04, 2015 at 02:19:36AM +0100, Thomas Monjalon wrote:
> > > 2015-11-03 12:00, Bruce Richardson:
> > > > Move the function ptr and port id checking mac
>
> Hi Jasvinder,
>
> > -Original Message-
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jasvinder Singh
> > Sent: Wednesday, November 04, 2015 3:05 PM
> > To: dev at dpdk.org
> > Subject: [dpdk-dev] [PATCH] doc: update release notes for ip_pipeline
> > app
> >
> > This patc
Yes. It will build by now.
Once we add the PF then BSP will be needed. I guess this is the same for
MLX PMDs needing specific Mellanox libraries.
On Wed, Nov 4, 2015 at 3:03 PM, Thomas Monjalon
wrote:
> 2015-11-02 12:25, Alejandro.Lucero:
> > +Before using the Netronome's DPDK PMD some NFP-6xxx
This patch removes l3_l4_xsum_errors from rx errors.
The reason to remove it is that UDP packets have an optional checksum, and
when not calculated the checksum field should be set to zero. When the
checksum is not calculated (zero-ed out), the hardware still counts a valid
UDP packet as an l3_l4_
> From: Martin Weiser [mailto:martin.weiser at allegro-packets.com]
> Subject: Re: [dpdk-dev] ixgbe: ierrors counter spuriously increasing in DPDK
> 2.1
> The
> rx-error which showed up immediately after starting the interface is
> gone since this was probably caused by mac_remote_errors.
Impro
2015-11-04 15:50, Harry van Haaren:
> This patch removes l3_l4_xsum_errors from rx errors.
>
> The reason to remove it is that UDP packets have an optional checksum, and
> when not calculated the checksum field should be set to zero. When the
> checksum is not calculated (zero-ed out), the hardwar
This patch removes l3_l4_xsum_errors from rx errors.
The reason to remove it is that UDP packets have an optional checksum, and
when not calculated the checksum field should be set to zero. When the
checksum is not calculated (zero-ed out), the hardware still counts a valid
UDP packet as an l3_l4_
2015-11-04 23:01, Jingjing Wu:
> When stop device, should disable interrupt of FDIR but not
> enable it. This patch corrected it.
>
> Signed-off-by: Jingjing Wu
Applied, thanks
On 04.11.15 16:54, Van Haaren, Harry wrote:
>> From: Martin Weiser [mailto:martin.weiser at allegro-packets.com]
>> Subject: Re: [dpdk-dev] ixgbe: ierrors counter spuriously increasing in DPDK
>> 2.1
>> The
>> rx-error which showed up immediately after starting the interface is
>> gone since this
> > Following the same approach taken with dev_started field
> > in rte_eth_dev_data structure, this patch adds two new fields
> > in it, rx_queue_state and tx_queue_state arrays, which track
> > which queues have been started and which not.
> >
> > This is important to avoid trying to start/stop
On Tue, Nov 03, 2015 at 03:00:15PM +, Polehn, Mike A wrote:
> Adds Eth driver prefetch variable structure to CPU cache 0 while calling into
> tx or rx
> device driver operation.
>
> RFC 2544 test of NIC task test measurement points show improvement of lower
> latency and/or better packet th
2015-11-04 16:14, Harry van Haaren:
> This patch removes l3_l4_xsum_errors from rx errors.
>
> The reason to remove it is that UDP packets have an optional checksum, and
> when not calculated the checksum field should be set to zero. When the
> checksum is not calculated (zero-ed out), the hardwar
No one is proposing any close door planning session and commits for
ARM port of DPDK already staretd.
Pradeep
On 11/3/15 3:35 PM, Stephen Hemminger wrote:
> On Mon, 2 Nov 2015 15:16:16 -0800
> Pradeep Kathail wrote:
>
>> Tim and Dave,
>>
>> I agree that an architecture board membership should b
This patch updates the release notes with the features
added to ip_pipeline application.
v2:
*placed the release notes under New Features header
Signed-off-by: Jasvinder Singh
---
doc/guides/rel_notes/release_2_2.rst | 33 +
1 file changed, 33 insertions(+)
diff
The i40e driver was using a #define value for the max number of rxtx interrupts
supported. This value was defined only for linux, giving an error when compiling
on FreeBSD.
CC i40e_ethdev.o
/usr/home/bruce/dpdk.org/drivers/net/i40e/i40e_ethdev.c:3885:9: fatal error:
use of undeclared
id
2015-11-04 17:05, Bruce Richardson:
> The i40e driver was using a #define value for the max number of rxtx
> interrupts
> supported. This value was defined only for linux, giving an error when
> compiling
> on FreeBSD.
>
> CC i40e_ethdev.o
> /usr/home/bruce/dpdk.org/drivers/net/i40e/i40e_ethde
Begin forwarded message:
Date: Wed, 04 Nov 2015 08:08:51 -0800
From: scan-ad...@coverity.com
To: stephen at networkplumber.org
Subject: New Defects reported by Coverity Scan for DPDK Data Plane Development
Kit
Hi,
Please find the latest report on new defect(s) introduced to DPDK Data Plane
2015-11-04 08:14, Qiu, Michael:
> On 2015/11/4 14:14, Chao Zhu wrote:
> > The fm10k vector driver is specific for x86 platform which can't compile
> > on IBM POWER for lacking of tmmintrin.h header file. This patch turns
> > off fm10k driver compilation on IBM POWER to prevent compile issue.
> >
>
On Wed, 28 Oct 2015 10:56:33 +0100
Simon Kagstrom wrote:
> Otherwise mbufs will leak when the port is destroyed. The
> rte_sched_port_qbase() and rte_sched_port_qsize() functions are used
> in free now, so move them up.
>
> Signed-off-by: Simon Kagstrom
Overall it looks good, and fixes a long
On Wed, 4 Nov 2015 08:51:03 +0100
Simon K?gstr?m wrote:
> Ping?
>
> // Simon
>
> On 2015-10-20 15:20, Simon Kagstrom wrote:
> > The tx_rate unit is Mbps.
> >
> > Gleaned from the ixgbe implementation, the 82599 datasheet and the use
> > in test-pmd.
> >
> > Signed-off-by: Simon Kagstrom
> >
On Fri, 2015-10-30 at 18:10 -0400, Mussar, Gary wrote:
> I have been attempting to build testpmd with the bnx2x pmd enabled to try and
> talk through a bcm57810 10G NIC card. I have tried using SR-IOV VFs and I
> have tried using the PFs. Neither seem to be able to be properly initialized
> in t
On Wed, 4 Nov 2015 12:29:01 +0100
Simon K?gstr?m wrote:
> On 2015-11-04 11:35, Thomas Monjalon wrote:
> > 2015-08-20 08:51, Simon Kagstrom:
> >> -ifeq ($(shell test -f /proc/version_signature && lsb_release -si
> >> 2>/dev/null),Ubuntu)
> >> +ifeq ($(shell lsb_release -si 2>/dev/null),Ubuntu)
>
On Wed, 4 Nov 2015 11:24:18 +0100
Adrien Mazarguil wrote:
> On Wed, Nov 04, 2015 at 02:19:36AM +0100, Thomas Monjalon wrote:
> > 2015-11-03 12:00, Bruce Richardson:
> > > Move the function ptr and port id checking macros to the header file, so
> > > that they can be used in the static inline func
Is there a way where we can just define the fields that ought to be there in
the mbuf structure, but the position and size is implementation dependent ? The
application can provide "mbuf_impl.h" that contains mbuf_rte fields in the
order that seems appropriate to application.
--
- Thanks
char *
> > This patch set adds new flow director behavior "passthru" on Intel X(L)710
> > NIC.
> > When this mode is selected flow director will direct packet to LAN while the
> > queue is defined by other filters. This can be used to extract flexible
> > payload
> > to RX desriptor with the flow directo
2015-10-20 15:20, Simon Kagstrom:
> The tx_rate unit is Mbps.
>
> Gleaned from the ixgbe implementation, the 82599 datasheet and the use
> in test-pmd.
>
> Signed-off-by: Simon Kagstrom
Applied, thanks
2015-10-27 17:38, Tim Shearer:
> Calling the Ethernet driver's link_update function from rte_eth_dev_start can
> result in a race condition if the NIC raises the link interrupt at the same
> time. Depending on the interrupt handler implementation, the race can cause
> the it to think that it rec
> > > Someone may need to call rte_eal_init() with a fake argc/argv array
> > > in the middle of using getopt() to parse its own unrelated argc/argv
> > > parameters. So getopt lib shouldn't be reset by rte_eal_init().
> > >
> > > Now eal will always save optind, optarg and optopt (and optreset on
> >Signed-off-by: Chas Williams <3chas3 at gmail.com>
>
> Acked-by: Harish Patil
Applied, thanks
2015-10-14 09:41, Charles Williams:
> On Tue, 2015-10-13 at 14:49 +0200, Olivier MATZ wrote:
> > For PCI devices that have several interfaces (I think it's the case for
> > some Mellanox boards), maybe we should not store the interface name?
>
> I am not sure what you mean here. If a device has
> > In case of creating bond device without add any slaves and
> > quit from testpmd, application crashed since rx/tx queues
> > are NULL.
> >
> > add checking of this paramters before trying to free.
> >
> > Signed-off-by: Raslsn Darawsheh
> > Signed-off-by: Yaacov Hazan
>
> Acked-by: Declan Do
101 - 160 of 160 matches
Mail list logo