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
F102061819: D35266.diff
Thu, Nov 7, 3:39 AM
Unknown Object (File)
Fri, Oct 11, 11:22 PM
Unknown Object (File)
Oct 6 2024, 11:42 AM
Unknown Object (File)
Oct 5 2024, 8:24 PM
Unknown Object (File)
Oct 3 2024, 7:28 AM
Unknown Object (File)
Oct 3 2024, 1:59 AM
Unknown Object (File)
Sep 28 2024, 4:19 AM
Unknown Object (File)
Sep 27 2024, 5:28 PM
Subscribers

Details

Summary

Initialize nvidia_devclass in nvidia_modevent instead.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 45645
Build 42533: arc lint + arc unit

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
1

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
1

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
1

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!