aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven <sluong@cisco.com>2017-04-11 12:24:47 -0700
committerDamjan Marion <dmarion.lists@gmail.com>2017-04-11 20:46:52 +0000
commit67e06070493607ccff463f6a2c812b76c7a43f1c (patch)
tree9a93fee7f2f4cf89702acba839d66e468acc139f
parent7a1bde061a919af42645e3bc09206b98fbfde9d9 (diff)
Devices: set interface placement does not remove the old interface placement
The command set interface placement is supposed to remove the existing interface placement and add a new interface placement based on the given options. My quick test for the CLI shows that the old interface placement continues to exist. But the new interface placement is also added. The bug exists in vnet_device_input_unassign_thread which checks the old thread index is the same as the passed thread index and skips the deletion if they are the same. The fix is to remove the check which is not supposed to be there. Change-Id: Ib055721fad47513949a03b3cb6dc292bd19fd1e8 Signed-off-by: Steven <sluong@cisco.com>
-rw-r--r--src/vnet/devices/devices.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/vnet/devices/devices.c b/src/vnet/devices/devices.c
index c8a95087ec6..2f55adcb20f 100644
--- a/src/vnet/devices/devices.c
+++ b/src/vnet/devices/devices.c
@@ -174,9 +174,6 @@ vnet_device_input_unassign_thread (vnet_main_t * vnm, u32 hw_if_index,
old_thread_index = hw->input_node_thread_index_by_queue[queue_id];
- if (old_thread_index == thread_index)
- return 0;
-
vm = vlib_mains[old_thread_index];
rt = vlib_node_get_runtime_data (vm, hw->input_node_index);