ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test/nvme: fix get_numa_node function

Browse Source 
.driver_specific.nvme is an array, so return all elements
of this array before trying to access .pci_address field.

We only expect a single PCIe address in this test, so it's
safe.

Signed-off-by: Maciej Wawryk <maciejx.wawryk@intel.com>
Change-Id: I0697ab2ffb1adff2c6f288bba460858bf594b219
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11406
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
This commit is contained in:
Maciej Wawryk 2022-02-04 10:43:12 +01:00 committed by Tomasz Zawadzki
parent 64454afb7c
commit 7bce0a3ead
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/nvme/perf/common.sh
View File

@ -136,7 +136,7 @@ function get_numa_node() {
bdevs=$(discover_bdevs $rootdir $testdir/bdev.conf) bdevs=$(discover_bdevs $rootdir $testdir/bdev.conf)
for name in $disks; do for name in $disks; do
local bdev_bdf local bdev_bdf
bdev_bdf=$(jq -r ".[] | select(.name==\"$name\").driver_specific.nvme.pci_address" <<< $bdevs) bdev_bdf=$(jq -r ".[] | select(.name==\"$name\").driver_specific.nvme[].pci_address" <<< "$bdevs")
cat /sys/bus/pci/devices/$bdev_bdf/numa_node cat /sys/bus/pci/devices/$bdev_bdf/numa_node
done done
else else