cpu: Remove automatic overriding of numThreads in SE on O3.
authorGabe Black <gabeblack@google.com>
Mon, 12 Oct 2020 00:26:10 +0000 (17:26 -0700)
committerGabe Black <gabeblack@google.com>
Wed, 14 Oct 2020 07:18:27 +0000 (07:18 +0000)
commit2ae079c6be9fea7d829aad9ac539bbc1f654ec0a
tree406b05c499efe75d707096522d06fd522de8cc93
parent05e60080dc2ecf24ebcb53097a0647ed5132c92f
cpu: Remove automatic overriding of numThreads in SE on O3.

On the O3 CPU, when the number of threads on the CPU (SMT) is too low to
hold all the old style CPU workload items, then it would increase the
number of threads to match. There are three problems with this.

1. This behavior was only implemented on O3.
2. It could silently hide a bug in the config where the number of
   workload items was accidentally too big.
3. It makes the DerivO3CPUParams struct tamper with itself in the
   create() method, which means not even config.ini will accurately
   reflect the actual config of the system.

Change-Id: I0aab70d4b98093f7f14156ca437e763f031049ab
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/35937
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/cpu/o3/cpu.cc
src/cpu/o3/deriv.cc