Page MenuHomeFreeBSD
Differential D31552

linux(4): Consolidate wait* facility into linux_common_wait().
ClosedPublic
Actions

Authored by dchagin on Aug 16 2021, 12:07 PM.
Tags
None
Referenced Files
F171413874: D31552.diff
Fri, Sep 11, 12:14 AM
F171320311: D31552.id.diff
Thu, Sep 10, 9:39 AM
F171319167: D31552.id104235.diff
Thu, Sep 10, 9:26 AM
Unknown Object (File)
Tue, Sep 8, 6:35 PM
Unknown Object (File)
Sun, Sep 6, 5:08 AM
Unknown Object (File)
Sat, Sep 5, 8:59 PM
Unknown Object (File)
Sat, Sep 5, 3:43 PM
Unknown Object (File)
Sat, Sep 5, 11:08 AM
View All Files
Subscribers
andrew
emaste
imp
kib
pauamma_chezdork.com

Details

Reviewers
manu
brooks
Group Reviewers
Linux Emulation
manpages
Commits
rG721dcd87abe3: linux(4): Consolidate wait* facility into linux_common_wait().
rG0c6b1ff7de56: linux(4): Consolidate wait* facility into linux_common_wait().
Summary

Also this fixes bug in waitid() implementation, use wru_self not wru_children.

Diff Detail

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

Event Timeline

dchagin created this revision.Aug 16 2021, 12:07 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 16 2021, 12:07 PM
Harbormaster completed remote builds in B41057: Diff 93742.Aug 16 2021, 12:07 PM
dchagin requested review of this revision.Aug 16 2021, 12:07 PM
dchagin added a reviewer: Linux Emulation.Aug 16 2021, 12:17 PM
dchagin changed the repository for this revision from rS FreeBSD src repository - subversion to rG FreeBSD src repository.
dchagin updated this revision to Diff 93777.Aug 16 2021, 7:01 PM

style(9)

Harbormaster completed remote builds in B41079: Diff 93777.Aug 16 2021, 7:01 PM
dchagin added a child revision: D31561: linux(4): Fixup wait4 handling pid equal 0 case..Aug 23 2021, 4:23 PM
dchagin mentioned this in D31561: linux(4): Fixup wait4 handling pid equal 0 case..
kib added a subscriber: kib.Sep 5 2021, 12:31 AM
kib added inline comments.
sys/compat/linux/linux_misc.c
1001

What is the point of mallocing them? The structures are not that large to not fit into stack for top-level.

1024

Same question about mallocing this one.

dchagin updated this revision to Diff 104235.Mar 26 2022, 5:26 PM

indeed, back to stack allocation of local vars.

Harbormaster completed remote builds in B44897: Diff 104235.Mar 26 2022, 5:26 PM
dchagin updated this revision to Diff 104343.Mar 29 2022, 8:47 PM

fix, add D31561 code

Herald added a reviewer: manu. · View Herald TranscriptMar 29 2022, 8:47 PM
Herald added a reviewer: brooks. · View Herald Transcript
Herald added a reviewer: manpages. · View Herald Transcript
Herald added subscribers: emaste, andrew. · View Herald Transcript
Harbormaster completed remote builds in B44921: Diff 104343.Mar 29 2022, 8:47 PM
dchagin updated this revision to Diff 104344.Mar 29 2022, 8:48 PM

whoops, ugly arc:(

Harbormaster completed remote builds in B44922: Diff 104344.Mar 29 2022, 8:48 PM
pauamma_chezdork.com added a subscriber: pauamma_chezdork.com.Mar 29 2022, 11:15 PM

I don't see any manual pages in this.

This revision was not accepted when it landed; it landed in state Needs Review.Mar 31 2022, 5:50 PM
Closed by commit rG0c6b1ff7de56: linux(4): Consolidate wait* facility into linux_common_wait(). (authored by dchagin). · Explain Why
This revision was automatically updated to reflect the committed changes.
dchagin added a commit: rG0c6b1ff7de56: linux(4): Consolidate wait* facility into linux_common_wait()..
dchagin added a commit: rG721dcd87abe3: linux(4): Consolidate wait* facility into linux_common_wait()..Jun 17 2022, 7:41 PM

Revision Contents
Changeset List

PathSize
sys/
compat/
linux/
107 lines

Diff 104344

View Options

sys/compat/linux/linux_misc.c

Loading...