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

scripts/gen_nvme.sh: make bdfs var local to a function

Browse Source 
No need to keep "bdfs" variable as global variable.

Signed-off-by: Karol Latecki <karol.latecki@intel.com>
Change-Id: I655ed9f1514f13a6460292c1298ef08f904157fa
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/8774
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Paul Luse <paul.e.luse@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Pawel Piatek <pawelx.piatek@intel.com>
This commit is contained in:
Karol Latecki 2021-07-13 19:00:46 +02:00 committed by Tomasz Zawadzki
parent 61f3c72cd2
commit 65b3c34b14
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
scripts/gen_nvme.sh
View File

@ -19,6 +19,9 @@ function usage() {
function create_json_config() { function create_json_config() {
local bdev_json_cfg=() local bdev_json_cfg=()
local bdfs=()
bdfs=($(nvme_in_userspace))
for i in "${!bdfs[@]}"; do for i in "${!bdfs[@]}"; do
bdev_json_cfg+=("$( bdev_json_cfg+=("$(
@ -70,8 +73,6 @@ while getopts 'h-:' optchar; do
esac esac
done done
bdfs=($(nvme_in_userspace))
if [[ $gen_subsystems == true ]]; then if [[ $gen_subsystems == true ]]; then
create_json_config_with_subsystems create_json_config_with_subsystems
else else