[dpdk-dev,PktGen/PCAP] let multi tx queues with pcap work

Message ID 1460706071-17876-1-git-send-email-zhouzhouyi@gmail.com (mailing list archive)
State Not Applicable, archived
Headers

Commit Message

Zhouyi Zhou April 15, 2016, 7:41 a.m. UTC
  Currently, dpdk-pktgen will panic when assigning multi tx queues with pcap
packet, for example:
./app/app/build/pktgen  -c f -n 2 -- -P -m "[0:1-3].0"  -s 0:pcap/large.pcap

This patch fix this bug by assigning qid to memory pool name.

Signed-off-by: Zhouyi Zhou <yizhouzhou@ict.ac.cn>
---
 app/pktgen-pcap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

Wiles, Keith April 15, 2016, 1:33 p.m. UTC | #1
Thank you.

>Currently, dpdk-pktgen will panic when assigning multi tx queues with pcap

>packet, for example:

>./app/app/build/pktgen  -c f -n 2 -- -P -m "[0:1-3].0"  -s 0:pcap/large.pcap

>

>This patch fix this bug by assigning qid to memory pool name.

>

>Signed-off-by: Zhouyi Zhou <yizhouzhou@ict.ac.cn>

>---

> app/pktgen-pcap.c | 2 +-

> 1 file changed, 1 insertion(+), 1 deletion(-)

>

>diff --git a/app/pktgen-pcap.c b/app/pktgen-pcap.c

>index 1af0a53..960294c 100644

>--- a/app/pktgen-pcap.c

>+++ b/app/pktgen-pcap.c

>@@ -337,7 +337,7 @@ pktgen_pcap_parse(pcap_info_t *pcap, port_info_t *info, unsigned qid)

> 

> 	wr_pcap_rewind(pcap);

> 

>-	snprintf(name, sizeof(name), "%-12s%d:%d", "PCAP TX", info->pid, 0);

>+	snprintf(name, sizeof(name), "%-12s%d:%d", "PCAP TX", info->pid, qid);

> 	rte_printf_status("    Process: %-*s ", 18, name);

> 

> 	pkt_sizes = elt_count = i = 0;

>-- 

>1.9.1

>

>

>



Regards,
Keith
  

Patch

diff --git a/app/pktgen-pcap.c b/app/pktgen-pcap.c
index 1af0a53..960294c 100644
--- a/app/pktgen-pcap.c
+++ b/app/pktgen-pcap.c
@@ -337,7 +337,7 @@  pktgen_pcap_parse(pcap_info_t *pcap, port_info_t *info, unsigned qid)
 
 	wr_pcap_rewind(pcap);
 
-	snprintf(name, sizeof(name), "%-12s%d:%d", "PCAP TX", info->pid, 0);
+	snprintf(name, sizeof(name), "%-12s%d:%d", "PCAP TX", info->pid, qid);
 	rte_printf_status("    Process: %-*s ", 18, name);
 
 	pkt_sizes = elt_count = i = 0;