Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@GalFawkes
Copy link

@GalFawkes GalFawkes commented Dec 16, 2025

re.IGNORECASE causes a hit on the CPU model number before the CPU name (this has been edited per my most recent comment on this PR). On a 6th-gen x86 Intel CPU, the CPU ID is 94, not a CPU model string.

Description

I would like to update arm/models/system_info.py to fix a regex issue when ARM is running in Docker on x86 CPUs. Currently, the model number is displayed instead of the CPU name (not expected behavior).

Context: I was wondering why the CPU was listed as 94 instead of as a 6th gen i5.
Motivation: I realized it was a bug.

Fixes #1646 (admittedly, I also created this issue a bit ago.)

Type of change

  • [ x ] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

I used a python shell with the relevant cpuinfo data dumped to a text file as well as a few different regex calculators online. As this is a relatively minor change, I didn't redeploy arm to validate. I am using Docker on Debian Trixie.

  • [ x ] Docker

Checklist:

  • [ x ] My code follows the style guidelines of this project
  • [ x ] I have performed a self-review of my own code
  • [ x ] I have commented my code, particularly in hard-to-understand areas
  • [ x ] I have made corresponding changes to the documentation
  • [ x ] My changes generate no new warnings
  • [ x ] I have tested that my fix is effective or that my feature works

Changelog:

Include the details of changes made here

  • Removed re.IGNORECASE from regex in line 47 of arm/models/system_info.py

Logs

N/A, minor bugfix

`re.IGNORECASE` also somehow ignores spaces. I'm not sure why, but on an x86 Intel CPU, the CPU ID is 94, not a CPU model string.
@sonarqubecloud
Copy link

@GalFawkes
Copy link
Author

GalFawkes commented Dec 17, 2025

Update to "regex ignores spaces":

It doesn't, it just matched "Model" case insensitively, meaning I got ths "model", not the "model name" of my CPU.

Whoops! Error and fix is still valid as far as I can tell 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CPU Names are Not Displayed for x86 Processors

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.