From 4f11fa5b6c046732054c1636c40435043d9578fb Mon Sep 17 00:00:00 2001 From: Konrad Sztyber Date: Wed, 31 Mar 2021 17:35:36 +0200 Subject: [PATCH] scripts/setup: use `test -e` to test for msr presence Current `test -f` never succeeds, because `/dev/cpu/0/msr` is a character device file. Signed-off-by: Konrad Sztyber Change-Id: Iaa28f04866ab24140668ad43351a37f0ccccef65 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7166 Tested-by: SPDK CI Jenkins Community-CI: Mellanox Build Bot Reviewed-by: Jim Harris Reviewed-by: Tomasz Zawadzki --- scripts/setup.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/setup.sh b/scripts/setup.sh index a2319ec71..4d2346424 100755 --- a/scripts/setup.sh +++ b/scripts/setup.sh @@ -509,7 +509,7 @@ function configure_linux() { fi fi - if [ ! -f /dev/cpu/0/msr ]; then + if [ ! -e /dev/cpu/0/msr ]; then # Some distros build msr as a module. Make sure it's loaded to ensure # DPDK can easily figure out the TSC rate rather than relying on 100ms # sleeps.