Minimal change to the way BACKUP_LA_LIMIT is calculated

[sahsanu]

The other day I saw that BACKUP_LA_LIMIT was showing 4 processors but I'm using 2 so the problem here is the name of the processor (Common KVM processor) and as the command is

BACKUP_LA_LIMIT=$(cat /proc/cpuinfo | grep processor | wc -l)

It counts 4 processors instead of 2:

$ cat /proc/cpuinfo | grep processor
processor       : 0
model name      : Common KVM processor
processor       : 1
model name      : Common KVM processor

So I changed it to avoid this issue:

BACKUP_LA_LIMIT=$(cat /proc/cpuinfo | grep '^processor' | wc -l)

[Skamasle]

Hi

I will recomend use more efficient way to this

Here we call cat, grep and wc to get a single CPU number, remember the unix way, keep it simple!

You can do it with one command

lscpu |awk ' $1 == "CPU(s):" { print $2 }'

or if you want check /proc/cpuinfo why not just grep

grep -c '^processor' /proc/cpuinfo
Same result but faster,

# time grep -c  '^processor' /proc/cpuinfo
1

real	0m0.002s
user	0m0.002s
sys	0m0.000s

# time cat /proc/cpuinfo | grep '^processor' | wc -l
1

real	0m0.038s
user	0m0.001s
sys	0m0.005s

time lscpu |awk ' $1 == "CPU(s):" { print $2 }'
1

real	0m0.006s
user	0m0.002s
sys	0m0.003s

[jaapmarcus]

Made the change