Page MenuHomeFreeBSD

ia32: Sync signal context type names with i386.
ClosedPublic

Authored by jhb on Jan 13 2022, 9:08 PM.
Tags
None
Referenced Files
F77944727: D33882.diff
Sun, Feb 25, 10:57 AM
Unknown Object (File)
Mon, Feb 12, 3:47 AM
Unknown Object (File)
Jan 12 2024, 7:55 AM
Unknown Object (File)
Dec 23 2023, 12:54 AM
Unknown Object (File)
Nov 12 2023, 7:42 PM
Unknown Object (File)
Oct 28 2023, 9:37 PM
Unknown Object (File)
Aug 5 2023, 1:37 AM
Unknown Object (File)
Aug 5 2023, 1:35 AM
Subscribers

Details

Summary
  • Use ia32_freebsd4_* instead of ia32_*4.
  • Use ia32_o* instead of ia32_*3.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 43909
Build 40797: arc lint + arc unit

Event Timeline

jhb requested review of this revision.Jan 13 2022, 9:08 PM
kib added inline comments.
sys/compat/ia32/ia32_signal.h
91

I suspect these ifdefs in the header can be removed.

This revision is now accepted and ready to land.Jan 13 2022, 9:29 PM
imp added inline comments.
sys/amd64/ia32/ia32_signal.c
366

Why osigframe? Is there a reason for that naming convetion?

I think this is right.

@imp COMPAT_43 syscalls get named o<syscall> so this follows that convention. The "3" suffix is weird.

jhb added inline comments.
sys/amd64/ia32/ia32_signal.c
366

Because the rest of things for COMPAT_43 use the 'o' prefix including syscalls, etc. These aren't FreeBSD 3.x things, but BSD43, so the 3 suffix is weird.

This revision was automatically updated to reflect the committed changes.