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

Support for GPUs older than pascal #9

Merged
merged 2 commits into from
Feb 5, 2019
Merged

Support for GPUs older than pascal #9

merged 2 commits into from
Feb 5, 2019

Conversation

RavenX8
Copy link
Contributor

@RavenX8 RavenX8 commented Sep 26, 2018

I have tested this using maxwell cards. To convert an NVIDIA vBIOS for VFIO, you do not need to remove the footer (even for pascal cards). The changes I have made here allow you to disable the checks looking for the footer, therefore allowing you to convert an older card for VFIO.

Added support for maxwell GPUs
fixed some syntax errors
@RavenX8 RavenX8 mentioned this pull request Sep 26, 2018
@Matoking
Copy link
Owner

Thanks for posting this pull request.

I tested this pull request by downloading VBIOSes from TechPowerup and checking if they correspond to the vBIOS dumps posted here. The dumps this script generates include some footer data that's not present in the user-submitted dumps over at Unraid forums, such as a very prominent -----BEGIN CERTIFICATE----- section that you can see in dumps downloaded from TechPowerup.

As you said, including the rest of the ROM is probably not a problem, but I'd like to have another confirmation just to be sure before merging this.

@RavenX8
Copy link
Contributor Author

RavenX8 commented Sep 27, 2018

No problem. I had actually used the vBIOSes from TechPowerup and even dumping from my cards directly. They also included the certs.

@MrMeeb
Copy link

MrMeeb commented Feb 1, 2019

Hi, since you were looking for another confirmation, I tested this script with a GT710 and it worked perfectly, both when using a BIOS from techpowerup, and using my own BIOS pulled using GPU-Z

@Matoking Matoking merged commit d269523 into Matoking:master Feb 5, 2019
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.

3 participants