----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3681/#review9027 -----------------------------------------------------------
src/sim/process.cc (line 124) <http://reviews.gem5.org/r/3681/#comment7770> This should be a fatal. In fact, maybe 2. One to check that it's less than MAX_PID, and another to make sure it's not already in use. src/sim/system.hh (line 543) <http://reviews.gem5.org/r/3681/#comment7769> Can this just be an std::set<int>? This is only used to make sure there are no duplicates, right? - Jason Lowe-Power On Nov. 7, 2016, 9:56 p.m., Brandon Potter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3681/ > ----------------------------------------------------------- > > (Updated Nov. 7, 2016, 9:56 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11703:9a6631ad6f41 > --------------------------- > syscall_emul: [patch 14/22] adds identifier system calls > > This changeset add fields to the process object and adds the following > three system calls: setpgid, gettid, getpid. > > > Diffs > ----- > > src/sim/syscall_emul.cc 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/sim/system.hh 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/arch/x86/linux/process.cc 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/sim/Process.py 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/sim/process.hh 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/sim/process.cc 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/sim/syscall_emul.hh 4a86763c0b30cccba0f56c7f48637a46a4663b06 > > Diff: http://reviews.gem5.org/r/3681/diff/ > > > Testing > ------- > > > Thanks, > > Brandon Potter > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
