mem: Add instruction sequence number to request

Information
Submitter:	Curtis Dunham
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

This patch adds the instruction sequence number to the request and provides a
request constructor that accepts a sequence number for initialization.

Testing Done

Issue Summary

Description	From	Last Updated	Status
Seems like the only way to set this variable is through the constructor. Why not mark as const?	Nilay Vaish	Dec. 1, 2015, 9:35 a.m.	Dropped

src/mem/request.hh (Diff revision 1)

Seems like the only way to set this variable is through the constructor. Why not mark as const?

Show all issues

Radhika Jagtap Nov. 5, 2015, 10:56 a.m. (Nov. 5, 2015, 10:56 a.m.)

In http://reviews.gem5.org/r/3090/ I added optionally tracing virtual address which meant calling another Request constructor. That also meant I needed to set the inst. sequence number using a setter. A bit unfortunate but don't think it's worth changing the Request constructor or other methods.

Ship It!

Radhika Jagtap Dec. 1, 2015, 9:49 a.m. (Dec. 1, 2015, 9:49 a.m.)

Thanks, I am making the reqInstSeqNum const in this patch (blocking out what #3090 intends to do). The patch will be refreshed soon anyways.

Diff:

Revision 2 (+40 -6)

Show changes

src/mem/request.hh

Ship It!

You have a pending review.

Review Board 2.0.15

This change has been marked as submitted.

Screenshots

Files

Issue Summary

Diff:

Status: Closed (submitted)