Hmm, CPUS_PER_TASK is actually a global variable masquerading as a local variable (it is fetched from conf).
Given this, will it still have an issue ?
I do agree with the problem statement btw ...