usertools: enhance device bind script module detection
Checks
Commit Message
From: Pavan Nikhilesh <pbhagavatula@marvell.com>
Some kernel modules have '-' in their name when populated in
'/sys/bus/pci/drivers/' where as the kernel always populates
'/sys/modules/' with '_'.
Replace '-' in module name with '_' to avoid script complaining
that module not loaded.
Signed-off-by: Phanendra Vukkisala <pvukkisala@marvell.com>
Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
usertools/dpdk-devbind.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
Comments
21/09/2019 20:16, pbhagavatula@marvell.com:
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> Some kernel modules have '-' in their name when populated in
> '/sys/bus/pci/drivers/' where as the kernel always populates
> '/sys/modules/' with '_'.
> Replace '-' in module name with '_' to avoid script complaining
> that module not loaded.
Sorry it does not look obvious to me.
Please, could you describe what is wrong and what is the real cause?
@@ -153,6 +153,9 @@ def check_output(args, stderr=None):
def module_is_loaded(module):
global loaded_modules
+ if module == 'vfio_pci':
+ module = 'vfio-pci'
+
if loaded_modules:
return module in loaded_modules
@@ -520,7 +523,7 @@ def bind_all(dev_list, driver, force=False):
pass
# check if we're attempting to bind to a driver that isn't loaded
- if not module_is_loaded(driver):
+ if not module_is_loaded(driver.replace('-','_')):
sys.exit("Error: Driver '%s' is not loaded." % driver)
try: