Commit b59e6ef8 authored by Ben Hutchings's avatar Ben Hutchings

sfc: Don't refer to 'stack' in filter implementation

Change all the 'stack' naming to 'auto' (or other meaningful term);
the device address list is based on more than just what the network
stack wants, and the no-match filters aren't really what the stack
wants at all.
Signed-off-by: default avatarBen Hutchings <bhutchings@solarflare.com>
parent 7665d1ab
...@@ -53,25 +53,25 @@ struct efx_ef10_filter_table { ...@@ -53,25 +53,25 @@ struct efx_ef10_filter_table {
struct { struct {
unsigned long spec; /* pointer to spec plus flag bits */ unsigned long spec; /* pointer to spec plus flag bits */
/* BUSY flag indicates that an update is in progress. STACK_OLD is /* BUSY flag indicates that an update is in progress. AUTO_OLD is
* used to mark and sweep stack-owned MAC filters. * used to mark and sweep MAC filters for the device address lists.
*/ */
#define EFX_EF10_FILTER_FLAG_BUSY 1UL #define EFX_EF10_FILTER_FLAG_BUSY 1UL
#define EFX_EF10_FILTER_FLAG_STACK_OLD 2UL #define EFX_EF10_FILTER_FLAG_AUTO_OLD 2UL
#define EFX_EF10_FILTER_FLAGS 3UL #define EFX_EF10_FILTER_FLAGS 3UL
u64 handle; /* firmware handle */ u64 handle; /* firmware handle */
} *entry; } *entry;
wait_queue_head_t waitq; wait_queue_head_t waitq;
/* Shadow of net_device address lists, guarded by mac_lock */ /* Shadow of net_device address lists, guarded by mac_lock */
#define EFX_EF10_FILTER_STACK_UC_MAX 32 #define EFX_EF10_FILTER_DEV_UC_MAX 32
#define EFX_EF10_FILTER_STACK_MC_MAX 256 #define EFX_EF10_FILTER_DEV_MC_MAX 256
struct { struct {
u8 addr[ETH_ALEN]; u8 addr[ETH_ALEN];
u16 id; u16 id;
} stack_uc_list[EFX_EF10_FILTER_STACK_UC_MAX], } dev_uc_list[EFX_EF10_FILTER_DEV_UC_MAX],
stack_mc_list[EFX_EF10_FILTER_STACK_MC_MAX]; dev_mc_list[EFX_EF10_FILTER_DEV_MC_MAX];
int stack_uc_count; /* negative for PROMISC */ int dev_uc_count; /* negative for PROMISC */
int stack_mc_count; /* negative for PROMISC/ALLMULTI */ int dev_mc_count; /* negative for PROMISC/ALLMULTI */
}; };
/* An arbitrary search limit for the software hash table */ /* An arbitrary search limit for the software hash table */
...@@ -2401,7 +2401,7 @@ static s32 efx_ef10_filter_insert(struct efx_nic *efx, ...@@ -2401,7 +2401,7 @@ static s32 efx_ef10_filter_insert(struct efx_nic *efx,
if (saved_spec->priority > EFX_FILTER_PRI_AUTO) if (saved_spec->priority > EFX_FILTER_PRI_AUTO)
saved_spec->flags |= EFX_FILTER_FLAG_RX_OVER_AUTO; saved_spec->flags |= EFX_FILTER_FLAG_RX_OVER_AUTO;
table->entry[ins_index].spec &= table->entry[ins_index].spec &=
~EFX_EF10_FILTER_FLAG_STACK_OLD; ~EFX_EF10_FILTER_FLAG_AUTO_OLD;
rc = ins_index; rc = ins_index;
goto out_unlock; goto out_unlock;
} }
...@@ -2514,13 +2514,13 @@ static void efx_ef10_filter_update_rx_scatter(struct efx_nic *efx) ...@@ -2514,13 +2514,13 @@ static void efx_ef10_filter_update_rx_scatter(struct efx_nic *efx)
} }
/* Remove a filter. /* Remove a filter.
* If !stack_requested, remove by ID * If !by_index, remove by ID
* If stack_requested, remove by index * If by_index, remove by index
* Filter ID may come from userland and must be range-checked. * Filter ID may come from userland and must be range-checked.
*/ */
static int efx_ef10_filter_remove_internal(struct efx_nic *efx, static int efx_ef10_filter_remove_internal(struct efx_nic *efx,
enum efx_filter_priority priority, enum efx_filter_priority priority,
u32 filter_id, bool stack_requested) u32 filter_id, bool by_index)
{ {
unsigned int filter_idx = filter_id % HUNT_FILTER_TBL_ROWS; unsigned int filter_idx = filter_id % HUNT_FILTER_TBL_ROWS;
struct efx_ef10_filter_table *table = efx->filter_state; struct efx_ef10_filter_table *table = efx->filter_state;
...@@ -2547,7 +2547,7 @@ static int efx_ef10_filter_remove_internal(struct efx_nic *efx, ...@@ -2547,7 +2547,7 @@ static int efx_ef10_filter_remove_internal(struct efx_nic *efx,
spec = efx_ef10_filter_entry_spec(table, filter_idx); spec = efx_ef10_filter_entry_spec(table, filter_idx);
if (!spec || if (!spec ||
(!stack_requested && (!by_index &&
efx_ef10_filter_rx_match_pri(table, spec->match_flags) != efx_ef10_filter_rx_match_pri(table, spec->match_flags) !=
filter_id / HUNT_FILTER_TBL_ROWS)) { filter_id / HUNT_FILTER_TBL_ROWS)) {
rc = -ENOENT; rc = -ENOENT;
...@@ -2558,7 +2558,7 @@ static int efx_ef10_filter_remove_internal(struct efx_nic *efx, ...@@ -2558,7 +2558,7 @@ static int efx_ef10_filter_remove_internal(struct efx_nic *efx,
priority == EFX_FILTER_PRI_AUTO) { priority == EFX_FILTER_PRI_AUTO) {
/* Just remove flags */ /* Just remove flags */
spec->flags &= ~EFX_FILTER_FLAG_RX_OVER_AUTO; spec->flags &= ~EFX_FILTER_FLAG_RX_OVER_AUTO;
table->entry[filter_idx].spec &= ~EFX_EF10_FILTER_FLAG_STACK_OLD; table->entry[filter_idx].spec &= ~EFX_EF10_FILTER_FLAG_AUTO_OLD;
rc = 0; rc = 0;
goto out_unlock; goto out_unlock;
} }
...@@ -2572,7 +2572,7 @@ static int efx_ef10_filter_remove_internal(struct efx_nic *efx, ...@@ -2572,7 +2572,7 @@ static int efx_ef10_filter_remove_internal(struct efx_nic *efx,
spin_unlock_bh(&efx->filter_lock); spin_unlock_bh(&efx->filter_lock);
if (spec->flags & EFX_FILTER_FLAG_RX_OVER_AUTO) { if (spec->flags & EFX_FILTER_FLAG_RX_OVER_AUTO) {
/* Reset steering of a stack-owned filter */ /* Reset to an automatic filter */
struct efx_filter_spec new_spec = *spec; struct efx_filter_spec new_spec = *spec;
...@@ -3086,15 +3086,15 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx) ...@@ -3086,15 +3086,15 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx)
/* Mark old filters that may need to be removed */ /* Mark old filters that may need to be removed */
spin_lock_bh(&efx->filter_lock); spin_lock_bh(&efx->filter_lock);
n = table->stack_uc_count < 0 ? 1 : table->stack_uc_count; n = table->dev_uc_count < 0 ? 1 : table->dev_uc_count;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
filter_idx = table->stack_uc_list[i].id % HUNT_FILTER_TBL_ROWS; filter_idx = table->dev_uc_list[i].id % HUNT_FILTER_TBL_ROWS;
table->entry[filter_idx].spec |= EFX_EF10_FILTER_FLAG_STACK_OLD; table->entry[filter_idx].spec |= EFX_EF10_FILTER_FLAG_AUTO_OLD;
} }
n = table->stack_mc_count < 0 ? 1 : table->stack_mc_count; n = table->dev_mc_count < 0 ? 1 : table->dev_mc_count;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
filter_idx = table->stack_mc_list[i].id % HUNT_FILTER_TBL_ROWS; filter_idx = table->dev_mc_list[i].id % HUNT_FILTER_TBL_ROWS;
table->entry[filter_idx].spec |= EFX_EF10_FILTER_FLAG_STACK_OLD; table->entry[filter_idx].spec |= EFX_EF10_FILTER_FLAG_AUTO_OLD;
} }
spin_unlock_bh(&efx->filter_lock); spin_unlock_bh(&efx->filter_lock);
...@@ -3103,28 +3103,28 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx) ...@@ -3103,28 +3103,28 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx)
*/ */
netif_addr_lock_bh(net_dev); netif_addr_lock_bh(net_dev);
if (net_dev->flags & IFF_PROMISC || if (net_dev->flags & IFF_PROMISC ||
netdev_uc_count(net_dev) >= EFX_EF10_FILTER_STACK_UC_MAX) { netdev_uc_count(net_dev) >= EFX_EF10_FILTER_DEV_UC_MAX) {
table->stack_uc_count = -1; table->dev_uc_count = -1;
} else { } else {
table->stack_uc_count = 1 + netdev_uc_count(net_dev); table->dev_uc_count = 1 + netdev_uc_count(net_dev);
memcpy(table->stack_uc_list[0].addr, net_dev->dev_addr, memcpy(table->dev_uc_list[0].addr, net_dev->dev_addr,
ETH_ALEN); ETH_ALEN);
i = 1; i = 1;
netdev_for_each_uc_addr(uc, net_dev) { netdev_for_each_uc_addr(uc, net_dev) {
memcpy(table->stack_uc_list[i].addr, memcpy(table->dev_uc_list[i].addr,
uc->addr, ETH_ALEN); uc->addr, ETH_ALEN);
i++; i++;
} }
} }
if (net_dev->flags & (IFF_PROMISC | IFF_ALLMULTI) || if (net_dev->flags & (IFF_PROMISC | IFF_ALLMULTI) ||
netdev_mc_count(net_dev) >= EFX_EF10_FILTER_STACK_MC_MAX) { netdev_mc_count(net_dev) >= EFX_EF10_FILTER_DEV_MC_MAX) {
table->stack_mc_count = -1; table->dev_mc_count = -1;
} else { } else {
table->stack_mc_count = 1 + netdev_mc_count(net_dev); table->dev_mc_count = 1 + netdev_mc_count(net_dev);
eth_broadcast_addr(table->stack_mc_list[0].addr); eth_broadcast_addr(table->dev_mc_list[0].addr);
i = 1; i = 1;
netdev_for_each_mc_addr(mc, net_dev) { netdev_for_each_mc_addr(mc, net_dev) {
memcpy(table->stack_mc_list[i].addr, memcpy(table->dev_mc_list[i].addr,
mc->addr, ETH_ALEN); mc->addr, ETH_ALEN);
i++; i++;
} }
...@@ -3132,27 +3132,27 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx) ...@@ -3132,27 +3132,27 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx)
netif_addr_unlock_bh(net_dev); netif_addr_unlock_bh(net_dev);
/* Insert/renew unicast filters */ /* Insert/renew unicast filters */
if (table->stack_uc_count >= 0) { if (table->dev_uc_count >= 0) {
for (i = 0; i < table->stack_uc_count; i++) { for (i = 0; i < table->dev_uc_count; i++) {
efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO, efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO,
EFX_FILTER_FLAG_RX_RSS, EFX_FILTER_FLAG_RX_RSS,
0); 0);
efx_filter_set_eth_local(&spec, EFX_FILTER_VID_UNSPEC, efx_filter_set_eth_local(&spec, EFX_FILTER_VID_UNSPEC,
table->stack_uc_list[i].addr); table->dev_uc_list[i].addr);
rc = efx_ef10_filter_insert(efx, &spec, true); rc = efx_ef10_filter_insert(efx, &spec, true);
if (rc < 0) { if (rc < 0) {
/* Fall back to unicast-promisc */ /* Fall back to unicast-promisc */
while (i--) while (i--)
efx_ef10_filter_remove_safe( efx_ef10_filter_remove_safe(
efx, EFX_FILTER_PRI_AUTO, efx, EFX_FILTER_PRI_AUTO,
table->stack_uc_list[i].id); table->dev_uc_list[i].id);
table->stack_uc_count = -1; table->dev_uc_count = -1;
break; break;
} }
table->stack_uc_list[i].id = rc; table->dev_uc_list[i].id = rc;
} }
} }
if (table->stack_uc_count < 0) { if (table->dev_uc_count < 0) {
efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO, efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO,
EFX_FILTER_FLAG_RX_RSS, EFX_FILTER_FLAG_RX_RSS,
0); 0);
...@@ -3160,34 +3160,34 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx) ...@@ -3160,34 +3160,34 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx)
rc = efx_ef10_filter_insert(efx, &spec, true); rc = efx_ef10_filter_insert(efx, &spec, true);
if (rc < 0) { if (rc < 0) {
WARN_ON(1); WARN_ON(1);
table->stack_uc_count = 0; table->dev_uc_count = 0;
} else { } else {
table->stack_uc_list[0].id = rc; table->dev_uc_list[0].id = rc;
} }
} }
/* Insert/renew multicast filters */ /* Insert/renew multicast filters */
if (table->stack_mc_count >= 0) { if (table->dev_mc_count >= 0) {
for (i = 0; i < table->stack_mc_count; i++) { for (i = 0; i < table->dev_mc_count; i++) {
efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO, efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO,
EFX_FILTER_FLAG_RX_RSS, EFX_FILTER_FLAG_RX_RSS,
0); 0);
efx_filter_set_eth_local(&spec, EFX_FILTER_VID_UNSPEC, efx_filter_set_eth_local(&spec, EFX_FILTER_VID_UNSPEC,
table->stack_mc_list[i].addr); table->dev_mc_list[i].addr);
rc = efx_ef10_filter_insert(efx, &spec, true); rc = efx_ef10_filter_insert(efx, &spec, true);
if (rc < 0) { if (rc < 0) {
/* Fall back to multicast-promisc */ /* Fall back to multicast-promisc */
while (i--) while (i--)
efx_ef10_filter_remove_safe( efx_ef10_filter_remove_safe(
efx, EFX_FILTER_PRI_AUTO, efx, EFX_FILTER_PRI_AUTO,
table->stack_mc_list[i].id); table->dev_mc_list[i].id);
table->stack_mc_count = -1; table->dev_mc_count = -1;
break; break;
} }
table->stack_mc_list[i].id = rc; table->dev_mc_list[i].id = rc;
} }
} }
if (table->stack_mc_count < 0) { if (table->dev_mc_count < 0) {
efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO, efx_filter_init_rx(&spec, EFX_FILTER_PRI_AUTO,
EFX_FILTER_FLAG_RX_RSS, EFX_FILTER_FLAG_RX_RSS,
0); 0);
...@@ -3195,20 +3195,20 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx) ...@@ -3195,20 +3195,20 @@ static void efx_ef10_filter_sync_rx_mode(struct efx_nic *efx)
rc = efx_ef10_filter_insert(efx, &spec, true); rc = efx_ef10_filter_insert(efx, &spec, true);
if (rc < 0) { if (rc < 0) {
WARN_ON(1); WARN_ON(1);
table->stack_mc_count = 0; table->dev_mc_count = 0;
} else { } else {
table->stack_mc_list[0].id = rc; table->dev_mc_list[0].id = rc;
} }
} }
/* Remove filters that weren't renewed. Since nothing else /* Remove filters that weren't renewed. Since nothing else
* changes the STACK_OLD flag or removes these filters, we * changes the AUTO_OLD flag or removes these filters, we
* don't need to hold the filter_lock while scanning for * don't need to hold the filter_lock while scanning for
* these filters. * these filters.
*/ */
for (i = 0; i < HUNT_FILTER_TBL_ROWS; i++) { for (i = 0; i < HUNT_FILTER_TBL_ROWS; i++) {
if (ACCESS_ONCE(table->entry[i].spec) & if (ACCESS_ONCE(table->entry[i].spec) &
EFX_EF10_FILTER_FLAG_STACK_OLD) { EFX_EF10_FILTER_FLAG_AUTO_OLD) {
if (efx_ef10_filter_remove_internal( if (efx_ef10_filter_remove_internal(
efx, EFX_FILTER_PRI_AUTO, i, true) < 0) efx, EFX_FILTER_PRI_AUTO, i, true) < 0)
remove_failed = true; remove_failed = true;
......
...@@ -2184,7 +2184,7 @@ efx_farch_filter_to_gen_spec(struct efx_filter_spec *gen_spec, ...@@ -2184,7 +2184,7 @@ efx_farch_filter_to_gen_spec(struct efx_filter_spec *gen_spec,
} }
static void static void
efx_farch_filter_init_rx_for_stack(struct efx_nic *efx, efx_farch_filter_init_rx_auto(struct efx_nic *efx,
struct efx_farch_filter_spec *spec) struct efx_farch_filter_spec *spec)
{ {
/* If there's only one channel then disable RSS for non VF /* If there's only one channel then disable RSS for non VF
...@@ -2547,7 +2547,7 @@ static int efx_farch_filter_remove(struct efx_nic *efx, ...@@ -2547,7 +2547,7 @@ static int efx_farch_filter_remove(struct efx_nic *efx,
return -ENOENT; return -ENOENT;
if (spec->flags & EFX_FILTER_FLAG_RX_OVER_AUTO) { if (spec->flags & EFX_FILTER_FLAG_RX_OVER_AUTO) {
efx_farch_filter_init_rx_for_stack(efx, spec); efx_farch_filter_init_rx_auto(efx, spec);
efx_farch_filter_push_rx_config(efx); efx_farch_filter_push_rx_config(efx);
} else { } else {
efx_farch_filter_table_clear_entry(efx, table, filter_idx); efx_farch_filter_table_clear_entry(efx, table, filter_idx);
...@@ -2815,7 +2815,7 @@ int efx_farch_filter_table_probe(struct efx_nic *efx) ...@@ -2815,7 +2815,7 @@ int efx_farch_filter_table_probe(struct efx_nic *efx)
for (i = 0; i < EFX_FARCH_FILTER_SIZE_RX_DEF; i++) { for (i = 0; i < EFX_FARCH_FILTER_SIZE_RX_DEF; i++) {
spec = &table->spec[i]; spec = &table->spec[i];
spec->type = EFX_FARCH_FILTER_UC_DEF + i; spec->type = EFX_FARCH_FILTER_UC_DEF + i;
efx_farch_filter_init_rx_for_stack(efx, spec); efx_farch_filter_init_rx_auto(efx, spec);
__set_bit(i, table->used_bitmap); __set_bit(i, table->used_bitmap);
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment