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
F156985920: D31552.id104343.diff
Sun, May 17, 7:45 PM
Unknown Object (File)
Sat, May 16, 11:22 PM
Unknown Object (File)
Wed, May 13, 9:44 PM
Unknown Object (File)
Wed, May 13, 9:35 PM
Unknown Object (File)
Wed, May 13, 9:35 PM
Unknown Object (File)
Wed, May 13, 9:35 PM
Unknown Object (File)
Wed, May 13, 9:35 PM
Unknown Object (File)
Wed, May 13, 9:35 PM
View All Files
Subscribers
andrew
emaste
imp
kib
pauamma_gundo.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 44897
Build 41785: 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
1003

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_gundo.com added a subscriber: pauamma_gundo.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/
100 lines

Diff 104235

View Options

sys/compat/linux/linux_misc.c

Loading...