----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3672/#review8931 -----------------------------------------------------------
Ship it! All that NXM support could probably go away at some point. We should also consider just eliminating SE-mode Tru64 support. I'm not sure anything uses it, and if so, it's probably stuff that people shouldn't be using anymore anyway. Of course, I'm sure spme people named Andreas would say that that sentiment applies to Alpha support on the whole... but I think the overhead of maintaining Alpha Linux support is pretty low, at least comparable to some of the Tru64 stuff, in terms of wonky things that aren't hidden away in their own source files. Though I haven't looked at the code in a while so I could be wrong. - Steve Reinhardt On Oct. 17, 2016, 8:19 a.m., Brandon Potter wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3672/ > ----------------------------------------------------------- > > (Updated Oct. 17, 2016, 8:19 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11694:ebff480ad584 > --------------------------- > syscall_emul: [patch 6/22] remove unused fields from Process class > > It looks like tru64 has some nxm* system calls, but the two fields that > are defined in the Process class are unused by any of the code. There doesn't > appear to be any reference in the tru64 code. > > > Diffs > ----- > > src/sim/process.cc 4a86763c0b30cccba0f56c7f48637a46a4663b06 > src/sim/process.hh 4a86763c0b30cccba0f56c7f48637a46a4663b06 > > Diff: http://reviews.gem5.org/r/3672/diff/ > > > Testing > ------- > > > Thanks, > > Brandon Potter > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
