> On Feb. 22, 2012, 6:16 p.m., Steve Reinhardt wrote: > > src/mem/bus.cc, line 72 > > <http://reviews.gem5.org/r/1047/diff/1/?file=23374#file23374line72> > > > > Does it make more sense to just maintain two separate vectors, one for > > masters and one for slaves? It'd require more extensive changes to the > > rest of the bus code, but it might actually clean it up a bit... maybe > > you're already doing this in some future patch I haven't seen yet. > > Andreas Hansson wrote: > Great minds think alike :). > > This master/slave split is indeed happening with the introduction of the > master and slave port in the C++ classes. Once this happens there will be a > slavePorts and masterPorts vector. I'd suggest to save it until that later > patch (two or three patches away from being posted to the board).
Great, I'll look forward to that. > On Feb. 22, 2012, 6:16 p.m., Steve Reinhardt wrote: > > src/cpu/testers/directedtest/RubyDirectedTester.cc, line 86 > > <http://reviews.gem5.org/r/1047/diff/1/?file=23354#file23354line86> > > > > Is this cast really necessary? Looks like ports.size() should be a > > size_t. I'm curious what error/warning we get if we leave it out... > > Andreas Hansson wrote: > size_t is unsigned, int signed, hence with the warning enabled: > > warning: comparison between signed and unsigned integer expressions > [-Wsign-compare] > > I'd rather keep the code as clean as possible and avoid issues with > clang, future versions of gcc etc (with the current flags I can remove the > cast and gcc 4.5 is still happy). Ah right, I have seen this one before... thanks for reminding me. - Steve ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1047/#review2192 ----------------------------------------------------------- On Feb. 14, 2012, 10:41 a.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1047/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2012, 10:41 a.m.) > > > Review request for Default. > > > Description > ------- > > MEM: Move port creation to the memory object(s) construction > > This patch moves all port creation from the getPort method to be > consistently done in the MemObject's constructor. This is possible > thanks to the Swig interface passing the length of the vector ports. > Previously there was a mix of: 1) creating the ports as members (at > object construction time) and using getPort for the name resolution, > or 2) dynamically creating the ports in the getPort call. This is now > uniform. Furthermore, objects that would not be complete without a > port have these ports as members rather than having pointers to > dynamically allocated ports. > > This patch also enables an elaboration-time enumeration of all the > ports in the system which can be used to determine the masterId. > > > Diffs > ----- > > src/arch/arm/table_walker.hh 2eaf1809c6c6 > src/arch/arm/table_walker.cc 2eaf1809c6c6 > src/arch/x86/interrupts.hh 2eaf1809c6c6 > src/arch/x86/interrupts.cc 2eaf1809c6c6 > src/cpu/testers/directedtest/RubyDirectedTester.cc 2eaf1809c6c6 > src/cpu/testers/rubytest/RubyTester.cc 2eaf1809c6c6 > src/dev/alpha/tsunami_pchip.cc 2eaf1809c6c6 > src/dev/arm/pl111.cc 2eaf1809c6c6 > src/dev/copy_engine.hh 2eaf1809c6c6 > src/dev/copy_engine.cc 2eaf1809c6c6 > src/dev/i8254xGBe.cc 2eaf1809c6c6 > src/dev/io_device.hh 2eaf1809c6c6 > src/dev/io_device.cc 2eaf1809c6c6 > src/dev/mips/malta_pchip.cc 2eaf1809c6c6 > src/dev/pcidev.hh 2eaf1809c6c6 > src/dev/pcidev.cc 2eaf1809c6c6 > src/dev/sinic.cc 2eaf1809c6c6 > src/dev/x86/i82094aa.hh 2eaf1809c6c6 > src/dev/x86/i82094aa.cc 2eaf1809c6c6 > src/dev/x86/intdev.hh 2eaf1809c6c6 > src/dev/x86/intdev.cc 2eaf1809c6c6 > src/mem/bridge.hh 2eaf1809c6c6 > src/mem/bridge.cc 2eaf1809c6c6 > src/mem/bus.hh 2eaf1809c6c6 > src/mem/bus.cc 2eaf1809c6c6 > src/mem/physical.cc 2eaf1809c6c6 > src/mem/ruby/system/RubyPort.hh 2eaf1809c6c6 > src/mem/ruby/system/RubyPort.cc 2eaf1809c6c6 > > Diff: http://reviews.gem5.org/r/1047/diff/diff > > > Testing > ------- > > util/regress all passing (disregarding t1000 and eio) > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
