Hello Gabe Black, Andreas Sandberg,

I'd like you to reexamine a change. Please visit

    https://gem5-review.googlesource.com/2880

to look at the new patch set (#2).

Change subject: cpu, configs: A different way to expose CPU names
......................................................................

cpu, configs: A different way to expose CPU names

Use Python magic to generate a list of CPUs instead of explicitly listing
them.

This is in contrast to
https://gem5-review.googlesource.com/c/2860
https://gem5-review.googlesource.com/c/2861
https://gem5-review.googlesource.com/c/2862

Currently just a draft proposal. Comments are welcome :)

This patch still doesn't solve one of the problems the above patches
solve: All files that define CPU models must be imported into a common
file. This patch imports the ARM CPU model into CpuConfig.py as a
stop-gap.

When adding the following code to se.py you get the following result.

from common import O3_ARM_v7a_3
for cpu,cls in m5._cpu_models.iteritems():
    print cpu, cls
...
detailed DerivO3CPU
O3_ARM_v7a_3 O3_ARM_v7a_3
AtomicSimpleCPU AtomicSimpleCPU
trace TraceCPU
O3Checker O3Checker
kvm X86KvmCPU
DummyChecker DummyChecker
DerivO3CPU DerivO3CPU
atomic AtomicSimpleCPU
TimingSimpleCPU TimingSimpleCPU
timing TimingSimpleCPU
X86KvmCPU X86KvmCPU
TraceCPU TraceCPU
...

This also supports adding aliases to classes derived in pure python.

If this patch has support, I will provide a similar patch for the DRAM
models.

Change-Id: Iae64910ce3b54a9dc05c19ed196a5b282f6ada8a
Signed-off-by: Jason Lowe-Power <[email protected]>
---
M configs/common/CpuConfig.py
M src/cpu/BaseCPU.py
M src/cpu/kvm/BaseKvmCPU.py
M src/cpu/minor/MinorCPU.py
M src/cpu/o3/O3CPU.py
M src/cpu/simple/AtomicSimpleCPU.py
M src/cpu/simple/TimingSimpleCPU.py
M src/cpu/trace/TraceCPU.py
8 files changed, 55 insertions(+), 75 deletions(-)


--
To view, visit https://gem5-review.googlesource.com/2880
To unsubscribe, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iae64910ce3b54a9dc05c19ed196a5b282f6ada8a
Gerrit-Change-Number: 2880
Gerrit-PatchSet: 2
Gerrit-Owner: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to