----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2462/#review5478 -----------------------------------------------------------
src/arch/arm/pagetable.hh <http://reviews.gem5.org/r/2462/#comment4926> setting AP=3 makes the page table user-read-only. one minor comment - Ali Saidi On Oct. 6, 2014, 6:58 p.m., Alexandru Dutu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2462/ > ----------------------------------------------------------- > > (Updated Oct. 6, 2014, 6:58 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10429:e0f8a026df2c > --------------------------- > mem: Page Table map api modification > > This patch adds uncacheable/cacheable and read-only/read-write attributes to > the map method of PageTableBase. It also modifies the constructor of TlbEntry > structs for all architectures to consider the new attributes. > > > Diffs > ----- > > src/arch/alpha/pagetable.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/arch/arm/pagetable.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/arch/mips/pagetable.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/arch/power/tlb.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/arch/sparc/pagetable.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/arch/x86/pagetable.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/arch/x86/pagetable.cc 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/mem/multi_level_page_table.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/mem/multi_level_page_table_impl.hh > 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/mem/page_table.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/mem/page_table.cc 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/sim/Process.py 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/sim/process.hh 148b96b7bc77190686f532d3b8f9f30cf911b36a > src/sim/process.cc 148b96b7bc77190686f532d3b8f9f30cf911b36a > > Diff: http://reviews.gem5.org/r/2462/diff/ > > > Testing > ------- > > Quick regressions testing done. > > > Thanks, > > Alexandru Dutu > > _______________________________________________ gem5-dev mailing list gem5-dev@gem5.org http://m5sim.org/mailman/listinfo/gem5-dev