Commit 2e9cd9d7 authored by David Marchand's avatar David Marchand Committed by Tomasz Zawadzki
Browse files

env_dpdk/pci: fix check on 20.11 EAL API change 



RTE_DEV_ALLOWED is an enum and has no associated define, hence checking
for its presence will always be false.
We could test for RTE_DEV_WHITELISTED define, but this macro added for
deprecation warning will be dropped in the future.
Switch to a check on DPDK version.

Fixes: 10ed0eb7 ("env_dpdk/pci: adapt to 20.11 EAL changes")
Signed-off-by: default avatarDavid Marchand <david.marchand@redhat.com>
Change-Id: I75270977b580065b36c753266cbaa5fb73f99eb1
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5165


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: default avatarAleksey Marchuk <alexeymar@mellanox.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 024179d8

lib/env_dpdk/pci.c

+1 −1
Original line number Diff line number Diff line
@@ -41,7 +41,7 @@ 
#define SYSFS_PCI_DRIVERS	"/sys/bus/pci/drivers"



/* Compatibility for versions < 20.11 */

#ifndef RTE_DEV_ALLOWED

#if RTE_VERSION < RTE_VERSION_NUM(20, 11, 0, 0)

#define RTE_DEV_ALLOWED RTE_DEV_WHITELISTED

#define RTE_DEV_BLOCKED RTE_DEV_BLACKLISTED

#define RTE_BUS_SCAN_ALLOWLIST RTE_BUS_SCAN_WHITELIST