Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix some bug in Processor.cpp #11

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

chenyxuan
Copy link

I tried to run ramulator-pim with an in-order core but failed.

Command:

 ./ramulator --config Configs/pim.cfg --disable-perf-scheduling true --mode=cpu --stats pim.stats --trace sample_traces/pim/pim-rodiniaBFS.out --core-org=inOrder --number-cores=4 --trace-format=zsim --split-trace=true

Result:

tick_gcd: 800
cpu_tick: 1
mem_tick: 1
Number of Cores: 4
Split Trace 
Core 0 trying to load trace sample_traces/pim/pim-rodiniaBFS.out.0
Trace opended: sample_traces/pim/pim-rodiniaBFS.out.0
Core 1 trying to load trace sample_traces/pim/pim-rodiniaBFS.out.1
Trace opended: sample_traces/pim/pim-rodiniaBFS.out.1
Core 2 trying to load trace sample_traces/pim/pim-rodiniaBFS.out.2
Trace opended: sample_traces/pim/pim-rodiniaBFS.out.2
Core 3 trying to load trace sample_traces/pim/pim-rodiniaBFS.out.3
Trace opended: sample_traces/pim/pim-rodiniaBFS.out.3
[0]retired: 0, clk, 1
[1]retired: 0, clk, 1
[2]retired: 0, clk, 1
[3]retired: 0, clk, 1

-> retired: 0
-> cycles: 1
-> ipc: 0
-> total instructions: 0
-> total time: -nanns
Simulation done. Statistics written to pim.stats

This should be due to the fact that the core didn't update more_reqs at the end of tick_inOrder() when split_trace is true.

I fixed it by copying some lines of code from tick_outOrder() and now it runs well.

@chenyxuan chenyxuan changed the title Update Processor.cpp Fix some bug in Processor.cpp Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant