> On Aug. 26, 2015, 7:42 a.m., Jason Power wrote: > > I'm not arguing against this patch, but I do see the original rationale. > > Running a binary natively, or appending "gem5.opt config.py -c" in front of > > the binary will produce the same outputs. I don't think I have an opinion > > one way or the other.
Good point---obviously this change works better for my use case, but I can see the confusion you point out. In particular, I can imagine a case where a program creates an output file, then closes and re-opens it, and the re-open will fail since the file did not get created in the cwd. So now I sort of feel like this ought to be controlled with a command-line flag. Another alternative would be to have an option to force a cwd change into the output directory, which I think would make this change irrelevant (since output files would then automatically get created in the output dir) and also solve the create/re-open problem, and would only be an issue if your program is reading files in the original dir, but if you specified the cd option then presumably you'd know that you need to fix that. Thoughts? - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3076/#review7053 ----------------------------------------------------------- On Aug. 23, 2015, 11:56 p.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3076/ > ----------------------------------------------------------- > > (Updated Aug. 23, 2015, 11:56 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11064:98f8322edfdb > --------------------------- > syscall_emul: create Process output files in output dir > > Redirected output files from Process objects were being created in > the current directory instead of in the output directory. > > > Diffs > ----- > > src/sim/process.cc 842f56345a421244a7a8988a5bc4fb1cfbf409ef > > Diff: http://reviews.gem5.org/r/3076/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
