Page MenuHomeFreeBSD

x11/nvidia-driver: Remove devclass from DRIVER_MODULE on recent main.
ClosedPublic

Authored by jhb on May 20 2022, 12:18 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jun 29, 2:06 AM
Unknown Object (File)
Thu, Jun 27, 1:16 PM
Unknown Object (File)
Jun 24 2024, 2:24 PM
Unknown Object (File)
Jun 9 2024, 9:00 AM
Unknown Object (File)
May 27 2024, 4:55 AM
Unknown Object (File)
May 27 2024, 4:48 AM
Unknown Object (File)
May 25 2024, 5:37 AM
Unknown Object (File)
May 25 2024, 5:21 AM
Subscribers

Details

Summary

Initialize nvidia_devclass in nvidia_modevent instead.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb requested review of this revision.May 20 2022, 12:18 AM

Looks mostly fine. The patch header is not quite by our standards (only *.orig file timestamped, UTC-relative, seconds-precision) but that's not a blocker.

x11/nvidia-driver/files/extra-patch-src-nvidia_subr.c.in
2

You may want to regenerate the patch header with make makepatch (or by hand) so it stays consistent with other patches.

This revision is now accepted and ready to land.Jun 24 2022, 6:59 AM
x11/nvidia-driver/files/extra-patch-src-nvidia_subr.c.in
2

make makepatch is not really practical for the nvidia port since it patches these files directly. This the output of diff -up. The porter's handbook simply says to use diff -u and doesn't say anything about trying to sanitizer or otherwise change patch headers?

x11/nvidia-driver/files/extra-patch-src-nvidia_subr.c.in
2

Hmm, I did manage to get something out of make makepatch I could use (by copying the header over from a patch it generated and massaging it to deal with %%NVSRC%%)

@jhb wrote:

make makepatch is not really practical for the nvidia port since it patches these files directly.

Yeah, I know, this is a bit of an annoyance, hence I did not insist. :-)

The porter's handbook simply says to use diff -u and doesn't say anything about trying to sanitizer or otherwise change patch headers?

Well, we try to follow the common standard, but it is not strictly enforced (albeit it probably should've mentioned in the PHB).

Hmm, I did manage to get something out of make makepatch I could use (by copying the header over from a patch it generated and massaging it to deal with %%NVSRC%%)

Right, that what I was thinking (I often do the same with non-trivial patches), thanks!