----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1922/#review4429 -----------------------------------------------------------
You want to remove some ugly code in a portion that is only really done once for most systems, and the "price" is a more complex object structure for the ports. My gut feeling is that these different ports have little in common. For example, their actual interfaces are very different. I also think this patch makes the port/interface separation much more difficult. - Andreas Hansson On June 13, 2013, 7:58 a.m., Steve Reinhardt wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1922/ > ----------------------------------------------------------- > > (Updated June 13, 2013, 7:58 a.m.) > > > Review request for Default. > > > Description > ------- > > Changeset 9757:a2acc94bd8d0 > --------------------------- > sim: unify memory & ethernet port binding mechanisms. > > Get rid of the ugly special-casing and dynamic casting going > on in connectPorts(), and make the port binding mechanism > more extensible. This is done by moving the port lookup > methods to SimObject and creating a common base Port class > from which both the old Port (now MemPort) and EtherInt > (which perhaps should be renamed EtherPort) derive. > > > Diffs > ----- > > src/arch/x86/bios/acpi.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/dev/etherdevice.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/dev/etherdevice.cc 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/dev/etherint.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/dev/etherint.cc 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/dev/etherlink.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/dev/etherobject.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/mem/mem_object.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/mem/mem_object.cc 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/mem/port.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/mem/port.cc 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/python/swig/pyobject.cc 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/sim/port.hh PRE-CREATION > src/sim/sim_object.hh 0b4a08751b42ac522e296bd1a12408b816068fa1 > src/sim/sim_object.cc 0b4a08751b42ac522e296bd1a12408b816068fa1 > > Diff: http://reviews.gem5.org/r/1922/diff/ > > > Testing > ------- > > > Thanks, > > Steve Reinhardt > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
