[DTrace-devel] [PATCH v2] sparc: remove architecture support

Kris Van Hees kris.van.hees at oracle.com
Thu Mar 5 05:01:23 UTC 2026


Signed-off-by: Kris Van Hees <kris.van.hees at oracle.com>
---
 COMMANDLINE-OPTIONS                           |   6 +-
 GNUmakefile                                   |  10 +-
 include/sparc/platform.h                      |  32 -----
 libdtrace/dt_link.c                           |   4 +-
 libdtrace/dt_open.c                           |  16 +--
 libdtrace/procfs.d.in                         |   4 +-
 libdtrace/sparc/regs.d                        |  99 -------------
 libproc/sparc/isadep.c                        | 133 ------------------
 libproc/sparc/isadep.h                        |  81 -----------
 test/stress/fbtsafety/tst.vahole.d            |  50 -------
 test/unittest/aggs/tst.neglquant.d            |  10 +-
 test/unittest/arrays/tst.uregsarray.sparc64.d |  69 ---------
 test/unittest/arrays/tst.uregsarray.sparc64.x |   4 -
 test/unittest/pid/test.sparc64.options        |   1 -
 test/unittest/usdt/tst.enabled2.sh            |   2 +-
 15 files changed, 16 insertions(+), 505 deletions(-)
 delete mode 100644 include/sparc/platform.h
 delete mode 100644 libdtrace/sparc/regs.d
 delete mode 100644 libproc/sparc/isadep.c
 delete mode 100644 libproc/sparc/isadep.h
 delete mode 100755 test/stress/fbtsafety/tst.vahole.d
 delete mode 100644 test/unittest/arrays/tst.uregsarray.sparc64.d
 delete mode 100755 test/unittest/arrays/tst.uregsarray.sparc64.x
 delete mode 100644 test/unittest/pid/test.sparc64.options

diff --git a/COMMANDLINE-OPTIONS b/COMMANDLINE-OPTIONS
index 73be89b1..67be7447 100644
--- a/COMMANDLINE-OPTIONS
+++ b/COMMANDLINE-OPTIONS
@@ -3,8 +3,8 @@ DTRACE OPTIONS
 
 This file lists the dtrace commandline options, and indicates whether
 or not they have been made to work on Linux and roughly when.  All options
-are believed to work equally well on all supported architectures (x86 and
-SPARC).
+are believed to work equally well on all supported architectures (x86_64 and
+ARM64).
 
 Notations:
 
@@ -317,8 +317,6 @@ definitions are always specified and valid in all modes:
     * __linux
     * __unix
     * __SVR4
-    * __sparc (on SPARC® systems only)
-    * __sparcv9 (on SPARC® systems only when 64–bit programs are compiled)
     * __i386 (on x86 systems only when 32–bit programs are compiled)
     * __amd64 (on x86 systems only when 64–bit programs are compiled)
     * __`uname -s` (for example, __Linux)
diff --git a/GNUmakefile b/GNUmakefile
index 835bc9b5..10530f3c 100644
--- a/GNUmakefile
+++ b/GNUmakefile
@@ -20,8 +20,8 @@ VERSION := $(shell ./libdtrace/mkvers -vcurrent=t libdtrace/versions.list)
 
 ARCH := $(shell uname -m)
 
-$(if $(subst sparc64,,$(subst aarch64,,$(subst x86_64,,$(ARCH)))), \
-    $(error "Error: DTrace for Linux only supports x86_64, ARM64 and sparc64"),)
+$(if $(subst aarch64,,$(subst x86_64,,$(ARCH))), \
+    $(error "Error: DTrace for Linux currently only supports x86_64 and ARM64"),)
 $(if $(subst Linux,,$(shell uname -s)), \
     $(error "Error: DTrace only supports Linux"),)
 
@@ -74,7 +74,7 @@ ifdef KERNELSRCDIR
 KERNELOBJDIR ?= $(KERNELSRCDIR)
 endif
 
