> On March 21, 2012, 9:32 p.m., Ali Saidi wrote: > > Seems like a reasonable start. i have same question below and would like a > > better description of the options.
I assume the description you are looking for is for the parameters of the AbstractMemory? I can either spill over the 80 char or add the descriptions as code comments in the Python code. Preferences? > On March 21, 2012, 9:32 p.m., Ali Saidi wrote: > > src/mem/AbstractMemory.py, line 52 > > <http://reviews.gem5.org/r/1113/diff/1/?file=24960#file24960line52> > > > > when would you have a memory that wasn't? The memtesters use a "reference" memory that shadows the normal memory system. These memories are overlapping in address ranges and should not appear in the global address map. > On March 21, 2012, 9:32 p.m., Ali Saidi wrote: > > src/dev/arm/RealView.py, line 460 > > <http://reviews.gem5.org/r/1113/diff/1/?file=24959#file24959line460> > > > > this looks like it's wrong Indeed, thanks. > On March 21, 2012, 9:32 p.m., Ali Saidi wrote: > > src/dev/arm/RealView.py, line 151 > > <http://reviews.gem5.org/r/1113/diff/1/?file=24959#file24959line151> > > > > i don't think this should be conf table reported Have I got this wrong then? This is used by the arm/linux/system.cc to populate the AtagMem struct, and if this is not "conf_table_reported" (or what ever we choose to call this property of a memory) then the realview regressions fail. Are you suggesting a name change for the parameter? > On March 21, 2012, 9:32 p.m., Ali Saidi wrote: > > src/mem/abstract_mem.hh, line 74 > > <http://reviews.gem5.org/r/1113/diff/1/?file=24964#file24964line74> > > > > wrong style and below Thanks. Fixed. - Andreas ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/1113/#review2350 ----------------------------------------------------------- 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
