Page MenuHomeFreeBSD

Make struct syscall_args machine independent
ClosedPublic

Authored by brooks on Dec 6 2021, 10:54 PM.
Tags
None
Referenced Files
F110458100: D33308.diff
Tue, Feb 18, 5:44 PM
Unknown Object (File)
Sun, Feb 9, 6:06 PM
Unknown Object (File)
Sun, Feb 9, 5:52 AM
Unknown Object (File)
Fri, Feb 7, 9:34 AM
Unknown Object (File)
Fri, Feb 7, 5:34 AM
Unknown Object (File)
Sat, Jan 25, 3:17 AM
Unknown Object (File)
Fri, Jan 24, 3:07 AM
Unknown Object (File)
Jan 10 2025, 5:49 AM

Details

Summary

After a round of cleanups in late 2020, all definitions are
functionally identical.

This removes a rotted __aligned(8) on arm. It was added in
b7112ead32bc50ef9744099bdbb1cfbd6e906b2a and was intended to align the
args member so that 64-bit types (off_t, etc) could be safely read on
armeb compiled with clang. With the removal of armev, this is no
longer needed (armv7 requires that 32-bit aligned reads of 64-bit
values be supported and we enable such support on armv6). As further
evidence this is unnecessary, cleanups to struct syscall_args have
resulted in args being 32-bit aligned on 32-bit systems. The sole
effect is to bloat the struct by 4 bytes.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib added inline comments.
sys/sys/proc.h
204

I believe MAXARGS definition should be removed in this change, or in immediate follow-up. It seems there are only two uses of the symbol in arm64 MD code, everything else uses nitems(sa->args).

This revision is now accepted and ready to land.Dec 6 2021, 11:00 PM
brooks added inline comments.
sys/sys/proc.h
204

While I'm not generally a fan of magic numbers this makes sense. I'll work that up.