-KERNELARCH := $(subst sparc64,sparc,$(subst aarch64,arm64,$(subst x86_64,x86,$(ARCH))))
+KERNELARCH := $(subst aarch64,arm64,$(subst x86_64,x86,$(ARCH)))
 
 # Paths.
 
@@ -120,7 +120,7 @@ export CC ?= gcc
 
 BITNESS := 64
 NATIVE_BITNESS_ONLY := $(shell echo 'int main (void) { }' | $(CC) -x c -o /dev/null -m32 - 2>/dev/null || echo t)
-ARCHINC := $(subst sparc64,sparc,$(subst aarch64,arm64,$(subst x86_64,i386,$(ARCH))))
+ARCHINC := $(subst aarch64,arm64,$(subst x86_64,i386,$(ARCH)))
 
 INVARIANT_CFLAGS := -std=gnu99 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 $(if $(NATIVE_BITNESS_ONLY),-DNATIVE_BITNESS_ONLY) -D_DT_VERSION=\"$(VERSION)\"
 CPPFLAGS += -Iinclude -Iuts/common -Iinclude/$(ARCHINC) -I$(objdir)
@@ -129,7 +129,7 @@ override CFLAGS += $(INVARIANT_CFLAGS)
 PREPROCESS = $(CC) -E
 export BPFC ?= bpf-unknown-none-gcc
 
-BPFCPPFLAGS += -D$(subst sparc64,__sparc,$(subst aarch64,__aarch64__,$(subst x86_64,__amd64,$(ARCH))))
+BPFCPPFLAGS += -D$(subst aarch64,__aarch64__,$(subst x86_64,__amd64,$(ARCH)))
 BPFCFLAGS ?= -O2 -Wall -Wno-unknown-pragmas $(if $(HAVE_BPFV3),-mcpu=v3) $(if $(HAVE_BPFMASM),-masm=normal) -ffreestanding
 export BPFLD ?= bpf-unknown-none-ld
 
