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)
Fri, Mar 29, 12:41 PM
Unknown Object (File)
Fri, Mar 22, 8:09 AM
Unknown Object (File)
Fri, Mar 22, 7:57 AM
Unknown Object (File)
Sun, Mar 17, 7:53 AM
Unknown Object (File)
Mar 11 2024, 9:07 PM
Unknown Object (File)
Jan 14 2024, 9:07 AM
Unknown Object (File)
Dec 3 2023, 10:13 PM
Unknown Object (File)
Nov 16 2023, 9:26 PM
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!