On Thu, Feb 23, 2017 at 03:11:42PM +0800, Zhiyong Yang wrote:
> The minor change aims to remove the redundant computing and make
> it easier to understand the code.
>
> Cc: yuanhan@linux.intel.com
> Cc: maxime.coque...@redhat.com
>
> Signed-off-by: Zhiyong Yang
Applied to dpdk-next-virtio.
The minor change aims to remove the redundant computing and make
it easier to understand the code.
Cc: yuanhan@linux.intel.com
Cc: maxime.coque...@redhat.com
Signed-off-by: Zhiyong Yang
---
v2 changes:
remove the prefix "fix" and fixline according to yuanhan's advice.
drivers/net/virtio/v
On 16/02/2017 10:53, Thomas Monjalon wrote:
2017-02-03 10:33, Remy Horton:
[..]
I think there are three remaining questions:
- When the metrics computation are done? (in which thread?)
Actual calculation is not done by libmetrics itself - it only handles
distribution. Calculation is done pri
On Mon, Feb 13, 2017 at 03:28:13PM +0100, Maxime Coquelin wrote:
> This series adds support to new Virtio's MTU feature[1].
Seems you missed a link here?
> The MTU
> value is set via QEMU parameters.
>
> If the feature is negotiated (i.e supported by both host andcguest,
> and valid MTU value is
Hi, yuanhan:
> -Original Message-
> From: Yuanhan Liu [mailto:yuanhan@linux.intel.com]
> Sent: Thursday, February 23, 2017 2:21 PM
> To: Yang, Zhiyong
> Cc: dev@dpdk.org; maxime.coque...@redhat.com
> Subject: Re: [PATCH] net/virtio: fix remove the redundant computing
>
> On Thu, Feb
On Wed, Feb 22, 2017 at 02:25:29PM +0530, mitali wrote:
> Hi Yuanhan,
>
> We are using virt-manager to start the VMs. We are attaching the image which
> shows the vNIC setup for the VMs.
>
> We have written our own dpdk application to send and receive UDP packets. The
> command we are using is
>
On Thu, Feb 23, 2017 at 12:28:33PM +0800, Zhiyong Yang wrote:
> This is not a bug.
Then adding "fix" prefix and fixline here doesn't seem proper to me.
> The minor change aims to remove the redundant
> computing and make it easier to understand the code.
> Fixes:01ad44fd374f("net/virtio: split Rx
On Wed, Feb 22, 2017 at 10:36:36AM +0100, Maxime Coquelin wrote:
>
>
> On 02/22/2017 02:37 AM, Yuanhan Liu wrote:
> >On Tue, Feb 21, 2017 at 06:32:43PM +0100, Maxime Coquelin wrote:
> >>This patch aligns the Virtio-net header on a cache-line boundary to
> >>optimize cache utilization, as it puts
Hi,
Could this be reviewed for few more acks (though changes are only for
ppc64le) ?.
If needed. I can send release notes update separately for this support.
Regards,
Gowrishankar
On Friday 10 February 2017 11:48 AM, Gowrishankar wrote:
From: Gowrishankar Muthukrishnan
Below changes adds p
The patch change the prototype of callback function
(rte_intr_callback_fn) by removing the unnecessary parameter.
Also with this change we can keep content of item in intr_source
read only, the inappropriate modifciation(like get_max_intr) can
be avoid.
Signed-off-by: Qi Zhang
---
drivers/net/bn
Update the test code to algin with callback function change.
interface is changed.
Signed-off-by: Qi Zhang
---
app/test/test_interrupts.c | 23 ---
1 file changed, 12 insertions(+), 11 deletions(-)
diff --git a/app/test/test_interrupts.c b/app/test/test_interrupts.c
index 37
In an interrupt callback function, the paramter intr_handle which
is registered at rte_intr_callback_register call is passed back.
It is supposed to feed rte_intr_enable, but it is not gareenteed
to contain paramters as required, since it is a copy of
rte_ethdev->intr_handle, if some field need
From: Gowrishankar Muthukrishnan
This patch enables i40e driver in powerpc along with its altivec
intrinsic support.
Changes:
v4 - docs and config update.
v3 - minor corrections for coding style standard.
v2 - minor corrections for gcc strict aliasing and coding style standard.
Signed-off-by:
On Wed, Feb 22, 2017 at 11:01:43AM +0100, Matthias Gatto wrote:
> On Wed, Feb 22, 2017 at 2:59 AM, Yuanhan Liu
> wrote:
> >
> > On Tue, Feb 21, 2017 at 03:25:30PM +0100, Matthias Gatto wrote:
> > > fdset_add increment pfdset->num, but fdset_del doesn't decrement
> > > pfdset->num,
> > > so if we
This is not a bug. The minor change aims to remove the redundant
computing and make it easier to understand the code.
Fixes:01ad44fd374f("net/virtio: split Rx/Tx queue")
Cc: yuanhan@linux.intel.com
Cc: maxime.coque...@redhat.com
Signed-off-by: Zhiyong Yang
---
drivers/net/virtio/virtio_rxtx
On Wed, Feb 22, 2017 at 05:40:28PM +0200, Ami Sabo wrote:
> diff --git a/drivers/net/virtio/virtio_user_ethdev.c
> b/drivers/net/virtio/virtio_user_ethdev.c
> index 5291294..9b3c266 100644
> --- a/drivers/net/virtio/virtio_user_ethdev.c
> +++ b/drivers/net/virtio/virtio_user_ethdev.c
> @@ -418,25
So that, as a stable maintainer while picking commits to a stable release,
I could pay less attention to those have it and pay more attention to those
don't have it.
Signed-off-by: Yuanhan Liu
---
devtools/git-log-fixes.sh | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/de
To miss updating the variable prev_tsc in function l2fwd_main_loop()
The patch fixes it.
Fixes:e2366e74e029("examples: use buffered Tx")
CC: sergio.gonzalez.mon...@intel.com
Signed-off-by: Zhiyong Yang
---
examples/multi_process/l2fwd_fork/main.c | 2 ++
1 file changed, 2 insertions(+)
diff --
>
>The default tx_free_thresh is potentially larger than the allocated queue
>which will result in TX queue cleanup never happening. To fix this,
>lower the default free threshold and ensure that the free threshold is
>never greater than the maximum outstanding transmit buffers.
>
>Fixes: 827ed2a1
Signed-off-by: Ferruh Yigit
---
mk/rte.app.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mk/rte.app.mk b/mk/rte.app.mk
index 92f3635..d46a33e 100644
--- a/mk/rte.app.mk
+++ b/mk/rte.app.mk
@@ -272,7 +272,7 @@ clean: _postclean
.PHONY: doclean
doclean:
$(Q)rm -rf
Signed-off-by: Ferruh Yigit
---
mk/rte.lib.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mk/rte.lib.mk b/mk/rte.lib.mk
index 266cd01..987553d 100644
--- a/mk/rte.lib.mk
+++ b/mk/rte.lib.mk
@@ -172,7 +172,7 @@ clean: _postclean
.PHONY: doclean
doclean:
$(Q)rm -rf
Signed-off-by: Ferruh Yigit
---
mk/rte.lib.mk | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mk/rte.lib.mk b/mk/rte.lib.mk
index 33a5f5a..266cd01 100644
--- a/mk/rte.lib.mk
+++ b/mk/rte.lib.mk
@@ -172,7 +172,7 @@ clean: _postclean
.PHONY: doclean
doclean:
$(Q)rm -rf
On 2/22/2017 9:57 AM, Nelio Laranjeiro wrote:
> When flows cannot be re-applied due to configuration modifications, the
> start function should rollback the configuration done.
>
> Fixes: 2097d0d1e2cc ("net/mlx5: support basic flow items and actions")
> Cc: sta...@dpdk.org
> Signed-off-by: Nelio
On 2/21/2017 2:37 PM, Shahaf Shuler wrote:
> Interface name is queried, however never used.
>
> Signed-off-by: Shahaf Shuler
> Acked-by: Nelio Laranjeiro
Series applied to dpdk-next-net/master, thanks.
...
> The impact of having separate repositories is to reduce the work of a
> contributor touching many areas in a rework. This cost is transfered
> to the maintainer of the separate repository impacted by the change
> in the main repository. So it becomes this question:
> Do we prefer requiring s
On 2/22/2017 1:42 PM, Vasily Philipov wrote:
> Make some structs/defines visible from different source files by placing
> them into mlx4.h header.
>
> Signed-off-by: Vasily Philipov
> ---
> drivers/net/mlx4/mlx4.c | 183 ++
> drivers/net/mlx4/mlx4.h |
...
> > > 1/
> > > I suggest that each new library must be developed in a separate
> repository
> > > on dpdk.org. Then it can be asked to integrate it in the main
> > > project/repo.
> > > Such discussion must happen on the mailing list and the techboard will
> vote
> > > for the integration of t
On 2/21/2017 4:04 PM, Mcnamara, John wrote:
>
>
>> -Original Message-
>> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Pascal Mazon
>> Sent: Tuesday, February 21, 2017 1:15 PM
>> To: Wiles, Keith
>> Cc: dev@dpdk.org; Pascal Mazon
>> Subject: [dpdk-dev] [PATCH] doc: add features d
On 2/20/2017 6:11 PM, Qi Zhang wrote:
> This patch fix the compile error when RTE_LIBRTE_I40E_RX_ALLOW_BULK_ALLOC
> is disabled. fake_mbuf is also required to be initialized and assigned to
> additional sw_ring entries for vector PMD.
>
> Fixes: 9ed94e5bb04e ("i40e: add vector Rx")
> Cc: sta...@dp
On 2/20/2017 12:15 PM, Gowrishankar wrote:
> From: Gowrishankar Muthukrishnan
>
> Changes:
> v3 - minor corrections for coding style standard.
> v2 - minor corrections for gcc strict aliasing and coding style standard.
>
> This patch enables i40e driver in powerpc along with its altivec
> intrin
This commit adds the option to disable TSO offload on a specific txq.
Signed-off-by: Shahaf Shuler
---
lib/librte_ether/rte_ethdev.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h
index 3ab8568..b93be09 100644
--- a/lib/librte_et
This patchset adds support for hardware TSO on mlx5 PMD.
* Patches 1/4 and 2/4 adds TSO flags and capabilities to ethdev layer.
* Patch 3/4 adds support for the flag introduced in patch 2/4. This patch
also simplifies the testing of patch 4/4.
* Patch 4/4 implement support for hardware TSO and
Implement support for hardware TSO.
Signed-off-by: Shahaf Shuler
---
Performance impact on tx function due to tso logic insertion is estimated ~4
clocks.
---
doc/guides/nics/features/mlx5.ini | 1 +
drivers/net/mlx5/mlx5.c | 8 +++
drivers/net/mlx5/mlx5.h | 2 +
driver
Add the option to globaly disable hardware TSO offload
from the command line.
Signed-off-by: Shahaf Shuler
---
app/test-pmd/parameters.c | 9 -
doc/guides/testpmd_app_ug/run_app.rst | 4
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/app/test-pmd/paramete
Many Tx offloads are performed by hardware. As such, each offload
has its own limitations.
This commit adds the option to query Tx offload limitations in
order to use them properly and avoid bugs.
The limitations should be filled by the PMD upon query device info.
Signed-off-by: Shahaf Shuler
---
Secondary process doesn't properly attach to the rte_eth_device
initialized by the primary process.
Accessing device from secondary process (e.g. via rte_eth_rx_burst),
causes process to crash. because rte_eth_dev_data is not properly set.
The issue was flood by
'commit 7f95f78a8aea ("ethdev: cl
On Fri, Sep 16, 2016 at 03:15:01PM +0200, Nikita Kozlov wrote:
> On 08/25/2016 00:59, Nikita Kozlov wrote:
> > This serie of pathes focus on improving the speed of deleting rules in lpm6.
> >
> > It also contains some other improvement like having a dynamic number of
> > rules in lpm6 and increasin
Make some structs/defines visible from different source files by placing
them into mlx4.h header.
Signed-off-by: Vasily Philipov
---
drivers/net/mlx4/mlx4.c | 183 ++
drivers/net/mlx4/mlx4.h | 187 +++-
2 fil
Adding support for the next items: eth, vlan, ipv4, udp, tcp and for the
next actions: queue, drop
---
drivers/net/mlx4/Makefile|3 +-
drivers/net/mlx4/mlx4.c | 103 -
drivers/net/mlx4/mlx4.h |3 +
drivers/net/mlx4/mlx4_flow.c | 1053 +
CC: dev@dpdk.org. Missed that address when pulling from email archive.
> -Original Message-
> From: Bruce Richardson [mailto:bruce.richard...@intel.com]
> Sent: Wednesday, February 22, 2017 1:39 PM
> To: chunguang yang
> Subject: Re: [dpdk-dev] [PATCH] lpm: rte_lpm_iterate() - iterate thr
Downstreams might want to provide different DPDK releases at the same
time to support multiple consumers of DPDK linked against older and newer
sonames.
Also due to the interdependencies that DPDK libraries can have applications
might end up with an executable space in which multiple versions of a
On Tue, Feb 14, 2017 at 9:31 PM, Jan Blunck wrote:
> > 1. Downstreams to insert Major version into soname
> > Distributions could insert the DPDK major version (like 16.11) into the
> > soname and package names. A common example of this is libboost [5].
> > That would perfectly allow 16.07. to co
On Tue, Feb 21, 2017 at 07:12:58PM +0530, Shreyansh Jain wrote:
> Thanks for the suggestions about rte_* renaming in DPAA2 PMD.
> I create a draft patch for a single symbol change. (applies over v7
> of DPAA2 PMD)
>
> Can you tell me if this is the direction you were suggesting?
>
> I see two iss
Hi Bruce,
Yes, v2 patch is same as v1 + indentation changes according to checkpatch
warnings.
Regards,
Vlad
> -Original Message-
> From: Bruce Richardson [mailto:bruce.richard...@intel.com]
> Sent: Wednesday, February 22, 2017 2:03 PM
> To: Vladyslav Buslov
> Cc: dev@dpdk.org
> Subject:
On Tue, Feb 21, 2017 at 04:46:39PM +0200, Vladyslav Buslov wrote:
> This patch extend next_hop field from 8-bits to 21-bits in LPM library
> for IPv6.
>
> Added versioning symbols to functions and updated
> library and applications that have a dependency on LPM library.
>
> Signed-off-by: Vladysl
Secondary process doesn't properly attach to the rte_eth_device
initialized by the primary process.
Accessing device from secondary process (e.g. via rte_eth_rx_burst),
causes process to crash. because rte_eth_dev_data is not properly set.
The issue was flood by commit 7f95f78a8aea6161b824288bdeb
On 2/21/2017 6:15 PM, Jerin Jacob wrote:
On Fri, Feb 10, 2017 at 02:05:49PM +, Cristian Dumitrescu wrote:
The rte_flow feature breaks the current monolithic approach for ethdev and
introduces the new generic flow API to ethdev using a plugin-like approach.
Basically, the rte_flow API is sti
> -Original Message-
> From: Wiles, Keith
> Sent: Tuesday, February 21, 2017 5:39 PM
> To: dpdk.org; dev@dpdk.org
> Cc: iryz...@nfware.com; thomas.monja...@6wind.com; Mcnamara, John
>
> Subject: [PATCH v4] doc: use corelist instead of coremask
>
> The coremask option in DPDK is difficult
On Wed, Feb 22, 2017 at 09:37:42AM +0100, Nélio Laranjeiro wrote:
> On Tue, Feb 21, 2017 at 02:07:03PM +, Vasily Philipov wrote:
> > Adding support for the next items: eth, vlan, ipv4, udp, tcp and for the
> > next actions: queue, drop
> >
> > Signed-off-by: Vasily Philipov
> > ---
> > drive
On Wed, Feb 22, 2017 at 2:59 AM, Yuanhan Liu
wrote:
>
> On Tue, Feb 21, 2017 at 03:25:30PM +0100, Matthias Gatto wrote:
> > fdset_add increment pfdset->num, but fdset_del doesn't decrement
> > pfdset->num,
> > so if we call fdset_add then fdset_del in a loop witout calling
> > fdset_shrink, we ca
When flows cannot be re-applied due to configuration modifications, the
start function should rollback the configuration done.
Fixes: 2097d0d1e2cc ("net/mlx5: support basic flow items and actions")
Cc: sta...@dpdk.org
Signed-off-by: Nelio Laranjeiro
---
drivers/net/mlx5/mlx5_trigger.c | 21
On 02/22/2017 03:49 AM, Yang, Zhiyong wrote:
-Original Message-
From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Yuanhan Liu
Sent: Wednesday, February 22, 2017 9:38 AM
To: Maxime Coquelin
Cc: Liang, Cunming ; Tan, Jianfeng
; dev@dpdk.org
Subject: Re: [dpdk-dev] [RFC PATCH] net/vi
On 02/22/2017 02:37 AM, Yuanhan Liu wrote:
On Tue, Feb 21, 2017 at 06:32:43PM +0100, Maxime Coquelin wrote:
This patch aligns the Virtio-net header on a cache-line boundary to
optimize cache utilization, as it puts the Virtio-net header (which
is always accessed) on the same cache line as the
On Mon, Feb 20, 2017 at 04:00:12PM +0530, Priyanka wrote:
> Hi All,
>
> We have 2 VM running on qemu-kvm hypervisor. We have provided vNIC to the
> VMs using macvtap device and virtio as the device model in bridge mode. We
> are running DPDK applications (udp client and server) on the VMs. We obse
On Tue, Feb 21, 2017 at 02:07:02PM +, Vasily Philipov wrote:
> Make some structs/defines visible from different source files by placing
> them into mlx4.h header.
>
> Signed-off-by: Vasily Philipov
> ---
> drivers/net/mlx4/mlx4.c | 183 ++
> driver
On Tue, Feb 21, 2017 at 02:07:03PM +, Vasily Philipov wrote:
> Adding support for the next items: eth, vlan, ipv4, udp, tcp and for the
> next actions: queue, drop
>
> Signed-off-by: Vasily Philipov
> ---
> drivers/net/mlx4/Makefile|3 +-
> drivers/net/mlx4/mlx4.c | 60 ++-
>
(Modified the subject to: 'Re: [PATCHv7 03/47] common/dpaa2: adding
qbman driver' from 'Re: Hello Ferruh, Neil,')
Hello Ferruh,
On Tuesday 21 February 2017 08:09 PM, Ferruh Yigit wrote:
On 2/21/2017 1:42 PM, Shreyansh Jain wrote:
Thanks for the suggestions about rte_* renaming in DPAA2 PMD.
I
57 matches
Mail list logo
