Skip to content

Commit

Permalink
Attempt to fix apps/intel_mp/test_1g_sw_sem.snabb
Browse files Browse the repository at this point in the history
I think the original intent of this test was to test intel_mp without
any rxq/txq, as in test_10g_sw_sem.snabb.  However in the refactoring of
the default values of rxq/txq and how lib.parse was used to provide
defaults, rxq/txq got turned on, which prevents the test from working as
nic:stop recursively acquires the semaphore.
  • Loading branch information
wingo committed Nov 14, 2017
1 parent d15db99 commit b9d1d5d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/apps/intel_mp/test_1g_sw_sem.snabb
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ local parse = require("core.lib").parse
local function new_intel (arg)
return intel.Intel:new(parse(arg, intel.Intel.config))
end
local nic = new_intel({pciaddr = pci0})
local nic = new_intel({pciaddr = pci0, rxq = false, txq = false})

nic:unlock_sw_sem()
nic:lock_sw_sem()
Expand Down

0 comments on commit b9d1d5d

Please sign in to comment.