-
Couldn't load subscription status.
- Fork 794
[CI][Bench] Install perf in proper version #20493
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: sycl
Are you sure you want to change the base?
Conversation
d1ba477 to
140e12a
Compare
140e12a to
3ab267b
Compare
|
Our precommit benchmark test is not working currently - @PatKamin is working on the fix - but my fix was checked in: https://github.com/intel/llvm/actions/runs/18881499077/job/53887826008?pr=20493#step:19:726 |
| # Linux tools installed during docker creation may not match the self-hosted | ||
| # kernel version, so we need to install the correct version here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How exactly would it match the host? Does uname -r report host information or container information?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has to be host, Docker isn't a real VM and uses the host kernel. Note I did zero research.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I did minimal research https://stackoverflow.com/questions/31012297/uname-a-returning-the-same-in-docker-host-or-any-docker-container
Docker uses the host operating system kernel, there is no custom or additional kernel inside the container. All containers running on the machine are sharing this "host" kernel.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, I tested this locally - kernel reported in docker was exactly like my host.
I pulled sycl image, where linux-tools package was installed (in version A), then run apt install -y linux-tools-$(uname -r) in docker and new version B appeared and perf started to work.
Also, as I mentioned above - I tested this on BMG machine with other kernel and worked as well (https://github.com/intel/llvm/actions/runs/18881499077/job/53887826008?pr=20493#step:19:726)
To fix issue:
WARNING: perf not found for kernel 6.17.0-6ref. https://github.com/intel/llvm/actions/runs/18870710633/job/53848872196#step:19:1241
For self-hosted machines kernel packages installed in docker may be unsuitable - install packages required for
perfin each workflow run (to match the actual host).