Skip to content
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

[BUG] Latest MLC does not use mlcdrv.sys by default #11

Closed
wiredroid opened this issue Jan 13, 2023 · 6 comments
Closed

[BUG] Latest MLC does not use mlcdrv.sys by default #11

wiredroid opened this issue Jan 13, 2023 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@wiredroid
Copy link

wiredroid commented Jan 13, 2023

Describe the bug
When I run latency check I hard wall on 50ns. When I run using Admin CMD I need to use the -e0 parameter to use Windows Driver to get the correct reading of 41.*ns.

mlc --latency_matrix -e0

image

image

@wiredroid wiredroid added the bug Something isn't working label Jan 13, 2023
@FarisR99 FarisR99 changed the title [BUG] [BUG] Latest MLC does not use mlcdrv.sys by default Jan 13, 2023
@FarisR99
Copy link
Owner

FarisR99 commented Jan 22, 2023

I was able to reproduce this issue last week and fix it, but haven't had the chance to test the fix.

Note that this is only an issue with MLC 3.10 and higher, v3.9a which the GUI downloads still works as expected. I'm going to try supporting new MLC versions as they support newer CPU platforms.

@wiredroid
Copy link
Author

wiredroid commented Jan 23, 2023 via email

@FarisR99
Copy link
Owner

FarisR99 commented Jan 28, 2023

@wiredroid Thanks for filing the bug report, can you please test and confirm it works for you with 3.10? https://github.com/FarisR99/IMLCGui/releases/tag/1.2.0-pre
The download button has also been updated to download 3.10 automatically.

@wiredroid
Copy link
Author

wiredroid commented Jan 28, 2023 via email

@sanyer
Copy link

sanyer commented Jan 28, 2023

@FarisR99 I never experienced this bug, but 1.2.0-pre works well and downloads 3.10.

FarisR99 pushed a commit that referenced this issue Jan 29, 2023
Fixes #11

Update the Download button to download MLC v3.10, which supports
newer CPU platforms including Raptorlake. MLC v3.10 has a different
benchmark behaviour to v3.9 by default, so we must add some flags
to the command line arguments to ensure the result of v3.10 is not
different to v3.9.
@FarisR99 FarisR99 mentioned this issue Jan 29, 2023
@FarisR99
Copy link
Owner

Bumped 1.2.0-pre to 1.2.0 as I won't have time to fix anything until next weekend. Please reopen this issue if the bug is still present, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants