From: Edward Cree
> Sent: 27 September 2016 17:36
...
> + case UDP_V4_FLOW:
> + if (efx->rx_hash_udp_4tuple)
> + /* fall through */
> + /* else fall further! */
If you invert the above and add a goto...
if (!efx->rx_hash_udp_4tuple)
goto set_ip;
> case TCP_V4_FLOW:
> - info->data |= RXH_L4_B_0_1 | RXH_L4_B_2_3;
> + info->data |= RXH_L4_B_0_1 | RXH_L4_B_2_3;
> /* fall through */
> - case UDP_V4_FLOW:
> case SCTP_V4_FLOW:
> case AH_ESP_V4_FLOW:
> case IPV4_FLOW:
set_ip:
> info->data |= RXH_IP_SRC | RXH_IP_DST;
> min_revision = EFX_REV_FALCON_B0;
> break;
It might look better.
David
