Re: [gem5-dev] Review Request 2316: mem: add tCCD to DRAM controller

2014-09-17 Thread Andreas Hansson via gem5-dev
> On July 28, 2014, 12:34 a.m., Andreas Hansson wrote: > > Thanks for the input Amin. One high-level question: what is the main aim of > > the patch? Until now we have tried to keep the timing constraints to a > > minimum (without sacrificing fidelity). In general you could argue that > > tBUR

Re: [gem5-dev] Review Request 2316: mem: add tCCD to DRAM controller

2014-07-29 Thread Amin Farmahini via gem5-dev
> On July 28, 2014, 12:34 a.m., Andreas Hansson wrote: > > Thanks for the input Amin. One high-level question: what is the main aim of > > the patch? Until now we have tried to keep the timing constraints to a > > minimum (without sacrificing fidelity). In general you could argue that > > tBUR

Re: [gem5-dev] Review Request 2316: mem: add tCCD to DRAM controller

2014-07-29 Thread Andreas Hansson via gem5-dev
> On July 28, 2014, 12:34 a.m., Andreas Hansson wrote: > > Thanks for the input Amin. One high-level question: what is the main aim of > > the patch? Until now we have tried to keep the timing constraints to a > > minimum (without sacrificing fidelity). In general you could argue that > > tBUR

Re: [gem5-dev] Review Request 2316: mem: add tCCD to DRAM controller

2014-07-29 Thread Amin Farmahini via gem5-dev
> On July 28, 2014, 12:34 a.m., Andreas Hansson wrote: > > Thanks for the input Amin. One high-level question: what is the main aim of > > the patch? Until now we have tried to keep the timing constraints to a > > minimum (without sacrificing fidelity). In general you could argue that > > tBUR

Re: [gem5-dev] Review Request 2316: mem: add tCCD to DRAM controller

2014-07-27 Thread Andreas Hansson via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2316/#review5228 --- Thanks for the input Amin. One high-level question: what is the main aim

[gem5-dev] Review Request 2316: mem: add tCCD to DRAM controller

2014-07-23 Thread Amin Farmahini via gem5-dev
--- This is an automatically generated e-mail. To reply, visit: http://reviews.gem5.org/r/2316/ --- Review request for Default. Repository: gem5 Description --- This patch adds