[PATCH v2] fm10k: set packet type for multi-segment packets

Subsystems: networking drivers, the rest

STALE3817d

3 messages, 3 authors, 2016-04-21 · open the first message on its own page

[PATCH v2] fm10k: set packet type for multi-segment packets

From: Michael Frasca <hidden>
Date: 2016-04-18 12:52:04

When building a chain of mbufs for a multi-segment packet, the
packet_type field resides at the end of the chain. It should be
copied forward to the head of the list.

Also, uses RTE_LIBRTE_FM10K_RX_OLFLAGS_ENABLE to guard packet-type
computation. The mbuf fields are not copied when this define is not set.

Fixes: fe65e1e1ce61 ("fm10k: add vector scatter Rx")

Signed-off-by: Michael Frasca <redacted>
---
v2:
- Only copy hash, ol_flags, and packet_type to 'start' when 
  RTE_LIBRTE_FM10K_RX_OLFLAGS_ENABLE is defined.

 drivers/net/fm10k/fm10k_rxtx_vec.c | 3 +++
 1 file changed, 3 insertions(+)
diff --git a/drivers/net/fm10k/fm10k_rxtx_vec.c b/drivers/net/fm10k/fm10k_rxtx_vec.c
index f8efe8f..03e4a5c 100644
--- a/drivers/net/fm10k/fm10k_rxtx_vec.c
+++ b/drivers/net/fm10k/fm10k_rxtx_vec.c
@@ -606,8 +606,11 @@ fm10k_reassemble_packets(struct fm10k_rx_queue *rxq,
 
 			if (!split_flags[buf_idx]) {
 				/* it's the last packet of the set */
+#ifdef RTE_LIBRTE_FM10K_RX_OLFLAGS_ENABLE
 				start->hash = end->hash;
 				start->ol_flags = end->ol_flags;
+				start->packet_type = end->packet_type;
+#endif
 				pkts[pkt_idx++] = start;
 				start = end = NULL;
 			}
-- 
2.5.0

Re: [PATCH v2] fm10k: set packet type for multi-segment packets

From: Chen, Jing D <hidden>
Date: 2016-04-18 14:07:23

Hi,
-----Original Message-----
From: Michael Frasca [mailto:michael.frasca@oracle.com]
Sent: Monday, April 18, 2016 8:52 PM
To: Chen, Jing D <redacted>
Cc: dev@dpdk.org; Michael Frasca <redacted>
Subject: [PATCH v2] fm10k: set packet type for multi-segment packets

When building a chain of mbufs for a multi-segment packet, the packet_type
field resides at the end of the chain. It should be copied forward to the head
of the list.

Also, uses RTE_LIBRTE_FM10K_RX_OLFLAGS_ENABLE to guard packet-type
computation. The mbuf fields are not copied when this define is not set.

Fixes: fe65e1e1ce61 ("fm10k: add vector scatter Rx")

Signed-off-by: Michael Frasca <redacted>
Acked-by : Jing Chen [off-list ref]

Re: [PATCH v2] fm10k: set packet type for multi-segment packets

From: Bruce Richardson <hidden>
Date: 2016-04-21 14:04:45

On Mon, Apr 18, 2016 at 02:05:23PM +0000, Chen, Jing D wrote:
Hi,
quoted
-----Original Message-----
From: Michael Frasca [mailto:michael.frasca@oracle.com]
Sent: Monday, April 18, 2016 8:52 PM
To: Chen, Jing D <redacted>
Cc: dev@dpdk.org; Michael Frasca <redacted>
Subject: [PATCH v2] fm10k: set packet type for multi-segment packets

When building a chain of mbufs for a multi-segment packet, the packet_type
field resides at the end of the chain. It should be copied forward to the head
of the list.

Also, uses RTE_LIBRTE_FM10K_RX_OLFLAGS_ENABLE to guard packet-type
computation. The mbuf fields are not copied when this define is not set.

Fixes: fe65e1e1ce61 ("fm10k: add vector scatter Rx")

Signed-off-by: Michael Frasca <redacted>
Acked-by : Jing Chen [off-list ref]
Applied to dpdk-next-net/rel_16_07

/Bruce
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help