Source-Changes-HG archive

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

[src/netbsd-10]: src/external/cddl/osnet/dist/lib/libdtrace/common Pull up fo...



details:   https://anonhg.NetBSD.org/src/rev/3aba1b553ed3
branches:  netbsd-10
changeset: 373424:3aba1b553ed3
user:      martin <martin%NetBSD.org@localhost>
date:      Wed Feb 08 17:25:56 2023 +0000

description:
Pull up following revision(s) (requested by chs in ticket #74):

        external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c: revision 1.16

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 263b51704c63 -r 3aba1b553ed3 external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c
--- a/external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c   Wed Feb 08 17:18:15 2023 +0000
+++ b/external/cddl/osnet/dist/lib/libdtrace/common/dt_subr.c   Wed Feb 08 17:25:56 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