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

Nvidia fixes #114

Closed
wants to merge 1 commit into from
Closed

Nvidia fixes #114

wants to merge 1 commit into from

Conversation

shkhln
Copy link

@shkhln shkhln commented Feb 19, 2023

  1. 304, 340, 390 are out of support: https://nvidia.custhelp.com/app/answers/detail/a_id/3142/~/support-timeframes-for-unix-legacy-gpu-releases. In addition to that 304 and 340 are also completely broken.
  2. The nvidia-modeset advice is simply wrong.

1. 304, 340, 390 are out of support: https://nvidia.custhelp.com/app/answers/detail/a_id/3142/~/support-timeframes-for-unix-legacy-gpu-releases. In addition to that 304 and 340 are also completely broken.
2. The nvidia-modeset advice is simply wrong.
@grahamperrin
Copy link
Contributor

@shkhln
Copy link
Author

shkhln commented Feb 25, 2023

Is there a reasonable time frame for the "due course"? As you are well aware, I publicly promised to drop my ports if this is not merged and we need some date for that.

@shkhln
Copy link
Author

shkhln commented Feb 25, 2023

drop my ports

Just in case somebody thinks this is too harsh: we are already ignoring the intention behind the nvidia-driver pkg message, actual people's experience and even the confirmation from one of the Nvidia Unix driver devs.

@sergio-carlavilla
Copy link
Contributor

I asked about this to the current nvidia maintainer in FreeBSD.
I'll make the commit when I have an answer from him.
Thanks for the patch and sorry for the delay

@grahamperrin
Copy link
Contributor

Is there a reasonable time frame for the "due course"?

There's the interim change that was made around two minutes after #114 (comment). I intended to await a commit here before making a more substantial change.

@danfe
Copy link

danfe commented Mar 1, 2023

In addition to that 304 and 340 are also completely broken.

That's somewhat an overstatement. While nVidia had apparently neglected an earlier request to catch up the -304 branch with xserver ABI 24, there are fresh reports of success with -340 version by ignoring the ABI:

Section "ServerFlags"
Option "IgnoreABI" "1"
EndSection

we are already ignoring the intention behind the nvidia-driver pkg message

We're not ignoring anything. If you read it carefully, you'd notice that commit you reference largely quotes the pkg-message text.

The nvidia-modeset advice is simply wrong.

This is true when it is advised out of the context, I presume that's the problem with the Handbook? The nvidia-modeset kernel module was introduced in driver version 358.09 to program the display engine of the GPU (changing resolutions, transformation settings, display device hot-plugging, etc.) and works in conjunction with the nvidia.ko kernel module. At the time of the writing, only the latest version of the driver shipped this module, most users would still go the kld_list="nvidia" way. We simply forgot to flip the switch as the time had passed. I'll update the port's pkg-message accordingly.

@shkhln
Copy link
Author

shkhln commented Mar 1, 2023

we are already ignoring the intention behind the nvidia-driver pkg message

We're not ignoring anything. If you read it carefully, you'd notice that commit you reference largely quotes the pkg-message text.

Not you, those two guys. I didn't want to make this personal, thus "we".

The nvidia-modeset advice is simply wrong.

This is true when it is advised out of the context, I presume that's the problem with the Handbook?

No, the problem with the Handbook is that now it downright tells users to avoid nvidia-modeset. There is no nuance there at all.

@sergio-carlavilla
Copy link
Contributor

I'll upgrade the handbook information with the commit https://cgit.freebsd.org/ports/commit/?id=a09185f8b66c
Thanks @danfe :)

@sergio-carlavilla
Copy link
Contributor

Fixed in https://cgit.freebsd.org/doc/commit/?id=9af61238

@grahamperrin
Copy link
Contributor

grahamperrin commented Mar 11, 2023

Likewise, thanks :-)

I'll update the wiki in due course.

Done.

For updates to the FreeBSD Handbook, I might put something in Phabricator this weekend. #122

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants