Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/external/cddl/osnet/dist/lib/libdtrace/common dtrace: remove...



details:   https://anonhg.NetBSD.org/src/rev/f79b04892aca
branches:  trunk
changeset: 373414:f79b04892aca
user:      chs <chs%NetBSD.org@localhost>
date:      Tue Feb 07 22:54:02 2023 +0000

description:
dtrace: remove unnecessary fflush()

Apply this commit from FreeBSD:

  commit f339a3ef6369b368f3a2455792a7a3a4c28f92c4
  Author: Chuck Silvers <chs%FreeBSD.org@localhost>
  Date:   Wed Feb 9 17:09:26 2022 -0800

      dtrace: remove unnecessary fflush()

      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
      Reviewed by:      imp, markj
      Differential Revision:    https://reviews.freebsd.org/D34216

diffstat:

 external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c |  1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diffs (11 lines):

diff -r 60f489de414b -r f79b04892aca external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c
--- a/external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c   Tue Feb 07 22:26:46 2023 +0000
+++ b/external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c   Tue Feb 07 22:54:02 2023 +0000
@@ -725,7 +725,6 @@
 
        va_copy(ap2, ap);
        n = vfprintf(fp, format, ap2);
-       fflush(fp);
        va_end(ap2);
        va_end(ap);
 



Home | Main Index | Thread Index | Old Index