[DTrace-devel] [oracle/dtrace-utils] 595d1f: Remove unused dpp_pc and dpp_size

Kris Van Hees noreply at github.com
Fri Feb 21 04:14:27 UTC 2025


  Branch: refs/heads/devel
  Home:   https://github.com/oracle/dtrace-utils
  Commit: 595d1fe2ddd93b7bb851e54957f8912f26e98d12
      https://github.com/oracle/dtrace-utils/commit/595d1fe2ddd93b7bb851e54957f8912f26e98d12
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    M libdtrace/dt_pid.c

  Log Message:
  -----------
  Remove unused dpp_pc and dpp_size

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: a97ded8aaafdd4cb6b434ff379de88482c211b91
      https://github.com/oracle/dtrace-utils/commit/a97ded8aaafdd4cb6b434ff379de88482c211b91
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    M libdtrace/dt_pid.c

  Log Message:
  -----------
  Simplify references to dtp

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: 788b3a3c0f4e8c167c79d05e3dae7d25b378b0cc
      https://github.com/oracle/dtrace-utils/commit/788b3a3c0f4e8c167c79d05e3dae7d25b378b0cc
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    M libdtrace/dt_pid.c

  Log Message:
  -----------
  Remove unused function arg

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: 22fb7502459d097acda12c0717f49f52d3ccdb02
      https://github.com/oracle/dtrace-utils/commit/22fb7502459d097acda12c0717f49f52d3ccdb02
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    M test/triggers/usdt-tst-multiprovider-prov.d

  Log Message:
  -----------
  test: Remove unneeded skip for trigger

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: e49316248a94216d6fd06eaa0dc8f6769d40c4e7
      https://github.com/oracle/dtrace-utils/commit/e49316248a94216d6fd06eaa0dc8f6769d40c4e7
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    M test/triggers/Build
    A test/triggers/usdt-tst-multiprov-dupprobe-prov.d
    A test/triggers/usdt-tst-multiprov-dupprobe.c
    A test/unittest/usdt/tst.multiprov-dupprobe.r
    A test/unittest/usdt/tst.multiprov-dupprobe.r.p
    A test/unittest/usdt/tst.multiprov-dupprobe.sh
    M test/unittest/usdt/tst.multiprovider.r
    M test/unittest/usdt/tst.multiprovider.r.p
    M test/unittest/usdt/tst.multiprovider.sh

  Log Message:
  -----------
  test: Embellish the USDT multiprovider test

Add a probe to two providers, each probe with the same name but
different arguments.

Do so in a new, separate test since the older test used to trigger
some crashes, and we would like to continue to catch such problems.

Add -v to these probe-list tests to check attributes and arguments.

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: 03381a3f7b64d26a5249cd589197a50a1ae43a2c
      https://github.com/oracle/dtrace-utils/commit/03381a3f7b64d26a5249cd589197a50a1ae43a2c
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    A test/unittest/usdt/tst.multiprov-dupprobe-fire.r
    A test/unittest/usdt/tst.multiprov-dupprobe-fire.r.p
    A test/unittest/usdt/tst.multiprov-dupprobe-fire.sh
    A test/unittest/usdt/tst.multiprovider-fire.r
    A test/unittest/usdt/tst.multiprovider-fire.r.p
    A test/unittest/usdt/tst.multiprovider-fire.sh

  Log Message:
  -----------
  test: Add USDT multiprovider tests that fire probes

The multiprovider tests simply list probes.  Add new tests that
fire probes.

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: 71317c48c479dd6be1fc7814b601aad84d76b59a
      https://github.com/oracle/dtrace-utils/commit/71317c48c479dd6be1fc7814b601aad84d76b59a
  Author: Eugene Loh <eugene.loh at oracle.com>
  Date:   2025-01-22 (Wed, 22 Jan 2025)

  Changed paths:
    A test/unittest/usdt/tst.multiprov-dupprobe-shlibs.r
    A test/unittest/usdt/tst.multiprov-dupprobe-shlibs.r.p
    A test/unittest/usdt/tst.multiprov-dupprobe-shlibs.sh

  Log Message:
  -----------
  test: USDT probes with multiple providers and shared libs

A simple executable links in two shared libraries.  There is
a different USDT provider for the executable and for each of
the two libraries.  Each provider has probes named the same
as for the other providers, but the probes differ in arguments.