diff --git a/include/sparc/platform.h b/include/sparc/platform.h
deleted file mode 100644
index 1dea6ea2..00000000
--- a/include/sparc/platform.h
+++ /dev/null
@@ -1,32 +0,0 @@
-/*
- * Oracle Linux DTrace.
- * Copyright (c) 2013, 2015, Oracle and/or its affiliates. All rights reserved.
- * Licensed under the Universal Permissive License v 1.0 as shown at
- * http://oss.oracle.com/licenses/upl.
- */
-
-#ifndef _SPARC_PLATFORM_H
-#define _SPARC_PLATFORM_H
-
-#include <inttypes.h>
-#include <asm/ptrace.h>
-#include <elf.h>
-#include <sys/syscall.h>			/* for __NR_* */
-
-/*
- * Must be no larger than an 'unsigned long'.
- */
-const static unsigned char plat_bkpt[] = { 0x91, 0xd0, 0x20, 0x01 };
-
-/*
- * Number of processor-specific dynamic tags on this platform.
- */
-#define DT_THISPROCNUM DT_SPARC_NUM
-
-/*
- * TRUE if this platform requires software singlestepping.
- */
-#define NEED_SOFTWARE_SINGLESTEP 1
-
-#endif
-
diff --git a/libdtrace/dt_link.c b/libdtrace/dt_link.c
index 6d733881..340273fb 100644
--- a/libdtrace/dt_link.c
+++ b/libdtrace/dt_link.c
@@ -524,9 +524,7 @@ dtrace_program_link(dtrace_hdl_t *dtp, dtrace_prog_t *pgp, uint_t dflags,
 
 		if (dtp->dt_oflags & DTRACE_O_ILP32) {
 			snprintf(drti, sizeof (drti), "%s/drti/drti32.o", libdir->dir_path);
-#if defined(__sparc)
-			emu = " -m elf32_sparc";
-#elif defined(__i386) || defined(__amd64)
+#if defined(__i386) || defined(__amd64)
 			emu = " -m elf_i386";
 #endif
 		} else {
diff --git a/libdtrace/dt_open.c b/libdtrace/dt_open.c
index 54adec02..30b8758d 100644
--- a/libdtrace/dt_open.c
+++ b/libdtrace/dt_open.c
@@ -827,24 +827,10 @@ dt_vopen(int version, int flags, int *errp,
 	else if (flags & DTRACE_O_ILP32)
 		dtp->dt_conf.dtc_ctfmodel = CTF_MODEL_ILP32;
 
-#ifdef __sparc
-	/*
-	 * On SPARC systems, __sparc is always defined for <sys/isa_defs.h>
-	 * and __sparcv9 is defined if we are doing a 64-bit compile.
-	 */
-	if (dt_cpp_add_arg(dtp, "-D__sparc") == NULL)
-		return set_open_errno(dtp, errp, EDT_NOMEM);
-
-	if (dtp->dt_conf.dtc_ctfmodel == CTF_MODEL_LP64 &&
-	    dt_cpp_add_arg(dtp, "-D__sparcv9") == NULL)
-		return set_open_errno(dtp, errp, EDT_NOMEM);
-#endif
-
 #ifdef __x86
 	/*
 	 * On x86 systems, __i386 is defined for <sys/isa_defs.h> for 32-bit
-	 * compiles and __amd64 is defined for 64-bit compiles.  Unlike SPARC,
-	 * they are defined exclusive of one another (see PSARC 2004/619).
+	 * compiles and __amd64 is defined for 64-bit compiles.
 	 */
 	if (dtp->dt_conf.dtc_ctfmodel == CTF_MODEL_LP64) {
 		if (dt_cpp_add_arg(dtp, "-D__amd64") == NULL)
diff --git a/libdtrace/procfs.d.in b/libdtrace/procfs.d.in
index 23454d63..4140608b 100644
--- a/libdtrace/procfs.d.in
+++ b/libdtrace/procfs.d.in
@@ -285,9 +285,7 @@ translator lwpsinfo_t < struct task_struct *T > {
 define_for_kernel([[on_cpu]], [[(m4_kver(5,16,0), m4_dnl
   [[((struct thread_info *)T->stack)->cpu]]), m4_dnl
 (m4_kver(4,9,0), m4_dnl
-  [[if_arch([[x86]],[[T->cpu]])]] m4_dnl
-  [[if_arch([[arm64]],[[T->cpu]])]] m4_dnl
-  [[if_arch([[sparc]],[[((struct thread_info *)T->stack)->cpu]])]])]], m4_dnl
+  [[T->cpu]])]], m4_dnl
 [[((struct thread_info *)T->stack)->cpu]])m4_dnl
 	pr_onpro = on_cpu;
 };
diff --git a/libdtrace/sparc/regs.d b/libdtrace/sparc/regs.d
deleted file mode 100644
index 112ae207..00000000
--- a/libdtrace/sparc/regs.d
+++ /dev/null
@@ -1,99 +0,0 @@
-/*
- * Oracle Linux DTrace.
- * Copyright (c) 2012, Oracle and/or its affiliates. All rights reserved.
- * Licensed under the Universal Permissive License v 1.0 as shown at
- * http://oss.oracle.com/licenses/upl.
- */
-
-inline int R_G0	= 0;
-#pragma D binding "1.0" R_G0
-inline int R_G1	= 1;
-#pragma D binding "1.0" R_G1
-inline int R_G2	= 2;
-#pragma D binding "1.0" R_G2
-inline int R_G3	= 3;
-#pragma D binding "1.0" R_G3
-inline int R_G4	= 4;
-#pragma D binding "1.0" R_G4
-inline int R_G5	= 5;
-#pragma D binding "1.0" R_G5
-inline int R_G6	= 6;
-#pragma D binding "1.0" R_G6
-inline int R_G7	= 7;
-#pragma D binding "1.0" R_G7
-
-inline int R_O0	= 8;
-#pragma D binding "1.0" R_O0
-inline int R_O1	= 9;
-#pragma D binding "1.0" R_O1
-inline int R_O2	= 10;
-#pragma D binding "1.0" R_O2
-inline int R_O3	= 11;
-#pragma D binding "1.0" R_O3
-inline int R_O4	= 12;
-#pragma D binding "1.0" R_O4
-inline int R_O5	= 13;
-#pragma D binding "1.0" R_O5
-inline int R_O6	= 14;
-#pragma D binding "1.0" R_O6
-inline int R_O7	= 15;
-#pragma D binding "1.0" R_O7
-
-inline int R_L0	= 16;
-#pragma D binding "1.0" R_L0
-inline int R_L1	= 17;
-#pragma D binding "1.0" R_L1
-inline int R_L2	= 18;
-#pragma D binding "1.0" R_L2
-inline int R_L3	= 19;
-#pragma D binding "1.0" R_L3
-inline int R_L4	= 20;
-#pragma D binding "1.0" R_L4
-inline int R_L5	= 21;
-#pragma D binding "1.0" R_L5
-inline int R_L6	= 22;
-#pragma D binding "1.0" R_L6
-inline int R_L7	= 23;
-#pragma D binding "1.0" R_L7
-
-inline int R_I0	= 24;
-#pragma D binding "1.0" R_I0
-inline int R_I1	= 25;
-#pragma D binding "1.0" R_I1
-inline int R_I2	= 26;
-#pragma D binding "1.0" R_I2
-inline int R_I3	= 27;
-#pragma D binding "1.0" R_I3
-inline int R_I4	= 28;
-#pragma D binding "1.0" R_I4
-inline int R_I5	= 29;
-#pragma D binding "1.0" R_I5
-inline int R_I6	= 30;
-#pragma D binding "1.0" R_I6
-inline int R_I7	= 31;
-#pragma D binding "1.0" R_I7
-
-inline int R_CCR = 32;
-#pragma D binding "1.0" R_CCR
-inline int R_PC = 33;
-#pragma D binding "1.0" R_PC
-inline int R_nPC = 34;
-#pragma D binding "1.0" R_nPC
-inline int R_NPC = R_nPC;
-#pragma D binding "1.0" R_NPC
-inline int R_Y = 35;
-#pragma D binding "1.0" R_Y
-inline int R_ASI = 36;
-#pragma D binding "1.0" R_ASI
-inline int R_FPRS = 37;
-#pragma D binding "1.0" R_FPRS
-inline int R_PS = R_CCR;
-#pragma D binding "1.0" R_PS
-inline int R_SP = R_O6;
-#pragma D binding "1.0" R_SP
-inline int R_FP = R_I6;
-#pragma D binding "1.0" R_FP
-inline int R_R0 = R_O0;
-#pragma D binding "1.0" R_R0
-inline int R_R1 = R_O1;
-#pragma D binding "1.0" R_R1
diff --git a/libproc/sparc/isadep.c b/libproc/sparc/isadep.c
deleted file mode 100644
index aa4f53f5..00000000
--- a/libproc/sparc/isadep.c
+++ /dev/null
@@ -1,133 +0,0 @@
-/*
- * ISA-dependent functions for SPARC and SPARC64.
- */
-
-/*
- * Oracle Linux DTrace.
- * Copyright (c) 2013, 2020, Oracle and/or its affiliates. All rights reserved.
- * Licensed under the Universal Permissive License v 1.0 as shown at
- * http://oss.oracle.com/licenses/upl.
- */
-
-#include <inttypes.h>
-#include <errno.h>
-#include <string.h>
-#include <sys/ptrace.h>
-#include <asm/ptrace.h>				/* for struct pt_regs */
-
-#include "Pcontrol.h"
-#include "libproc.h"
-#include "platform.h"
-
-#define PTRACE_GETREGSET        0x4204
-
-/*
- * Get a SPARC64 regset.
- */
-static struct pt_regs *
-getregs_sparc64(struct ps_prochandle *P, struct pt_regs *regs)
-{
-	if (Pstate(P) == PS_RUN ||
-	    Pstate(P) == PS_DEAD)
-		return NULL;
-
-	if (wrapped_ptrace(P, PTRACE_GETREGS64, P->pid, regs) < 0)
-		return NULL;
-
-	return regs;
-}
-
-static struct pt_regs32 *
-getregs_sparc32(struct ps_prochandle *P, struct pt_regs32 *regs)
-{
-	if (Pstate(P) == PS_RUN ||
-	    Pstate(P) == PS_DEAD)
-		return NULL;
-
-	if (wrapped_ptrace(P, PTRACE_GETREGS, P->pid, regs) < 0)
-		return NULL;
-
-	return regs;
-}
-
-/*
- * Read the first argument of the function at which the process P is halted,
- * which must be a pointer (though other integral arguments happen to work for
- * the SPARC and SPARC64 ABI).
- *
- * For SPARC, the first integral argument to a function will always be in %i0.
- * Horrifically, the value of all the UREG_* constants is off by 1, so we have
- * to correct for that.
- *
- * On error, -1 cast to a uintptr_t is returned, and errno is set.
- */
-uintptr_t
-Pread_first_arg_sparc64(struct ps_prochandle *P)
-{
-	struct pt_regs regs;
-
-	if (getregs_sparc64(P, &regs) == NULL)
-		return (uintptr_t)-1;
-
-	return regs.u_regs[UREG_I0 - 1];
-}
-
-uintptr_t
-Pread_first_arg_sparc32(struct ps_prochandle *P)
-{
-	struct pt_regs32 regs;
-
-	if (getregs_sparc32(P, &regs) == NULL)
-		return (uintptr_t)-1;
-
-	return regs.u_regs[UREG_I0 - 1];
-}
-
-/*
- * Get the address at which we stopped at a breakpoint.  Unlike on x86, no post
- * facto adjustment is required.
- */
-uintptr_t
-Pget_bkpt_ip_sparc64(struct ps_prochandle *P, int expect_esrch)
-{
-	struct pt_regs regs;
-
-	if (getregs_sparc64(P, &regs) == NULL) {
-		if ((errno == ESRCH) && (expect_esrch))
-			return 0;
-		else {
-			_dprintf("Unexpected ptrace (PTRACE_GETREGSET) "
-			    "error: %s\n", strerror(errno));
-			return -1;
-		}
-	}
-
-	return regs.tpc;
-}
-
-/*
- * Reset the instruction pointer address at which the process P is stopped.
- * (Not needed on SPARC.)
- */
-long
-Preset_bkpt_ip_sparc(struct ps_prochandle *P, uintptr_t addr)
-{
-    return 0; /* nothing doing */
-}
-
-/*
- * Get the address at which the temporary breakpoint should be dropped when a
- * breakpoint is hit.
- */
-extern long Pget_next_ip_sparc64(struct ps_prochandle *P, uintptr_t addr)
-{
-	struct pt_regs regs;
-
-	if (getregs_sparc64(P, &regs) == NULL) {
-		_dprintf("Unexpected ptrace (PTRACE_GETREGSET) "
-		    "error: %s\n", strerror(errno));
-		return -1;
-	}
-
-	return regs.tnpc;
-}
diff --git a/libproc/sparc/isadep.h b/libproc/sparc/isadep.h
deleted file mode 100644
index 005e6008..00000000
--- a/libproc/sparc/isadep.h
+++ /dev/null
@@ -1,81 +0,0 @@
-/*
- * Underlying ISA-dependent functions for SPARC and SPARC64.
- */
-
-/*
- * Oracle Linux DTrace.
- * Copyright (c) 2013, 2015, Oracle and/or its affiliates. All rights reserved.
- * Licensed under the Universal Permissive License v 1.0 as shown at
- * http://oss.oracle.com/licenses/upl.
- */
-
-/*
- * Not protected against multiple inclusion: this file is included by
- * isadep_dispatch.h repeatedly, at local scope, to get different prototypes and
- * dispatch variables.
- */
-
-#ifdef WANT_FIRST_ARG_DISPATCH
-
-extern	uintptr_t Pread_first_arg_sparc64(struct ps_prochandle *P);
-extern	uintptr_t Pread_first_arg_sparc32(struct ps_prochandle *P);
-
-isa_dispatch_t dispatch[] = {
-    {B_FALSE, EM_SPARCV9, (dispatch_fun_t *) Pread_first_arg_sparc32},
-    {B_TRUE, EM_SPARCV9, (dispatch_fun_t *) Pread_first_arg_sparc64},
-    {B_FALSE, EM_SPARC32PLUS, (dispatch_fun_t *) Pread_first_arg_sparc32},
-    {B_TRUE, EM_SPARC32PLUS, (dispatch_fun_t *) Pread_first_arg_sparc64},
-    {B_FALSE, EM_SPARC, (dispatch_fun_t *) Pread_first_arg_sparc32},
-    {B_TRUE, EM_SPARC, (dispatch_fun_t *) Pread_first_arg_sparc64}, /* ??? */
-    {0, 0, NULL}};
-
-#endif
-
-#ifdef WANT_GET_BKPT_IP
-
-extern uintptr_t Pget_bkpt_ip_sparc64(struct ps_prochandle *P, int expect_esrch);
-
-/*
- * I hope 32-on-64 traps are treated like 64-on-64 traps.
- */
-
-isa_dispatch_t dispatch[] = {
-    {B_FALSE, EM_SPARCV9, (dispatch_fun_t *) Pget_bkpt_ip_sparc64},
-    {B_TRUE, EM_SPARCV9, (dispatch_fun_t *) Pget_bkpt_ip_sparc64},
-    {B_FALSE, EM_SPARC32PLUS, (dispatch_fun_t *) Pget_bkpt_ip_sparc64},
-    {B_TRUE, EM_SPARC32PLUS, (dispatch_fun_t *) Pget_bkpt_ip_sparc64},
-    {B_FALSE, EM_SPARC, (dispatch_fun_t *) Pget_bkpt_ip_sparc64},
-    {B_TRUE, EM_SPARC, (dispatch_fun_t *) Pget_bkpt_ip_sparc64}, /* ??? */
-    {0, 0, NULL}};
-
-#endif
-
-#ifdef WANT_RESET_BKPT_IP
-
-extern	long Preset_bkpt_ip_sparc(struct ps_prochandle *P, uintptr_t addr);
-
-isa_dispatch_t dispatch[] = {
-    {B_FALSE, EM_SPARCV9, (dispatch_fun_t *) Preset_bkpt_ip_sparc},
-    {B_TRUE, EM_SPARCV9, (dispatch_fun_t *) Preset_bkpt_ip_sparc},
-    {B_FALSE, EM_SPARC32PLUS, (dispatch_fun_t *) Preset_bkpt_ip_sparc},
-    {B_TRUE, EM_SPARC32PLUS, (dispatch_fun_t *) Preset_bkpt_ip_sparc},
-    {B_FALSE, EM_SPARC, (dispatch_fun_t *) Preset_bkpt_ip_sparc},
-    {B_TRUE, EM_SPARC, (dispatch_fun_t *) Preset_bkpt_ip_sparc}, /* ??? */
-    {0, 0, NULL}};
-
-#endif
-
-#ifdef WANT_GET_NEXT_IP
-
-extern	long Pget_next_ip_sparc64(struct ps_prochandle *P, uintptr_t addr);
-
-isa_dispatch_t dispatch[] = {
-    {B_FALSE, EM_SPARCV9, (dispatch_fun_t *) Pget_next_ip_sparc64},
-    {B_TRUE, EM_SPARCV9, (dispatch_fun_t *) Pget_next_ip_sparc64},
-    {B_FALSE, EM_SPARC32PLUS, (dispatch_fun_t *) Pget_next_ip_sparc64},
-    {B_TRUE, EM_SPARC32PLUS, (dispatch_fun_t *) Pget_next_ip_sparc64},
-    {B_FALSE, EM_SPARC, (dispatch_fun_t *) Pget_next_ip_sparc64},
-    {B_TRUE, EM_SPARC, (dispatch_fun_t *) Pget_next_ip_sparc64}, /* ??? */
-    {0, 0, NULL}};
-
-#endif
diff --git a/test/stress/fbtsafety/tst.vahole.d b/test/stress/fbtsafety/tst.vahole.d
deleted file mode 100755
index 732ae3ca..00000000
--- a/test/stress/fbtsafety/tst.vahole.d
+++ /dev/null
@@ -1,50 +0,0 @@
-#!dtrace -Cs
-
-/*
- * Oracle Linux DTrace.
- * Copyright (c) 2006, 2026, Oracle and/or its affiliates. All rights reserved.
- * Licensed under the Universal Permissive License v 1.0 as shown at
- * http://oss.oracle.com/licenses/upl.
- */
-
-/* @@skip: nonportable */
-
-/*
- * We set our buffer size absurdly low to prevent a flood of errors that we
- * don't care about.
- */
-#pragma D option bufsize=16
-#pragma D option bufpolicy=ring
-
-fbt:::
-{
-	n++;
-#ifdef __sparc
-	trace(*(int *)0x8000000000000000 ^ rand());
-#else
-	trace(*(int *)(`kernelbase - 1));
-#endif
-}
-
-dtrace:::ERROR
-{
-	err++;
-}
-
-tick-1sec
-/sec++ == 10/
-{
-	exit(2);
-}
-
-END
-/n == 0 || err == 0/
-{
-	exit(1);
-}
-
-END
-/n != 0 && err != 0/
-{
-	exit(0);
-}
diff --git a/test/unittest/aggs/tst.neglquant.d b/test/unittest/aggs/tst.neglquant.d
index c09d7aab..5e6d1dc0 100644
--- a/test/unittest/aggs/tst.neglquant.d
+++ b/test/unittest/aggs/tst.neglquant.d
@@ -90,11 +90,11 @@ BEGIN
 	 * We want to test the ability to sort very large quantizations
 	 * that differ by a small amount.  Ideally, they would differ only
 	 * by 1 -- but that is smaller than the precision of long doubles of
-	 * this magnitude on x86.  To assure that the same test works on x86
-	 * just as it does on SPARC, we pick a value that is just larger than
-	 * the precision at this magnitude.  It should go without saying that
-	 * this robustness on new ISAs very much depends on the precision
-	 * of the long double representation.
+	 * this magnitude on x86_64.  To assure that the same test works for
+	 * the architectures DTrace is developed for, we pick a value that is
+	 * just larger than the precision at this magnitude.  It should go
+	 * without saying that this robustness on new ISAs very much depends on
+	 * the precision of the long double representation.
 	 */
 	this->val = (((long long) 1) << 63) - 7;
 
diff --git a/test/unittest/arrays/tst.uregsarray.sparc64.d b/test/unittest/arrays/tst.uregsarray.sparc64.d
deleted file mode 100644
index 2179a0db..00000000
--- a/test/unittest/arrays/tst.uregsarray.sparc64.d
+++ /dev/null
@@ -1,69 +0,0 @@
-/*
- * Oracle Linux DTrace.
- * Copyright (c) 2006, Oracle and/or its affiliates. All rights reserved.
- * Licensed under the Universal Permissive License v 1.0 as shown at
- * http://oss.oracle.com/licenses/upl.
- */
-/* @@xfail: dtv2 */
-
-/*
- * ASSERTION:
- *	Positive test to make sure that we can invoke sparc
- *	ureg[] aliases.
- *
- * SECTION: User Process Tracing/uregs Array
- *
- * NOTES: This test does no verification - the value of the output
- *	is not deterministic.
- */
-
-#pragma D option quiet
-
-BEGIN
-{
-	printf("R_G0 = 0x%x\n", uregs[R_G0]);
-	printf("R_G1 = 0x%x\n", uregs[R_G1]);
-	printf("R_G2 = 0x%x\n", uregs[R_G2]);
-	printf("R_G3 = 0x%x\n", uregs[R_G3]);
-	printf("R_G4 = 0x%x\n", uregs[R_G4]);
-	printf("R_G5 = 0x%x\n", uregs[R_G5]);
-	printf("R_G6 = 0x%x\n", uregs[R_G6]);
-	printf("R_G7 = 0x%x\n", uregs[R_G7]);
-	printf("R_O0 = 0x%x\n", uregs[R_O0]);
-	printf("R_O1 = 0x%x\n", uregs[R_O1]);
-	printf("R_O2 = 0x%x\n", uregs[R_O2]);
-	printf("R_O3 = 0x%x\n", uregs[R_O3]);
-	printf("R_O4 = 0x%x\n", uregs[R_O4]);
-	printf("R_O5 = 0x%x\n", uregs[R_O5]);
-	printf("R_O6 = 0x%x\n", uregs[R_O6]);
-	printf("R_O7 = 0x%x\n", uregs[R_O7]);
-	printf("R_L0 = 0x%x\n", uregs[R_L0]);
-	printf("R_L1 = 0x%x\n", uregs[R_L1]);
-	printf("R_L2 = 0x%x\n", uregs[R_L2]);
-	printf("R_L3 = 0x%x\n", uregs[R_L3]);
-	printf("R_L4 = 0x%x\n", uregs[R_L4]);
-	printf("R_L5 = 0x%x\n", uregs[R_L5]);
-	printf("R_L6 = 0x%x\n", uregs[R_L6]);
-	printf("R_L7 = 0x%x\n", uregs[R_L7]);
-	printf("R_I0 = 0x%x\n", uregs[R_I0]);
-	printf("R_I1 = 0x%x\n", uregs[R_I1]);
-	printf("R_I2 = 0x%x\n", uregs[R_I2]);
-	printf("R_I3 = 0x%x\n", uregs[R_I3]);
-	printf("R_I4 = 0x%x\n", uregs[R_I4]);
-	printf("R_I5 = 0x%x\n", uregs[R_I5]);
-	printf("R_I6 = 0x%x\n", uregs[R_I6]);
-	printf("R_I7 = 0x%x\n", uregs[R_I7]);
-	printf("R_CCR = 0x%x\n", uregs[R_CCR]);
-	printf("R_PC = 0x%x\n", uregs[R_PC]);
-	printf("R_NPC = 0x%x\n", uregs[R_NPC]);
-	printf("R_Y = 0x%x\n", uregs[R_Y]);
-	printf("R_ASI = 0x%x\n", uregs[R_ASI]);
-	printf("R_FPRS = 0x%x\n", uregs[R_FPRS]);
-	exit(0);
-}
-
-ERROR
-{
-	printf("uregs access failed.\n");
-	exit(1);
-}
diff --git a/test/unittest/arrays/tst.uregsarray.sparc64.x b/test/unittest/arrays/tst.uregsarray.sparc64.x
deleted file mode 100755
index 2c34e04d..00000000
--- a/test/unittest/arrays/tst.uregsarray.sparc64.x
+++ /dev/null
@@ -1,4 +0,0 @@
-#!/bin/sh
-
-[ `uname -m` = "sparc64" ] && exit 0
-exit 2
diff --git a/test/unittest/pid/test.sparc64.options b/test/unittest/pid/test.sparc64.options
deleted file mode 100644
index 9e81427e..00000000
--- a/test/unittest/pid/test.sparc64.options
+++ /dev/null
@@ -1 +0,0 @@
-@@skip: pid provider not yet implemented (sparc64)
diff --git a/test/unittest/usdt/tst.enabled2.sh b/test/unittest/usdt/tst.enabled2.sh
index b174ef4c..c33bca07 100755
--- a/test/unittest/usdt/tst.enabled2.sh
+++ b/test/unittest/usdt/tst.enabled2.sh
@@ -5,7 +5,7 @@
 # Licensed under the Universal Permissive License v 1.0 as shown at
 # http://oss.oracle.com/licenses/upl.
 #
-# This test is primarily intended to verify a fix for SPARC, but there's no
+# This test was primarily developed to verify a fix for SPARC, but there's no
 # harm in running it on other platforms. Here, we verify that is-enabled
 # probes don't interfere with return values from previously invoked functions.
 
-- 
2.51.0




More information about the DTrace-devel mailing list