[dpdk-dev] tx_stop_queue and flow control

2015-08-11 Thread Jia Yu
Hello, DPDK drivers support tx_stop_queue, but they don't use it for flow control, I.e. Stop upper layer calling the device tx_pkt_burst routine when there is no resource. I wonder if flow control is needed here, if yes, is there plan to add this support? Thanks, Jia

[dpdk-dev] [PATCH v2] librte_pmd_bond: remove memory alloc for rte_pci_driver

2015-03-23 Thread Jia Yu
eth_driver already contains rte_pci_driver data structure. Allocating rte_pci_driver without referencing it after bond creation causes memory leakage. Added signed off information. Signed-off-by: Jia Yu --- lib/librte_pmd_bond/rte_eth_bond_api.c | 11 ++- 1 file changed, 2 insertions

[dpdk-dev] [PATCH] librte_pmd_bond: remove memory alloc for rte_pci_driver

2015-03-12 Thread Jia Yu
eth_driver already contains rte_pci_driver data structure. Allocating rte_pci_driver without referencing it after bond creation causes memory leakage. --- lib/librte_pmd_bond/rte_eth_bond_api.c | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git

[dpdk-dev] [PATCH 1/2] rte_ethdev: update link status (speed, duplex, link_up) after rte_eth_dev_start

2015-02-03 Thread Jia Yu
c != 0). Therefore, for hardware NIC (e.g. 82599) that disabled lsc by default, the link_update callback will not be executed. Please let me know if you have other concerns. Thanks, Jia On 2/3/15, 12:35 AM, "Zhang, Helin" wrote: > > >> -Original Message- >> From

[dpdk-dev] [PATCH 1/2] rte_ethdev: update link status (speed, duplex, link_up) after rte_eth_dev_start

2015-02-03 Thread Jia Yu
t;2014-11-12 03:57, Zhang, Helin: >> Hi Jia >> >> > -Original Message- >> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jia Yu >> > Sent: Saturday, November 8, 2014 1:32 AM >> > To: dev at dpdk.org >> > Subject: [dpdk-dev

[dpdk-dev] lib: include rte_memory.h for __rte_cache_aligned

2014-12-10 Thread Jia Yu
orman" wrote: >On Tue, Dec 09, 2014 at 09:53:18AM +0100, Olivier MATZ wrote: >> Hi Neil, >> >> On 12/08/2014 04:04 PM, Neil Horman wrote: >> >On Fri, Nov 07, 2014 at 09:28:09AM -0800, Jia Yu wrote: >> >>Include rte_memory.h for lib files that use __rte

[dpdk-dev] lib: include rte_memory.h for __rte_cache_aligned

2014-12-09 Thread Jia Yu
(as validated by parole). For good practice, it?s better to explicitly include rte_memory.h to avoid the problem. Thanks, Jia On 12/9/14, 12:53 AM, "Olivier MATZ" wrote: >Hi Neil, > >On 12/08/2014 04:04 PM, Neil Horman wrote: >> On Fri, Nov 07, 2014 at 09:28:09AM -0800,

[dpdk-dev] [PATCH] lib: include rte_memory.h for __rte_cache_aligned

2014-11-17 Thread Jia Yu
2014-11-07 09:28, Jia Yu: >> Include rte_memory.h for lib files that use __rte_cache_aligned >> attribute. > >Please could you explain what was the error? >As I suspect it's a fix, it would be clearer to start your title with >"fix". > >Thanks >-- >Thomas

[dpdk-dev] [PATCH 0/2] rte_ethdev fix/improvement

2014-11-11 Thread Jia Yu
Thanks, Thomas. The two patches are minor fixes. No new API is introduced. I will revise the description. Thanks, Jia On 11/10/14, 1:40 AM, "Thomas Monjalon" wrote: >Hi Jia, > >2014-11-07 09:31, Jia Yu: >> This patch series include a fix and an improvement

[dpdk-dev] [PATCH] librte_kni: Add buildtime checks for rte_kni_mbuf and rte_mbuf

2014-11-07 Thread Jia Yu
Adding this check is to avoid breakage from future data structure changes. Signed-off-by: Jia Yu --- lib/librte_kni/rte_kni.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/lib/librte_kni/rte_kni.c b/lib/librte_kni/rte_kni.c index 31ecb33..f5c449c 100644 --- a/lib

[dpdk-dev] [PATCH] Bond: set {rx|tx}_offload_capa flags

2014-11-07 Thread Jia Yu
devices are unused (i.e. linked down, standby). Signed-off-by: Jia Yu --- lib/librte_pmd_bond/rte_eth_bond_api.c | 16 lib/librte_pmd_bond/rte_eth_bond_pmd.c | 5 + lib/librte_pmd_bond/rte_eth_bond_private.h | 2 ++ 3 files changed, 23 insertions(+) diff --git a/lib

[dpdk-dev] [PATCH 1/2] rte_ethdev: update link status (speed, duplex, link_up) after rte_eth_dev_start

2014-11-07 Thread Jia Yu
link status from hardware register at device start time. Issue: Change-Id: Ib57a1c9114f922485c7b0f4338bfe7b3d3f87d65 Signed-off-by: Jia Yu --- lib/librte_ether/rte_ethdev.c | 4 1 file changed, 4 insertions(+) diff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c index

[dpdk-dev] [PATCH 0/2] rte_ethdev fix/improvement

2014-11-07 Thread Jia Yu
This patch series include a fix and an improvement to rte_ethdev lib. Jia Yu (2): rte_ethdev: update link status (speed, duplex, link_up) after rte_eth_dev_start rte_ethdev: add return status for rte_eth_stats_get lib/librte_ether/rte_ethdev.c | 11 --- lib/librte_ether

[dpdk-dev] [PATCH] lib: include rte_memory.h for __rte_cache_aligned

2014-11-07 Thread Jia Yu
Include rte_memory.h for lib files that use __rte_cache_aligned attribute. Signed-off-by: Jia Yu --- lib/librte_distributor/rte_distributor.c| 1 + lib/librte_eal/common/include/rte_malloc_heap.h | 1 + lib/librte_ip_frag/rte_ip_frag.h| 1 + lib/librte_malloc

[dpdk-dev] [PATCH] app/test-pmd: Use the new rss_hf size changed in 1.7

2014-11-07 Thread Jia Yu
According to rss_hf definition in rte_eth_rss_conf, it shall be uint64 type. Using uint16 will get truncated value, and cause incorrect output. This fix corrected this issue. Signed-off-by: Jia Yu --- app/test-pmd/config.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app