mem-ruby: HTMSequencer stats initialized twice
HTMSequencer stats are already initialized in the constructor This is a bug from: [1]: https://gem5-review.googlesource.com/c/public/gem5/+/36478 Change-Id: Id7d9b11f45035a46af32584ed86470c65d2a80b6 Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com> Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/51407 Reviewed-by: Hoa Nguyen <hoanguyen@ucdavis.edu> Reviewed-by: Jason Lowe-Power <power.jg@gmail.com> Maintainer: Jason Lowe-Power <power.jg@gmail.com> Tested-by: kokoro <noreply+kokoro@google.com>
This commit is contained in:
@@ -209,40 +209,6 @@ HTMSequencer::htmCallback(Addr address,
|
||||
}
|
||||
}
|
||||
|
||||
void
|
||||
HTMSequencer::regStats()
|
||||
{
|
||||
// hardware transactional memory
|
||||
m_htm_transaction_cycles
|
||||
.init(10)
|
||||
.name(name() + ".htm_transaction_cycles")
|
||||
.desc("number of cycles spent in an outer transaction")
|
||||
.flags(statistics::pdf | statistics::dist | statistics::nozero |
|
||||
statistics::nonan)
|
||||
;
|
||||
m_htm_transaction_instructions
|
||||
.init(10)
|
||||
.name(name() + ".htm_transaction_instructions")
|
||||
.desc("number of instructions spent in an outer transaction")
|
||||
.flags(statistics::pdf | statistics::dist | statistics::nozero |
|
||||
statistics::nonan)
|
||||
;
|
||||
auto num_causes = static_cast<int>(HtmFailureFaultCause::NUM_CAUSES);
|
||||
m_htm_transaction_abort_cause
|
||||
.init(num_causes)
|
||||
.name(name() + ".htm_transaction_abort_cause")
|
||||
.desc("cause of htm transaction abort")
|
||||
.flags(statistics::total | statistics::pdf | statistics::dist |
|
||||
statistics::nozero)
|
||||
;
|
||||
|
||||
for (unsigned cause_idx = 0; cause_idx < num_causes; ++cause_idx) {
|
||||
m_htm_transaction_abort_cause.subname(
|
||||
cause_idx,
|
||||
htmFailureToStr(HtmFailureFaultCause(cause_idx)));
|
||||
}
|
||||
}
|
||||
|
||||
void
|
||||
HTMSequencer::rubyHtmCallback(PacketPtr pkt,
|
||||
const HtmFailedInCacheReason htm_return_code)
|
||||
|
||||
@@ -68,7 +68,6 @@ class HTMSequencer : public Sequencer
|
||||
|
||||
bool empty() const override;
|
||||
void print(std::ostream& out) const override;
|
||||
void regStats() override;
|
||||
void wakeup() override;
|
||||
|
||||
private:
|
||||
|
||||
Reference in New Issue
Block a user