----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3582/#review8848 -----------------------------------------------------------
Please mark this as submitted. Thanks. - Jason Lowe-Power On Aug. 7, 2016, 4:12 p.m., Nicolas Derumigny wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3582/ > ----------------------------------------------------------- > > (Updated Aug. 7, 2016, 4:12 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > syscall_emul: Corrected a bug on the syscall "read()". > Read() should not write anything when returning 0 (EOF). > This patch does not correct the same bug occuring for : > > nbr_read=read(file, buf, nbytes) > > When nbr_read<nbytes, nbytes bytes are copied into the virtual > RAM instead of nbr_read. If buf is smaller than nbytes, a > page fault occurs, even if buf is in fact bigger than nbr_read. > > > Diffs > ----- > > src/sim/syscall_emul.cc cdb94f2332a6 > > Diff: http://reviews.gem5.org/r/3582/diff/ > > > Testing > ------- > > Works with CERE codelets (https://github.com/benchmark-subsetting/cere), > correcting the issue when replacing dumps in memory. > > > Thanks, > > Nicolas Derumigny > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev