User Details
- User Since
- Jun 6 2021, 5:11 PM (271 w, 3 d)
Yesterday
This looks reasonable. Please see comments for some small items to check.
Tue, Aug 18
This looks reasonable, but let's wait for some more feedback from the others.
Mon, Aug 17
I like it. Please land the new header and kernel fixes separately in case reversal is needed.
Sun, Aug 16
Looks like a good patch. Could you add unit tests that make sure the constraint handles actually trigger on error?
Please mention in the commit message that this also documents bsearch_b.
Sat, Aug 15
Fri, Aug 14
Approved for commit.
Do you wish to have these changes merged into stable/15? If yes, consider adding an MFC after tag. I recommend a wait of 1 month for new features, 1 week for bug fixes.
Same goes for your other C23 changes. You can MFC even if you did not add the relevant tag, but you'll only get a reminder if the tag is present.
If you wish to add "MFC after", just do it on commit. You do not need to ask for approval again.
Please add the desired commit messages for reference.
Mon, Aug 10
Sun, Aug 9
Nice work. Some feedback is attached.
For example, the build-option survey is not trying to solve an equation but test for combinations of build-options that break the build when combined together.
Sat, Aug 8
I'm relieved, I never managed to find time to get one.
Fri, Aug 7
Please post the commit message you plan to use so I can then give approval for commit.
Thu, Aug 6
It sounds like this could perhaps be simplified by having a pdsyscall system call that executes a system call on the given process descriptor. Then all manipulations on the zygote process just become normal system calls carried out through pdsyscall and activating a process image becomes a call to execve() or similar through pdsyscall. In particular this avoids having to add new stuff every time we come up with something new that we want to set up before sending off the new process.