Page MenuHomeFreeBSD

libc: don't needlessly add vfork.o to NOASM
ClosedPublic

Authored by brooks on Dec 5 2023, 7:09 PM.
Tags
None
Referenced Files
F109527330: D42914.diff
Thu, Feb 6, 6:44 AM
Unknown Object (File)
Fri, Jan 24, 2:01 PM
Unknown Object (File)
Nov 25 2024, 7:51 AM
Unknown Object (File)
Oct 6 2024, 12:46 AM
Unknown Object (File)
Oct 5 2024, 3:25 AM
Unknown Object (File)
Oct 3 2024, 10:43 PM
Unknown Object (File)
Oct 3 2024, 11:56 AM
Unknown Object (File)
Oct 1 2024, 1:15 PM
Subscribers

Details

Summary

For architectures where vfork.S was named Ovfork.S this was needed, but
it was always pointless here as an entry in either MDASM or NOASM is
equivalent.

Sponsored by: DARPA

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 54801
Build 51690: arc lint + arc unit

Event Timeline

brooks requested review of this revision.Dec 5 2023, 7:09 PM

Are you sure about amd64? There is the arch-specific manually written vfork.S.
Does the patch rely on vfork.S in MD directory to override vfork.s automatically generated for the syscall?

In D42914#978872, @kib wrote:

Are you sure about amd64? There is the arch-specific manually written vfork.S.
Does the patch rely on vfork.S in MD directory to override vfork.s automatically generated for the syscall?

Yes, I'm sure. The presence of vfork.S in MDASM prevents the default stub from being generated in the same loop that handles NOASM. I'm pretty sure they only reasons these were there is that i386 put the real implementation in Ovfork.S and then porters copied it around.

This revision is now accepted and ready to land.Dec 6 2023, 12:59 AM
This revision was automatically updated to reflect the committed changes.