From: Kalesh AP
Change verbosity of a message to DEBUG from ERROR.
This is just debug message.
Fixes: 3e92fd4e4ec0c7b ("net/bnxt: use dynamic log type")
Signed-off-by: Kalesh AP
Reviewed-by: Somnath Kotur
Signed-off-by: Somnath Kotur
---
drivers/net/bnxt/bnxt_hwrm.c | 2 +-
1 file changed,
From: Kalesh AP
Use rte_cpu_to_le_16/32 while parsing the hwrm command response.
Fixes: 11e5e19695c7bfb3 ("net/bnxt: support redirecting tunnel packets to VF")
Signed-off-by: Kalesh AP
Reviewed-by: Somnath Kotur
Signed-off-by: Somnath Kotur
---
drivers/net/bnxt/bnxt_hwrm.c | 4 ++--
1 file
From: Kalesh AP
1. Added couple of missing return value checks
2. Moved the log message to right place.
Fixes: 14255b35153721 ("net/bnxt: fix queue start/stop operations")
Signed-off-by: Kalesh AP
Signed-off-by: Somnath Kotur
---
drivers/net/bnxt/bnxt_ring.c | 4
drivers/net/bnxt/bnxt_
From: Kalesh AP
Modified to send the tunnel redirect commands to Chimp as Kong does not
support these commands.
Fixes: 11e5e19695c7bfb3 ("net/bnxt: support redirecting tunnel packets to VF")
Signed-off-by: Kalesh AP
Signed-off-by: Somnath Kotur
---
drivers/net/bnxt/bnxt_hwrm.c | 16 -
This set contains only bug fixes, that need to get into 19.08
Please apply.
Kalesh AP (4):
net/bnxt: fix rxq start
net/bnxt: fix to use chimp hwrm channel for few commands
net/bnxt: fix endianness issue
net/bnxt: reduce verbosity of a message
Santoshkumar Karanappa Rastapur (1):
net/bn
From: Santoshkumar Karanappa Rastapur
We were trying to fill in more rx extended stats than the size allocated
for stats causing segfault. Fixed this by adding an explicit check.
Rearranged the code to return statistic values in xstats_get as per the
names returned in xstats_get_names.
Fixes: f5
> -Original Message-
> From: Dekel Peled
> Sent: Tuesday, July 23, 2019 17:55
> To: Yongseok Koh ; Slava Ovsiienko
> ; Shahaf Shuler
> Cc: Ori Kam ; dev@dpdk.org
> Subject: [PATCH] net/mlx5: fix doorbell release on Rx queue release
>
> Function mlx5_rxq_release() calls mlx5_release_dbr()
> -Original Message-
> From: Dekel Peled
> Sent: Tuesday, July 23, 2019 18:35
> To: Yongseok Koh ; Slava Ovsiienko
> ; Shahaf Shuler
> Cc: Ori Kam ; dev@dpdk.org
> Subject: [PATCH] net/mlx5: fix Rx queue release of resources
>
> Function rxq_release_rq_resources() releases resources of R
Signed-off-by: Xiaolong Ye
---
drivers/net/e1000/base/e1000_80003es2lan.c | 35 ++
drivers/net/e1000/base/e1000_80003es2lan.h | 35 ++
drivers/net/e1000/base/e1000_82540.c | 35 ++
drivers/net/e1000/base/e1000_82541.c | 35 ++
Signed-off-by: Xiaolong Ye
---
drivers/net/i40e/i40e_rxtx_vec_altivec.c | 35 ++--
drivers/net/i40e/i40e_rxtx_vec_avx2.c| 33 ++
2 files changed, 5 insertions(+), 63 deletions(-)
diff --git a/drivers/net/i40e/i40e_rxtx_vec_altivec.c
b/drivers/net/i40e
Signed-off-by: Xiaolong Ye
---
drivers/net/fm10k/base/fm10k_api.c| 35 +++
drivers/net/fm10k/base/fm10k_api.h| 35 +++
drivers/net/fm10k/base/fm10k_common.c | 35 +++
drivers/net/fm10k/base/fm10k_common.h | 35 +++
Signed-off-by: Xiaolong Ye
---
drivers/net/iavf/base/iavf_adminq.c | 35 +++--
drivers/net/iavf/base/iavf_adminq.h | 35 +++--
drivers/net/iavf/base/iavf_adminq_cmd.h | 35 +++--
drivers/net/iavf/base/iavf_alloc.h | 35 +++--
This patchset replaces the remaining full license headers in a number
of intel driver files with SPDX tags to achieve DPDK-license compliance.
Xiaolong Ye (4):
net/iavf: replace license text with SPDX tag
net/fm10k: replace license text with SPDX tag
net/e1000: replace license text with SPDX
RCU library now includes stdbool.h header itself.
There is no need to include this header file in RCU user files.
Signed-off-by: Ruifeng Wang
Reviewed-by: Gavin Hu
---
app/test/test_rcu_qsbr.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/app/test/test_rcu_qsbr.c b/app/test/test_rcu_qsbr.c
>
> 23/07/2019 21:15, Honnappa Nagarahalli:
> > Hi Gavin,
> > I think this should have been V1 (I mean, no versioning, just 'PATCH'),
> since it is converted to patch. I think we should be able to resend it as V1
> and
> mark this V3 as 'superseded'.
> >
> > Hi Thomas,
> > Please let us k
On 07/24, Haiyue Wang wrote:
>The memzone's start virtual address pointer (addr) is of type void *,
>no need to add type cast.
>
>Signed-off-by: Haiyue Wang
>---
> drivers/net/ice/ice_rxtx.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/net/ice/ice_rxtx.c b/dri
The memzone's start virtual address pointer (addr) is of type void *,
no need to add type cast.
Signed-off-by: Haiyue Wang
---
drivers/net/ice/ice_rxtx.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ice/ice_rxtx.c b/drivers/net/ice/ice_rxtx.c
index 3353f23.
> -Original Message-
> From: Yang, Qiming
> Sent: Wednesday, July 24, 2019 5:26 AM
> To: Zhang, Qi Z ; Lu, Wenzhuo
> Cc: Stillwell Jr, Paul M ; dev@dpdk.org
> Subject: RE: [PATCH 0/6] net/ice/base: couple share code fix
>
>
> > -Original Message-
> > From: Zhang, Qi Z
> > Sent
> -Original Message-
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Xiao Zhang
> Sent: Monday, July 22, 2019 11:12 PM
> To: dev@dpdk.org
> Cc: Lu, Wenzhuo ; Zhao1, Wei
> ; Ye, Xiaolong ; Zhang, Xiao
> ; sta...@dpdk.org
> Subject: [dpdk-dev] [v8] net/e1000: fix i219 hang on reset/
Konstantin,
The recommended use of rte_eth_tx_prepare() had no effect, which after looking
at it, makes sense. We are using "large" mbufs to support Jumbo frames, so
nb-seg will always == 1. Additionally, we are not currently leveraging any HW
offload capabilities. As such, rte_eth_tx_prepare()
A new DPDK release candidate is ready for testing:
https://git.dpdk.org/dpdk/tag/?id=v19.08-rc2
277 patches were integrated.
The release notes so far:
http://doc.dpdk.org/guides/rel_notes/release_19_08.html
It should be completed with a list of tested hardware.
Highlights of 19.08
On Mon, Jul 22, 2019 at 2:34 PM Ferruh Yigit wrote:
> I dropped this patch from next-net since discussion is going on, I did my best
> to resolve the conflicts but please confirm the final result in next-net.
>
I did some basic testing of the current head of dpdk-next-net and diffed
against the
23/07/2019 21:15, Honnappa Nagarahalli:
> Hi Gavin,
> I think this should have been V1 (I mean, no versioning, just 'PATCH'),
> since it is converted to patch. I think we should be able to resend it as V1
> and mark this V3 as 'superseded'.
>
> Hi Thomas,
> Please let us know if it i
> -Original Message-
> From: Zhang, Qi Z
> Sent: Monday, July 22, 2019 8:51 PM
> To: Yang, Qiming ; Lu, Wenzhuo
>
> Cc: Stillwell Jr, Paul M ; dev@dpdk.org; Zhang,
> Qi Z
> Subject: [PATCH 0/6] net/ice/base: couple share code fix
>
> Qi Zhang (6):
> net/ice/base: fix inner TCP and UD
On 7/23/19 3:28 PM, Vamsi Krishna Attunuru wrote:
-Original Message-
From: Andrew Rybchenko
Sent: Tuesday, July 23, 2019 4:38 PM
To: Vamsi Krishna Attunuru ; dev@dpdk.org
Cc: tho...@monjalon.net; Jerin Jacob Kollanukkaran ;
olivier.m...@6wind.com; ferruh.yi...@intel.com;
anatoly.bura...@
Hi Gavin,
I think this should have been V1 (I mean, no versioning, just 'PATCH'),
since it is converted to patch. I think we should be able to resend it as V1
and mark this V3 as 'superseded'.
Hi Thomas,
Please let us know if it is worth/helps fixing the version.
Thanks,
Honnapp
>
> On Tue, 23 Jul 2019 23:43:46 +0800
> Gavin Hu wrote:
>
> > Add the RTE_USE_WFE configuration entry for aarch64, disabled by default.
> > It can be enabled selectively based on the performance benchmarking.
> >
> > Signed-off-by: Gavin Hu
> > Reviewed-by: Ruifeng Wang
> > Reviewed-by: Steve
On Tue, Jul 23, 2019 at 6:27 PM Bruce Richardson
wrote:
>
> Some builds with clang report an error because '<>' rather than '""' were
> used for including the ioat spec header file.
>
> Target: x86_64-native-bsdapp-clang
> 22:10: error: 'rte_ioat_spec.h' file not found with include; use
> "q
23/07/2019 18:26, Bruce Richardson:
> Some builds with clang report an error because '<>' rather than '""' were
> used for including the ioat spec header file.
>
> Target: x86_64-native-bsdapp-clang
> 22:10: error: 'rte_ioat_spec.h' file not found with include; use
> "quotes" instead
> #in
23/07/2019 12:30, Sirvys, Andrius:
> When using IOVA as VA mode, there is no need to map segments page by page.
> This normally isn't a problem, but it becomes one when attempting to use DPDK
> in no-huge mode, where VFIO subsystem simply runs out of space to store
> mappings.
>
> Fix this for
On Tue, 23 Jul 2019 13:30:33 +0100
Bruce Richardson wrote:
> On Mon, Jul 22, 2019 at 11:53:26AM -0700, Stephen Hemminger wrote:
> > On Mon, 22 Jul 2019 19:31:08 +0200
> > Thomas Monjalon wrote:
> >
> > > 22/07/2019 19:13, Stephen Hemminger:
> > > > Thomas Monjalon wrote:
> > > > > Are
23/07/2019 15:39, Thomas Monjalon:
> 23/07/2019 15:34, Andrew Rybchenko:
> > On 7/23/19 4:14 PM, Thomas Monjalon wrote:
> > > 23/07/2019 14:11, Andrew Rybchenko:
> > >> rte_eth_dev_info_get() returns void and caller does know if the function
> > >> does its job or not. Changing of the return value
On Tue, 23 Jul 2019 14:19:53 +0100
Anatoly Burakov wrote:
> diff --git a/lib/librte_eal/linux/eal/eal.c b/lib/librte_eal/linux/eal/eal.c
> index 34db78753..54feb24a3 100644
> --- a/lib/librte_eal/linux/eal/eal.c
> +++ b/lib/librte_eal/linux/eal/eal.c
> @@ -83,6 +83,13 @@ static struct flock wr_lo
23/07/2019 20:29, Stephen Hemminger:
> On Tue, 23 Jul 2019 09:59:04 +0200
> Thomas Monjalon wrote:
>
> > 22/07/2019 20:34, Stephen Hemminger:
> > > On Mon, 22 Jul 2019 19:31:08 +0200
> > > Thomas Monjalon wrote:
> > >
> > > > 22/07/2019 19:13, Stephen Hemminger:
> > > > > Thomas Monjalon w
On Tue, 23 Jul 2019 09:59:04 +0200
Thomas Monjalon wrote:
> 22/07/2019 20:34, Stephen Hemminger:
> > On Mon, 22 Jul 2019 19:31:08 +0200
> > Thomas Monjalon wrote:
> >
> > > 22/07/2019 19:13, Stephen Hemminger:
> > > > Thomas Monjalon wrote:
> > > > > Are the constructors run on dlopen
On Tue, 23 Jul 2019 13:30:33 +0100
Bruce Richardson wrote:
> On Mon, Jul 22, 2019 at 11:53:26AM -0700, Stephen Hemminger wrote:
> > On Mon, 22 Jul 2019 19:31:08 +0200
> > Thomas Monjalon wrote:
> >
> > > 22/07/2019 19:13, Stephen Hemminger:
> > > > Thomas Monjalon wrote:
> > > > > Are
On Tue, 23 Jul 2019 23:43:46 +0800
Gavin Hu wrote:
> Add the RTE_USE_WFE configuration entry for aarch64, disabled by default.
> It can be enabled selectively based on the performance benchmarking.
>
> Signed-off-by: Gavin Hu
> Reviewed-by: Ruifeng Wang
> Reviewed-by: Steve Capper
> Reviewed-
Konstantin,
Thank you for the prompt reply on this posting. In looking at the single
use-case in test-pmd's csumonly.c, it would seem prepare + retry_enabled may
have some shortcomings as currently coded when nb_prep < nb_rx. Has anyone
looked at this? I happened to notice this when looking for
Some builds with clang report an error because '<>' rather than '""' were
used for including the ioat spec header file.
Target: x86_64-native-bsdapp-clang
22:10: error: 'rte_ioat_spec.h' file not found with include; use
"quotes" instead
#include
^
"rt
>
> Hello,
>
> We are chasing an interesting NIC transmit issue where after some period of
> time with normal operation the NIC enters a state where it
> refuses to transmit frames from our DPDK application via rte_eth_tx_burst().
> All indications are the port is up and otherwise operation
On 23-Jul-19 3:36 PM, Jerin Jacob Kollanukkaran wrote:
-Original Message-
From: Burakov, Anatoly
Sent: Tuesday, July 23, 2019 8:00 PM
To: Thomas Monjalon
Cc: Jerin Jacob Kollanukkaran ; Stojaczyk, Dariusz
; David Marchand
; dev@dpdk.org
Subject: [EXT] Re: [dpdk-dev] [PATCH v4 0/4] Fixes
Hi Stephen,
> -Original Message-
> From: Pavan Nikhilesh Bhagavatula
> Sent: Saturday, July 20, 2019 3:03 PM
> To: Gavin Hu (Arm Technology China) ; dev@dpdk.org
> Cc: nd
> Subject: RE: [dpdk-dev] [RFC v2 5/5] config: add WFE config entry for
> aarch64
>
>
>
> >-Original Message---
DPDK has multiple use cases where the core repeatedly polls a location in
memory. This polling results in many cache and memory transactions.
Arm architecture provides WFE (Wait For Event) instruction, which allows
the cpu core to enter a low power state until woken up by the update to the
memory
Add the RTE_USE_WFE configuration entry for aarch64, disabled by default.
It can be enabled selectively based on the performance benchmarking.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Steve Capper
Reviewed-by: Honnappa Nagarahalli
Acked-by: Pavan Nikhilesh
---
config/ar
While using ticket lock, cores repeatedly poll the lock variable.
This is replaced by rte_wait_until_equal API.
Running ticketlock_autotest on ThunderX2, with different numbers of cores
and depths of rings, 3%~8% performance gains were measured.
Signed-off-by: Gavin Hu
Reviewed-by: Honnappa Naga
Instead of polling for tail to be updated, use wfe instruction.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Steve Capper
Reviewed-by: Ola Liljedahl
Reviewed-by: Honnappa Nagarahalli
---
lib/librte_ring/rte_ring_c11_mem.h | 4 ++--
lib/librte_ring/rte_ring_generic.h | 3 +--
In acquiring a spinlock, cores repeatedly poll the lock variable.
This is replaced by rte_wait_until_equal API.
5~10% performance gain was measured by running spinlock_autotest on
14 isolated cores of ThunderX2.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Phil Yang
Reviewed-
The rte_wait_until_equalxx APIs abstract the functionality of 'polling
for a memory location to become equal to a given value'.
Signed-off-by: Gavin Hu
Reviewed-by: Ruifeng Wang
Reviewed-by: Steve Capper
Reviewed-by: Ola Liljedahl
Reviewed-by: Honnappa Nagarahalli
Acked-by: Pavan Nikhilesh
-
Function rxq_release_rq_resources() releases resources of RQ object
created by DevX API.
This patch updates this function to properly clear the released
resources, to avoid repeated release of the same resource.
Fixes: 69abf7b1db4a ("net/mlx5: create advanced RxQ using new API")
Signed-off-by: D
On 07/23, Wei Zhao wrote:
>There is a bug in function ixgbevf_dev_info_get(), it do not
>contain the reta table size get function for vf port, and this
>will cause error when update reta for x550 vf port. And also
>when function ixgbe_reta_size_get() is called, it should return
>other ixgbe NIC vf
Hello,
We are chasing an interesting NIC transmit issue where after some period of
time with normal operation the NIC enters a state where it refuses to transmit
frames from our DPDK application via rte_eth_tx_burst(). All indications are
the port is up and otherwise operational and is still re
Function mlx5_rxq_release() calls mlx5_release_dbr() to release the
doorbell allocated for this Rx queue.
This call is relevant only for Rx queue objects created using
DevX API.
This patch adds the required check, to call mlx5_release_dbr()
only when relevant.
It also updates mlx5_release_dbr() to
On 7/23/19 5:38 PM, Stephen Hemminger wrote:
On Tue, 23 Jul 2019 15:07:51 +0100
Andrew Rybchenko wrote:
void return value is bad for get API (like rte_eth_dev_info-get())
since caller does not know if the function does its job or not and
output value is filled in.
void return value is bad for
On Tue, 23 Jul 2019 15:07:51 +0100
Andrew Rybchenko wrote:
> void return value is bad for get API (like rte_eth_dev_info-get())
> since caller does not know if the function does its job or not and
> output value is filled in.
>
> void return value is bad for state changing API (like
> rte_eth_pr
> -Original Message-
> From: Burakov, Anatoly
> Sent: Tuesday, July 23, 2019 8:00 PM
> To: Thomas Monjalon
> Cc: Jerin Jacob Kollanukkaran ; Stojaczyk, Dariusz
> ; David Marchand
> ; dev@dpdk.org
> Subject: [EXT] Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection
> On 23-Jul-19 2:
On 23-Jul-19 2:56 PM, Burakov, Anatoly wrote:
On 23-Jul-19 11:25 AM, Thomas Monjalon wrote:
23/07/2019 11:57, Burakov, Anatoly:
A machine without an IOMMU shouldn't have picked IOVA as VA in the first
place. Perhaps this is something we could fix? I'm not sure how to
detected that condition tho
> -Original Message-
> From: Burakov, Anatoly
> Sent: Tuesday, July 23, 2019 7:27 PM
> To: Thomas Monjalon
> Cc: Jerin Jacob Kollanukkaran ; Stojaczyk, Dariusz
> ; David Marchand
> ; dev@dpdk.org
> Subject: [EXT] Re: [dpdk-dev] [PATCH v4 0/4] Fixes on IOVA mode selection
>
>
On 7/23/2019 3:07 PM, Andrew Rybchenko wrote:
> void return value is bad for get API (like rte_eth_dev_info-get())
> since caller does not know if the function does its job or not and
> output value is filled in.
>
> void return value is bad for state changing API (like
> rte_eth_promiscuous_enabl
void return value is bad for get API (like rte_eth_dev_info-get())
since caller does not know if the function does its job or not and
output value is filled in.
void return value is bad for state changing API (like
rte_eth_promiscuous_enable()) since caller should use get API
to understand if stat
On 23-Jul-19 11:25 AM, Thomas Monjalon wrote:
23/07/2019 11:57, Burakov, Anatoly:
A machine without an IOMMU shouldn't have picked IOVA as VA in the first
place. Perhaps this is something we could fix? I'm not sure how to
detected that condition though, i don't think there's a mechanism to
know
23/07/2019 15:34, Andrew Rybchenko:
> On 7/23/19 4:14 PM, Thomas Monjalon wrote:
> > 23/07/2019 14:11, Andrew Rybchenko:
> >> rte_eth_dev_info_get() returns void and caller does know if the function
> >> does its job or not. Changing of the return value to int would be
> >> API/ABI breakage which r
On 7/23/19 4:14 PM, Thomas Monjalon wrote:
23/07/2019 14:11, Andrew Rybchenko:
rte_eth_dev_info_get() returns void and caller does know if the function
does its job or not. Changing of the return value to int would be
API/ABI breakage which requires deprecation process and cannot be
backported t
Thanks David.
> -Original Message-
> From: David Marchand [mailto:david.march...@redhat.com]
> Sent: Tuesday, July 23, 2019 6:00 AM
> To: dev@dpdk.org
> Cc: Eads, Gage ; Van Haaren, Harry
>
> Subject: [PATCH] service: remove experimental banner in API description
>
> This was missed when
Currently, primary process holds an exclusive lock on the config
file, thereby preventing other primaries from spinning up. However,
when the primary dies, the lock is no longer being held, even though
there might be other secondary processes still running.
The fix is two-fold. First of all, downg
On Tue, Jul 23, 2019 at 09:05:50AM +, Krzysztof Kanas wrote:
> On 19-07-22 14:44, Bruce Richardson wrote:
> > External Email
> >
> > --
> > On Mon, Jul 22, 2019 at 02:39:59PM +0200, kka...@marvell.com wrote:
> > > From: Krzysz
23/07/2019 14:11, Andrew Rybchenko:
> rte_eth_dev_info_get() returns void and caller does know if the function
> does its job or not. Changing of the return value to int would be
> API/ABI breakage which requires deprecation process and cannot be
> backported to stable branches. For now, make sure
23/07/2019 14:16, Andrew Rybchenko:
> On 7/23/19 1:40 PM, Ferruh Yigit wrote:
> > On 7/23/2019 10:13 AM, Andrew Rybchenko wrote:
> >> rte_eth_dev_info_get() returns void and caller does know if the function
> >> does its job or not. Changing of the return value to int would be
> >> API/ABI breakage
> -Original Message-
> From: Ferruh Yigit
> Sent: Wednesday, July 10, 2019 5:07 PM
> To: Jakub Grajciar -X (jgrajcia - PANTHEON TECHNOLOGIES at Cisco)
> ; dev@dpdk.org
> Cc: anatoly.bura...@intel.com
> Subject: Re: [dpdk-dev] [PATCH v4] net/memif: zero-copy slave
>
> On 7/9/2019 9:22 AM
On Mon, Jul 22, 2019 at 11:53:26AM -0700, Stephen Hemminger wrote:
> On Mon, 22 Jul 2019 19:31:08 +0200
> Thomas Monjalon wrote:
>
> > 22/07/2019 19:13, Stephen Hemminger:
> > > Thomas Monjalon wrote:
> > > > Are the constructors run on dlopen of the bus driver?
> > >
> > > Yes, constructor
> -Original Message-
> From: Andrew Rybchenko
> Sent: Tuesday, July 23, 2019 4:38 PM
> To: Vamsi Krishna Attunuru ; dev@dpdk.org
> Cc: tho...@monjalon.net; Jerin Jacob Kollanukkaran ;
> olivier.m...@6wind.com; ferruh.yi...@intel.com;
> anatoly.bura...@intel.com; Kiran Kumar Kokkilagadda
This patchset contains unit testcases added to increase the
functional and decision coverage for the library functions
defined in rte_mbuf.h and rte_mbuf.c
1/2: unit test cases added for rte_mbuf.h
2/2: unit test cases added for rte_mbuf.c
Patch 2/2 depends on 1/2
Signed-off-by: Lavanya Govinda
On 7/23/19 1:40 PM, Ferruh Yigit wrote:
On 7/23/2019 10:13 AM, Andrew Rybchenko wrote:
rte_eth_dev_info_get() returns void and caller does know if the function
does its job or not. Changing of the return value to int would be
API/ABI breakage which requires deprecation process and cannot be
back
From: Pallantla Poornima
Added UT for the below four functions in test_mbuf.c
rte_get_rx_ol_flag_list
rte_get_tx_ol_flag_list
rte_get_rx_ol_flag_name
rte_get_tx_ol_flag_name
Signed-off-by: Pallantla Poornima
---
app/test/test_mbuf.c | 260 +++
1 file cha
Added new unit test cases to cover the below
functions defined in rte_mbuf.h
rte_validate_tx_offload,
rte_pktmbuf_alloc_bulk,
rte_pktmbuf_read,
rte_pktmbuf_ext_shinfo_init_helper,
rte_pktmbuf_attach_extbuf,
rte_mbuf_ext_refcnt_read,
rte_mbuf_ext_refcnt_update,
rte_mbuf_ext_refcnt_set,
rte_pktmbuf_d
rte_eth_dev_info_get() returns void and caller does know if the function
does its job or not. Changing of the return value to int would be
API/ABI breakage which requires deprecation process and cannot be
backported to stable branches. For now, make sure that device info is
initialized even in the
Hi Ferruh
From: Ferruh Yigit
> On 7/22/2019 3:51 PM, Matan Azrad wrote:
> > From: Dekel Peled
> >
> > A variable of type struct ibv_cq_ex is declared in 2 unions, but isn't
> > used.
> > This patch removes the 2 redundant declarations.
> This is not a fix, but I think it still make sense to req
Hi,
>-Original Message-
>From: Bruce Richardson [mailto:bruce.richard...@intel.com]
>Sent: Monday, July 22, 2019 7:23 PM
>To: Thomas Monjalon
>Cc: Parthasarathy, JananeeX M ;
>dev@dpdk.org; Michael Santana Francisco ; Aaron
>Conole ; Hari Kumar Vemula
>; Pattan, Reshma
>; Mcnamara, John ;
On 7/19/19 7:56 PM, Stephen Hemminger wrote:
Tell users about upcoming changes to rte_ether_addr and
rte_ether_header.
Signed-off-by: Stephen Hemminger
Acked-by: Bruce Richardson
Acked-by: Andrew Rybchenko
On Tue, Jul 16, 2019 at 5:29 PM Ferruh Yigit wrote:
>
> On 7/16/2019 4:00 PM, Andrew Rybchenko wrote:
> > On 7/16/19 5:28 PM, Bruce Richardson wrote:
> >> On Tue, Jul 16, 2019 at 03:23:03PM +0100, Ferruh Yigit wrote:
> >>> On 7/16/2019 1:23 PM, Andrew Rybchenko wrote:
> On 7/15/19 5:53 PM, Fe
On 7/23/19 8:38 AM, vattun...@marvell.com wrote:
From: Vamsi Attunuru
Patch adds a routine to populate mempool from page aligned and
page sized chunks of memory to ensures memory objs do not fall
across the page boundaries. It's useful for applications that
require physically contiguous mbuf me
CID 340857: Null pointer dereferences (NULL_RETURNS)
CID 340856: (CONSTANT_EXPRESSION_RESULT)
Fixes: 3be12ea52ad8 ("test/compress: improve debug trace setup")
Signed-off-by: Adam Dybkowski
---
app/test/test_compressdev.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --
> -Original Message-
> From: Andrew Rybchenko
> Sent: Tuesday, July 23, 2019 4:20 PM
> To: Vamsi Krishna Attunuru ; dev@dpdk.org
> Cc: tho...@monjalon.net; Jerin Jacob Kollanukkaran ;
> olivier.m...@6wind.com; ferruh.yi...@intel.com; anatoly.bura...@intel.com;
> Kiran Kumar Kokkilagadda
This was missed when promoting this API to stable.
Fixes: 7a0ac7cdb454 ("service: promote experimental functions to stable")
Signed-off-by: David Marchand
---
lib/librte_eal/common/include/rte_service.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/lib/librte_eal/common/include/rte_serv
On 7/23/19 8:38 AM, vattun...@marvell.com wrote:
From: Vamsi Attunuru
Current KNI implementation only operates in IOVA=PA mode, patch adds
required functionality in KNI lib to support IOVA=VA mode.
KNI kernel module requires device info to get iommu domain related
information for IOVA addr rel
Yes, the platform can be detected at runtime
from the PCI device ID. This would be a better
approach than adding a configuration parameter,
I think. I'll take a look.
Thanks,
Lance
On Tue, Jul 23, 2019 at 4:04 AM Thomas Monjalon wrote:
>
> 22/07/2019 20:34, Ferruh Yigit:
> > On 7/22/2019 6:5
On 7/22/2019 3:51 PM, Matan Azrad wrote:
> From: Dekel Peled
>
> A variable of type struct ibv_cq_ex is declared in 2 unions, but
> isn't used.
> This patch removes the 2 redundant declarations.
This is not a fix, but I think it still make sense to request backporting these
kind of refactoring to
On 7/23/19 8:38 AM, vattun...@marvell.com wrote:
From: Vamsi Attunuru
When KNI operates in IOVA = VA mode, it requires mbuf memory
to be physically contiguous to ensure KNI kernel module could
translate IOVA addresses properly. Patch adds a KNI specific
mempool create routine to populate the KN
On 7/23/2019 7:39 AM, Raslan Darawsheh wrote:
> Tested-by: Raslan Darawsheh
> Seems OK for me,
>
> Kindest regards,
> Raslan Darawsheh
>
>> -Original Message-
>> From: Stephen Hemminger
>> Sent: Monday, July 22, 2019 8:20 PM
>> To: Ferruh Yigit
>> Cc: Adrien Mazarguil ; Wenzhuo Lu
>>
On 7/23/2019 10:13 AM, Andrew Rybchenko wrote:
> rte_eth_dev_info_get() returns void and caller does know if the function
> does its job or not. Changing of the return value to int would be
> API/ABI breakage which requires deprecation process and cannot be
> backported to stable branches. For now,
Subject: [PATCH] vfio: use contiguous mapping for IOVA as VA mode
When using IOVA as VA mode, there is no need to map segments page by page. This
normally isn't a problem, but it becomes one when attempting to use DPDK in
no-huge mode, where VFIO subsystem simply runs out of space to store ma
23/07/2019 11:57, Burakov, Anatoly:
> A machine without an IOMMU shouldn't have picked IOVA as VA in the first
> place. Perhaps this is something we could fix? I'm not sure how to
> detected that condition though, i don't think there's a mechanism to
> know that for sure. Some kernels create a "
This patch fixes the issue of memory overwrite after the end of
the output buffer by calculating its size as the number of all
segments multipled by the output segment size.
Additionally buffer overflow errors returned by PMD driver are
detected and shown, ending the test.
Also the output buffer s
23/07/2019 10:04, David Marchand:
> From Nithin original cover letter:
> This patch set basically reverts vfio patch that moves eventfd setup
> to probe time (1/3) because msix initialization is broken.
> It adds new ack interrupt api for purposes of simple and light
> weight ack method and also t
When using IOVA as VA mode, there is no need to map segments
page by page. This normally isn't a problem, but it becomes one
when attempting to use DPDK in no-huge mode, where VFIO subsystem
simply runs out of space to store mappings.
Fix this for x86 by triggering different callbacks based on whe
On 23-Jul-19 6:27 AM, Jerin Jacob Kollanukkaran wrote:
-Original Message-
From: Stojaczyk, Dariusz
Sent: Tuesday, July 23, 2019 10:24 AM
To: Jerin Jacob Kollanukkaran ; Thomas Monjalon
; David Marchand ;
Burakov, Anatoly
Cc: dev@dpdk.org
Subject: [EXT] RE: [dpdk-dev] [PATCH v4 0/4] Fixe
rte_eth_dev_info_get() returns void and caller does know if the function
does its job or not. Changing of the return value to int would be
API/ABI breakage which requires deprecation process and cannot be
backported to stable branches. For now, make sure that device info is
initialized even in the
On 22-Jul-19 6:03 PM, Thomas Monjalon wrote:
22/07/2019 18:55, Thomas Monjalon:
When using --no-huge mode, dynamic allocation is not supported.
Because of this limitation, the option --legacy-mem is implied
and -m may be needed to specify the amount of memory to allocate.
Otherwise the default a
Hi Ferruh,
> >> Duplication is bad, it is bad to have two different ways to do same thing.
> >> I am not sure if Vivek is aware of duplicated functions but
> >> explicitly prefers to update old versions.
> >> I suggest lets stop improving these ones and get rid of them asap.
> >
> > Testpmd is w
On 19-07-22 14:44, Bruce Richardson wrote:
> External Email
>
> --
> On Mon, Jul 22, 2019 at 02:39:59PM +0200, kka...@marvell.com wrote:
> > From: Krzysztof Kanas
> >
> > gcc prior 9 don't will add additional warning for unrecog
From: Kiran Kumar K
Add new rte_flow_item_igmp in order to match the Internet Group
Management Protocol based on RFC 2236.
Signed-off-by: Kiran Kumar K
---
lib/librte_ethdev/rte_flow.c | 1 +
lib/librte_ethdev/rte_flow.h | 31 +++
2 files changed, 32 insertions(+)
1 - 100 of 119 matches
Mail list logo
