[dpdk-dev] [PATCH] fm10k: fix vlan flag bug in scattered RX

[Bruce Richardson]

On Fri, Dec 18, 2015 at 05:35:35AM +, He, Shaopeng wrote:
> 
> > -Original Message-
> > From: Wang, Xiao W
> > Sent: Friday, December 18, 2015 11:09 AM
> > To: Chen, Jing D
> > Cc: He, Shaopeng; dev at dpdk.org; Wang, Xiao W
> > Subject: [PATCH] fm10k: fix vlan flag bug in scattered RX
> > 
> > In fm10k_recv_scattered_pkts function, a packet is stored in a linked list,
> > offload flags such as PKT_RX_VLAN_PKT should be set in the first segment.
> > 
> > Signed-off-by: Wang Xiao W 
> 
> Thanks for the catch.
> 
> Acked-by: Shaopeng He 

First commit applied to dpdk-next-net tree.

Applied to dpdk-next-net/rel_16_04 with added fixes line:
Fixes: 6b59a3bc82b1 ("fm10k: fix VLAN in Rx mbuf")

I hope to apply more patches that are ready in the coming days.

Regards,
/Bruce

[dpdk-dev] [PATCH] fm10k: fix vlan flag bug in scattered RX

[Wang Xiao W]

In fm10k_recv_scattered_pkts function, a packet is stored in a linked list,
offload flags such as PKT_RX_VLAN_PKT should be set in the first segment.

Signed-off-by: Wang Xiao W 
---
 drivers/net/fm10k/fm10k_rxtx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/fm10k/fm10k_rxtx.c b/drivers/net/fm10k/fm10k_rxtx.c
index e958865..de31cad 100644
--- a/drivers/net/fm10k/fm10k_rxtx.c
+++ b/drivers/net/fm10k/fm10k_rxtx.c
@@ -305,7 +305,7 @@ fm10k_recv_scattered_pkts(void *rx_queue, struct rte_mbuf 
**rx_pkts,
 * So, always PKT_RX_VLAN_PKT flag is set and vlan_tci
 * is valid for each RX packet's mbuf.
 */
-   mbuf->ol_flags |= PKT_RX_VLAN_PKT;
+   first_seg->ol_flags |= PKT_RX_VLAN_PKT;
first_seg->vlan_tci = desc.w.vlan;

/* Prefetch data of first segment, if configured to do so. */
-- 
1.9.3

[dpdk-dev] [PATCH] fm10k: fix vlan flag bug in scattered RX

[He, Shaopeng]

> -Original Message-
> From: Wang, Xiao W
> Sent: Friday, December 18, 2015 11:09 AM
> To: Chen, Jing D
> Cc: He, Shaopeng; dev at dpdk.org; Wang, Xiao W
> Subject: [PATCH] fm10k: fix vlan flag bug in scattered RX
> 
> In fm10k_recv_scattered_pkts function, a packet is stored in a linked list,
> offload flags such as PKT_RX_VLAN_PKT should be set in the first segment.
> 
> Signed-off-by: Wang Xiao W 

Thanks for the catch.

Acked-by: Shaopeng He