----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1113/#review2350 -----------------------------------------------------------
Seems like a reasonable start. i have same question below and would like a better description of the options. src/dev/arm/RealView.py <http://reviews.gem5.org/r/1113/#comment2779> i don't think this should be conf table reported src/dev/arm/RealView.py <http://reviews.gem5.org/r/1113/#comment2780> this looks like it's wrong src/mem/AbstractMemory.py <http://reviews.gem5.org/r/1113/#comment2781> when would you have a memory that wasn't? src/mem/abstract_mem.hh <http://reviews.gem5.org/r/1113/#comment2782> wrong style and below - Ali Saidi On March 21, 2012, 4:21 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/1113/ > ----------------------------------------------------------- > > (Updated March 21, 2012, 4:21 p.m.) > > > Review request for Default. > > > Description > ------- > > MEM: Enable multiple distributed generalized memories > > This patch removes the assumption on having on single instance of > PhysicalMemory, and enables a distributed memory where the individual > memories in the system are each responsible for a single contiguous > address range. > > All memories inherit from an AbstractMemory that encompasses the basic > behaviuor of a random access memory, and provides untimed access > methods. What was previously called PhysicalMemory is now > SimpleMemory, and a subclass of AbstractMemory. All future types of > memory controllers should inherit from AbstractMemory. > > To enable e.g. the atomic CPU and RubyPort to access the now > distributed memory, the system has a wrapper class, called > PhysicalMemory that is aware of all the memories in the system and > their associated address ranges. This class thus acts as an > infinitely-fast bus and performs address decoding for these "shortcut" > accesses. Each memory can specify that it should not be part of the > global address map (used e.g. by the functional memories by some > testers). Moreover, each memory can be configured to be reported to > the OS configuration table, useful for populating ATAG structures, and > any potential ACPI tables. > > Checkpointing support currently assumes that all memories have the > same size and organisation when creating and resuming from the > checkpoint. A future patch will enable a more flexible > re-organisation. > > > Diffs > ----- > > configs/common/FSConfig.py 4da2ea94319f > configs/example/memtest.py 4da2ea94319f > configs/example/ruby_direct_test.py 4da2ea94319f > configs/example/ruby_mem_test.py 4da2ea94319f > configs/example/ruby_network_test.py 4da2ea94319f > configs/example/ruby_random_test.py 4da2ea94319f > configs/example/se.py 4da2ea94319f > configs/ruby/MESI_CMP_directory.py 4da2ea94319f > configs/ruby/MI_example.py 4da2ea94319f > configs/ruby/MOESI_CMP_directory.py 4da2ea94319f > configs/ruby/MOESI_CMP_token.py 4da2ea94319f > configs/ruby/MOESI_hammer.py 4da2ea94319f > configs/ruby/Network_test.py 4da2ea94319f > configs/ruby/Ruby.py 4da2ea94319f > configs/splash2/cluster.py 4da2ea94319f > configs/splash2/run.py 4da2ea94319f > src/arch/alpha/remote_gdb.hh 4da2ea94319f > src/arch/alpha/remote_gdb.cc 4da2ea94319f > src/arch/arm/ArmSystem.py 4da2ea94319f > src/arch/arm/linux/system.cc 4da2ea94319f > src/arch/arm/system.cc 4da2ea94319f > src/arch/sparc/SparcSystem.py 4da2ea94319f > src/base/remote_gdb.hh 4da2ea94319f > src/base/remote_gdb.cc 4da2ea94319f > src/cpu/checker/thread_context.hh 4da2ea94319f > src/cpu/inorder/thread_context.hh 4da2ea94319f > src/cpu/o3/fetch_impl.hh 4da2ea94319f > src/cpu/simple/atomic.hh 4da2ea94319f > src/cpu/simple/atomic.cc 4da2ea94319f > src/dev/alpha/backdoor.cc 4da2ea94319f > src/dev/arm/RealView.py 4da2ea94319f > src/mem/AbstractMemory.py PRE-CREATION > src/mem/PhysicalMemory.py 4da2ea94319f > src/mem/SConscript 4da2ea94319f > src/mem/SimpleMemory.py PRE-CREATION > src/mem/abstract_mem.hh PRE-CREATION > src/mem/abstract_mem.cc PRE-CREATION > src/mem/physical.hh 4da2ea94319f > src/mem/physical.cc 4da2ea94319f > src/mem/ruby/system/RubyPort.cc 4da2ea94319f > src/mem/simple_mem.hh PRE-CREATION > src/mem/simple_mem.cc PRE-CREATION > src/sim/System.py 4da2ea94319f > src/sim/system.hh 4da2ea94319f > src/sim/system.cc 4da2ea94319f > tests/configs/inorder-timing.py 4da2ea94319f > tests/configs/memtest-ruby.py 4da2ea94319f > tests/configs/memtest.py 4da2ea94319f > tests/configs/o3-timing-checker.py 4da2ea94319f > tests/configs/o3-timing-mp.py 4da2ea94319f > tests/configs/o3-timing.py 4da2ea94319f > tests/configs/rubytest-ruby.py 4da2ea94319f > tests/configs/simple-atomic-dummychecker.py 4da2ea94319f > tests/configs/simple-atomic-mp.py 4da2ea94319f > tests/configs/simple-atomic.py 4da2ea94319f > tests/configs/simple-timing-mp-ruby.py 4da2ea94319f > tests/configs/simple-timing-mp.py 4da2ea94319f > tests/configs/simple-timing-ruby.py 4da2ea94319f > tests/configs/simple-timing.py 4da2ea94319f > > Diff: http://reviews.gem5.org/r/1113/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
