Differential D34216

dtrace: remove unnecessary fflush()
ClosedPublic
Actions

Authored by chs on Feb 8 2022, 11:40 PM.

Tags

None

Referenced Files

	F153485738: D34216.id102529.diff
	Tue, Apr 21, 10:32 AM

	Unknown Object (File)
	Thu, Apr 16, 3:49 PM

	Unknown Object (File)
	Sun, Apr 12, 1:12 AM

	Unknown Object (File)
	Thu, Apr 9, 12:03 PM

	Unknown Object (File)
	Wed, Mar 25, 4:41 AM

	Unknown Object (File)
	Mon, Mar 23, 11:03 PM

	Unknown Object (File)
	Mar 17 2026, 3:18 AM

	Unknown Object (File)
	Mar 16 2026, 11:57 PM

Subscribers

Details

Reviewers

markj
imp
gnn

Commits

rGf339a3ef6369: dtrace: remove unnecessary fflush()

Summary

This call was added back in the early days of dtrace porting and
no one knows why anymore. The extra flushing causes lots of
unnecessary CPU overhead when a script produces lots of output,
as well as easily losing output because the command can't keep up.

Sponsored by: Netflix

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 44354
Build 41242: arc lint + arc unit

Event Timeline

chs created this revision.Feb 8 2022, 11:40 PM

Herald added a subscriber: imp. · View Herald TranscriptFeb 8 2022, 11:40 PM

chs requested review of this revision.Feb 8 2022, 11:40 PM

Harbormaster completed remote builds in B44354: Diff 102529.Feb 8 2022, 11:40 PM

chs added a reviewer: markj.Feb 8 2022, 11:41 PM

given all the early returns that *DONT* do this, I have trouble seeing how it would be needed.

imp accepted this revision.Feb 8 2022, 11:51 PM

This revision is now accepted and ready to land.Feb 8 2022, 11:51 PM

markj accepted this revision.Feb 9 2022, 11:53 PM

Closed by commit rGf339a3ef6369: dtrace: remove unnecessary fflush() (authored by chs). · Explain WhyFeb 10 2022, 2:49 AM

This revision was automatically updated to reflect the committed changes.

chs added a commit: rGf339a3ef6369: dtrace: remove unnecessary fflush().

Herald added a reviewer: gnn. · View Herald TranscriptFeb 10 2022, 2:49 AM

Revision Contents
Changeset List

Path

Size

cddl/

contrib/

opensolaris/

lib/

libdtrace/

common/

1 line

Diff 102529

cddl/contrib/opensolaris/lib/libdtrace/common/dt_subr.c

Loading...