[DTrace-devel] [PATCH] test: Change open() to openat() in specopen

eugene.loh at oracle.com eugene.loh at oracle.com
Tue Nov 19 19:18:15 UTC 2024


From: Eugene Loh <eugene.loh at oracle.com>

In commit 751f8f46dd29 "test: Clean up tst.specopen.d", there was
minor cleanup to make this example run.  Actually, it should have
converted to openat() calls, since open() is deprecated.

Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
---
 test/demo/spec/specopen.d | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/test/demo/spec/specopen.d b/test/demo/spec/specopen.d
index 700bdd9ff..e7dbc3b25 100644
--- a/test/demo/spec/specopen.d
+++ b/test/demo/spec/specopen.d
@@ -7,7 +7,7 @@
  * http://oss.oracle.com/licenses/upl.
  */
 
-syscall::open:entry
+syscall::openat:entry
 {
 	/*
 	 * The call to speculation() creates a new speculation.  If this fails,
@@ -23,7 +23,7 @@ syscall::open:entry
 	 * speculatively traced; it will only appear in the data buffer if the
 	 * speculation is subsequently commited.
 	 */
-	printf("%s", stringof(copyinstr(arg0)));
+	printf("%s", stringof(copyinstr(arg1)));
 }
 
 fbt:::
@@ -35,20 +35,20 @@ fbt:::
 	speculate(self->spec);
 }
 
-syscall::open:return
+syscall::openat:return
 /self->spec/
 {
 	/*
 	 * To balance the output with the -F option, we want to be sure that
 	 * every entry has a matching return.  Because we speculated the
-	 * open entry above, we want to also speculate the open return.
+	 * openat entry above, we want to also speculate the openat return.
 	 * This is also a convenient time to trace the errno value.
 	 */
 	speculate(self->spec);
 	trace(errno);
 }
 
-syscall::open:return
+syscall::openat:return
 /self->spec && errno != 0/
 {
 	/*
@@ -58,7 +58,7 @@ syscall::open:return
 	self->spec = 0;
 }
 
-syscall::open:return
+syscall::openat:return
 /self->spec && errno == 0/
 {
 	/*
-- 
2.43.5




More information about the DTrace-devel mailing list