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
F135355907: D31481.diff
Sun, Nov 9, 1:49 AM
Unknown Object (File)
Mon, Nov 3, 9:29 AM
Unknown Object (File)
Mon, Nov 3, 6:57 AM
Unknown Object (File)
Sun, Nov 2, 6:11 PM
Unknown Object (File)
Sat, Nov 1, 10:04 AM
Unknown Object (File)
Thu, Oct 30, 1:58 AM
Unknown Object (File)
Wed, Oct 29, 6:01 PM
Unknown Object (File)
Tue, Oct 28, 7:22 AM
View All 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...