User Details
- User Since
- May 18 2014, 4:14 AM (489 w, 18 h)
Thu, Sep 7
Please address the comments from markj@ but otherwise this looks good, and more extensive than what we came up with on CADETS.
Tue, Sep 5
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
Nov 25 2020
Clean up various bits for hopeful commit.
Updates based on conversations with cperciva and emaste
Address issues pointed out by philip@
Nov 24 2020
Nov 17 2020
Nov 13 2020
Nov 12 2020
Committed revision 367628.
Address melifaro@ request to move code into a function.
Nov 11 2020
Address the comment about the comment.