Nice catch.  This bug was introduced 10/9/08, I'm surprised it didn't come
up until now.  I am committing it now.

Thanks.
Lisa

On Tue, Sep 29, 2009 at 2:12 PM, soumyaroop roy <[email protected]> wrote:

> There was a tiny typographical bug in the code due to which
> "max_insts_all_threads" wasn't working. In fact, shouldn't this be called
> "min_insts_all_threads"?
>
> diff --git a/src/cpu/base.cc b/src/cpu/base.cc
> --- a/src/cpu/base.cc
> +++ b/src/cpu/base.cc
> @@ -151,7 +151,7 @@
>          *counter = numThreads;
>          for (ThreadID tid = 0; tid < numThreads; ++tid) {
>              Event *event = new CountedExitEvent(cause, *counter);
> -            comInstEventQueue[tid]->schedule(event,
> p->max_insts_any_thread);
> +            comInstEventQueue[tid]->schedule(event,
> p->max_insts_all_threads);
>          }
>      }
>
>
> --
> Soumyaroop Roy
> Ph.D. Candidate
> Department of Computer Science and Engineering
> University of South Florida, Tampa
> http://www.csee.usf.edu/~sroy
> _______________________________________________
> m5-dev mailing list
> [email protected]
> http://m5sim.org/mailman/listinfo/m5-dev
>
>
_______________________________________________
m5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/m5-dev

Reply via email to