----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2325/#review5288 -----------------------------------------------------------
Ship it! Not important, but I'd suggest changing the description to: "In cases where the DRAM burst size (e.g. 32 bytes in a rank with a single LPDDR3 x32) was smaller than the channel interleaving size (e.g. systems with a 64-byte cache line) ..." I'd also use the "channel interleaving/striping" term instead of just simply "interleaving/striping" if that is what you mean. - Amin Farmahini On Aug. 18, 2014, 12:27 p.m., Andreas Hansson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.gem5.org/r/2325/ > ----------------------------------------------------------- > > (Updated Aug. 18, 2014, 12:27 p.m.) > > > Review request for Default. > > > Repository: gem5 > > > Description > ------- > > Changeset 10286:08c7971d5f0a > --------------------------- > mem: Fix address interleaving bug in DRAM controller > > This patch fixes a bug in the DRAM controller address decoding. In > cases where the DRAM burst size was smaller than the interleaving > stripe size (e.g. LPDDR3 x32 with a 64 byte cache line) one address > bit effectively got used as a channel bit when it should have been a > low-order column bit. > > This patch adds a notion of "columns per stripe", and more clearly > deals with the low-order column bits and high-order column bits. The > patch also relaxes the granularity check such that it is possible to > use interleaving granularities other than the cache line size. > > The patch also adds a missing M5_CLASS_VAR_USED to the tCK member as > it is only used in the debug build for now. > > > Diffs > ----- > > src/mem/dram_ctrl.hh 79fde1c67ed8 > src/mem/dram_ctrl.cc 79fde1c67ed8 > > Diff: http://reviews.gem5.org/r/2325/diff/ > > > Testing > ------- > > > Thanks, > > Andreas Hansson > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
