Brandon Potter has submitted this change and it was merged. ( https://gem5-review.googlesource.com/c/public/gem5/+/12124 )

Change subject: sim-se: use DPRINTF_SYSCALL for ioctl/wait4
......................................................................

sim-se: use DPRINTF_SYSCALL for ioctl/wait4

Change-Id: I4fbaf1a0653f13ae964a2574cc26bbaac2dc0686
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/12124
Maintainer: Brandon Potter <[email protected]>
Tested-by: kokoro <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
---
M src/sim/syscall_emul.hh
1 file changed, 3 insertions(+), 4 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved
  Brandon Potter: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/sim/syscall_emul.hh b/src/sim/syscall_emul.hh
index 892d9d7..7996027 100644
--- a/src/sim/syscall_emul.hh
+++ b/src/sim/syscall_emul.hh
@@ -754,7 +754,7 @@
     int tgt_fd = p->getSyscallArg(tc, index);
     unsigned req = p->getSyscallArg(tc, index);

-    DPRINTF(SyscallVerbose, "ioctl(%d, 0x%x, ...)\n", tgt_fd, req);
+    DPRINTF_SYSCALL(Verbose, "ioctl(%d, 0x%x, ...)\n", tgt_fd, req);

     if (OS::isTtyReq(req))
         return -ENOTTY;
@@ -2741,10 +2741,9 @@
     Addr rusagePtr = p->getSyscallArg(tc, index);

     if (rusagePtr)
-        DPRINTFR(SyscallVerbose,
-                 "%d: %s: syscall wait4: rusage pointer provided however "
+ DPRINTF_SYSCALL(Verbose, "wait4: rusage pointer provided %lx, however "
                  "functionality not supported. Ignoring rusage pointer.\n",
-                 curTick(), tc->getCpuPtr()->name());
+                 rusagePtr);

     /**
* Currently, wait4 is only implemented so that it will wait for children

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/12124
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I4fbaf1a0653f13ae964a2574cc26bbaac2dc0686
Gerrit-Change-Number: 12124
Gerrit-PatchSet: 23
Gerrit-Owner: Brandon Potter <[email protected]>
Gerrit-Reviewer: Alexandru DuČ›u <[email protected]>
Gerrit-Reviewer: Brandon Potter <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: John Alsop <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to