On 11/12/20 6:10 PM, Maxime Coquelin wrote:
> This series fixes several leaks in Vhost-user requests
> handling.
>
> Thanks to Xuan Ding from Intel for reporting these
> issues.
>
> Changes in v4:
> - Fix wrong return value in set_mem_table (Xueming)
>
> Changes in v3:
> - Remove the right cl
On 11/12/20 4:49 PM, Cheng Jiang wrote:
> Fix vhost-switch compiling issue when ioat dependency is missing.
> Change 'RTE_x86' check into 'RTE_RAW_IOAT' check in meson build file.
> Use 'RTE_RAW_IOAT' to control conditional compiling in source file.
> Clean some codes.
>
> Fixes: abec60e7115d (
10/11/2020 18:43, David Christensen:
> The SPAPR IOMMU requires that a DMA window size be defined before memory
> can be mapped for DMA. Current code dynamically modifies the DMA window
> size in response to every new memory allocation which is potentially
> dangerous because all existing mappings
On 11/6/20 4:24 AM, Cheng Jiang wrote:
> Add args length check before copying to fix the coverity issue.
>
> Fixes: 3a04ecb21420 ("examples/vhost: add async vhost args parsing")
> Coverity issue: 363741
>
> Signed-off-by: Cheng Jiang
> ---
> examples/vhost/main.c | 6 ++
> 1 file changed
On 11/11/20 10:08 AM, Cheng Jiang wrote:
> Add checking return value of string split function to fix the
> coverity issue.
>
> Fixes: 3a04ecb21420 ("examples/vhost: add async vhost args parsing")
> Coverity issue: 363739
>
> Signed-off-by: Cheng Jiang
> ---
> v2: checked args_nr explicitly
>
Fixes: eda1cc20c3bc ("common/sfc_efx/base: add 256-bit type")
Signed-off-by: Andrew Rybchenko
Reviewed-by: Andy Moreton
---
drivers/common/sfc_efx/base/efx_types.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/common/sfc_efx/base/efx_types.h
b/drivers/common/sfc_e
> > When doing Clang build with '-mcpu=native' on N1 platform, build failed
> > with:
> > ../lib/librte_eal/arm/include/rte_atomic_64.h:76:39:
> > error: instruction requires: lse
> > __ATOMIC128_CAS_OP(__cas_128_release, "caspl")
> >
> > This is because native detection for Neoverse N1 was ad
05/11/2020 17:11, Bruce Richardson:
> On Thu, Nov 05, 2020 at 09:33:15AM -0600, Dharmik Thakkar wrote:
> > With pkg-config support available within AArch64crypto library,
> > meson option 'armv8_crypto_dir' can be removed.
> > PKG_CONFIG_PATH environment variable should be set appropriately
> > to
13/11/2020 10:29, Thomas Monjalon:
> 05/11/2020 17:11, Bruce Richardson:
> > On Thu, Nov 05, 2020 at 09:33:15AM -0600, Dharmik Thakkar wrote:
> > > With pkg-config support available within AArch64crypto library,
> > > meson option 'armv8_crypto_dir' can be removed.
> > > PKG_CONFIG_PATH environment
13/11/2020 08:06, Dmitry Kozlyuk:
> Hi Luca,
>
> Thank you for taking the initiative. However, the patch fails to apply on my
> system, and that's the reason I've been delaying my own):
>
> $ git am \[kmods\ PATCH\ v2\]\ windows:\ normalize\ line-endings
> Applying: windows: normalize line-endin
From: Luca Boccassi
It forces files line ending conversions, making applying patches
impossible. Remove it. Line endings will be normalized in the next patch.
Signed-off-by: Luca Boccassi
---
v3: split the patch, and remove first the gitattributes, making it
possible to apply the second pat
From: Luca Boccassi
MSBuild XML files have to use CRLF line-endings, otherwise any change
results in the whole file being rewritten with CRLF line-endings by
Visual Studio. However, it's inconvenient to have such files checked-out
with CLRF in Unix environments.
Suggested-by: John Alexander
Sig
On Fri, 2020-11-13 at 10:43 +0100, Thomas Monjalon wrote:
> 13/11/2020 08:06, Dmitry Kozlyuk:
> > Hi Luca,
> >
> > Thank you for taking the initiative. However, the patch fails to apply on my
> > system, and that's the reason I've been delaying my own):
> >
> > $ git am \[kmods\ PATCH\ v2\]\ wind
On Thu, Nov 12, 2020 at 2:43 AM Chen, Mike Ximing
wrote:
>
>
>
> > -Original Message-
> > From: dev On Behalf Of Timothy McDaniel
> > Sent: Wednesday, November 11, 2020 3:27 PM
> > Cc: dev@dpdk.org; Carrillo, Erik G ; Eads, Gage
> > ; Van Haaren, Harry ;
> > jer...@marvell.com; tho...@mon
On Thu, Nov 12, 2020 at 11:13 PM wrote:
>
> From: Pavan Nikhilesh
>
> Fix SEGFAULT when nb_timer_adapters command line parameter is
> set to 0.
>
> Fixes: 98c6292105d4 ("app/eventdev: add options for event timer adapter")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Pavan Nikhilesh
Acked-by: Jerin
The following changes since commit 4630290af46ed44a58515067b6d2add9c044252a:
mbuf: move pool pointer in first half (2020-11-12 16:39:10 +0100)
are available in the Git repository at:
http://dpdk.org/git/next/dpdk-next-eventdev
for you to fetch changes up to c919c1039c07408da9b4ea0fd341818
On Fri, Nov 13, 2020 at 04:55:57AM +, Honnappa Nagarahalli wrote:
>
>
> >
> > To allow per-build override of some settings, without having to change
> > DPDK source-code files, i.e. rte_config.h, we can add an option to
> > allow the user to pass in a file containing their own defines for th
> -Original Message-
> From: dev On Behalf Of Ferruh Yigit
> Sent: Friday, November 6, 2020 2:10 AM
> To: Lu, Wenzhuo ; Xing, Beilei
> ; Iremonger, Bernard
> Cc: Yigit, Ferruh ; dev@dpdk.org; Yang, SteveX
> ; Thomas Monjalon ;
> Andrew Rybchenko ; Ananyev, Konstantin
> ; Olivier Matz ;
In pcap pmd, the timestamp mbuf dynamic field is mandatory. When the
pcap pmd is created in a secondary process (this is the case for pdump),
it cannot be registered because this is not allowed from a secondary
process.
To ensure that the field is properly registered, do it from probe()
instead of
The current machine='default' build name is not descriptive. The actual
default build is machine='native'. Add an alternative string which does
the same build and better describes what we're building:
machine='generic'. Leave machine='default' for backwards compatibility.
Signed-off-by: Juraj Link
Rename Arm build variables and values so that they better conform to Arm
specifications. Also rename generically sounding variable to names that
better capture what the variables hold.
Rename machine_args_generic to part_number_config_arm since the
variable contains more than just the generic mach
The current way of specifying Arm configuration options is insufficient
since we can't identify the SoC we're building for from the MIDR
information. For example, we can't distinguish between N1SDP, Graviton2
or Ampere Altra.
Add a way to specify the cpu count and numa node count for cross builds
Remove variables that were either not used, referenced just once or not
needed.
Signed-off-by: Juraj Linkeš
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
config/arm/meson.build | 28 +++-
1 file changed, 7 insertions(+), 21 deletions(-)
diff --git a/c
Set flags in one loop. Append flags to a list and use the list in the
loop.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnappa Nagarahalli
---
config/arm/meson.build | 37 +
1 file changed, 17 insertions(+), 20 deletions(-)
diff --git a/config/arm/meson.build
Change formatting so that it's more consistent and readable, add/modify
comments/stdout messages, move configuration options to more appropriate
places and make the order consistent according to these rules:
1. First list generic configuration options, then list options that may
be overwritten.
Letting the compiler decide is going to yield the best results for
native builds, so use native machine args usable for both GCC and Clang.
Signed-off-by: Juraj Linkeš
---
config/arm/meson.build | 54 ++
1 file changed, 28 insertions(+), 26 deletions(-)
d
Use generic configuration for the only build where it makes sense - the
generic build. For other builds, if we don't know either of implementer
ID or part number, the build is not supported.
Add part numbers to cross files where fallback to generic configuration
is assumed.
Signed-off-by: Juraj Li
Use dictionary lookup instead of checking for existing variables,
iterating over all elements in the list or checking lists for optional
configuration. Move variable contents into the dictionary for variables
that would be referenced only once.
Fallback to generic part number if the discovered part
From: Ruifeng Wang
When doing Clang build with '-mcpu=native' on N1 platform, build failed
with:
../lib/librte_eal/arm/include/rte_atomic_64.h:76:39:
error: instruction requires: lse
__ATOMIC128_CAS_OP(__cas_128_release, "caspl")
This is because native detection for Neoverse N1 was added
Add an option to automatically discover the host's numa and cpu counts
and use those values for a non cross-build.
Give users the option to override the per-arch default values or values
from cross files by specifying them on the command line with -Dmax_lcores
and -Dmax_numa_nodes.
Signed-off-by:
Add support for setting core count and numa nodes in cross files. The
values specified in cross files will override the default values.
Also add missing default values to Arm config.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnappa Nagarahalli
---
config/arm/arm64_armada_linux_gcc
A few options that disabled drivers in the old makefiles were improperly
ported to the meson build system. Fix this by adding a to the list of
disabled drivers, similarly how the command line option works. Remove
unneeded driver options ported from the old makefile system.
Add support for removing
Fix the implementer and part number of DPAA and ARMADA SoCs.
The current values of 16 cores and 1 NUMA node don't cover all SoCs from
the Arm implementer, e.g. Taishan 2280 has 64 cores and 4 NUMA nodes.
Increase these to 64 and 4 to widen the coverage.
Add configuration to SoC options where smalle
Some Arm SoCs are not NUMA systems. Add the capability to disable NUMA
for cross build and disable NUMA in Arm cross files.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnappa Nagarahalli
---
config/arm/arm64_armada_linux_gcc | 1 +
config/arm/arm64_bluefield_linux_gcc | 1
Add Arm SoC configuration to Arm meson.build and add a meson option to
enable those options for native builds. This is preferable to
specifying a cross file when doing aarch64 -> aarch64 builds, since the
cross file specifies the toolchain as well.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnapp
The current machine='default' build name is not descriptive. The actual
default build is machine='native'. Add an alternative string which does
the same build and better describes what we're building:
machine='generic'. Leave machine='default' for backwards compatibility.
Signed-off-by: Juraj Link
The current way of specifying Arm configuration options is insufficient
since we can't identify the SoC we're building for from the MIDR
information. For example, we can't distinguish between N1SDP, Graviton2
or Ampere Altra.
Add a way to specify the cpu count and numa node count for cross builds
From: Dharmik Thakkar
With pkg-config support available within AArch64crypto library,
meson option 'armv8_crypto_dir' can be removed.
PKG_CONFIG_PATH environment variable should be set appropriately
to use the crypto library.
Suggested-by: Thomas Monjalon
Signed-off-by: Dharmik Thakkar
Reviewe
Remove variables that were either not used, referenced just once or not
needed.
Signed-off-by: Juraj Linkeš
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
config/arm/meson.build | 28 +++-
1 file changed, 7 insertions(+), 21 deletions(-)
diff --git a/c
Rename Arm build variables and values so that they better conform to Arm
specifications. Also rename generically sounding variable to names that
better capture what the variables hold.
Rename machine_args_generic to part_number_config_arm since the
variable contains more than just the generic mach
Change formatting so that it's more consistent and readable, add/modify
comments/stdout messages, move configuration options to more appropriate
places and make the order consistent according to these rules:
1. First list generic configuration options, then list options that may
be overwritten.
Set flags in one loop. Append flags to a list and use the list in the
loop.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnappa Nagarahalli
---
config/arm/meson.build | 37 +
1 file changed, 17 insertions(+), 20 deletions(-)
diff --git a/config/arm/meson.build
Use dictionary lookup instead of checking for existing variables,
iterating over all elements in the list or checking lists for optional
configuration. Move variable contents into the dictionary for variables
that would be referenced only once.
Fallback to generic part number if the discovered part
Use generic configuration for the only build where it makes sense - the
generic build. For other builds, if we don't know either of implementer
ID or part number, the build is not supported.
Add part numbers to cross files where fallback to generic configuration
is assumed.
Signed-off-by: Juraj Li
Letting the compiler decide is going to yield the best results for
native builds, so use native machine args usable for both GCC and Clang.
Signed-off-by: Juraj Linkeš
---
config/arm/meson.build | 54 ++
1 file changed, 28 insertions(+), 26 deletions(-)
d
Add an option to automatically discover the host's numa and cpu counts
and use those values for a non cross-build.
Give users the option to override the per-arch default values or values
from cross files by specifying them on the command line with -Dmax_lcores
and -Dmax_numa_nodes.
Signed-off-by:
A few options that disabled drivers in the old makefiles were improperly
ported to the meson build system. Fix this by adding a to the list of
disabled drivers, similarly how the command line option works. Remove
unneeded driver options ported from the old makefile system.
Add support for removing
Add Arm SoC configuration to Arm meson.build and add a meson option to
enable those options for native builds. This is preferable to
specifying a cross file when doing aarch64 -> aarch64 builds, since the
cross file specifies the toolchain as well.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnapp
Add support for setting core count and numa nodes in cross files. The
values specified in cross files will override the default values.
Also add missing default values to Arm config.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnappa Nagarahalli
---
config/arm/arm64_armada_linux_gcc
Some Arm SoCs are not NUMA systems. Add the capability to disable NUMA
for cross build and disable NUMA in Arm cross files.
Signed-off-by: Juraj Linkeš
Reviewed-by: Honnappa Nagarahalli
---
config/arm/arm64_armada_linux_gcc | 1 +
config/arm/arm64_bluefield_linux_gcc | 1
Fix the implementer and part number of DPAA and ARMADA SoCs.
The current values of 16 cores and 1 NUMA node don't cover all SoCs from
the Arm implementer, e.g. Taishan 2280 has 64 cores and 4 NUMA nodes.
Increase these to 64 and 4 to widen the coverage.
Add configuration to SoC options where smalle
In 'rte_eth_dev_configure()', if 'DEV_RX_OFFLOAD_JUMBO_FRAME' is not set
the max frame size is limited to 'RTE_ETHER_MAX_LEN' (1518).
This is mistake because for the PMDs that has frame size bigger than
"RTE_ETHER_HDR_LEN + RTE_ETHER_CRC_LEN" (18 bytes), the MTU becomes
less than 1500, causing a va
On 11/9/2020 6:06 AM, Murphy Yang wrote:
Currently, driver does not support parse UDP outer checksum flag of
tunneled packets.
When execute 'csum set outer-udp hw 0' and 'csum parse-tunnel on 0'
commands to enable hardware UDP outer checksum. This patch supports
parse UDP outer checksum flag of
+ more people into this conversation. IMHO, restore to previous state is the
best choice by now.
> -Original Message-
> From: David Marchand
> Sent: Friday, November 13, 2020 3:27 PM
> To: Liu, Yong
> Cc: Maxime Coquelin ; Xia, Chenbo
> ; dev ; Richardson, Bruce
>
> Subject: Re: [dpdk
I tested external compilation for all possible examples (in my env).
The result is this series.
--
David Marchand
Changelog since v1:
- added patch 7 to fix performance-thread example,
- reworked last patch by introducing a new -Dexamples=buildall special
value and removed exceptions for examp
flow.c: In function ‘parse_flow_tokens’:
flow.c:153:23: error: taking address of packed member of ‘struct
rte_ipv4_hdr’ may result in an unaligned pointer value
[-Werror=address-of-packed-member]
153 | if (ipv4_addr_cpy(&rule->ipv4.spec.hdr.src_addr,
| ^~~~
When this example started using rte_cryptodev_sym_session_pool_create,
the part for pkg-config builds was not updated.
Fixes: 261bbff75e34 ("examples: use separate crypto session mempools")
Cc: sta...@dpdk.org
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
examples/fips_validatio
Two issues fixed here.
First add the experimental flag.
Then fix a link issue with the crypto scheduler driver:
/usr/bin/ld: /tmp/cchr7aHA.o: in function `main':
main.c:(.text.startup+0x1673): undefined reference to
`rte_cryptodev_scheduler_workers_get'
collect2: error: ld returned 1 exit status
rm -f build/kni build/kni-static build/kni-shared
test -d build && rmdir -p build || true
[...]
/usr/bin/ld: /tmp/cc72ssnK.o: undefined reference to symbol
'pthread_join@@GLIBC_2.2.5'
This example explicitly call pthread API and should be linked against
the pthread library.
Fixes: 724beb913b44 ("
This example missed the rework from commit 8549295db07b
("build/pkg-config: improve static linking flags").
Fixes: 08bd1a174461 ("examples/l3fwd-graph: add graph-based l3fwd skeleton")
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
examples/l3fwd-graph/Makefile | 2 +-
1 file cha
When introducing this example, the cleanup from commit 7e9562a107f1
("examples: fix make clean when using pkg-config") was missed.
Fixes: c5eebf85badc ("examples/ntb: add example for NTB")
Cc: sta...@dpdk.org
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
Acked-by: Xiaoyun Li
---
ex
main.c: In function ‘lthread_tx’:
main.c:2091:25: error: implicit declaration of function ‘sched_getcpu’;
did you mean ‘sched_getparam’? [-Werror=implicit-function-declaration]
2091 | tx_conf->conf.cpu_id = sched_getcpu();
| ^~~~
|
vhost_blk.c: In function ‘ctrlr_worker’:
vhost_blk.c:543:2: warning: implicit declaration of function ‘CPU_ZERO’
[-Wimplicit-function-declaration]
543 | CPU_ZERO(&cpuset);
| ^~~~
vhost_blk.c:544:2: warning: implicit declaration of function ‘CPU_SET’
[-Wimplicit-function-declaration]
This example is missing the experimental flag since it uses an
experimental API.
Fixes: cd1dadeb9b2a ("examples/rxtx_callbacks: support HW timestamp")
Cc: sta...@dpdk.org
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
examples/rxtx_callbacks/Makefile | 2 ++
1 file changed, 2 ins
Before make removal, those examples were built with experimental flag
for tracepoints to be compiled in but the pkg-config part of those
makefiles were missed.
Fixes: 78d44153de8f ("ethdev: add tracepoints")
Signed-off-by: David Marchand
Acked-by: Bruce Richardson
---
examples/cmdline/Makefile
Rather than just installing all examples, we can use the build checks to
filter out any examples that are missing dependencies or are otherwise
unbuildable on the current system.
Introduce a new "buildable" special value for the -Dexamples= meson
option, this way existing behavior on installing all
Hi,
>
> Instead of using getopt_long return value, strcmp was used to
> compare the input parameters with the struct option array. This
> patch get rid of all those strcmp by directly binding each longopt
> with an int enum. This is to improve readability and consistency in
> all examples.
>
>
Thomas,
On Fri, Nov 13, 2020 at 1:28 PM David Marchand
wrote:
> - a fix for the vhost is waiting in next-virtio, I assume it will get
> pulled before this series,
If next-virtio is not pulled yet, we can add a temp:
dpdk_examples_exclude += ['vhost']
--
David Marchand
On 11/12/2020 4:35 PM, Ciara Loftus wrote:
'uint64_t' is used to hold pointers in multiple locations in the
copy-mode code (used for kernels before 5.4). For a 32-bit build
this assumption is wrong and results in build errors. This commit
replaces such instances of 'uint64_t' with 'uintptr_t'.
W
On 11/13/20 1:24 PM, David Marchand wrote:
> vhost_blk.c: In function ‘ctrlr_worker’:
> vhost_blk.c:543:2: warning: implicit declaration of function ‘CPU_ZERO’
> [-Wimplicit-function-declaration]
> 543 | CPU_ZERO(&cpuset);
> | ^~~~
> vhost_blk.c:544:2: warning: implicit declaration
On 11/11/2020 9:31 AM, Ferruh Yigit wrote:
On 11/11/2020 7:23 AM, Michael Pfeiffer wrote:
Hi,
On Tue, 2020-11-10 at 15:59 +, Ferruh Yigit wrote:
On 11/9/2020 2:22 PM, Michael Pfeiffer wrote:
Unlike TCP, UDP checksums are optional and may be zero to indicate "not
set" [RFC 768] (except for
On Thu, Nov 5, 2020 at 4:27 PM Cristian Dumitrescu
wrote:
>
> Fix for the exact match lookup function.
>
> Fixes: d0a00966618b ("table: add exact match SWX table")
>
> Signed-off-by: Cristian Dumitrescu
> Signed-off-by: Churchill Khangar
Applied, thanks.
--
David Marchand
On Tue, Nov 10, 2020 at 9:03 PM Cristian Dumitrescu
wrote:
>
> Add the new SWX pipeline type to the Programmer's Guide.
Thanks for this high level description.
Could you work on documenting how to start and use the new pipeline example too?
>
> V2 changes: fixed 2 typos.
Patch changelog should
11/11/2020 14:18, Juraj Linkeš:
> Use generic configuration for the only build where it makes sense - the
> generic build. For other builds, if we don't know either of implementer
> ID or part number, the build is not supported.
> Add part numbers to cross files where fallback to generic configurat
On Fri, Nov 13, 2020 at 11:39:57AM +0100, Olivier Matz wrote:
> In pcap pmd, the timestamp mbuf dynamic field is mandatory. When the
> pcap pmd is created in a secondary process (this is the case for pdump),
> it cannot be registered because this is not allowed from a secondary
> process.
>
> To e
...
> diff --git a/lib/librte_gso/rte_gso.c b/lib/librte_gso/rte_gso.c
> index 896350e..fb76a21 100644
> --- a/lib/librte_gso/rte_gso.c
> +++ b/lib/librte_gso/rte_gso.c
> @@ -11,6 +11,7 @@
> #include "gso_common.h"
> #include "gso_tcp4.h"
> #include "gso_tunnel_tcp4.h"
> +#include "gso_tunnel_ud
On Thu, Nov 5, 2020 at 9:57 AM David Marchand wrote:
>
> On Mon, Nov 2, 2020 at 12:59 PM Ibtisam Tariq
> wrote:
> >
> > The short option written for interactive mode is --i in usage of
> > this qos_sched example. Actually, it is -i.
> >
> > Fixes: cfd5c971e5e ("examples/qos_sched: add stats")
Cc
On Thu, Nov 5, 2020 at 1:37 PM Gregory Etelson wrote:
>
> DPDK relies on pkg-config(1) to provide correct parameters for
> compiler and linker used in application build.
> Inaccurate build parameters, produced by pkg-config from DPDK .pc
> files could fail application build or cause unpredicted re
31/10/2020 07:56, Nick Connolly:
> Don't run symlink-drivers-solibs.sh as part of 'install' because
> Windows doesn't support shell scripts.
>
> Signed-off-by: Nick Connolly
> Tested-by: Ranjit Menon
> Acked-by: Ranjit Menon
Applied with title "build: fix install on Windows", thanks.
> -Original Message-
> From: Juraj Linkeš
> Sent: Friday, November 13, 2020 7:38 PM
> To: bruce.richard...@intel.com; Ruifeng Wang ;
> Honnappa Nagarahalli ; Phil Yang
> ; vcchu...@amazon.com; Dharmik Thakkar
> ; jerinjac...@gmail.com;
> hemant.agra...@nxp.com; Ajit Khaparde (ajit.khapa...
On Fri, Nov 13, 2020 at 01:24:23PM +0100, David Marchand wrote:
> Two issues fixed here.
>
> First add the experimental flag.
> Then fix a link issue with the crypto scheduler driver:
> /usr/bin/ld: /tmp/cchr7aHA.o: in function `main':
> main.c:(.text.startup+0x1673): undefined reference to
> `rte
Thanks for your reply Thomas, that's no problem.
I will expand the explanation for this patch when I push a v2,
v2 will hopefully also address the issues it caused with the icc build.
/Conor
> -Original Message-
> From: Thomas Monjalon
> Sent: Thursday 12 November 2020 18:47
> To: Walsh,
> -Original Message-
> From: Ruifeng Wang
> Sent: Friday, November 13, 2020 2:52 PM
> To: Juraj Linkeš ; bruce.richard...@intel.com;
> Honnappa Nagarahalli ; Phil Yang
> ; vcchu...@amazon.com; Dharmik Thakkar
> ; jerinjac...@gmail.com;
> hemant.agra...@nxp.com; Ajit Khaparde (ajit.khapa.
On Thu, Nov 5, 2020 at 10:39 AM Dharmappa, Savinay
wrote:
> Hi Thomas,
> >> I see that this patch is not applied on rc2. When can this will be applied
> >> ? will it be part of rc3 ?. Please let me know
Please, pay attention when editing replies, this mail of yours added
Cristian token again in
On 11/13/2020 1:13 PM, Olivier Matz wrote:
On Fri, Nov 13, 2020 at 11:39:57AM +0100, Olivier Matz wrote:
In pcap pmd, the timestamp mbuf dynamic field is mandatory. When the
pcap pmd is created in a secondary process (this is the case for pdump),
it cannot be registered because this is not allow
Unlike TCP, UDP checksums are optional and may be zero to indicate "not
set" [RFC 768] (except for IPv6, where this prohibited [RFC 8200]). Add
this special case to the checksum offload emulation in net/tap.
Signed-off-by: Michael Pfeiffer
---
v2:
* Added comment.
drivers/net/tap/rte_eth_tap.c
12/11/2020 22:35, Tal Shnaiderman:
> > Subject: Re: [dpdk-dev] [PATCH v2] windows: minor build fixes
> >
> > Missing / incorrect '--in-reply-to'. Please ignore - still learning the
> > process!
> >
> > On 31/10/2020 06:43, Nick Connolly wrote:
> > > Meson versions >= 0.54.0 include support for h
> -Original Message-
> From: Thomas Monjalon
> Sent: Friday, November 13, 2020 2:11 PM
> To: Juraj Linkeš
> Cc: bruce.richard...@intel.com; ruifeng.w...@arm.com;
> honnappa.nagaraha...@arm.com; phil.y...@arm.com;
> vcchu...@amazon.com; dharmik.thak...@arm.com; jerinjac...@gmail.com;
> he
Hi, Maxime
> -Original Message-
> From: Maxime Leroy
> Sent: Thursday, November 12, 2020 17:39
> To: Matan Azrad ; Shahaf Shuler ;
> Slava Ovsiienko ; Olivier Matz
>
> Cc: dev@dpdk.org; Didier Pallard
> Subject: [PATCH v2 2/2] net/mlx5: fix Rx descriptor status returned value
>
> From:
On Fri, Nov 13, 2020 at 01:24:30PM +0100, David Marchand wrote:
> Rather than just installing all examples, we can use the build checks to
> filter out any examples that are missing dependencies or are otherwise
> unbuildable on the current system.
> Introduce a new "buildable" special value for th
On Fri, Nov 13, 2020 at 3:06 PM Bruce Richardson
wrote:
>
> On Fri, Nov 13, 2020 at 01:24:30PM +0100, David Marchand wrote:
> > Rather than just installing all examples, we can use the build checks to
> > filter out any examples that are missing dependencies or are otherwise
> > unbuildable on the
On Fri, Nov 13, 2020 at 2:53 PM Bruce Richardson
wrote:
> > +NEED_CRYPTO_SCHEDULER = $(shell echo RTE_CRYPTO_SCHEDULER | $(CPP)
> > $(CFLAGS) -P - | tail -1)
> > +ifeq ($(NEED_CRYPTO_SCHEDULER), 1)
>
> Sorry for the last-minute comment, but I wonder for this check if we can do
> better by adding
> > Meson versions >= 0.54.0 include support for handling /implib
> > with msvc link. Specifying it explicitly causes failures when
> > linking against the dll. Tested using Link 14.27.29112.0 and
> > Clang 11.0.0.
> >
> > There were a number of changes to the way that import libraries
> > are han
13/11/2020 15:03, Juraj Linkeš:
> From: Thomas Monjalon
> > 11/11/2020 14:18, Juraj Linkeš:
> > > Use generic configuration for the only build where it makes sense -
> > > the generic build. For other builds, if we don't know either of
> > > implementer ID or part number, the build is not supporte
> -Original Message-
> From: Thomas Monjalon
> Sent: Friday, November 13, 2020 3:18 PM
> To: Juraj Linkeš
> Cc: bruce.richard...@intel.com; ruifeng.w...@arm.com;
> honnappa.nagaraha...@arm.com; phil.y...@arm.com;
> vcchu...@amazon.com; dharmik.thak...@arm.com; jerinjac...@gmail.com;
> h
On Fri, Nov 13, 2020 at 03:14:14PM +0100, David Marchand wrote:
> On Fri, Nov 13, 2020 at 3:06 PM Bruce Richardson
> wrote:
> >
> > On Fri, Nov 13, 2020 at 01:24:30PM +0100, David Marchand wrote:
> > > Rather than just installing all examples, we can use the build checks to
> > > filter out any ex
05/11/2020 14:58, Olivier Matz:
> On Thu, Nov 05, 2020 at 01:51:14PM +, David Hunt wrote:
> > Coverity flags that rte_mempool_ops_dequeue_bulk() is called without
> > checking the return value.
> >
> > This ignoring is intentional, so this patch gets the return code, then
> > uses RTE_SET_USED
The current way of specifying Arm configuration options is insufficient
since we can't identify the SoC we're building for from the MIDR
information. For example, we can't distinguish between N1SDP, Graviton2
or Ampere Altra.
Add a way to specify the cpu count and numa node count for cross builds
The current machine='default' build name is not descriptive. The actual
default build is machine='native'. Add an alternative string which does
the same build and better describes what we're building:
machine='generic'. Leave machine='default' for backwards compatibility.
Signed-off-by: Juraj Link
Remove variables that were either not used, referenced just once or not
needed.
Signed-off-by: Juraj Linkeš
Reviewed-by: Ruifeng Wang
Reviewed-by: Honnappa Nagarahalli
---
config/arm/meson.build | 28 +++-
1 file changed, 7 insertions(+), 21 deletions(-)
diff --git a/c
1 - 100 of 159 matches
Mail list logo
