Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

makesyscall.lua cleanups #2226

Open
wants to merge 13 commits into
base: dev
Choose a base branch
from
Open

makesyscall.lua cleanups #2226

wants to merge 13 commits into from

Conversation

brooksdavis
Copy link
Member

Assorted cleanups in preparation for an upstream refactoring of this code. A mix of changes to converge the overall output and cleanups of local remnants.

brooksdavis and others added 13 commits October 3, 2024 23:20
sysc is local by virtue of being an iterator
Nothing uses them now so reduce diff to freebsd.
We don't need sysargmask for non-CHERI architectures.
Rename variable to match default value.
Fixes:		3c6211e Add cheri_cidcap_alloc(2)
Comment aligning was inconsistent and required a ton of book-keeping.
Replaced comment aligning with a simple, single tab out.

Pull Request:	freebsd/freebsd-src#1441
Signed-off-by: agge3 <sterspark@gmail.com>
(cherry picked from commit e668866c82f487fe36c4e732e3ce18ae9c44260f)
An upcoming refactor appends do-not-merge comments to all headers
centrally to do the same to reduce the final diff.  Headers also start
with a comment line (for /*) and end with a blank line.

(cherry picked from commit 0d490c6a445a5e94e5d50b65aa96ab72eeafe15f)
Add blank like after comment to align with upcoming refactor for
makesysent.lua.

Fixes:		0d490c6a445a sysent: make header comments more consistent

(cherry picked from commit 0f1d5bfb7297aca850a8c499a7b7fd0fd440cd50)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants