X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=src%2Fvnet%2Fbonding%2Fcli.c;h=cb344c611c28363901d88f79018771d9cbb0d0fa;hb=59fa121f8953f7b07f0cc02149ca28182f959f42;hp=14c52f70b0beb67c4048b397303c80d77f2c2ec1;hpb=b7b929931a07fbb27b43d5cd105f366c3e29807e;p=vpp.git diff --git a/src/vnet/bonding/cli.c b/src/vnet/bonding/cli.c index 14c52f70b0b..cb344c611c2 100644 --- a/src/vnet/bonding/cli.c +++ b/src/vnet/bonding/cli.c @@ -28,6 +28,8 @@ bond_disable_collecting_distributing (vlib_main_t * vm, slave_if_t * sif) bond_if_t *bif; int i; uword p; + vnet_main_t *vnm = vnet_get_main (); + vnet_hw_interface_t *hw; u8 switching_active = 0; bif = bond_get_master_by_dev_instance (sif->bif_dev_instance); @@ -37,22 +39,53 @@ bond_disable_collecting_distributing (vlib_main_t * vm, slave_if_t * sif) p = *vec_elt_at_index (bif->active_slaves, i); if (p == sif->sw_if_index) { - /* Are we disabling the very 1st slave? */ - if (sif->sw_if_index == *vec_elt_at_index (bif->active_slaves, 0)) - switching_active = 1; - + if (sif->sw_if_index == bif->sw_if_index_working) + { + switching_active = 1; + if (bif->mode == BOND_MODE_ACTIVE_BACKUP) + bif->is_local_numa = 0; + } vec_del1 (bif->active_slaves, i); hash_unset (bif->active_slave_by_sw_if_index, sif->sw_if_index); - - /* We got a new slave just becoming active? */ - if ((vec_len (bif->active_slaves) >= 1) && - (bif->mode == BOND_MODE_ACTIVE_BACKUP) && switching_active) - vlib_process_signal_event (bm->vlib_main, bond_process_node.index, - BOND_SEND_GARP_NA, bif->hw_if_index); break; } } + + /* We get a new slave just becoming active */ + if ((bif->mode == BOND_MODE_ACTIVE_BACKUP) && switching_active) + { + if ((vec_len (bif->active_slaves) >= 1)) + { + /* scan all slaves and try to find the first slave with local numa node. */ + vec_foreach_index (i, bif->active_slaves) + { + p = *vec_elt_at_index (bif->active_slaves, i); + hw = vnet_get_sup_hw_interface (vnm, p); + if (vm->numa_node == hw->numa_node) + { + bif->sw_if_index_working = p; + bif->is_local_numa = 1; + vlib_process_signal_event (bm->vlib_main, + bond_process_node.index, + BOND_SEND_GARP_NA, + bif->hw_if_index); + break; + } + } + } + + /* No local numa node is found in the active slave set. Use the first slave */ + if ((bif->is_local_numa == 0) && (vec_len (bif->active_slaves) >= 1)) + { + p = *vec_elt_at_index (bif->active_slaves, 0); + bif->sw_if_index_working = p; + vlib_process_signal_event (bm->vlib_main, bond_process_node.index, + BOND_SEND_GARP_NA, bif->hw_if_index); + } + } clib_spinlock_unlock_if_init (&bif->lockp); + + return; } void @@ -60,6 +93,10 @@ bond_enable_collecting_distributing (vlib_main_t * vm, slave_if_t * sif) { bond_if_t *bif; bond_main_t *bm = &bond_main; + vnet_main_t *vnm = vnet_get_main (); + vnet_hw_interface_t *hw = vnet_get_sup_hw_interface (vnm, sif->sw_if_index); + int i; + uword p; bif = bond_get_master_by_dev_instance (sif->bif_dev_instance); clib_spinlock_lock_if_init (&bif->lockp); @@ -72,10 +109,41 @@ bond_enable_collecting_distributing (vlib_main_t * vm, slave_if_t * sif) /* First slave becomes active? */ if ((vec_len (bif->active_slaves) == 1) && (bif->mode == BOND_MODE_ACTIVE_BACKUP)) - vlib_process_signal_event (bm->vlib_main, bond_process_node.index, - BOND_SEND_GARP_NA, bif->hw_if_index); + { + bif->sw_if_index_working = sif->sw_if_index; + bif->is_local_numa = (vm->numa_node == hw->numa_node) ? 1 : 0; + vlib_process_signal_event (bm->vlib_main, bond_process_node.index, + BOND_SEND_GARP_NA, bif->hw_if_index); + } + else if ((vec_len (bif->active_slaves) > 1) + && (bif->mode == BOND_MODE_ACTIVE_BACKUP) + && bif->is_local_numa == 0) + { + if (vm->numa_node == hw->numa_node) + { + vec_foreach_index (i, bif->active_slaves) + { + p = *vec_elt_at_index (bif->active_slaves, 0); + if (p == sif->sw_if_index) + break; + + vec_del1 (bif->active_slaves, 0); + hash_unset (bif->active_slave_by_sw_if_index, p); + vec_add1 (bif->active_slaves, p); + hash_set (bif->active_slave_by_sw_if_index, p, p); + } + bif->sw_if_index_working = sif->sw_if_index; + bif->is_local_numa = 1; + vlib_process_signal_event (bm->vlib_main, + bond_process_node.index, + BOND_SEND_GARP_NA, bif->hw_if_index); + + } + } } clib_spinlock_unlock_if_init (&bif->lockp); + + return; } int @@ -92,6 +160,7 @@ bond_dump_ifs (bond_interface_details_t ** out_bondifs) pool_foreach (bif, bm->interfaces, vec_add2(r_bondifs, bondif, 1); clib_memset (bondif, 0, sizeof (*bondif)); + bondif->id = bif->id; bondif->sw_if_index = bif->sw_if_index; hi = vnet_get_hw_interface (vnm, bif->hw_if_index); clib_memcpy(bondif->interface_name, hi->name, @@ -231,6 +300,7 @@ bond_delete_if (vlib_main_t * vm, u32 sw_if_index) clib_bitmap_free (bif->port_number_bitmap); hash_unset (bm->bond_by_sw_if_index, bif->sw_if_index); + hash_unset (bm->id_used, bif->id); clib_memset (bif, 0, sizeof (*bif)); pool_put (bm->interfaces, bif); @@ -266,9 +336,21 @@ bond_create_if (vlib_main_t * vm, bond_create_if_args_t * args) pool_get (bm->interfaces, bif); clib_memset (bif, 0, sizeof (*bif)); bif->dev_instance = bif - bm->interfaces; + bif->id = args->id; bif->lb = args->lb; bif->mode = args->mode; + // Adjust requested interface id + if (bif->id == ~0) + bif->id = bif->dev_instance; + if (hash_get (bm->id_used, bif->id)) + { + args->rv = VNET_API_ERROR_INSTANCE_IN_USE; + pool_put (bm->interfaces, bif); + return; + } + hash_set (bm->id_used, bif->id, 1); + // Special load-balance mode used for rr and bc if (bif->mode == BOND_MODE_ROUND_ROBIN) bif->lb = BOND_LB_RR; @@ -291,13 +373,14 @@ bond_create_if (vlib_main_t * vm, bond_create_if_args_t * args) } memcpy (bif->hw_address, args->hw_addr, 6); args->error = ethernet_register_interface - (vnm, bond_dev_class.index, bif - bm->interfaces /* device instance */ , + (vnm, bond_dev_class.index, bif->dev_instance /* device instance */ , bif->hw_address /* ethernet address */ , &bif->hw_if_index, 0 /* flag change */ ); if (args->error) { args->rv = VNET_API_ERROR_INVALID_REGISTRATION; + hash_unset (bm->id_used, bif->id); pool_put (bm->interfaces, bif); return; } @@ -329,6 +412,7 @@ bond_create_command_fn (vlib_main_t * vm, unformat_input_t * input, if (!unformat_user (input, unformat_line_input, line_input)) return clib_error_return (0, "Missing required arguments."); + args.id = ~0; args.mode = -1; args.lb = BOND_LB_L2; while (unformat_check_input (line_input) != UNFORMAT_END_OF_INPUT) @@ -342,6 +426,8 @@ bond_create_command_fn (vlib_main_t * vm, unformat_input_t * input, else if (unformat (line_input, "hw-addr %U", unformat_ethernet_address, args.hw_addr)) args.hw_addr_set = 1; + else if (unformat (line_input, "id %u", &args.id)) + ; else return clib_error_return (0, "unknown input `%U'", format_unformat_error, input); @@ -360,7 +446,8 @@ bond_create_command_fn (vlib_main_t * vm, unformat_input_t * input, VLIB_CLI_COMMAND (bond_create_command, static) = { .path = "create bond", .short_help = "create bond mode {round-robin | active-backup | broadcast | " - "{lacp | xor} [load-balance { l2 | l23 | l34 }]} [hw-addr ]", + "{lacp | xor} [load-balance { l2 | l23 | l34 }]} [hw-addr ] " + "[id ]", .function = bond_create_command_fn, }; /* *INDENT-ON* */ @@ -512,11 +599,13 @@ bond_enslave (vlib_main_t * vm, bond_enslave_args_t * args) ethernet_set_rx_redirect (vnm, sif_hw, 1); } - if ((bif->mode == BOND_MODE_LACP) && bm->lacp_enable_disable) + if (bif->mode == BOND_MODE_LACP) { - (*bm->lacp_enable_disable) (vm, bif, sif, 1); + if (bm->lacp_enable_disable) + (*bm->lacp_enable_disable) (vm, bif, sif, 1); } - else + else if (sif->port_enabled && + (sif_hw->flags & VNET_HW_INTERFACE_FLAG_LINK_UP)) { bond_enable_collecting_distributing (vm, sif); } @@ -716,6 +805,7 @@ show_bond_details (vlib_main_t * vm) vnet_get_main (), *sw_if_index); } vlib_cli_output (vm, " device instance: %d", bif->dev_instance); + vlib_cli_output (vm, " interface id: %d", bif->id); vlib_cli_output (vm, " sw_if_index: %d", bif->sw_if_index); vlib_cli_output (vm, " hw_if_index: %d", bif->hw_if_index); }));