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