commit f4d21fa24a3a61ac10b764ea97ce4746f60820fb
author Brandon Potter <brandon.potter@amd.com> Fri Aug 09 14:03:59 2019 -0400
committer Brandon Potter <Brandon.Potter@amd.com> Mon Aug 12 16:49:54 2019 +0000
tree 9fb2c6ea1ab6ec2d067e789fbfbb827c923130ce
parent dbf18ac11125f08114b8bc13a9140a1b7653054c

sim-se: rename Process::setpgid member

The getter methods to access these types of members do not
have a 'get' string in the method names. To make the interface
a bit more consistent, remove the 'set' part of the member name.

Change-Id: I04c56bd9d9feb1cf68ff50a1152083ea57ea7c62
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/20008
Reviewed-by: Anthony Gutierrez <anthony.gutierrez@amd.com>
Maintainer: Brandon Potter <Brandon.Potter@amd.com>
Tested-by: kokoro <noreply+kokoro@google.com>

src/sim/process.hh

diff --git a/src/sim/process.hh b/src/sim/process.hh
index 23ed6d6..ae23de4 100644
--- a/src/sim/process.hh
+++ b/src/sim/process.hh
@@ -87,8 +87,8 @@
     inline uint64_t pid() { return _pid; }
     inline uint64_t ppid() { return _ppid; }
     inline uint64_t pgid() { return _pgid; }
+    inline void pgid(uint64_t pgid) { _pgid = pgid; }
     inline uint64_t tgid() { return _tgid; }
-    inline void setpgid(uint64_t pgid) { _pgid = pgid; }
 
     const char *progName() const { return executable.c_str(); }
 

src/sim/syscall_emul.cc

diff --git a/src/sim/syscall_emul.cc b/src/sim/syscall_emul.cc
index 088dcbc..f2f4f27 100644
--- a/src/sim/syscall_emul.cc
+++ b/src/sim/syscall_emul.cc
@@ -1001,7 +1001,7 @@
         return -EINVAL;
 
     if (pid == 0) {
-        process->setpgid(process->pid());
+        process->pgid(process->pid());
         return 0;
     }
 
@@ -1020,7 +1020,7 @@
     }
 
     assert(matched_ph);
-    matched_ph->setpgid((pgid == 0) ? matched_ph->pid() : pgid);
+    matched_ph->pgid((pgid == 0) ? matched_ph->pid() : pgid);
 
     return 0;
 }