diff options
| author |   Damjan Marion <damarion@cisco.com> | 2018-11-21 12:52:12 +0100 |
|---|---|---|
| committer | Damjan Marion <damarion@cisco.com> | 2018-11-21 12:52:12 +0100 |
| commit | fdd19c46655f6dc3ab4bc6773c681e423d2f0152 (patch) | |
| tree | 351d76282e24f3f5183e0972c80cfd6766c21ee2 | |
| parent | 18934e6d2b3b0b9dad286bd83cc42c66a7160284 (diff) | |
dpdk: don't shift flags twice
Change-Id: I3555230ef7b7677d6187282fb0c0f02af216e0a0
Signed-off-by: Damjan Marion <damarion@cisco.com>
| -rw-r--r-- | src/plugins/dpdk/device/node.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/plugins/dpdk/device/node.c b/src/plugins/dpdk/device/node.c index a0144f702c6..194c359dbac 100644 --- a/src/plugins/dpdk/device/node.c +++ b/src/plugins/dpdk/device/node.c @@ -341,7 +341,7 @@ dpdk_device_input (vlib_main_t * vm, dpdk_main_t * dm, dpdk_device_t * xd, else n_rx_bytes = dpdk_process_rx_burst (vm, ptd, n_rx_packets, 0, &or_flags); - if (PREDICT_FALSE (or_flags & (1 << PKT_RX_FDIR))) + if (PREDICT_FALSE (or_flags & PKT_RX_FDIR)) { /* some packets will need to go to different next nodes */ for (n = 0; n < n_rx_packets; n++) @@ -350,7 +350,7 @@ dpdk_device_input (vlib_main_t * vm, dpdk_main_t * dm, dpdk_device_t * xd, /* flow offload - process if rx flow offload enabled and at least one packet is marked */ if (PREDICT_FALSE ((xd->flags & DPDK_DEVICE_FLAG_RX_FLOW_OFFLOAD) && - (or_flags & (1 << PKT_RX_FDIR)))) + (or_flags & PKT_RX_FDIR))) dpdk_process_flow_offload (xd, ptd, n_rx_packets); /* enqueue buffers to the next node */ @@ -387,7 +387,7 @@ dpdk_device_input (vlib_main_t * vm, dpdk_main_t * dm, dpdk_device_t * xd, marked as ip4 checksum bad we can notify ethernet input so it can send pacets to ip4-input-no-checksum node */ if (xd->flags & DPDK_DEVICE_FLAG_RX_IP4_CKSUM && - (or_flags & (1 << PKT_RX_IP_CKSUM_BAD)) == 0) + (or_flags & PKT_RX_IP_CKSUM_BAD) == 0) f->flags |= ETH_INPUT_FRAME_F_IP4_CKSUM_OK; } n_left_to_next -= n_rx_packets; |