ctld: Return from the inner "main" loop on EINTR
ClosedPublic
Actions

Authored by jhb on Mon, Apr 20, 2:26 PM.

Tags

None

Referenced Files

	F154558883: D56525.diff
	Tue, Apr 28, 10:12 PM

	Unknown Object (File)
	Mon, Apr 27, 9:28 AM

	Unknown Object (File)
	Mon, Apr 27, 9:27 AM

	Unknown Object (File)
	Sun, Apr 26, 7:57 AM

	Unknown Object (File)
	Sat, Apr 25, 10:47 AM

	Unknown Object (File)
	Sat, Apr 25, 6:32 AM

	Unknown Object (File)
	Fri, Apr 24, 10:47 PM

	Unknown Object (File)
	Thu, Apr 23, 7:55 PM

View All 10 Files

Subscribers

imp

Details

Reviewers

asomers

Commits

rGe56687bb2d32: ctld: Return from the inner "main" loop on EINTR

Summary

This ensures that child processes are reaped in the outer loop in
main().

PR: 294035
Fixes: 66b107e82b2f ("ctld: Use kevent(2) for socket events rather than select(2)")
Sponsored by: Chelsio Communications

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 72366
Build 69249: arc lint + arc unit

Event Timeline

jhb created this revision.Mon, Apr 20, 2:26 PM

Herald added a subscriber: imp. · View Herald TranscriptMon, Apr 20, 2:26 PM

Harbormaster completed remote builds in B72366: Diff 175901.Mon, Apr 20, 2:26 PM

jhb requested review of this revision.Mon, Apr 20, 2:26 PM

jhb added a child revision: D56526: ctld: Update nchildren directly in wait_for_children.

asomers accepted this revision.Mon, Apr 20, 4:22 PM

This revision is now accepted and ready to land.Mon, Apr 20, 4:22 PM

Closed by commit rGe56687bb2d32: ctld: Return from the inner "main" loop on EINTR (authored by Nikolay Denev <ndenev@gmail.com>, committed by jhb). · Explain WhyMon, Apr 20, 5:20 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rGe56687bb2d32: ctld: Return from the inner "main" loop on EINTR.

Revision Contents
Changeset List

Path

Size

usr.sbin/

ctld/

ctld.cc

2 lines

Diff 175901

View Options

ctld: Return from the inner "main" loop on EINTRClosedPublicActions