Run dtrace on the executable.  The D script first dumps the
target pid for later post-processing.  All USDT probes are
traced and their argument values tested.

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
Reviewed-by: Kris Van Hees <kris.van.hees at oracle.com>


  Commit: 8bd26415bc78f1c6f8ccf6e629ad677aeba0e534
      https://github.com/oracle/dtrace-utils/commit/8bd26415bc78f1c6f8ccf6e629ad677aeba0e534
  Author: Kris Van Hees <kris.van.hees at oracle.com>
  Date:   2025-01-31 (Fri, 31 Jan 2025)

  Changed paths:
    M bpf/get_bvar.c
    M libdtrace/dt_cg.c
    M libdtrace/dt_dis.c
    M test/demo/builtin/ipl.d
    M test/demo/builtin/vtimestamp.d
    M test/demo/io/iocpu.d
    M test/unittest/buffering/tst.cputime.sh
    M test/unittest/disasm/tst.ann-bvar.r
    M test/unittest/disasm/tst.ann-bvar.sh
    M test/unittest/disasm/tst.vartab-bvar.r
    M test/unittest/disasm/tst.vartab-bvar.sh

  Log Message:
  -----------
  bpf: separate bvar implementation into separate functions

The handling of builtin variables was done with a single big function
that therefore got linked into every single BPF program (twice if the
builtin variable might raise a fault).  Providing separate functions
for most builtin variables reduces BPF program size significantly.

dtrace -n 'BEGIN { trace(arg0); exit(0); '
  Before this change:
    BEGIN = 1195 BPF instructions
    END = 739 BPF instructions

dtrace -n 'BEGIN { trace(arg0); trace(arg1); exit(0); '
  Before this change:
    BEGIN = 1205 BPF instructions
    END = 748 BPF instructions

dtrace -n 'BEGIN { trace(execname); exit(0); '
  Before this change:
    BEGIN = 1209 BPF instructions
    END = 753 BPF instructions

dtrace -n 'BEGIN { trace(probename); exit(0); '
  Before this change:
    BEGIN = 1209 BPF instructions
    END = 787 BPF instructions

dtrace -n 'BEGIN { trace(ppid); exit(0); '
  Before this change:
    BEGIN = 1195 BPF instructions
    END = 779 BPF instructions

This also reduces pressure on the BPF verifies.

Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
Reviewed-by: Eugene Loh <eugene.loh at oracle.com>


  Commit: c768cb785f8648cf41f9490c1578595e969c06d6
      https://github.com/oracle/dtrace-utils/commit/c768cb785f8648cf41f9490c1578595e969c06d6
  Author: Kris Van Hees <kris.van.hees at oracle.com>
  Date:   2025-01-31 (Fri, 31 Jan 2025)

  Changed paths:
    M include/dtrace/faults_defines.h

  Log Message:
  -----------
  include: remove unnecessary include of universal.h

Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
Reviewed-by: Eugene Loh <eugene.loh at oracle.com>


  Commit: 24fc774afb1f00b2422ee9398d328ca4c6732ad9
      https://github.com/oracle/dtrace-utils/commit/24fc774afb1f00b2422ee9398d328ca4c6732ad9
  Author: Kris Van Hees <kris.van.hees at oracle.com>
  Date:   2025-01-31 (Fri, 31 Jan 2025)

  Changed paths:
    M include/port.h
    M libdtrace/dt_as.c
    M libdtrace/dt_cc.c
    M libdtrace/dt_dis.c

  Log Message:
  -----------
  bpf: implement additional relocation types

The relocation names we were using are not entirely correct in view of
the "official" names.  GCC is also generating code that uses some of
the relocations that were not implemented yet.

Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
Reviewed-by: Eugene Loh <eugene.loh at oracle.com>


  Commit: 3a551bfd7d5875463553619d0fbbe6d3c91f6d8c
      https://github.com/oracle/dtrace-utils/commit/3a551bfd7d5875463553619d0fbbe6d3c91f6d8c
  Author: Kris Van Hees <kris.van.hees at oracle.com>
  Date:   2025-02-07 (Fri, 07 Feb 2025)

  Changed paths:
    M include/dtrace/dif.h
    M libdtrace/dt_cg.c
    M libdtrace/dt_consume.c
    M libdtrace/dt_parser.c
    A test/unittest/actions/trace/tst.array-char-multi-nul.d
    A test/unittest/actions/trace/tst.array-char-multi-nul.r
    A test/unittest/actions/trace/tst.array-char-str-multi-nul.d
    A test/unittest/actions/trace/tst.array-char-str-multi-nul.r
    A test/unittest/actions/trace/tst.array-char-str-no-nul.d
    A test/unittest/actions/trace/tst.array-char-str-no-nul.r
    A test/unittest/actions/trace/tst.array-char-str.d
    A test/unittest/actions/trace/tst.array-char-str.r
    A test/unittest/actions/trace/tst.array-char-unprintable.d
    A test/unittest/actions/trace/tst.array-char-unprintable.r
    M test/unittest/actions/trace/tst.array.d
    M test/unittest/actions/trace/tst.array.r
    R test/unittest/actions/trace/tst.array.r.p

  Log Message:
  -----------
  trace: fix char-array handling

