> On Aug. 19, 2015, 2:56 p.m., Steve Reinhardt wrote: > > src/mem/packet.hh, line 580 > > <http://reviews.gem5.org/r/3044/diff/1/?file=49083#file49083line580> > > > > Looks like you need to update this comment. I assume that no one ever > > used the capability described here?
Indeed, it was never used. I'll update the comment. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/3044/#review7015 ----------------------------------------------------------- On Aug. 19, 2015, 9:06 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/3044/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2015, 9:06 a.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 11055:dc0a15e6e113 > --------------------------- > mem: Reflect that packet address and size are always valid > > This patch simplifies the packet, and removes the possibility of > creating a packet without a valid address and/or size. Under no > circumstances are these fields set at a later point, and thus they > really have to be provided at construction time. > > The patch also fixes a case there the MinorCPU creates a packet > without a valid address and size, only to later delete it. > > > Diffs > ----- > > src/cpu/minor/lsq.cc 110cce93d398 > src/mem/packet.hh 110cce93d398 > > Diff: http://reviews.gem5.org/r/3044/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
