Page MenuHomeFreeBSD
Differential D31481

linux(4): Implement CLONE_CLEAR_SIGHAND option bit.
ClosedPublic
Actions

Authored by dchagin on Aug 9 2021, 7:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 7, 8:26 AM
Unknown Object (File)
Sat, May 3, 10:45 AM
Unknown Object (File)
Thu, Apr 24, 1:19 PM
Unknown Object (File)
Thu, Apr 24, 6:48 AM
Unknown Object (File)
Apr 11 2025, 2:50 PM
Unknown Object (File)
Apr 6 2025, 2:34 AM
Unknown Object (File)
Mar 13 2025, 3:12 PM
Unknown Object (File)
Mar 12 2025, 4:19 PM
View All 92 Files
Subscribers
imp

Details

Reviewers
kib
Group Reviewers
Linux Emulation
Commits
rG3af6bc9d02cd: linux(4): Implement CLONE_CLEAR_SIGHAND option bit.
rGc5fc9fe7f36a: linux(4): Implement CLONE_CLEAR_SIGHAND option bit.

Diff Detail

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

Event Timeline

dchagin created this revision.Aug 9 2021, 7:53 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 9 2021, 7:53 PM
dchagin requested review of this revision.Aug 9 2021, 7:53 PM
Harbormaster completed remote builds in B40951: Diff 93462.Aug 9 2021, 7:53 PM
dchagin added reviewers: kib, Linux Emulation.Aug 9 2021, 9:53 PM
dchagin changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.
kib added a comment.Aug 10 2021, 12:28 AM

Can you explain the semantic of CLEAR_SIGHAND, or provide a reference to the official doc?

sys/compat/linux/linux_fork.c
142

This leaves f2 uninitialize if CLEAR_SIGHAND is not specified, no?

dchagin updated this revision to Diff 93476.Aug 10 2021, 8:36 AM

fixed, CLONE_CLEAR_SIGHAND is designed to reset all signal handlers of the child
not set to SIG_IGN to SIG_DFL.

Harbormaster completed remote builds in B40958: Diff 93476.Aug 10 2021, 8:36 AM
kib added inline comments.Aug 10 2021, 9:18 AM
sys/compat/linux/linux_fork.c
155

Please do '|=' instead of '='.

dchagin updated this revision to Diff 93477.Aug 10 2021, 9:23 AM

fixed

Harbormaster completed remote builds in B40959: Diff 93477.Aug 10 2021, 9:23 AM
kib accepted this revision.Aug 10 2021, 9:44 AM
kib added inline comments.
sys/compat/linux/linux_fork.c
142

I would also moved the initialization of ff there, from the declaration block.

This revision is now accepted and ready to land.Aug 10 2021, 9:44 AM
Closed by commit rGc5fc9fe7f36a: linux(4): Implement CLONE_CLEAR_SIGHAND option bit. (authored by dchagin). · Explain WhyAug 12 2021, 8:57 AM
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rGc5fc9fe7f36a: linux(4): Implement CLONE_CLEAR_SIGHAND option bit..
dchagin added a commit: rG3af6bc9d02cd: linux(4): Implement CLONE_CLEAR_SIGHAND option bit..Jun 17 2022, 7:39 PM

Revision Contents
Changeset List

PathSize
sys/
compat/
linux/
2 lines

Diff 93606

View Options

sys/compat/linux/linux_fork.c

Loading...