ShellCheck - shell script analysis tool version: 0.10.0 license: GNU General Public License, version 3 website: https://www.shellcheck.net Redirect to /tmp/tmp.fWR0XeBbMj and /tmp/tmp.V9wIJswn5b Tree base: 474b4c08fe15 ("netdevsim: register a new devlink param with default value interface") Now at: c0ad641b3102 ("selftest: netdevsim: test devlink default params") ====== Checking before the patch ====== Checking tools/testing/selftests/drivers/net/netdevsim/devlink.sh - f1444de82439af96e37a8691454de2b2a7f6ecedf0f4a3b16dfb249a22e93677 In devlink.sh line 4: lib_dir=$(dirname $0)/../../../net/forwarding ^-- SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: lib_dir=$(dirname "$0")/../../../net/forwarding In devlink.sh line 6: ALL_TESTS="fw_flash_test params_test regions_test reload_test \ ^-------^ SC2034 (warning): ALL_TESTS appears unused. Verify use (or export if used externally). In devlink.sh line 9: NUM_NETIFS=0 ^--------^ SC2034 (warning): NUM_NETIFS appears unused. Verify use (or export if used externally). In devlink.sh line 10: source $lib_dir/lib.sh ^-------------^ SC1091 (info): Not following: ./lib.sh: openBinaryFile: does not exist (No such file or directory) ^------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: source "$lib_dir"/lib.sh In devlink.sh line 22: "$@" | grep -q $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$@" | grep -q "$DL_HANDLE" In devlink.sh line 43: echo "10"> $DEBUGFS_DIR/fw_update_flash_chunk_time_ms ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "10"> "$DEBUGFS_DIR"/fw_update_flash_chunk_time_ms In devlink.sh line 45: devlink dev flash $DL_HANDLE file $DUMMYFILE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" In devlink.sh line 48: devlink dev flash $DL_HANDLE file $DUMMYFILE component fw.mgmt ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" component fw.mgmt In devlink.sh line 51: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite settings ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite settings In devlink.sh line 54: echo "1"> $DEBUGFS_DIR/fw_update_overwrite_mask ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "1"> "$DEBUGFS_DIR"/fw_update_overwrite_mask In devlink.sh line 57: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite settings ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite settings In devlink.sh line 60: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite identifiers ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite identifiers In devlink.sh line 63: echo "3"> $DEBUGFS_DIR/fw_update_overwrite_mask ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "3"> "$DEBUGFS_DIR"/fw_update_overwrite_mask In devlink.sh line 66: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite identifiers overwrite settings ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite identifiers overwrite settings In devlink.sh line 69: echo "n"> $DEBUGFS_DIR/fw_update_status ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "n"> "$DEBUGFS_DIR"/fw_update_status In devlink.sh line 72: devlink dev flash $DL_HANDLE file $DUMMYFILE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" In devlink.sh line 91: devlink dev param set $DL_HANDLE name $name cmode driverinit value $value ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev param set "$DL_HANDLE" name "$name" cmode driverinit value "$value" In devlink.sh line 102: value=$(param_get $name) ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: value=$(param_get "$name") In devlink.sh line 106: value=$(<$DEBUGFS_DIR/$name) ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: value=$(<"$DEBUGFS_DIR"/"$name") In devlink.sh line 116: local max_macs ^------^ SC2034 (warning): max_macs appears unused. Verify use (or export if used externally). In devlink.sh line 117: local test1 ^---^ SC2034 (warning): test1 appears unused. Verify use (or export if used externally). In devlink.sh line 130: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 143: size=$(devlink region show $DL_HANDLE/$name -j | jq -e -r '.[][].size') ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: size=$(devlink region show "$DL_HANDLE"/"$name" -j | jq -e -r '.[][].size') In devlink.sh line 145: [ $size -eq 32768 ] ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$size" -eq 32768 ] In devlink.sh line 156: count=$(devlink region show $DL_HANDLE/$name -j | jq -e -r '.[][].snapshot | length') ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: count=$(devlink region show "$DL_HANDLE"/"$name" -j | jq -e -r '.[][].snapshot | length') In devlink.sh line 157: [ $count -eq $expected_count ] ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$count" -eq "$expected_count" ] In devlink.sh line 170: echo ""> $DEBUGFS_DIR/take_snapshot ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo ""> "$DEBUGFS_DIR"/take_snapshot In devlink.sh line 174: echo ""> $DEBUGFS_DIR/take_snapshot ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo ""> "$DEBUGFS_DIR"/take_snapshot In devlink.sh line 178: echo ""> $DEBUGFS_DIR/take_snapshot ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo ""> "$DEBUGFS_DIR"/take_snapshot In devlink.sh line 182: devlink region del $DL_HANDLE/dummy snapshot 1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region del "$DL_HANDLE"/dummy snapshot 1 In devlink.sh line 187: devlink region new $DL_HANDLE/dummy snapshot 25 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region new "$DL_HANDLE"/dummy snapshot 25 In devlink.sh line 192: devlink region dump $DL_HANDLE/dummy snapshot 25 >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region dump "$DL_HANDLE"/dummy snapshot 25 >> /dev/null In devlink.sh line 195: devlink region read $DL_HANDLE/dummy snapshot 25 addr 0 len 1 >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr 0 len 1 >> /dev/null In devlink.sh line 198: devlink region read $DL_HANDLE/dummy snapshot 25 addr 128 len 128 >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr 128 len 128 >> /dev/null In devlink.sh line 201: devlink region read $DL_HANDLE/dummy snapshot 25 addr 128 len $((1<<32)) >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr 128 len $((1<<32)) >> /dev/null In devlink.sh line 204: devlink region read $DL_HANDLE/dummy snapshot 25 addr $((1<<32)) len 128 >> /dev/null 2>&1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr $((1<<32)) len 128 >> /dev/null 2>&1 In devlink.sh line 207: devlink region del $DL_HANDLE/dummy snapshot 25 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region del "$DL_HANDLE"/dummy snapshot 25 In devlink.sh line 212: sid=$(devlink -j region new $DL_HANDLE/dummy | jq '.[][][][]') ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sid=$(devlink -j region new "$DL_HANDLE"/dummy | jq '.[][][][]') In devlink.sh line 217: devlink region dump $DL_HANDLE/dummy snapshot $sid >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region dump "$DL_HANDLE"/dummy snapshot "$sid" >> /dev/null In devlink.sh line 220: devlink region del $DL_HANDLE/dummy snapshot $sid ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region del "$DL_HANDLE"/dummy snapshot "$sid" In devlink.sh line 232: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 235: echo "y"> $DEBUGFS_DIR/fail_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "y"> "$DEBUGFS_DIR"/fail_reload In devlink.sh line 238: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 241: echo "n"> $DEBUGFS_DIR/fail_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "n"> "$DEBUGFS_DIR"/fail_reload In devlink.sh line 244: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 247: echo "y"> $DEBUGFS_DIR/dont_allow_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "y"> "$DEBUGFS_DIR"/dont_allow_reload In devlink.sh line 250: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 253: echo "n"> $DEBUGFS_DIR/dont_allow_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "n"> "$DEBUGFS_DIR"/dont_allow_reload In devlink.sh line 256: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 271: devlink dev reload $DL_HANDLE netns testns1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" netns testns1 In devlink.sh line 274: devlink -N testns1 dev reload $DL_HANDLE netns testns2 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 dev reload "$DL_HANDLE" netns testns2 In devlink.sh line 309: devlink dev reload $DL_HANDLE netns testns1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" netns testns1 In devlink.sh line 314: ip -n testns1 link add name $DUMMYDEV type dummy ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns1 link add name "$DUMMYDEV" type dummy In devlink.sh line 316: ip -n testns1 link set $DUMMYDEV up ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns1 link set "$DUMMYDEV" up In devlink.sh line 318: ip -n testns1 a a 192.0.1.1/24 dev $DUMMYDEV ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns1 a a 192.0.1.1/24 dev "$DUMMYDEV" In devlink.sh line 321: local occ=$(res_val_get testns1 IPv4 fib occ) ^-^ SC2155 (warning): Declare and assign separately to avoid masking return values. In devlink.sh line 326: devlink -N testns1 resource set $DL_HANDLE path IPv4/fib size $limit ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 resource set "$DL_HANDLE" path IPv4/fib size "$limit" In devlink.sh line 328: local size_new=$(res_val_get testns1 IPv4 fib size_new) ^------^ SC2155 (warning): Declare and assign separately to avoid masking return values. In devlink.sh line 332: devlink -N testns1 dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 dev reload "$DL_HANDLE" In devlink.sh line 334: local size=$(res_val_get testns1 IPv4 fib size) ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. In devlink.sh line 352: ip -n testns2 link add name $DUMMYDEV type dummy ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns2 link add name "$DUMMYDEV" type dummy In devlink.sh line 354: ip -n testns2 link set $DUMMYDEV up ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns2 link set "$DUMMYDEV" up In devlink.sh line 356: ip -n testns2 a a 192.0.1.1/24 dev $DUMMYDEV ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns2 a a 192.0.1.1/24 dev "$DUMMYDEV" In devlink.sh line 363: devlink -N testns1 dev reload $DL_HANDLE netns testns2 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 dev reload "$DL_HANDLE" netns testns2 In devlink.sh line 366: devlink -N testns2 resource set $DL_HANDLE path IPv4/fib size ' -1' ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns2 resource set "$DL_HANDLE" path IPv4/fib size ' -1' In devlink.sh line 369: devlink -N testns2 dev reload $DL_HANDLE netns 1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns2 dev reload "$DL_HANDLE" netns 1 In devlink.sh line 404: devlink health show $DL_HANDLE reporter empty >/dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health show "$DL_HANDLE" reporter empty >/dev/null In devlink.sh line 407: devlink health dump show $DL_HANDLE reporter empty >/dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health dump show "$DL_HANDLE" reporter empty >/dev/null In devlink.sh line 410: devlink health diagnose $DL_HANDLE reporter empty >/dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health diagnose "$DL_HANDLE" reporter empty >/dev/null In devlink.sh line 413: devlink health recover $DL_HANDLE reporter empty ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter empty In devlink.sh line 428: local show=$(devlink health show $DL_HANDLE reporter $name -j | jq -e -r ".[][][]") ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local show=$(devlink health show "$DL_HANDLE" reporter "$name" -j | jq -e -r ".[][][]") In devlink.sh line 431: local state=$(echo $show | jq -r ".state") ^---^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local state=$(echo "$show" | jq -r ".state") In devlink.sh line 435: local error=$(echo $show | jq -r ".error") ^---^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local error=$(echo "$show" | jq -r ".error") In devlink.sh line 439: local recover=`echo $show | jq -r ".recover"` ^-----^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-----------------------------^ SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local recover=$(echo "$show" | jq -r ".recover") In devlink.sh line 443: local grace_period=$(echo $show | jq -r ".grace_period") ^----------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local grace_period=$(echo "$show" | jq -r ".grace_period") In devlink.sh line 448: local auto_recover=$(echo $show | jq -r ".auto_recover") ^----------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local auto_recover=$(echo "$show" | jq -r ".auto_recover") In devlink.sh line 459: devlink health set $DL_HANDLE reporter dummy auto_recover false ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health set "$DL_HANDLE" reporter dummy auto_recover false In devlink.sh line 465: echo "$BREAK_MSG"> $DEBUGFS_DIR/health/break_health ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$BREAK_MSG"> "$DEBUGFS_DIR"/health/break_health In devlink.sh line 470: local dump=$(devlink health dump show $DL_HANDLE reporter dummy -j) ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local dump=$(devlink health dump show "$DL_HANDLE" reporter dummy -j) In devlink.sh line 473: local dump_break_msg=$(echo $dump | jq -r ".break_message") ^------------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local dump_break_msg=$(echo "$dump" | jq -r ".break_message") In devlink.sh line 477: devlink health dump clear $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health dump clear "$DL_HANDLE" reporter dummy In devlink.sh line 480: devlink health recover $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter dummy In devlink.sh line 485: devlink health set $DL_HANDLE reporter dummy auto_recover true ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health set "$DL_HANDLE" reporter dummy auto_recover true In devlink.sh line 490: echo "$BREAK_MSG"> $DEBUGFS_DIR/health/break_health ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$BREAK_MSG"> "$DEBUGFS_DIR"/health/break_health In devlink.sh line 495: local diagnose=$(devlink health diagnose $DL_HANDLE reporter dummy -j -p) ^------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local diagnose=$(devlink health diagnose "$DL_HANDLE" reporter dummy -j -p) In devlink.sh line 498: local rcvrd_break_msg=$(echo $diagnose | jq -r ".recovered_break_message") ^-------------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local rcvrd_break_msg=$(echo "$diagnose" | jq -r ".recovered_break_message") In devlink.sh line 502: devlink health set $DL_HANDLE reporter dummy grace_period 10 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health set "$DL_HANDLE" reporter dummy grace_period 10 In devlink.sh line 507: echo "Y"> $DEBUGFS_DIR/health/fail_recover ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "Y"> "$DEBUGFS_DIR"/health/fail_recover In devlink.sh line 510: echo "$BREAK_MSG"> $DEBUGFS_DIR/health/break_health ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$BREAK_MSG"> "$DEBUGFS_DIR"/health/break_health In devlink.sh line 515: devlink health recover $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter dummy In devlink.sh line 518: echo "N"> $DEBUGFS_DIR/health/fail_recover ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "N"> "$DEBUGFS_DIR"/health/fail_recover In devlink.sh line 521: devlink health recover $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter dummy In devlink.sh line 526: echo 8192 > $DEBUGFS_DIR/health/binary_len ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo 8192 > "$DEBUGFS_DIR"/health/binary_len In devlink.sh line 529: local dump=$(devlink health dump show $DL_HANDLE reporter dummy -j) ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local dump=$(devlink health dump show "$DL_HANDLE" reporter dummy -j) In devlink.sh line 532: devlink health dump clear $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health dump clear "$DL_HANDLE" reporter dummy In devlink.sh line 543: '.[] | to_entries | .[] | select(.value.type == "leaf") | .key | select(contains("'$handle'"))' ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: '.[] | to_entries | .[] | select(.value.type == "leaf") | .key | select(contains("'"$handle"'"))' In devlink.sh line 551: '.[] | to_entries | .[] | select(.value.type == "node") | .key | select(contains("'$handle'"))' ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: '.[] | to_entries | .[] | select(.value.type == "node") | .key | select(contains("'"$handle"'"))' In devlink.sh line 561: devlink port function rate set $handle $name $value$units ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink port function rate set "$handle" "$name" "$value""$units" In devlink.sh line 569: cmd_jq "devlink port function rate show $handle -j" '.[][].'$name ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: cmd_jq "devlink port function rate show $handle -j" '.[][].'"$name" In devlink.sh line 579: rate_attr_set $handle $name $rate mbit ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_set "$handle" "$name" "$rate" mbit In devlink.sh line 582: local debug_value=$(cat $debug_file) ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local debug_value=$(cat "$debug_file") In devlink.sh line 587: local api_value=$(( $(rate_attr_get $handle $name) * 8 / 1000000 )) ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local api_value=$(( $(rate_attr_get "$handle" "$name") * 8 / 1000000 )) In devlink.sh line 599: rate_attr_set $handle parent $parent ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_set "$handle" parent "$parent" In devlink.sh line 602: debug_value=$(cat $debug_file) ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: debug_value=$(cat "$debug_file") In devlink.sh line 607: api_value=$(rate_attr_get $r_obj parent) ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: api_value=$(rate_attr_get "$r_obj" parent) In devlink.sh line 657: devlink port function rate add $handle ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink port function rate add "$handle" In devlink.sh line 664: devlink port function rate del $handle ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink port function rate del "$handle" In devlink.sh line 669: RET=0 ^-^ SC2034 (warning): RET appears unused. Verify use (or export if used externally). In devlink.sh line 671: echo $VF_COUNT > /sys/bus/netdevsim/devices/$DEV_NAME/sriov_numvfs ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$VF_COUNT" > /sys/bus/netdevsim/devices/"$DEV_NAME"/sriov_numvfs In devlink.sh line 672: devlink dev eswitch set $DL_HANDLE mode switchdev ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev eswitch set "$DL_HANDLE" mode switchdev In devlink.sh line 673: local leafs=`rate_leafs_get $DL_HANDLE` ^---^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-------------------------^ SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local leafs=$(rate_leafs_get "$DL_HANDLE") In devlink.sh line 674: local num_leafs=`echo $leafs | wc -w` ^-------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-------------------^ SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local num_leafs=$(echo "$leafs" | wc -w) In devlink.sh line 681: rate_attr_tx_rate_check $r_obj tx_share $rate \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$r_obj" tx_share "$rate" \ In devlink.sh line 682: $DEBUGFS_DIR/ports/${r_obj##*/}/tx_share ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/ports/"${r_obj##*/}"/tx_share In devlink.sh line 683: rate=$(($rate+10)) ^---^ SC2004 (style): $/${} is unnecessary on arithmetic variables. In devlink.sh line 689: rate_attr_tx_rate_check $r_obj tx_max $rate \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$r_obj" tx_max "$rate" \ In devlink.sh line 690: $DEBUGFS_DIR/ports/${r_obj##*/}/tx_max ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/ports/"${r_obj##*/}"/tx_max In devlink.sh line 691: rate=$(($rate+100)) ^---^ SC2004 (style): $/${} is unnecessary on arithmetic variables. In devlink.sh line 706: local num_nodes=`rate_nodes_get $DL_HANDLE | wc -w` ^-------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-- SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local num_nodes=$(rate_nodes_get "$DL_HANDLE" | wc -w) In devlink.sh line 707: [ $num_nodes == 1 ] ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$num_nodes" == 1 ] In devlink.sh line 711: rate_attr_tx_rate_check $node1 tx_share $node_tx_share \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$node1" tx_share "$node_tx_share" \ In devlink.sh line 712: $DEBUGFS_DIR/rate_nodes/${node1##*/}/tx_share ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/rate_nodes/"${node1##*/}"/tx_share In devlink.sh line 715: rate_attr_tx_rate_check $node1 tx_max $node_tx_max \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$node1" tx_max "$node_tx_max" \ In devlink.sh line 716: $DEBUGFS_DIR/rate_nodes/${node1##*/}/tx_max ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/rate_nodes/"${node1##*/}"/tx_max In devlink.sh line 720: rate_attr_tc_bw_check $node1 "$tc_bw" \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tc_bw_check "$node1" "$tc_bw" \ In devlink.sh line 726: local num_nodes=`rate_nodes_get $DL_HANDLE | wc -w` ^-------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-- SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local num_nodes=$(rate_nodes_get "$DL_HANDLE" | wc -w) In devlink.sh line 727: [ $num_nodes == 0 ] ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$num_nodes" == 0 ] In devlink.sh line 735: rate_attr_parent_check $r_obj $node1_name \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_parent_check "$r_obj" "$node1_name" \ In devlink.sh line 736: $DEBUGFS_DIR/ports/${r_obj##*/}/rate_parent ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/ports/"${r_obj##*/}"/rate_parent In devlink.sh line 743: rate_attr_parent_check $node2 $node1_name \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_parent_check "$node2" "$node1_name" \ In devlink.sh line 744: $DEBUGFS_DIR/rate_nodes/$node2_name/rate_parent ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/rate_nodes/"$node2_name"/rate_parent In devlink.sh line 775: exit $EXIT_STATUS ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: exit "$EXIT_STATUS" For more information: https://www.shellcheck.net/wiki/SC2034 -- ALL_TESTS appears unused. Verify ... https://www.shellcheck.net/wiki/SC2155 -- Declare and assign separately to ... https://www.shellcheck.net/wiki/SC1091 -- Not following: ./lib.sh: openBina... ====== Checking the tree with the patch ====== Checking tools/testing/selftests/drivers/net/netdevsim/devlink.sh - f1444de82439af96e37a8691454de2b2a7f6ecedf0f4a3b16dfb249a22e93677 In devlink.sh line 4: lib_dir=$(dirname $0)/../../../net/forwarding ^-- SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: lib_dir=$(dirname "$0")/../../../net/forwarding In devlink.sh line 6: ALL_TESTS="fw_flash_test params_test \ ^-------^ SC2034 (warning): ALL_TESTS appears unused. Verify use (or export if used externally). In devlink.sh line 10: NUM_NETIFS=0 ^--------^ SC2034 (warning): NUM_NETIFS appears unused. Verify use (or export if used externally). In devlink.sh line 11: source $lib_dir/lib.sh ^-------------^ SC1091 (info): Not following: ./lib.sh: openBinaryFile: does not exist (No such file or directory) ^------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: source "$lib_dir"/lib.sh In devlink.sh line 23: "$@" | grep -q $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$@" | grep -q "$DL_HANDLE" In devlink.sh line 44: echo "10"> $DEBUGFS_DIR/fw_update_flash_chunk_time_ms ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "10"> "$DEBUGFS_DIR"/fw_update_flash_chunk_time_ms In devlink.sh line 46: devlink dev flash $DL_HANDLE file $DUMMYFILE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" In devlink.sh line 49: devlink dev flash $DL_HANDLE file $DUMMYFILE component fw.mgmt ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" component fw.mgmt In devlink.sh line 52: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite settings ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite settings In devlink.sh line 55: echo "1"> $DEBUGFS_DIR/fw_update_overwrite_mask ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "1"> "$DEBUGFS_DIR"/fw_update_overwrite_mask In devlink.sh line 58: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite settings ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite settings In devlink.sh line 61: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite identifiers ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite identifiers In devlink.sh line 64: echo "3"> $DEBUGFS_DIR/fw_update_overwrite_mask ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "3"> "$DEBUGFS_DIR"/fw_update_overwrite_mask In devlink.sh line 67: devlink dev flash $DL_HANDLE file $DUMMYFILE overwrite identifiers overwrite settings ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" overwrite identifiers overwrite settings In devlink.sh line 70: echo "n"> $DEBUGFS_DIR/fw_update_status ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "n"> "$DEBUGFS_DIR"/fw_update_status In devlink.sh line 73: devlink dev flash $DL_HANDLE file $DUMMYFILE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev flash "$DL_HANDLE" file "$DUMMYFILE" In devlink.sh line 95: devlink dev param set $DL_HANDLE name $name cmode $cmode value $value ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev param set "$DL_HANDLE" name "$name" cmode "$cmode" value "$value" In devlink.sh line 103: devlink dev param set $DL_HANDLE name $name default cmode $cmode ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev param set "$DL_HANDLE" name "$name" default cmode "$cmode" In devlink.sh line 120: value=$(param_get $name $attr $cmode) ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: value=$(param_get "$name" "$attr" "$cmode") In devlink.sh line 124: value=$(<$DEBUGFS_DIR/$name) ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: value=$(<"$DEBUGFS_DIR"/"$name") In devlink.sh line 134: local max_macs ^------^ SC2034 (warning): max_macs appears unused. Verify use (or export if used externally). In devlink.sh line 135: local test1 ^---^ SC2034 (warning): test1 appears unused. Verify use (or export if used externally). In devlink.sh line 148: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 179: local default_debugfs=$(value_to_debugfs $expected_default) ^-------------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local default_debugfs=$(value_to_debugfs "$expected_default") In devlink.sh line 180: local new_debugfs=$(value_to_debugfs $new_value) ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local new_debugfs=$(value_to_debugfs "$new_value") In devlink.sh line 184: check_value $param_name initial-default $expected_default $expected_debugfs $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_value "$param_name" initial-default "$expected_default" "$expected_debugfs" "$cmode" In devlink.sh line 186: param_set $param_name $new_value $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: param_set "$param_name" "$new_value" "$cmode" In devlink.sh line 190: check_value $param_name post-set $new_value $expected_debugfs $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_value "$param_name" post-set "$new_value" "$expected_debugfs" "$cmode" In devlink.sh line 192: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 196: check_value $param_name post-reload-new-value $new_value $expected_debugfs $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_value "$param_name" post-reload-new-value "$new_value" "$expected_debugfs" "$cmode" In devlink.sh line 198: param_set_default $param_name $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: param_set_default "$param_name" "$cmode" In devlink.sh line 202: check_value $param_name post-set-default $expected_default $expected_debugfs $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_value "$param_name" post-set-default "$expected_default" "$expected_debugfs" "$cmode" In devlink.sh line 204: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 208: check_value $param_name post-reload-default $expected_default $expected_debugfs $cmode ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: check_value "$param_name" post-reload-default "$expected_default" "$expected_debugfs" "$cmode" In devlink.sh line 229: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 244: size=$(devlink region show $DL_HANDLE/$name -j | jq -e -r '.[][].size') ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: size=$(devlink region show "$DL_HANDLE"/"$name" -j | jq -e -r '.[][].size') In devlink.sh line 246: [ $size -eq 32768 ] ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$size" -eq 32768 ] In devlink.sh line 257: count=$(devlink region show $DL_HANDLE/$name -j | jq -e -r '.[][].snapshot | length') ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: count=$(devlink region show "$DL_HANDLE"/"$name" -j | jq -e -r '.[][].snapshot | length') In devlink.sh line 258: [ $count -eq $expected_count ] ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$count" -eq "$expected_count" ] In devlink.sh line 271: echo ""> $DEBUGFS_DIR/take_snapshot ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo ""> "$DEBUGFS_DIR"/take_snapshot In devlink.sh line 275: echo ""> $DEBUGFS_DIR/take_snapshot ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo ""> "$DEBUGFS_DIR"/take_snapshot In devlink.sh line 279: echo ""> $DEBUGFS_DIR/take_snapshot ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo ""> "$DEBUGFS_DIR"/take_snapshot In devlink.sh line 283: devlink region del $DL_HANDLE/dummy snapshot 1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region del "$DL_HANDLE"/dummy snapshot 1 In devlink.sh line 288: devlink region new $DL_HANDLE/dummy snapshot 25 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region new "$DL_HANDLE"/dummy snapshot 25 In devlink.sh line 293: devlink region dump $DL_HANDLE/dummy snapshot 25 >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region dump "$DL_HANDLE"/dummy snapshot 25 >> /dev/null In devlink.sh line 296: devlink region read $DL_HANDLE/dummy snapshot 25 addr 0 len 1 >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr 0 len 1 >> /dev/null In devlink.sh line 299: devlink region read $DL_HANDLE/dummy snapshot 25 addr 128 len 128 >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr 128 len 128 >> /dev/null In devlink.sh line 302: devlink region read $DL_HANDLE/dummy snapshot 25 addr 128 len $((1<<32)) >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr 128 len $((1<<32)) >> /dev/null In devlink.sh line 305: devlink region read $DL_HANDLE/dummy snapshot 25 addr $((1<<32)) len 128 >> /dev/null 2>&1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region read "$DL_HANDLE"/dummy snapshot 25 addr $((1<<32)) len 128 >> /dev/null 2>&1 In devlink.sh line 308: devlink region del $DL_HANDLE/dummy snapshot 25 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region del "$DL_HANDLE"/dummy snapshot 25 In devlink.sh line 313: sid=$(devlink -j region new $DL_HANDLE/dummy | jq '.[][][][]') ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: sid=$(devlink -j region new "$DL_HANDLE"/dummy | jq '.[][][][]') In devlink.sh line 318: devlink region dump $DL_HANDLE/dummy snapshot $sid >> /dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region dump "$DL_HANDLE"/dummy snapshot "$sid" >> /dev/null In devlink.sh line 321: devlink region del $DL_HANDLE/dummy snapshot $sid ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^--^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink region del "$DL_HANDLE"/dummy snapshot "$sid" In devlink.sh line 333: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 336: echo "y"> $DEBUGFS_DIR/fail_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "y"> "$DEBUGFS_DIR"/fail_reload In devlink.sh line 339: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 342: echo "n"> $DEBUGFS_DIR/fail_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "n"> "$DEBUGFS_DIR"/fail_reload In devlink.sh line 345: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 348: echo "y"> $DEBUGFS_DIR/dont_allow_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "y"> "$DEBUGFS_DIR"/dont_allow_reload In devlink.sh line 351: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 354: echo "n"> $DEBUGFS_DIR/dont_allow_reload ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "n"> "$DEBUGFS_DIR"/dont_allow_reload In devlink.sh line 357: devlink dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" In devlink.sh line 372: devlink dev reload $DL_HANDLE netns testns1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" netns testns1 In devlink.sh line 375: devlink -N testns1 dev reload $DL_HANDLE netns testns2 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 dev reload "$DL_HANDLE" netns testns2 In devlink.sh line 410: devlink dev reload $DL_HANDLE netns testns1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev reload "$DL_HANDLE" netns testns1 In devlink.sh line 415: ip -n testns1 link add name $DUMMYDEV type dummy ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns1 link add name "$DUMMYDEV" type dummy In devlink.sh line 417: ip -n testns1 link set $DUMMYDEV up ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns1 link set "$DUMMYDEV" up In devlink.sh line 419: ip -n testns1 a a 192.0.1.1/24 dev $DUMMYDEV ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns1 a a 192.0.1.1/24 dev "$DUMMYDEV" In devlink.sh line 422: local occ=$(res_val_get testns1 IPv4 fib occ) ^-^ SC2155 (warning): Declare and assign separately to avoid masking return values. In devlink.sh line 427: devlink -N testns1 resource set $DL_HANDLE path IPv4/fib size $limit ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 resource set "$DL_HANDLE" path IPv4/fib size "$limit" In devlink.sh line 429: local size_new=$(res_val_get testns1 IPv4 fib size_new) ^------^ SC2155 (warning): Declare and assign separately to avoid masking return values. In devlink.sh line 433: devlink -N testns1 dev reload $DL_HANDLE ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 dev reload "$DL_HANDLE" In devlink.sh line 435: local size=$(res_val_get testns1 IPv4 fib size) ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. In devlink.sh line 453: ip -n testns2 link add name $DUMMYDEV type dummy ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns2 link add name "$DUMMYDEV" type dummy In devlink.sh line 455: ip -n testns2 link set $DUMMYDEV up ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns2 link set "$DUMMYDEV" up In devlink.sh line 457: ip -n testns2 a a 192.0.1.1/24 dev $DUMMYDEV ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: ip -n testns2 a a 192.0.1.1/24 dev "$DUMMYDEV" In devlink.sh line 464: devlink -N testns1 dev reload $DL_HANDLE netns testns2 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns1 dev reload "$DL_HANDLE" netns testns2 In devlink.sh line 467: devlink -N testns2 resource set $DL_HANDLE path IPv4/fib size ' -1' ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns2 resource set "$DL_HANDLE" path IPv4/fib size ' -1' In devlink.sh line 470: devlink -N testns2 dev reload $DL_HANDLE netns 1 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink -N testns2 dev reload "$DL_HANDLE" netns 1 In devlink.sh line 505: devlink health show $DL_HANDLE reporter empty >/dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health show "$DL_HANDLE" reporter empty >/dev/null In devlink.sh line 508: devlink health dump show $DL_HANDLE reporter empty >/dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health dump show "$DL_HANDLE" reporter empty >/dev/null In devlink.sh line 511: devlink health diagnose $DL_HANDLE reporter empty >/dev/null ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health diagnose "$DL_HANDLE" reporter empty >/dev/null In devlink.sh line 514: devlink health recover $DL_HANDLE reporter empty ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter empty In devlink.sh line 529: local show=$(devlink health show $DL_HANDLE reporter $name -j | jq -e -r ".[][][]") ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local show=$(devlink health show "$DL_HANDLE" reporter "$name" -j | jq -e -r ".[][][]") In devlink.sh line 532: local state=$(echo $show | jq -r ".state") ^---^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local state=$(echo "$show" | jq -r ".state") In devlink.sh line 536: local error=$(echo $show | jq -r ".error") ^---^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local error=$(echo "$show" | jq -r ".error") In devlink.sh line 540: local recover=`echo $show | jq -r ".recover"` ^-----^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-----------------------------^ SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local recover=$(echo "$show" | jq -r ".recover") In devlink.sh line 544: local grace_period=$(echo $show | jq -r ".grace_period") ^----------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local grace_period=$(echo "$show" | jq -r ".grace_period") In devlink.sh line 549: local auto_recover=$(echo $show | jq -r ".auto_recover") ^----------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local auto_recover=$(echo "$show" | jq -r ".auto_recover") In devlink.sh line 560: devlink health set $DL_HANDLE reporter dummy auto_recover false ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health set "$DL_HANDLE" reporter dummy auto_recover false In devlink.sh line 566: echo "$BREAK_MSG"> $DEBUGFS_DIR/health/break_health ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$BREAK_MSG"> "$DEBUGFS_DIR"/health/break_health In devlink.sh line 571: local dump=$(devlink health dump show $DL_HANDLE reporter dummy -j) ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local dump=$(devlink health dump show "$DL_HANDLE" reporter dummy -j) In devlink.sh line 574: local dump_break_msg=$(echo $dump | jq -r ".break_message") ^------------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local dump_break_msg=$(echo "$dump" | jq -r ".break_message") In devlink.sh line 578: devlink health dump clear $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health dump clear "$DL_HANDLE" reporter dummy In devlink.sh line 581: devlink health recover $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter dummy In devlink.sh line 586: devlink health set $DL_HANDLE reporter dummy auto_recover true ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health set "$DL_HANDLE" reporter dummy auto_recover true In devlink.sh line 591: echo "$BREAK_MSG"> $DEBUGFS_DIR/health/break_health ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$BREAK_MSG"> "$DEBUGFS_DIR"/health/break_health In devlink.sh line 596: local diagnose=$(devlink health diagnose $DL_HANDLE reporter dummy -j -p) ^------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local diagnose=$(devlink health diagnose "$DL_HANDLE" reporter dummy -j -p) In devlink.sh line 599: local rcvrd_break_msg=$(echo $diagnose | jq -r ".recovered_break_message") ^-------------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local rcvrd_break_msg=$(echo "$diagnose" | jq -r ".recovered_break_message") In devlink.sh line 603: devlink health set $DL_HANDLE reporter dummy grace_period 10 ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health set "$DL_HANDLE" reporter dummy grace_period 10 In devlink.sh line 608: echo "Y"> $DEBUGFS_DIR/health/fail_recover ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "Y"> "$DEBUGFS_DIR"/health/fail_recover In devlink.sh line 611: echo "$BREAK_MSG"> $DEBUGFS_DIR/health/break_health ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$BREAK_MSG"> "$DEBUGFS_DIR"/health/break_health In devlink.sh line 616: devlink health recover $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter dummy In devlink.sh line 619: echo "N"> $DEBUGFS_DIR/health/fail_recover ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "N"> "$DEBUGFS_DIR"/health/fail_recover In devlink.sh line 622: devlink health recover $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health recover "$DL_HANDLE" reporter dummy In devlink.sh line 627: echo 8192 > $DEBUGFS_DIR/health/binary_len ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo 8192 > "$DEBUGFS_DIR"/health/binary_len In devlink.sh line 630: local dump=$(devlink health dump show $DL_HANDLE reporter dummy -j) ^--^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local dump=$(devlink health dump show "$DL_HANDLE" reporter dummy -j) In devlink.sh line 633: devlink health dump clear $DL_HANDLE reporter dummy ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink health dump clear "$DL_HANDLE" reporter dummy In devlink.sh line 644: '.[] | to_entries | .[] | select(.value.type == "leaf") | .key | select(contains("'$handle'"))' ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: '.[] | to_entries | .[] | select(.value.type == "leaf") | .key | select(contains("'"$handle"'"))' In devlink.sh line 652: '.[] | to_entries | .[] | select(.value.type == "node") | .key | select(contains("'$handle'"))' ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: '.[] | to_entries | .[] | select(.value.type == "node") | .key | select(contains("'"$handle"'"))' In devlink.sh line 662: devlink port function rate set $handle $name $value$units ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink port function rate set "$handle" "$name" "$value""$units" In devlink.sh line 670: cmd_jq "devlink port function rate show $handle -j" '.[][].'$name ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: cmd_jq "devlink port function rate show $handle -j" '.[][].'"$name" In devlink.sh line 680: rate_attr_set $handle $name $rate mbit ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_set "$handle" "$name" "$rate" mbit In devlink.sh line 683: local debug_value=$(cat $debug_file) ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local debug_value=$(cat "$debug_file") In devlink.sh line 688: local api_value=$(( $(rate_attr_get $handle $name) * 8 / 1000000 )) ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local api_value=$(( $(rate_attr_get "$handle" "$name") * 8 / 1000000 )) In devlink.sh line 700: rate_attr_set $handle parent $parent ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_set "$handle" parent "$parent" In devlink.sh line 703: debug_value=$(cat $debug_file) ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: debug_value=$(cat "$debug_file") In devlink.sh line 708: api_value=$(rate_attr_get $r_obj parent) ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: api_value=$(rate_attr_get "$r_obj" parent) In devlink.sh line 758: devlink port function rate add $handle ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink port function rate add "$handle" In devlink.sh line 765: devlink port function rate del $handle ^-----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink port function rate del "$handle" In devlink.sh line 770: RET=0 ^-^ SC2034 (warning): RET appears unused. Verify use (or export if used externally). In devlink.sh line 772: echo $VF_COUNT > /sys/bus/netdevsim/devices/$DEV_NAME/sriov_numvfs ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^-------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: echo "$VF_COUNT" > /sys/bus/netdevsim/devices/"$DEV_NAME"/sriov_numvfs In devlink.sh line 773: devlink dev eswitch set $DL_HANDLE mode switchdev ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: devlink dev eswitch set "$DL_HANDLE" mode switchdev In devlink.sh line 774: local leafs=`rate_leafs_get $DL_HANDLE` ^---^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-------------------------^ SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local leafs=$(rate_leafs_get "$DL_HANDLE") In devlink.sh line 775: local num_leafs=`echo $leafs | wc -w` ^-------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-------------------^ SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local num_leafs=$(echo "$leafs" | wc -w) In devlink.sh line 782: rate_attr_tx_rate_check $r_obj tx_share $rate \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$r_obj" tx_share "$rate" \ In devlink.sh line 783: $DEBUGFS_DIR/ports/${r_obj##*/}/tx_share ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/ports/"${r_obj##*/}"/tx_share In devlink.sh line 784: rate=$(($rate+10)) ^---^ SC2004 (style): $/${} is unnecessary on arithmetic variables. In devlink.sh line 790: rate_attr_tx_rate_check $r_obj tx_max $rate \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$r_obj" tx_max "$rate" \ In devlink.sh line 791: $DEBUGFS_DIR/ports/${r_obj##*/}/tx_max ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/ports/"${r_obj##*/}"/tx_max In devlink.sh line 792: rate=$(($rate+100)) ^---^ SC2004 (style): $/${} is unnecessary on arithmetic variables. In devlink.sh line 807: local num_nodes=`rate_nodes_get $DL_HANDLE | wc -w` ^-------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-- SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local num_nodes=$(rate_nodes_get "$DL_HANDLE" | wc -w) In devlink.sh line 808: [ $num_nodes == 1 ] ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$num_nodes" == 1 ] In devlink.sh line 812: rate_attr_tx_rate_check $node1 tx_share $node_tx_share \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^------------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$node1" tx_share "$node_tx_share" \ In devlink.sh line 813: $DEBUGFS_DIR/rate_nodes/${node1##*/}/tx_share ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/rate_nodes/"${node1##*/}"/tx_share In devlink.sh line 816: rate_attr_tx_rate_check $node1 tx_max $node_tx_max \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tx_rate_check "$node1" tx_max "$node_tx_max" \ In devlink.sh line 817: $DEBUGFS_DIR/rate_nodes/${node1##*/}/tx_max ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/rate_nodes/"${node1##*/}"/tx_max In devlink.sh line 821: rate_attr_tc_bw_check $node1 "$tc_bw" \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_tc_bw_check "$node1" "$tc_bw" \ In devlink.sh line 827: local num_nodes=`rate_nodes_get $DL_HANDLE | wc -w` ^-------^ SC2155 (warning): Declare and assign separately to avoid masking return values. ^-- SC2006 (style): Use $(...) notation instead of legacy backticks `...`. ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: local num_nodes=$(rate_nodes_get "$DL_HANDLE" | wc -w) In devlink.sh line 828: [ $num_nodes == 0 ] ^--------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: [ "$num_nodes" == 0 ] In devlink.sh line 836: rate_attr_parent_check $r_obj $node1_name \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_parent_check "$r_obj" "$node1_name" \ In devlink.sh line 837: $DEBUGFS_DIR/ports/${r_obj##*/}/rate_parent ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/ports/"${r_obj##*/}"/rate_parent In devlink.sh line 844: rate_attr_parent_check $node2 $node1_name \ ^----^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: rate_attr_parent_check "$node2" "$node1_name" \ In devlink.sh line 845: $DEBUGFS_DIR/rate_nodes/$node2_name/rate_parent ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. ^---------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: "$DEBUGFS_DIR"/rate_nodes/"$node2_name"/rate_parent In devlink.sh line 876: exit $EXIT_STATUS ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting. Did you mean: exit "$EXIT_STATUS" For more information: https://www.shellcheck.net/wiki/SC2034 -- ALL_TESTS appears unused. Verify ... https://www.shellcheck.net/wiki/SC2155 -- Declare and assign separately to ... https://www.shellcheck.net/wiki/SC1091 -- Not following: ./lib.sh: openBina...