The cpu_topology script has a section of code to extract values of fields in /proc/cpuinfo that it is interested in. Unfortunately the logic gets tripped up where the word 'processor' is contained in the model name of a CPU, for example:
model name: Intel(R) Celeron(R) M processor 900MHz
Instead of using a simple 'string in string' statement it might be better to use string.startswith. I wouldn't go as far as saying regexes are necessary, but the possibility that the string might occur somewhere other than the start of the line needs to be guarded against.
The cpu_topology script has a section of code to extract values of fields in /proc/cpuinfo that it is interested in. Unfortunately the logic gets tripped up where the word 'processor' is contained in the model name of a CPU, for example:
model name: Intel(R) Celeron(R) M processor 900MHz
Instead of using a simple 'string in string' statement it might be better to use string.startswith. I wouldn't go as far as saying regexes are necessary, but the possibility that the string might occur somewhere other than the start of the line needs to be guarded against.