-
Notifications
You must be signed in to change notification settings - Fork 119
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
CPU-X does not start anymore since 4.5.0 #246
Comments
It seems it founds 2 OpenCL platforms:
Then it crashes later, but backtrace output overlap with other messages, weird. Segmentation fault appears in libnvidia-opencl.so. Maybe a threading issue. |
@ptr1337 if I understand well, you did not face this issue in v4.4.0? |
Yes, correct. Actually I have compiled the 4.4 version again and started it with --verbose:
It seems to be a regression between 4.4.0 and 4.5.0. |
Hmmm, when I check changes in Can you retry to build and install the |
Sure. Actually I have compiled a cpu-x-git and reverted this commit, and it seems not to help.
|
Please the result of |
That is the content of cpu-x.log file. 😉 @ptr1337 thank you for bisecting. It is pointing the Vulkan code here, but Valgrind output is not helpful unfortunately. |
@Umio-Yasuno
cpu-x --debug -v (4.5.0)
clinfo:
|
|
By setting the path of the OpenCL ICD file to |
Sorry for the dumb question, I have never done anything with opencl. Which program should I test ? |
@ptr1337 Please run But, Arch Linux may have ICD installed in a different location. |
Ah, then I did understood it right first.
|
Hmm... I have no idea... |
@ptr1337 thank you for the new dump, this is helpful. @Umio-Yasuno the bug seems not related to OpenCL code ( |
If @Umio-Yasuno in case below, you are evaluating if(((uint32_t)dev->domain == bus_info.pciDomain &&
dev->bus == bus_info.pciBus &&
dev->dev == bus_info.pciDevice &&
dev->func == bus_info.pciFunction) ||
(use_device_id &&
(uint32_t)dev->device_id == prop2.properties.deviceID)) I added some commits on master, maybe a18f9c8 fixed something. @ptr1337 could you please rebuild the cpu-x-git package from master, then rerun |
Hey! Here the log of valgrind:
|
Version strings must be keep as it: there is no need to remove stuff from string. Related to #246
Ok, thanks. I fixed the If it does not crash anymore, let's close this issue. |
Much thanks for fixing so fast the issue! Actually one question I would have - Is it possible to show at cpu-x (like at cpu-z on windows) the timings/freq of the ram? |
It was requested some time ago in #96. So that is why this feature is missing in CPU-X. |
Yes, I noticed that, that at linux general is a missing possibility for that. Maybe in the future! Thanks for your fast info and your work! |
Sorry, it was a mistake in my code. |
Describe the bug/Expected behavior
Additional information
Bug information
The text was updated successfully, but these errors were encountered: