User Details
- User Since
- May 18 2014, 4:14 AM (545 w, 3 d)
Jul 29 2024
May 1 2024
Apr 20 2024
LGTM
Mar 21 2024
LGTM. Do it!
Nov 14 2023
commit 029848334f85e03895234db8b89f5544ffb6e720 (HEAD -> main, freebsd/main, fr>
Author: George V. Neville-Neil <gnn@FreeBSD.org>
Date: Tue Nov 14 16:09:18 2023 -0500
@jhb Just sent a private email asking for more of these and I'll organize and commit, thanks!
Nov 12 2023
I have to say I prefer the version with the module_data, even though it is admittedly longer, because it shows the reader how to explicitly add module data. A much simpler verison with SYSINIT and SYSUNINIT doesn't easily show someone how to not only make a module but how to create their private module data. I could create a version with both using a #define #if setup, so we can show both styles. Let me know is that would be preferred or if what we have here is acceptable.
- Update the minor issues found by jhb@, other discussion continues.
Oct 5 2023
Sep 7 2023
Please address the comments from markj@ but otherwise this looks good, and more extensive than what we came up with on CADETS.
Sep 5 2023
May 5 2023
Apr 10 2023
Approved
Apr 5 2023
Mar 28 2023
Mar 25 2023
Mar 24 2023
Approved.
Feb 24 2023
Feb 18 2023
Feb 13 2023
Dec 21 2022
Dec 14 2022
May 31 2022
LGTM
Mar 28 2022
Mar 21 2022
Mar 20 2022
Mar 17 2022
For some reason phk@ can't currently log in, so he emailed me his comments. I paste them here, verbatim:
Mar 16 2022
Is it possible to find out how this was tested?
Jan 27 2022
Jan 22 2022
LGTM and Thanks!
Nov 16 2021
- Fix two more nits from 0mp
Nov 15 2021
- Address review comments from 0mp, debdrup and oshogbo
Nov 12 2021
Sep 5 2021
Aug 26 2021
Pretty much anything marked as KAME compatibility can be removed as KAME ended long ago.
Jun 21 2021
Jun 17 2021
May 28 2021
Apr 21 2021
Apr 20 2021
Apr 16 2021
Apr 9 2021
Apr 7 2021
Now that this review has devolved to the point of discussing democracy, rather than the deprecation of a piece of long surviving, but, inherently insecure code in our base system, it's time for people to step back from their keyboards and realize that removing this from base does not make it impossible to install or use, it simply reduces the default attack surface of our system. So, let's let emaste put this deprecation in for 14, and move on.
Apr 2 2021
Mar 18 2021
Mar 11 2021
LGTM
Mar 10 2021
Feb 19 2021
Couple of nits but otherwise LGTM.
Feb 12 2021
Feb 8 2021
Jan 22 2021
Jan 19 2021
Yup, I understand that we do not currently have that function. I think you ought to add that as part of this fix.
I believe the use of this boolean is the wrong approach. If you want to check and then lock you'll need to an an INP_RLOCKED similar to INP_WLOCKED which can be found in in_pcb.h.
Jan 13 2021
Jan 5 2021
Dec 13 2020
Dec 7 2020
Update with suggestions from emaste