Skip to content

Conversation

@sigreturn
Copy link
Contributor

Title.

I ended up going with the simpler solution proposed in the earlier pull request (#4019)

@sigreturn sigreturn requested a review from Torxed as a code owner December 30, 2025 04:59
Comment on lines +49 to +50
if gfx_driver == 'Nvidia (proprietary)':
_gfx_driver = GfxDriver.NvidiaOpenSource
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, I understand the "why".
But I'm not sure it's a great idea to let the user be un-informed when they chose "proprietary" but got "nouveau" :/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you propose then? I'm not sure how we could clearly communicate this unless there is some pop up when the installer is first ran telling them what is going to happen and that they should probably update their config (and read the announcement regarding packaging changes)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think one option is to not show the nvidia prop driver at all, if it's not supported?
But not sure how to do that based on hardware ID's and stuff

Copy link
Contributor Author

@sigreturn sigreturn Dec 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This pull request removes the proprietary driver from the menus, but if you read the previous pull request (#4019) @svartkanin was worried about backwards compatibility with older configs, hence why this is here.

As of driver release 590.48.01, they are no longer supporting the proprietary driver hence it was removed from the repositories which only leaves the open kernel module available and it only supports Turing+ hardware (GTX 1650/RTX 2000 and newer)

If we fall back to the open kernel module for these older configs and the user happens to have pre-Turing hardware, they're going to be met with a black screen when they boot after installation, which I find undesirable in my opinion. I'd rather someone end up getting the 'incorrect' but working graphics driver that they can fix themselves after the installation than them be met with a broken install that may leave a new user confused with a system they can't use.

@sigreturn
Copy link
Contributor Author

Bumping this as I'd like to hopefully get this merged in time for the February .iso release since it didn't make it in time for the January one.

@Torxed
Copy link
Member

Torxed commented Jan 15, 2026

I think the issue of #4053 (comment) is valid tho and needs to be addressed.

@sigreturn
Copy link
Contributor Author

sigreturn commented Jan 16, 2026

I'm open to suggestions because I'm just not too sure what would be the best way to handle such a thing. I really dislike the idea of installing nvidia-open if they have the legacy nvidia package selected in their config because if we go that route and the user has pre-Turing hardware they will be left with an unusable system after installation since the new driver release does not support hardware that is pre-Turing like I addressed in the previous comments.

The way this pull request technically handles things could be seen as "misleading" since we're installing the open-source Mesa stack instead for people that have the legacy nvidia package selected in their configs, but at least this way they'd have a functioning system that they could at least get to the desktop with.

Another option is to display a warning message when they run the installer with a config that has the old driver selected that warns them their config is out of date and will that it will install a graphics driver that may not be compatible with their hardware, and if they ignore the warning, I guess that falls on them?

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.

2 participants