The special handling of strings in the consumer's trace implementation
did not correctly account for char-arrays that do not necessarily are
intended to hold string data.  The trace action is documented to print
data as an ASCII string if it can be represented as such, using the
heuristics implemented in dt_print_bytes().

The implementation for trace on both producer and consumer side was
using the data alignment as a marker to determine whether data should
be printed as a string (alignment == 1) or not.  The real alignment
of the data should be used.

The dtrace_diftype struct has its dtdt_pad member renamed as dtdt_align.

Added several new test cases to cover various possibilities.

Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
Reviewed-by: Eugene Loh <eugene.loh at oracle.com>


  Commit: 40148f2055039f62448ff95e5c23619abac5e4f6
      https://github.com/oracle/dtrace-utils/commit/40148f2055039f62448ff95e5c23619abac5e4f6
  Author: Kris Van Hees <kris.van.hees at oracle.com>
  Date:   2025-02-20 (Thu, 20 Feb 2025)

  Changed paths:
    M bpf/Build
    A bpf/bvar_execargs.S
    M dlibs/aarch64/5.11/procfs.d
    M dlibs/aarch64/5.12/procfs.d
    M dlibs/aarch64/5.14/procfs.d
    M dlibs/aarch64/5.16/procfs.d
    M dlibs/aarch64/5.2/procfs.d
    M dlibs/aarch64/5.6/procfs.d
    M dlibs/aarch64/6.1/procfs.d
    M dlibs/aarch64/6.10/procfs.d
    M dlibs/x86_64/5.11/procfs.d
    M dlibs/x86_64/5.12/procfs.d
    M dlibs/x86_64/5.14/procfs.d
    M dlibs/x86_64/5.16/procfs.d
    M dlibs/x86_64/5.2/procfs.d
    M dlibs/x86_64/5.6/procfs.d
    M dlibs/x86_64/6.1/procfs.d
    M dlibs/x86_64/6.10/procfs.d
    M include/dtrace/dif_defines.h
    M libdtrace/dt_bpf.h
    M libdtrace/dt_cc.c
    M libdtrace/dt_cg.c
    M libdtrace/dt_dlibs.c
    M libdtrace/dt_open.c
    M libdtrace/procfs.d.in
    M test/unittest/builtinvar/tst.psinfo-bug21974606.d
    M test/unittest/builtinvar/tst.psinfo-bug22561297.d
    M test/unittest/builtinvar/tst.psinfo.d
    M test/unittest/builtinvar/tst.psinfo1.d
    A test/unittest/proc/tst.pr_psargs.d
    A test/unittest/proc/tst.pr_psargs.r
    A test/unittest/proc/tst.pr_psargs_other_task.d
    A test/unittest/proc/tst.pr_psargs_other_task.r
    A test/unittest/variables/bvar/tst.execargs.d

  Log Message:
  -----------
  procfs: implement execargs for pr_psargs translator support

Implement a new execargs built-in variable to provide the ps-style
argument string for the current task.  It is used in the psinfo
translator for the pr_psargs member.

Due to BPF limitations, it is only possible to retrieve this information
for the current task.  The data is stored in the task's address space,
and the only BPF helper that can access data in another task's address
space is not allowed to be used in non-sleepable programs (such as
tracing programs).

Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
Reviewed-by: Eugene Loh <eugene.loh at oracle.com>


Compare: https://github.com/oracle/dtrace-utils/compare/94e53e27cbb4...40148f205503

To unsubscribe from these emails, change your notification settings at https://github.com/oracle/dtrace-utils/settings/notifications



More information about the DTrace-devel mailing list