From: Steven Date: Tue, 11 Apr 2017 19:24:47 +0000 (-0700) Subject: Devices: set interface placement does not remove the old interface placement X-Git-Tag: v17.07-rc1~331 X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=67e06070493607ccff463f6a2c812b76c7a43f1c;p=vpp.git 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 --- 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);